Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34991 )
Change subject: arch/x86: Include stack_top region into postcar_frame ......................................................................
Patch Set 1:
(1 comment)
I'm not sure we want to go down this route because of how painful all the various corner cases are when wanting WB cache type for increasing loading speed ramstage. microarchitecture details are making consistency hard.
https://review.coreboot.org/c/coreboot/+/34991/1/src/arch/x86/postcar_loader... File src/arch/x86/postcar_loader.c:
https://review.coreboot.org/c/coreboot/+/34991/1/src/arch/x86/postcar_loader... PS1, Line 69: This absolutely needs to stay.