git: 60847070f908 - main - vm: Eliminate a redundant call to vm_reserv_break_all()
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Wed, 05 Jun 2024 17:42:19 UTC
The branch main has been updated by alc:
URL: https://cgit.FreeBSD.org/src/commit/?id=60847070f908c7c5ebb2ea4c851f8b98680fd01a
commit 60847070f908c7c5ebb2ea4c851f8b98680fd01a
Author: Alan Cox <alc@FreeBSD.org>
AuthorDate: 2024-06-05 06:40:20 +0000
Commit: Alan Cox <alc@FreeBSD.org>
CommitDate: 2024-06-05 17:39:47 +0000
vm: Eliminate a redundant call to vm_reserv_break_all()
When vm_object_collapse() was changed in commit 98087a0 to call
vm_object_terminate(), rather than destroying the object directly, its
call to vm_reserv_break_all() should have been removed, as
vm_object_terminate() calls vm_reserv_break_all().
Reviewed by: kib, markj
MFC after: 1 week
Differential Revision: https://reviews.freebsd.org/D45495
---
sys/vm/vm_object.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/sys/vm/vm_object.c b/sys/vm/vm_object.c
index 905df5454355..0af4402938ba 100644
--- a/sys/vm/vm_object.c
+++ b/sys/vm/vm_object.c
@@ -1953,14 +1953,6 @@ vm_object_collapse(vm_object_t object)
*/
vm_object_collapse_scan(object);
-#if VM_NRESERVLEVEL > 0
- /*
- * Break any reservations from backing_object.
- */
- if (__predict_false(!LIST_EMPTY(&backing_object->rvq)))
- vm_reserv_break_all(backing_object);
-#endif
-
/*
* Move the pager from backing_object to object.
*