[coreboot] [PATCH try 2] Reduce duplicate definition in CAR code.

ron minnich rminnich at gmail.com
Wed Oct 6 00:18:09 CEST 2010


It's proven dangerous in the past to cross mtrr settings across vendors.

Which is what you are doing. Then somebody patches, e.g.,
cpu/x86/mtrr.h for some fix to via, and we find out a year later it is
not right for some flavor of AMD. MTRRs have been a rolling headache
for 10 years now.

Sure they should all be the same. Sometimes there are weird issues.

So what I'd prefer, personally: leave the settings in each vendor
file: amd, via, whatever, don't make the common settings in
cpu/x86/mtrr.h. But use your nice macros to set those up.

ron




More information about the coreboot mailing list