Re: [PATCH] perf vendor events arm64: Update FUJITSU-MONAKA pmu event

[James Clark]

On 18/06/2025 7:35 am, Kotaro, Tokai wrote:
> Update pmu events for FUJITSU-MONAKA.
> And, also updated common-and-microarch.json.
>
> FUJITSU-MONAKA PMU Events Specification v1.1 and Errata v1.0 URL:
> https://github.com/fujitsu/FUJITSU-MONAKA
>
> Arm Architecture Reference Version L.b URL:
> https://developer.arm.com/documentation/ddi0487/lb/?lang=en
>
> Signed-off-by: Kotaro, Tokai <fj0635gf@xxxxxxxxxxxxxxxxx>
> ---
>   .../arch/arm64/common-and-microarch.json      | 70 +++++++++++++
>   .../arm64/fujitsu/monaka/core-imp-def.json    |  2 +-
>   .../fujitsu/monaka/cycle_accounting.json      |  4 +-
>   .../arch/arm64/fujitsu/monaka/exception.json  |  2 +-
>   .../arm64/fujitsu/monaka/fp_operation.json    | 98 +++++++++++++++----
>   .../arch/arm64/fujitsu/monaka/l1d_cache.json  | 10 +-
>   .../arch/arm64/fujitsu/monaka/l1i_cache.json  |  8 +-
>   .../arch/arm64/fujitsu/monaka/l2_cache.json   | 28 +++---
>   .../arch/arm64/fujitsu/monaka/l3_cache.json   | 63 ++++++------
>   .../arch/arm64/fujitsu/monaka/ll_cache.json   |  2 +-
>   .../arch/arm64/fujitsu/monaka/pipeline.json   |  6 +-
>   .../arm64/fujitsu/monaka/spec_operation.json  | 12 +--
>   .../arch/arm64/fujitsu/monaka/stall.json      |  4 +-
>   .../arch/arm64/fujitsu/monaka/sve.json        | 44 ++++-----
>   .../arch/arm64/fujitsu/monaka/tlb.json        | 56 +++++------
>   15 files changed, 265 insertions(+), 144 deletions(-)
[...]
> diff --git a/tools/perf/pmu-events/arch/arm64/fujitsu/monaka/fp_operation.json b/tools/perf/pmu-events/arch/arm64/fujitsu/monaka/fp_operation.json
> index a3c368959199..2ffdc16530dd 100644
> --- a/tools/perf/pmu-events/arch/arm64/fujitsu/monaka/fp_operation.json
> +++ b/tools/perf/pmu-events/arch/arm64/fujitsu/monaka/fp_operation.json
> @@ -2,7 +2,7 @@
>       {
>           "EventCode": "0x0105",
>           "EventName": "FP_MV_SPEC",
> -        "BriefDescription": "This event counts architecturally executed floating-point move operations."
> +        "BriefDescription": "This event counts architecturally executed floating-point move operation."

The original was more grammatically correct, but the updates are just 
from the pdf so:

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