[edk2-devel] [PATCH 1/1] Platform/RaspberryPi: Setup option for disabling Fast Boot
Sunny Wang
Sunny.Wang at arm.com
Fri Apr 16 13:45:42 UTC 2021
Hi Samer,
Thanks for checking and putting all related issues here. However, I missed your comment in v2. Sorry about that. Therefore, I directly added the references to the commit.
https://github.com/tianocore/edk2-platforms/commit/efdc159ef7c9f15581a0f63d755a1530ff475156
This commit turned out to fix three issues below. I also added comments/updates to these four issues and closed the following three.
https://github.com/pftf/RPi4/issues/144
https://github.com/pftf/RPi4/issues/114
https://github.com/pftf/RPi4/issues/117
Best Regards,
Sunny Wang
-----Original Message-----
From: Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud at arm.com>
Sent: Tuesday, April 13, 2021 5:29 AM
To: Sunny Wang <Sunny.Wang at arm.com>; devel at edk2.groups.io
Cc: Sunny Wang <Sunny.Wang at arm.com>; Jeremy Linton <Jeremy.Linton at arm.com>; Pete Batard <pete at akeo.ie>; Ard Biesheuvel <ardb+tianocore at kernel.org>; Sunny Wang <Sunny.Wang at arm.com>; Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud at arm.com>
Subject: RE: [PATCH 1/1] Platform/RaspberryPi: Setup option for disabling Fast Boot
Sunny,
Thanks for sending this!
This was tested by several RPi4 users and confirmed to fix at least the following issue:
https://github.com/pftf/RPi4/issues/144
https://github.com/pftf/RPi4/issues/114
It *may* also fix the following issues (to be confirmed):
https://github.com/pftf/RPi4/issues/117
https://github.com/pftf/RPi4/issues/136
Please add references to these issues (at least confirmed ones) in the commit message when you send the v2 of the patch.
With that,
Acked-by: Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud at arm.com>
> -----Original Message-----
> From: Sunny Wang <Sunny.Wang at arm.com>
> Sent: Monday, April 12, 2021 5:06 AM
> To: devel at edk2.groups.io
> Cc: Sunny Wang <Sunny.Wang at arm.com>; Samer El-Haj-Mahmoud
> <Samer.El-Haj-Mahmoud at arm.com>; Jeremy Linton <Jeremy.Linton at arm.com>;
> Pete Batard <pete at akeo.ie>; Ard Biesheuvel
> <ardb+tianocore at kernel.org>; Sunny Wang <Sunny.Wang at arm.com>
> Subject: [PATCH 1/1] Platform/RaspberryPi: Setup option for disabling
> Fast Boot
>
> This is a fix for https://github.com/pftf/RPi4/issues/114.
>
> Changes:
> 1. Add a setup option called Boot Policy and consume the setting
> during boot to whether perform or skip ConnectAll.
> 2. The Default setting is set to Full discovery because it is not
> worth enabling Fast boot by default on RaspberryPi systems.
> Enabling it just saves boot time about 1 second, but caused a
> lot of issues.
>
> Testing Done:
> - Booted to Standalone UEFI shell on SD card and use drivers
> command to check the result with Fast Boot and Full discovery
> settings. Then, child/device handles are created as expected.
>
> Note and to-do items:
> - The root cause looks like that boot loaders and some tools like
> grub and iPXE haven't supported selective connect/Fast boot.
> However, system firmware should still provide a setup option for
> user to enable Fast boot with old version boot loaders and tools
> , which is why we proposed this change. We will also report this
> issue to boot loader and tool vendors/open source GitHubs.
> - We Will add more options for connecting specific type devices so
> that we can still have the shortest boot time for all use cases.
>
> Cc: Samer El-Haj-Mahmoud <samer.el-haj-mahmoud at arm.com>
> Cc: Jeremy Linton <jeremy.linton at arm.com>
> Cc: Pete Batard <pete at akeo.ie>
> Cc: Ard Biesheuvel <ardb+tianocore at kernel.org>
> Signed-off-by: Sunny Wang <sunny.wang at arm.com>
> ---
> .../RaspberryPi/Drivers/ConfigDxe/ConfigDxe.c | 11 ++++++++++-
> .../RaspberryPi/Drivers/ConfigDxe/ConfigDxe.inf | 3 ++-
> .../Drivers/ConfigDxe/ConfigDxeHii.uni | 10 +++++++++-
> .../Drivers/ConfigDxe/ConfigDxeHii.vfr | 15 ++++++++++++++-
> Platform/RaspberryPi/Include/ConfigVars.h | 12 +++++++++++-
> .../Library/PlatformBootManagerLib/PlatformBm.c | 16 ++++++++++++++--
> .../PlatformBootManagerLib.inf | 1 +
> Platform/RaspberryPi/RPi3/RPi3.dsc | 10 +++++++++-
> Platform/RaspberryPi/RPi4/RPi4.dsc | 9 ++++++++-
> Platform/RaspberryPi/RaspberryPi.dec | 2 ++
> 10 files changed, 80 insertions(+), 9 deletions(-)
>
> diff --git a/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.c
> b/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.c
> index 22f86d4d44..d3c5869949 100644
> --- a/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.c
> +++ b/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.c
> @@ -1,6 +1,6 @@
> /** @file *- * Copyright (c) 2019 - 2020, ARM Limited. All rights
> reserved.+ * Copyright (c) 2019 - 2021, ARM Limited. All rights
> reserved. * Copyright (c)
> 2018 - 2020, Andrei Warkentin <andrey.warkentin at gmail.com> * *
> SPDX-
> License-Identifier: BSD-2-Clause-Patent@@ -281,6 +281,15 @@
> SetupVariables (
> ); } + Size = sizeof (UINT32);+ Status = gRT->GetVariable
> (L"BootPolicy",+ &gConfigDxeFormSetGuid,+ NULL, &Size,
> &Var32);+ if (EFI_ERROR (Status)) {+ Status = PcdSet32S (PcdBootPolicy,
> PcdGet32 (PcdBootPolicy));+ ASSERT_EFI_ERROR (Status);+ }+ Size =
> sizeof (UINT32); Status = gRT->GetVariable (L"SdIsArasan",
> &gConfigDxeFormSetGuid,diff --git
> a/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.inf
> b/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.inf
> index d51e54e010..032e40b0c3 100644
> --- a/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.inf
> +++ b/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.inf
> @@ -2,7 +2,7 @@
> # # Component description file for the RasbperryPi DXE platform
> config driver. #-# Copyright (c) 2019 - 2020, ARM Limited. All rights
> reserved.+# Copyright (c) 2019 - 2021, ARM Limited. All rights
> reserved. # Copyright (c)
> 2018 - 2020, Andrei Warkentin <andrey.warkentin at gmail.com> # # SPDX-
> License-Identifier: BSD-2-Clause-Patent@@ -93,6 +93,7 @@
> gRaspberryPiTokenSpaceGuid.PcdRamLimitTo3GB
> gRaspberryPiTokenSpaceGuid.PcdFanOnGpio
> gRaspberryPiTokenSpaceGuid.PcdFanTemp+
> gRaspberryPiTokenSpaceGuid.PcdBootPolicy [Depex] gPcdProtocolGuid
> AND gRaspberryPiFirmwareProtocolGuiddiff --git
> a/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxeHii.uni
> b/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxeHii.uni
> index 466fa852cb..7b14fdf39f 100644
> --- a/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxeHii.uni
> +++ b/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxeHii.uni
> @@ -1,7 +1,7 @@
> /** @file * * Copyright (c) 2018, Andrei Warkentin
> <andrey.warkentin at gmail.com>- * Copyright (c) 2020, ARM Limited. All
> rights reserved.+ * Copyright (c) 2020 - 2021, ARM Limited. All
> rights reserved. * * SPDX-License-Identifier: BSD-2-Clause-Patent
> *@@ -60,6
> +60,14 @@
> #string STR_ADVANCED_ASSET_TAG_PROMPT #language en-US "Asset Tag"
> #string STR_ADVANCED_ASSET_TAG_HELP #language en-US "Set the
> system Asset Tag" +#string STR_BOOT_POLICY_PROMPT #language en-
> US "Boot Policy"+#string STR_BOOT_POLICY_HELP #language en-US
> "When Fast Boot is selected, only required devices will be discoverd for
> reducing "+ "the boot time."+
> "When Full Discovery is selected, all the devices will be discoverd
> for some "+ "scenarios such as system deployement and diagnostic tests"+#string
> STR_FAST_BOOT #language en-US "Fast Boot"+#string
> STR_FULL_DISCOVERY #language en-US "Full Discovery"+ /* *
> MMC/SD configuration. */diff --git
> a/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxeHii.vfr
> b/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxeHii.vfr
> index cc7a09cfb7..5dc558ec08 100644
> --- a/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxeHii.vfr
> +++ b/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxeHii.vfr
> @@ -1,7 +1,7 @@
> /** @file * * Copyright (c) 2018 Andrei Warkentin
> <andrey.warkentin at gmail.com>- * Copyright (c) 2020, ARM Limited. All
> rights reserved.+ * Copyright (c) 2020 - 2021, ARM Limited. All
> rights reserved. * * SPDX-License-Identifier: BSD-2-Clause-Patent
> *@@ -116,6
> +116,11 @@ formset
> name = DisplayEnableSShot, guid = CONFIGDXE_FORM_SET_GUID; +
> efivarstore BOOT_POLICY_VARSTORE_DATA,+ attribute =
> EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_RUNTIME_ACCESS |
> EFI_VARIABLE_NON_VOLATILE,+ name = BootPolicy,+ guid =
> CONFIGDXE_FORM_SET_GUID;+ form formid = 1, title =
> STRING_TOKEN(STR_FORM_SET_TITLE); subtitle text =
> STRING_TOKEN(STR_NULL_STRING);@@ -220,6 +225,14 @@ formset
> minsize = 0, maxsize = ASSET_TAG_STR_MAX_LEN,
> endstring;++ oneof varid = BootPolicy.BootPolicy,+ prompt =
> STRING_TOKEN(STR_BOOT_POLICY_PROMPT),+ help =
> STRING_TOKEN(STR_BOOT_POLICY_HELP),+ flags =
> NUMERIC_SIZE_4 | INTERACTIVE | RESET_REQUIRED,+ option text =
> STRING_TOKEN(STR_FAST_BOOT ), value = FAST_BOOT , flags = 0;+ option
> text = STRING_TOKEN(STR_FULL_DISCOVERY), value =
> FULL_DISCOVERY, flags = DEFAULT;+ endoneof; endform; form
> formid = 0x1003,diff --git a/Platform/RaspberryPi/Include/ConfigVars.h
> b/Platform/RaspberryPi/Include/ConfigVars.h
> index 142317985a..3347f899df 100644
> --- a/Platform/RaspberryPi/Include/ConfigVars.h
> +++ b/Platform/RaspberryPi/Include/ConfigVars.h
> @@ -1,7 +1,7 @@
> /** @file * * Copyright (c) 2020, Andrei Warkentin
> <andrey.warkentin at gmail.com>- * Copyright (c) 2020, ARM Limited. All
> rights reserved.+ * Copyright (c) 2020 - 2021, ARM Limited. All
> rights reserved. * * SPDX-License-Identifier: BSD-2-Clause-Patent
> *@@ -143,4
> +143,14 @@ typedef struct {
> UINT32 EnableDma; } MMC_EMMC_DMA_VARSTORE_DATA; +#define
> FAST_BOOT 0+#define FULL_DISCOVERY 1+typedef struct {+ /*+ * 0 -
> Fast Boot+ * 1 - Full Discovrey (Connect All)+ */+ UINT32 BootPolicy;+}
> BOOT_POLICY_VARSTORE_DATA;+ #endif /* CONFIG_VARS_H */diff --git
> a/Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBm.c
> b/Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBm.c
> index c2fc40b8ea..abbe4fb3d0 100644
> --- a/Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBm.c
> +++ b/Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBm.c
> @@ -4,7 +4,7 @@
> * Copyright (c) 2017-2018, Andrei Warkentin
> <andrey.warkentin at gmail.com> * Copyright (c) 2016, Linaro Ltd. All
> rights reserved. * Copyright (c) 2015-2016, Red Hat, Inc.- *
> Copyright (c) 2014- 2020, ARM Ltd. All rights reserved.+ * Copyright
> (c) 2014-2021, ARM Ltd. All rights reserved. * Copyright (c)
> 2004-2016, Intel Corporation. All rights reserved. * *
> SPDX-License-Identifier: BSD-2-Clause-Patent@@ -26,9
> +26,11 @@ #include <Guid/EventGroup.h> #include <Guid/TtyTerm.h>
> +#include <ConfigVars.h>+ #include "PlatformBm.h" -#define
> BOOT_PROMPT L"ESC (setup), F1 (shell), ENTER (boot)"+#define
> BOOT_PROMPT L"ESC (setup), F1 (shell), ENTER (boot)\n" #define
> DP_NODE_LEN(Type) { (UINT8)sizeof (Type), (UINT8)(sizeof (Type) >> 8)
> } @@ -633,6 +635,16 @@ PlatformBootManagerAfterConsole (
> Print (BOOT_PROMPT); } + //+ // Connect the rest of the devices if the
> boot polcy is set to Full discovery+ //+ if (PcdGet32 (PcdBootPolicy) ==
> FULL_DISCOVERY) {+ DEBUG ((DEBUG_INFO, "Boot Policy is Full Discovery.
> Connect All devices\n"));+ EfiBootManagerConnectAll ();+ } else if
> (PcdGet32 (PcdBootPolicy) == FAST_BOOT) {+ DEBUG ((DEBUG_INFO,
> "Boot Policy is Fast Boot. Skip connecting all devices\n"));+ }+ Status = gBS-
> >LocateProtocol (&gEsrtManagementProtocolGuid, NULL,
> (VOID**)&EsrtManagement); if (!EFI_ERROR (Status)) { EsrtManagement-
> >SyncEsrtFmp ();diff --git
> a/Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBootMan
> agerLib.inf
> b/Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBootMan
> agerLib.inf
> index 88f6f8fe09..fbf510ab96 100644
> ---
> a/Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBootMan
> agerLib.inf
> +++
> b/Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBootMa
> +++ nagerLib.inf
> @@ -63,6 +63,7 @@
> [Pcd] gEfiMdePkgTokenSpaceGuid.PcdPlatformBootTimeOut
> gRaspberryPiTokenSpaceGuid.PcdSdIsArasan+
> gRaspberryPiTokenSpaceGuid.PcdBootPolicy [Guids] gEfiFileInfoGuiddiff --
> git a/Platform/RaspberryPi/RPi3/RPi3.dsc
> b/Platform/RaspberryPi/RPi3/RPi3.dsc
> index 0961133ae9..ddb03e405f 100644
> --- a/Platform/RaspberryPi/RPi3/RPi3.dsc
> +++ b/Platform/RaspberryPi/RPi3/RPi3.dsc
> @@ -1,6 +1,6 @@
> # @file #-# Copyright (c) 2011 - 2020, ARM Limited. All rights
> reserved.+# Copyright (c) 2011 - 2021, ARM Limited. All rights
> reserved. # Copyright (c) 2014, Linaro Limited. All rights reserved.
> # Copyright (c) 2015 - 2021, Intel Corporation. All rights reserved.
> # Copyright (c) 2017 - 2018, Andrei Warkentin
> <andrey.warkentin at gmail.com>@@ -512,6 +512,14 @@
>
> gRaspberryPiTokenSpaceGuid.PcdFanOnGpio|L"FanOnGpio"|gConfigDxeFor
> mSetGuid|0x0|0
> gRaspberryPiTokenSpaceGuid.PcdFanTemp|L"FanTemp"|gConfigDxeFormS
> etGuid|0x0|0 + #+ # Boot Policy+ # 0 - Fast Boot+ # 1 - Full
> etGuid|0x0|Discovrey
> (Connect All)+ #+
> gRaspberryPiTokenSpaceGuid.PcdBootPolicy|L"BootPolicy"|gConfigDxeFor
> mSetGuid|0x0|1++ # # Reset-related. #diff --git
> a/Platform/RaspberryPi/RPi4/RPi4.dsc
> b/Platform/RaspberryPi/RPi4/RPi4.dsc
> index ff802d8347..8ee1922a44 100644
> --- a/Platform/RaspberryPi/RPi4/RPi4.dsc
> +++ b/Platform/RaspberryPi/RPi4/RPi4.dsc
> @@ -1,6 +1,6 @@
> # @file #-# Copyright (c) 2011 - 2020, ARM Limited. All rights
> reserved.+# Copyright (c) 2011 - 2021, ARM Limited. All rights
> reserved. # Copyright (c)
> 2017 - 2018, Andrei Warkentin <andrey.warkentin at gmail.com> #
> Copyright
> (c) 2015 - 2021, Intel Corporation. All rights reserved. # Copyright
> (c) 2014, Linaro Limited. All rights reserved.@@ -528,6 +528,13 @@
>
> gRaspberryPiTokenSpaceGuid.PcdFanOnGpio|L"FanOnGpio"|gConfigDxeFor
> mSetGuid|0x0|0
> gRaspberryPiTokenSpaceGuid.PcdFanTemp|L"FanTemp"|gConfigDxeFormS
> etGuid|0x0|60 + #+ # Boot Policy+ # 0 - Fast Boot+ # 1 - Full
> etGuid|0x0|Discovrey
> (Connect All)+ #+
> gRaspberryPiTokenSpaceGuid.PcdBootPolicy|L"BootPolicy"|gConfigDxeFor
> mSetGuid|0x0|1+ # # Reset-related. #diff --git
> a/Platform/RaspberryPi/RaspberryPi.dec
> b/Platform/RaspberryPi/RaspberryPi.dec
> index 08135717ed..8eb1c2bac7 100644
> --- a/Platform/RaspberryPi/RaspberryPi.dec
> +++ b/Platform/RaspberryPi/RaspberryPi.dec
> @@ -2,6 +2,7 @@
> # # Copyright (c) 2016, Linaro, Ltd. All rights reserved. #
> Copyright (c) 2017- 2018, Andrei Warkentin
> <andrey.warkentin at gmail.com>+# Copyright (c) 2021, ARM Limited. All
> rights reserved. # # SPDX-License-Identifier: BSD-2- Clause-Patent #@@ -70,3 +71,4 @@
> gRaspberryPiTokenSpaceGuid.PcdFanTemp|0|UINT32|0x0000001D
> gRaspberryPiTokenSpaceGuid.PcdPlatformResetDelay|0|UINT32|0x0000001
> E
> gRaspberryPiTokenSpaceGuid.PcdMmcEnableDma|0|UINT32|0x0000001F+
> gRaspberryPiTokenSpaceGuid.PcdBootPolicy|0|UINT32|0x00000020--
> 2.31.0.windows.1
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#74209): https://edk2.groups.io/g/devel/message/74209
Mute This Topic: https://groups.io/mt/82033650/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list