Hello Hao/Jian,

As per Maintainers.txt, I think you guys are the ov= erall maintainers of MdeModulePkg. Bringing this patch to your attention so= that it could be submitted. Ray who is the reviewer of BDS changes has alr= eady reviewed this patch.

Thanks

Ashish

From: Ni, Ray <ray.ni@intel.com>
Sent: Wednesday, January 1, 2020 7:05 PM
To: devel@edk2.groups.io; Ashish Singhal <ashishsingha@nvidia.co= m>; Wang, Sunny (HPS SW) <sunnywang@hpe.com>; Wang, Jian J <jia= n.j.wang@intel.com>; Wu, Hao A <hao.a.wu@intel.com>; Gao, Zhichao = <zhichao.gao@intel.com>; Kinney, Michael D <michael.d.kinney@intel= .com>; afish@apple.com
Cc: Spottswood, Jason <jason.spottswood@hpe.com>
Subject: RE: [edk2-devel] [PATCH v7] MdeModulePkg: Add EDK2 Platfor= m Boot Manager Protocol

External email: Use caution opening = links or attachments

Ashish, no follow up is required from your side.

MdeModulePkg maintainers will help to push the patc= hes.

From: devel@edk2.groups.io <deve= l@edk2.groups.io> On Behalf Of Ashish Singhal
Sent: Thursday, December 26, 2019 11:42 AM
To: Ni, Ray <ray.ni@intel.co= m>; Wang, Sunny (HPS SW) <su= nnywang@hpe.com>; devel@edk2.groups.io; Wang, Ji= an J <jian.j.wang@intel.com= >; Wu, Hao A <hao.a.wu@intel.co= m>; Gao, Zhichao <zhicha= o.gao@intel.com>; Kinney, Michael D <micha= el.d.kinney@intel.com>; afish@apple.com
Cc: Spottswood, Jason <jason.spottswood@hpe.com>
Subject: Re: [edk2-devel] [PATCH v7] MdeModulePkg: Add EDK2 Platfor= m Boot Manager Protocol

Hi Ray,

Following up on this. Are we waiting on more sugges= tions or clarifications?

Thanks

Ashish

Get Outlook for i= OS

From: Ni, Ray <ra= y.ni@intel.com>
Sent: Monday, December 23, 2019 10:41:50 PM
To: Wang, Sunny (HPS SW) <s= unnywang@hpe.com>; devel@edk2.groups.io <devel@edk2.groups.io>; Ashish Sin= ghal <ashishsingha@nvidia.com= >; Wang, Jian J <jian.j.= wang@intel.com>; Wu, Hao A <hao.a.wu@intel.com>; Gao, Zhichao <zhichao.ga= o@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>; afish@apple.com <afish@apple.com>
Cc: Spottswood, Jason <jason.spottswood@hpe.com>
Subject: RE: [edk2-devel] [PATCH v7] MdeModulePkg: Add EDK2 Platfor= m Boot Manager Protocol

External email: Use = caution opening links or attachments

Reviewed-by: Ray Ni <ray.ni@intel.c= om>

