From db316236605edf9e1beda3f7710f304eb2950dcb Mon Sep 17 00:00:00 2001 From: Toomas Soome Date: Fri, 27 Mar 2020 18:01:01 +0000 Subject: [PATCH] loader.efi: conout->QueryMode() can fail to return screen dimensions Some systems are reported to fail to report screen dimensions unless the screen mode is set first. --- stand/efi/libefi/efi_console.c | 18 +++++++++++++----- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/stand/efi/libefi/efi_console.c b/stand/efi/libefi/efi_console.c index deca014999bc..60b5c0f984c5 100644 --- a/stand/efi/libefi/efi_console.c +++ b/stand/efi/libefi/efi_console.c @@ -388,10 +388,18 @@ efi_cons_probe(struct console *cp) conout = ST->ConOut; conin = ST->ConIn; - status = BS->OpenProtocol(ST->ConsoleInHandle, &simple_input_ex_guid, - (void **)&coninex, IH, NULL, EFI_OPEN_PROTOCOL_GET_PROTOCOL); - if (status != EFI_SUCCESS) - coninex = NULL; + /* + * Call SetMode to work around buggy firmware. + */ + status = conout->SetMode(conout, conout->Mode->Mode); + + if (coninex == NULL) { + status = BS->OpenProtocol(ST->ConsoleInHandle, + &simple_input_ex_guid, (void **)&coninex, + IH, NULL, EFI_OPEN_PROTOCOL_GET_PROTOCOL); + if (status != EFI_SUCCESS) + coninex = NULL; + } cp->c_flags |= C_PRESENTIN | C_PRESENTOUT; } @@ -824,7 +832,7 @@ efi_cons_update_mode(void) char env[8]; status = conout->QueryMode(conout, conout->Mode->Mode, &cols, &rows); - if (EFI_ERROR(status)) { + if (EFI_ERROR(status) || cols * rows == 0) { cols = 80; rows = 24; }