* [PATCH v2 1/2] perf mem: Simplify Arm SPE event config
2025-12-12 17:24 [PATCH v2 0/2] perf arm_spe: Minor event configs housekeeping Leo Yan
@ 2025-12-12 17:24 ` Leo Yan
2025-12-12 17:25 ` [PATCH v2 2/2] perf c2c: Update documentation for adding memory event table Leo Yan
2026-01-16 11:09 ` [PATCH v2 0/2] perf arm_spe: Minor event configs housekeeping Leo Yan
2 siblings, 0 replies; 6+ messages in thread
From: Leo Yan @ 2025-12-12 17:24 UTC (permalink / raw)
To: Arnaldo Carvalho de Melo, Namhyung Kim, Ian Rogers, James Clark,
Mike Leach, Will Deacon, Mark Rutland, Jiri Olsa, Adrian Hunter,
Al Grant
Cc: linux-arm-kernel, linux-perf-users, linux-kernel, Leo Yan
Since configuration fields default to zero, the zero assignments are
redundant, remove them.
Reviewed-by: James Clark <james.clark@linaro.org>
Signed-off-by: Leo Yan <leo.yan@arm.com>
---
tools/perf/arch/arm64/util/mem-events.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/perf/arch/arm64/util/mem-events.c b/tools/perf/arch/arm64/util/mem-events.c
index 9f8da7937255cc9b14c3c58a1119b40bd0c76f6b..eaf00e0609c6c1b7d939a02fe3794471d1ed119b 100644
--- a/tools/perf/arch/arm64/util/mem-events.c
+++ b/tools/perf/arch/arm64/util/mem-events.c
@@ -6,7 +6,7 @@
#define E(t, n, s, l, a) { .tag = t, .name = n, .event_name = s, .ldlat = l, .aux_event = a }
struct perf_mem_event perf_mem_events_arm[PERF_MEM_EVENTS__MAX] = {
- E("spe-load", "%s/ts_enable=1,pa_enable=1,load_filter=1,store_filter=0,min_latency=%u/", NULL, true, 0),
- E("spe-store", "%s/ts_enable=1,pa_enable=1,load_filter=0,store_filter=1/", NULL, false, 0),
+ E("spe-load", "%s/ts_enable=1,pa_enable=1,load_filter=1,min_latency=%u/", NULL, true, 0),
+ E("spe-store", "%s/ts_enable=1,pa_enable=1,store_filter=1/", NULL, false, 0),
E("spe-ldst", "%s/ts_enable=1,pa_enable=1,load_filter=1,store_filter=1,min_latency=%u/", NULL, true, 0),
};
--
2.34.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH v2 2/2] perf c2c: Update documentation for adding memory event table
2025-12-12 17:24 [PATCH v2 0/2] perf arm_spe: Minor event configs housekeeping Leo Yan
2025-12-12 17:24 ` [PATCH v2 1/2] perf mem: Simplify Arm SPE event config Leo Yan
@ 2025-12-12 17:25 ` Leo Yan
2025-12-15 8:40 ` James Clark
2026-01-16 11:09 ` [PATCH v2 0/2] perf arm_spe: Minor event configs housekeeping Leo Yan
2 siblings, 1 reply; 6+ messages in thread
From: Leo Yan @ 2025-12-12 17:25 UTC (permalink / raw)
To: Arnaldo Carvalho de Melo, Namhyung Kim, Ian Rogers, James Clark,
Mike Leach, Will Deacon, Mark Rutland, Jiri Olsa, Adrian Hunter,
Al Grant
Cc: linux-arm-kernel, linux-perf-users, linux-kernel, Leo Yan
Users may occasionally need to see which options are applied to memory
events. This helps to understand the behavior of "perf c2c" and
"perf mem", and provides guidance for configuring memory event options
directly.
Add a table to track memory events and their corresponding options, and
include the Arm SPE events in it.
Suggested-by: Al Grant <al.grant@arm.com>
Signed-off-by: Leo Yan <leo.yan@arm.com>
---
tools/perf/Documentation/perf-c2c.txt | 51 +++++++++++++++++++++++++----------
1 file changed, 37 insertions(+), 14 deletions(-)
diff --git a/tools/perf/Documentation/perf-c2c.txt b/tools/perf/Documentation/perf-c2c.txt
index 40b0f71a2c44eb642ff3bb234631a614b7c4fc9d..e57a122b8719f1b2e20dd509959fc1563a6c20fd 100644
--- a/tools/perf/Documentation/perf-c2c.txt
+++ b/tools/perf/Documentation/perf-c2c.txt
@@ -160,20 +160,43 @@ Following perf record options are configured by default:
-W,-d,--phys-data,--sample-cpu
-Unless specified otherwise with '-e' option, following events are monitored by
-default on Intel:
-
- cpu/mem-loads,ldlat=30/P
- cpu/mem-stores/P
-
-following on AMD:
-
- ibs_op//
-
-and following on PowerPC:
-
- cpu/mem-loads/
- cpu/mem-stores/
+The following table lists the events monitored on different architectures.
+Unless specified otherwise with the -e option, the tool will select the
+default events.
+
+ +--------+---------------+-----------------+--------------------------------------------------------------------------------+
+ | Arch | Configuration | Options | Events |
+ +--------+---------------+-----------------+--------------------------------------------------------------------------------+
+ | Intel | Default | -e ldlat-loads | cpu/mem-loads,ldlat=30/P |
+ | | | -e ldlat-stores | cpu/mem-stores/P |
+ | |---------------+-----------------+--------------------------------------------------------------------------------+
+ | | Load only | -e ldlat-loads | cpu/mem-loads,ldlat=30/P |
+ | |---------------+-----------------+--------------------------------------------------------------------------------+
+ | | Store only | -e ldlat-stores | cpu/mem-stores/P |
+ +--------+---------------+-----------------+--------------------------------------------------------------------------------+
+ | Intel | Default | -e ldlat-loads | {cpu/mem-loads-aux/,cpu/mem-loads,ldlat=30/}:P |
+ | with | | -e ldlat-stores | cpu/mem-stores/P |
+ | AUX |--------------+------------------+--------------------------------------------------------------------------------+
+ | | Load only | -e ldlat-loads | {cpu/mem-loads-aux/,cpu/mem-loads,ldlat=30/}:P |
+ | |---------------+-----------------+--------------------------------------------------------------------------------+
+ | | Store only | -e ldlat-stores | cpu/mem-stores/P |
+ +--------+---------------+-----------------+--------------------------------------------------------------------------------+
+ | AMD | Default | -e mem-ldst | ibs_op// (without latency support) |
+ | | | | ibs_op/ldlat=30/ (with latency support) |
+ +--------+---------------+-----------------+--------------------------------------------------------------------------------+
+ | PowerPC| Default | -e ldlat-loads | cpu/mem-loads/ |
+ | | | -e ldlat-stores | cpu/mem-stores/ |
+ | |---------------+-----------------+--------------------------------------------------------------------------------+
+ | | Load only | -e ldlat-loads | cpu/mem-loads/ |
+ | |---------------+-----------------+--------------------------------------------------------------------------------+
+ | | Store only | -e ldlat-stores | cpu/mem-stores/ |
+ +--------+---------------+-----------------+--------------------------------------------------------------------------------+
+ | Arm | Default | -e spe-ldst | arm_spe_0/ts_enable=1,pa_enable=1,load_filter=1,store_filter=1,min_latency=30/ |
+ | SPE |---------------+-----------------+--------------------------------------------------------------------------------+
+ | | Load only | -e spe-load | arm_spe_0/ts_enable=1,pa_enable=1,load_filter=1,min_latency=30/ |
+ | |---------------+-----------------+--------------------------------------------------------------------------------+
+ | | Store only | -e spe-store | arm_spe_0/ts_enable=1,pa_enable=1,store_filter=1/ |
+ +--------+---------------+-----------------+--------------------------------------------------------------------------------+
User can pass any 'perf record' option behind '--' mark, like (to enable
callchains and system wide monitoring):
--
2.34.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH v2 2/2] perf c2c: Update documentation for adding memory event table
2025-12-12 17:25 ` [PATCH v2 2/2] perf c2c: Update documentation for adding memory event table Leo Yan
@ 2025-12-15 8:40 ` James Clark
2026-01-20 20:32 ` Arnaldo Carvalho de Melo
0 siblings, 1 reply; 6+ messages in thread
From: James Clark @ 2025-12-15 8:40 UTC (permalink / raw)
To: Leo Yan
Cc: linux-arm-kernel, linux-perf-users, linux-kernel,
Arnaldo Carvalho de Melo, Namhyung Kim, Ian Rogers, Mike Leach,
Will Deacon, Mark Rutland, Jiri Olsa, Adrian Hunter, Al Grant
On 12/12/2025 19:25, Leo Yan wrote:
> Users may occasionally need to see which options are applied to memory
> events. This helps to understand the behavior of "perf c2c" and
> "perf mem", and provides guidance for configuring memory event options
> directly.
>
> Add a table to track memory events and their corresponding options, and
> include the Arm SPE events in it.
>
> Suggested-by: Al Grant <al.grant@arm.com>
> Signed-off-by: Leo Yan <leo.yan@arm.com>
Reviewed-by: James Clark <james.clark@linaro.org>
> ---
> tools/perf/Documentation/perf-c2c.txt | 51 +++++++++++++++++++++++++----------
> 1 file changed, 37 insertions(+), 14 deletions(-)
>
> diff --git a/tools/perf/Documentation/perf-c2c.txt b/tools/perf/Documentation/perf-c2c.txt
> index 40b0f71a2c44eb642ff3bb234631a614b7c4fc9d..e57a122b8719f1b2e20dd509959fc1563a6c20fd 100644
> --- a/tools/perf/Documentation/perf-c2c.txt
> +++ b/tools/perf/Documentation/perf-c2c.txt
> @@ -160,20 +160,43 @@ Following perf record options are configured by default:
>
> -W,-d,--phys-data,--sample-cpu
>
> -Unless specified otherwise with '-e' option, following events are monitored by
> -default on Intel:
> -
> - cpu/mem-loads,ldlat=30/P
> - cpu/mem-stores/P
> -
> -following on AMD:
> -
> - ibs_op//
> -
> -and following on PowerPC:
> -
> - cpu/mem-loads/
> - cpu/mem-stores/
> +The following table lists the events monitored on different architectures.
> +Unless specified otherwise with the -e option, the tool will select the
> +default events.
> +
> + +--------+---------------+-----------------+--------------------------------------------------------------------------------+
> + | Arch | Configuration | Options | Events |
> + +--------+---------------+-----------------+--------------------------------------------------------------------------------+
> + | Intel | Default | -e ldlat-loads | cpu/mem-loads,ldlat=30/P |
> + | | | -e ldlat-stores | cpu/mem-stores/P |
> + | |---------------+-----------------+--------------------------------------------------------------------------------+
> + | | Load only | -e ldlat-loads | cpu/mem-loads,ldlat=30/P |
> + | |---------------+-----------------+--------------------------------------------------------------------------------+
> + | | Store only | -e ldlat-stores | cpu/mem-stores/P |
> + +--------+---------------+-----------------+--------------------------------------------------------------------------------+
> + | Intel | Default | -e ldlat-loads | {cpu/mem-loads-aux/,cpu/mem-loads,ldlat=30/}:P |
> + | with | | -e ldlat-stores | cpu/mem-stores/P |
> + | AUX |--------------+------------------+--------------------------------------------------------------------------------+
> + | | Load only | -e ldlat-loads | {cpu/mem-loads-aux/,cpu/mem-loads,ldlat=30/}:P |
> + | |---------------+-----------------+--------------------------------------------------------------------------------+
> + | | Store only | -e ldlat-stores | cpu/mem-stores/P |
> + +--------+---------------+-----------------+--------------------------------------------------------------------------------+
> + | AMD | Default | -e mem-ldst | ibs_op// (without latency support) |
> + | | | | ibs_op/ldlat=30/ (with latency support) |
> + +--------+---------------+-----------------+--------------------------------------------------------------------------------+
> + | PowerPC| Default | -e ldlat-loads | cpu/mem-loads/ |
> + | | | -e ldlat-stores | cpu/mem-stores/ |
> + | |---------------+-----------------+--------------------------------------------------------------------------------+
> + | | Load only | -e ldlat-loads | cpu/mem-loads/ |
> + | |---------------+-----------------+--------------------------------------------------------------------------------+
> + | | Store only | -e ldlat-stores | cpu/mem-stores/ |
> + +--------+---------------+-----------------+--------------------------------------------------------------------------------+
> + | Arm | Default | -e spe-ldst | arm_spe_0/ts_enable=1,pa_enable=1,load_filter=1,store_filter=1,min_latency=30/ |
> + | SPE |---------------+-----------------+--------------------------------------------------------------------------------+
> + | | Load only | -e spe-load | arm_spe_0/ts_enable=1,pa_enable=1,load_filter=1,min_latency=30/ |
> + | |---------------+-----------------+--------------------------------------------------------------------------------+
> + | | Store only | -e spe-store | arm_spe_0/ts_enable=1,pa_enable=1,store_filter=1/ |
> + +--------+---------------+-----------------+--------------------------------------------------------------------------------+
>
> User can pass any 'perf record' option behind '--' mark, like (to enable
> callchains and system wide monitoring):
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH v2 2/2] perf c2c: Update documentation for adding memory event table
2025-12-15 8:40 ` James Clark
@ 2026-01-20 20:32 ` Arnaldo Carvalho de Melo
0 siblings, 0 replies; 6+ messages in thread
From: Arnaldo Carvalho de Melo @ 2026-01-20 20:32 UTC (permalink / raw)
To: James Clark
Cc: Leo Yan, linux-arm-kernel, linux-perf-users, linux-kernel,
Namhyung Kim, Ian Rogers, Mike Leach, Will Deacon, Mark Rutland,
Jiri Olsa, Adrian Hunter, Al Grant
On Mon, Dec 15, 2025 at 10:40:04AM +0200, James Clark wrote:
> On 12/12/2025 19:25, Leo Yan wrote:
> > Users may occasionally need to see which options are applied to memory
> > events. This helps to understand the behavior of "perf c2c" and
> > "perf mem", and provides guidance for configuring memory event options
> > directly.
> >
> > Add a table to track memory events and their corresponding options, and
> > include the Arm SPE events in it.
> >
> > Suggested-by: Al Grant <al.grant@arm.com>
> > Signed-off-by: Leo Yan <leo.yan@arm.com>
>
> Reviewed-by: James Clark <james.clark@linaro.org>
Thanks, applied to perf-tools-next,
- Arnaldo
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 0/2] perf arm_spe: Minor event configs housekeeping
2025-12-12 17:24 [PATCH v2 0/2] perf arm_spe: Minor event configs housekeeping Leo Yan
2025-12-12 17:24 ` [PATCH v2 1/2] perf mem: Simplify Arm SPE event config Leo Yan
2025-12-12 17:25 ` [PATCH v2 2/2] perf c2c: Update documentation for adding memory event table Leo Yan
@ 2026-01-16 11:09 ` Leo Yan
2 siblings, 0 replies; 6+ messages in thread
From: Leo Yan @ 2026-01-16 11:09 UTC (permalink / raw)
To: Arnaldo Carvalho de Melo, Namhyung Kim, Ian Rogers, James Clark,
Mike Leach, Will Deacon, Mark Rutland, Jiri Olsa, Adrian Hunter,
Al Grant
Cc: linux-arm-kernel, linux-perf-users, linux-kernel
On Fri, Dec 12, 2025 at 05:24:58PM +0000, Leo Yan wrote:
> This series is to clean up event configs and update the doc for the
> memory events and related options on different archs.
>
> A quick verifying on Orion6 board with commands:
>
> perf c2c record -e spe-load -- test
> perf c2c record -e spe-store -- test
Gentle ping for this short series, thanks!
^ permalink raw reply [flat|nested] 6+ messages in thread