[PATCH] powerpc/32bit,PREEMPT:Load TI_FLAGS to check NEED_RESCHED

[Jain Priyanka-B32167]

Yes,
Ben is also pointing to same thing.
I will send another patch with changes suggested (with both subject and description modified) which will supersede this patch.

Regards
Priyanka

> -----Original Message-----
> From: tiejun.chen [mailto:tiejun.chen at windriver.com]
> Sent: Monday, May 27, 2013 12:49 PM
> To: Jain Priyanka-B32167
> Cc: linuxppc-dev at lists.ozlabs.org; Wood Scott-B07421
> Subject: Re: [PATCH] powerpc/32bit,PREEMPT:Load TI_FLAGS to check
> NEED_RESCHED
> 
> On 05/27/2013 02:55 PM, Jain Priyanka-B32167 wrote:
> >
> > If we go some more lines up in the same file, the code is
> >
> > resume_kernel:
> >          /* check current_thread_info, _TIF_EMULATE_STACK_STORE */
> >          CURRENT_THREAD_INFO(r9, r1)
> >          lwz     r8,TI_FLAGS(r9)
> >          andis.  r8,r8,_TIF_EMULATE_STACK_STORE at h
> 
> Okay, but could you fix this directly like:
> 
> diff --git a/arch/powerpc/kernel/entry_32.S
> b/arch/powerpc/kernel/entry_32.S index e514de5..4498467 100644
> --- a/arch/powerpc/kernel/entry_32.S
> +++ b/arch/powerpc/kernel/entry_32.S
> @@ -851,7 +851,7 @@ resume_kernel:
>          /* check current_thread_info, _TIF_EMULATE_STACK_STORE */
>          CURRENT_THREAD_INFO(r9, r1)
>          lwz     r8,TI_FLAGS(r9)
> -       andis.  r8,r8,_TIF_EMULATE_STACK_STORE at h
> +       andis.  r0,r8,_TIF_EMULATE_STACK_STORE at h
>          beq+    1f
> 
>          addi    r8,r1,INT_FRAME_SIZE    /* Get the kprobed function
> entry */
> 
> Tiejun