perf stat: Move STAT_RECORD out of perf_evlist__print_counters()
authorJiri Olsa <jolsa@kernel.org>
Thu, 30 Aug 2018 06:32:25 +0000 (08:32 +0200)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Thu, 30 Aug 2018 18:52:22 +0000 (15:52 -0300)
It's stat related and should stay in the 'perf stat' command.  The
perf_evlist__print_counters function will be moved out in the following
patches.

Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Andi Kleen <andi@firstfloor.org>
Cc: David Ahern <dsahern@gmail.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/20180830063252.23729-17-jolsa@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/builtin-stat.c

index db11832bbdbcb117b9905854c6b11b6189b19ccf..4ffbb6594eb648e8c75d63dde83c4d605f82754e 100644 (file)
@@ -1701,10 +1701,6 @@ perf_evlist__print_counters(struct perf_evlist *evlist,
        struct perf_evsel *counter;
        char buf[64], *prefix = NULL;
 
-       /* Do not print anything if we record to the pipe. */
-       if (STAT_RECORD && perf_stat.data.is_pipe)
-               return;
-
        if (interval)
                print_interval(prefix = buf, ts);
        else
@@ -1766,6 +1762,10 @@ perf_evlist__print_counters(struct perf_evlist *evlist,
 
 static void print_counters(struct timespec *ts, int argc, const char **argv)
 {
+       /* Do not print anything if we record to the pipe. */
+       if (STAT_RECORD && perf_stat.data.is_pipe)
+               return;
+
        perf_evlist__print_counters(evsel_list, ts, argc, argv);
 }