Re: [PATCH v3 06/11] perf, tools, stat: Support JSON metrics in perf stat

[Arnaldo Carvalho de Melo]

Em Tue, Sep 05, 2017 at 11:19:52AM -0700, Andi Kleen escreveu:
> On Tue, Sep 05, 2017 at 03:09:19PM -0300, Arnaldo Carvalho de Melo wrote:
> > Em Mon, Sep 04, 2017 at 10:37:25AM -0700, Andi Kleen escreveu:
> > > But you can get it here

> > > git://git.kernel.org/pub/scm/linux/kernel/git/ak/linux-misc.git perf/intel-json-metrics-2

> > Ok, so I tried installing the Broadwell ones (one of my test machines)
> > and got into:
 
> I would suggest to apply all the patches from the patchkit before you test.
> Some of the later ones add/fix stuff needed by some of the metrics.
> 
> > 
> > [root@jouet ~]# perf stat -M Pipeline flops
> > bpf: builtin compilation failed: -95, try external compiler
> > ERROR: problems with path cpu/uops_executed.c: No such file or directory
> > Cannot set up events {uops_retired.retire_slots,inst_retired.any}:W,{inst_retired.any,cycles}:W,{uops_executed.thread,cpu/uops_executed.core,cmask=1/,uops_executed.cycles_ge_1_uop_exec}:W
> > 
> >  Usage: perf stat [<options>] [<command>]
> > 
> >     -M, --metrics <metric/metric group list>
> >                           monitor specified metrics or metric groups (separated by ,)
> > [root@jouet ~]# perf stat -M Pipeline flops
> > 
> > 
> > Wasn't this already fixed?
> 
> Yes that was supposed to be fixed with 
 
>     perf bpf: Tighten detection of BPF events
>     
>       perf stat -e cpu/uops_executed.core,cmask=1/
 
> I'll check.

Ok, I couldn't reproduce it anymore, after you check this, please let me
know if I should pull that perf/intel-json-metrics-2 branch so that
someone wanting to test this can have it all in one place, ok?

- Arnaldo