Re: [PATCH] x86/ia32: State that IA32 emulation is disabled

From: Borislav Petkov
Date: Thu Nov 30 2023 - 11:09:15 EST


On Thu, Nov 30, 2023 at 04:52:13PM +0100, Borislav Petkov wrote:
> +static inline bool ia32_enabled_verbose(void)
> +{
> +#ifdef CONFIG_IA32_EMULATION
> + pr_notice_once("32-bit emulation disabled. You can reenable with ia32_emulation=on\n");
> +#endif
> + return ia32_enabled();
> +}

As Nik rightfully points out offlist, this should rather be something
like:

---
bool enabled = ia32_enabled();

#ifdef CONFIG_IA32_EMULATION
if (!enabled)
pr_notice_once("32-bit emulation disabled. You can reenable with ia32_emulation=on\n");
#endif

return enabled;
---

Thx.

--
Regards/Gruss,
Boris.

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