The definition of NumberOfSmmReservedRegions is incorrect in Framework SmmCis specifi...

[mirror_edk2.git] / IntelFrameworkPkg / FrameworkSpecConformance.txt

diff --git a/IntelFrameworkPkg/FrameworkSpecConformance.txt b/IntelFrameworkPkg/FrameworkSpecConformance.txt
index f70386b728dbe357b3a03733dcfcd411dbf61cb7..22f940dfc6f951b6f34d2ec9d670af6c31f4e4c6 100644 (file)
--- a/IntelFrameworkPkg/FrameworkSpecConformance.txt
+++ b/IntelFrameworkPkg/FrameworkSpecConformance.txt
@@ -1119,13 +1119,18 @@
 ##\r
   1. Include/Guid/SmramMemoryReserve.h\r
     typedef struct {\r
+      UINT32                NumberOfSmmReservedRegions;\r
       ...\r
     } EFI_SMRAM_HOB_DESCRIPTOR_BLOCK;\r
 \r
-    The name of the definition is *NOT* consistent with Framework SmmCis specification 0.91, in which it's \r
+    1) The name of the definition is *NOT* consistent with Framework SmmCis specification 0.91, in which it's \r
     defined as "EFI_HOB_SMRAM_DESCRIPTOR_BLOCK" rather than "EFI_SMRAM_HOB_DESCRIPTOR_BLOCK". \r
     Keeping this inconsistency for backward compatibility.\r
 \r
+    2) The definition of NumberOfSmmReservedRegions is *NOT* consistent with Framework SmmCis specification 0.91,\r
+    in which the type of this field is defined as UINTN. However, HOBs are supposed to be CPU neutral, so UINTN\r
+    is incorrect and UINT32 should be used.\r
+\r
   2. Include/Guid/SmramMemoryReserve.h\r
     typedef enum {\r
       ...\r