Re: [PATCH][RT][PPC64] Fix a previously reverted "fix"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



  Hi Chirag,

On Thu, 10 Jul 2008 22:34:56 +0530 Chirag Jog <chirag@xxxxxxxxxxxxxxxxxx> wrote:

> 
> This patch reintroduces a "fix" that got reverted.
> Here was the original patch http://lkml.org/lkml/2007/5/22/133
> 
> Here is the new patch
> This patch also fixes OOPS reported here: http://lkml.org/lkml/2008/6/19/146
> 
> From tsutomu.owa@xxxxxxxxxxxxx
> Signed-Off-By: Chirag <chirag@xxxxxxxxxxxxxxxxxx>

  This indeed fixes the oops we get with matrix_mult, good job.

  You can add my
 
  Tested-by: Sebastien Dugue <sebastien.dugue@xxxxxxxx>

  Sebastien.


> 
> 
> 
> --- linux-2.6.25.8-rt7/arch/powerpc/kernel/entry_64.S	2008-07-10 16:37:17.000000000 +0000
> +++ linux-2.6.25.8-rt7.mod/arch/powerpc/kernel/entry_64.S	2008-07-10 07:58:56.000000000 +0000
> @@ -579,14 +579,9 @@ do_work:
>  	cmpdi	r0,0
>  	crandc	eq,cr1*4+eq,eq
>  	bne	restore
> -	/* here we are preempting the current task */
>  1:
> -	li	r0,1
> -	stb	r0,PACASOFTIRQEN(r13)
> -	stb	r0,PACAHARDIRQEN(r13)
> -	ori	r10,r10,MSR_EE
> -	mtmsrd	r10,1		/* reenable interrupts */
> -	bl	.preempt_schedule
> +	/* preempt_schedule_irq() expects interrupts disabled. */
> +	bl	.preempt_schedule_irq
>  	mfmsr	r10
>  	clrrdi	r9,r1,THREAD_SHIFT
>  	rldicl	r10,r10,48,1	/* disable interrupts again */
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [RT Stable]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]

  Powered by Linux