[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1268234645.19976.1.camel@falcon>
Date: Wed, 10 Mar 2010 23:24:04 +0800
From: Wu Zhangjin <wuzhangjin@...il.com>
To: Sergei Shtylyov <sshtylyov@...sta.com>
Cc: Ralf Baechle <ralf@...ux-mips.org>, Greg KH <gregkh@...e.de>,
linux-mips@...ux-mips.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/3] Loongson-2F: Flush the branch target history such
as BTB and RAS
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@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists