On Wed, 20 May 2020 at 18:38, Arvind Sankar <nivedita@xxxxxxxxxxxx> wrote: > > On Tue, May 19, 2020 at 11:07:55AM -0400, Arvind Sankar wrote: > > On Tue, May 19, 2020 at 10:22:40AM +0200, Ard Biesheuvel wrote: > > > On Mon, 18 May 2020 at 21:07, Arvind Sankar <nivedita@xxxxxxxxxxxx> wrote: > > > > @@ -100,7 +123,9 @@ efi_status_t efi_parse_options(char const *cmdline) > > > > if (!strcmp(param, "nokaslr")) { > > > > efi_nokaslr = true; > > > > } else if (!strcmp(param, "quiet")) { > > > > - efi_quiet = true; > > > > + efi_loglevel = CONSOLE_LOGLEVEL_QUIET; > > > > + } else if (!strcmp(param, "debug")) { > > > > + efi_loglevel = CONSOLE_LOGLEVEL_DEBUG; > > > > > > Should we wire this to 'efi=debug' instead? > > > > > > > Sure. > > Do you prefer it wired up to both or just efi=debug? > Let's stick with efi=debug, that is what all other EFI code uses.