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
This commit is contained in:
Alan Cox 2024-06-05 01:40:20 -05:00
parent 56b822a17c
commit 60847070f9

View File

@ -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.
*