Attention is currently required from: Anjaneya "Reddy" Chagam, Jonathan Zhang, Johnny Lin, Arthur Heymans, Morgan Jang. Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55046 )
Change subject: arch/x86/bootblock.ld: Align the bottom of the bootblock to 64 bytes ......................................................................
Patch Set 1:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/55046/comment/cf8d23f5_a729670f PS1, Line 11: - Bootguard/CBnT requires a 64 byte alignment I thought reading comments once that Bootquard needed fixed bootblock size?
https://review.coreboot.org/c/coreboot/+/55046/comment/dc6c8d11_683dc460 PS1, Line 12: - It can reduce the bootblock footprint as the size is aligned upwards Is potential of 48 bytes saved worth mentioning?
File src/arch/x86/bootblock.ld:
https://review.coreboot.org/c/coreboot/+/55046/comment/5287a705_50dc614f PS1, Line 32: PROGRAM_SZ = SIZEOF(.text) + 512; There is more potential to save size by decreasing from 512. But one value will not work for all platforms.