@@ -58,6 +58,18 @@
enum {
#include "power7-events-list.h"
+ /*
+ * Normally, generic events like 'cycles' are aliases for a real
+ * event PM_CYC. So we want both the event and the alias listed in
+ * sysfs. But mem-loads and mem-stores are just aliases - they are
+ * not listed in power7_generic_events[] for instance. Adding them
+ * to power7-events-list.h will unnecessariliy create PM_MEM_LOADS
+ * and PM_MEM_STORES events in sysfs when that file is processed
+ * again below. To save a couple of sysfs entries, define these
+ * separately.
+ */
+ EVENT(PM_MEM_LOADS, 0x40030)
+ EVENT(PM_MEM_STORES, 0x40030)
};
#undef EVENT
@@ -382,6 +394,8 @@ GENERIC_EVENT_ATTR(cache-references, PM_LD_REF_L1);
GENERIC_EVENT_ATTR(cache-misses, PM_LD_MISS_L1);
GENERIC_EVENT_ATTR(branch-instructions, PM_BRU_FIN);
GENERIC_EVENT_ATTR(branch-misses, PM_BR_MPRED);
+GENERIC_EVENT_ATTR(mem-loads, PM_MEM_LOADS);
+GENERIC_EVENT_ATTR(mem-stores, PM_MEM_STORES);
#define EVENT(_name, _code) POWER_EVENT_ATTR(_name, _name);
#include "power7-events-list.h"
@@ -398,6 +412,8 @@ static struct attribute *power7_events_attr[] = {
GENERIC_EVENT_PTR(PM_LD_MISS_L1),
GENERIC_EVENT_PTR(PM_BRU_FIN),
GENERIC_EVENT_PTR(PM_BR_MPRED),
+ GENERIC_EVENT_PTR(PM_MEM_LOADS),
+ GENERIC_EVENT_PTR(PM_MEM_STORES),
#include "power7-events-list.h"
#undef EVENT
[PATCH 4/7] powerpc/perf: Create mem-loads/mem-stores events for Power7 'perf mem' command depends on the support for generic hardware events 'mem-loads' and 'mem-stores'. Create those events for Power7 and map them both to the event PM_MRK_GRP_CMPL. While PM_MRK_GRP_CMPL is strictly not restricted to loads and stores, that seems to be a close/resonable match. Cc: Stephane Eranian <eranian@google.com> Cc: Paul Mckerras <paulus@samba.org> Cc: Michael Ellerman <michael@ellerman.id.au> Signed-off-by: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com> --- arch/powerpc/perf/power7-pmu.c | 16 ++++++++++++++++ 1 files changed, 16 insertions(+), 0 deletions(-)