Re: O2 RM7000 Issues

[Ralf Baechle <ralf@linux-mips.org>]

On Mon, Sep 17, 2007 at 04:20:28PM -0700, David Daney wrote:

Hi David,

> Ralf Baechle wrote:
> 
> >Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
> >
> >diff --git a/arch/mips/kernel/cpu-probe.c b/arch/mips/kernel/cpu-probe.c
> >index f599e79..7ee0cb0 100644
> >--- a/arch/mips/kernel/cpu-probe.c
> >+++ b/arch/mips/kernel/cpu-probe.c
> >@@ -75,6 +75,26 @@ static void r4k_wait_irqoff(void)
> > 	local_irq_enable();
> > }
> > 
> >+/*
> >+ * The RM7000 variant has to handle erratum 38.  The workaround is to not
> >+ * have any pending stores when the WAIT instruction is executed.
> >+ */
> >+static void rm7k_wait_irqoff(void)
> >+{
> >+	local_irq_disable();
> >+	if (!need_resched())
> >+		__asm__(
> >+		"	.set	push		\n"
> >+		"	.set	mips3		\n"
> >+		"	.set	noat		\n"
> >+		"	mfc0	$1, $12		\n"
> >+		"	sync			\n"
> >+		"	mtc0	$1, $12		\n"
> >+		"	wait			\n"
> >+		"	.set	pop		\n");
> >+	local_irq_enable();
> >+}
> >+
> 
> Technically, shouldn't that __asm__ be volatile?

Gcc won't delete this asm because it has no return value that is it will
treat it like a volatile asm.

  Ralf