All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ian Campbell <Ian.Campbell@citrix.com>
To: Daniel Kiper <daniel.kiper@oracle.com>
Cc: "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
	Ian Jackson <Ian.Jackson@eu.citrix.com>,
	"konrad.wilk@oracle.com" <konrad.wilk@oracle.com>
Subject: Re: [PATCH v3 1/3] libxl: xl mem-max et consortes must update static-max in xenstore too
Date: Wed, 10 Apr 2013 14:13:39 +0100	[thread overview]
Message-ID: <1365599619.27868.53.camel@zakaz.uk.xensource.com> (raw)
In-Reply-To: <20130409140857.GA31533@debian70-amd64.local.net-space.pl>

On Tue, 2013-04-09 at 15:08 +0100, Daniel Kiper wrote:
> On Tue, Apr 09, 2013 at 02:10:53PM +0100, Ian Jackson wrote:
> > Daniel Kiper writes ("Re: [Xen-devel] [PATCH v3 1/3] libxl: xl mem-max et consortes must update static-max in xenstore too"):
> > > On Mon, Apr 08, 2013 at 05:21:27PM +0100, Ian Jackson wrote:
> > > > Doesn't this, together with your previous patch, conflate the "static
> > > > maximum" (ie, boot-time memory size which in the absence of memory
> > > > hotplug can never be exceeded), with the "xen maximum" (ie the
> > > > enforced memory limit) ?
> > > >
> > > > I thought that the static-max xenstore key was used for the former.
> > >
> > > To some extent. However, static-max has always a bit larger value
> > > than "xen maximum". xl uses static-max to enforce limits on guests
> > > but it is just an info for guest itself. "xen maximum" is a kind
> > > of hard limit which could not be exceeded and is enforced on guest
> > > by Xen hypervisor.
> >
> > The reason for xl using static-max is that in the absence of memory
> > hotplug, attempting to raise a guest above static-max will not work.
> > And this check takes effect in xl.
> 
> OK but now it is quiet difficult (or close to impossible) to know
> in advance that a given guest supports memory hotplug or not.

This is why I previously queried whether memory hotplug ought not to be
a conscious admin decision.

  parent reply	other threads:[~2013-04-10 13:13 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-04 20:34 [PATCH v3 0/3] libxl: memory management patches Daniel Kiper
2013-04-04 20:34 ` [PATCH v3 1/3] libxl: xl mem-max et consortes must update static-max in xenstore too Daniel Kiper
2013-04-08 16:21   ` Ian Jackson
2013-04-09 11:54     ` Daniel Kiper
2013-04-09 13:10       ` Ian Jackson
2013-04-09 14:08         ` Daniel Kiper
2013-04-09 14:18           ` Ian Jackson
2013-04-09 21:44             ` Daniel Kiper
2013-04-10 15:56               ` [PATCH v3 1/3] libxl: xl mem-max et consortes must update static-max in xenstore too [and 1 more messages] Ian Jackson
2013-04-11 10:15                 ` George Dunlap
2013-04-11 12:25                   ` Daniel Kiper
2013-04-11 12:24                 ` Daniel Kiper
2013-04-11 13:47                   ` Ian Campbell
2013-04-11 14:28                     ` Daniel Kiper
2013-04-16 12:10                       ` Daniel Kiper
2013-04-16 12:21                         ` Ian Campbell
2013-04-16 13:47                           ` Daniel Kiper
2013-04-16 14:09                             ` Ian Jackson
2013-04-10 13:13           ` Ian Campbell [this message]
2013-04-11 11:20             ` [PATCH v3 1/3] libxl: xl mem-max et consortes must update static-max in xenstore too Daniel Kiper
2013-04-11 13:35               ` Ian Campbell
2013-04-11 14:32                 ` Daniel Kiper
2013-04-16 12:12                   ` Daniel Kiper
2013-04-04 20:34 ` [PATCH v3 2/3] xl: Allow user to configure xl mem-set behavior Daniel Kiper
2013-04-04 20:34 ` [PATCH v3 3/3] xl: Improve xl documentation in regards to guest memory management Daniel Kiper
2013-04-08 16:19   ` Ian Jackson
2013-04-09 12:18     ` Daniel Kiper
2013-04-09 14:33       ` Ian Jackson
2013-04-09 21:57         ` Daniel Kiper

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=1365599619.27868.53.camel@zakaz.uk.xensource.com \
    --to=ian.campbell@citrix.com \
    --cc=Ian.Jackson@eu.citrix.com \
    --cc=daniel.kiper@oracle.com \
    --cc=konrad.wilk@oracle.com \
    --cc=xen-devel@lists.xensource.com \
    /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.