[2/3] perf vendor events arm64: AmpereOne: Rename BPU_FLUSH_MEM_FAULT to GPC_FLUSH_MEM_FAULT

Message ID 20231201021550.1109196-3-ilkka@os.amperecomputing.com
State New
Headers
Series perf vendor events arm64: Add AmpereOneX events and metrics and fix AmpereOne bugs |

Commit Message

Ilkka Koskinen Dec. 1, 2023, 2:15 a.m. UTC
  The documentation wrongly called the event as BPU_FLUSH_MEM_FAULT and now
has been fixed. Correct the name in the perf tool as well.

Fixes: a9650b7f6fc0 ("perf vendor events arm64: Add AmpereOne core PMU events")
Signed-off-by: Ilkka Koskinen <ilkka@os.amperecomputing.com>
---
 .../pmu-events/arch/arm64/ampere/ampereone/core-imp-def.json    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Ian Rogers Dec. 4, 2023, 7:03 p.m. UTC | #1
On Thu, Nov 30, 2023 at 6:16 PM Ilkka Koskinen
<ilkka@os.amperecomputing.com> wrote:
>
> The documentation wrongly called the event as BPU_FLUSH_MEM_FAULT and now
> has been fixed. Correct the name in the perf tool as well.
>
> Fixes: a9650b7f6fc0 ("perf vendor events arm64: Add AmpereOne core PMU events")
> Signed-off-by: Ilkka Koskinen <ilkka@os.amperecomputing.com>

Reviewed-by: Ian Rogers <irogers@google.com>

Thanks,
Ian

> ---
>  .../pmu-events/arch/arm64/ampere/ampereone/core-imp-def.json    | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/perf/pmu-events/arch/arm64/ampere/ampereone/core-imp-def.json b/tools/perf/pmu-events/arch/arm64/ampere/ampereone/core-imp-def.json
> index 88b23b85e33c..879ff21e0b17 100644
> --- a/tools/perf/pmu-events/arch/arm64/ampere/ampereone/core-imp-def.json
> +++ b/tools/perf/pmu-events/arch/arm64/ampere/ampereone/core-imp-def.json
> @@ -110,7 +110,7 @@
>      {
>          "PublicDescription": "Flushes due to memory hazards",
>          "EventCode": "0x121",
> -        "EventName": "BPU_FLUSH_MEM_FAULT",
> +        "EventName": "GPC_FLUSH_MEM_FAULT",
>          "BriefDescription": "Flushes due to memory hazards"
>      },
>      {
> --
> 2.41.0
>
  

Patch

diff --git a/tools/perf/pmu-events/arch/arm64/ampere/ampereone/core-imp-def.json b/tools/perf/pmu-events/arch/arm64/ampere/ampereone/core-imp-def.json
index 88b23b85e33c..879ff21e0b17 100644
--- a/tools/perf/pmu-events/arch/arm64/ampere/ampereone/core-imp-def.json
+++ b/tools/perf/pmu-events/arch/arm64/ampere/ampereone/core-imp-def.json
@@ -110,7 +110,7 @@ 
     {
         "PublicDescription": "Flushes due to memory hazards",
         "EventCode": "0x121",
-        "EventName": "BPU_FLUSH_MEM_FAULT",
+        "EventName": "GPC_FLUSH_MEM_FAULT",
         "BriefDescription": "Flushes due to memory hazards"
     },
     {