All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Vrabel <david.vrabel@citrix.com>
To: Jan Beulich <JBeulich@suse.com>
Cc: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>,
	"Keir (Xen.org)" <keir@xen.org>,
	xen-devel <xen-devel@lists.xen.org>
Subject: Re: [PATCH] x86: set dom0's default maximum reservation to the initial number of pages
Date: Wed, 21 Mar 2012 10:33:57 +0000	[thread overview]
Message-ID: <4F69AE95.9010707@citrix.com> (raw)
In-Reply-To: <4F69B8010200007800079D86@nat28.tlf.novell.com>

On 21/03/12 10:14, Jan Beulich wrote:
> On 21.03.12 at 10:58, David Vrabel <david.vrabel@citrix.com> wrote:
>> The docs still need updating and I think the min:MMM option should be
>> deprecated.
> 
> Why should we deprecate min:? It can be useful when combined with
> other sub-options, can't it?

Not convinced of its utility but it does do something, so yes, it should
be kept.

>>>> This change means that with Linux 3.0.5 and later kernels,
>>>> dom0_mem=512M has the same result as older, 'classic Xen' kernels. The
>>>> older kernels used the initial number of pages to set the maximum
>>>> number of pages and did not query the hypervisor for the maximum
>>>> reservation.
>>>
>>> If the pv-ops kernels behave differently in this aspect than the forward
>>> ported ones, then it should really be the newer one to get adjusted to
>>> the original behavior (if so intended), unless the old behavior was
>>> completely insane. Adjusting the hypervisor for a shortcoming(?) in a
>>> particular implementation of a Dom0 kernel doesn't seem to be the right
>>> route - please also consider non-Linux Dom0 kernels that might expect
>>> the current behavior.
>>
>> The old behaviour did not allow dom0 to balloon up beyond the initial
>> allocation so we don't what to go back to that.
> 
> In pv-ops Linux Dom0 you mean? While I can't speak of non-Linux,
> ballooning up beyond the initial allocation works fine in the legacy
> and forward ported kernels.

Perhaps I'm looking at a XenServer specific limitation.

David

      reply	other threads:[~2012-03-21 10:33 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-20 18:21 [PATCH] x86: set dom0's default maximum reservation to the initial number of pages David Vrabel
2012-03-20 23:41 ` Konrad Rzeszutek Wilk
2012-03-21  8:36 ` Jan Beulich
2012-03-21  9:58   ` David Vrabel
2012-03-21 10:14     ` Jan Beulich
2012-03-21 10:33       ` David Vrabel [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=4F69AE95.9010707@citrix.com \
    --to=david.vrabel@citrix.com \
    --cc=JBeulich@suse.com \
    --cc=keir@xen.org \
    --cc=konrad.wilk@oracle.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.