Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751077AbaKXUfz (ORCPT ); Mon, 24 Nov 2014 15:35:55 -0500 Received: from mx1.redhat.com ([209.132.183.28]:34336 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750819AbaKXUfx (ORCPT ); Mon, 24 Nov 2014 15:35:53 -0500 Date: Mon, 24 Nov 2014 18:35:33 -0200 From: Arnaldo Carvalho de Melo To: Jiri Olsa Cc: linux-kernel@vger.kernel.org, Andi Kleen , Corey Ashford , David Ahern , Frederic Weisbecker , Ingo Molnar , Matt Fleming , Namhyung Kim , Paul Mackerras , Peter Zijlstra , Stephane Eranian Subject: Re: [PATCH 04/11] perf stat: Use perf_evsel__read_cb in read_counter Message-ID: <20141124203533.GB12236@redhat.com> References: <1416562275-12404-1-git-send-email-jolsa@kernel.org> <1416562275-12404-5-git-send-email-jolsa@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1416562275-12404-5-git-send-email-jolsa@kernel.org> X-Url: http://acmel.wordpress.com User-Agent: Mutt/1.5.20 (2009-12-10) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Em Fri, Nov 21, 2014 at 10:31:08AM +0100, Jiri Olsa escreveu: > Replacing __perf_evsel__read_on_cpu function with perf_evsel__read_cb > function. The read_cb callback will be used later for global aggregation > counter values as well. But the changeset makes it look like there are no changes made... Can you elaborate here? > Cc: Andi Kleen > Cc: Arnaldo Carvalho de Melo > Cc: Corey Ashford > Cc: David Ahern > Cc: Frederic Weisbecker > Cc: Ingo Molnar > Cc: Matt Fleming > Cc: Namhyung Kim > Cc: Paul Mackerras > Cc: Peter Zijlstra > Cc: Stephane Eranian > Signed-off-by: Jiri Olsa > --- > tools/perf/builtin-stat.c | 27 +++++++++++++++++++++------ > 1 file changed, 21 insertions(+), 6 deletions(-) > > diff --git a/tools/perf/builtin-stat.c b/tools/perf/builtin-stat.c > index 055ce9232c9e..9cc0db1d7f06 100644 > --- a/tools/perf/builtin-stat.c > +++ b/tools/perf/builtin-stat.c > @@ -388,6 +388,26 @@ static void update_shadow_stats(struct perf_evsel *counter, u64 *count) > update_stats(&runtime_itlb_cache_stats[0], count[0]); > } > > +static int read_cb(struct perf_evsel *evsel, int cpu, int thread __maybe_unused, > + struct perf_counts_values *count) > +{ > + switch (aggr_mode) { > + case AGGR_CORE: > + case AGGR_SOCKET: > + case AGGR_NONE: > + perf_evsel__compute_deltas(evsel, cpu, count); > + perf_counts_values__scale(count, scale, NULL); > + evsel->counts->cpu[cpu] = *count; > + update_shadow_stats(evsel, count->values); > + break; > + case AGGR_GLOBAL: > + default: > + break; > + } > + > + return 0; > +} > + > /* > * Read out the results of a single counter: > * aggregate counts across CPUs in system-wide mode > @@ -424,16 +444,11 @@ static int read_counter_aggr(struct perf_evsel *counter) > */ > static int read_counter(struct perf_evsel *counter) > { > - u64 *count; > int cpu; > > for (cpu = 0; cpu < perf_evsel__nr_cpus(counter); cpu++) { > - if (__perf_evsel__read_on_cpu(counter, cpu, 0, scale) < 0) > + if (perf_evsel__read_cb(counter, cpu, 0, read_cb)) > return -1; > - > - count = counter->counts->cpu[cpu].values; > - > - update_shadow_stats(counter, count); > } > > return 0; > -- > 1.9.3 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/