Re: [PATCH] x86/MCE: Get microcode revision from cpu_data instead of boot_cpu_data

[Borislav Petkov]

On Fri, Dec 01, 2023 at 07:56:59PM +0000, Sironi, Filippo wrote:
> Boris, I just took a quick look and I might be missing something. If cores
> fail to load the microcode or timeout, we taint the kernel, print an error
> message, and then bubble up an error to userspace via:
> 
> load_late_stop_cpus
> load_late_locked
> reload_store
> 
> Right?

Yap.

> We would take servers that fail out of production;

And I'd like to hear about such issues. We added this failure checking
only recently because something might go wrong and we want to warn. But
it all updates fine here so kinda hard to test.

My expectation is that if microcode fails loading on a subset of
machines, the machine would more or less freeze. Depending, ofc, on what
the microcode is updating...

> however, for others it might be interesting to have the correct
> information. The patch - with a reworked commit message - might still
> be useful to a few.

https://lore.kernel.org/r/20231118193248.1296798-3-yazen.ghannam@xxxxxxx

:)

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette