diff options
| author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2010-05-14 10:36:42 -0300 |
|---|---|---|
| committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2010-05-14 10:36:42 -0300 |
| commit | c8446b9bdabcb0caa61bb341bd73c58f7104b503 (patch) | |
| tree | b2530bfbbd893017e9515bd48f2ffa8845860759 /tools/perf/builtin-report.c | |
| parent | 5d2be7cb198a0a6bc6088d3806fb7261b184ad89 (diff) | |
| download | lwn-c8446b9bdabcb0caa61bb341bd73c58f7104b503.tar.gz lwn-c8446b9bdabcb0caa61bb341bd73c58f7104b503.zip | |
perf hist: Make event__totals per hists
This is one more thing that started global but are more useful per hist
or per session.
Cc: Frédéric Weisbecker <fweisbec@gmail.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Tom Zanussi <tzanussi@gmail.com>
LKML-Reference: <new-submission>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/builtin-report.c')
| -rw-r--r-- | tools/perf/builtin-report.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c index 04de3387de3f..f13cda1ef059 100644 --- a/tools/perf/builtin-report.c +++ b/tools/perf/builtin-report.c @@ -139,6 +139,12 @@ static int add_event_total(struct perf_session *session, return -ENOMEM; hists->stats.total += data->period; + /* + * FIXME: add_event_total should be moved from here to + * perf_session__process_event so that the proper hist is passed to + * the event_op methods. + */ + hists__inc_nr_events(hists, PERF_RECORD_SAMPLE); session->hists.stats.total += data->period; return 0; } @@ -293,7 +299,7 @@ static int __cmd_report(void) goto out_delete; if (dump_trace) { - event__print_totals(); + perf_session__fprintf_nr_events(session, stdout); goto out_delete; } |