Re: [PATCH] x86/microcode: Consolidate the loader enablement

From: Borislav Petkov
Date: Fri Apr 11 2025 - 07:12:05 EST


On Thu, Apr 10, 2025 at 01:53:25PM +0200, Thomas Gleixner wrote:
> It just exists to make the above a one time operation. What's wrong with
> having:
>
> static void __init microcode_check_cmdline(void)
> {
> if (cmdline_find_option_bool(boot_command_line, "dis_ucode_ldr") <= 0)
> dis_ucode_ldr = false;
> }
>
> and call that once at the proper place?

Yeah, I had that done for the mk_early_pgtbl_32() to avoid doing __pa gunk in
the loader. But if we're going to do it unconditionally now, I don't need the
tristate anymore.

Lemme zap it.

--
Regards/Gruss,
Boris.

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