diff options
author | Star Zeng <star.zeng@intel.com> | 2016-07-08 10:37:46 +0800 |
---|---|---|
committer | Star Zeng <star.zeng@intel.com> | 2016-07-09 10:35:28 +0800 |
commit | 072a3ca1d36a42aec97f871c808776ee7038ca06 (patch) | |
tree | 83193e092190ada76551d7f7d3b3ea8ed8a3dac8 | |
parent | 2a3a91eb9dd9c45bce2647988519bf3754fe8a5d (diff) | |
download | edk2-platforms-072a3ca1d36a42aec97f871c808776ee7038ca06.tar.xz |
MdeModulePkg MemoryProfile.h:two bytes of Reserved[4] as ActionStringOffset
At 94092aa60341a3e4b1e1ea7c362781b8404ac538, we extended
MEMORY_PROFILE_ALLOC_INFO to add ActionStringOffset and
Reserved2[6] fields, that was not aware the Reserved[4]
field can be reused to have better memory profile database
size efficiency.
With the patch, PcdMemoryProfilePropertyMask|0x3 and
PcdMemoryProfileMemoryType|0xffff, the memory profile
database size can be reduced as below on my sample platform.
UefiMemoryProfileSize - 0x1597A8 -> UefiMemoryProfileSize - 0x12AB28
SmramProfileSize - 0xCF68 -> SmramProfileSize - 0xB8E8
Cc: Jiewen Yao <jiewen.yao@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Star Zeng <star.zeng@intel.com>
Reviewed-by: Jiewen Yao <jiewen.yao@intel.com>
-rw-r--r-- | MdeModulePkg/Include/Guid/MemoryProfile.h | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/MdeModulePkg/Include/Guid/MemoryProfile.h b/MdeModulePkg/Include/Guid/MemoryProfile.h index 38a64945e3..777f950db7 100644 --- a/MdeModulePkg/Include/Guid/MemoryProfile.h +++ b/MdeModulePkg/Include/Guid/MemoryProfile.h @@ -146,13 +146,12 @@ typedef struct { MEMORY_PROFILE_COMMON_HEADER Header;
PHYSICAL_ADDRESS CallerAddress;
UINT32 SequenceId;
- UINT8 Reserved[4];
+ UINT8 Reserved[2];
+ UINT16 ActionStringOffset;
MEMORY_PROFILE_ACTION Action;
EFI_MEMORY_TYPE MemoryType;
PHYSICAL_ADDRESS Buffer;
UINT64 Size;
- UINT16 ActionStringOffset;
- UINT8 Reserved2[6];
//CHAR8 ActionString[];
} MEMORY_PROFILE_ALLOC_INFO;
|