perf report: Print session information only if --stdio is given
authorNamhyung Kim <namhyung.kim@lge.com>
Fri, 20 Dec 2013 05:11:13 +0000 (14:11 +0900)
committerArnaldo Carvalho de Melo <acme@redhat.com>
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 <namhyung@kernel.org>
Cc: David Ahern <dsahern@gmail.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1387516278-17024-3-git-send-email-namhyung@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/builtin-report.c

index 0c9ec3e..bf8dd2e 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;