diff options
| author | Tejun Heo <tj@kernel.org> | 2011-03-23 10:37:00 +0100 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2011-03-23 10:37:00 +0100 |
| commit | 71db5eb99c960e9c30e4b3ed04103c513b6251b5 (patch) | |
| tree | 89732a4c7bef3c0740ebe0212fc2b432de2d8609 /kernel/signal.c | |
| parent | ptrace: Remove the extra wake_up_state() from ptrace_detach() (diff) | |
| download | linux-71db5eb99c960e9c30e4b3ed04103c513b6251b5.tar.xz linux-71db5eb99c960e9c30e4b3ed04103c513b6251b5.zip | |
signal: Remove superflous try_to_freeze() loop in do_signal_stop()
do_signal_stop() is used only by get_signal_to_deliver() and after a
successful signal stop, it always calls try_to_freeze(), so the
try_to_freeze() loop around schedule() in do_signal_stop() is
superflous and confusing. Remove it.
Signed-off-by: Tejun Heo <tj@kernel.org>
Acked-by: Rafael J. Wysocki <rjw@sisk.pl>
Acked-by: Oleg Nesterov <oleg@redhat.com>
Acked-by: Roland McGrath <roland@redhat.com>
Diffstat (limited to 'kernel/signal.c')
| -rw-r--r-- | kernel/signal.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/kernel/signal.c b/kernel/signal.c index e26274abf3a9..f4db76986ec1 100644 --- a/kernel/signal.c +++ b/kernel/signal.c @@ -1781,9 +1781,7 @@ static int do_signal_stop(int signr) } /* Now we don't run again until woken by SIGCONT or SIGKILL */ - do { - schedule(); - } while (try_to_freeze()); + schedule(); tracehook_finish_jctl(); current->exit_code = 0; |