git: f1b656f14464 - main - vm_fault_busy_sleep(): update comment after addition of allocflags arg
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Mon, 15 Sep 2025 14:12:48 UTC
The branch main has been updated by kib:
URL: https://cgit.FreeBSD.org/src/commit/?id=f1b656f14464c2e3ec4ab2eeade3b00dce4bd459
commit f1b656f14464c2e3ec4ab2eeade3b00dce4bd459
Author: Konstantin Belousov <kib@FreeBSD.org>
AuthorDate: 2025-09-15 00:52:47 +0000
Commit: Konstantin Belousov <kib@FreeBSD.org>
CommitDate: 2025-09-15 14:11:57 +0000
vm_fault_busy_sleep(): update comment after addition of allocflags arg
Reviewed by: markj
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
Differential revision: https://reviews.freebsd.org/D52538
---
sys/vm/vm_fault.c | 9 ++++-----
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/sys/vm/vm_fault.c b/sys/vm/vm_fault.c
index 3dddc7a307d4..524aed2be2ff 100644
--- a/sys/vm/vm_fault.c
+++ b/sys/vm/vm_fault.c
@@ -1456,11 +1456,10 @@ vm_fault_getpages(struct faultstate *fs, int *behindp, int *aheadp)
* and we could end up trying to pagein and pageout the same page
* simultaneously.
*
- * We can theoretically allow the busy case on a read fault if the page
- * is marked valid, but since such pages are typically already pmap'd,
- * putting that special case in might be more effort then it is worth.
- * We cannot under any circumstances mess around with a shared busied
- * page except, perhaps, to pmap it.
+ * We allow the busy case on a read fault if the page is valid. We
+ * cannot under any circumstances mess around with a shared busied
+ * page except, perhaps, to pmap it. This is controlled by the
+ * VM_ALLOC_SBUSY bit in the allocflags argument.
*/
static void
vm_fault_busy_sleep(struct faultstate *fs, int allocflags)