Re: [PATCH V0 RESEND] tools/perf: Add PVN for HX-C2000 CPU with Power8 Architecture

[Madhavan Srinivasan]

On 12/21/23 11:32 AM, JiaLong.Yang wrote:
> HX-C2000 is a new CPU made by HEXIN Technologies Co., Ltd. And a new
> PVN 0x0066 has been applied from the OpenPower Community for this CPU. Here is
> a patch to make perf tool run in the CPU.
>
> Signed-off-by: JiaLong.Yang <jialong.yang@xxxxxxxxxxxx>

Patch seems to be fine to me.

Reviewed-by: Madhavan Srinivasanan <maddy@xxxxxxxxxxxxx>

> ---
> Here tells detail info.
> Link: https://lore.kernel.org/all/20231129075845.57976-1-ke.zhao@xxxxxxxxxxxx/
>
>   tools/perf/pmu-events/arch/powerpc/mapfile.csv | 3 +--
>   1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/tools/perf/pmu-events/arch/powerpc/mapfile.csv b/tools/perf/pmu-events/arch/powerpc/mapfile.csv
> index f4908af7ad66..599a588dbeb4 100644
> --- a/tools/perf/pmu-events/arch/powerpc/mapfile.csv
> +++ b/tools/perf/pmu-events/arch/powerpc/mapfile.csv
> @@ -11,8 +11,7 @@
>   #
>   # Multiple PVRs could map to a single JSON file.
>   #
> -
> -# Power8 entries
>   0x004[bcd][[:xdigit:]]{4},1,power8,core
> +0x0066[[:xdigit:]]{4},1,power8,core
>   0x004e[[:xdigit:]]{4},1,power9,core
>   0x0080[[:xdigit:]]{4},1,power10,core