linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] tools: Fix typo error in json file
@ 2025-10-31  3:17 Chu Guangqing
  2025-10-31 10:00 ` James Clark
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Chu Guangqing @ 2025-10-31  3:17 UTC (permalink / raw)
  To: john.g.garry, will, james.clark, mike.leach, leo.yan, peterz,
	mingo, acme, namhyung, mark.rutland, alexander.shishkin, jolsa,
	irogers, adrian.hunter, linux-arm-kernel, linux-perf-users,
	linux-kernel
  Cc: Chu Guangqing

Correct instruction spelling errors.

Signed-off-by: Chu Guangqing <chuguangqing@inspur.com>
---
 tools/perf/pmu-events/arch/arm64/ampere/emag/cache.json | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/perf/pmu-events/arch/arm64/ampere/emag/cache.json b/tools/perf/pmu-events/arch/arm64/ampere/emag/cache.json
index 4cc50b7da526..4001cc5753a7 100644
--- a/tools/perf/pmu-events/arch/arm64/ampere/emag/cache.json
+++ b/tools/perf/pmu-events/arch/arm64/ampere/emag/cache.json
@@ -81,7 +81,7 @@
         "BriefDescription": "L2D TLB access"
     },
     {
-        "PublicDescription": "Level 2 access to instruciton TLB that caused a page table walk. This event counts on any instruciton access which causes L2I_TLB_REFILL to count",
+        "PublicDescription": "Level 2 access to instruction TLB that caused a page table walk. This event counts on any instruction access which causes L2I_TLB_REFILL to count",
         "EventCode": "0x35",
         "EventName": "L2I_TLB_ACCESS",
         "BriefDescription": "L2I TLB access"
-- 
2.43.7



^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] tools: Fix typo error in json file
  2025-10-31  3:17 [PATCH] tools: Fix typo error in json file Chu Guangqing
@ 2025-10-31 10:00 ` James Clark
  2025-10-31 19:25 ` Ilkka Koskinen
  2025-11-03 17:53 ` Namhyung Kim
  2 siblings, 0 replies; 5+ messages in thread
From: James Clark @ 2025-10-31 10:00 UTC (permalink / raw)
  To: Chu Guangqing
  Cc: john.g.garry, will, mike.leach, leo.yan, peterz, mingo, acme,
	namhyung, mark.rutland, alexander.shishkin, jolsa, irogers,
	adrian.hunter, linux-arm-kernel, linux-perf-users, linux-kernel



On 31/10/2025 3:17 am, Chu Guangqing wrote:
> Correct instruction spelling errors.
> 
> Signed-off-by: Chu Guangqing <chuguangqing@inspur.com>
> ---
>   tools/perf/pmu-events/arch/arm64/ampere/emag/cache.json | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/perf/pmu-events/arch/arm64/ampere/emag/cache.json b/tools/perf/pmu-events/arch/arm64/ampere/emag/cache.json
> index 4cc50b7da526..4001cc5753a7 100644
> --- a/tools/perf/pmu-events/arch/arm64/ampere/emag/cache.json
> +++ b/tools/perf/pmu-events/arch/arm64/ampere/emag/cache.json
> @@ -81,7 +81,7 @@
>           "BriefDescription": "L2D TLB access"
>       },
>       {
> -        "PublicDescription": "Level 2 access to instruciton TLB that caused a page table walk. This event counts on any instruciton access which causes L2I_TLB_REFILL to count",
> +        "PublicDescription": "Level 2 access to instruction TLB that caused a page table walk. This event counts on any instruction access which causes L2I_TLB_REFILL to count",
>           "EventCode": "0x35",
>           "EventName": "L2I_TLB_ACCESS",
>           "BriefDescription": "L2I TLB access"

Reviewed-by: James Clark <james.clark@linaro.org>



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] tools: Fix typo error in json file
  2025-10-31  3:17 [PATCH] tools: Fix typo error in json file Chu Guangqing
  2025-10-31 10:00 ` James Clark
@ 2025-10-31 19:25 ` Ilkka Koskinen
  2025-10-31 19:30   ` Namhyung Kim
  2025-11-03 17:53 ` Namhyung Kim
  2 siblings, 1 reply; 5+ messages in thread
From: Ilkka Koskinen @ 2025-10-31 19:25 UTC (permalink / raw)
  To: Chu Guangqing
  Cc: john.g.garry, will, james.clark, mike.leach, leo.yan, peterz,
	mingo, acme, namhyung, mark.rutland, alexander.shishkin, jolsa,
	irogers, adrian.hunter, linux-arm-kernel, linux-perf-users,
	linux-kernel


