Re: [edk2-devel] [Patch V3] MdeModulePkg: Retrive boot manager menu from any fv

2021-05-11 Thread Ni, Ray
Reviewed-by: Ray Ni 


> -Original Message-
> From: Liu, Zhiguang 
> Sent: Tuesday, May 11, 2021 10:32 AM
> To: devel@edk2.groups.io
> Cc: Wang, Jian J ; Wu, Hao A ;
> Gao, Zhichao ; Ni, Ray 
> Subject: [Patch V3] MdeModulePkg: Retrive boot manager menu from any fv
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3384
> 
> Currently, UefiBootManagerLib has the below assumption:
>   Assume the BootManagerMenuFile is in the same FV as the module links to
> this library.
> It has some limitation now, so remove the assumption.
> 
> Cc: Jian J Wang 
> Cc: Hao A Wu 
> Cc: Zhichao Gao 
> Cc: Ray Ni 
> Reviewed-by: Hao A Wu 
> Signed-off-by: Zhiguang Liu 
> ---
>  MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c | 9 +
>  1 file changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c
> b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c
> index 6cc34d29c0..bef41ae102 100644
> --- a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c
> +++ b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c
> @@ -2,7 +2,7 @@
>Library functions which relates with booting.
> 
> 
> 
>  Copyright (c) 2019, NVIDIA CORPORATION. All rights reserved.
> 
> -Copyright (c) 2011 - 2020, Intel Corporation. All rights reserved.
> 
> +Copyright (c) 2011 - 2021, Intel Corporation. All rights reserved.
> 
>  (C) Copyright 2015-2021 Hewlett Packard Enterprise Development LP
> 
>  SPDX-License-Identifier: BSD-2-Clause-Patent
> 
> 
> 
> @@ -2385,7 +2385,8 @@ EfiBootManagerRefreshAllBootOption (
>This function is called to get or create the boot option for the Boot 
> Manager
> Menu.
> 
> 
> 
>The Boot Manager Menu is shown after successfully booting a boot option.
> 
> -  Assume the BootManagerMenuFile is in the same FV as the module links to
> this library.
> 
> +  This function will first try to search the BootManagerMenuFile is in the
> same FV as
> 
> +  the module links to this library. If fails, it will search in all FVs.
> 
> 
> 
>@param  BootOptionReturn the boot option of the Boot Manager Menu
> 
> 
> 
> @@ -2437,7 +2438,7 @@ BmRegisterBootManagerMenu (
> 
> 
>if (DevicePath == NULL) {
> 
>  Data = NULL;
> 
> -Status = GetSectionFromFv (
> 
> +Status = GetSectionFromAnyFv (
> 
> PcdGetPtr (PcdBootManagerMenuFile),
> 
> EFI_SECTION_PE32,
> 
> 0,
> 
> @@ -2455,7 +2456,7 @@ BmRegisterBootManagerMenu (
>  //
> 
>  // Get BootManagerMenu application's description from EFI User
> Interface Section.
> 
>  //
> 
> -Status = GetSectionFromFv (
> 
> +Status = GetSectionFromAnyFv (
> 
> PcdGetPtr (PcdBootManagerMenuFile),
> 
> EFI_SECTION_USER_INTERFACE,
> 
> 0,
> 
> --
> 2.30.0.windows.2



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#75057): https://edk2.groups.io/g/devel/message/75057
Mute This Topic: https://groups.io/mt/82737690/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




[edk2-devel] [Patch V3] MdeModulePkg: Retrive boot manager menu from any fv

2021-05-10 Thread Zhiguang Liu
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3384

Currently, UefiBootManagerLib has the below assumption:
  Assume the BootManagerMenuFile is in the same FV as the module links to this 
library.
It has some limitation now, so remove the assumption.

Cc: Jian J Wang 
Cc: Hao A Wu 
Cc: Zhichao Gao 
Cc: Ray Ni 
Reviewed-by: Hao A Wu 
Signed-off-by: Zhiguang Liu 
---
 MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c | 9 +
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c 
b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c
index 6cc34d29c0..bef41ae102 100644
--- a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c
+++ b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c
@@ -2,7 +2,7 @@
   Library functions which relates with booting.
 
 Copyright (c) 2019, NVIDIA CORPORATION. All rights reserved.
-Copyright (c) 2011 - 2020, Intel Corporation. All rights reserved.
+Copyright (c) 2011 - 2021, Intel Corporation. All rights reserved.
 (C) Copyright 2015-2021 Hewlett Packard Enterprise Development LP
 SPDX-License-Identifier: BSD-2-Clause-Patent
 
@@ -2385,7 +2385,8 @@ EfiBootManagerRefreshAllBootOption (
   This function is called to get or create the boot option for the Boot 
Manager Menu.
 
   The Boot Manager Menu is shown after successfully booting a boot option.
-  Assume the BootManagerMenuFile is in the same FV as the module links to this 
library.
+  This function will first try to search the BootManagerMenuFile is in the 
same FV as
+  the module links to this library. If fails, it will search in all FVs.
 
   @param  BootOptionReturn the boot option of the Boot Manager Menu
 
@@ -2437,7 +2438,7 @@ BmRegisterBootManagerMenu (
 
   if (DevicePath == NULL) {
 Data = NULL;
-Status = GetSectionFromFv (
+Status = GetSectionFromAnyFv (
PcdGetPtr (PcdBootManagerMenuFile),
EFI_SECTION_PE32,
0,
@@ -2455,7 +2456,7 @@ BmRegisterBootManagerMenu (
 //
 // Get BootManagerMenu application's description from EFI User Interface 
Section.
 //
-Status = GetSectionFromFv (
+Status = GetSectionFromAnyFv (
PcdGetPtr (PcdBootManagerMenuFile),
EFI_SECTION_USER_INTERFACE,
0,
-- 
2.30.0.windows.2



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#74934): https://edk2.groups.io/g/devel/message/74934
Mute This Topic: https://groups.io/mt/82737690/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-