summaryrefslogtreecommitdiffstats
path: root/src/soc/amd
diff options
context:
space:
mode:
authorMarshall Dawson <marshall.dawson@scarletltd.com>2018-08-05 10:42:17 -0600
committerMartin Roth <martinroth@google.com>2018-08-08 17:52:31 +0000
commit74473ec65b2f003c48ed2c3ea101a9db4d087b15 (patch)
tree97950769265d4448cee6cc450f34c897fa8204b2 /src/soc/amd
parentf9a63c0a848699bf96adeba8d3faab573a920ff1 (diff)
downloadcoreboot-74473ec65b2f003c48ed2c3ea101a9db4d087b15.tar.gz
coreboot-74473ec65b2f003c48ed2c3ea101a9db4d087b15.tar.bz2
coreboot-74473ec65b2f003c48ed2c3ea101a9db4d087b15.zip
amd/stoneyridge: Dump MCA registers
Add a function to provide a rudimentary dump of the Machine Check Architecture registers. These values survive a warm reset. BUG=b:65445599 TEST=Verify on a Grunt having propensity for #MC errors Change-Id: Ib6875cabe3041e65c811d8b2232f7ac6bedd1a02 Signed-off-by: Marshall Dawson <marshalldawson3rd@gmail.com> Reviewed-on: https://review.coreboot.org/27926 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/soc/amd')
-rw-r--r--src/soc/amd/stoneyridge/cpu.c51
1 files changed, 47 insertions, 4 deletions
diff --git a/src/soc/amd/stoneyridge/cpu.c b/src/soc/amd/stoneyridge/cpu.c
index 52b1c9cbb165..09543b6b157e 100644
--- a/src/soc/amd/stoneyridge/cpu.c
+++ b/src/soc/amd/stoneyridge/cpu.c
@@ -117,22 +117,65 @@ void stoney_init_cpus(struct device *dev)
set_warm_reset_flag();
}
-static void model_15_init(struct device *dev)
-{
- printk(BIOS_DEBUG, "Model 15 Init.\n");
+static const char *const mca_bank_name[] = {
+ "Load-store unit",
+ "Instruction fetch unit",
+ "Combined unit",
+ "Reserved",
+ "Northbridge",
+ "Execution unit",
+ "Floating point unit"
+};
+static void check_mca(void)
+{
int i;
msr_t msr;
int num_banks;
- /* zero the machine check error status registers */
msr = rdmsr(MCG_CAP);
num_banks = msr.lo & MCA_BANKS_MASK;
+
+ if (is_warm_reset()) {
+ for (i = 0 ; i < num_banks ; i++) {
+ if (i == 3) /* Reserved in Family 15h */
+ continue;
+
+ msr = rdmsr(MC0_STATUS + (i * 4));
+ if (msr.hi || msr.lo) {
+ int core = cpuid_ebx(1) >> 24;
+
+ printk(BIOS_WARNING, "#MC Error: core %d, bank %d %s\n",
+ core, i, mca_bank_name[i]);
+
+ printk(BIOS_WARNING, " MC%d_STATUS = %08x_%08x\n",
+ i, msr.hi, msr.lo);
+ msr = rdmsr(MC0_ADDR + (i * 4));
+ printk(BIOS_WARNING, " MC%d_ADDR = %08x_%08x\n",
+ i, msr.hi, msr.lo);
+ msr = rdmsr(MC0_MISC + (i * 4));
+ printk(BIOS_WARNING, " MC%d_MISC = %08x_%08x\n",
+ i, msr.hi, msr.lo);
+ msr = rdmsr(MC0_CTL + (i * 4));
+ printk(BIOS_WARNING, " MC%d_CTL = %08x_%08x\n",
+ i, msr.hi, msr.lo);
+ msr = rdmsr(MC0_CTL_MASK + i);
+ printk(BIOS_WARNING, " MC%d_CTL_MASK = %08x_%08x\n",
+ i, msr.hi, msr.lo);
+ }
+ }
+ }
+
+ /* zero the machine check error status registers */
msr.lo = 0;
msr.hi = 0;
for (i = 0 ; i < num_banks ; i++)
wrmsr(MC0_STATUS + (i * 4), msr);
+}
+static void model_15_init(struct device *dev)
+{
+ check_mca();
setup_lapic();
}