diff options
author | alc <alc@FreeBSD.org> | 2010-05-07 05:23:15 +0000 |
---|---|---|
committer | alc <alc@FreeBSD.org> | 2010-05-07 05:23:15 +0000 |
commit | ec2322fb0053a7bcacf6002a572400947f373e74 (patch) | |
tree | 89ce22ca005f48ab502b68edb9db98d02859e2f1 /sys/vm/vm_object.c | |
parent | ebbcafea2444e9d6d3defd83ca63250518014836 (diff) | |
download | FreeBSD-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.c | 11 |
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; |