Re: [PATCH] MdePkg-UefiSpec.h: Add UEFI 2.8 new memory attributes

[qing.huang@intel.com]

[-- Attachment #1: Type: text/plain, Size: 1699 bytes --]

Add V2 patch to add some description for these new memory type from UEFI 2.8 spec.

Thanks  & Best regards,
Huang, Qing


> -----Original Message-----
> From: Gao, Liming
> Sent: Wednesday, October 30, 2019 9:17 AM
> To: Huang, Qing <qing.huang@intel.com>; devel@edk2.groups.io
> Subject: RE: [PATCH] MdePkg-UefiSpec.h: Add UEFI 2.8 new memory
> attributes
> 
> Reviewed-by: Liming Gao <liming.gao@intel.com>
> 
> >-----Original Message-----
> >From: Huang, Qing
> >Sent: Thursday, October 24, 2019 10:34 AM
> >To: devel@edk2.groups.io
> >Cc: Gao, Liming <liming.gao@intel.com>
> >Subject: [PATCH] MdePkg-UefiSpec.h: Add UEFI 2.8 new memory attributes
> >
> >BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2309
> >Add EFI_MEMORY_SP and EFI_MEMORY_CPU_CRYPTO definition into UEFI
> >specification header file.
> >
> >Signed-off-by: Qing Huang <qing.huang@intel.com>
> >Cc: Liming Gao <liming.gao@intel.com>
> >---
> > MdePkg/Include/Uefi/UefiSpec.h | 7 +++++++
> > 1 file changed, 7 insertions(+)
> >
> >diff --git a/MdePkg/Include/Uefi/UefiSpec.h
> >b/MdePkg/Include/Uefi/UefiSpec.h index 44a0a6a7fa..50fae6f013 100644
> >--- a/MdePkg/Include/Uefi/UefiSpec.h
> >+++ b/MdePkg/Include/Uefi/UefiSpec.h
> >@@ -82,6 +82,13 @@ typedef enum {
> > // If all memory has the same reliability, then this bit is not used.
> > //
> > #define EFI_MEMORY_MORE_RELIABLE    0x0000000000010000ULL
> >+
> >+//
> >+// Note: UEFI spec 2.8 and following:
> >+//
> >+#define EFI_MEMORY_SP               0x0000000000040000ULL
> >+#define EFI_MEMORY_CPU_CRYPTO       0x0000000000080000ULL
> >+
> > //
> > // Runtime memory attribute
> > //
> >--
> >2.17.0.windows.1


[-- Attachment #2: 0001-MdePkg-UefiSpec.h-Add-UEFI-2.8-new-memory-attributes-V2.patch --]
[-- Type: application/octet-stream, Size: 1788 bytes --]

From f5a951a52a4926499a51d08ea6ea423948735ae2 Mon Sep 17 00:00:00 2001
From: Qing Huang <qing.huang@intel.com>
Date: Thu, 24 Oct 2019 10:34:15 +0800
Subject: [PATCH] MdePkg-UefiSpec.h: Add UEFI 2.8 new memory attributes

BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2309
Add EFI_MEMORY_SP and EFI_MEMORY_CPU_CRYPTO definition into UEFI specification header file.

Signed-off-by: Qing Huang <qing.huang@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
---
 MdePkg/Include/Uefi/UefiSpec.h | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

diff --git a/MdePkg/Include/Uefi/UefiSpec.h b/MdePkg/Include/Uefi/UefiSpec.h
index 44a0a6a7fa..5d58faf7ed 100644
--- a/MdePkg/Include/Uefi/UefiSpec.h
+++ b/MdePkg/Include/Uefi/UefiSpec.h
@@ -82,6 +82,26 @@ typedef enum {
 // If all memory has the same reliability, then this bit is not used.
 //
 #define EFI_MEMORY_MORE_RELIABLE    0x0000000000010000ULL
+
+//
+// Note: UEFI spec 2.8 and following:
+//
+// Specific-purpose memory (SPM). The memory is earmarked for
+// specific purposes such as for specific device drivers or applications.
+// The SPM attribute serves as a hint to the OS to avoid allocating this
+// memory for core OS data or code that can not be relocated.
+//
+#define EFI_MEMORY_SP               0x0000000000040000ULL
+//
+// If this flag is set, the memory region is capable of being
+// protected with the CPU’s memory cryptographic
+// capabilities. If this flag is clear, the memory region is not
+// capable of being protected with the CPU’s memory
+// cryptographic capabilities or the CPU does not support CPU
+// memory cryptographic capabilities.
+//
+#define EFI_MEMORY_CPU_CRYPTO       0x0000000000080000ULL
+
 //
 // Runtime memory attribute
 //
-- 
2.17.0.windows.1