On Fri, 31 Oct 2025, Chu Guangqing wrote:
> Correct instruction spelling errors.
>
> Signed-off-by: Chu Guangqing <chuguangqing@inspur.com>


Thanks for the patch. Just one comment, would you mind changing the 
subject line slightly. Something like

"perf vendor events arm64: Fix typo in Ampere eMag json file"


With that change

Reviewed-by: Ilkka Koskinen <ilkka@os.amperecomputing.com>

Cheers, Ilkka


> ---
> tools/perf/pmu-events/arch/arm64/ampere/emag/cache.json | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/perf/pmu-events/arch/arm64/ampere/emag/cache.json b/tools/perf/pmu-events/arch/arm64/ampere/emag/cache.json
> index 4cc50b7da526..4001cc5753a7 100644
> --- a/tools/perf/pmu-events/arch/arm64/ampere/emag/cache.json
> +++ b/tools/perf/pmu-events/arch/arm64/ampere/emag/cache.json
> @@ -81,7 +81,7 @@
>         "BriefDescription": "L2D TLB access"
>     },
>     {
> -        "PublicDescription": "Level 2 access to instruciton TLB that caused a page table walk. This event counts on any instruciton access which causes L2I_TLB_REFILL to count",
> +        "PublicDescription": "Level 2 access to instruction TLB that caused a page table walk. This event counts on any instruction access which causes L2I_TLB_REFILL to count",
>         "EventCode": "0x35",
>         "EventName": "L2I_TLB_ACCESS",
>         "BriefDescription": "L2I TLB access"
> -- 
> 2.43.7
>
>
>


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] tools: Fix typo error in json file
  2025-10-31 19:25 ` Ilkka Koskinen
@ 2025-10-31 19:30   ` Namhyung Kim
  0 siblings, 0 replies; 5+ messages in thread
From: Namhyung Kim @ 2025-10-31 19:30 UTC (permalink / raw)
  To: Ilkka Koskinen
  Cc: Chu Guangqing, john.g.garry, will, james.clark, mike.leach,
	leo.yan, peterz, mingo, acme, mark.rutland, alexander.shishkin,
	jolsa, irogers, adrian.hunter, linux-arm-kernel, linux-perf-users,
	linux-kernel

Hello,

On Fri, Oct 31, 2025 at 12:25:37PM -0700, Ilkka Koskinen wrote:
> 
> On Fri, 31 Oct 2025, Chu Guangqing wrote:
> > Correct instruction spelling errors.
> > 
> > Signed-off-by: Chu Guangqing <chuguangqing@inspur.com>
> 
> 
> Thanks for the patch. Just one comment, would you mind changing the subject
> line slightly. Something like
> 
> "perf vendor events arm64: Fix typo in Ampere eMag json file"
> 
> 
> With that change
> 
> Reviewed-by: Ilkka Koskinen <ilkka@os.amperecomputing.com>

Thanks, I'll handle the both changes.
Namhyung



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] tools: Fix typo error in json file
  2025-10-31  3:17 [PATCH] tools: Fix typo error in json file Chu Guangqing
  2025-10-31 10:00 ` James Clark
  2025-10-31 19:25 ` Ilkka Koskinen
@ 2025-11-03 17:53 ` Namhyung Kim
  2 siblings, 0 replies; 5+ messages in thread
From: Namhyung Kim @ 2025-11-03 17:53 UTC (permalink / raw)
  To: john.g.garry, will, james.clark, mike.leach, leo.yan, peterz,
	mingo, acme, mark.rutland, alexander.shishkin, jolsa, irogers,
	adrian.hunter, linux-arm-kernel, linux-perf-users, linux-kernel,
	Chu Guangqing

On Fri, 31 Oct 2025 11:17:29 +0800, Chu Guangqing wrote:
> Correct instruction spelling errors.
> 
> 
Applied to perf-tools-next, thanks!

Best regards,
Namhyung




^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2025-11-03 17:53 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-10-31  3:17 [PATCH] tools: Fix typo error in json file Chu Guangqing
2025-10-31 10:00 ` James Clark
2025-10-31 19:25 ` Ilkka Koskinen
2025-10-31 19:30   ` Namhyung Kim
2025-11-03 17:53 ` Namhyung Kim

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).