Re: [PATCH] ____call_usermodehelper: don't flush_signals()

2007-02-27 Thread Oleg Nesterov
On 02/27, Rusty Russell wrote: > > On Tue, 2007-02-27 at 00:53 +0300, Oleg Nesterov wrote: > > call_usermodehelper() has no reason for flush_signals(). It is a fresh > > forked process which is going to exec a user-space application or exit on > > failure. > > And the flush_signal_handlers() c

Re: [PATCH] ____call_usermodehelper: don't flush_signals()

2007-02-26 Thread Rusty Russell
On Tue, 2007-02-27 at 00:53 +0300, Oleg Nesterov wrote: > call_usermodehelper() has no reason for flush_signals(). It is a fresh > forked process which is going to exec a user-space application or exit on > failure. And the flush_signal_handlers() call? Your patch looks correct; this code was

[PATCH] ____call_usermodehelper: don't flush_signals()

2007-02-26 Thread Oleg Nesterov
call_usermodehelper() has no reason for flush_signals(). It is a fresh forked process which is going to exec a user-space application or exit on failure. Signed-off-by: Oleg Nesterov <[EMAIL PROTECTED]> --- WQ/kernel/kmod.c~ 2007-02-18 22:56:49.0 +0300 +++ WQ/kernel/kmod.c2007-0