git: a114891356c4 - main - vm_page: drop find_least()

From: Doug Moore <dougm_at_FreeBSD.org>
Date: Sat, 26 Apr 2025 18:12:05 UTC
The branch main has been updated by dougm:

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

commit a114891356c4912c989f06d408c18bb4dee12a9a
Author:     Doug Moore <dougm@FreeBSD.org>
AuthorDate: 2025-04-26 18:09:24 +0000
Commit:     Doug Moore <dougm@FreeBSD.org>
CommitDate: 2025-04-26 18:09:24 +0000

    vm_page: drop find_least()
    
    Remove the function vm_page_find_least and in the place it is used,
    replace it with vm_radix_lookup_ge().
    
    Reviewed by:    alc
    Differential Revision:  https://reviews.freebsd.org/D50042
---
 sys/kern/kern_proc.c |  3 ++-
 sys/vm/vm_page.c     | 19 -------------------
 sys/vm/vm_page.h     |  1 -
 3 files changed, 2 insertions(+), 21 deletions(-)

diff --git a/sys/kern/kern_proc.c b/sys/kern/kern_proc.c
index ee15efb5fb31..191143443d42 100644
--- a/sys/kern/kern_proc.c
+++ b/sys/kern/kern_proc.c
@@ -92,6 +92,7 @@
 #include <vm/vm_object.h>
 #include <vm/vm_page.h>
 #include <vm/vm_pager.h>
+#include <vm/vm_radix.h>
 #include <vm/uma.h>
 
 #include <fs/devfs/devfs.h>
@@ -2560,7 +2561,7 @@ kern_proc_vmmap_resident(vm_map_t map, vm_map_entry_t entry,
 			pi_adv = atop(entry->end - addr);
 			pindex = pi;
 			for (tobj = obj;; tobj = tobj->backing_object) {
-				m = vm_page_find_least(tobj, pindex);
+				m = vm_radix_lookup_ge(&tobj->rtree, pindex);
 				if (m != NULL) {
 					if (m->pindex == pindex)
 						break;
diff --git a/sys/vm/vm_page.c b/sys/vm/vm_page.c
index 9e41735460db..e2f11eb57100 100644
--- a/sys/vm/vm_page.c
+++ b/sys/vm/vm_page.c
@@ -1875,25 +1875,6 @@ vm_page_busy_release(vm_page_t m)
 	}
 }
 
-/*
- *	vm_page_find_least:
- *
- *	Returns the page associated with the object with least pindex
- *	greater than or equal to the parameter pindex, or NULL.
- *
- *	The object must be locked.
- */
-vm_page_t
-vm_page_find_least(vm_object_t object, vm_pindex_t pindex)
-{
-	vm_page_t m;
-
-	VM_OBJECT_ASSERT_LOCKED(object);
-	if ((m = TAILQ_FIRST(&object->memq)) != NULL && m->pindex < pindex)
-		m = vm_radix_lookup_ge(&object->rtree, pindex);
-	return (m);
-}
-
 /*
  * Uses the page mnew as a replacement for an existing page at index
  * pindex which must be already present in the object.
diff --git a/sys/vm/vm_page.h b/sys/vm/vm_page.h
index c24ae8b64d28..6d1982767cc8 100644
--- a/sys/vm/vm_page.h
+++ b/sys/vm/vm_page.h
@@ -649,7 +649,6 @@ void vm_page_deactivate(vm_page_t);
 void vm_page_deactivate_noreuse(vm_page_t);
 void vm_page_dequeue(vm_page_t m);
 void vm_page_dequeue_deferred(vm_page_t m);
-vm_page_t vm_page_find_least(vm_object_t, vm_pindex_t);
 void vm_page_free_invalid(vm_page_t);
 vm_page_t vm_page_getfake(vm_paddr_t paddr, vm_memattr_t memattr);
 void vm_page_initfake(vm_page_t m, vm_paddr_t paddr, vm_memattr_t memattr);