why build runtime memory allocation hob here? The PayloadEntry should already got the information and build a new HOB list to DXE core, will anyone access these region late? If yes, maybe you need add LINUXBOOT_PAYLOAD flag for this code, and update commit message on this.
Thanks, Guo -----Original Message----- From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Cheng-Chieh Huang via groups.io Sent: Wednesday, July 21, 2021 6:23 AM To: devel@edk2.groups.io Cc: Cheng-Chieh Huang <chengch...@google.com> Subject: [edk2-devel] [PATCH v1 4/6] UefiPayloadPkg: Reserve Payload config in runtime services data Signed-off-by: Cheng-Chieh Huang <chengch...@google.com> --- UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c b/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c index ae16f25c7c0e..70afbf83ed4a 100644 --- a/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c +++ b/UefiPayloadPkg/UefiPayloadEntry/UefiPayloadEntry.c @@ -517,6 +517,8 @@ BuildGenericHob ( // The UEFI payload FV BuildMemoryAllocationHob (PcdGet32 (PcdPayloadFdMemBase), PcdGet32 (PcdPayloadFdMemSize), EfiBootServicesData); + // The UEFI payload config FV + BuildMemoryAllocationHob (PcdGet32 (PcdPayloadFdMemBase) - SIZE_64KB, SIZE_64KB, EfiRuntimeServicesData); // // Build CPU memory space and IO space hob -- 2.32.0.402.g57bb445576-goog -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#78627): https://edk2.groups.io/g/devel/message/78627 Mute This Topic: https://groups.io/mt/84357536/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
