Re: [PATCH] vr41xx: fix problem with vr41xx_cpu_wait

[Atsushi Nemoto <anemo@mba.ocn.ne.jp>]

On Wed, 6 Aug 2008 13:42:13 +0900, Yoichi Yuasa <yoichi_yuasa@tripeaks.co.jp> 
wrote:
> > Just now I got my hands on the manual, I can see that the standby
> > instruction sets IE bit to 1 but only on Vr4131 and Vr4181A cores, all
> > others (such as my Vr4121) need to have interrupts enabled before going
> > into standby.
> > 
> > The patch will make it work on all Vr4100 derivates, or we could also
> > add code to build the function depending on CPU type. What do you think?
> 
> local_irq_disable() is included in the sample code on the User's Manul. 
> I think the following patch is good way of this.
...
> +static void old_vr41xx_cpu_wait(void)
> +{
> +     __asm__("standby;\n");
> +}

Then, old vr41 CPUs have potential latency problem as like as other
CPUs with WAIT instruction.

Please refer "WAIT vs. tickless kernel" thread on linux-mips ML
archive for details.

I don't complain about this patch itself.
---
Atsushi Nemoto