Re: [PATCH] efistub: Lower default log level

[Aaron Kling]

On Tue, Jul 8, 2025 at 2:30 AM Aaron Kling via B4 Relay
<devnull+webgeek1234.gmail.com@xxxxxxxxxx> wrote:
>
> From: Aaron Kling <webgeek1234@xxxxxxxxx>
>
> Some uefi implementations will write the efistub logs to the display
> over a splash image. This is not desirable for debug and info logs, so
> lower the default efi log level to exclude them.
>
> Suggested-by: Ard Biesheuvel <ardb@xxxxxxxxxx>
> Signed-off-by: Aaron Kling <webgeek1234@xxxxxxxxx>
> ---
>  drivers/firmware/efi/libstub/printk.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/firmware/efi/libstub/printk.c b/drivers/firmware/efi/libstub/printk.c
> index 3a67a2cea7bdf1aa215d48dbf9ece4ceec6e4c28..bc599212c05dd746a9c54abbbe61a4bf70f1a8c4 100644
> --- a/drivers/firmware/efi/libstub/printk.c
> +++ b/drivers/firmware/efi/libstub/printk.c
> @@ -5,13 +5,13 @@
>  #include <linux/ctype.h>
>  #include <linux/efi.h>
>  #include <linux/kernel.h>
> -#include <linux/printk.h> /* For CONSOLE_LOGLEVEL_* */
> +#include <linux/kern_levels.h>
>  #include <asm/efi.h>
>  #include <asm/setup.h>
>
>  #include "efistub.h"
>
> -int efi_loglevel = CONSOLE_LOGLEVEL_DEFAULT;
> +int efi_loglevel = LOGLEVEL_NOTICE;
>
>  /**
>   * efi_char16_puts() - Write a UCS-2 encoded string to the console
>
> ---
> base-commit: d7b8f8e20813f0179d8ef519541a3527e7661d3a
> change-id: 20250708-efi-default-loglevel-4da5a36cac87
>
> Best regards,
> --
> Aaron Kling <webgeek1234@xxxxxxxxx>

This patch was originally suggested a few months ago [0], but as far
as I can tell was never queued for merge. Since I'm also hitting a
case where this is relevant, I'm sending this in to bring attention
back to it.

Sincerely,
Aaron

[0] https://lore.kernel.org/all/CAMj1kXFKSrVx4SO_T8sHU=4hie8pjk8CUWQDV8oh+bz=xMn4rw@xxxxxxxxxxxxxx/