Date: Wed, 25 Jun 2014 16:09:32 +0400 From: Gleb Smirnoff <glebius@FreeBSD.org> To: Attilio Rao <attilio@freebsd.org> Cc: Alan Cox <alc@freebsd.org>, "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, Roger Pau =?iso-8859-1?Q?Monn=E9?= <royger@freebsd.org>, "src-committers@freebsd.org" <src-committers@freebsd.org> Subject: Re: svn commit: r267858 - in head/sys/dev: virtio/balloon xen/balloon Message-ID: <20140625120932.GM28199@FreeBSD.org> In-Reply-To: <CAJ-FndD4M8QkejyTv4R_9qoThqvFinYbHP4YSby6mQS9EKSYPA@mail.gmail.com> References: <201406250951.s5P9p8YR017159@svn.freebsd.org> <CAJ-FndD4M8QkejyTv4R_9qoThqvFinYbHP4YSby6mQS9EKSYPA@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Jun 25, 2014 at 01:58:29PM +0200, Attilio Rao wrote: A> > Log: A> > xen/virtio: fix balloon drivers to not mark pages as WIRED A> > A> > Prevent the Xen and VirtIO balloon drivers from marking pages as A> > wired. This prevents them from increasing the system wired page count, A> > which can lead to mlock failing because of hitting the limit in A> > vm.max_wired. A> A> This change is conceptually wrong. A> The pages balloon is allocating are unmanaged and they should be wired A> by definition. Alan and I are considering enforcing this (mandatory A> wired pages for unmanaged pages allocation) directly in the KPI. A> This in practice just seem an artifact to deal with scarce wired A> memory limit. I suggest that for the XEN case this limit gets bumped A> rather relying on similar type of hacks. Proper limit would be to count pages wired by userland via mlock(2) and enforce limit only on those pages. Pages wired by kernel should be either unlimited or controled by a separate limit. -- Totus tuus, Glebius.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20140625120932.GM28199>