diff options
author | Tejun Heo <tj@kernel.org> | 2011-03-23 05:37:00 -0400 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2011-03-23 05:37:00 -0400 |
commit | 71db5eb99c960e9c30e4b3ed04103c513b6251b5 (patch) | |
tree | 89732a4c7bef3c0740ebe0212fc2b432de2d8609 /kernel/signal.c | |
parent | 9f2bf6513a6cca0b00cbbf67ba6197017cfba548 (diff) |
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) | |||
1781 | } | 1781 | } |
1782 | 1782 | ||
1783 | /* Now we don't run again until woken by SIGCONT or SIGKILL */ | 1783 | /* Now we don't run again until woken by SIGCONT or SIGKILL */ |
1784 | do { | 1784 | schedule(); |
1785 | schedule(); | ||
1786 | } while (try_to_freeze()); | ||
1787 | 1785 | ||
1788 | tracehook_finish_jctl(); | 1786 | tracehook_finish_jctl(); |
1789 | current->exit_code = 0; | 1787 | current->exit_code = 0; |