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

Jeff Brasen posted 3 patches 32 weeks ago
Failed in applying to current master (apply log)
.../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(-)

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

Posted by Jeff Brasen 32 weeks ago
 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

 .../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


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#51378): https://edk2.groups.io/g/devel/message/51378
Mute This Topic: https://groups.io/mt/63268463/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

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

Posted by Ard Biesheuvel 29 weeks ago
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
>
>
> 
>

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#52282): https://edk2.groups.io/g/devel/message/52282
Mute This Topic: https://groups.io/mt/63268463/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-