perf report: Print session information only if --stdio is given
authorNamhyung Kim <[email protected]>
Fri, 20 Dec 2013 05:11:13 +0000 (14:11 +0900)
committerArnaldo Carvalho de Melo <[email protected]>
Fri, 20 Dec 2013 16:36:41 +0000 (13:36 -0300)
Move those print functions under "if (use_browser == 0)" so that they
don't interfere with TUI output.

Maybe they can handle other UIs later.

Signed-off-by: Namhyung Kim <[email protected]>
Cc: David Ahern <[email protected]>
Cc: Ingo Molnar <[email protected]>
Cc: Jiri Olsa <[email protected]>
Cc: Paul Mackerras <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
tools/perf/builtin-report.c

index 0c9ec3e3f0fce25842987f1b0716d798daa1b8c7..bf8dd2e893e4d21b52a40e38f9b1e58956f30cec 100644 (file)
@@ -469,15 +469,17 @@ static int __cmd_report(struct report *rep)
                desc);
        }
 
-       if (verbose > 3)
-               perf_session__fprintf(session, stdout);
+       if (use_browser == 0) {
+               if (verbose > 3)
+                       perf_session__fprintf(session, stdout);
 
-       if (verbose > 2)
-               perf_session__fprintf_dsos(session, stdout);
+               if (verbose > 2)
+                       perf_session__fprintf_dsos(session, stdout);
 
-       if (dump_trace) {
-               perf_session__fprintf_nr_events(session, stdout);
-               return 0;
+               if (dump_trace) {
+                       perf_session__fprintf_nr_events(session, stdout);
+                       return 0;
+               }
        }
 
        nr_samples = 0;