Skip to content

Commit

Permalink
perf session: Move dump code to event delivery path
Browse files Browse the repository at this point in the history
Preparatory patch for ordered perf report -D

Acked-by: Ian Munsie <imunsie@au1.ibm.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ian Munsie <imunsie@au1.ibm.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Peter Zijlstra <peterz@infradead.org>
LKML-Reference: <20101207124550.918655066@linutronix.de>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
  • Loading branch information
Thomas Gleixner authored and Arnaldo Carvalho de Melo committed Dec 9, 2010
1 parent f74725d commit 532e726
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions tools/perf/util/session.c
Original file line number Diff line number Diff line change
Expand Up @@ -702,10 +702,13 @@ static int perf_session_deliver_event(struct perf_session *session,
event_t *event,
struct sample_data *sample,
struct perf_event_ops *ops,
u64 file_offset __used)
u64 file_offset)
{
dump_event(session, event, file_offset, sample);

switch (event->header.type) {
case PERF_RECORD_SAMPLE:
dump_sample(session, event, sample);
return ops->sample(event, sample, session);
case PERF_RECORD_MMAP:
return ops->mmap(event, sample, session);
Expand Down Expand Up @@ -747,10 +750,8 @@ static int perf_session__process_event(struct perf_session *session,

if (event->header.type >= PERF_RECORD_USER_TYPE_START)
dump_event(session, event, file_offset, NULL);
else {
else
event__parse_sample(event, session, &sample);
dump_event(session, event, file_offset, &sample);
}

/* These events are processed right away */
switch (event->header.type) {
Expand All @@ -765,7 +766,6 @@ static int perf_session__process_event(struct perf_session *session,
return 0;
}
}
dump_sample(session, event, &sample);
break;

case PERF_RECORD_HEADER_ATTR:
Expand Down

0 comments on commit 532e726

Please sign in to comment.