[edk2-devel] [PATCH 0/4] Add a pcd PcdBootManagerInBootOrder to control whether BootManager is in BootOrder
Ni, Ray
ray.ni at intel.com
Wed Jul 17 08:07:05 UTC 2019
Zhichao,
Firstly, patch 4/4 is a fix to patch 3/4, why not merge them into one patch? Do we have a policy to strictly cherry-pick the patches from MU?
Secondly, the change that skips to add the #### to BootOrder may cause the Boot#### pointing to the BootManagerMenu be overwritten later. Reason is BmGetFreeOptionNumber() replies on BootOrder/BootNext to get a free option number.
If the requirement is to let BootManagerMenuApp hide the BootManagerMenu, we could change the BootManagerMenuApp using a PCD to control.
Thanks,
Ray
> -----Original Message-----
> From: Gao, Zhichao
> Sent: Wednesday, July 17, 2019 3:50 PM
> To: devel at edk2.groups.io
> Cc: Wang, Jian J <jian.j.wang at intel.com>; Wu, Hao A <hao.a.wu at intel.com>;
> Ni, Ray <ray.ni at intel.com>; Zeng, Star <star.zeng at intel.com>; Gao, Liming
> <liming.gao at intel.com>; Sean Brogan <sean.brogan at microsoft.com>;
> Michael Turner <Michael.Turner at microsoft.com>; Bret Barkelew
> <Bret.Barkelew at microsoft.com>
> Subject: [PATCH 0/4] Add a pcd PcdBootManagerInBootOrder to control
> whether BootManager is in BootOrder
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1979
>
> Add a pcd to control whether the BootManager is in BootOrder variable.
> From a UI perspective, it never want to see a Front Page UI in the boot order
> list. So add a pcd PcdBootManagerInBootOrder to control it. If it is FALSE, the
> BootManager App wouldn't be added to BootOrder. And it wouldn't be
> shown in the BootManagerMenuApp.
>
> Notes:
> BootManager means the application with file guid PcdBootManagerMenuFile.
> It's default value is EEC25BDC-67F2-4D95-B1D5-F81B2039D11D, and users
> usually change the pcd to UiApp.inf's file guid. BootManagerMenuApp
> means uefi application with file guid EEC25BDC-67F2-4D95-B1D5-
> F81B2039D11D.
>
> Cc: Jian J Wang <jian.j.wang at intel.com>
> Cc: Hao A Wu <hao.a.wu at intel.com>
> Cc: Ray Ni <ray.ni at intel.com>
> Cc: Star Zeng <star.zeng at intel.com>
> Cc: Liming gao <liming.gao at intel.com>
> Cc: Sean Brogan <sean.brogan at microsoft.com>
> Cc: Michael Turner <Michael.Turner at microsoft.com>
> Cc: Bret Barkelew <Bret.Barkelew at microsoft.com>
> Signed-off-by: Zhichao Gao <zhichao.gao at intel.com>
>
> Bret Barkelew (2):
> MdeModulePkg: Add PcdBootManagerInBootOrder to dec file
> MdeModulePkg: Use pcd to control if BootManager in BootOrder
>
> Zhichao Gao (2):
> MdeModulePkg: Add prompt and help string for
> PcdBootManagerInBootOrder
> MdeModulePkg: PcdBootManagerInBootOrder should only affect
> BootOrder
>
> .../Library/UefiBootManagerLib/BmBoot.c | 29 +++++++++++++++----
> .../UefiBootManagerLib/UefiBootManagerLib.inf | 1 +
> MdeModulePkg/MdeModulePkg.dec | 6 ++++
> MdeModulePkg/MdeModulePkg.uni | 6 ++++
> 4 files changed, 36 insertions(+), 6 deletions(-)
>
> --
> 2.21.0.windows.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#43857): https://edk2.groups.io/g/devel/message/43857
Mute This Topic: https://groups.io/mt/32501044/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