d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Avoid calling _PHOLD(p1) with p2's lock held, since _PHOLD()

Browse Source 
may block to swap in p1.  Instead, call _PHOLD earlier, at a
point where the only lock held happens to be p1's.
This commit is contained in:
David Schultz 2004-10-01 05:01:29 +00:00
parent 616b5f90d3
commit 299bc7367d
1 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
sys/kern/kern_fork.c
View File

@ -533,8 +533,14 @@ again:
* p_limit is copy-on-write. Bump its refcount.
*/
p2->p_limit = lim_hold(p1->p_limit);
PROC_UNLOCK(p1);
/*
* We don't need to prevent the parent from being swapped out just yet,
* but this is a convenient point to block to swap it in if needed.
*/
PROC_UNLOCK(p2);
_PHOLD(p1);
PROC_UNLOCK(p1);
/* Bump references to the text vnode (for procfs) */
if (p2->p_textvp)
@ -629,9 +635,8 @@ again:
/*
* This begins the section where we must prevent the parent
* from being swapped.
* from being swapped. We already got a hold on the parent earlier.
*/
_PHOLD(p1);
PROC_UNLOCK(p1);
/*