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

2014-07-13 Thread Andi Kleen
On Sun, Jul 13, 2014 at 10:09:45PM +0200, Jiri Olsa wrote: > On Sun, Jul 13, 2014 at 01:49:59AM +0200, Andi Kleen wrote: > > > missing the tests/aliases.o file > > > > > > AR libperf.a > > > ar: tests/aliases.o: No such file or directory > > > > Fixed here (and in git) > > got it, would

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

2014-07-13 Thread Jiri Olsa
On Sun, Jul 13, 2014 at 01:49:59AM +0200, Andi Kleen wrote: > > missing the tests/aliases.o file > > > > AR libperf.a > > ar: tests/aliases.o: No such file or directory > > Fixed here (and in git) got it, would you be ok with following change? using pr_* instead of fprintf calls..

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

2014-07-13 Thread Jiri Olsa
On Sun, Jul 13, 2014 at 01:49:59AM +0200, Andi Kleen wrote: missing the tests/aliases.o file AR libperf.a ar: tests/aliases.o: No such file or directory Fixed here (and in git) got it, would you be ok with following change? using pr_* instead of fprintf calls.. jirka ---

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

2014-07-13 Thread Andi Kleen
On Sun, Jul 13, 2014 at 10:09:45PM +0200, Jiri Olsa wrote: On Sun, Jul 13, 2014 at 01:49:59AM +0200, Andi Kleen wrote: missing the tests/aliases.o file AR libperf.a ar: tests/aliases.o: No such file or directory Fixed here (and in git) got it, would you be ok with

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

2014-07-12 Thread Andi Kleen
> missing the tests/aliases.o file > > AR libperf.a > ar: tests/aliases.o: No such file or directory Fixed here (and in git) --- 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

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

2014-07-12 Thread Jiri Olsa
On Fri, Jul 11, 2014 at 04:59:24PM -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 > > v2: Rename identifiers > v3:

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

2014-07-12 Thread Jiri Olsa
On Fri, Jul 11, 2014 at 04:59:24PM -0700, Andi Kleen wrote: From: Andi Kleen a...@linux.intel.com 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

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

2014-07-12 Thread Andi Kleen
missing the tests/aliases.o file AR libperf.a ar: tests/aliases.o: No such file or directory Fixed here (and in git) --- From: Andi Kleen a...@linux.intel.com Add a simple test case to perf test that runs perf download and parses all the available events, including json events.

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

2014-07-11 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 v3: Only iterate cpu pmu to avoid bogus errors. Move pmu iterator to extra patch

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

2014-07-11 Thread Andi Kleen
From: Andi Kleen a...@linux.intel.com 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 v3: Only iterate cpu pmu to avoid bogus errors. Move pmu iterator