diff options
author | Eric B Munson <ebmunson@us.ibm.com> | 2010-03-05 10:51:08 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2010-03-10 07:53:49 -0500 |
commit | eefc465cdd49cb89a742083fac2807c718ddad31 (patch) | |
tree | 9b16f03050d831ab357b2d0e40f694369135e787 /tools/perf/builtin-report.c | |
parent | cb8f09393646c5058056db771583c86e0ed1d92f (diff) |
perf session: Change perf_session post processing functions to take histogram tree
Now that report can store historgrams for multiple events we
need to be able to do the post processing work for each
histogram. This patch changes the post processing functions so
that they can be called individually for each event's histogram.
Signed-off-by: Eric B Munson <ebmunson@us.ibm.com>
[ Guarantee bisectabilty by fixing up builtin-report.c ]
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Paul Mackerras <paulus@samba.org>
LKML-Reference: <1267804269-22660-5-git-send-email-acme@infradead.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'tools/perf/builtin-report.c')
-rw-r--r-- | tools/perf/builtin-report.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c index cd16e6a7d6d0..294b4cf105f2 100644 --- a/tools/perf/builtin-report.c +++ b/tools/perf/builtin-report.c | |||
@@ -225,10 +225,12 @@ static int __cmd_report(void) | |||
225 | if (verbose > 2) | 225 | if (verbose > 2) |
226 | dsos__fprintf(stdout); | 226 | dsos__fprintf(stdout); |
227 | 227 | ||
228 | perf_session__collapse_resort(session); | 228 | perf_session__collapse_resort(&session->hists); |
229 | perf_session__output_resort(session, session->events_stats.total); | 229 | perf_session__output_resort(&session->hists, |
230 | session->events_stats.total); | ||
230 | fprintf(stdout, "# Samples: %Ld\n#\n", session->events_stats.total); | 231 | fprintf(stdout, "# Samples: %Ld\n#\n", session->events_stats.total); |
231 | perf_session__fprintf_hists(session, NULL, false, stdout); | 232 | perf_session__fprintf_hists(&session->hists, NULL, false, stdout, |
233 | session->events_stats.total); | ||
232 | if (sort_order == default_sort_order && | 234 | if (sort_order == default_sort_order && |
233 | parent_pattern == default_parent_pattern) | 235 | parent_pattern == default_parent_pattern) |
234 | fprintf(stdout, "#\n# (For a higher level overview, try: perf report --sort comm,dso)\n#\n"); | 236 | fprintf(stdout, "#\n# (For a higher level overview, try: perf report --sort comm,dso)\n#\n"); |