git: 1ee4b5fd824a - main - fork: Drop an uneeded PHOLD/PRELE pair
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Sat, 06 Jun 2026 13:03:50 UTC
The branch main has been updated by markj:
URL: https://cgit.FreeBSD.org/src/commit/?id=1ee4b5fd824aa93f9a20df6bd5dad50c7e32fe05
commit 1ee4b5fd824aa93f9a20df6bd5dad50c7e32fe05
Author: Mark Johnston <markj@FreeBSD.org>
AuthorDate: 2026-06-06 12:52:15 +0000
Commit: Mark Johnston <markj@FreeBSD.org>
CommitDate: 2026-06-06 12:52:15 +0000
fork: Drop an uneeded PHOLD/PRELE pair
Support for swapping out kernel stacks was removed, so the PHOLD has no
purpose. (And even before that, it's not clear why a swapout here would
have been problematic.)
Reviewed by: kib
MFC after: 1 week
Differential Revision: https://reviews.freebsd.org/D57486
---
sys/kern/kern_fork.c | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/sys/kern/kern_fork.c b/sys/kern/kern_fork.c
index 36647ae44f08..75f8413e5f36 100644
--- a/sys/kern/kern_fork.c
+++ b/sys/kern/kern_fork.c
@@ -675,11 +675,6 @@ do_fork(struct thread *td, struct fork_req *fr, struct proc *p2, struct thread *
callout_init_mtx(&p2->p_itcallout, &p2->p_mtx, 0);
- /*
- * This begins the section where we must prevent the parent
- * from being swapped.
- */
- _PHOLD(p1);
PROC_UNLOCK(p1);
/*
@@ -786,10 +781,6 @@ do_fork(struct thread *td, struct fork_req *fr, struct proc *p2, struct thread *
*/
knote_fork(p1->p_klist, p2->p_pid);
- /*
- * Now can be swapped.
- */
- _PRELE(p1);
PROC_UNLOCK(p1);
SDT_PROBE3(proc, , , create, p2, p1, fr->fr_flags);