From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752563AbcDFOc2 (ORCPT ); Wed, 6 Apr 2016 10:32:28 -0400 Received: from mail.kernel.org ([198.145.29.136]:50821 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751070AbcDFOc1 (ORCPT ); Wed, 6 Apr 2016 10:32:27 -0400 Date: Wed, 6 Apr 2016 11:32:20 -0300 From: Arnaldo Carvalho de Melo To: Andi Kleen Cc: peterz@infradead.org, jolsa@kernel.org, linux-kernel@vger.kernel.org, mingo@kernel.org, Andi Kleen Subject: Re: [PATCH 11/11] perf, tools, stat: Add extra output of counter values with -v Message-ID: <20160406143220.GD5945@kernel.org> References: <1459802514-31501-1-git-send-email-andi@firstfloor.org> <1459802514-31501-12-git-send-email-andi@firstfloor.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1459802514-31501-12-git-send-email-andi@firstfloor.org> X-Url: http://acmel.wordpress.com User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Em Mon, Apr 04, 2016 at 01:41:54PM -0700, Andi Kleen escreveu: > From: Andi Kleen > > Add debug output of raw counter values per CPU when > perf stat -v is specified, together with their cpu numbers. > This is very useful to debug problems with per core counters, > where we can normally only see aggregated values. Can we do it only when per CPU is in place? [acme@jouet linux]$ perf stat -v usleep 1 task-clock: 0: 1426718 1426718 1426718 task-clock: 1426718 1426718 1426718 context-switches: 0: 3 1426718 1426718 context-switches: 3 1426718 1426718 cpu-migrations: 0: 1 1426718 1426718 cpu-migrations: 1 1426718 1426718 page-faults: 0: 52 1426718 1426718 page-faults: 52 1426718 1426718 cycles: 0: 4338775 1437628 1437628 cycles: 4338775 1437628 1437628 instructions: 0: 754838 1437628 1437628 instructions: 754838 1437628 1437628 branches: 0: 146638 1437628 1437628 branches: 146638 1437628 1437628 branch-misses: 0: 7871 1437628 1437628 branch-misses: 7871 1437628 1437628 Performance counter stats for 'usleep 1': 1.426718 task-clock (msec) # 0.370 CPUs utilized 3 context-switches # 0.002 M/sec 1 cpu-migrations # 0.701 K/sec 52 page-faults # 0.036 M/sec 4,338,775 cycles # 3.041 GHz 754,838 instructions # 0.17 insn per cycle 146,638 branches # 102.780 M/sec 7,871 branch-misses # 5.37% of all branches 0.003859416 seconds time elapsed [acme@jouet linux]$ > Signed-off-by: Andi Kleen > --- > tools/perf/builtin-stat.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/tools/perf/builtin-stat.c b/tools/perf/builtin-stat.c > index 6d8ce72cd8c1..0ee224b80056 100644 > --- a/tools/perf/builtin-stat.c > +++ b/tools/perf/builtin-stat.c > @@ -314,6 +314,14 @@ static int read_counter(struct perf_evsel *counter) > return -1; > } > } > + > + if (verbose) { > + fprintf(stat_config.output, > + "%s: %d: %" PRIu64 " %" PRIu64 " %" PRIu64 "\n", > + perf_evsel__name(counter), > + cpu, > + count->val, count->ena, count->run); > + } > } > } > > -- > 2.5.5