* [PATCH] perf vendor events arm64 AmpereOneX: Fix typo - should be l1d_cache_access_prefetches
@ 2025-09-09 21:01 Ilkka Koskinen
2025-09-10 9:07 ` James Clark
0 siblings, 1 reply; 4+ messages in thread
From: Ilkka Koskinen @ 2025-09-09 21:01 UTC (permalink / raw)
To: John Garry, Will Deacon, James Clark, Mike Leach, Leo Yan,
Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo,
Namhyung Kim, Mark Rutland, Alexander Shishkin, Jiri Olsa,
Ian Rogers, Adrian Hunter, Liang, Kan
Cc: Ilkka Koskinen, linux-arm-kernel, linux-perf-users, linux-kernel
Add missing 'h' to l1d_cache_access_prefetces
Also fix a couple of typos and use consistent term in brief descriptions
Fixes: 16438b652b46 ("perf vendor events arm64 AmpereOneX: Add core PMU events and metrics")
Signed-off-by: Ilkka Koskinen <ilkka@os.amperecomputing.com>
---
.../arch/arm64/ampere/ampereonex/metrics.json | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/tools/perf/pmu-events/arch/arm64/ampere/ampereonex/metrics.json b/tools/perf/pmu-events/arch/arm64/ampere/ampereonex/metrics.json
index 5228f94a793f..6817cac149e0 100644
--- a/tools/perf/pmu-events/arch/arm64/ampere/ampereonex/metrics.json
+++ b/tools/perf/pmu-events/arch/arm64/ampere/ampereonex/metrics.json
@@ -113,7 +113,7 @@
{
"MetricName": "load_store_spec_rate",
"MetricExpr": "LDST_SPEC / INST_SPEC",
- "BriefDescription": "The rate of load or store instructions speculatively executed to overall instructions speclatively executed",
+ "BriefDescription": "The rate of load or store instructions speculatively executed to overall instructions speculatively executed",
"MetricGroup": "Operation_Mix",
"ScaleUnit": "100percent of operations"
},
@@ -132,7 +132,7 @@
{
"MetricName": "pc_write_spec_rate",
"MetricExpr": "PC_WRITE_SPEC / INST_SPEC",
- "BriefDescription": "The rate of software change of the PC speculatively executed to overall instructions speclatively executed",
+ "BriefDescription": "The rate of software change of the PC speculatively executed to overall instructions speculatively executed",
"MetricGroup": "Operation_Mix",
"ScaleUnit": "100percent of operations"
},
@@ -195,14 +195,14 @@
{
"MetricName": "stall_frontend_cache_rate",
"MetricExpr": "STALL_FRONTEND_CACHE / CPU_CYCLES",
- "BriefDescription": "Proportion of cycles stalled and no ops delivered from frontend and cache miss",
+ "BriefDescription": "Proportion of cycles stalled and no operations delivered from frontend and cache miss",
"MetricGroup": "Stall",
"ScaleUnit": "100percent of cycles"
},
{
"MetricName": "stall_frontend_tlb_rate",
"MetricExpr": "STALL_FRONTEND_TLB / CPU_CYCLES",
- "BriefDescription": "Proportion of cycles stalled and no ops delivered from frontend and TLB miss",
+ "BriefDescription": "Proportion of cycles stalled and no operations delivered from frontend and TLB miss",
"MetricGroup": "Stall",
"ScaleUnit": "100percent of cycles"
},
@@ -391,7 +391,7 @@
"ScaleUnit": "100percent of cache acceses"
},
{
- "MetricName": "l1d_cache_access_prefetces",
+ "MetricName": "l1d_cache_access_prefetches",
"MetricExpr": "L1D_CACHE_PRFM / L1D_CACHE",
"BriefDescription": "L1D cache access - prefetch",
"MetricGroup": "Cache",
--
2.51.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] perf vendor events arm64 AmpereOneX: Fix typo - should be l1d_cache_access_prefetches
2025-09-09 21:01 [PATCH] perf vendor events arm64 AmpereOneX: Fix typo - should be l1d_cache_access_prefetches Ilkka Koskinen
@ 2025-09-10 9:07 ` James Clark
2025-09-10 19:46 ` Ilkka Koskinen
0 siblings, 1 reply; 4+ messages in thread
From: James Clark @ 2025-09-10 9:07 UTC (permalink / raw)
To: Ilkka Koskinen
Cc: linux-arm-kernel, linux-perf-users, linux-kernel, John Garry,
Will Deacon, Mike Leach, Leo Yan, Peter Zijlstra, Ingo Molnar,
Arnaldo Carvalho de Melo, Namhyung Kim, Mark Rutland,
Alexander Shishkin, Jiri Olsa, Ian Rogers, Adrian Hunter,
Liang, Kan
On 09/09/2025 10:01 pm, Ilkka Koskinen wrote:
> Add missing 'h' to l1d_cache_access_prefetces
>
> Also fix a couple of typos and use consistent term in brief descriptions
>
> Fixes: 16438b652b46 ("perf vendor events arm64 AmpereOneX: Add core PMU events and metrics")
> Signed-off-by: Ilkka Koskinen <ilkka@os.amperecomputing.com>
> ---
> .../arch/arm64/ampere/ampereonex/metrics.json | 10 +++++-----
The same typos are in arch/arm64/ampere/ampereone/metrics.json as well.
Reviewed-by: James Clark <james.clark@linaro.org>
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/tools/perf/pmu-events/arch/arm64/ampere/ampereonex/metrics.json b/tools/perf/pmu-events/arch/arm64/ampere/ampereonex/metrics.json
> index 5228f94a793f..6817cac149e0 100644
> --- a/tools/perf/pmu-events/arch/arm64/ampere/ampereonex/metrics.json
> +++ b/tools/perf/pmu-events/arch/arm64/ampere/ampereonex/metrics.json
> @@ -113,7 +113,7 @@
> {
> "MetricName": "load_store_spec_rate",
> "MetricExpr": "LDST_SPEC / INST_SPEC",
> - "BriefDescription": "The rate of load or store instructions speculatively executed to overall instructions speclatively executed",
> + "BriefDescription": "The rate of load or store instructions speculatively executed to overall instructions speculatively executed",
> "MetricGroup": "Operation_Mix",
> "ScaleUnit": "100percent of operations"
> },
> @@ -132,7 +132,7 @@
> {
> "MetricName": "pc_write_spec_rate",
> "MetricExpr": "PC_WRITE_SPEC / INST_SPEC",
> - "BriefDescription": "The rate of software change of the PC speculatively executed to overall instructions speclatively executed",
> + "BriefDescription": "The rate of software change of the PC speculatively executed to overall instructions speculatively executed",
> "MetricGroup": "Operation_Mix",
> "ScaleUnit": "100percent of operations"
> },
> @@ -195,14 +195,14 @@
> {
> "MetricName": "stall_frontend_cache_rate",
> "MetricExpr": "STALL_FRONTEND_CACHE / CPU_CYCLES",
> - "BriefDescription": "Proportion of cycles stalled and no ops delivered from frontend and cache miss",
> + "BriefDescription": "Proportion of cycles stalled and no operations delivered from frontend and cache miss",
> "MetricGroup": "Stall",
> "ScaleUnit": "100percent of cycles"
> },
> {
> "MetricName": "stall_frontend_tlb_rate",
> "MetricExpr": "STALL_FRONTEND_TLB / CPU_CYCLES",
> - "BriefDescription": "Proportion of cycles stalled and no ops delivered from frontend and TLB miss",
> + "BriefDescription": "Proportion of cycles stalled and no operations delivered from frontend and TLB miss",
> "MetricGroup": "Stall",
> "ScaleUnit": "100percent of cycles"
> },
> @@ -391,7 +391,7 @@
> "ScaleUnit": "100percent of cache acceses"
> },
> {
> - "MetricName": "l1d_cache_access_prefetces",
> + "MetricName": "l1d_cache_access_prefetches",
> "MetricExpr": "L1D_CACHE_PRFM / L1D_CACHE",
> "BriefDescription": "L1D cache access - prefetch",
> "MetricGroup": "Cache",
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] perf vendor events arm64 AmpereOneX: Fix typo - should be l1d_cache_access_prefetches
2025-09-10 9:07 ` James Clark
@ 2025-09-10 19:46 ` Ilkka Koskinen
2025-09-12 21:31 ` Arnaldo Carvalho de Melo
0 siblings, 1 reply; 4+ messages in thread
From: Ilkka Koskinen @ 2025-09-10 19:46 UTC (permalink / raw)
To: James Clark
Cc: Ilkka Koskinen, linux-arm-kernel, linux-perf-users, linux-kernel,
John Garry, Will Deacon, Mike Leach, Leo Yan, Peter Zijlstra,
Ingo Molnar, Arnaldo Carvalho de Melo, Namhyung Kim, Mark Rutland,
Alexander Shishkin, Jiri Olsa, Ian Rogers, Adrian Hunter,
Liang, Kan
On Wed, 10 Sep 2025, James Clark wrote:
> On 09/09/2025 10:01 pm, Ilkka Koskinen wrote:
>> Add missing 'h' to l1d_cache_access_prefetces
>>
>> Also fix a couple of typos and use consistent term in brief descriptions
>>
>> Fixes: 16438b652b46 ("perf vendor events arm64 AmpereOneX: Add core PMU
>> events and metrics")
>> Signed-off-by: Ilkka Koskinen <ilkka@os.amperecomputing.com>
>> ---
>> .../arch/arm64/ampere/ampereonex/metrics.json | 10 +++++-----
>
> The same typos are in arch/arm64/ampere/ampereone/metrics.json as well.
Ah, that's right. The descriptions do have the same typos. I'll prepare
another patch for AmpereOne and submit them together as v2.
Cheers, Ilkka
>
> Reviewed-by: James Clark <james.clark@linaro.org>
>
>> 1 file changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git
>> a/tools/perf/pmu-events/arch/arm64/ampere/ampereonex/metrics.json
>> b/tools/perf/pmu-events/arch/arm64/ampere/ampereonex/metrics.json
>> index 5228f94a793f..6817cac149e0 100644
>> --- a/tools/perf/pmu-events/arch/arm64/ampere/ampereonex/metrics.json
>> +++ b/tools/perf/pmu-events/arch/arm64/ampere/ampereonex/metrics.json
>> @@ -113,7 +113,7 @@
>> {
>> "MetricName": "load_store_spec_rate",
>> "MetricExpr": "LDST_SPEC / INST_SPEC",
>> - "BriefDescription": "The rate of load or store instructions
>> speculatively executed to overall instructions speclatively executed",
>> + "BriefDescription": "The rate of load or store instructions
>> speculatively executed to overall instructions speculatively executed",
>> "MetricGroup": "Operation_Mix",
>> "ScaleUnit": "100percent of operations"
>> },
>> @@ -132,7 +132,7 @@
>> {
>> "MetricName": "pc_write_spec_rate",
>> "MetricExpr": "PC_WRITE_SPEC / INST_SPEC",
>> - "BriefDescription": "The rate of software change of the PC
>> speculatively executed to overall instructions speclatively executed",
>> + "BriefDescription": "The rate of software change of the PC
>> speculatively executed to overall instructions speculatively executed",
>> "MetricGroup": "Operation_Mix",
>> "ScaleUnit": "100percent of operations"
>> },
>> @@ -195,14 +195,14 @@
>> {
>> "MetricName": "stall_frontend_cache_rate",
>> "MetricExpr": "STALL_FRONTEND_CACHE / CPU_CYCLES",
>> - "BriefDescription": "Proportion of cycles stalled and no ops
>> delivered from frontend and cache miss",
>> + "BriefDescription": "Proportion of cycles stalled and no
>> operations delivered from frontend and cache miss",
>> "MetricGroup": "Stall",
>> "ScaleUnit": "100percent of cycles"
>> },
>> {
>> "MetricName": "stall_frontend_tlb_rate",
>> "MetricExpr": "STALL_FRONTEND_TLB / CPU_CYCLES",
>> - "BriefDescription": "Proportion of cycles stalled and no ops
>> delivered from frontend and TLB miss",
>> + "BriefDescription": "Proportion of cycles stalled and no
>> operations delivered from frontend and TLB miss",
>> "MetricGroup": "Stall",
>> "ScaleUnit": "100percent of cycles"
>> },
>> @@ -391,7 +391,7 @@
>> "ScaleUnit": "100percent of cache acceses"
>> },
>> {
>> - "MetricName": "l1d_cache_access_prefetces",
>> + "MetricName": "l1d_cache_access_prefetches",
>> "MetricExpr": "L1D_CACHE_PRFM / L1D_CACHE",
>> "BriefDescription": "L1D cache access - prefetch",
>> "MetricGroup": "Cache",
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] perf vendor events arm64 AmpereOneX: Fix typo - should be l1d_cache_access_prefetches
2025-09-10 19:46 ` Ilkka Koskinen
@ 2025-09-12 21:31 ` Arnaldo Carvalho de Melo
0 siblings, 0 replies; 4+ messages in thread
From: Arnaldo Carvalho de Melo @ 2025-09-12 21:31 UTC (permalink / raw)
To: Ilkka Koskinen
Cc: James Clark, linux-arm-kernel, linux-perf-users, linux-kernel,
John Garry, Will Deacon, Mike Leach, Leo Yan, Peter Zijlstra,
Ingo Molnar, Namhyung Kim, Mark Rutland, Alexander Shishkin,
Jiri Olsa, Ian Rogers, Adrian Hunter, Liang, Kan
On Wed, Sep 10, 2025 at 12:46:24PM -0700, Ilkka Koskinen wrote:
>
>
> On Wed, 10 Sep 2025, James Clark wrote:
> > On 09/09/2025 10:01 pm, Ilkka Koskinen wrote:
> > > Add missing 'h' to l1d_cache_access_prefetces
> > >
> > > Also fix a couple of typos and use consistent term in brief descriptions
> > >
> > > Fixes: 16438b652b46 ("perf vendor events arm64 AmpereOneX: Add core
> > > PMU events and metrics")
> > > Signed-off-by: Ilkka Koskinen <ilkka@os.amperecomputing.com>
> > > ---
> > > .../arch/arm64/ampere/ampereonex/metrics.json | 10 +++++-----
> >
> > The same typos are in arch/arm64/ampere/ampereone/metrics.json as well.
>
> Ah, that's right. The descriptions do have the same typos. I'll prepare
> another patch for AmpereOne and submit them together as v2.
>
> Cheers, Ilkka
Thanks, applied to perf-tools-next,
- Arnaldo
> >
> > Reviewed-by: James Clark <james.clark@linaro.org>
> >
> > > 1 file changed, 5 insertions(+), 5 deletions(-)
> > >
> > > diff --git
> > > a/tools/perf/pmu-events/arch/arm64/ampere/ampereonex/metrics.json
> > > b/tools/perf/pmu-events/arch/arm64/ampere/ampereonex/metrics.json
> > > index 5228f94a793f..6817cac149e0 100644
> > > --- a/tools/perf/pmu-events/arch/arm64/ampere/ampereonex/metrics.json
> > > +++ b/tools/perf/pmu-events/arch/arm64/ampere/ampereonex/metrics.json
> > > @@ -113,7 +113,7 @@
> > > {
> > > "MetricName": "load_store_spec_rate",
> > > "MetricExpr": "LDST_SPEC / INST_SPEC",
> > > - "BriefDescription": "The rate of load or store instructions
> > > speculatively executed to overall instructions speclatively
> > > executed",
> > > + "BriefDescription": "The rate of load or store instructions
> > > speculatively executed to overall instructions speculatively
> > > executed",
> > > "MetricGroup": "Operation_Mix",
> > > "ScaleUnit": "100percent of operations"
> > > },
> > > @@ -132,7 +132,7 @@
> > > {
> > > "MetricName": "pc_write_spec_rate",
> > > "MetricExpr": "PC_WRITE_SPEC / INST_SPEC",
> > > - "BriefDescription": "The rate of software change of the PC
> > > speculatively executed to overall instructions speclatively
> > > executed",
> > > + "BriefDescription": "The rate of software change of the PC
> > > speculatively executed to overall instructions speculatively
> > > executed",
> > > "MetricGroup": "Operation_Mix",
> > > "ScaleUnit": "100percent of operations"
> > > },
> > > @@ -195,14 +195,14 @@
> > > {
> > > "MetricName": "stall_frontend_cache_rate",
> > > "MetricExpr": "STALL_FRONTEND_CACHE / CPU_CYCLES",
> > > - "BriefDescription": "Proportion of cycles stalled and no
> > > ops delivered from frontend and cache miss",
> > > + "BriefDescription": "Proportion of cycles stalled and no
> > > operations delivered from frontend and cache miss",
> > > "MetricGroup": "Stall",
> > > "ScaleUnit": "100percent of cycles"
> > > },
> > > {
> > > "MetricName": "stall_frontend_tlb_rate",
> > > "MetricExpr": "STALL_FRONTEND_TLB / CPU_CYCLES",
> > > - "BriefDescription": "Proportion of cycles stalled and no
> > > ops delivered from frontend and TLB miss",
> > > + "BriefDescription": "Proportion of cycles stalled and no
> > > operations delivered from frontend and TLB miss",
> > > "MetricGroup": "Stall",
> > > "ScaleUnit": "100percent of cycles"
> > > },
> > > @@ -391,7 +391,7 @@
> > > "ScaleUnit": "100percent of cache acceses"
> > > },
> > > {
> > > - "MetricName": "l1d_cache_access_prefetces",
> > > + "MetricName": "l1d_cache_access_prefetches",
> > > "MetricExpr": "L1D_CACHE_PRFM / L1D_CACHE",
> > > "BriefDescription": "L1D cache access - prefetch",
> > > "MetricGroup": "Cache",
> >
> >
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-09-12 21:31 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-09-09 21:01 [PATCH] perf vendor events arm64 AmpereOneX: Fix typo - should be l1d_cache_access_prefetches Ilkka Koskinen
2025-09-10 9:07 ` James Clark
2025-09-10 19:46 ` Ilkka Koskinen
2025-09-12 21:31 ` Arnaldo Carvalho de Melo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).