Acked-by: Jonas Bonn <[email protected]> ...for OpenRISC, all 4 patches (20-23 in this series).
Should I take these in the OpenRISC tree or how did you want to do? /Jonas On Tue, 2012-02-14 at 11:40 +0000, Matt Fleming wrote: > From: Matt Fleming <[email protected]> > > Instead of open coding the sequence from force_sigsegv() just call > it. This also fixes a bug because we were modifying ka->sa.sa_handler > (which is a copy of sighand->action[]), whereas the intention of the > code was to modify sighand->action[] directly. > > As the original code was working with a copy it had no effect on > signal delivery. > > Acked-by: Oleg Nesterov <[email protected]> > Cc: Jonas Bonn <[email protected]> > Cc: Arnd Bergmann <[email protected]> > Cc: [email protected] > Signed-off-by: Matt Fleming <[email protected]> > --- > arch/openrisc/kernel/signal.c | 4 +--- > 1 files changed, 1 insertions(+), 3 deletions(-) > > diff --git a/arch/openrisc/kernel/signal.c b/arch/openrisc/kernel/signal.c > index 95207ab..53741ba 100644 > --- a/arch/openrisc/kernel/signal.c > +++ b/arch/openrisc/kernel/signal.c > @@ -250,9 +250,7 @@ static void setup_rt_frame(int sig, struct k_sigaction > *ka, siginfo_t *info, > return; > > give_sigsegv: > - if (sig == SIGSEGV) > - ka->sa.sa_handler = SIG_DFL; > - force_sig(SIGSEGV, current); > + force_sigsegv(sig, current); > } > > static inline void -- Jonas Bonn South Pole AB Phone : +46-8-56237100 Anderstorpsvägen 16 Mobile: +46-768-527755 SE-17154 Solna [email protected] www.southpoleconsulting.com _______________________________________________ Linux mailing list [email protected] http://lists.openrisc.net/listinfo/linux
