[PATCH 09/10] perf, tools: Support DividedBy header in JSON event list

2016-11-18 Thread Andi Kleen
From: Andi Kleen Add support for parsing the DividedBy header in the JSON event lists and storing them in the alias structure. Used in the next patch. Signed-off-by: Andi Kleen --- tools/perf/pmu-events/jevents.c| 18 ++

[PATCH 09/10] perf, tools: Support DividedBy header in JSON event list

2016-11-18 Thread Andi Kleen
From: Andi Kleen Add support for parsing the DividedBy header in the JSON event lists and storing them in the alias structure. Used in the next patch. Signed-off-by: Andi Kleen --- tools/perf/pmu-events/jevents.c| 18 ++ tools/perf/pmu-events/jevents.h| 2 +-

Re: [PATCH 09/10] perf, tools: Support DividedBy header in JSON event list

2016-10-17 Thread Andi Kleen
> > so.. > > - you put 'DividedBy' into JSON event's defition any further > explanation how or why the format we use for event defs will > be used now used to describe ratios > > - then you force perf stat to merge together all 'same' uncore events > to get just one number.. The ratios

Re: [PATCH 09/10] perf, tools: Support DividedBy header in JSON event list

2016-10-17 Thread Andi Kleen
> > so.. > > - you put 'DividedBy' into JSON event's defition any further > explanation how or why the format we use for event defs will > be used now used to describe ratios > > - then you force perf stat to merge together all 'same' uncore events > to get just one number.. The ratios

Re: [PATCH 09/10] perf, tools: Support DividedBy header in JSON event list

2016-10-17 Thread Jiri Olsa
On Mon, Oct 17, 2016 at 07:43:12PM +0200, Jiri Olsa wrote: > On Mon, Oct 17, 2016 at 09:27:54AM -0700, Andi Kleen wrote: > > On Mon, Oct 17, 2016 at 01:44:43PM +0200, Jiri Olsa wrote: > > > On Thu, Oct 13, 2016 at 02:15:31PM -0700, Andi Kleen wrote: > > > > From: Andi Kleen

Re: [PATCH 09/10] perf, tools: Support DividedBy header in JSON event list

2016-10-17 Thread Jiri Olsa
On Mon, Oct 17, 2016 at 07:43:12PM +0200, Jiri Olsa wrote: > On Mon, Oct 17, 2016 at 09:27:54AM -0700, Andi Kleen wrote: > > On Mon, Oct 17, 2016 at 01:44:43PM +0200, Jiri Olsa wrote: > > > On Thu, Oct 13, 2016 at 02:15:31PM -0700, Andi Kleen wrote: > > > > From: Andi Kleen > > > > > > > > Add

Re: [PATCH 09/10] perf, tools: Support DividedBy header in JSON event list

2016-10-17 Thread Jiri Olsa
On Mon, Oct 17, 2016 at 09:27:54AM -0700, Andi Kleen wrote: > On Mon, Oct 17, 2016 at 01:44:43PM +0200, Jiri Olsa wrote: > > On Thu, Oct 13, 2016 at 02:15:31PM -0700, Andi Kleen wrote: > > > From: Andi Kleen > > > > > > Add support for parsing the DividedBy header in the

Re: [PATCH 09/10] perf, tools: Support DividedBy header in JSON event list

2016-10-17 Thread Jiri Olsa
On Mon, Oct 17, 2016 at 09:27:54AM -0700, Andi Kleen wrote: > On Mon, Oct 17, 2016 at 01:44:43PM +0200, Jiri Olsa wrote: > > On Thu, Oct 13, 2016 at 02:15:31PM -0700, Andi Kleen wrote: > > > From: Andi Kleen > > > > > > Add support for parsing the DividedBy header in the JSON event lists and > >

Re: [PATCH 09/10] perf, tools: Support DividedBy header in JSON event list

2016-10-17 Thread Andi Kleen
On Mon, Oct 17, 2016 at 01:44:43PM +0200, Jiri Olsa wrote: > On Thu, Oct 13, 2016 at 02:15:31PM -0700, Andi Kleen wrote: > > From: Andi Kleen > > > > Add support for parsing the DividedBy header in the JSON event lists and > > storing them in the alias structure. > > I

Re: [PATCH 09/10] perf, tools: Support DividedBy header in JSON event list

2016-10-17 Thread Andi Kleen
On Mon, Oct 17, 2016 at 01:44:43PM +0200, Jiri Olsa wrote: > On Thu, Oct 13, 2016 at 02:15:31PM -0700, Andi Kleen wrote: > > From: Andi Kleen > > > > Add support for parsing the DividedBy header in the JSON event lists and > > storing them in the alias structure. > > I wish you'd add JSON tags

Re: [PATCH 09/10] perf, tools: Support DividedBy header in JSON event list

2016-10-17 Thread Jiri Olsa
On Thu, Oct 13, 2016 at 02:15:31PM -0700, Andi Kleen wrote: > From: Andi Kleen > > Add support for parsing the DividedBy header in the JSON event lists and > storing them in the alias structure. I wish you'd add JSON tags always one by one as you did in here ;-) however

Re: [PATCH 09/10] perf, tools: Support DividedBy header in JSON event list

2016-10-17 Thread Jiri Olsa
On Thu, Oct 13, 2016 at 02:15:31PM -0700, Andi Kleen wrote: > From: Andi Kleen > > Add support for parsing the DividedBy header in the JSON event lists and > storing them in the alias structure. I wish you'd add JSON tags always one by one as you did in here ;-) however Ithink we'll need more

[PATCH 09/10] perf, tools: Support DividedBy header in JSON event list

2016-10-13 Thread Andi Kleen
From: Andi Kleen Add support for parsing the DividedBy header in the JSON event lists and storing them in the alias structure. Used in the next patch. Signed-off-by: Andi Kleen --- tools/perf/pmu-events/jevents.c| 18 ++

[PATCH 09/10] perf, tools: Support DividedBy header in JSON event list

2016-10-13 Thread Andi Kleen
From: Andi Kleen Add support for parsing the DividedBy header in the JSON event lists and storing them in the alias structure. Used in the next patch. Signed-off-by: Andi Kleen --- tools/perf/pmu-events/jevents.c| 18 ++ tools/perf/pmu-events/jevents.h| 2 +-