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

Ian Rogers
2023-12-04T19:03:26Z

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
>