Re: [PATCH 22/25] perf: Make EVENT_ATTR global

2013-02-01 Thread Ingo Molnar

* Arnaldo Carvalho de Melo a...@infradead.org wrote:

 From: Sukadev Bhattiprolu suka...@linux.vnet.ibm.com
 
 Rename EVENT_ATTR() to PMU_EVENT_ATTR() and make it global so it is
 available to all architectures.
 
 Further to allow architectures flexibility, have PMU_EVENT_ATTR() pass
 in the variable name as a parameter.
 
 Changelog[v2]
   - [Jiri Olsa] No need to define PMU_EVENT_PTR()
 
 Signed-off-by: Sukadev Bhattiprolu suka...@linux.vnet.ibm.com
 Acked-by: Jiri Olsa jo...@redhat.com
 Cc: Andi Kleen a...@linux.intel.com
 Cc: Anton Blanchard an...@au1.ibm.com
 Cc: Ingo Molnar mi...@redhat.com
 Cc: Jiri Olsa jo...@redhat.com
 Cc: Paul Mackerras pau...@samba.org
 Cc: Peter Zijlstra a.p.zijls...@chello.nl
 Cc: Robert Richter robert.rich...@amd.com
 Cc: Stephane Eranian eran...@google.com
 Cc: linuxppc-...@ozlabs.org
 Link: http://lkml.kernel.org/r/20130123062422.gc13...@us.ibm.com
 Signed-off-by: Arnaldo Carvalho de Melo a...@redhat.com
 ---
  arch/x86/kernel/cpu/perf_event.c | 13 +++--
  include/linux/perf_event.h   | 11 +++
  2 files changed, 14 insertions(+), 10 deletions(-)

so this one started conflicting non-trivially with tip:perf/x86 
- the pending memory profiling kernel-side bits.

Can we merge the memory profiling tooling side bits together 
with the kernel side bits - or does it need more work?

For now I've excluded perf/x86 from tip:master until this is 
resolved.

Thanks,

Ingo
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


[PATCH 22/25] perf: Make EVENT_ATTR global

2013-01-31 Thread Arnaldo Carvalho de Melo
From: Sukadev Bhattiprolu suka...@linux.vnet.ibm.com

Rename EVENT_ATTR() to PMU_EVENT_ATTR() and make it global so it is
available to all architectures.

Further to allow architectures flexibility, have PMU_EVENT_ATTR() pass
in the variable name as a parameter.

Changelog[v2]
- [Jiri Olsa] No need to define PMU_EVENT_PTR()

Signed-off-by: Sukadev Bhattiprolu suka...@linux.vnet.ibm.com
Acked-by: Jiri Olsa jo...@redhat.com
Cc: Andi Kleen a...@linux.intel.com
Cc: Anton Blanchard an...@au1.ibm.com
Cc: Ingo Molnar mi...@redhat.com
Cc: Jiri Olsa jo...@redhat.com
Cc: Paul Mackerras pau...@samba.org
Cc: Peter Zijlstra a.p.zijls...@chello.nl
Cc: Robert Richter robert.rich...@amd.com
Cc: Stephane Eranian eran...@google.com
Cc: linuxppc-...@ozlabs.org
Link: http://lkml.kernel.org/r/20130123062422.gc13...@us.ibm.com
Signed-off-by: Arnaldo Carvalho de Melo a...@redhat.com
---
 arch/x86/kernel/cpu/perf_event.c | 13 +++--
 include/linux/perf_event.h   | 11 +++
 2 files changed, 14 insertions(+), 10 deletions(-)

diff --git a/arch/x86/kernel/cpu/perf_event.c b/arch/x86/kernel/cpu/perf_event.c
index 6774c17..c0df5ed2 100644
--- a/arch/x86/kernel/cpu/perf_event.c
+++ b/arch/x86/kernel/cpu/perf_event.c
@@ -1310,11 +1310,6 @@ static struct attribute_group x86_pmu_format_group = {
.attrs = NULL,
 };
 
-struct perf_pmu_events_attr {
-   struct device_attribute attr;
-   u64 id;
-};
-
 /*
  * Remove all undefined events (x86_pmu.event_map(id) == 0)
  * out of events_attr attributes.
@@ -1348,11 +1343,9 @@ static ssize_t events_sysfs_show(struct device *dev, 
struct device_attribute *at
 #define EVENT_VAR(_id)  event_attr_##_id
 #define EVENT_PTR(_id) event_attr_##_id.attr.attr
 
-#define EVENT_ATTR(_name, _id) \
-static struct perf_pmu_events_attr EVENT_VAR(_id) = {  \
-   .attr = __ATTR(_name, 0444, events_sysfs_show, NULL),   \
-   .id   =  PERF_COUNT_HW_##_id,   \
-};
+#define EVENT_ATTR(_name, _id) \
+   PMU_EVENT_ATTR(_name, EVENT_VAR(_id), PERF_COUNT_HW_##_id,  \
+   events_sysfs_show)
 
 EVENT_ATTR(cpu-cycles, CPU_CYCLES  );
 EVENT_ATTR(instructions,   INSTRUCTIONS);
diff --git a/include/linux/perf_event.h b/include/linux/perf_event.h
index 6bfb2faa..42adf01 100644
--- a/include/linux/perf_event.h
+++ b/include/linux/perf_event.h
@@ -817,6 +817,17 @@ do {   
\
 } while (0)
 
 
+struct perf_pmu_events_attr {
+   struct device_attribute attr;
+   u64 id;
+};
+
+#define PMU_EVENT_ATTR(_name, _var, _id, _show)
\
+static struct perf_pmu_events_attr _var = {\
+   .attr = __ATTR(_name, 0444, _show, NULL),   \
+   .id   =  _id,   \
+};
+
 #define PMU_FORMAT_ATTR(_name, _format)
\
 static ssize_t \
 _name##_show(struct device *dev,   \
-- 
1.8.1.1.361.gec3ae6e

___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev