hi, following up on the 'perf timechart' FIXME note and changing its tracepoint match not to use event types data.
In my tests the old and new timechart outputs look the same, so.. any other tester would be appreciated ;-) Also as this was the only user of the event types data the rest of the patchset is removing it out of the perf. The event types data are referenced from the perf data file header. The reference (offset & size) stays in the header with 0s. Also the event types data is already duplicated via the event_desc FEATURE, so there's no information loss. thanks for comments, jirka Signed-off-by: Jiri Olsa <[email protected]> Cc: Arnaldo Carvalho de Melo <[email protected]> Cc: Peter Zijlstra <[email protected]> Cc: Ingo Molnar <[email protected]> Cc: Paul Mackerras <[email protected]> Cc: Corey Ashford <[email protected]> Cc: Frederic Weisbecker <[email protected]> Cc: Namhyung Kim <[email protected]> Cc: David Ahern <[email protected]> Cc: Thomas Renninger <[email protected]> --- tools/perf/builtin-inject.c | 7 ------ tools/perf/builtin-record.c | 13 ----------- tools/perf/builtin-report.c | 1 - tools/perf/builtin-script.c | 1 - tools/perf/builtin-timechart.c | 94 ++++++++++++++++++++++++++++++++++++++++++------------------------------------- tools/perf/util/event.h | 2 +- tools/perf/util/header.c | 118 +--------------------------------------------------------------------------------------------------- tools/perf/util/header.h | 16 +------------- tools/perf/util/session.c | 11 ---------- tools/perf/util/tool.h | 3 --- 10 files changed, 54 insertions(+), 212 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
