Reviewed-by: Star Zeng <star.z...@intel.com>
-----Original Message-----
From: Ni, Ray
Sent: Thursday, November 21, 2019 1:46 PM
To: Jeff Brasen <jbra...@nvidia.com>; edk2-de...@lists.01.org;
devel@edk2.groups.io
Cc: Gao, Liming <liming....@intel.com>; Kinney, Michael D
<michael.d.kin...@intel.com>; Wu, Hao A <hao.a...@intel.com>; Gao, Zhichao
<zhichao....@intel.com>; Zeng, Star <star.z...@intel.com>
Subject: RE: [PATCH 2/3] MdePkg/MdeModule: Add support for
RuntimeServicesSupported variable
Reviewed-by: Ray Ni <ray...@intel.com>
+ VarCheck expert Star Zeng.
> -----Original Message-----
> From: Jeff Brasen <jbra...@nvidia.com>
> Sent: Saturday, November 16, 2019 1:43 AM
> To: edk2-de...@lists.01.org; devel@edk2.groups.io
> Cc: Jeff Brasen <jbra...@nvidia.com>; Gao, Liming <liming....@intel.com>;
> Kinney, Michael D <michael.d.kin...@intel.com>; Wu, Hao A
> <hao.a...@intel.com>; Ni, Ray <ray...@intel.com>; Gao, Zhichao
> <zhichao....@intel.com>
> Subject: [PATCH 2/3] MdePkg/MdeModule: Add support for
> RuntimeServicesSupported variable
>
> Add support for new global variable defined in the UEFI 2.8 specification.
> This
> provides a bitmask of which calls are implemented by the firmware during
> runtime services.
>
> Signed-off-by: Jeff Brasen <jbra...@nvidia.com>
> ---
> .../Library/VarCheckUefiLib/VarCheckUefiLibNullClass.c | 11
> +++++++++++
> MdePkg/Include/Guid/GlobalVariable.h | 7 +++++++
> 2 files changed, 18 insertions(+)
>
> diff --git
> a/MdeModulePkg/Library/VarCheckUefiLib/VarCheckUefiLibNullClass.c
> b/MdeModulePkg/Library/VarCheckUefiLib/VarCheckUefiLibNullClass.c
> index e3bf04a..4264892 100644
> --- a/MdeModulePkg/Library/VarCheckUefiLib/VarCheckUefiLibNullClass.c
> +++ b/MdeModulePkg/Library/VarCheckUefiLib/VarCheckUefiLibNullClass.c
> @@ -553,6 +553,17 @@ UEFI_DEFINED_VARIABLE_ENTRY
> mGlobalVariableList[] = {
> },
> NULL
> },
> + {
> + EFI_RUNTIME_SERVICES_SUPPORTED_VARIABLE_NAME,
> + {
> + VAR_CHECK_VARIABLE_PROPERTY_REVISION,
> + VAR_CHECK_VARIABLE_PROPERTY_READ_ONLY,
> + VARIABLE_ATTRIBUTE_BS_RT,
> + sizeof (UINT16),
> + sizeof (UINT16)
> + },
> + NULL
> + },
> };
>
> UEFI_DEFINED_VARIABLE_ENTRY mGlobalVariableList2[] = { diff --git
> a/MdePkg/Include/Guid/GlobalVariable.h
> b/MdePkg/Include/Guid/GlobalVariable.h
> index 7abc103..06a8a12 100644
> --- a/MdePkg/Include/Guid/GlobalVariable.h
> +++ b/MdePkg/Include/Guid/GlobalVariable.h
> @@ -182,5 +182,12 @@ extern EFI_GUID gEfiGlobalVariableGuid; /// Its
> attribute is BS+RT.
> ///
> #define EFI_VENDOR_KEYS_VARIABLE_NAME L"VendorKeys"
> +///
> +/// Bitmask of which calls are implemented by the firmware during runtime
> services.
> +/// RT access is required only if GetVariable() is implemented by runtime
> services.
> +/// Should be treated as read-only.
> +/// Its attribute is BS+RT.
> +///
> +#define EFI_RUNTIME_SERVICES_SUPPORTED_VARIABLE_NAME
> L"RuntimeServicesSupported"
>
> #endif
> --
> 2.7.4
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#50997): https://edk2.groups.io/g/devel/message/50997
Mute This Topic: https://groups.io/mt/59298932/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
