diff options
author | Arnaldo Carvalho de Melo <[email protected]> | 2011-01-11 12:42:00 -0200 |
---|---|---|
committer | Arnaldo Carvalho de Melo <[email protected]> | 2011-01-11 12:51:03 -0200 |
commit | bd3bfe9eda94d3c050830217c1e1c338808de5b2 (patch) | |
tree | d2be87a7e681192994c2434592dac685f5178cc9 /tools/perf/util/trace-event-scripting.c | |
parent | 3d03e2ea74103a50c23d6ab1906cf73399c0dafb (diff) |
perf evsel: Fix order of event list deletion
We need to defer calling perf_evsel_list__delete() till after atexit
registered routines, because we need to traverse the events being
recorded at that time at least on 'perf record'.
This fixes the problem reported by Thomas Renninger where cmd_record
called by cmd_timechart would not write the tracing data to the perf.data
file header because the evsel_list at atexit (control+C on 'perf timechart
record') time would be empty, being already deleted by run_builtin(),
and thus 'perf timechart' when trying to process such perf.data file would
die with:
"no trace data in the file"
Problem introduced in 70d544d.
Reported-by: Thomas Renninger <[email protected]>
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: Thomas Renninger <[email protected]>
Cc: Tom Zanussi <[email protected]>
LKML-Reference: <new-submission>
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
Diffstat (limited to 'tools/perf/util/trace-event-scripting.c')
0 files changed, 0 insertions, 0 deletions