From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH] libxc: correct memory range check in domain builder Date: Mon, 8 Feb 2016 15:45:59 +0000 Message-ID: <1454946359.26508.99.camel@citrix.com> References: <1454941432-24262-1-git-send-email-jgross@suse.com> <20160208152028.GB1810@perard.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20160208152028.GB1810@perard.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Anthony PERARD , Juergen Gross Cc: wei.liu2@citrix.com, stefano.stabellini@eu.citrix.com, ian.jackson@eu.citrix.com, xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On Mon, 2016-02-08 at 15:20 +0000, Anthony PERARD wrote: > On Mon, Feb 08, 2016 at 03:23:52PM +0100, Juergen Gross wrote: > > Commit 81a76e4b12961a9f54f5021809074196dfe6dbba ("libxc: rework of > > domain builder's page table handler") introduced a regression with > > checking the required memory size of the domain. The needed maximum pfn > > of the initial kernel mapping was added to the currently last used pfn > > resulting in doubling the estimated memory need. > > > > Correct the calculation of the last needed pfn to enable booting of > > small domains again. > > > > Reported-by: Anthony Perard > > Signed-off-by: Juergen Gross > > It works for me, using xl. I have not try with the full stack > (openstack/libvirt), but I expect it to work. > > Tested-by: Anthony PERARD thanks, acked + applied.