lkml.org 
[lkml]   [2017]   [Sep]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/1] perf stat: Only auto-merge events that are PMU aliases
On Thu, Aug 31, 2017 at 03:41:22PM -0300, Arnaldo Carvalho de Melo wrote:
> Hi Andi, Jiri,
>
> Please check the patch below, would be glad to have your acks,
>
> - Arnaldo
>
> Peter reported that when he explicitely asked for multiple events with
> the same name on the command line it got coalesced into just one line,
> i.e.:
>
> # perf stat -e cycles -e cycles -e cycles usleep 1
>
> Performance counter stats for 'usleep 1':
>
> 3,269,652 cycles
>
> 0.000884123 seconds time elapsed
>
> #
>
> And while there is the --no-merges option to disable that auto-merging, this is
> an blunt change in behaviour for such explicit request, so change the code so
> that this auto merging is done only when handling the multi PMU aliases with
> the same name that introduced this coalescing, restoring the previous behaviour
> for explicit case:
>
> # perf stat -e cycles -e cycles -e cycles usleep 1
>
> Performance counter stats for 'usleep 1':
>
> 1,472,837 cycles
> 1,472,837 cycles
> 1,472,837 cycles
>
> 0.001764870 seconds time elapsed
>
> #
>
> Reported-by: Peter Zijlstra <peterz@infradead.org>
> Cc: Adrian Hunter <adrian.hunter@intel.com>
> Cc: Andi Kleen <ak@linux.intel.com>
> Cc: Jiri Olsa <jolsa@kernel.org>
> Cc: Mark Rutland <mark.rutland@arm.com>
> Cc: Namhyung Kim <namhyung@kernel.org>
> Cc: Wang Nan <wangnan0@huawei.com>
> Fixes: 430daf2dc7af ("perf stat: Collapse identically named events")
> Link: http://lkml.kernel.org/n/tip-phjedddz2oe8udy9co3alo7d@git.kernel.org
> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

Thanks Arnaldo!

\
 
 \ /
  Last update: 2017-09-01 13:21    [W:0.052 / U:2.480 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site