Date: Thu, 2 Aug 2012 17:22:48 +0300 From: Konstantin Belousov <kostikbel@gmail.com> To: Andrey Zonov <andrey@zonov.org> Cc: freebsd-hackers@freebsd.org Subject: Re: system wide major/minor page faults counters Message-ID: <20120802142248.GY2676@deviant.kiev.zoral.com.ua> In-Reply-To: <501A85A1.3050106@zonov.org> References: <501A85A1.3050106@zonov.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--v/49yhf2B++JedBI Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Aug 02, 2012 at 05:50:25PM +0400, Andrey Zonov wrote: > Hi, >=20 > It would be useful to have system wide major and minor page faults=20 > counters. Attached patch makes this possible. >=20 > Are there any objections to have it? I thought that struct vmmeter was engraved into our ABI much deeper, but inspection shows that at least base system successfully avoids dragging the structure into ABI at all. So what follows is only some comments and not objections. The description of v_majflt as 'page faults' and v_minflt as 'page reclaim' is very confusing, IMO. First, the v_majflt is not inclusive in regard of v_minflt. Second, reclaim usually has a connotation of thing being repurposed (I am probably wrong there), which is not case for minor page faults. As an additional twist, please note that not all page faults reach vm_fault_hold() at all. See for instance x86 handling of the spurious page faults, I believe that other architectures like MIPS also completely consume some page faults in MD code. The minfault counter probably should be incremented there too. And, not all calls to vm_fault_hold() are caused by hardware page faults. See the use of vm_fault_hold() and vm_fault_quick_hold_pages() scattered around the sys/. --v/49yhf2B++JedBI Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.12 (FreeBSD) iEYEARECAAYFAlAajTgACgkQC3+MBN1Mb4iq0QCfZWYDDEbz/exhrtcNLV2GenOu F2wAnA4WRg+h03J1D7vQCF8CZjuBhcNs =mT/I -----END PGP SIGNATURE----- --v/49yhf2B++JedBI--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20120802142248.GY2676>