[PATCH] powerpc/32bit,PREEMPT:Load TI_FLAGS to check NEED_RESCHED
Jain Priyanka-B32167
B32167 at freescale.com
Mon May 27 17:21:54 EST 2013
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
More information about the Linuxppc-dev
mailing list