Re: [PATCH] vr41xx: fix problem with vr41xx_cpu_wait

[Ricardo Mendoza <ricmm@gentoo.org>]

On Wed, Aug 06, 2008 at 10:30:33PM +0900, Atsushi Nemoto wrote:
 
> > 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.

I see, I hadn't read that thread before. Yoichi's rev of the patch
should go in but we should put some effort in finding a better route for
CPUs that are not forcefully aware by hardware of interrupts in their
wait call.

Was some work ever done on Ralf's idea in the "WAIT vs. tickless kernel"
ML thread? Regarding patching the return from exception code to contain
a check for this particular loop.


     Ricardo