[Top] [All Lists]

Re: [PATCH] vr41xx: fix problem with vr41xx_cpu_wait

Subject: Re: [PATCH] vr41xx: fix problem with vr41xx_cpu_wait
From: Atsushi Nemoto <>
Date: Wed, 06 Aug 2008 22:30:33 +0900 (JST)
In-reply-to: <>
Original-recipient: rfc822;
References: <> <> <>
On Wed, 6 Aug 2008 13:42:13 +0900, Yoichi Yuasa <> 
> > 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

<Prev in Thread] Current Thread [Next in Thread>