diff options
| author | Eugene Teo <eugeneteo@kernel.sg> | 2007-10-18 23:40:38 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-10-19 11:53:42 -0700 |
| commit | 270f722d4d5f94b02fd48eed47e57917ab00a858 (patch) | |
| tree | f6f58a0730c8968d2b4b983965b4409761616ab2 /kernel/sched.c | |
| parent | d85f50d5e1aa99ab082035f94265847521819e58 (diff) | |
| download | lwn-270f722d4d5f94b02fd48eed47e57917ab00a858.tar.gz lwn-270f722d4d5f94b02fd48eed47e57917ab00a858.zip | |
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/sched.c')
| -rw-r--r-- | kernel/sched.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/sched.c b/kernel/sched.c index 72a809a54d5b..9d458504e3a6 100644 --- a/kernel/sched.c +++ b/kernel/sched.c @@ -5279,7 +5279,7 @@ static void migrate_dead(unsigned int dead_cpu, struct task_struct *p) struct rq *rq = cpu_rq(dead_cpu); /* Must be exiting, otherwise would be on tasklist. */ - BUG_ON(p->exit_state != EXIT_ZOMBIE && p->exit_state != EXIT_DEAD); + BUG_ON(!p->exit_state); /* Cannot have done final schedule yet: would have vanished. */ BUG_ON(p->state == TASK_DEAD); |