Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36908 )
Change subject: lib/fmap.c: Properly handle cache for !C_ENVIRONMENT_BOOTBLOCK ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/36908/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/36908/1//COMMIT_MSG@9 PS1, Line 9: first stage : accessing FMAP is romstage
Why wouldn't it be accessed from bootblock? I don't understand the reasoning for this assertion.
I didn't read 'without' properly in the first part of the sentence.
https://review.coreboot.org/c/coreboot/+/36908/1/src/lib/fmap.c File src/lib/fmap.c:
https://review.coreboot.org/c/coreboot/+/36908/1/src/lib/fmap.c@81 PS1, Line 81: if (!ENV_BOOTBLOCK && CONFIG(C_ENVIRONMENT_BOOTBLOCK)) { Looks like this will result in no signature checking for the !c_env_bootblock platforms post romstage. Do we need to expand this check to deal with both cases?
((CONFIG(C_ENVIRONMENT_BOOTBLOCK) && !ENV_BOOTBLOCK) || (!CONFIG(C_ENVIRONMENT_BOOTBLOCK) && !ENV_ROMSTAGE))