Re: [PATCH 8/9] perf, tools, test: Add test case for alias and JSON parsing v2

2014-07-09 Thread Jiri Olsa
On Fri, Jun 27, 2014 at 04:16:03PM -0700, Andi Kleen wrote: > From: Andi Kleen > > Add a simple test case to perf test that runs perf download and parses > all the available events, including json events. > > This needs adding an all event iterator to pmu.c > SNIP > + > +int pmu_iterate_event

Re: [PATCH 8/9] perf, tools, test: Add test case for alias and JSON parsing v2

2014-07-09 Thread Jiri Olsa
On Fri, Jun 27, 2014 at 04:16:03PM -0700, Andi Kleen wrote: > From: Andi Kleen > > Add a simple test case to perf test that runs perf download and parses > all the available events, including json events. > > This needs adding an all event iterator to pmu.c I still get this error display 33: T

[PATCH 8/9] perf, tools, test: Add test case for alias and JSON parsing v2

2014-06-27 Thread Andi Kleen
From: Andi Kleen Add a simple test case to perf test that runs perf download and parses all the available events, including json events. This needs adding an all event iterator to pmu.c v2: Rename identifiers Acked-by: Namhyung Kim Signed-off-by: Andi Kleen --- tools/perf/Makefile.perf

[PATCH 8/9] perf, tools, test: Add test case for alias and JSON parsing v2

2014-06-13 Thread Andi Kleen
From: Andi Kleen Add a simple test case to perf test that runs perf download and parses all the available events, including json events. This needs adding an all event iterator to pmu.c v2: Rename identifiers Acked-by: Namhyung Kim Signed-off-by: Andi Kleen --- tools/perf/Makefile.perf

[PATCH 8/9] perf, tools, test: Add test case for alias and JSON parsing v2

2014-05-30 Thread Andi Kleen
From: Andi Kleen Add a simple test case to perf test that runs perf download and parses all the events. This needs adding an all event iterator to pmu.c v2: Rename identifiers Signed-off-by: Andi Kleen --- tools/perf/Makefile.perf| 1 + tools/perf/tests/aliases.c | 58 +++

[PATCH 8/9] perf, tools, test: Add test case for alias and JSON parsing v2

2014-05-15 Thread Andi Kleen
From: Andi Kleen Add a simple test case to perf test that runs perf download and parses all the events. This needs adding an all event iterator to pmu.c v2: Rename identifiers Signed-off-by: Andi Kleen --- tools/perf/Makefile.perf| 1 + tools/perf/tests/aliases.c | 58 +++

Re: [PATCH 8/9] perf, tools, test: Add test case for alias and JSON parsing

2014-05-13 Thread Namhyung Kim
On Mon, 12 May 2014 15:51:13 -0700, Andi Kleen wrote: [SNIP] > +int test__aliases(void) Just a nitpick. Please use a more descriptive name for the testcase like test__event_aliases(). > +{ > + int err; > + > + /* Download JSON files */ > + /* XXX assumes perf is installed */ > +

[PATCH 8/9] perf, tools, test: Add test case for alias and JSON parsing

2014-05-12 Thread Andi Kleen
From: Andi Kleen Add a simple test case to perf test that runs perf download and parses all the events. This needs adding an all event iterator to pmu.c Signed-off-by: Andi Kleen --- tools/perf/Makefile.perf| 1 + tools/perf/tests/aliases.c | 52 ++