Re: [PATCH v2 05/19] perf, tools: Support weak groups

2017-08-23 Thread Jiri Olsa
On Tue, Aug 22, 2017 at 01:58:12PM -0700, Andi Kleen wrote: > On Tue, Aug 22, 2017 at 10:34:15AM +0200, Jiri Olsa wrote: > > On Fri, Aug 11, 2017 at 04:26:20PM -0700, Andi Kleen wrote: > > > > SNIP > > > > > diff --git a/tools/perf/Documentation/perf-list.txt > > > b/tools/perf/Documentation/per

Re: [PATCH v2 05/19] perf, tools: Support weak groups

2017-08-22 Thread Andi Kleen
> also if I put 'cycles' instead of the l2_rqsts.all_code_rd, > I get clean open but 'not counted' as result.. I wonder > there's some counter scheduling issue > > [root@krava perf]# ./perf stat -v -e > '{branches,branch-misses,l1d.replacement,l2_lines_in.all,cycles}:W' -a sleep 1 > Using CPUID G

Re: [PATCH v2 05/19] perf, tools: Support weak groups

2017-08-22 Thread Andi Kleen
On Tue, Aug 22, 2017 at 10:34:15AM +0200, Jiri Olsa wrote: > On Fri, Aug 11, 2017 at 04:26:20PM -0700, Andi Kleen wrote: > > SNIP > > > diff --git a/tools/perf/Documentation/perf-list.txt > > b/tools/perf/Documentation/perf-list.txt > > index f709de54707b..d432965d728d 100644 > > --- a/tools/per

Re: [PATCH v2 05/19] perf, tools: Support weak groups

2017-08-22 Thread Jiri Olsa
On Fri, Aug 11, 2017 at 04:26:20PM -0700, Andi Kleen wrote: > From: Andi Kleen > > Setting up groups can be complicated due to the > complicated scheduling restrictions of different PMUs. > User tools usually don't understand all these restrictions. > Still in many cases it is useful to set up gr

Re: [PATCH v2 05/19] perf, tools: Support weak groups

2017-08-22 Thread Jiri Olsa
On Fri, Aug 11, 2017 at 04:26:20PM -0700, Andi Kleen wrote: SNIP > diff --git a/tools/perf/Documentation/perf-list.txt > b/tools/perf/Documentation/perf-list.txt > index f709de54707b..d432965d728d 100644 > --- a/tools/perf/Documentation/perf-list.txt > +++ b/tools/perf/Documentation/perf-list.tx