Hi Guo, Sounds like a good proposal, but it would be great to merge this change temporarily. In some cases of the current implementation edk2 does override the memory area, where the coreboot table pointer is located. Therefore the kernel and cbmem tool is not able to locate the tables anymore.
Thanks, Marcello On Tue, Sep 8, 2020 at 11:40 PM Dong, Guo <[email protected]> wrote: > > Hi Marcello, > > In the UEFI payload, we should not hardcoded any memory usage. It means > UEFI payload should use the memory map whatever reported from the > bootloader. I plan to remove this hardcoded memory usage soon. > Before that, it is OK for me to merge this change if you want. > BTW, did you see any issue with current implement? > > Thanks, > Guo > > > -----Original Message----- > > From: [email protected] <[email protected]> On Behalf Of Marcello > > Sylvester Bauer > > Sent: Wednesday, July 8, 2020 5:01 AM > > To: [email protected] > > Cc: Ma, Maurice <[email protected]>; Desimone, Nathaniel L > > <[email protected]>; Zeng, Star <[email protected]> > > Subject: [edk2-devel] [PATCH v1 1/1] UefipayloadPkg: Protect coreboot > tables > > > > From: Patrick Rudolph <[email protected]> > > > > Signed-off-by: Patrick Rudolph <[email protected]> > > Signed-off-by: Marcello Sylvester Bauer <[email protected]> > > Cc: Maurice Ma <[email protected]> > > Cc: Nate DeSimone <[email protected]> > > Cc: Star Zeng <[email protected]> > > --- > > UefiPayloadPkg/BlSupportPei/BlSupportPei.c | 26 ++++++++++++++------ > > 1 file changed, 19 insertions(+), 7 deletions(-) > > > > diff --git a/UefiPayloadPkg/BlSupportPei/BlSupportPei.c > > b/UefiPayloadPkg/BlSupportPei/BlSupportPei.c > > index 22972453117a..b3ff065a537e 100644 > > --- a/UefiPayloadPkg/BlSupportPei/BlSupportPei.c > > +++ b/UefiPayloadPkg/BlSupportPei/BlSupportPei.c > > @@ -390,24 +390,36 @@ BlPeiEntryPoint ( > > EFI_PEI_GRAPHICS_DEVICE_INFO_HOB GfxDeviceInfo; > > > > EFI_PEI_GRAPHICS_DEVICE_INFO_HOB *NewGfxDeviceInfo; > > > > > > > > - > > > > - // > > > > - // Report lower 640KB of RAM. Attribute EFI_RESOURCE_ATTRIBUTE_TESTED > > > > - // is intentionally omitted to prevent erasing of the coreboot header > > > > - // record before it is processed by ParseMemoryInfo. > > > > + // Report lower 640KB of RAM. > > > > + // Mark memory as reserved to keep coreboot header in place. > > > > // > > > > BuildResourceDescriptorHob ( > > > > - EFI_RESOURCE_SYSTEM_MEMORY, > > > > + EFI_RESOURCE_MEMORY_RESERVED, > > > > ( > > > > EFI_RESOURCE_ATTRIBUTE_PRESENT | > > > > EFI_RESOURCE_ATTRIBUTE_INITIALIZED | > > > > + EFI_RESOURCE_ATTRIBUTE_TESTED | > > > > EFI_RESOURCE_ATTRIBUTE_UNCACHEABLE | > > > > EFI_RESOURCE_ATTRIBUTE_WRITE_COMBINEABLE | > > > > EFI_RESOURCE_ATTRIBUTE_WRITE_THROUGH_CACHEABLE | > > > > EFI_RESOURCE_ATTRIBUTE_WRITE_BACK_CACHEABLE > > > > ), > > > > (EFI_PHYSICAL_ADDRESS)(0), > > > > - (UINT64)(0xA0000) > > > > + (UINT64)(0x1000) > > > > + ); > > > > + > > > > + BuildResourceDescriptorHob ( > > > > + EFI_RESOURCE_SYSTEM_MEMORY, > > > > + ( > > > > + EFI_RESOURCE_ATTRIBUTE_PRESENT | > > > > + EFI_RESOURCE_ATTRIBUTE_INITIALIZED | > > > > + EFI_RESOURCE_ATTRIBUTE_UNCACHEABLE | > > > > + EFI_RESOURCE_ATTRIBUTE_WRITE_COMBINEABLE | > > > > + EFI_RESOURCE_ATTRIBUTE_WRITE_THROUGH_CACHEABLE | > > > > + EFI_RESOURCE_ATTRIBUTE_WRITE_BACK_CACHEABLE > > > > + ), > > > > + (EFI_PHYSICAL_ADDRESS)(0x1000), > > > > + (UINT64)(0x9F000) > > > > ); > > > > > > > > BuildResourceDescriptorHob ( > > > > -- > > 2.27.0 > > > > > > -=-=-=-=-=-= > > Groups.io Links: You receive all messages sent to this group. > > > > View/Reply Online (#62229): https://edk2.groups.io/g/devel/message/62229 > > Mute This Topic: https://groups.io/mt/75374752/1781375 > > Group Owner: [email protected] > > Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] > > -=-=-=-=-=-= > > -- *[Marcello Sylvester Bauer]* 9elements Agency GmbH, Kortumstraße 19-21, 44787 Bochum, Germany Email: [DEINE EMAIL ADDRESSE] <https://static.9elements.com/email_signatur.html> Phone: *+49 234 68 94 188 <+492346894188>* Mobile: *+49 1722847618 <+491722847618>* Sitz der Gesellschaft: Bochum Handelsregister: Amtsgericht Bochum, HRB 17519 Geschäftsführung: Sebastian Deutsch, Eray Basar Datenschutzhinweise nach Art. 13 DSGVO <https://9elements.com/privacy> -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#65217): https://edk2.groups.io/g/devel/message/65217 Mute This Topic: https://groups.io/mt/75374752/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
