perf session: Move perf report specific hits out of perf_session__fprintf_hists
authorArnaldo Carvalho de Melo <acme@redhat.com>
Wed, 16 Dec 2009 14:27:10 +0000 (12:27 -0200)
committerIngo Molnar <mingo@elte.hu>
Wed, 16 Dec 2009 15:51:50 +0000 (16:51 +0100)
Those don't make sense for tools such as 'perf diff'.

Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Frédéric Weisbecker <fweisbec@gmail.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Paul Mackerras <paulus@samba.org>
LKML-Reference: <1260973631-28035-2-git-send-email-acme@infradead.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
tools/perf/builtin-report.c
tools/perf/util/hist.c

index 24d20e7d125a8e2c4af0fee4f8ef005e66326633..c4f3a08f7c4831e3062c88f8f93c350eb47de220 100644 (file)
@@ -224,7 +224,12 @@ static int __cmd_report(void)
 
        perf_session__collapse_resort(session);
        perf_session__output_resort(session, session->events_stats.total);
+       fprintf(stdout, "# Samples: %ld\n#\n", session->events_stats.total);
        perf_session__fprintf_hists(session, stdout);
+       if (sort_order == default_sort_order &&
+           parent_pattern == default_parent_pattern)
+               fprintf(stdout, "#\n# (For a higher level overview, try: perf report --sort comm,dso)\n#\n");
+
        if (show_threads) {
                bool raw_printing_style = !strcmp(pretty_printing_style, "raw");
                perf_read_values_display(stdout, &show_threads_values,
index 270eb8f3dceeec2870d4fff00b01c4effbb8f3a0..6e416a62e0d603cd28fd604cde95304a8151a9d3 100644 (file)
@@ -515,9 +515,6 @@ size_t perf_session__fprintf_hists(struct perf_session *self, FILE *fp)
 
        init_rem_hits();
 
-       fprintf(fp, "# Samples: %ld\n", self->events_stats.total);
-       fprintf(fp, "#\n");
-
        fprintf(fp, "# Overhead");
        if (symbol_conf.show_nr_samples) {
                if (symbol_conf.field_sep)
@@ -578,14 +575,6 @@ print_entries:
                ret += hist_entry__fprintf(fp, pos, self);
        }
 
-       if (sort_order == default_sort_order &&
-                       parent_pattern == default_parent_pattern) {
-               fprintf(fp, "#\n");
-               fprintf(fp, "# (For a higher level overview, try: perf report --sort comm,dso)\n");
-               fprintf(fp, "#\n");
-       }
-       fprintf(fp, "\n");
-
        free(rem_sq_bracket);
 
        return ret;