[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Wed, 6 Jun 2018 16:19:02 -0700
From: Andi Kleen <ak@...ux.intel.com>
To: Jiri Olsa <jolsa@...nel.org>
Cc: Arnaldo Carvalho de Melo <acme@...nel.org>,
Peter Zijlstra <a.p.zijlstra@...llo.nl>,
Kan Liang <kan.liang@...ux.intel.com>,
Agustin Vega-Frias <agustinv@...eaurora.org>,
lkml <linux-kernel@...r.kernel.org>,
Ingo Molnar <mingo@...nel.org>,
Namhyung Kim <namhyung@...nel.org>,
David Ahern <dsahern@...il.com>,
Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
Stephane Eranian <eranian@...gle.com>,
Milian Wolff <milian.wolff@...b.com>,
Andi Kleen <andi@...stfloor.org>,
Frederic Weisbecker <frederic@...nel.org>
Subject: Re: [PATCH 01/10] perf tools: Uniquify the event name if there's no
other matched event
On Thu, Jun 07, 2018 at 12:15:04AM +0200, Jiri Olsa wrote:
> Currently by default we try to match the user specified PMU
> name to all PMU units available and use them to aggregate
> all matched PMUs event counts into one 'pattern' event.
>
> While this is useful for uncore events, it screws up names
> for other events, where this is not desirable, like:
>
> Before:
> # perf stat -e cp/cpu-cycles/ kill
I assume you mean cpU/cpu-cycles/
>
> Performance counter stats for 'kill':
>
> 1,573,757 cp/cpu-cycles/
>
> Keeping the pattern matching logic, but making the name unique
> in case there's no other match found. That fixes the example
> above and hopefully does not screw up anything else.
>
> After:
> # perf stat -e cp/cpu-cycles/ kill
>
> Performance counter stats for 'kill':
>
> 1,573,757 cpu/cpu-cycles/
The output is 100% identical?
-Andi
Powered by blists - more mailing lists