Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <501e5a2f-4e5a-1d32-b933-5b74e9775aa5@linux.intel.com>
Date:   Thu, 9 Jun 2022 09:07:17 -0400
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.9.1
Subject: Re: [PATCH v2 1/2] perf vendor events intel: Rename tremontx to
 snowridgex
Content-Language: en-US
To:     zhengjun.xing@linux.intel.com, acme@kernel.org,
        peterz@infradead.org, mingo@redhat.com,
        alexander.shishkin@intel.com, jolsa@redhat.com
Cc:     linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org,
        irogers@google.com, adrian.hunter@intel.com, ak@linux.intel.com
References: <20220609094222.2030167-1-zhengjun.xing@linux.intel.com>
From:   "Liang, Kan" <kan.liang@linux.intel.com>
In-Reply-To: <20220609094222.2030167-1-zhengjun.xing@linux.intel.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 6/9/2022 5:42 AM, zhengjun.xing@linux.intel.com wrote:
> From: Zhengjun Xing <zhengjun.xing@linux.intel.com>
> 
> Tremontx was an old name for Snowridgex, so rename Tremontx to Snowridgex.
> 
> Signed-off-by: Zhengjun Xing <zhengjun.xing@linux.intel.com>

The patch series look good to me.

Reviewed-by: Kan Liang <kan.liang@linux.intel.com>

Thanks,
Kan

