diff options
author | Benjamin Doron <benjamin.doron00@gmail.com> | 2023-03-31 16:41:25 -0400 |
---|---|---|
committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2023-04-12 11:50:30 +0000 |
commit | dc5f2905ebfdf68ae28ce1081d435af0f8641dd9 (patch) | |
tree | f73b0d31e77bf3f98a77a347c8476e78f4acdc05 /.mailmap | |
parent | b991aec0509f24ae7573d732ba337549ecee310c (diff) | |
download | edk2-dc5f2905ebfdf68ae28ce1081d435af0f8641dd9.zip edk2-dc5f2905ebfdf68ae28ce1081d435af0f8641dd9.tar.gz edk2-dc5f2905ebfdf68ae28ce1081d435af0f8641dd9.tar.bz2 |
UefiPayloadPkg: Always build MemoryTypeInformation HOB for DXE GCD
MemoryType information assists GCD with defragmenting the memory map.
When the DXE core starts, GCD adds memory descriptors for the resource
descriptors HOBs. This allocates heap space which can be reused later
as the bins by memory type. It seems memory allocation prefers low
ranges.
It seems "below 4G" is an artifact of this heap reuse. However, the
memory type information determines the DXE core's
`MinimalMemorySizeNeeded`, determining which system memory descriptor
HOB may be used by DXE. Furthermore, it's important that the memory
type information be correct, for an S4 memory map.
Therefore, follow other bootloaders, such as [MinPlatform][1], and do
this unconditionally. As of [edk2-stable202011][2], it was.
[1]: https://github.com/tianocore/edk2-platforms/blob/b6f96743891be51541184bf61dd2970c008e2c43/Platform/Intel/MinPlatformPkg/PlatformInit/PlatformInitPei/PlatformInitPreMem.c#L164-L201
[2]: https://github.com/tianocore/edk2/blob/edk2-stable202011/UefiPayloadPkg/BlSupportPei/BlSupportPei.c#L462-L466
Cc: Guo Dong <guo.dong@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Sean Rhodes <sean@starlabs.systems>
Cc: James Lu <james.lu@intel.com>
Cc: Gua Guo <gua.guo@intel.com>
Reviewed-by: Sean Rhodes <sean@starlabs.systems>
Signed-off-by: Benjamin Doron <benjamin.doron00@gmail.com>
Diffstat (limited to '.mailmap')
0 files changed, 0 insertions, 0 deletions