Received-SPF: pass (google.com: domain of linux-kernel+bounces-8183-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <7a2623b2-2c56-43db-b19b-3e9488b4be6d@linux.ibm.com>
Date: Thu, 21 Dec 2023 15:41:31 +0530
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH V0 RESEND] tools/perf: Add PVN for HX-C2000 CPU with
 Power8 Architecture
To: "JiaLong.Yang" <jialong.yang@shingroup.cn>,
        Peter Zijlstra <peterz@infradead.org>, Ingo Molnar <mingo@redhat.com>,
        Arnaldo Carvalho de Melo <acme@kernel.org>,
        Mark Rutland <mark.rutland@arm.com>,
        Alexander Shishkin <alexander.shishkin@linux.intel.com>,
        Jiri Olsa <jolsa@kernel.org>, Namhyung Kim <namhyung@kernel.org>,
        Ian Rogers <irogers@google.com>,
        Adrian Hunter <adrian.hunter@intel.com>
Cc: shenghui.qu@shingroup.cn, ke.zhao@shingroup.cn, zhijie.ren@shingroup.cn,
        linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20231221060242.4532-1-jialong.yang@shingroup.cn>
Content-Language: en-US
From: Madhavan Srinivasan <maddy@linux.ibm.com>
In-Reply-To: <20231221060242.4532-1-jialong.yang@shingroup.cn>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0


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@shingroup.cn>

Patch seems to be fine to me.

Reviewed-by: Madhavan Srinivasanan <maddy@linux.ibm.com>

> ---
> Here tells detail info.
> Link: https://lore.kernel.org/all/20231129075845.57976-1-ke.zhao@shingroup.cn/
>
>   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