summaryrefslogtreecommitdiffstats
path: root/sys/vm/vm_object.c
diff options
context:
space:
mode:
authoralc <alc@FreeBSD.org>2010-05-07 05:23:15 +0000
committeralc <alc@FreeBSD.org>2010-05-07 05:23:15 +0000
commitec2322fb0053a7bcacf6002a572400947f373e74 (patch)
tree89ce22ca005f48ab502b68edb9db98d02859e2f1 /sys/vm/vm_object.c
parentebbcafea2444e9d6d3defd83ca63250518014836 (diff)
downloadFreeBSD-src-ec2322fb0053a7bcacf6002a572400947f373e74.zip
FreeBSD-src-ec2322fb0053a7bcacf6002a572400947f373e74.tar.gz
Eliminate acquisitions of the page queues lock that are no longer needed.
Switch to a per-processor counter for the number of pages freed during process termination.
Diffstat (limited to 'sys/vm/vm_object.c')
-rw-r--r--sys/vm/vm_object.c11
1 files changed, 2 insertions, 9 deletions
diff --git a/sys/vm/vm_object.c b/sys/vm/vm_object.c
index 3986d14..5428c98 100644
--- a/sys/vm/vm_object.c
+++ b/sys/vm/vm_object.c
@@ -722,14 +722,11 @@ vm_object_terminate(vm_object_t object)
("vm_object_terminate: freeing busy page %p "
"p->busy = %d, p->oflags %x\n", p, p->busy, p->oflags));
vm_page_lock(p);
- vm_page_lock_queues();
if (p->wire_count == 0) {
vm_page_free(p);
- cnt.v_pfree++;
- } else {
+ PCPU_INC(cnt.v_pfree);
+ } else
vm_page_remove(p);
- }
- vm_page_unlock_queues();
vm_page_unlock(p);
}
@@ -1634,14 +1631,12 @@ vm_object_backing_scan(vm_object_t object, int op)
* can simply destroy it.
*/
vm_page_lock(p);
- vm_page_lock_queues();
KASSERT(!pmap_page_is_mapped(p),
("freeing mapped page %p", p));
if (p->wire_count == 0)
vm_page_free(p);
else
vm_page_remove(p);
- vm_page_unlock_queues();
vm_page_unlock(p);
p = next;
continue;
@@ -1660,14 +1655,12 @@ vm_object_backing_scan(vm_object_t object, int op)
* Leave the parent's page alone
*/
vm_page_lock(p);
- vm_page_lock_queues();
KASSERT(!pmap_page_is_mapped(p),
("freeing mapped page %p", p));
if (p->wire_count == 0)
vm_page_free(p);
else
vm_page_remove(p);
- vm_page_unlock_queues();
vm_page_unlock(p);
p = next;
continue;
OpenPOWER on IntegriCloud