diff options
| author | Robert Morris <[email protected]> | 2020-11-05 07:32:10 -0500 |
|---|---|---|
| committer | Robert Morris <[email protected]> | 2020-11-05 07:32:10 -0500 |
| commit | bc943c230960c414fd43b93317c3825f1a99c611 (patch) | |
| tree | a2b89fe071e83d7e9b6d49d582bee8301e40dc9c /kernel/proc.c | |
| parent | 7dea4b93c82354d5a24e8882860af5a1d92282e6 (diff) | |
| download | xv6-labs-bc943c230960c414fd43b93317c3825f1a99c611.tar.gz xv6-labs-bc943c230960c414fd43b93317c3825f1a99c611.tar.bz2 xv6-labs-bc943c230960c414fd43b93317c3825f1a99c611.zip | |
don't over-lock in exit()
Diffstat (limited to 'kernel/proc.c')
| -rw-r--r-- | kernel/proc.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/kernel/proc.c b/kernel/proc.c index 54c965d..bd77dad 100644 --- a/kernel/proc.c +++ b/kernel/proc.c @@ -358,14 +358,14 @@ exit(int status) p->cwd = 0; acquire(&proc_tree_lock); - - acquire(&p->lock); // Give any children to init. reparent(p); // Parent might be sleeping in wait(). wakeup(p->parent); + + acquire(&p->lock); p->xstate = status; p->state = ZOMBIE; @@ -393,7 +393,9 @@ wait(uint64 addr) havekids = 0; for(np = proc; np < &proc[NPROC]; np++){ if(np->parent == p){ + // make sure the child isn't still in exit() or swtch(). acquire(&np->lock); + havekids = 1; if(np->state == ZOMBIE){ // Found one. |