Subject: Re: [patch 5/6] PS3: Fix system slowdown

From: Benjamin Herrenschmidt <[email-masked]>
Date: Tue, 01 May 2007 15:31:06 +1000

On Mon, 2007-04-30 at 14:01 -0700, Geoff Levand wrote:
> From: Takao Shinohara <[email-masked]>
> 
> The PS3 HV will deliver soft-disabled interrupts at the next HV call or
> interrupt.  Add an HV call to local_irq_restore() to force the timely
> delivery of any pending interrupts.
> 
> This fixes the system slowdown bug reported here
> http://bugzilla.kernel.org/show_bug.cgi?id=8260
> 
> From: Takao Shinohara <[email-masked]>
> Signed-off-by: Geoff Levand <[email-masked]>

Acked-by: Benjamin Herrenschmidt <[email-masked]>

Note: It would be nicer to do the HV call from assembly, using a
firmware feature section, when the interrupt actually happens. The main
reason I haven't done a patch yet is that it happens in an exception
vectors with only few registers available nad the HV call will clobber a
lot more than we have saved.

Ben.

> ---
> 
> I made a request to change the behavior of the lv1 hypervisor to better
> support the way we do soft-disable on Linux, but it is still not been
> decided if and when that support will be done.
> 
>  arch/powerpc/kernel/irq.c |   11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> --- ps3-linux-dev.orig/arch/powerpc/kernel/irq.c
> +++ ps3-linux-dev/arch/powerpc/kernel/irq.c
> @@ -67,6 +67,7 @@
>  #ifdef CONFIG_PPC64
>  #include <asm/paca.h>
>  #include <asm/firmware.h>
> +#include <asm/lv1call.h>
>  #endif
>  
>  int __irq_offset_value;
> @@ -162,6 +163,16 @@ void local_irq_restore(unsigned long en)
>  	local_paca->hard_enabled = en;
>  	if ((int)mfspr(SPRN_DEC) < 0)
>  		mtspr(SPRN_DEC, 1);
> +
> +	/*
> +	 * Force the delivery of pending soft-disabled interrupts on PS3.
> +	 * Any HV call will have this side effect.
> +	 */
> +	if (firmware_has_feature(FW_FEATURE_PS3_LV1)) {
> +		u64 tmp;
> +		lv1_get_version_info(&tmp);
> +	}
> +
>  	hard_irq_enable();
>  }
>  #endif /* CONFIG_PPC64 */
>