From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web10.50007.1680252502968215375 for ; Fri, 31 Mar 2023 01:48:23 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: pierre.gondois@arm.com) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id C64632F4; Fri, 31 Mar 2023 01:49:06 -0700 (PDT) Received: from [10.57.20.151] (unknown [10.57.20.151]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id AD45C3F7D8; Fri, 31 Mar 2023 01:48:20 -0700 (PDT) Message-ID: <81c4bdc8-b584-661e-803e-e80c95e9cecf@arm.com> Date: Fri, 31 Mar 2023 10:47:56 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.9.0 Subject: Re: [PATCH 1/1] ArmPkg/PlatformBootManagerLib: Add path to boot UEFI Shell over UiApp From: "PierreGondois" To: Ard Biesheuvel Cc: devel@edk2.groups.io, ardb+tianocore@kernel.org, leif@nuviainc.com, sami.mujawar@arm.com, patrik.berglund@arm.com, marcin.juszkiewicz@linaro.org References: <20230207090653.395992-1-pierre.gondois@arm.com> <4d1c3209-87d6-546d-e638-b15d0239e8f3@arm.com> In-Reply-To: <4d1c3209-87d6-546d-e638-b15d0239e8f3@arm.com> Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Hello Ard, On 2/13/23 09:39, Pierre Gondois wrote: > Hello Ard, > > On 2/9/23 17:57, Ard Biesheuvel wrote: >> On Tue, 7 Feb 2023 at 10:07, wrote: >>> >>> From: Pierre Gondois >>> >>> The UEFI Shell is a non-active boot option, at the opposite of UiApp. >>> If no valid boot option is found, UiApp is selected. UiApp requires a >>> human interaction. When installing a new EDKII image in CIs or when >>> scripting is required, this is problematic. >>> >>> If no valid boot option is discovered, add a path to directly go to >>> the UEFI Shell where the startup.nsh script is automatically executed. >>> The UEFI Shell is launched after connecting possible devices, but >>> before the reset that is meant to automatically make them visible. >>> >>> The new PcdUefiShellDefaultBootEnable must be set to TRUE to enable >>> this behaviour. The Pcd is set to false by default. >>> >> >> Is this similar to how we implemented this on RPi4? IIRC, a similar >> issue came up there as well. > > I'm not finding an equivalent for the Rpi4. I see that the > BootDiscoveryPolicy was added, but this regards which devices to connect > during boot IIUC. The UEFI Shell also seems to have been added to the > Boot Manager Menu, but this should not select the UEFI Shell by default. > Are you referring to the following ? https://edk2.groups.io/g/devel/message/60719 If yes I don't think this patch aims to achieve the same thing. The goal here is to reach the UEFI shell (i.e. make it an active option). I believe the patch mentioned above doesn't modify that and is more about making it visible in UiApp BootManager. Please let me know if you were referring to another modification, Regards, Pierre