Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 27 Aug 2015 23:32:22 -0700
From:      hiren panchasara <hiren@strugglingcoder.info>
To:        Mark Johnston <markj@FreeBSD.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r287235 - in head/sys: sys vm
Message-ID:  <20150828063222.GO82991@strugglingcoder.info>
In-Reply-To: <201508280044.t7S0iIBY038698@repo.freebsd.org>
References:  <201508280044.t7S0iIBY038698@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help

--wxNrNKeE/ptr8Zbn
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On 08/28/15 at 12:44P, Mark Johnston wrote:
> Author: markj
> Date: Fri Aug 28 00:44:17 2015
> New Revision: 287235
> URL: https://svnweb.freebsd.org/changeset/base/287235
>=20
> Log:
>   Remove weighted page handling from vm_page_advise().
>  =20
>   This was added in r51337 as part of the implementation of
>   madvise(MADV_DONTNEED).  Its objective was to ensure that the page daem=
on
>   would eventually reclaim other unreferenced pages (i.e., unreferenced p=
ages
>   not touched by madvise()) from the active queue.
>  =20
>   Now that the pagedaemon performs steady scanning of the active page que=
ue,
>   this weighted handling is unnecessary.  Instead, always "cache" clean p=
ages
>   by moving them to the head of the inactive page queue.  This simplifies=
 the
>   implementation of vm_page_advise() and eliminates the fragmentation that
>   resulted from the distribution of pages among multiple queues.
>  =20
>   Suggested by:	alc
>   Reviewed by:	alc
>   Sponsored by:	EMC / Isilon Storage Division
>   Differential Revision:	https://reviews.freebsd.org/D3401

Nice work!

Can this be mfc'd to 10?

Cheers,
Hiren

--wxNrNKeE/ptr8Zbn
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.22 (FreeBSD)

iQF8BAEBCgBmBQJV4AB2XxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w
ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXRBNEUyMEZBMUQ4Nzg4RjNGMTdFNjZGMDI4
QjkyNTBFMTU2M0VERkU1AAoJEIuSUOFWPt/lvvcH+gJhwJiuyC3FJeOtKGrlHM0s
fcDgrxzq2VXMPSnzEeNr5VKIEDIPo56KG/2PhQEj0hg8Yy9vcDDvx4ZX5brZjFwv
Xvg+LfO3PA2Ywb6FOGJwy1AH12RQ4GLAx1wrm545LLlI33c/5ipH+hSXrzx2vO4t
1lHzprODHD16c5ofsxx3x095hIB2Xu+ktqnH2VqaFTwQXJihJy2sOwPPquUXN7RP
NNADLJEa/lKXLIiKF5yKoINbqVuzCN8H7U7e5jp+Oays8/KOsDjxea3E5s/5jI2T
MnO8UbDp1f1gWUkBLV21pSdXEbe2gE2UCxI+S+g6YO6CxcBXodjag024IhjNZVg=
=5ZOc
-----END PGP SIGNATURE-----

--wxNrNKeE/ptr8Zbn--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20150828063222.GO82991>