> ---
> Change log:
>    v2:
>      * no change since v1.
> 
>   tools/perf/pmu-events/arch/x86/mapfile.csv                      | 2 +-
>   .../pmu-events/arch/x86/{tremontx => snowridgex}/cache.json     | 0
>   .../arch/x86/{tremontx => snowridgex}/floating-point.json       | 0
>   .../pmu-events/arch/x86/{tremontx => snowridgex}/frontend.json  | 0
>   .../pmu-events/arch/x86/{tremontx => snowridgex}/memory.json    | 0
>   .../pmu-events/arch/x86/{tremontx => snowridgex}/other.json     | 0
>   .../pmu-events/arch/x86/{tremontx => snowridgex}/pipeline.json  | 0
>   .../arch/x86/{tremontx => snowridgex}/uncore-memory.json        | 0
>   .../arch/x86/{tremontx => snowridgex}/uncore-other.json         | 0
>   .../arch/x86/{tremontx => snowridgex}/uncore-power.json         | 0
>   .../arch/x86/{tremontx => snowridgex}/virtual-memory.json       | 0
>   11 files changed, 1 insertion(+), 1 deletion(-)
>   rename tools/perf/pmu-events/arch/x86/{tremontx => snowridgex}/cache.json (100%)
>   rename tools/perf/pmu-events/arch/x86/{tremontx => snowridgex}/floating-point.json (100%)
>   rename tools/perf/pmu-events/arch/x86/{tremontx => snowridgex}/frontend.json (100%)
>   rename tools/perf/pmu-events/arch/x86/{tremontx => snowridgex}/memory.json (100%)
>   rename tools/perf/pmu-events/arch/x86/{tremontx => snowridgex}/other.json (100%)
>   rename tools/perf/pmu-events/arch/x86/{tremontx => snowridgex}/pipeline.json (100%)
>   rename tools/perf/pmu-events/arch/x86/{tremontx => snowridgex}/uncore-memory.json (100%)
>   rename tools/perf/pmu-events/arch/x86/{tremontx => snowridgex}/uncore-other.json (100%)
>   rename tools/perf/pmu-events/arch/x86/{tremontx => snowridgex}/uncore-power.json (100%)
>   rename tools/perf/pmu-events/arch/x86/{tremontx => snowridgex}/virtual-memory.json (100%)
> 
> diff --git a/tools/perf/pmu-events/arch/x86/mapfile.csv b/tools/perf/pmu-events/arch/x86/mapfile.csv
> index f5a382421a60..9852bca98d59 100644
> --- a/tools/perf/pmu-events/arch/x86/mapfile.csv
> +++ b/tools/perf/pmu-events/arch/x86/mapfile.csv
> @@ -40,7 +40,7 @@ GenuineIntel-6-8[CD],v1,tigerlake,core
>   GenuineIntel-6-A7,v1,icelake,core
>   GenuineIntel-6-6A,v1,icelakex,core
>   GenuineIntel-6-6C,v1,icelakex,core
> -GenuineIntel-6-86,v1,tremontx,core
> +GenuineIntel-6-86,v1,snowridgex,core
>   GenuineIntel-6-96,v1,elkhartlake,core
>   GenuineIntel-6-97,v1,alderlake,core
>   GenuineIntel-6-9A,v1,alderlake,core
> diff --git a/tools/perf/pmu-events/arch/x86/tremontx/cache.json b/tools/perf/pmu-events/arch/x86/snowridgex/cache.json
> similarity index 100%
> rename from tools/perf/pmu-events/arch/x86/tremontx/cache.json
> rename to tools/perf/pmu-events/arch/x86/snowridgex/cache.json
> diff --git a/tools/perf/pmu-events/arch/x86/tremontx/floating-point.json b/tools/perf/pmu-events/arch/x86/snowridgex/floating-point.json
> similarity index 100%
> rename from tools/perf/pmu-events/arch/x86/tremontx/floating-point.json
> rename to tools/perf/pmu-events/arch/x86/snowridgex/floating-point.json
> diff --git a/tools/perf/pmu-events/arch/x86/tremontx/frontend.json b/tools/perf/pmu-events/arch/x86/snowridgex/frontend.json
> similarity index 100%
> rename from tools/perf/pmu-events/arch/x86/tremontx/frontend.json
> rename to tools/perf/pmu-events/arch/x86/snowridgex/frontend.json
> diff --git a/tools/perf/pmu-events/arch/x86/tremontx/memory.json b/tools/perf/pmu-events/arch/x86/snowridgex/memory.json
> similarity index 100%
> rename from tools/perf/pmu-events/arch/x86/tremontx/memory.json
> rename to tools/perf/pmu-events/arch/x86/snowridgex/memory.json
> diff --git a/tools/perf/pmu-events/arch/x86/tremontx/other.json b/tools/perf/pmu-events/arch/x86/snowridgex/other.json
> similarity index 100%
> rename from tools/perf/pmu-events/arch/x86/tremontx/other.json
> rename to tools/perf/pmu-events/arch/x86/snowridgex/other.json
> diff --git a/tools/perf/pmu-events/arch/x86/tremontx/pipeline.json b/tools/perf/pmu-events/arch/x86/snowridgex/pipeline.json
> similarity index 100%
> rename from tools/perf/pmu-events/arch/x86/tremontx/pipeline.json
> rename to tools/perf/pmu-events/arch/x86/snowridgex/pipeline.json
> diff --git a/tools/perf/pmu-events/arch/x86/tremontx/uncore-memory.json b/tools/perf/pmu-events/arch/x86/snowridgex/uncore-memory.json
> similarity index 100%
> rename from tools/perf/pmu-events/arch/x86/tremontx/uncore-memory.json
> rename to tools/perf/pmu-events/arch/x86/snowridgex/uncore-memory.json
> diff --git a/tools/perf/pmu-events/arch/x86/tremontx/uncore-other.json b/tools/perf/pmu-events/arch/x86/snowridgex/uncore-other.json
> similarity index 100%
> rename from tools/perf/pmu-events/arch/x86/tremontx/uncore-other.json
> rename to tools/perf/pmu-events/arch/x86/snowridgex/uncore-other.json
> diff --git a/tools/perf/pmu-events/arch/x86/tremontx/uncore-power.json b/tools/perf/pmu-events/arch/x86/snowridgex/uncore-power.json
> similarity index 100%
> rename from tools/perf/pmu-events/arch/x86/tremontx/uncore-power.json
> rename to tools/perf/pmu-events/arch/x86/snowridgex/uncore-power.json
> diff --git a/tools/perf/pmu-events/arch/x86/tremontx/virtual-memory.json b/tools/perf/pmu-events/arch/x86/snowridgex/virtual-memory.json
> similarity index 100%
> rename from tools/perf/pmu-events/arch/x86/tremontx/virtual-memory.json
> rename to tools/perf/pmu-events/arch/x86/snowridgex/virtual-memory.json