Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43401 )
Change subject: program layout: align to 64 bytes when Boot Guard is enabled ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/43401/1/src/arch/x86/memlayout.ld File src/arch/x86/memlayout.ld:
https://review.coreboot.org/c/coreboot/+/43401/1/src/arch/x86/memlayout.ld@5... PS1, Line 51: /* Each program stage needs to be 64 bytes aligned to be added as FIT entry */ I don't immediately see any SECTIONS declarations following the ALIGN(64) below. Can you improve the commit message with the actual symbols names this affects/fixes.
https://review.coreboot.org/c/coreboot/+/43401/1/src/arch/x86/memlayout.ld@5... PS1, Line 52: #if CONFIG_INTEL_BOOTGUARD CONFIG(INTEL_BOOTGUARD) ?