diff options
author | Eugene Teo <eugeneteo@kernel.sg> | 2007-10-19 02:40:38 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-10-19 14:53:42 -0400 |
commit | 270f722d4d5f94b02fd48eed47e57917ab00a858 (patch) | |
tree | f6f58a0730c8968d2b4b983965b4409761616ab2 /kernel/fork.c | |
parent | d85f50d5e1aa99ab082035f94265847521819e58 (diff) |
Fix tsk->exit_state usage
tsk->exit_state can only be 0, EXIT_ZOMBIE, or EXIT_DEAD. A non-zero test
is the same as tsk->exit_state & (EXIT_ZOMBIE | EXIT_DEAD), so just testing
tsk->exit_state is sufficient.
Signed-off-by: Eugene Teo <eugeneteo@kernel.sg>
Cc: Roland McGrath <roland@redhat.com>
Cc: Oleg Nesterov <oleg@tv-sign.ru>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'kernel/fork.c')
-rw-r--r-- | kernel/fork.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/fork.c b/kernel/fork.c index a794bfcf6003..240aa6601f5b 100644 --- a/kernel/fork.c +++ b/kernel/fork.c | |||
@@ -117,7 +117,7 @@ EXPORT_SYMBOL(free_task); | |||
117 | 117 | ||
118 | void __put_task_struct(struct task_struct *tsk) | 118 | void __put_task_struct(struct task_struct *tsk) |
119 | { | 119 | { |
120 | WARN_ON(!(tsk->exit_state & (EXIT_DEAD | EXIT_ZOMBIE))); | 120 | WARN_ON(!tsk->exit_state); |
121 | WARN_ON(atomic_read(&tsk->usage)); | 121 | WARN_ON(atomic_read(&tsk->usage)); |
122 | WARN_ON(tsk == current); | 122 | WARN_ON(tsk == current); |
123 | 123 | ||