Re: [PATCH v4 3/4] perf stat: Support 'percore' event qualifier

[Arnaldo Carvalho de Melo]

Em Mon, Apr 15, 2019 at 08:09:09PM +0530, Ravi Bangoria escreveu:
> 
> 
> On 4/12/19 7:29 PM, Jin Yao wrote:
> > diff --git a/tools/perf/Documentation/perf-stat.txt b/tools/perf/Documentation/perf-stat.txt
> > index 39c05f8..1e312c2 100644
> > --- a/tools/perf/Documentation/perf-stat.txt
> > +++ b/tools/perf/Documentation/perf-stat.txt
> > @@ -43,6 +43,10 @@ report::
> >  	  param1 and param2 are defined as formats for the PMU in
> >  	  /sys/bus/event_source/devices/<pmu>/format/*
> >  
> > +	  'percore' is a event qualifier that sums up the event counts for both
> > +	  hardware threads in a core.
> 
> s/both/all/  :
> 
>   $ lscpu | grep Thread
>   Thread(s) per core:    4
> 
> 
> Apart from that, for the series:
> Tested-by: Ravi Bangoria <ravi.bangoria@xxxxxxxxxxxxx>

Yeah I had fixed up that already, fixed up the patch description this
time around, "both" was being used there as well, changed to "all".

Thanks, added your Tested-by tag, appreciated.

- Arnaldo