Re: [PATCHv4 0/8] perf, tool: Allow to use hw events in PMU syntax

2012-10-23 Thread Peter Zijlstra
On Wed, 2012-10-10 at 14:53 +0200, Jiri Olsa wrote: > arch/x86/kernel/cpu/perf_event.c | 121 > + > arch/x86/kernel/cpu/perf_event.h | 2 ++ > arch/x86/kernel/cpu/perf_event_amd.c | 9 +++

Re: [PATCHv4 0/8] perf, tool: Allow to use hw events in PMU syntax

2012-10-23 Thread Peter Zijlstra
On Wed, 2012-10-10 at 14:53 +0200, Jiri Olsa wrote: arch/x86/kernel/cpu/perf_event.c | 121 + arch/x86/kernel/cpu/perf_event.h | 2 ++ arch/x86/kernel/cpu/perf_event_amd.c | 9 +++

Re: [PATCHv4 0/8] perf, tool: Allow to use hw events in PMU syntax

2012-10-10 Thread Stephane Eranian
On Wed, Oct 10, 2012 at 3:40 PM, Jiri Olsa wrote: > On Wed, Oct 10, 2012 at 03:34:53PM +0200, Stephane Eranian wrote: >> On Wed, Oct 10, 2012 at 2:53 PM, Jiri Olsa wrote: >> > hi, >> > here's the change to make following syntax available: >> > perf stat -e cpu/event=instructions/u ls >> >

Re: [PATCHv4 0/8] perf, tool: Allow to use hw events in PMU syntax

2012-10-10 Thread Jiri Olsa
On Wed, Oct 10, 2012 at 03:34:53PM +0200, Stephane Eranian wrote: > On Wed, Oct 10, 2012 at 2:53 PM, Jiri Olsa wrote: > > hi, > > here's the change to make following syntax available: > > perf stat -e cpu/event=instructions/u ls > > perf stat -e cpu/cycles/u ls > > > Unless, I am mistaken but

Re: [PATCHv4 0/8] perf, tool: Allow to use hw events in PMU syntax

2012-10-10 Thread Stephane Eranian
On Wed, Oct 10, 2012 at 2:53 PM, Jiri Olsa wrote: > hi, > here's the change to make following syntax available: > perf stat -e cpu/event=instructions/u ls > perf stat -e cpu/cycles/u ls > Unless, I am mistaken but I did not see in your patch where you adjust the list of generic Hw events (and

[PATCHv4 0/8] perf, tool: Allow to use hw events in PMU syntax

2012-10-10 Thread Jiri Olsa
hi, here's the change to make following syntax available: perf stat -e cpu/event=instructions/u ls perf stat -e cpu/cycles/u ls this is identical to: perf stat -e instructions:u ls perf stat -e cycles:u ls v4 changes: - kernel change is now structured differently allowing to pick

[PATCHv4 0/8] perf, tool: Allow to use hw events in PMU syntax

2012-10-10 Thread Jiri Olsa
hi, here's the change to make following syntax available: perf stat -e cpu/event=instructions/u ls perf stat -e cpu/cycles/u ls this is identical to: perf stat -e instructions:u ls perf stat -e cycles:u ls v4 changes: - kernel change is now structured differently allowing to pick

Re: [PATCHv4 0/8] perf, tool: Allow to use hw events in PMU syntax

2012-10-10 Thread Stephane Eranian
On Wed, Oct 10, 2012 at 2:53 PM, Jiri Olsa jo...@redhat.com wrote: hi, here's the change to make following syntax available: perf stat -e cpu/event=instructions/u ls perf stat -e cpu/cycles/u ls Unless, I am mistaken but I did not see in your patch where you adjust the list of generic Hw

Re: [PATCHv4 0/8] perf, tool: Allow to use hw events in PMU syntax

2012-10-10 Thread Jiri Olsa
On Wed, Oct 10, 2012 at 03:34:53PM +0200, Stephane Eranian wrote: On Wed, Oct 10, 2012 at 2:53 PM, Jiri Olsa jo...@redhat.com wrote: hi, here's the change to make following syntax available: perf stat -e cpu/event=instructions/u ls perf stat -e cpu/cycles/u ls Unless, I am mistaken

Re: [PATCHv4 0/8] perf, tool: Allow to use hw events in PMU syntax

2012-10-10 Thread Stephane Eranian
On Wed, Oct 10, 2012 at 3:40 PM, Jiri Olsa jo...@redhat.com wrote: On Wed, Oct 10, 2012 at 03:34:53PM +0200, Stephane Eranian wrote: On Wed, Oct 10, 2012 at 2:53 PM, Jiri Olsa jo...@redhat.com wrote: hi, here's the change to make following syntax available: perf stat -e