* [PATCH 1/2] powerpc/perf: Remove PME_ prefix for power7 events @ 2016-01-11 22:55 Sukadev Bhattiprolu 2016-01-11 22:55 ` [PATCH 2/2] powerpc/perf: Export Power8 generic and cache events to sysfs Sukadev Bhattiprolu 2016-03-11 0:30 ` [1/2] powerpc/perf: Remove PME_ prefix for power7 events Michael Ellerman 0 siblings, 2 replies; 5+ messages in thread From: Sukadev Bhattiprolu @ 2016-01-11 22:55 UTC (permalink / raw) To: Michael Ellerman, Arnaldo Carvalho de Melo, Jiri Olsa Cc: linuxppc-dev, linux-kernel We used the PME_ prefix earlier to avoid some macro/variable name collisions. We have since changed the way we define/use the event macros so we no longer need the prefix. By dropping the prefix, we keep the the event macros consistent with their official names. Reported-by: Michael Ellerman <ellerman@au1.ibm.com> Signed-off-by: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com> --- arch/powerpc/include/asm/perf_event_server.h | 2 +- arch/powerpc/perf/power7-pmu.c | 18 +++++++++--------- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/arch/powerpc/include/asm/perf_event_server.h b/arch/powerpc/include/asm/perf_event_server.h index 8146221..0691087 100644 --- a/arch/powerpc/include/asm/perf_event_server.h +++ b/arch/powerpc/include/asm/perf_event_server.h @@ -141,7 +141,7 @@ extern ssize_t power_events_sysfs_show(struct device *dev, #define EVENT_PTR(_id, _suffix) &EVENT_VAR(_id, _suffix).attr.attr #define EVENT_ATTR(_name, _id, _suffix) \ - PMU_EVENT_ATTR(_name, EVENT_VAR(_id, _suffix), PME_##_id, \ + PMU_EVENT_ATTR(_name, EVENT_VAR(_id, _suffix), _id, \ power_events_sysfs_show) #define GENERIC_EVENT_ATTR(_name, _id) EVENT_ATTR(_name, _id, _g) diff --git a/arch/powerpc/perf/power7-pmu.c b/arch/powerpc/perf/power7-pmu.c index 5b62f238..a383c23 100644 --- a/arch/powerpc/perf/power7-pmu.c +++ b/arch/powerpc/perf/power7-pmu.c @@ -54,7 +54,7 @@ * Power7 event codes. */ #define EVENT(_name, _code) \ - PME_##_name = _code, + _name = _code, enum { #include "power7-events-list.h" @@ -318,14 +318,14 @@ static void power7_disable_pmc(unsigned int pmc, unsigned long mmcr[]) } static int power7_generic_events[] = { - [PERF_COUNT_HW_CPU_CYCLES] = PME_PM_CYC, - [PERF_COUNT_HW_STALLED_CYCLES_FRONTEND] = PME_PM_GCT_NOSLOT_CYC, - [PERF_COUNT_HW_STALLED_CYCLES_BACKEND] = PME_PM_CMPLU_STALL, - [PERF_COUNT_HW_INSTRUCTIONS] = PME_PM_INST_CMPL, - [PERF_COUNT_HW_CACHE_REFERENCES] = PME_PM_LD_REF_L1, - [PERF_COUNT_HW_CACHE_MISSES] = PME_PM_LD_MISS_L1, - [PERF_COUNT_HW_BRANCH_INSTRUCTIONS] = PME_PM_BRU_FIN, - [PERF_COUNT_HW_BRANCH_MISSES] = PME_PM_BR_MPRED, + [PERF_COUNT_HW_CPU_CYCLES] = PM_CYC, + [PERF_COUNT_HW_STALLED_CYCLES_FRONTEND] = PM_GCT_NOSLOT_CYC, + [PERF_COUNT_HW_STALLED_CYCLES_BACKEND] = PM_CMPLU_STALL, + [PERF_COUNT_HW_INSTRUCTIONS] = PM_INST_CMPL, + [PERF_COUNT_HW_CACHE_REFERENCES] = PM_LD_REF_L1, + [PERF_COUNT_HW_CACHE_MISSES] = PM_LD_MISS_L1, + [PERF_COUNT_HW_BRANCH_INSTRUCTIONS] = PM_BRU_FIN, + [PERF_COUNT_HW_BRANCH_MISSES] = PM_BR_MPRED, }; #define C(x) PERF_COUNT_HW_CACHE_##x -- 2.5.3 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 2/2] powerpc/perf: Export Power8 generic and cache events to sysfs 2016-01-11 22:55 [PATCH 1/2] powerpc/perf: Remove PME_ prefix for power7 events Sukadev Bhattiprolu @ 2016-01-11 22:55 ` Sukadev Bhattiprolu 2016-01-14 11:57 ` Madhavan Srinivasan 2016-03-11 0:30 ` [1/2] powerpc/perf: Remove PME_ prefix for power7 events Michael Ellerman 1 sibling, 1 reply; 5+ messages in thread From: Sukadev Bhattiprolu @ 2016-01-11 22:55 UTC (permalink / raw) To: Michael Ellerman, Arnaldo Carvalho de Melo, Jiri Olsa Cc: linuxppc-dev, linux-kernel Power8 supports a large number of events in each susbystem so when a user runs: perf stat -e branch-instructions sleep 1 perf stat -e L1-dcache-loads sleep 1 it is not clear as to which PMU events were monitored. Export the generic hardware and cache perf events for Power8 to sysfs, so users can precisely determine the PMU event monitored by the generic event. Eg: cat /sys/bus/event_source/devices/cpu/events/branch-instructions event=0x10068 $ cat /sys/bus/event_source/devices/cpu/events/L1-dcache-loads event=0x100ee Signed-off-by: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com> --- arch/powerpc/include/asm/perf_event_server.h | 8 ++ arch/powerpc/perf/power8-events-list.h | 51 +++++++++++++ arch/powerpc/perf/power8-pmu.c | 110 ++++++++++++++++++--------- 3 files changed, 131 insertions(+), 38 deletions(-) create mode 100644 arch/powerpc/perf/power8-events-list.h diff --git a/arch/powerpc/include/asm/perf_event_server.h b/arch/powerpc/include/asm/perf_event_server.h index 0691087..e157489 100644 --- a/arch/powerpc/include/asm/perf_event_server.h +++ b/arch/powerpc/include/asm/perf_event_server.h @@ -136,6 +136,11 @@ extern ssize_t power_events_sysfs_show(struct device *dev, * event 'cpu-cycles' can have two entries in sysfs: 'cpu-cycles' and * 'PM_CYC' where the latter is the name by which the event is known in * POWER CPU specification. + * + * Similarly, some hardware and cache events use the same event code. Eg. + * on POWER8, both "cache-references" and "L1-dcache-loads" events refer + * to the same event, PM_LD_REF_L1. The suffix, allows us to have two + * sysfs objects for the same event and thus two entries/aliases in sysfs. */ #define EVENT_VAR(_id, _suffix) event_attr_##_id##_suffix #define EVENT_PTR(_id, _suffix) &EVENT_VAR(_id, _suffix).attr.attr @@ -147,5 +152,8 @@ extern ssize_t power_events_sysfs_show(struct device *dev, #define GENERIC_EVENT_ATTR(_name, _id) EVENT_ATTR(_name, _id, _g) #define GENERIC_EVENT_PTR(_id) EVENT_PTR(_id, _g) +#define CACHE_EVENT_ATTR(_name, _id) EVENT_ATTR(_name, _id, _c) +#define CACHE_EVENT_PTR(_id) EVENT_PTR(_id, _c) + #define POWER_EVENT_ATTR(_name, _id) EVENT_ATTR(_name, _id, _p) #define POWER_EVENT_PTR(_id) EVENT_PTR(_id, _p) diff --git a/arch/powerpc/perf/power8-events-list.h b/arch/powerpc/perf/power8-events-list.h new file mode 100644 index 0000000..741b77e --- /dev/null +++ b/arch/powerpc/perf/power8-events-list.h @@ -0,0 +1,51 @@ +/* + * Performance counter support for POWER8 processors. + * + * Copyright 2014 Sukadev Bhattiprolu, IBM Corporation. + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License + * as published by the Free Software Foundation; either version + * 2 of the License, or (at your option) any later version. + */ + +/* + * Power8 event codes. + */ +EVENT(PM_CYC, 0x0001e) +EVENT(PM_GCT_NOSLOT_CYC, 0x100f8) +EVENT(PM_CMPLU_STALL, 0x4000a) +EVENT(PM_INST_CMPL, 0x00002) +EVENT(PM_BRU_FIN, 0x10068) +EVENT(PM_BR_MPRED_CMPL, 0x400f6) + +/* All L1 D cache load references counted at finish, gated by reject */ +EVENT(PM_LD_REF_L1, 0x100ee) +/* Load Missed L1 */ +EVENT(PM_LD_MISS_L1, 0x3e054) +/* Store Missed L1 */ +EVENT(PM_ST_MISS_L1, 0x300f0) +/* L1 cache data prefetches */ +EVENT(PM_L1_PREF, 0x0d8b8) +/* Instruction fetches from L1 */ +EVENT(PM_INST_FROM_L1, 0x04080) +/* Demand iCache Miss */ +EVENT(PM_L1_ICACHE_MISS, 0x200fd) +/* Instruction Demand sectors wriittent into IL1 */ +EVENT(PM_L1_DEMAND_WRITE, 0x0408c) +/* Instruction prefetch written into IL1 */ +EVENT(PM_IC_PREF_WRITE, 0x0408e) +/* The data cache was reloaded from local core's L3 due to a demand load */ +EVENT(PM_DATA_FROM_L3, 0x4c042) +/* Demand LD - L3 Miss (not L2 hit and not L3 hit) */ +EVENT(PM_DATA_FROM_L3MISS, 0x300fe) +/* All successful D-side store dispatches for this thread */ +EVENT(PM_L2_ST, 0x17080) +/* All successful D-side store dispatches for this thread that were L2 Miss */ +EVENT(PM_L2_ST_MISS, 0x17082) +/* Total HW L3 prefetches(Load+store) */ +EVENT(PM_L3_PREF_ALL, 0x4e052) +/* Data PTEG reload */ +EVENT(PM_DTLB_MISS, 0x300fc) +/* ITLB Reloaded */ +EVENT(PM_ITLB_MISS, 0x400fc) diff --git a/arch/powerpc/perf/power8-pmu.c b/arch/powerpc/perf/power8-pmu.c index 7d5e295..08ab998 100644 --- a/arch/powerpc/perf/power8-pmu.c +++ b/arch/powerpc/perf/power8-pmu.c @@ -17,48 +17,16 @@ #include <asm/firmware.h> #include <asm/cputable.h> - /* * Some power8 event codes. */ -#define PM_CYC 0x0001e -#define PM_GCT_NOSLOT_CYC 0x100f8 -#define PM_CMPLU_STALL 0x4000a -#define PM_INST_CMPL 0x00002 -#define PM_BRU_FIN 0x10068 -#define PM_BR_MPRED_CMPL 0x400f6 - -/* All L1 D cache load references counted at finish, gated by reject */ -#define PM_LD_REF_L1 0x100ee -/* Load Missed L1 */ -#define PM_LD_MISS_L1 0x3e054 -/* Store Missed L1 */ -#define PM_ST_MISS_L1 0x300f0 -/* L1 cache data prefetches */ -#define PM_L1_PREF 0x0d8b8 -/* Instruction fetches from L1 */ -#define PM_INST_FROM_L1 0x04080 -/* Demand iCache Miss */ -#define PM_L1_ICACHE_MISS 0x200fd -/* Instruction Demand sectors wriittent into IL1 */ -#define PM_L1_DEMAND_WRITE 0x0408c -/* Instruction prefetch written into IL1 */ -#define PM_IC_PREF_WRITE 0x0408e -/* The data cache was reloaded from local core's L3 due to a demand load */ -#define PM_DATA_FROM_L3 0x4c042 -/* Demand LD - L3 Miss (not L2 hit and not L3 hit) */ -#define PM_DATA_FROM_L3MISS 0x300fe -/* All successful D-side store dispatches for this thread */ -#define PM_L2_ST 0x17080 -/* All successful D-side store dispatches for this thread that were L2 Miss */ -#define PM_L2_ST_MISS 0x17082 -/* Total HW L3 prefetches(Load+store) */ -#define PM_L3_PREF_ALL 0x4e052 -/* Data PTEG reload */ -#define PM_DTLB_MISS 0x300fc -/* ITLB Reloaded */ -#define PM_ITLB_MISS 0x400fc +#define EVENT(_name, _code) _name = _code, + +enum { +#include "power8-events-list.h" +}; +#undef EVENT /* * Raw event encoding for POWER8: @@ -604,6 +572,71 @@ static void power8_disable_pmc(unsigned int pmc, unsigned long mmcr[]) mmcr[1] &= ~(0xffUL << MMCR1_PMCSEL_SHIFT(pmc + 1)); } +GENERIC_EVENT_ATTR(cpu-cycles, PM_CYC); +GENERIC_EVENT_ATTR(stalled-cycles-frontend, PM_GCT_NOSLOT_CYC); +GENERIC_EVENT_ATTR(stalled-cycles-backend, PM_CMPLU_STALL); +GENERIC_EVENT_ATTR(instructions, PM_INST_CMPL); +GENERIC_EVENT_ATTR(branch-instructions, PM_BRU_FIN); +GENERIC_EVENT_ATTR(branch-misses, PM_BR_MPRED_CMPL); +GENERIC_EVENT_ATTR(cache-references, PM_LD_REF_L1); +GENERIC_EVENT_ATTR(cache-misses, PM_LD_MISS_L1); + +CACHE_EVENT_ATTR(L1-dcache-load-misses, PM_LD_MISS_L1); +CACHE_EVENT_ATTR(L1-dcache-loads, PM_LD_REF_L1); + +CACHE_EVENT_ATTR(L1-dcache-prefetches, PM_L1_PREF); +CACHE_EVENT_ATTR(L1-dcache-store-misses, PM_ST_MISS_L1); +CACHE_EVENT_ATTR(L1-icache-load-misses, PM_L1_ICACHE_MISS); +CACHE_EVENT_ATTR(L1-icache-loads, PM_INST_FROM_L1); +CACHE_EVENT_ATTR(L1-icache-prefetches, PM_IC_PREF_WRITE); + +CACHE_EVENT_ATTR(LLC-load-misses, PM_DATA_FROM_L3MISS); +CACHE_EVENT_ATTR(LLC-loads, PM_DATA_FROM_L3); +CACHE_EVENT_ATTR(LLC-prefetches, PM_L3_PREF_ALL); +CACHE_EVENT_ATTR(LLC-store-misses, PM_L2_ST_MISS); +CACHE_EVENT_ATTR(LLC-stores, PM_L2_ST); + +CACHE_EVENT_ATTR(branch-load-misses, PM_BR_MPRED_CMPL); +CACHE_EVENT_ATTR(branch-loads, PM_BRU_FIN); +CACHE_EVENT_ATTR(dTLB-load-misses, PM_DTLB_MISS); +CACHE_EVENT_ATTR(iTLB-load-misses, PM_ITLB_MISS); + +static struct attribute *power8_events_attr[] = { + GENERIC_EVENT_PTR(PM_CYC), + GENERIC_EVENT_PTR(PM_GCT_NOSLOT_CYC), + GENERIC_EVENT_PTR(PM_CMPLU_STALL), + GENERIC_EVENT_PTR(PM_INST_CMPL), + GENERIC_EVENT_PTR(PM_BRU_FIN), + GENERIC_EVENT_PTR(PM_BR_MPRED_CMPL), + GENERIC_EVENT_PTR(PM_LD_REF_L1), + GENERIC_EVENT_PTR(PM_LD_MISS_L1), + + CACHE_EVENT_PTR(PM_LD_MISS_L1), + CACHE_EVENT_PTR(PM_LD_REF_L1), + CACHE_EVENT_PTR(PM_L1_PREF), + CACHE_EVENT_PTR(PM_ST_MISS_L1), + CACHE_EVENT_PTR(PM_L1_ICACHE_MISS), + CACHE_EVENT_PTR(PM_INST_FROM_L1), + CACHE_EVENT_PTR(PM_IC_PREF_WRITE), + CACHE_EVENT_PTR(PM_DATA_FROM_L3MISS), + CACHE_EVENT_PTR(PM_DATA_FROM_L3), + CACHE_EVENT_PTR(PM_L3_PREF_ALL), + CACHE_EVENT_PTR(PM_L2_ST_MISS), + CACHE_EVENT_PTR(PM_L2_ST), + + CACHE_EVENT_PTR(PM_BR_MPRED_CMPL), + CACHE_EVENT_PTR(PM_BRU_FIN), + + CACHE_EVENT_PTR(PM_DTLB_MISS), + CACHE_EVENT_PTR(PM_ITLB_MISS), + NULL +}; + +static struct attribute_group power8_pmu_events_group = { + .name = "events", + .attrs = power8_events_attr, +}; + PMU_FORMAT_ATTR(event, "config:0-49"); PMU_FORMAT_ATTR(pmcxsel, "config:0-7"); PMU_FORMAT_ATTR(mark, "config:8"); @@ -640,6 +673,7 @@ struct attribute_group power8_pmu_format_group = { static const struct attribute_group *power8_pmu_attr_groups[] = { &power8_pmu_format_group, + &power8_pmu_events_group, NULL, }; -- 2.5.3 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] powerpc/perf: Export Power8 generic and cache events to sysfs 2016-01-11 22:55 ` [PATCH 2/2] powerpc/perf: Export Power8 generic and cache events to sysfs Sukadev Bhattiprolu @ 2016-01-14 11:57 ` Madhavan Srinivasan 2016-01-15 19:44 ` Sukadev Bhattiprolu 0 siblings, 1 reply; 5+ messages in thread From: Madhavan Srinivasan @ 2016-01-14 11:57 UTC (permalink / raw) To: Sukadev Bhattiprolu, Michael Ellerman, Arnaldo Carvalho de Melo, Jiri Olsa Cc: linuxppc-dev, linux-kernel On Tuesday 12 January 2016 04:25 AM, Sukadev Bhattiprolu wrote: > Power8 supports a large number of events in each susbystem so when a > user runs: > > perf stat -e branch-instructions sleep 1 > perf stat -e L1-dcache-loads sleep 1 > > it is not clear as to which PMU events were monitored. > > Export the generic hardware and cache perf events for Power8 to sysfs, > so users can precisely determine the PMU event monitored by the generic > event. > > Eg: > cat /sys/bus/event_source/devices/cpu/events/branch-instructions > event=0x10068 > > $ cat /sys/bus/event_source/devices/cpu/events/L1-dcache-loads > event=0x100ee > > Signed-off-by: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com> > --- > arch/powerpc/include/asm/perf_event_server.h | 8 ++ > arch/powerpc/perf/power8-events-list.h | 51 +++++++++++++ > arch/powerpc/perf/power8-pmu.c | 110 ++++++++++++++++++--------- > 3 files changed, 131 insertions(+), 38 deletions(-) > create mode 100644 arch/powerpc/perf/power8-events-list.h > > diff --git a/arch/powerpc/include/asm/perf_event_server.h b/arch/powerpc/include/asm/perf_event_server.h > index 0691087..e157489 100644 > --- a/arch/powerpc/include/asm/perf_event_server.h > +++ b/arch/powerpc/include/asm/perf_event_server.h > @@ -136,6 +136,11 @@ extern ssize_t power_events_sysfs_show(struct device *dev, > * event 'cpu-cycles' can have two entries in sysfs: 'cpu-cycles' and > * 'PM_CYC' where the latter is the name by which the event is known in > * POWER CPU specification. > + * > + * Similarly, some hardware and cache events use the same event code. Eg. > + * on POWER8, both "cache-references" and "L1-dcache-loads" events refer > + * to the same event, PM_LD_REF_L1. The suffix, allows us to have two > + * sysfs objects for the same event and thus two entries/aliases in sysfs. > */ > #define EVENT_VAR(_id, _suffix) event_attr_##_id##_suffix > #define EVENT_PTR(_id, _suffix) &EVENT_VAR(_id, _suffix).attr.attr > @@ -147,5 +152,8 @@ extern ssize_t power_events_sysfs_show(struct device *dev, > #define GENERIC_EVENT_ATTR(_name, _id) EVENT_ATTR(_name, _id, _g) > #define GENERIC_EVENT_PTR(_id) EVENT_PTR(_id, _g) > > +#define CACHE_EVENT_ATTR(_name, _id) EVENT_ATTR(_name, _id, _c) > +#define CACHE_EVENT_PTR(_id) EVENT_PTR(_id, _c) > + > #define POWER_EVENT_ATTR(_name, _id) EVENT_ATTR(_name, _id, _p) > #define POWER_EVENT_PTR(_id) EVENT_PTR(_id, _p) > diff --git a/arch/powerpc/perf/power8-events-list.h b/arch/powerpc/perf/power8-events-list.h > new file mode 100644 > index 0000000..741b77e > --- /dev/null > +++ b/arch/powerpc/perf/power8-events-list.h > @@ -0,0 +1,51 @@ > +/* > + * Performance counter support for POWER8 processors. > + * > + * Copyright 2014 Sukadev Bhattiprolu, IBM Corporation. > + * > + * This program is free software; you can redistribute it and/or > + * modify it under the terms of the GNU General Public License > + * as published by the Free Software Foundation; either version > + * 2 of the License, or (at your option) any later version. > + */ > + > +/* > + * Power8 event codes. > + */ > +EVENT(PM_CYC, 0x0001e) > +EVENT(PM_GCT_NOSLOT_CYC, 0x100f8) > +EVENT(PM_CMPLU_STALL, 0x4000a) > +EVENT(PM_INST_CMPL, 0x00002) > +EVENT(PM_BRU_FIN, 0x10068) > +EVENT(PM_BR_MPRED_CMPL, 0x400f6) > + > +/* All L1 D cache load references counted at finish, gated by reject */ > +EVENT(PM_LD_REF_L1, 0x100ee) > +/* Load Missed L1 */ > +EVENT(PM_LD_MISS_L1, 0x3e054) > +/* Store Missed L1 */ > +EVENT(PM_ST_MISS_L1, 0x300f0) > +/* L1 cache data prefetches */ > +EVENT(PM_L1_PREF, 0x0d8b8) > +/* Instruction fetches from L1 */ > +EVENT(PM_INST_FROM_L1, 0x04080) > +/* Demand iCache Miss */ > +EVENT(PM_L1_ICACHE_MISS, 0x200fd) > +/* Instruction Demand sectors wriittent into IL1 */ > +EVENT(PM_L1_DEMAND_WRITE, 0x0408c) > +/* Instruction prefetch written into IL1 */ > +EVENT(PM_IC_PREF_WRITE, 0x0408e) > +/* The data cache was reloaded from local core's L3 due to a demand load */ > +EVENT(PM_DATA_FROM_L3, 0x4c042) > +/* Demand LD - L3 Miss (not L2 hit and not L3 hit) */ > +EVENT(PM_DATA_FROM_L3MISS, 0x300fe) > +/* All successful D-side store dispatches for this thread */ > +EVENT(PM_L2_ST, 0x17080) > +/* All successful D-side store dispatches for this thread that were L2 Miss */ > +EVENT(PM_L2_ST_MISS, 0x17082) > +/* Total HW L3 prefetches(Load+store) */ > +EVENT(PM_L3_PREF_ALL, 0x4e052) > +/* Data PTEG reload */ > +EVENT(PM_DTLB_MISS, 0x300fc) > +/* ITLB Reloaded */ > +EVENT(PM_ITLB_MISS, 0x400fc) > diff --git a/arch/powerpc/perf/power8-pmu.c b/arch/powerpc/perf/power8-pmu.c > index 7d5e295..08ab998 100644 > --- a/arch/powerpc/perf/power8-pmu.c > +++ b/arch/powerpc/perf/power8-pmu.c > @@ -17,48 +17,16 @@ > #include <asm/firmware.h> > #include <asm/cputable.h> > > - > /* > * Some power8 event codes. > */ > -#define PM_CYC 0x0001e > -#define PM_GCT_NOSLOT_CYC 0x100f8 > -#define PM_CMPLU_STALL 0x4000a > -#define PM_INST_CMPL 0x00002 > -#define PM_BRU_FIN 0x10068 > -#define PM_BR_MPRED_CMPL 0x400f6 > - > -/* All L1 D cache load references counted at finish, gated by reject */ > -#define PM_LD_REF_L1 0x100ee > -/* Load Missed L1 */ > -#define PM_LD_MISS_L1 0x3e054 > -/* Store Missed L1 */ > -#define PM_ST_MISS_L1 0x300f0 > -/* L1 cache data prefetches */ > -#define PM_L1_PREF 0x0d8b8 > -/* Instruction fetches from L1 */ > -#define PM_INST_FROM_L1 0x04080 > -/* Demand iCache Miss */ > -#define PM_L1_ICACHE_MISS 0x200fd > -/* Instruction Demand sectors wriittent into IL1 */ > -#define PM_L1_DEMAND_WRITE 0x0408c > -/* Instruction prefetch written into IL1 */ > -#define PM_IC_PREF_WRITE 0x0408e > -/* The data cache was reloaded from local core's L3 due to a demand load */ > -#define PM_DATA_FROM_L3 0x4c042 > -/* Demand LD - L3 Miss (not L2 hit and not L3 hit) */ > -#define PM_DATA_FROM_L3MISS 0x300fe > -/* All successful D-side store dispatches for this thread */ > -#define PM_L2_ST 0x17080 > -/* All successful D-side store dispatches for this thread that were L2 Miss */ > -#define PM_L2_ST_MISS 0x17082 > -/* Total HW L3 prefetches(Load+store) */ > -#define PM_L3_PREF_ALL 0x4e052 > -/* Data PTEG reload */ > -#define PM_DTLB_MISS 0x300fc > -/* ITLB Reloaded */ > -#define PM_ITLB_MISS 0x400fc > +#define EVENT(_name, _code) _name = _code, > + > +enum { > +#include "power8-events-list.h" > +}; > IIUC we dont use this enum as a index to some array, so can we do something like this? #define NO_ATTRS(n) #define ENUM(x,y) enum{x = y} #define EVENTS__ATTR(a,b,c,d,e,f) ENUM(a,b);\ c(d,a);\ e(f,a) #define EVENTS(a,b) ENUM(a,b) EVENTS_ATTR(PM_CYC, 0x0001e, GENERIC_EVENT_ATTR, cpu-cycles, NO_ATTRS); EVENTS_ATTR(PM_LD_REF_L1, 0x000ee, GENERIC_EVENT_ATTR, cache-references, CACHE_EVENT_ATTR, L1-dcache-loads); EVENTS(__FOR_EX__, 0x00111) This was we can include the header file and get a way with the weird #def,undef EVENT Maddy > +#undef EVENT > > /* > * Raw event encoding for POWER8: > @@ -604,6 +572,71 @@ static void power8_disable_pmc(unsigned int pmc, unsigned long mmcr[]) > mmcr[1] &= ~(0xffUL << MMCR1_PMCSEL_SHIFT(pmc + 1)); > } > > +GENERIC_EVENT_ATTR(cpu-cycles, PM_CYC); > +GENERIC_EVENT_ATTR(stalled-cycles-frontend, PM_GCT_NOSLOT_CYC); > +GENERIC_EVENT_ATTR(stalled-cycles-backend, PM_CMPLU_STALL); > +GENERIC_EVENT_ATTR(instructions, PM_INST_CMPL); > +GENERIC_EVENT_ATTR(branch-instructions, PM_BRU_FIN); > +GENERIC_EVENT_ATTR(branch-misses, PM_BR_MPRED_CMPL); > +GENERIC_EVENT_ATTR(cache-references, PM_LD_REF_L1); > +GENERIC_EVENT_ATTR(cache-misses, PM_LD_MISS_L1); > + > +CACHE_EVENT_ATTR(L1-dcache-load-misses, PM_LD_MISS_L1); > +CACHE_EVENT_ATTR(L1-dcache-loads, PM_LD_REF_L1); > + > +CACHE_EVENT_ATTR(L1-dcache-prefetches, PM_L1_PREF); > +CACHE_EVENT_ATTR(L1-dcache-store-misses, PM_ST_MISS_L1); > +CACHE_EVENT_ATTR(L1-icache-load-misses, PM_L1_ICACHE_MISS); > +CACHE_EVENT_ATTR(L1-icache-loads, PM_INST_FROM_L1); > +CACHE_EVENT_ATTR(L1-icache-prefetches, PM_IC_PREF_WRITE); > + > +CACHE_EVENT_ATTR(LLC-load-misses, PM_DATA_FROM_L3MISS); > +CACHE_EVENT_ATTR(LLC-loads, PM_DATA_FROM_L3); > +CACHE_EVENT_ATTR(LLC-prefetches, PM_L3_PREF_ALL); > +CACHE_EVENT_ATTR(LLC-store-misses, PM_L2_ST_MISS); > +CACHE_EVENT_ATTR(LLC-stores, PM_L2_ST); > + > +CACHE_EVENT_ATTR(branch-load-misses, PM_BR_MPRED_CMPL); > +CACHE_EVENT_ATTR(branch-loads, PM_BRU_FIN); > +CACHE_EVENT_ATTR(dTLB-load-misses, PM_DTLB_MISS); > +CACHE_EVENT_ATTR(iTLB-load-misses, PM_ITLB_MISS); > + > +static struct attribute *power8_events_attr[] = { > + GENERIC_EVENT_PTR(PM_CYC), > + GENERIC_EVENT_PTR(PM_GCT_NOSLOT_CYC), > + GENERIC_EVENT_PTR(PM_CMPLU_STALL), > + GENERIC_EVENT_PTR(PM_INST_CMPL), > + GENERIC_EVENT_PTR(PM_BRU_FIN), > + GENERIC_EVENT_PTR(PM_BR_MPRED_CMPL), > + GENERIC_EVENT_PTR(PM_LD_REF_L1), > + GENERIC_EVENT_PTR(PM_LD_MISS_L1), > + > + CACHE_EVENT_PTR(PM_LD_MISS_L1), > + CACHE_EVENT_PTR(PM_LD_REF_L1), > + CACHE_EVENT_PTR(PM_L1_PREF), > + CACHE_EVENT_PTR(PM_ST_MISS_L1), > + CACHE_EVENT_PTR(PM_L1_ICACHE_MISS), > + CACHE_EVENT_PTR(PM_INST_FROM_L1), > + CACHE_EVENT_PTR(PM_IC_PREF_WRITE), > + CACHE_EVENT_PTR(PM_DATA_FROM_L3MISS), > + CACHE_EVENT_PTR(PM_DATA_FROM_L3), > + CACHE_EVENT_PTR(PM_L3_PREF_ALL), > + CACHE_EVENT_PTR(PM_L2_ST_MISS), > + CACHE_EVENT_PTR(PM_L2_ST), > + > + CACHE_EVENT_PTR(PM_BR_MPRED_CMPL), > + CACHE_EVENT_PTR(PM_BRU_FIN), > + > + CACHE_EVENT_PTR(PM_DTLB_MISS), > + CACHE_EVENT_PTR(PM_ITLB_MISS), > + NULL > +}; > + > +static struct attribute_group power8_pmu_events_group = { > + .name = "events", > + .attrs = power8_events_attr, > +}; > + > PMU_FORMAT_ATTR(event, "config:0-49"); > PMU_FORMAT_ATTR(pmcxsel, "config:0-7"); > PMU_FORMAT_ATTR(mark, "config:8"); > @@ -640,6 +673,7 @@ struct attribute_group power8_pmu_format_group = { > > static const struct attribute_group *power8_pmu_attr_groups[] = { > &power8_pmu_format_group, > + &power8_pmu_events_group, > NULL, > }; > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] powerpc/perf: Export Power8 generic and cache events to sysfs 2016-01-14 11:57 ` Madhavan Srinivasan @ 2016-01-15 19:44 ` Sukadev Bhattiprolu 0 siblings, 0 replies; 5+ messages in thread From: Sukadev Bhattiprolu @ 2016-01-15 19:44 UTC (permalink / raw) To: Madhavan Srinivasan Cc: Michael Ellerman, Arnaldo Carvalho de Melo, Jiri Olsa, linuxppc-dev, linux-kernel Madhavan Srinivasan [maddy@linux.vnet.ibm.com] wrote: > > +enum { > > +#include "power8-events-list.h" > > +}; > > > > IIUC we dont use this enum as a index to > some array, so can we do something like this? Possibly, but its already a macro soup and we would add another layer of indirection? I agree though that the #define and #undefs are not intuitive. Since it is only a few entries for Power8, was keeping it consistent with Power7. Sukadev ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [1/2] powerpc/perf: Remove PME_ prefix for power7 events 2016-01-11 22:55 [PATCH 1/2] powerpc/perf: Remove PME_ prefix for power7 events Sukadev Bhattiprolu 2016-01-11 22:55 ` [PATCH 2/2] powerpc/perf: Export Power8 generic and cache events to sysfs Sukadev Bhattiprolu @ 2016-03-11 0:30 ` Michael Ellerman 1 sibling, 0 replies; 5+ messages in thread From: Michael Ellerman @ 2016-03-11 0:30 UTC (permalink / raw) To: Sukadev Bhattiprolu, Arnaldo Carvalho de Melo, Jiri Olsa Cc: linuxppc-dev, linux-kernel On Mon, 2016-11-01 at 22:55:25 UTC, Sukadev Bhattiprolu wrote: > We used the PME_ prefix earlier to avoid some macro/variable name > collisions. We have since changed the way we define/use the event > macros so we no longer need the prefix. > > By dropping the prefix, we keep the the event macros consistent with > their official names. > > Reported-by: Michael Ellerman <ellerman@au1.ibm.com> > Signed-off-by: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com> Series applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/d4969e2459c6e852a6862256cf cheers ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-03-11 0:30 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-01-11 22:55 [PATCH 1/2] powerpc/perf: Remove PME_ prefix for power7 events Sukadev Bhattiprolu 2016-01-11 22:55 ` [PATCH 2/2] powerpc/perf: Export Power8 generic and cache events to sysfs Sukadev Bhattiprolu 2016-01-14 11:57 ` Madhavan Srinivasan 2016-01-15 19:44 ` Sukadev Bhattiprolu 2016-03-11 0:30 ` [1/2] powerpc/perf: Remove PME_ prefix for power7 events Michael Ellerman
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.