The branch main has been updated by alc:

URL: 
https://cgit.FreeBSD.org/src/commit/?id=60847070f908c7c5ebb2ea4c851f8b98680fd01a

commit 60847070f908c7c5ebb2ea4c851f8b98680fd01a
Author:     Alan Cox <a...@freebsd.org>
AuthorDate: 2024-06-05 06:40:20 +0000
Commit:     Alan Cox <a...@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.
                         *

Reply via email to