All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ian Campbell <ian.campbell@citrix.com>
To: Anthony PERARD <anthony.perard@citrix.com>,
	Juergen Gross <jgross@suse.com>
Cc: wei.liu2@citrix.com, stefano.stabellini@eu.citrix.com,
	ian.jackson@eu.citrix.com, xen-devel@lists.xen.org
Subject: Re: [PATCH] libxc: correct memory range check in domain builder
Date: Mon, 8 Feb 2016 15:45:59 +0000	[thread overview]
Message-ID: <1454946359.26508.99.camel@citrix.com> (raw)
In-Reply-To: <20160208152028.GB1810@perard.uk.xensource.com>

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 <anthony.perard@citrix.com>
> > Signed-off-by: Juergen Gross <jgross@suse.com>
> 
> 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 <anthony.perard@citrix.com>

thanks, acked + applied.

      reply	other threads:[~2016-02-08 15:45 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-08 14:23 [PATCH] libxc: correct memory range check in domain builder Juergen Gross
2016-02-08 15:20 ` Anthony PERARD
2016-02-08 15:45   ` Ian Campbell [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1454946359.26508.99.camel@citrix.com \
    --to=ian.campbell@citrix.com \
    --cc=anthony.perard@citrix.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=jgross@suse.com \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=wei.liu2@citrix.com \
    --cc=xen-devel@lists.xen.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.