From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Stabellini Subject: Re: [PATCH 0 of 4] videoram allocation and stdvga improvements Date: Wed, 14 Jan 2009 19:00:38 +0000 Message-ID: <496E3656.7000206@eu.citrix.com> References: <496C866B.1010307@eu.citrix.com> <20090113140108.GH5033@const.bordeaux.inria.fr> <20090113143136.GI5033@const.bordeaux.inria.fr> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <20090113143136.GI5033@const.bordeaux.inria.fr> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Samuel Thibault , Stefano Stabellini , xen-devel List-Id: xen-devel@lists.xenproject.org Samuel Thibault wrote: > Samuel Thibault, le Tue 13 Jan 2009 15:01:08 +0100, a =C3=A9crit : >> Stefano Stabellini, le Tue 13 Jan 2009 12:17:47 +0000, a =C3=A9crit : >>> In order to simplify the codepath and the libxc API, this patch serie= s >>> removes the videoram allocation code from hvmloader and lets qemu do = the >>> job, using xc_domain_memory_populate_physmap. >> That's what it was doing before I made hvmloader do it. The reason wa= s >> to make save/restore work fine with stubdomains dirty log, etc. >=20 > Ah, now I remember the precise reason, written in the 18383 changelog: >=20 > When creating an HVM domain, if e.g. another domain is created before > qemu allocates video memory, the extra 8MB memory ballooning is not > available any more, because it got consumed by the other domain. >=20 The current design of the autoballoner is limited, as a consequence the autoballoner is racy and it should not be used at all if the user plans to start multiple domains at the same time.