Attention is currently required from: Christopher Meis, Patrick Rudolph. Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55513 )
Change subject: intel/fit - Prepare for BootGuard support ......................................................................
Patch Set 1:
(1 comment)
File src/cpu/intel/fit/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/55513/comment/86c37572_ab74532c PS1, Line 16: intel_fit-position := 0xffff0000 # Set position for BootGuard
Where does this requirement come from? reading #558294 I don't see the need for that.
I'm looking at 557867. The requirements are: - FIT, KM, BPM needs to be in the top 4M - ACM in top 2M - All IBBs need to be in the top 4M
Maybe we can get rid of this and just use a small RO region with VBOOT? The tooling has to warn about ill configurations though.
-
Arthur Heymans (Code Review)