Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 16 May 2025 08:20:20 GMT
From:      Doug Moore <dougm@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: efde1e2c8b76 - main - vm_page: remove unneeded use of mpred
Message-ID:  <202505160820.54G8KKN4075353@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by dougm:

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

commit efde1e2c8b765e8dbd831e6af15c9706d38937a0
Author:     Doug Moore <dougm@FreeBSD.org>
AuthorDate: 2025-05-16 07:55:14 +0000
Commit:     Doug Moore <dougm@FreeBSD.org>
CommitDate: 2025-05-16 07:55:14 +0000

    vm_page: remove unneeded use of mpred
    
    Change 4c65b51f44af eliminated the need for some uses of the mpred
    variable in vm_page_alloc_contig_domain.  Remove those uses.
    
    Reviewed by:    alc, kib, markj
    Differential Revision:  https://reviews.freebsd.org/D50263
---
 sys/vm/vm_page.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/sys/vm/vm_page.c b/sys/vm/vm_page.c
index 0fdda1d5fc48..035b144ccd1e 100644
--- a/sys/vm/vm_page.c
+++ b/sys/vm/vm_page.c
@@ -2333,9 +2333,6 @@ vm_page_alloc_contig_domain(vm_object_t object, vm_pindex_t pindex, int domain,
 	KASSERT(npages > 0, ("vm_page_alloc_contig: npages is zero"));
 
 	vm_page_iter_init(&pages, object);
-	mpred = vm_radix_iter_lookup_lt(&pages, pindex);
-	KASSERT(mpred == NULL || mpred->pindex != pindex,
-	    ("vm_page_alloc_contig: pindex already allocated"));
 	for (;;) {
 #if VM_NRESERVLEVEL > 0
 		/*
@@ -2406,7 +2403,6 @@ vm_page_alloc_contig_domain(vm_object_t object, vm_pindex_t pindex, int domain,
 			}
 			return (NULL);
 		}
-		mpred = m;
 		if (memattr != VM_MEMATTR_DEFAULT)
 			pmap_page_set_memattr(m, memattr);
 		pindex++;



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202505160820.54G8KKN4075353>