Re: [PATCH] PPC64: lockfix for rtas error log

[linas@austin.ibm.com]

[-- Attachment #1: Type: text/plain, Size: 781 bytes --]

On Wed, Jun 30, 2004 at 09:27:09PM +1000, Paul Mackerras wrote:
> Linas,
> 
> > This patch moves the location of a lock in order to protect
> > the contents of a buffer until it has been copied to its final
> > destination. Prior to this, a race existed whereby the buffer
> > could be filled even while it was being emptied.
> 
> Given that log_error() seems to be a no-op at the moment AFAICT, and

!!
Too many kernel trees.  Someone must have whacked the ameslab tree
by accident, or on purpose, because they got sick of seeing rtas 
messages.  I don't find the RTAS messages to be pleasent, but simply
whacking them is not the right solution.  The following diff is 
between an older tree and the current tree.  If you could re-add 
these lines, that would be great.

--linas


[-- Attachment #2: chrp-setup.patch --]
[-- Type: text/plain, Size: 491 bytes --]

--- ameslab-2.6.7-25june/arch/ppc64/kernel/chrp_setup.c	2004-06-29 17:04:30.000000000 -0500
+++ linux-2.6.5-20040514/arch/ppc64/kernel/chrp_setup.c	2004-06-29 17:28:48.000000000 -0500
@@ -243,6 +243,12 @@ chrp_init(unsigned long r3, unsigned lon
 		ppc_md.get_irq        = xics_get_irq;
 	}
 
+#ifdef CONFIG_PPC_PSERIES
+	ppc_md.log_error = pSeries_log_error;
+#else
+	ppc_md.log_error = NULL;
+#endif
+
 	ppc_md.init           = chrp_init2;
 
 	ppc_md.pcibios_fixup  = pSeries_final_fixup;