Date: Sun, 9 May 2010 16:27:42 +0000 (UTC) From: Alan Cox <alc@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/vm swap_pager.c Message-ID: <201005091627.o49GRoeg017832@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
alc 2010-05-09 16:27:42 UTC FreeBSD src repository Modified files: sys/vm swap_pager.c Log: SVN rev 207822 on 2010-05-09 16:27:42Z by alc Call vm_page_deactivate() rather than vm_page_dontneed() in swp_pager_force_pagein(). By dirtying the page, swp_pager_force_pagein() forces vm_page_dontneed() to insert the page at the head of the inactive queue, just like vm_page_deactivate() does. Moreover, because the page was invalid, it can't have been mapped, and thus the other effect of vm_page_dontneed(), clearing the page's reference bits has no effect. In summary, there is no reason to call vm_page_dontneed() since its effect will be identical to calling the simpler vm_page_deactivate(). Revision Changes Path 1.321 +2 -4 src/sys/vm/swap_pager.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201005091627.o49GRoeg017832>