Don't attempt to wake tasks that aren't blocked.

It's possible for a supervised task to kill and wake its supervising
task then immediately try to wake it again if the supervising task
has joined the supervised. This is the easiest way to prevent that.
This commit is contained in:
Brian Anderson 2010-12-29 00:33:52 -05:00 committed by Graydon Hoare
parent 2ec4325fe9
commit 73fd87baf5
2 changed files with 37 additions and 1 deletions

View File

@ -420,7 +420,7 @@ rust_task::notify_tasks_waiting_to_join() {
delete waiting_task;
} else {
rust_task *task = waiting_task->referent();
if (task->dead() == false) {
if (task->blocked() == true) {
task->wakeup(this);
}
}

View File

@ -0,0 +1,36 @@
// Create a task that is supervised by another task,
// join the supervised task from the supervising task,
// then fail the supervised task. The supervised task
// will kill the supervising task, waking it up. The
// supervising task no longer needs to be wakened when
// the supervised task exits.
fn supervised() {
// Yield to make sure the supervisor joins before we
// fail. This is currently not needed because the supervisor
// runs first, but I can imagine that changing.
yield;
fail;
}
fn supervisor() {
let task t = spawn "supervised" supervised();
join t;
}
fn main() {
// Start the test in another domain so that
// the process doesn't return a failure status as a result
// of the main task being killed.
let task dom2 = spawn thread "supervisor" supervisor();
join dom2;
}
// Local Variables:
// mode: rust;
// fill-column: 78;
// indent-tabs-mode: nil
// c-basic-offset: 4
// buffer-file-coding-system: utf-8-unix
// compile-command: "make -k -C .. 2>&1 | sed -e 's/\\/x\\//x:\\//g'";
// End: