Re: [PATCH 3/4] perf/POWER7: Make event translations available in sysfs

2012-11-19 Thread Jiri Olsa
On Fri, Nov 16, 2012 at 11:35:37AM -0800, Sukadev Bhattiprolu wrote: > Jiri Olsa [jo...@redhat.com] wrote: > | > > | > Can we remove the assumption that the event id is a generic event that > | > has PERF_COUNT_HW_ prefix and also let the architectures pass in a "show" > | > function ? This would

Re: [PATCH 3/4] perf/POWER7: Make event translations available in sysfs

2012-11-19 Thread Jiri Olsa
On Fri, Nov 16, 2012 at 11:35:37AM -0800, Sukadev Bhattiprolu wrote: Jiri Olsa [jo...@redhat.com] wrote: | | Can we remove the assumption that the event id is a generic event that | has PERF_COUNT_HW_ prefix and also let the architectures pass in a show | function ? This would allow

Re: [PATCH 3/4] perf/POWER7: Make event translations available in sysfs

2012-11-16 Thread Sukadev Bhattiprolu
Jiri Olsa [jo...@redhat.com] wrote: | > | > Can we remove the assumption that the event id is a generic event that | > has PERF_COUNT_HW_ prefix and also let the architectures pass in a "show" | > function ? This would allow architectures to display any arch specific | > events that don't yet

Re: [PATCH 3/4] perf/POWER7: Make event translations available in sysfs

2012-11-16 Thread Jiri Olsa
On Wed, Nov 14, 2012 at 10:20:45AM -0800, Sukadev Bhattiprolu wrote: > Jiri Olsa [jo...@redhat.com] wrote: > | On Wed, Nov 07, 2012 at 11:19:28AM -0800, Sukadev Bhattiprolu wrote: > | > | SNIP > | > | > +struct perf_pmu_events_attr { > | > + struct device_attribute attr; > | > + u64 id; > | >

Re: [PATCH 3/4] perf/POWER7: Make event translations available in sysfs

2012-11-16 Thread Jiri Olsa
On Wed, Nov 14, 2012 at 10:20:45AM -0800, Sukadev Bhattiprolu wrote: Jiri Olsa [jo...@redhat.com] wrote: | On Wed, Nov 07, 2012 at 11:19:28AM -0800, Sukadev Bhattiprolu wrote: | | SNIP | | +struct perf_pmu_events_attr { | + struct device_attribute attr; | + u64 id; | +}; | + |

Re: [PATCH 3/4] perf/POWER7: Make event translations available in sysfs

2012-11-16 Thread Sukadev Bhattiprolu
Jiri Olsa [jo...@redhat.com] wrote: | | Can we remove the assumption that the event id is a generic event that | has PERF_COUNT_HW_ prefix and also let the architectures pass in a show | function ? This would allow architectures to display any arch specific | events that don't yet have a

Re: [PATCH 3/4] perf/POWER7: Make event translations available in sysfs

2012-11-14 Thread Sukadev Bhattiprolu
Jiri Olsa [jo...@redhat.com] wrote: | On Wed, Nov 07, 2012 at 11:19:28AM -0800, Sukadev Bhattiprolu wrote: | | SNIP | | > +struct perf_pmu_events_attr { | > + struct device_attribute attr; | > + u64 id; | > +}; | > + | > +extern ssize_t power_events_sysfs_show(struct device *dev, | > +

Re: [PATCH 3/4] perf/POWER7: Make event translations available in sysfs

2012-11-14 Thread Jiri Olsa
On Wed, Nov 07, 2012 at 11:19:28AM -0800, Sukadev Bhattiprolu wrote: SNIP > +struct perf_pmu_events_attr { > + struct device_attribute attr; > + u64 id; > +}; > + > +extern ssize_t power_events_sysfs_show(struct device *dev, > + struct device_attribute *attr,

Re: [PATCH 3/4] perf/POWER7: Make event translations available in sysfs

2012-11-14 Thread Jiri Olsa
On Wed, Nov 07, 2012 at 11:19:28AM -0800, Sukadev Bhattiprolu wrote: SNIP +struct perf_pmu_events_attr { + struct device_attribute attr; + u64 id; +}; + +extern ssize_t power_events_sysfs_show(struct device *dev, + struct device_attribute *attr, char

Re: [PATCH 3/4] perf/POWER7: Make event translations available in sysfs

2012-11-14 Thread Sukadev Bhattiprolu
Jiri Olsa [jo...@redhat.com] wrote: | On Wed, Nov 07, 2012 at 11:19:28AM -0800, Sukadev Bhattiprolu wrote: | | SNIP | | +struct perf_pmu_events_attr { | + struct device_attribute attr; | + u64 id; | +}; | + | +extern ssize_t power_events_sysfs_show(struct device *dev, | +

[PATCH 3/4] perf/POWER7: Make event translations available in sysfs

2012-11-07 Thread Sukadev Bhattiprolu
>From d05d1ce6d55bf339eee6230ded9f5dd1351f60e5 Mon Sep 17 00:00:00 2001 From: Sukadev Bhattiprolu Date: Tue, 6 Nov 2012 14:07:36 -0800 Subject: [PATCH 3/4] perf/POWER7: Make event translations available in sysfs Make the perf events supported by POWER7 available via sysfs. $ ls /

[PATCH 3/4] perf/POWER7: Make event translations available in sysfs

2012-11-07 Thread Sukadev Bhattiprolu
From d05d1ce6d55bf339eee6230ded9f5dd1351f60e5 Mon Sep 17 00:00:00 2001 From: Sukadev Bhattiprolu suka...@linux.vnet.ibm.com Date: Tue, 6 Nov 2012 14:07:36 -0800 Subject: [PATCH 3/4] perf/POWER7: Make event translations available in sysfs Make the perf events supported by POWER7 available via