Re: [edk2-devel] [PATCH v2 0/3] Support for RuntimeServicesSupported global variable

["Ard Biesheuvel" <ard.biesheuvel@linaro.org>]

On Thu, 28 Nov 2019 at 01:25, Jeff Brasen <jbrasen@nvidia.com> wrote:
>
>  Add support for the new UEFI 2.8 runtime services supported variable
>  that is used to indicate which runtime services a platform supports.
>  Also, add support for initializing this variable based on a PCD.
>
> Change Log:
> v1 - Initial version
> v2 - Move pcd from MdeModulePkg to MdePkg and update uni file
>
>
> Jeff Brasen (3):
>   MdePkg-UefiSpec.h: Add UEFI 2.8 RuntimeServicesSuppported definations
>   MdePkg/MdeModule: Add support for RuntimeServicesSupported variable
>   MdeModulePkg/BdsDxe: Set RuntimeServicesSupported variable
>

Hi Jeff,

Apologies if I am derailing your attempt at getting this support
merged, but I am actually proposing to the USWG to make this a config
table rather than a EFI variable, for the obvious reason that using a
runtime service to decide which runtime services are supported is a
bit problematic, and it forces the OS to invent its own way to
exchange this information between the OS loader and the OS proper.

A configuration table does not have this limitation, so it seems much
better suited for this.


>  .../VarCheckUefiLib/VarCheckUefiLibNullClass.c     | 11 +++++++
>  MdeModulePkg/Universal/BdsDxe/BdsDxe.inf           |  1 +
>  MdeModulePkg/Universal/BdsDxe/BdsEntry.c           | 35 +++++++++++++++++++++-
>  MdePkg/Include/Guid/GlobalVariable.h               |  7 +++++
>  MdePkg/Include/Uefi/UefiSpec.h                     | 18 +++++++++++
>  MdePkg/MdePkg.dec                                  | 18 +++++++++++
>  MdePkg/MdePkg.uni                                  | 17 +++++++++++
>  7 files changed, 106 insertions(+), 1 deletion(-)
>
> --
> 2.7.4
>
>
> 
>