Skip to content

Commit

Permalink
perf script: Use the default lost event handler
Browse files Browse the repository at this point in the history
That already does what was being done here. The warning is now unconditionally
given by __perf_session__process_pipe_events, just like for non pipe processing.

Cc: Frederic Weisbecker <[email protected]>
Cc: Ingo Molnar <[email protected]>
Cc: Mike Galbraith <[email protected]>
Cc: Paul Mackerras <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: Stephane Eranian <[email protected]>
Cc: Tom Zanussi <[email protected]>
LKML-Reference: <new-submission>
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
  • Loading branch information
acmel committed Jan 5, 2011
1 parent 1109599 commit 6d8afb5
Showing 1 changed file with 1 addition and 14 deletions.
15 changes: 1 addition & 14 deletions tools/perf/builtin-script.c
Original file line number Diff line number Diff line change
Expand Up @@ -99,24 +99,13 @@ static int process_sample_event(event_t *event, struct sample_data *sample,
return 0;
}

static u64 nr_lost;

static int process_lost_event(event_t *event, struct sample_data *sample __used,
struct perf_session *session __used)
{
nr_lost += event->lost.lost;

return 0;
}

static struct perf_event_ops event_ops = {
.sample = process_sample_event,
.comm = event__process_comm,
.attr = event__process_attr,
.event_type = event__process_event_type,
.tracing_data = event__process_tracing_data,
.build_id = event__process_build_id,
.lost = process_lost_event,
.ordering_requires_timestamps = true,
.ordered_samples = true,
};
Expand All @@ -136,10 +125,8 @@ static int __cmd_script(struct perf_session *session)

ret = perf_session__process_events(session, &event_ops);

if (debug_mode) {
if (debug_mode)
pr_err("Misordered timestamps: %llu\n", nr_unordered);
pr_err("Lost events: %llu\n", nr_lost);
}

return ret;
}
Expand Down

0 comments on commit 6d8afb5

Please sign in to comment.