> -----Original Message-----
> From: Wang, Sunny (HPS SW) <s= unnywang@hpe.com>
> Sent: Tuesday, December 24, 2019 12:44 PM
> To: devel@edk2.groups.io;= ashishsingha@nvidia.com; Ni, Ray
> <ray.ni@intel.com>; Wan= g, Jian J <jian.j.wang@intel.co= m>; Wu, Hao A
> <hao.a.wu@intel.com>;= Gao, Zhichao <zhichao.gao@inte= l.com>; Kinney,
> Michael D <michael.d= .kinney@intel.com>; afish@apple.com
> Cc: Wang, Sunny (HPS SW) <sun= nywang@hpe.com>; Spottswood, Jason
> <jason.spottswood@hpe.= com>
> Subject: RE: [edk2-devel] [PATCH v7] MdeModulePkg: Add EDK2 Platform<= br> > Boot Manager Protocol
>
> Looks good enough to me. Thanks for addressing my comments, Ashish. > Reviewed-by: Sunny Wang <sunn= ywang@hpe.com>
>
> -----Original Message-----
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io]= On Behalf Of
> Ashish Singhal
> Sent: Tuesday, December 24, 2019 10:58 AM
> To: devel@edk2.groups.io;= ray.ni@intel.com; jian.j.wang= @intel.com;
> hao.a.wu@intel.com; zhichao.gao@intel.com; m= ichael.d.kinney@intel.com;
> afish@apple.com
> Cc: Ashish Singhal <ash= ishsingha@nvidia.com>
> Subject: [edk2-devel] [PATCH v7] MdeModulePkg: Add EDK2 Platform Boot=
> Manager Protocol
>
> Add edk2 platform boot manager protocol which would have platform
> specific refreshes to the auto enumerated as well as NV boot options = for the
> platform.
>
> Signed-off-by: Ashish Singhal <ashishsingha@nvidia.com>
> ---
> .../Include/Protocol/PlatformBootManager.h &nb= sp; | 82
> ++++++++++++++= ;++++++++
> MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c | = 41 +++++++++--
> .../Library/UefiBootManagerLib/InternalBm.h &n= bsp; | 2 +
> .../UefiBootManagerLib/UefiBootManagerLib.inf = | 2 +
> MdeModulePkg/MdeModulePkg.dec &nbs= p; &= nbsp; | 4 ++
> 5 files changed, 124 insertions(+), 7 deletions(-) cr= eate mode 100644
> MdeModulePkg/Include/Protocol/PlatformBootManager.h
>
> diff --git a/MdeModulePkg/Include/Protocol/PlatformBootManager.h
> b/MdeModulePkg/Include/Protocol/PlatformBootManager.h
> new file mode 100644
> index 0000000..26b9ce4
> --- /dev/null
> +++ b/MdeModulePkg/Include/Protocol/PlatformBootManager.h=
> @@ -0,0 +1,82 @@
> +/** @file
> +
> + Copyright (c) 2019, NVIDIA CORPORATION. All rights reserv= ed.
> +
> + SPDX-License-Identifier: BSD-2-Clause-Patent
> +
> +**/
> +
> +#ifndef __PLATFORM_BOOT_MANAGER_PROTOCOL_H__
> +#define __PLATFORM_BOOT_MANAGER_PROTOCOL_H__
> +
> +#include <Library/UefiBootManagerLib.h>
> +
> +//
> +// Platform Boot Manager Protocol GUID value // #define
> +EDKII_PLATFORM_BOOT_MANAGER_PROTOCOL_GUID \
> + { \
> + 0xaa17add4, 0x756c, 0x460d, { 0x9= 4, 0xb8, 0x43, 0x88, 0xd7, 0xfb, 0x3e,
> 0x59 } \
> + }
> +
> +//
> +// Protocol interface structure
> +//
> +typedef struct _EDKII_PLATFORM_BOOT_MANAGER_PROTOCOL
> +EDKII_PLATFORM_BOOT_MANAGER_PROTOCOL;
> +
> +//
> +// Revision The revision to which the protocol interface adheres= .
> +// All fut= ure revisions must be backwards compatible.
> +// If a fu= ture version is not back wards compatible it is not the same
> GUID.
> +//
> +#define EDKII_PLATFORM_BOOT_MANAGER_PROTOCOL_REVISION
> 0x00000001
> +
> +//
> +// Function Prototypes
> +//
> +
> +/*
> + This function allows platform to refresh all boot options= specific to
> +the platform. Within
> + this function, platform can make modifications to the aut= o enumerated
> +platform boot options
> + as well as NV boot options.
> +
> + @param[in const] BootOptions = ; An array of auto enumerated
> platform boot options.
> + &nbs= p; &= nbsp; &nbs= p; This array will be freed by caller u= pon successful
> + &nbs= p; &= nbsp; &nbs= p; exit of this function and output arr= ay would be used.
> +
> + @param[in const] BootOptionsCount = The number of elements in
> BootOptions.
> +
> + @param[out] UpdatedBootOpti= ons An array of boot options that
> have been customized
> + &nbs= p; &= nbsp; &nbs= p; for the platform on top of input boo= t options. This
> + &nbs= p; &= nbsp; &nbs= p; array would be allocated by
> REFRESH_ALL_BOOT_OPTIONS
> + &nbs= p; &= nbsp; &nbs= p; and would be freed by caller after c= onsuming it.
> +
> + @param[out] UpdatedBootOpti= onsCount The number of elements in
> UpdatedBootOptions.
> +
> +
> + @retval EFI_SUCCESS &n= bsp; = ; Platform refresh to input BootOptions and
> + &nbs= p; &= nbsp; &nbs= p; BootCount have been done.
> +
> + @retval EFI_OUT_OF_RESOURCES = ; Memory allocation failed.
> +
> + @retval EFI_INVALID_PARAMETER &nbs= p; Input is not correct.
> +
> + @retval EFI_UNSUPPORTED &nbs= p; Platfo= rm specific overrides are not
> supported.
> +*/
> +typedef
> +EFI_STATUS
> +(EFIAPI *PLATFORM_BOOT_MANAGER_REFRESH_ALL_BOOT_OPTIONS) (
> + IN CONST EFI_BOOT_MANAGER_LOAD_OPTION *BootOptions,=
> + IN CONST UINTN &= nbsp; &nbs= p; BootOptionsCount,
> + OUT EFI_BOOT_MANAGER_= LOAD_OPTION **UpdatedBootOptions,
> + OUT UINTN = &nb= sp; *UpdatedBootOptionsCoun= t
> + );
> +
> +struct _EDKII_PLATFORM_BOOT_MANAGER_PROTOCOL {
> + UINT64 &nb= sp; = &nb= sp; Revision;
> + PLATFORM_BOOT_MANAGER_REFRESH_ALL_BOOT_OPTIONS
> RefreshAllBootOptions;
> +};
> +
> +extern EFI_GUID gEdkiiPlatformBootManagerProtocolGuid;
> +
> +#endif /* __PLATFORM_BOOT_MANAGER_PROTOCOL_H__ */
> diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c
> b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c
> index 760d764..62c5b2dc 100644
> --- a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c
> +++ b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c > @@ -1,6 +1,7 @@
> /** @file
> Library functions which relates with booting.
>
> +Copyright (c) 2019, NVIDIA CORPORATION. All rights reserved.
> Copyright (c) 2011 - 2019, Intel Corporation. All rights reserv= ed. 
> (C) Copyright 2015-2016 Hewlett Packard Enterprise Development = LP 
> SPDX-License-Identifier: BSD-2-Clause-Patent @@ -2258,12 +2= 259,15 @@
> EfiBootManagerRefreshAllBootOption (
> VOID
> )
> {
> - EFI_STATUS &nb= sp; Status;
> - EFI_BOOT_MANAGER_LOAD_OPTION *NvBootOptions;
> - UINTN &n= bsp; = ; NvBootOptionCount;
> - EFI_BOOT_MANAGER_LOAD_OPTION *BootOptions;
> - UINTN &n= bsp; = ; BootOptionCount;
> - UINTN &n= bsp; = ; Index;
> + EFI_STATUS = ; &n= bsp; Status;
> + EFI_BOOT_MANAGER_LOAD_OPTION = ; *NvBootOptions;
> + UINTN &nbs= p; &= nbsp; NvBootOptionCou= nt;
> + EFI_BOOT_MANAGER_LOAD_OPTION = ; *BootOptions;
> + UINTN &nbs= p; &= nbsp; BootOptionCount= ;
> + EFI_BOOT_MANAGER_LOAD_OPTION = ; *UpdatedBootOptions;
> + UINTN &nbs= p; &= nbsp; UpdatedBootOpti= onCount;
> + UINTN &nbs= p; &= nbsp; Index;
> + EDKII_PLATFORM_BOOT_MANAGER_PROTOCOL *PlatformBootManager= ;
>
> //
> // Optionally refresh the legacy boot option @@ -22= 73,7 +2277,6 @@
> EfiBootManagerRefreshAllBootOption (
> }
>
> BootOptions =3D BmEnumerateBootOptions = (&BootOptionCount);
> - NvBootOptions =3D EfiBootManagerGetLoadOptions (&NvBootOp= tionCount,
> LoadOptionTypeBoot);
>
> //
> // Mark the boot option as added by BDS by setting = OptionalData to a
> special GUID @@ -2284,6 +2287,30 @@
> EfiBootManagerRefreshAllBootOption (
> }
>
> //
> + // Locate Platform Boot Options Protocol // S= tatus =3D
> + gBS->LocateProtocol (&gEdkiiPlatformBootManagerProtocolG= uid,
> + &nbs= p; &= nbsp; NULL,
> + &nbs= p; &= nbsp; (VOID **)&PlatformBootM= anager); if
> + (!EFI_ERROR (Status)) {
> + //
> + // If found, call platform specific refresh t= o all auto enumerated and NV
> + // boot options.
> + //
> + Status =3D PlatformBootManager->RefreshAll= BootOptions ((CONST
> EFI_BOOT_MANAGER_LOAD_OPTION *)BootOptions,
> + &nbs= p; &= nbsp; &nbs= p; &= nbsp; (CONST UINTN)BootOptionCoun= t,
> + &nbs= p; &= nbsp; &nbs= p; &= nbsp; &UpdatedBootOptions, > + &nbs= p; &= nbsp; &nbs= p; &= nbsp; &UpdatedBootOptionCount= );
> + if (!EFI_ERROR (Status)) {
> + EfiBootManagerFreeLoadOptions (Bo= otOptions, BootOptionCount);
> + BootOptions =3D UpdatedBootOption= s;
> + BootOptionCount =3D UpdatedBootOp= tionCount;
> + }
> + }
> +
> + NvBootOptions =3D EfiBootManagerGetLoadOptions (&NvBo= otOptionCount,
> + LoadOptionTypeBoot);
> +
> + //
> // Remove invalid EFI boot options from NV
> //
> for (Index =3D 0; Index < NvBootOptionCount; Ind= ex++) { diff --git
> a/MdeModulePkg/Library/UefiBootManagerLib/InternalBm.h
> b/MdeModulePkg/Library/UefiBootManagerLib/InternalBm.h
> index 027eb25..ac866ac 100644
> --- a/MdeModulePkg/Library/UefiBootManagerLib/InternalBm.h
> +++ b/MdeModulePkg/Library/UefiBootManagerLib/InternalBm.= h
> @@ -1,6 +1,7 @@
> /** @file
> BDS library definition, include the file and data s= tructure
>
> +Copyright (c) 2019, NVIDIA CORPORATION. All rights reserved.
> Copyright (c) 2004 - 2018, Intel Corporation. All rights reserv= ed. 
> (C) Copyright 2015 Hewlett Packard Enterprise Development LP<= ;BR>
> SPDX-License-Identifier: BSD-2-Clause-Patent @@ -41,6 +42,7= @@ SPDX-
> License-Identifier: BSD-2-Clause-Patent #include <Protocol/V= ariableLock.h>
> #include <Protocol/RamDisk.h> #include <Protocol/Defer= redImageLoad.h>
> +#include <Protocol/PlatformBootManager.h>
>
> #include <Guid/MemoryTypeInformation.h> #include &l= t;Guid/FileInfo.h>
> diff --git
> a/MdeModulePkg/Library/UefiBootManagerLib/UefiBootManagerLib.inf
> b/MdeModulePkg/Library/UefiBootManagerLib/UefiBootManagerLib.inf
> index ed6b467..cf59086 100644
> --- a/MdeModulePkg/Library/UefiBootManagerLib/UefiBootManagerLib.inf<= br> > +++
> b/MdeModulePkg/Library/UefiBootManagerLib/UefiBootManagerLib.inf
> @@ -5,6 +5,7 @@
> # manipulation, hotkey registration, UEFI boot, connect/d= isconnect, console
> # manipulation, driver health checking and etc.
> #
> +# Copyright (c) 2019, NVIDIA CORPORATION. All rights reser= ved.
> # Copyright (c) 2007 - 2018, Intel Corporation. All right= s reserved. # (C)
> Copyright 2016 Hewlett Packard Enterprise Development LP &nb= sp; # SPDX-
> License-Identifier: BSD-2-Clause-Patent @@ -107,6 +108,7 @@
> gEfiFormBrowser2ProtocolGuid = ; &n= bsp; ## SOMETIMES_CONSUMES
> gEfiRamDiskProtocolGuid &nbs= p; &= nbsp; ## SOMETIMES_CONSUMES
> gEfiDeferredImageLoadProtocolGuid = ## SOMETIMES_CONSUME= S
> + gEdkiiPlatformBootManagerProtocolGuid &n= bsp; ## SOMETIMES_CONSUMES
>
> [Pcd]
>
> gEfiMdeModulePkgTokenSpaceGuid.PcdResetOnMemoryTypeInformationC
> hange ## SOMETIMES_CONSUMES
> diff --git a/MdeModulePkg/MdeModulePkg.dec
> b/MdeModulePkg/MdeModulePkg.dec index 41b9e70..cc238e9 100644
> --- a/MdeModulePkg/MdeModulePkg.dec
> +++ b/MdeModulePkg/MdeModulePkg.dec
> @@ -3,6 +3,7 @@
> # It also provides the definitions(including PPIs/PROTOCOLs/GUI= Ds and
> library classes) # and libraries instances, which are used for = those modules.
> #
> +# Copyright (c) 2019, NVIDIA CORPORATION. All rights reserved. > # Copyright (c) 2007 - 2019, Intel Corporation. All rights rese= rved. #
> Copyright (c) 2016, Linaro Ltd. All rights reserved. = # (C) Copyright 2016 -
> 2019 Hewlett Packard Enterprise Development LP @@ -609,6 &#= 43;610,9
> @@
> ## Include/Protocol/PeCoffImageEmulator.h
> gEdkiiPeCoffImageEmulatorProtocolGuid =3D { 0x96f46= 153, 0x97a7, 0x4793,
> { 0xac, 0xc1, 0xfa, 0x19, 0xbf, 0x78, 0xea, 0x97 } }
>
> + ## Include/Protocol/PlatformBootManager.h
> + gEdkiiPlatformBootManagerProtocolGuid =3D { 0xaa17add4, 0= x756c, 0x460d,
> + { 0x94, 0xb8, 0x43, 0x88, 0xd7, 0xfb, 0x3e, 0x59 } }
> +
> #
> # [Error.gEfiMdeModulePkgTokenSpaceGuid]
> # 0x80000001 | Invalid value provided.
> --
> 2.7.4
>
>
>
>

This email message is for the sole use of the inten= ded recipient(s) and may contain confidential information. Any unauth= orized review, use, disclosure or distribution is prohibited. If you = are not the intended recipient, please contact the sender by reply email and destroy all copies of the original message.=