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

Eliminate unnecessary page hold_count checks. These checks predate

Browse Source 
r90944, which introduced a general mechanism for handling the freeing
of held pages.

Reviewed by:	kib@
This commit is contained in:
Alan Cox 2011-02-03 14:42:46 +00:00
parent e119a7bcdb
commit 8189ac85e9
2 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
sys/kern/uipc_syscalls.c
View File

@ -2115,8 +2115,7 @@ kern_sendfile(struct thread *td, struct sendfile_args *uap,
* then free it.
*/
if (pg->wire_count == 0 && pg->valid == 0 &&
pg->busy == 0 && !(pg->oflags & VPO_BUSY) &&
pg->hold_count == 0)
pg->busy == 0 && !(pg->oflags & VPO_BUSY))
vm_page_free(pg);
vm_page_unlock(pg);
VM_OBJECT_UNLOCK(obj);

3
sys/kern/vfs_bio.c
View File

@ -1647,8 +1647,7 @@ vfs_vmio_release(struct buf *bp)
* no valid data. We also free the page if the
* buffer was used for direct I/O
*/
if ((bp->b_flags & B_ASYNC) == 0 && !m->valid &&
m->hold_count == 0) {
if ((bp->b_flags & B_ASYNC) == 0 && !m->valid) {
vm_page_free(m);
} else if (bp->b_flags & B_DIRECT) {
vm_page_try_to_free(m);