Re: [PATCH 1/3] Loongson-2F: Flush the branch target history suchas BTB and RAS

From: Wu Zhangjin
Date: Wed Mar 10 2010 - 10:30:41 EST


Hi,

Thanks very much for your careful review, will resend a new one later.

Regards,
Wu Zhangjin

On Wed, 2010-03-10 at 13:44 +0300, Sergei Shtylyov wrote:
[...]
> > diff --git a/arch/mips/include/asm/stackframe.h b/arch/mips/include/asm/stackframe.h
> > index 3b6da33..b84cfda 100644
> > --- a/arch/mips/include/asm/stackframe.h
> > +++ b/arch/mips/include/asm/stackframe.h
> > @@ -121,6 +121,25 @@
> > .endm
> > #else
> > .macro get_saved_sp /* Uniprocessor variation */
> > +#ifdef CONFIG_CPU_LOONGSON2F
> > + /*
> > + * Clear BTB(branch target buffer), forbid RAS(row address
> > + * strobe)
>
> No spaces before the left paren...
>
> > to workaround the Out-of-oder Issue in Loongson2F
> > + * via it's diagnostic register.
> >
>
> Only "its".
>
> > + */
> > + move k0, ra
> >
>
> Operands misalined...
>
> WBR, Sergei
>
>


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/