From nobody Wed May 15 13:34:17 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+110104+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+110104+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1698328434; cv=none; d=zohomail.com; s=zohoarc; b=Kr/tBghRFdYjso5YZIjp2K9QDjwK32cUgVn4PsgXZELZFBGK/Bv9+V9DRL2tFsGZo/ALZLaUcwf1PsxTgcCJzLv2kQVAR5TJgCOh9fiJfpBxAg96FMBSnoXMF2HwVlhbpRIbiwsuxkZDmxdTr/MRr7UY30j7dgSkmIi5l933lXA= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1698328434; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:List-Subscribe:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:Reply-To:Sender:Subject:Subject:To:To:Message-Id; bh=tVbOB3Q05fbuNFiszgvAtpmtB8HISF8XR1gK04BuZiI=; b=RVA7+xQ1evj8GoPx6qGZjhLF2cbW2Aj+XsyGn6YliV3S1UavBMvsDiimwV3wWqOdSqyyHOHFokNczkh+oKBFAKliGl2eI2E72dwwQ1oPZbXu77nPb3G05X/lCd6Lct/MZw+ag/XI4TRIBixj2RiXCqRBGx+u70AS9NEUbxcMZt4= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+110104+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1698328434347965.555668704904; Thu, 26 Oct 2023 06:53:54 -0700 (PDT) Return-Path: DKIM-Signature: a=rsa-sha256; bh=8UOOKnRdgEn3s++TNLZOCNHgniyBsnbUlTsl+YLsXYw=; c=relaxed/simple; d=groups.io; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Type:Content-Transfer-Encoding; s=20140610; t=1698328434; v=1; b=dD7ldjRR8Gn8KeTUnvDB8yyGCyLKDngQ1YKEq7cEQR750tME7l8xmW3XuEs6vdQVvmgebHNj rr/Me6kCzvICjthn8MRu2YAbZfE8P7LwafM/KUk4bul/0zIvxCUPkfb/AFd+qhEZgO9CcPu0aOn b8bUsy/H/OcuUHl8guhDiwgw= X-Received: by 127.0.0.2 with SMTP id HRjrYY1788612x7eaiOjHdaZ; Thu, 26 Oct 2023 06:53:54 -0700 X-Received: from mail-oo1-f50.google.com (mail-oo1-f50.google.com [209.85.161.50]) by mx.groups.io with SMTP id smtpd.web11.71155.1698328433448353517 for ; Thu, 26 Oct 2023 06:53:53 -0700 X-Received: by mail-oo1-f50.google.com with SMTP id 006d021491bc7-586abec2c8cso321845eaf.3 for ; Thu, 26 Oct 2023 06:53:53 -0700 (PDT) X-Gm-Message-State: dFNfr7Izt0GC3unPEzBHLu2Ox1787277AA= X-Google-Smtp-Source: AGHT+IGNokS1KnwkNMkfmQ2bWRaIVPNEBNjH7OB/VKVqpTIJoGS046SdI/NqUvQxji+ZTMAUo/IHeg== X-Received: by 2002:a05:6870:659e:b0:1dc:723d:b8d0 with SMTP id fp30-20020a056870659e00b001dc723db8d0mr22447582oab.27.1698328432186; Thu, 26 Oct 2023 06:53:52 -0700 (PDT) X-Received: from Belldandy-Slimbook.tail03774.ts.net ([32.221.209.96]) by smtp.gmail.com with ESMTPSA id dg3-20020a056214084300b00656373f9c30sm5226665qvb.75.2023.10.26.06.53.51 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 26 Oct 2023 06:53:51 -0700 (PDT) From: "Neal Gompa" To: devel@edk2.groups.io Cc: Neal Gompa , Pete Batard , =?UTF-8?q?Daniel=20P=20=2E=20Berrang=C3=A9?= , Gerd Hoffmann , Samer El-Haj-Mahmoud Subject: [edk2-devel] [PATCH] MdeModulePkg/UefiBootManagerLib: Signal ReadyToBoot on platform recovery Date: Thu, 26 Oct 2023 09:53:24 -0400 Message-ID: <20231026135324.15914-1-ngompa@fedoraproject.org> MIME-Version: 1.0 Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,ngompa13@gmail.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @groups.io) X-ZM-MESSAGEID: 1698328436247100003 From: Neal Gompa Currently, the ReadyToBoot event is only signaled when a formal Boot Manager option is executed (in BmBoot.c -> EfiBootManagerBoot ()). However, the introduction of Platform Recovery in UEFI 2.5 makes it necessary to signal ReadyToBoot when a Platform Recovery boot loader runs because otherwise it may lead to the execution of a boot loader that has similar requirements to a regular one that is not launched as a Boot Manager option. This is especially critical to ensuring that the graphical console is actually usable during platform recovery, as some platforms do rely on the ConsolePrefDxe driver, which only performs console initialization after ReadyToBoot is triggered. This patch fixes that behavior by calling EfiSignalEventReadyToBoot () in EfiBootManagerProcessLoadOption (), which is the function that sets up the platform recovery boot process. The expected behavior has been clarified in the UEFI 2.10 specification to explicitly indicate this behavior is required for correct operation. This is a rebased version of the patch originally written by Pete Batard. Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3D2831 Cc: Pete Batard Cc: Daniel P. Berrang=C3=A9 Cc: Gerd Hoffmann Cc: Samer El-Haj-Mahmoud Co-authored-by: Pete Batard Signed-off-by: Neal Gompa --- MdeModulePkg/Library/UefiBootManagerLib/BmLoadOption.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmLoadOption.c b/MdeMo= dulePkg/Library/UefiBootManagerLib/BmLoadOption.c index 2087f0b91d..31ed608817 100644 --- a/MdeModulePkg/Library/UefiBootManagerLib/BmLoadOption.c +++ b/MdeModulePkg/Library/UefiBootManagerLib/BmLoadOption.c @@ -1416,6 +1416,15 @@ EfiBootManagerProcessLoadOption ( return EFI_SUCCESS; } =20 + // + // Signal the EVT_SIGNAL_READY_TO_BOOT event when we are about to load a= nd execute the boot option. + // + EfiSignalEventReadyToBoot (); + // + // Report Status Code to indicate ReadyToBoot was signalled + // + REPORT_STATUS_CODE (EFI_PROGRESS_CODE, (EFI_SOFTWARE_DXE_BS_DRIVER | EFI= _SW_DXE_BS_PC_READY_TO_BOOT_EVENT)); + // // Load and start the load option. // --=20 2.41.0 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#110104): https://edk2.groups.io/g/devel/message/110104 Mute This Topic: https://groups.io/mt/102200076/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-