2023-12-21 06:05:23

by Yang Jialong 杨佳龙

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

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 <[email protected]>
---
Here tells detail info.
Link: https://lore.kernel.org/all/[email protected]/

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
--
2.25.1

2023-12-21 10:12:19

by Madhavan Srinivasan

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

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 <[email protected]>

Patch seems to be fine to me.

Reviewed-by: Madhavan Srinivasanan <[email protected]>

> ---
> Here tells detail info.
> Link: https://lore.kernel.org/all/[email protected]/
>
> 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