Attention is currently required from: Kane Chen. Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63486 )
Change subject: soc/intel/common: Enable rom cache on all CPU threads ......................................................................
Patch Set 2:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/63486/comment/ca1fba5b_dc07e566 PS2, Line 11: However the BSP doesn't wait all threads to finish _x86_setup_mtrrs I think it's better to fix this than to program something temporary on APs too. It's already hard enough to keep track of what is going on, on the BSP ;-)
Maybe you can add another AP call with a NOOP function to make sure that they all completed the previous call?
Patchset:
PS2: Is restoring the temp mtrr also done APs. I thought it didn't.