Skip site navigation (1)Skip section navigation (2)
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>