[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <DS7PR11MB6077065ABA534F0EC39B84ACFC63A@DS7PR11MB6077.namprd11.prod.outlook.com>
Date: Wed, 11 Feb 2026 22:32:50 +0000
From: "Luck, Tony" <tony.luck@...el.com>
To: "Mehta, Sohil" <sohil.mehta@...el.com>, "Hansen, Dave"
<dave.hansen@...el.com>
CC: Dave Hansen <dave.hansen@...ux.intel.com>, "Liu, Zhao1"
<zhao1.liu@...el.com>, Borislav Petkov <bp@...en8.de>, "H. Peter Anvin"
<hpa@...or.com>, Ingo Molnar <mingo@...hat.com>, "Kohler, Jon"
<jon@...anix.com>, Pawan Gupta <pawan.kumar.gupta@...ux.intel.com>, "Peter
Zijlstra (Intel)" <peterz@...radead.org>, Thomas Gleixner <tglx@...nel.org>,
"x86@...nel.org" <x86@...nel.org>, "Winiarska, Iwona"
<iwona.winiarska@...el.com>, LKML <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH 1/6] x86/cpu: Break Vendor/Family/Model macros into
separate header
>> + device->info.device_id = cpu_id >> 4;
>>
>
> I imagine this shifting the stepping. Should this be cpu_id & 0xF?
Fixing here seems to be better than changing the constants. But needs a "~"
/* Ignore the stepping */
device->info.device_id = cpu_id & ~ 0xF;
-Tony
Powered by blists - more mailing lists