Re: [PATCH 5/6] installer: suppress boot console status noise

[Alyssa Ross <hi@alyssa.is>]

[-- Attachment #1: Type: text/plain, Size: 2290 bytes --]

colbyt <colby@colbyt.com> writes:

> The installer can show systemd or udev messages between Plymouth exiting
> and cage starting, which breaks the intended flicker-free graphical
> handoff.
>
> Add quiet/systemd/udev kernel parameters to suppress status output and
> stop forcing systemd.journald.forward_to_console in the QEMU runner.
> This keeps the installer VM aligned with the quiet graphical boot path.
>
> Signed-off-by: colbyt <colby@colbyt.com>
> ---
>  release/installer/configuration.nix | 9 ++++++++-
>  release/installer/run-vm.nix        | 5 +----
>  2 files changed, 9 insertions(+), 5 deletions(-)
>
> diff --git a/release/installer/configuration.nix b/release/installer/configuration.nix
> index 3f9ef24..daaf5b7 100644
> --- a/release/installer/configuration.nix
> +++ b/release/installer/configuration.nix
> @@ -11,7 +11,14 @@ in
>    imports = [ (modulesPath + "/profiles/all-hardware.nix") ];
>  
>    boot.consoleLogLevel = lib.mkDefault 2;
> -  boot.kernelParams = [ "udev.log_priority=5" ];
> +  boot.kernelParams = [
> +    "quiet"
> +    "loglevel=3"
> +    "systemd.show_status=false"
> +    "rd.systemd.show_status=false"
> +    "udev.log_level=3"
> +    "rd.udev.log_level=3"
> +  ];

Why 3 in particular?  Experimentation?  Inspiration from some other
distribution?

>    boot.initrd.verbose = false;
>  
>    boot.kernelPackages = pkgs.linuxPackages_latest;
> diff --git a/release/installer/run-vm.nix b/release/installer/run-vm.nix
> index b9907f7..6639c8b 100644
> --- a/release/installer/run-vm.nix
> +++ b/release/installer/run-vm.nix
> @@ -40,8 +40,5 @@ writeShellScript "run-spectrum-installer-vm.sh" ''
>      -drive file=/proc/self/fd/3,format=raw,if=virtio \
>      -kernel ${installer.kernel} \
>      -initrd ${installer.initramfs} \
> -    -append ${escapeShellArg (toString [
> -      installer.kernelParams
> -      "systemd.journald.forward_to_console"
> -    ])}
> +    -append ${escapeShellArg installer.kernelParams}
>  '') (_: {})

The divergence here was intentional, since in development it's very
useful to be able to see logs on the serial console, and it should have
a minimal impact on anything else.  Did you see this causing problems
with flicker-free boot or something?

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]