From: Dave Hansen <dave.hansen@linux.intel.com>
To: linux-kernel@vger.kernel.org
Cc: sohil.mehta@intel.com, zhao1.liu@intel.com,
Dave Hansen <dave.hansen@linux.intel.com>,
Borislav Petkov <bp@alien8.de>, "H. Peter Anvin" <hpa@zytor.com>,
Ingo Molnar <mingo@redhat.com>, Jon Kohler <jon@nutanix.com>,
Pawan Gupta <pawan.kumar.gupta@linux.intel.com>,
"Peter Zijlstra (Intel)" <peterz@infradead.org>,
Thomas Gleixner <tglx@kernel.org>,
Tony Luck <tony.luck@intel.com>,
x86@kernel.org
Subject: [PATCH v4 0/4] x86/cpu: Take Intel platform into account for old microcode checks
Date: Wed, 04 Mar 2026 10:10:16 -0800 [thread overview]
Message-ID: <20260304181016.85EC87C7@davehans-spike.ostc.intel.com> (raw)
Chagnes from v3:
* Fix handling of the Pentium II's "no platform" goofiness.
Note: this was found in a Github Copilot review of the
series. I wrote the solution.
* Rebase to 7.0-rc2
Changes from v2:
* Move new helpers from .h into .c file. This avoids the
#include dependency issues for now. That can be dealt
with another day.
* Rename x86_platform_id to have an intel_ prefix and add a
union to make it clear that AMD can use the space for
something else
Changes from v1:
* Fix non-x86 PECI compile issues by lifting some x86 macros
into an arch-independent header.
* Make intel_get_platform_id() match its name and return an ID,
not a mask.
* Sort #includes
* Move ->x86_platform_id comment
The platform ID vs. platform mask confusion meant that v1
worked by total accident. If you tested v1, I'd really
appreciate you retest this as well.
--
There was a report[1] that CPUs running updated microcode were being
reported as running old microcode. The reason is that the old
microcode list neglects to take the platform ID into account.
The platform ID is an Intel-only construct that allows CPUs that
otherwise have the same model/family/stepping to take different
microcode revisions. The microcode loader itself already checks this.
Only the recent "old_microcode" checker failed here.
Treat the platform ID as a peer of model/family/stepping. Store it
in 'struct cpuinfo_x86', enable matching on it with with 'struct
x86_cpu_id', and flesh out the 'old_microcode' list with it.
This fixes the report of an inaccurate, false positive in the
'old_microcode' vulnerability file.
1. https://lore.kernel.org/all/38660F8F-499E-48CD-B58B-4822228A5941@nutanix.com/
Cc: Thomas Gleixner <tglx@kernel.org>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Dave Hansen <dave.hansen@linux.intel.com>
Cc: "H. Peter Anvin" <hpa@zytor.com>
Cc: Tony Luck <tony.luck@intel.com>
Cc: Pawan Gupta <pawan.kumar.gupta@linux.intel.com>
Cc: "Peter Zijlstra (Intel)" <peterz@infradead.org>
Cc: x86@kernel.org
Cc: Jon Kohler <jon@nutanix.com>
arch/x86/include/asm/microcode.h | 2
arch/x86/include/asm/processor.h | 5
arch/x86/kernel/cpu/intel.c | 1
arch/x86/kernel/cpu/match.c | 3
arch/x86/kernel/cpu/microcode/intel-ucode-defs.h | 398 +++++++++++++----------
arch/x86/kernel/cpu/microcode/intel.c | 55 ++-
include/linux/mod_devicetable.h | 2
7 files changed, 294 insertions(+), 172 deletions(-)
next reply other threads:[~2026-03-04 18:10 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-04 18:10 Dave Hansen [this message]
2026-03-04 18:10 ` [PATCH v4 1/4] x86/microcode: Refactor platform ID enumeration into a helper Dave Hansen
2026-03-05 1:38 ` Pawan Gupta
2026-03-05 6:28 ` Sohil Mehta
2026-03-05 18:01 ` Pawan Gupta
2026-03-05 18:37 ` Dave Hansen
2026-03-05 13:16 ` Borislav Petkov
2026-03-05 19:07 ` [tip: x86/microcode] " tip-bot2 for Dave Hansen
2026-03-05 20:28 ` tip-bot2 for Dave Hansen
2026-03-04 18:10 ` [PATCH v4 2/4] x86/cpu: Add platform ID to CPU info structure Dave Hansen
2026-03-05 2:26 ` Pawan Gupta
2026-03-05 18:39 ` Dave Hansen
2026-03-05 19:07 ` [tip: x86/microcode] " tip-bot2 for Dave Hansen
2026-03-05 20:28 ` tip-bot2 for Dave Hansen
[not found] ` <20260306004923.ln6ai2zl23gjsdov@desk>
2026-03-06 0:56 ` [PATCH v4 2/4] " Pawan Gupta
2026-03-04 18:10 ` [PATCH v4 3/4] x86/cpu: Add platform ID to CPU matching structure Dave Hansen
2026-03-05 19:07 ` [tip: x86/microcode] " tip-bot2 for Dave Hansen
2026-03-05 20:28 ` tip-bot2 for Dave Hansen
2026-03-04 18:10 ` [PATCH v4 4/4] x86/microcode: Add platform mask to Intel microcode "old" list Dave Hansen
2026-03-05 19:07 ` [tip: x86/microcode] " tip-bot2 for Dave Hansen
2026-03-05 20:28 ` tip-bot2 for Dave Hansen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260304181016.85EC87C7@davehans-spike.ostc.intel.com \
--to=dave.hansen@linux.intel.com \
--cc=bp@alien8.de \
--cc=hpa@zytor.com \
--cc=jon@nutanix.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=pawan.kumar.gupta@linux.intel.com \
--cc=peterz@infradead.org \
--cc=sohil.mehta@intel.com \
--cc=tglx@kernel.org \
--cc=tony.luck@intel.com \
--cc=x86@kernel.org \
--cc=zhao1.liu@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®