[coreboot] [PATCH] Streamline CPU_ADDR_BITS usage
Scott Duplichan
scott at notabs.org
Mon Oct 4 02:14:36 CEST 2010
]
]But note that the current form is also a bit dangerous. It hardcodes 40bits
]for K8 and 48bits for Fam10h here unconditionally. I don't know if this
]assumption is always correct for all CPUs. Using the correct per-CPU
]CONFIG_CPU_ADDR_BITS_MASK would definately be safer (if this mechanism
]can work here at all). Are we sure there are no K8 systems that support
]CPUs with bits != 40? Are we sure there are no Fam10h CPUs with
]bits != 48 (and that there never will be in the future)?
Getting this info from cpuid is the way to go (It is reliable on AMD
systems). Hard-coding is next best. It is fixed per family:
0Fh 40
10h 48
11h 40
12h 40
13h (no product)
14h 36
15h 48
Hopefully AMD will start supplying coreboot-ready support code, which
will allow coreboot to run exactly the same reference code as every
other AMD BIOS.
Thanks,
Scott
]Uwe.
]--
]http://hermann-uwe.de | http://sigrok.org
]http://randomprojects.org | http://unmaintained-free-software.org
More information about the coreboot
mailing list