<html><head><meta name="color-scheme" content="light dark"></head><body><pre style="word-wrap: break-word; white-space: pre-wrap;">
From: Paul Mackerras &lt;paulus@samba.org&gt;

This patch is from Jake Moilanen &lt;moilanen@austin.ibm.com&gt;.

Log machine check errors to error log and NVRAM.

Signed-off-by: Dave Altobelli &lt;dalto@austin.ibm.com&gt;
Signed-off-by: Jake Moilanen &lt;moilanen@austin.ibm.com&gt;
Signed-off-by: Paul Mackerras &lt;paulus@samba.org&gt;
Signed-off-by: Andrew Morton &lt;akpm@osdl.org&gt;
---

 25-akpm/arch/ppc64/kernel/ras.c |   46 ++++++++++++++++++++++++++--------------
 1 files changed, 31 insertions(+), 15 deletions(-)

diff -puN arch/ppc64/kernel/ras.c~ppc64-log-machine-check-errors-to-error-log-and-nvram arch/ppc64/kernel/ras.c
--- 25/arch/ppc64/kernel/ras.c~ppc64-log-machine-check-errors-to-error-log-and-nvram	2005-01-05 15:36:42.596110440 -0800
+++ 25-akpm/arch/ppc64/kernel/ras.c	2005-01-05 15:36:42.600109832 -0800
@@ -55,9 +55,13 @@
 static unsigned char ras_log_buf[RTAS_ERROR_LOG_MAX];
 static spinlock_t ras_log_buf_lock = SPIN_LOCK_UNLOCKED;
 
+char mce_data_buf[RTAS_ERROR_LOG_MAX]
+;
 /* This is true if we are using the firmware NMI handler (typically LPAR) */
 extern int fwnmi_active;
 
+extern void _exception(int signr, struct pt_regs *regs, int code, unsigned long addr);
+
 static int ras_get_sensor_state_token;
 static int ras_check_exception_token;
 
@@ -242,6 +246,13 @@ ras_error_interrupt(int irq, void *dev_i
  * FWNMI vectors.  The pt_regs' r3 will be updated to reflect
  * the actual r3 if possible, and a ptr to the error log entry
  * will be returned if found.
+ *
+ * The mce_data_buf does not have any locks or protection around it,
+ * if a second machine check comes in, or a system reset is done
+ * before we have logged the error, then we will get corruption in the
+ * error log.  This is preferable over holding off on calling
+ * ibm,nmi-interlock which would result in us checkstopping if a
+ * second machine check did come in.
  */
 static struct rtas_error_log *fwnmi_get_errinfo(struct pt_regs *regs)
 {
@@ -253,7 +264,9 @@ static struct rtas_error_log *fwnmi_get_
 	    (errdata &gt;= rtas.base &amp;&amp; errdata &lt; rtas.base + rtas.size - 16)) {
 		savep = __va(errdata);
 		regs-&gt;gpr[3] = savep[0];	/* restore original r3 */
-		errhdr = (struct rtas_error_log *)(savep + 1);
+		memset(mce_data_buf, 0, RTAS_ERROR_LOG_MAX);
+		memcpy(mce_data_buf, (char *)(savep + 1), RTAS_ERROR_LOG_MAX);
+		errhdr = (struct rtas_error_log *)mce_data_buf;
 	} else {
 		printk("FWNMI: corrupt r3\n");
 	}
@@ -291,26 +304,31 @@ void pSeries_system_reset_exception(stru
  * Return 1 if corrected (or delivered a signal).
  * Return 0 if there is nothing we can do.
  */
-static int recover_mce(struct pt_regs *regs, struct rtas_error_log err)
+static int recover_mce(struct pt_regs *regs, struct rtas_error_log * err)
 {
-	if (err.disposition == RTAS_DISP_FULLY_RECOVERED) {
+	int nonfatal = 0;
+
+	if (err-&gt;disposition == RTAS_DISP_FULLY_RECOVERED) {
 		/* Platform corrected itself */
-		return 1;
+		nonfatal = 1;
 	} else if ((regs-&gt;msr &amp; MSR_RI) &amp;&amp;
 		   user_mode(regs) &amp;&amp;
-		   err.severity == RTAS_SEVERITY_ERROR_SYNC &amp;&amp;
-		   err.disposition == RTAS_DISP_NOT_RECOVERED &amp;&amp;
-		   err.target == RTAS_TARGET_MEMORY &amp;&amp;
-		   err.type == RTAS_TYPE_ECC_UNCORR &amp;&amp;
+		   err-&gt;severity == RTAS_SEVERITY_ERROR_SYNC &amp;&amp;
+		   err-&gt;disposition == RTAS_DISP_NOT_RECOVERED &amp;&amp;
+		   err-&gt;target == RTAS_TARGET_MEMORY &amp;&amp;
+		   err-&gt;type == RTAS_TYPE_ECC_UNCORR &amp;&amp;
 		   !(current-&gt;pid == 0 || current-&gt;pid == 1)) {
 		/* Kill off a user process with an ECC error */
 		printk(KERN_ERR "MCE: uncorrectable ecc error for pid %d\n",
 		       current-&gt;pid);
 		/* XXX something better for ECC error? */
 		_exception(SIGBUS, regs, BUS_ADRERR, regs-&gt;nip);
-		return 1;
+		nonfatal = 1;
 	}
-	return 0;
+
+ 	log_error((char *)err, ERR_TYPE_RTAS_LOG, !nonfatal);
+
+	return nonfatal;
 }
 
 /*
@@ -325,14 +343,12 @@ static int recover_mce(struct pt_regs *r
  */
 int pSeries_machine_check_exception(struct pt_regs *regs)
 {
-	struct rtas_error_log err, *errp;
+	struct rtas_error_log *errp;
 
 	if (fwnmi_active) {
 		errp = fwnmi_get_errinfo(regs);
-		if (errp)
-			err = *errp;
-		fwnmi_release_errinfo();	/* frees errp */
-		if (errp &amp;&amp; recover_mce(regs, err))
+		fwnmi_release_errinfo();
+		if (errp &amp;&amp; recover_mce(regs, errp))
 			return 1;
 	}
 
_
</pre></body></html>