git: d7441aa9b09a - main - vm_object: use iterator in page_noreuse()

From: Doug Moore <dougm_at_FreeBSD.org>
Date: Mon, 21 Apr 2025 03:10:40 UTC
The branch main has been updated by dougm:

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

commit d7441aa9b09ae537c943c422df43281f087d5757
Author:     Doug Moore <dougm@FreeBSD.org>
AuthorDate: 2025-04-21 03:08:53 +0000
Commit:     Doug Moore <dougm@FreeBSD.org>
CommitDate: 2025-04-21 03:10:31 +0000

    vm_object: use iterator in page_noreuse()
    
    Use an iterator instead of tailq pointers for the loop in
    vm_object_page_noreuse().
    
    Reviewed by:    kib
    Differential Revision:  https://reviews.freebsd.org/D49935
---
 sys/vm/vm_object.c | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/sys/vm/vm_object.c b/sys/vm/vm_object.c
index bb9cb6b6f554..39f9f0a50d46 100644
--- a/sys/vm/vm_object.c
+++ b/sys/vm/vm_object.c
@@ -2106,23 +2106,18 @@ wired:
 void
 vm_object_page_noreuse(vm_object_t object, vm_pindex_t start, vm_pindex_t end)
 {
-	vm_page_t p, next;
+	struct pctrie_iter pages;
+	vm_page_t p;
 
 	VM_OBJECT_ASSERT_LOCKED(object);
 	KASSERT((object->flags & (OBJ_FICTITIOUS | OBJ_UNMANAGED)) == 0,
 	    ("vm_object_page_noreuse: illegal object %p", object));
 	if (object->resident_page_count == 0)
 		return;
-	p = vm_page_find_least(object, start);
 
-	/*
-	 * Here, the variable "p" is either (1) the page with the least pindex
-	 * greater than or equal to the parameter "start" or (2) NULL. 
-	 */
-	for (; p != NULL && (p->pindex < end || end == 0); p = next) {
-		next = TAILQ_NEXT(p, listq);
+	vm_page_iter_limit_init(&pages, object, end);
+	VM_RADIX_FOREACH_FROM(p, &pages, start)
 		vm_page_deactivate_noreuse(p);
-	}
 }
 
 /*