talexander
/
freebsd
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

vm_page: fuse two allocation loops 

Combine two loops, each iterating over the same array of pages to
initialize them, into a single loop.

Reviewed by:	kib
Differential Revision:	https://reviews.freebsd.org/D46609
This commit is contained in:
Doug Moore 2024-09-12 23:41:01 -05:00
parent c1c989588d
commit 4876924c0c
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
sys/vm/vm_page.c
View File

@ -2352,10 +2352,6 @@ vm_page_alloc_contig_domain(vm_object_t object, vm_pindex_t pindex, int domain,
if (!vm_domain_alloc_fail(VM_DOMAIN(domain), object, req))
return (NULL);
}
for (m = m_ret; m < &m_ret[npages]; m++) {
vm_page_dequeue(m);
vm_page_alloc_check(m);
}
/*
* Initialize the pages. Only the PG_ZERO flag is inherited.
@ -2376,6 +2372,8 @@ vm_page_alloc_contig_domain(vm_object_t object, vm_pindex_t pindex, int domain,
memattr == VM_MEMATTR_DEFAULT)
memattr = object->memattr;
for (m = m_ret; m < &m_ret[npages]; m++) {
vm_page_dequeue(m);
vm_page_alloc_check(m);
m->a.flags = 0;
m->flags = (m->flags | PG_NODUMP) & flags;
m->busy_lock = busy_lock;