Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36000 )
Change subject: vendorcode/amd/agesa/f12/Proc/CPU: Fix set MTRR6 and MTRR7 to default ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/36000/2/src/vendorcode/amd/agesa/f1... File src/vendorcode/amd/agesa/f12/Proc/CPU/cahalt.c:
https://review.coreboot.org/c/coreboot/+/36000/2/src/vendorcode/amd/agesa/f1... PS2, Line 152: for (msrno = 0x20F; msrno >= 0x20C; msrno--) With more or less unmaintained code I think it is better to test this on hardware as fixing some bugs (no-ops) may unravel others...
BTW to any AGESA experts: this code seems to be run on AP's. Does coreboot redo this later/earlier on?