Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-07 Thread Jiri Olsa
On Mon, May 07, 2018 at 04:24:08PM -0300, Arnaldo Carvalho de Melo wrote: > Em Mon, May 07, 2018 at 03:37:49PM -0300, Arnaldo Carvalho de Melo escreveu: > > Em Sat, May 05, 2018 at 08:43:11PM -0700, Andi Kleen escreveu: > > > Jiri Olsa writes: > > > > > > Please fix this

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-07 Thread Jiri Olsa
On Mon, May 07, 2018 at 04:24:08PM -0300, Arnaldo Carvalho de Melo wrote: > Em Mon, May 07, 2018 at 03:37:49PM -0300, Arnaldo Carvalho de Melo escreveu: > > Em Sat, May 05, 2018 at 08:43:11PM -0700, Andi Kleen escreveu: > > > Jiri Olsa writes: > > > > > > Please fix this quickly, PT is currently

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-07 Thread Arnaldo Carvalho de Melo
Em Mon, May 07, 2018 at 09:16:53PM +0200, Jiri Olsa escreveu: > On Mon, May 07, 2018 at 03:37:49PM -0300, Arnaldo Carvalho de Melo wrote: > > Em Sat, May 05, 2018 at 08:43:11PM -0700, Andi Kleen escreveu: > > > Jiri Olsa writes: > > > > > > Please fix this quickly, PT is

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-07 Thread Arnaldo Carvalho de Melo
Em Mon, May 07, 2018 at 09:16:53PM +0200, Jiri Olsa escreveu: > On Mon, May 07, 2018 at 03:37:49PM -0300, Arnaldo Carvalho de Melo wrote: > > Em Sat, May 05, 2018 at 08:43:11PM -0700, Andi Kleen escreveu: > > > Jiri Olsa writes: > > > > > > Please fix this quickly, PT is currently totally non

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-07 Thread Arnaldo Carvalho de Melo
Em Mon, May 07, 2018 at 03:37:49PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Sat, May 05, 2018 at 08:43:11PM -0700, Andi Kleen escreveu: > > Jiri Olsa writes: > > > > Please fix this quickly, PT is currently totally non functional in Linus > > mainline. > > Ok, so I'm

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-07 Thread Arnaldo Carvalho de Melo
Em Mon, May 07, 2018 at 03:37:49PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Sat, May 05, 2018 at 08:43:11PM -0700, Andi Kleen escreveu: > > Jiri Olsa writes: > > > > Please fix this quickly, PT is currently totally non functional in Linus > > mainline. > > Ok, so I'm reverting this patch,

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-07 Thread Jiri Olsa
On Mon, May 07, 2018 at 03:37:49PM -0300, Arnaldo Carvalho de Melo wrote: > Em Sat, May 05, 2018 at 08:43:11PM -0700, Andi Kleen escreveu: > > Jiri Olsa writes: > > > > Please fix this quickly, PT is currently totally non functional in Linus > > mainline. > > Ok, so I'm

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-07 Thread Jiri Olsa
On Mon, May 07, 2018 at 03:37:49PM -0300, Arnaldo Carvalho de Melo wrote: > Em Sat, May 05, 2018 at 08:43:11PM -0700, Andi Kleen escreveu: > > Jiri Olsa writes: > > > > Please fix this quickly, PT is currently totally non functional in Linus > > mainline. > > Ok, so I'm reverting this patch,

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-07 Thread Arnaldo Carvalho de Melo
Em Sat, May 05, 2018 at 08:43:11PM -0700, Andi Kleen escreveu: > Jiri Olsa writes: > > Please fix this quickly, PT is currently totally non functional in Linus > mainline. Ok, so I'm reverting this patch, the previous situation was just a misleading error message, so it can

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-07 Thread Arnaldo Carvalho de Melo
Em Sat, May 05, 2018 at 08:43:11PM -0700, Andi Kleen escreveu: > Jiri Olsa writes: > > Please fix this quickly, PT is currently totally non functional in Linus > mainline. Ok, so I'm reverting this patch, the previous situation was just a misleading error message, so it can wait for the

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-05 Thread Andi Kleen
Jiri Olsa writes: Please fix this quickly, PT is currently totally non functional in Linus mainline. -Andi

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-05 Thread Andi Kleen
Jiri Olsa writes: Please fix this quickly, PT is currently totally non functional in Linus mainline. -Andi

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-04 Thread Jiri Olsa
On Thu, May 03, 2018 at 02:38:15PM +0300, Adrian Hunter wrote: SNIP > >>> index 7afeb80cc39e..d14464c42714 100644 > >>> --- a/tools/perf/util/parse-events.y > >>> +++ b/tools/perf/util/parse-events.y > >>> @@ -224,15 +224,15 @@ event_def: event_pmu | > >>> event_bpf_file > >>> > >>>

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-04 Thread Jiri Olsa
On Thu, May 03, 2018 at 02:38:15PM +0300, Adrian Hunter wrote: SNIP > >>> index 7afeb80cc39e..d14464c42714 100644 > >>> --- a/tools/perf/util/parse-events.y > >>> +++ b/tools/perf/util/parse-events.y > >>> @@ -224,15 +224,15 @@ event_def: event_pmu | > >>> event_bpf_file > >>> > >>>

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-03 Thread Jiri Olsa
On Thu, May 03, 2018 at 02:38:15PM +0300, Adrian Hunter wrote: > On 03/05/18 13:37, Jiri Olsa wrote: > > On Thu, May 03, 2018 at 11:25:16AM +0300, Adrian Hunter wrote: > >> Hi > >> > >> This breaks Intel PT i.e. > >> > >> $ perf record -e intel_pt//u uname > >> event syntax error: 'intel_pt//u' >

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-03 Thread Jiri Olsa
On Thu, May 03, 2018 at 02:38:15PM +0300, Adrian Hunter wrote: > On 03/05/18 13:37, Jiri Olsa wrote: > > On Thu, May 03, 2018 at 11:25:16AM +0300, Adrian Hunter wrote: > >> Hi > >> > >> This breaks Intel PT i.e. > >> > >> $ perf record -e intel_pt//u uname > >> event syntax error: 'intel_pt//u' >

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-03 Thread Adrian Hunter
On 03/05/18 13:37, Jiri Olsa wrote: > On Thu, May 03, 2018 at 11:25:16AM +0300, Adrian Hunter wrote: >> Hi >> >> This breaks Intel PT i.e. >> >> $ perf record -e intel_pt//u uname >> event syntax error: 'intel_pt//u' >> \___ parser error >> Run 'perf list' for a list

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-03 Thread Adrian Hunter
On 03/05/18 13:37, Jiri Olsa wrote: > On Thu, May 03, 2018 at 11:25:16AM +0300, Adrian Hunter wrote: >> Hi >> >> This breaks Intel PT i.e. >> >> $ perf record -e intel_pt//u uname >> event syntax error: 'intel_pt//u' >> \___ parser error >> Run 'perf list' for a list

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-03 Thread Jiri Olsa
On Thu, May 03, 2018 at 11:25:16AM +0300, Adrian Hunter wrote: > Hi > > This breaks Intel PT i.e. > > $ perf record -e intel_pt//u uname > event syntax error: 'intel_pt//u' > \___ parser error > Run 'perf list' for a list of valid events > > Usage: perf record []

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-03 Thread Jiri Olsa
On Thu, May 03, 2018 at 11:25:16AM +0300, Adrian Hunter wrote: > Hi > > This breaks Intel PT i.e. > > $ perf record -e intel_pt//u uname > event syntax error: 'intel_pt//u' > \___ parser error > Run 'perf list' for a list of valid events > > Usage: perf record []

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-03 Thread Adrian Hunter
Hi This breaks Intel PT i.e. $ perf record -e intel_pt//u uname event syntax error: 'intel_pt//u' \___ parser error Run 'perf list' for a list of valid events Usage: perf record [] [] or: perf record [] -- [] -e, --eventevent selector. use 'perf list'

Re: [PATCH 05/12] perf pmu: Fix pmu events parsing rule

2018-05-03 Thread Adrian Hunter
Hi This breaks Intel PT i.e. $ perf record -e intel_pt//u uname event syntax error: 'intel_pt//u' \___ parser error Run 'perf list' for a list of valid events Usage: perf record [] [] or: perf record [] -- [] -e, --eventevent selector. use 'perf list'