* exit.c -prerelease-diff - Jan 4
@ 2001-01-04 12:13 Frank Jacobberger
2001-01-04 18:20 ` Linus Torvalds
0 siblings, 1 reply; 2+ messages in thread
From: Frank Jacobberger @ 2001-01-04 12:13 UTC (permalink / raw)
To: linux-kernel
+ current->state = TASK_ZOMBIE;
Why the update on exit.c to include TASK_ZOMBIE?
Thanks,
Frank -
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: exit.c -prerelease-diff - Jan 4
2001-01-04 12:13 exit.c -prerelease-diff - Jan 4 Frank Jacobberger
@ 2001-01-04 18:20 ` Linus Torvalds
0 siblings, 0 replies; 2+ messages in thread
From: Linus Torvalds @ 2001-01-04 18:20 UTC (permalink / raw)
To: linux-kernel
In article <3A5468F0.AC7388AE@xmission.com>,
Frank Jacobberger <f1j@xmission.com> wrote:
>+ current->state = TASK_ZOMBIE;
>
>Why the update on exit.c to include TASK_ZOMBIE?
There's a subtle race in the exit path, where we need to make sure that
"wait4()" does not pick up the process before it is ready to be picked
up.
As the flag for "I'm ready to be picked up" is "TASK_ZOMBIE", we need to
make sure that it maintains serialization. Look at how "exit_code" is
used, for example - as it used to be, we didn't actually have any
guarantees that "exit_code" had even been _written_ yet when wait4()
started using it.
(That's the bug I _know_ about - there may be others here, because the
exit sequence is damn subtle sometimes. So the prerelease-diff fixes at
least that one thing, but I'm still trying checking all the other
exit-paths, things like __exit_mm()/mmput()/swap_out() etc)
Linus
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2001-01-04 18:21 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2001-01-04 12:13 exit.c -prerelease-diff - Jan 4 Frank Jacobberger
2001-01-04 18:20 ` Linus Torvalds
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox