[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <a606211b5b454ef5b658858e0cb361f0@intel.com>
Date: Thu, 11 Mar 2021 21:09:57 +0000
From: "Luck, Tony" <tony.luck@...el.com>
To: Peter Zijlstra <peterz@...radead.org>,
"Liang, Kan" <kan.liang@...ux.intel.com>
CC: Andi Kleen <ak@...ux.intel.com>,
"mingo@...nel.org" <mingo@...nel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"acme@...nel.org" <acme@...nel.org>,
"tglx@...utronix.de" <tglx@...utronix.de>,
"bp@...en8.de" <bp@...en8.de>,
"namhyung@...nel.org" <namhyung@...nel.org>,
"jolsa@...hat.com" <jolsa@...hat.com>,
"yao.jin@...ux.intel.com" <yao.jin@...ux.intel.com>,
"alexander.shishkin@...ux.intel.com"
<alexander.shishkin@...ux.intel.com>,
"Hunter, Adrian" <adrian.hunter@...el.com>,
Mark Rutland <mark.rutland@....com>,
Ricardo Neri <ricardo.neri-calderon@...ux.intel.com>
Subject: RE: [PATCH V2 20/25] perf/x86/intel: Add Alder Lake Hybrid support
>> I think the "sapphire_rapids" is the code name for the server platform.
>
> If that's really the case, then:
>
> #define INTEL_FAM6_SAPPHIRERAPIDS_X 0x8F
>
> is wrong, those things should be uarch name, not platform name. Tony?
0x8F is the model number of the CPU that is named Sapphire Rapids that
goes into the Eagle Stream platform
If you want a uarch name, that might be the name of the core
(.... something-cove ... I can't keep track of the names of all the
coves). But that would likely lead to different confusion later if that
*cove core is used in some other CPU model number that doesn't
neatly fit into our _X, _L etc. suffix scheme)
-Tony
Powered by blists - more mailing lists