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/swap_pager.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/swap_pager.c')
-rw-r--r-- | sys/vm/swap_pager.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/sys/vm/swap_pager.c b/sys/vm/swap_pager.c index 5041276..d5e942e 100644 --- a/sys/vm/swap_pager.c +++ b/sys/vm/swap_pager.c @@ -1706,18 +1706,19 @@ swp_pager_force_pagein(vm_object_t object, vm_pindex_t pindex) vm_page_t m; vm_object_pip_add(object, 1); - 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_object_pip_subtract(object, 1); vm_page_dirty(m); vm_page_lock(m); vm_page_activate(m); vm_page_unlock(m); - vm_page_wakeup(m); vm_pager_page_unswapped(m); return; } + vm_page_busy(m); if (swap_pager_getpages(object, &m, 1, 0) != VM_PAGER_OK) panic("swap_pager_force_pagein: read from swap failed");/*XXX*/ vm_object_pip_subtract(object, 1); |