git: d7441aa9b09a - main - vm_object: use iterator in page_noreuse()
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
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); - } } /*