Re: [PATCH v5 1/3] perf jevents: Add support for parsing perchip/percore events

2020-08-18 Thread kajoljain
On 8/16/20 10:05 PM, Andi Kleen wrote: >> @@ -321,7 +331,7 @@ static void print_events_table_prefix(FILE *fp, const >> char *tblname) >> static int print_events_table_entry(void *data, char *name, char *event, >> char *desc, char *long_desc, >>

Re: [PATCH v5 1/3] perf jevents: Add support for parsing perchip/percore events

2020-08-16 Thread Andi Kleen
> @@ -321,7 +331,7 @@ static void print_events_table_prefix(FILE *fp, const > char *tblname) > static int print_events_table_entry(void *data, char *name, char *event, > char *desc, char *long_desc, > char *pmu, char *unit, char

[PATCH v5 1/3] perf jevents: Add support for parsing perchip/percore events

2020-08-16 Thread Kajol Jain
Initially, every time we want to add new terms like chip, core thread etc, we need to create corrsponding fields in pmu_events and event struct. This patch adds an enum called 'aggr_mode_class' which store all these aggregation like perchip/percore. It also adds new field 'aggr_mode' to capture