[coreboot-gerrit] Change in coreboot[master]: soc/intel/denverton_ns: Enable MC Exception

[Angel Pons (Code Review)]

Angel Pons has submitted this change. ( https://review.coreboot.org/c/coreboot/+/25434 )

Change subject: soc/intel/denverton_ns: Enable MC Exception
......................................................................

soc/intel/denverton_ns: Enable MC Exception

Change-Id: I9773c61d06bb6c68612e498a35b5ad22cd5a8a6e
Signed-off-by: Julien Viard de Galbert 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/25434
Tested-by: build bot (Jenkins) 
Reviewed-by: Angel Pons 
---
M src/soc/intel/denverton_ns/cpu.c
1 file changed, 7 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Angel Pons: Looks good to me, approved

diff --git a/src/soc/intel/denverton_ns/cpu.c b/src/soc/intel/denverton_ns/cpu.c
index 7cee5be..1fd233a 100644
--- a/src/soc/intel/denverton_ns/cpu.c
+++ b/src/soc/intel/denverton_ns/cpu.c
@@ -2,6 +2,7 @@

#include <console/console.h>
 #include <cpu/cpu.h>
+#include <cpu/x86/cr.h>
 #include <cpu/x86/mp.h>
 #include <cpu/x86/msr.h>
 #include <cpu/x86/mtrr.h>
@@ -44,6 +45,12 @@
       of these banks are core vs package scope. For now every CPU clears
       every bank. */
    mca_configure();
+
+	/* TODO install a fallback MC handler for each core in case OS does
+	   not provide one. Is it really needed? */
+
+	/* Enable the machine check exception */
+	write_cr4(read_cr4() | CR4_MCE);
 }

static void denverton_core_init(struct device *cpu)