diff options
author | xli24 <xli24@6f19259b-4bc3-4df7-8a09-765794883524> | 2010-08-20 17:32:04 +0000 |
---|---|---|
committer | xli24 <xli24@6f19259b-4bc3-4df7-8a09-765794883524> | 2010-08-20 17:32:04 +0000 |
commit | 7ae58ec2eaaad7a4a22092266c407d76c12ed69a (patch) | |
tree | 161cd5dc22cd650e951858e75e27eb12fe92ddad /IntelFrameworkPkg/FrameworkSpecConformance.txt | |
parent | 9e375eb11f5ad03f8d13357be232077879971cab (diff) | |
download | edk2-platforms-7ae58ec2eaaad7a4a22092266c407d76c12ed69a.tar.xz |
The definition of NumberOfSmmReservedRegions is incorrect in Framework SmmCis specification 0.91, in which the type of this field is defined as UINTN. However, HOBs are supposed to be CPU neutral, so UINT32 should be used. Here the code is updated and comments in FrameworkSpecConformance.txt is added.
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@10813 6f19259b-4bc3-4df7-8a09-765794883524
Diffstat (limited to 'IntelFrameworkPkg/FrameworkSpecConformance.txt')
-rw-r--r-- | IntelFrameworkPkg/FrameworkSpecConformance.txt | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/IntelFrameworkPkg/FrameworkSpecConformance.txt b/IntelFrameworkPkg/FrameworkSpecConformance.txt index f70386b728..22f940dfc6 100644 --- a/IntelFrameworkPkg/FrameworkSpecConformance.txt +++ b/IntelFrameworkPkg/FrameworkSpecConformance.txt @@ -1119,13 +1119,18 @@ ##
1. Include/Guid/SmramMemoryReserve.h
typedef struct {
+ UINT32 NumberOfSmmReservedRegions;
...
} EFI_SMRAM_HOB_DESCRIPTOR_BLOCK;
- The name of the definition is *NOT* consistent with Framework SmmCis specification 0.91, in which it's
+ 1) The name of the definition is *NOT* consistent with Framework SmmCis specification 0.91, in which it's
defined as "EFI_HOB_SMRAM_DESCRIPTOR_BLOCK" rather than "EFI_SMRAM_HOB_DESCRIPTOR_BLOCK".
Keeping this inconsistency for backward compatibility.
+ 2) The definition of NumberOfSmmReservedRegions is *NOT* consistent with Framework SmmCis specification 0.91,
+ in which the type of this field is defined as UINTN. However, HOBs are supposed to be CPU neutral, so UINTN
+ is incorrect and UINT32 should be used.
+
2. Include/Guid/SmramMemoryReserve.h
typedef enum {
...
|