Messages in this thread | | | Date | Mon, 9 Oct 2017 11:51:28 -0300 | From | Arnaldo Carvalho de Melo <> | Subject | Re: [PATCH 2/2] perf, tools: Don't force MetricExprs to lower case |
| |
Em Mon, Oct 09, 2017 at 04:39:53PM +0200, Jiri Olsa escreveu: > On Mon, Oct 09, 2017 at 11:12:58AM -0300, Arnaldo Carvalho de Melo wrote: > > Em Mon, Oct 09, 2017 at 07:07:29AM -0700, Andi Kleen escreveu: > > > On Mon, Oct 09, 2017 at 03:41:51PM +0200, Jiri Olsa wrote: > > > > On Wed, Oct 04, 2017 at 09:27:11AM -0700, Andi Kleen wrote: > > > > > I tried similar patches, but I always ran into more complex > > > > > situations where it still matched incorrectly. > > > > > > > > > > e.g. try it with cpu/uops_executed.core,... vs uops_executed.core > > > > > > > > hm, both works for me with the change: > > > > > > > > perf stat -e cpu/uops_executed.core/ ls > > > > perf stat -e uops_executed.core ls > > > > > > Ok. If it works it's fine for me. > > well it works, but it means that bpf file cannot contains any directory > part.. which im not sure is ok with bpf folks ;-) anyone?
That is a big limitation :-\
- Arnaldo
| |