Re: [PATCH] powerpc: signals: Discard transaction state from signal frames

2016-08-22 Thread Cyril Bur
On Mon, 2016-08-22 at 19:21 +0800, kbuild test robot wrote: > Hi Cyril, > > [auto build test ERROR on powerpc/next] > [also build test ERROR on v4.8-rc3 next-20160822] > [if your patch is applied to the wrong git tree, please drop us a > note to help improve the system] > [Suggest to use

Re: [PATCH] powerpc: signals: Discard transaction state from signal frames

2016-08-22 Thread Cyril Bur
On Sat, 2016-08-20 at 18:03 +0800, Simon Guo wrote: > Hi Cyril, > On Mon, Aug 22, 2016 at 05:32:06PM +1000, Cyril Bur wrote: > > > > diff --git a/arch/powerpc/kernel/signal_32.c > > b/arch/powerpc/kernel/signal_32.c > > index b6aa378..31e4e15 100644 > > --- a/arch/powerpc/kernel/signal_32.c > >

Re: [PATCH] powerpc: signals: Discard transaction state from signal frames

2016-08-22 Thread Simon Guo
Hi Cyril, On Mon, Aug 22, 2016 at 05:32:06PM +1000, Cyril Bur wrote: > diff --git a/arch/powerpc/kernel/signal_32.c b/arch/powerpc/kernel/signal_32.c > index b6aa378..31e4e15 100644 > --- a/arch/powerpc/kernel/signal_32.c > +++ b/arch/powerpc/kernel/signal_32.c > @@ -1226,7 +1226,19 @@ long

Re: [PATCH] powerpc: signals: Discard transaction state from signal frames

2016-08-22 Thread kbuild test robot
Hi Cyril, [auto build test ERROR on powerpc/next] [also build test ERROR on v4.8-rc3 next-20160822] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto for convenience) to record

Re: [PATCH] powerpc: signals: Discard transaction state from signal frames

2016-08-22 Thread Laurent Dufour
On 22/08/2016 09:32, Cyril Bur wrote: > Userspace can begin and suspend a transaction within the signal > handler which means they might enter sys_rt_sigreturn() with the > processor in suspended state. > > sys_rt_sigreturn() wants to restore process context (which may have > been in a

Re: [PATCH] powerpc: signals: Discard transaction state from signal frames

2016-08-22 Thread kbuild test robot
Hi Cyril, [auto build test ERROR on powerpc/next] [also build test ERROR on v4.8-rc3 next-20160819] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto for convenience) to record

Re: [PATCH] powerpc: signals: Discard transaction state from signal frames

2016-08-22 Thread Cyril Bur
On Mon, 2016-08-22 at 17:32 +1000, Cyril Bur wrote: The subject prefix should have contained V2. Sorry. > Userspace can begin and suspend a transaction within the signal > handler which means they might enter sys_rt_sigreturn() with the > processor in suspended state. > > sys_rt_sigreturn()

[PATCH] powerpc: signals: Discard transaction state from signal frames

2016-08-22 Thread Cyril Bur
Userspace can begin and suspend a transaction within the signal handler which means they might enter sys_rt_sigreturn() with the processor in suspended state. sys_rt_sigreturn() wants to restore process context (which may have been in a transaction before signal delivery). To do this it must

Re: [PATCH] powerpc: signals: Discard transaction state from signal frames

2016-08-22 Thread Cyril Bur
On Mon, 2016-08-22 at 17:07 +1000, Michael Neuling wrote: > On Mon, 2016-08-22 at 15:15 +1000, Cyril Bur wrote: > > > > Userspace can begin and suspend a transaction within the signal > > handler which means they might enter sys_rt_sigreturn() with the > > processor in suspended state. > > > >

Re: [PATCH] powerpc: signals: Discard transaction state from signal frames

2016-08-22 Thread Michael Neuling
On Mon, 2016-08-22 at 15:15 +1000, Cyril Bur wrote: > Userspace can begin and suspend a transaction within the signal > handler which means they might enter sys_rt_sigreturn() with the > processor in suspended state. > > sys_rt_sigreturn() wants to restore process context (which may have > been

[PATCH] powerpc: signals: Discard transaction state from signal frames

2016-08-21 Thread Cyril Bur
Userspace can begin and suspend a transaction within the signal handler which means they might enter sys_rt_sigreturn() with the processor in suspended state. sys_rt_sigreturn() wants to restore process context (which may have been in a transaction before signal delivery). To do this it must