d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Avoid deactivating the page if it is already on a queue, only requeue

Browse Source 
the page.  This both reduces the number of queues locking and avoids
moving the active page to inactive list just because the page was read
or written.

Based on the suggestion by:	alc
Reviewed by: alc
Tested by:   pho
This commit is contained in:
kib 2013-05-06 21:04:42 +00:00
parent 7608fe3425
commit 37807bfa8a
1 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
sys/fs/tmpfs/tmpfs_vnops.c
View File

@ -493,10 +493,12 @@ tmpfs_nocacheread(vm_object_t tobj, vm_pindex_t idx,
VM_OBJECT_WLOCK(tobj);
vm_page_lock(m);
vm_page_unhold(m);
vm_page_deactivate(m);
/* Requeue to maintain LRU ordering. */
if (m->queue != PQ_NONE)
if (m->queue == PQ_NONE) {
vm_page_deactivate(m);
} else {
/* Requeue to maintain LRU ordering. */
vm_page_requeue(m);
}
vm_page_unlock(m);
VM_OBJECT_WUNLOCK(tobj);
@ -609,10 +611,12 @@ tmpfs_mappedwrite(vm_object_t tobj, size_t len, struct uio *uio)
vm_page_dirty(tpg);
vm_page_lock(tpg);
vm_page_unhold(tpg);
vm_page_deactivate(tpg);
/* Requeue to maintain LRU ordering. */
if (tpg->queue != PQ_NONE)
if (tpg->queue == PQ_NONE) {
vm_page_deactivate(tpg);
} else {
/* Requeue to maintain LRU ordering. */
vm_page_requeue(tpg);
}
vm_page_unlock(tpg);
VM_OBJECT_WUNLOCK(tobj);