perf stat: Move perf_stat initialization counter process code
authorJiri Olsa <jolsa@kernel.org>
Fri, 26 Jun 2015 09:29:22 +0000 (11:29 +0200)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Fri, 26 Jun 2015 14:50:19 +0000 (11:50 -0300)
Moving perf_stat initialization counter process code,
to make the reading path free of processing logic.

Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Stephane Eranian <eranian@google.com>
Link: http://lkml.kernel.org/r/1435310967-14570-18-git-send-email-jolsa@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/builtin-stat.c

index b7c184bd559ccc22cb1f65772199bf86b96daf8e..526f67753664bd53f4421d6c61864be367e47f67 100644 (file)
@@ -284,6 +284,7 @@ static int process_counter(struct perf_evsel *counter)
        int i, ret;
 
        aggr->val = aggr->ena = aggr->run = 0;
+       memset(ps->res_stats, 0, sizeof(ps->res_stats));
 
        if (counter->per_pkg)
                zero_per_pkg(counter);
@@ -347,12 +348,8 @@ static int read_counter(struct perf_evsel *counter)
 static void read_counters(bool close)
 {
        struct perf_evsel *counter;
-       struct perf_stat *ps;
 
        evlist__for_each(evsel_list, counter) {
-               ps = counter->priv;
-               memset(ps->res_stats, 0, sizeof(ps->res_stats));
-
                if (read_counter(counter))
                        pr_warning("failed to read counter %s\n", counter->name);