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

Jeff Brasen posted 3 patches 4 years, 4 months 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 4 years, 4 months 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 4 years, 3 months 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]
-=-=-=-=-=-=-=-=-=-=-=-