Commit be1b670f authored by Ashok Raj's avatar Ashok Raj Committed by Borislav Petkov (AMD)

x86/microcode/intel: Do not retry microcode reloading on the APs

The retries in load_ucode_intel_ap() were in place to support systems
with mixed steppings. Mixed steppings are no longer supported and there is
only one microcode image at a time. Any retries will simply reattempt to
apply the same image over and over without making progress.

  [ bp: Zap the circumstantial reasoning from the commit message. ]

Fixes: 06b8534c ("x86/microcode: Rework microcode loading")
Signed-off-by: default avatarAshok Raj <ashok.raj@intel.com>
Signed-off-by: default avatarBorislav Petkov (AMD) <bp@alien8.de>
Reviewed-by: default avatarThomas Gleixner <tglx@linutronix.de>
Cc: stable@vger.kernel.org
Link: https://lore.kernel.org/r/20221129210832.107850-3-ashok.raj@intel.com
parent 5b1586ab
...@@ -495,7 +495,6 @@ void load_ucode_intel_ap(void) ...@@ -495,7 +495,6 @@ void load_ucode_intel_ap(void)
else else
iup = &intel_ucode_patch; iup = &intel_ucode_patch;
reget:
if (!*iup) { if (!*iup) {
patch = __load_ucode_intel(&uci); patch = __load_ucode_intel(&uci);
if (!patch) if (!patch)
...@@ -506,12 +505,7 @@ void load_ucode_intel_ap(void) ...@@ -506,12 +505,7 @@ void load_ucode_intel_ap(void)
uci.mc = *iup; uci.mc = *iup;
if (apply_microcode_early(&uci, true)) { apply_microcode_early(&uci, true);
/* Mixed-silicon system? Try to refetch the proper patch: */
*iup = NULL;
goto reget;
}
} }
static struct microcode_intel *find_patch(struct ucode_cpu_info *uci) static struct microcode_intel *find_patch(struct ucode_cpu_info *uci)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment