diff options
author | attilio <attilio@FreeBSD.org> | 2013-06-06 18:19:26 +0000 |
---|---|---|
committer | attilio <attilio@FreeBSD.org> | 2013-06-06 18:19:26 +0000 |
commit | 3b60ec551b6c8611ddc22835dc4f9ed46211ac1a (patch) | |
tree | 4b783acfa06db7b7287a56d02232844f86677851 /sys/vm/vm_glue.c | |
parent | 190d5ac85ba0340f44b918b60adeb77b8acfa649 (diff) | |
download | FreeBSD-src-3b60ec551b6c8611ddc22835dc4f9ed46211ac1a.zip FreeBSD-src-3b60ec551b6c8611ddc22835dc4f9ed46211ac1a.tar.gz |
Complete r251452:
Avoid to busy/unbusy a page in cases where there is no need to drop the
vm_obj lock, more nominally when the page is full valid after
vm_page_grab().
Sponsored by: EMC / Isilon storage division
Reviewed by: alc
Diffstat (limited to 'sys/vm/vm_glue.c')
-rw-r--r-- | sys/vm/vm_glue.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/sys/vm/vm_glue.c b/sys/vm/vm_glue.c index e0a8bf7..6183518 100644 --- a/sys/vm/vm_glue.c +++ b/sys/vm/vm_glue.c @@ -241,8 +241,10 @@ vm_imgact_hold_page(vm_object_t object, vm_ooffset_t offset) VM_OBJECT_WLOCK(object); pindex = OFF_TO_IDX(offset); - m = vm_page_grab(object, pindex, VM_ALLOC_NORMAL | VM_ALLOC_RETRY); + m = vm_page_grab(object, pindex, VM_ALLOC_NORMAL | VM_ALLOC_RETRY | + VM_ALLOC_NOBUSY); if (m->valid != VM_PAGE_BITS_ALL) { + vm_page_busy(m); ma[0] = m; rv = vm_pager_get_pages(object, ma, 1, 0); m = vm_page_lookup(object, pindex); @@ -255,11 +257,11 @@ vm_imgact_hold_page(vm_object_t object, vm_ooffset_t offset) m = NULL; goto out; } + vm_page_wakeup(m); } vm_page_lock(m); vm_page_hold(m); vm_page_unlock(m); - vm_page_wakeup(m); out: VM_OBJECT_WUNLOCK(object); return (m); |