All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dan Magenheimer <dan.magenheimer@oracle.com>
To: Jeremy Fitzhardinge <jeremy@goop.org>
Cc: xen-devel@lists.xensource.com,
	Ian Jackson <Ian.Jackson@eu.citrix.com>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Subject: RE: [PATCH 2 of 8] libxl: introduce libxl_set_relative_memory_target
Date: Wed, 1 Sep 2010 14:49:22 -0700 (PDT)	[thread overview]
Message-ID: <7352ab89-2dd6-4d15-99cd-04867849f399@default> (raw)
In-Reply-To: <4C7EC2F3.3050704@goop.org>

> From: Jeremy Fitzhardinge [mailto:jeremy@goop.org]
> 
>  On 09/01/2010 01:03 PM, Dan Magenheimer wrote:
> > Indeed, that's what selfballooning does.  The xenstore watch
> > is irrelevant for selfballooning (though the watch also can be used
> > asynchronously for backwards compatibility).
> 
> There's no mechanism to make the balloon driver ignore the target
> watch,
> so any updates to xenstore will update the driver's target.

The selfballooning patch currently applies to the balloon driver
so it could easily disable the target watch, though it does not.
 
> > So, frankly, I think the "xm memset" functionality is largely
> > useless, but agree that it should be maintained in xl for backwards
> > compatibility.  But trying to comingle the concepts of maxmem
> > and target is a bad idea.
> 
> In the general case I think you're probably right (I can't see it being
> useful in a VPS hosting service, for example), but there are definitely
> special cases where it is useful.  Squashing down existing domains to
> make room for a new one, for example, or more app-specific uses.

Agreed in general, though I suspect sysadmins would be rather
peeved if/when a simple xm command in dom0 causes kernel OOMs
and application kills... or, worse, guest kernel crashes (which are
circumvented by minimum-target code in the linux-xen balloon
driver but NOT in the pvops balloon driver).

So "useless" is an overstatement, but "must be used extremely
carefully with knowledge of current activity in the guest
and/or willingness for the targeted guest or its applications
to die for a greater cause" is not an overstatement.
 
> Giving domains some real incentive to be economical with memory would
> probably change the landscape a lot.  But I don't think there's a real
> solution without knowing the specifics of that incentive.

Agreed.  Lacking a clear incentive though, reducing the disincentive
is a reasonable approach... which is what tmem+selfballooning does.

My long term view of the incentive is something like a VPS hoster
that offers service for $10/month, but offers it for $5/month
if using a tmem(+selfballooning)-enabled kernel.  This is essentially
like the electric utility companies that give customers a discount
if the customer allows them a remote-kill switch to turn off your air
conditioning if system-wide conditions warrant.

Dan

  reply	other threads:[~2010-09-01 21:49 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-27 11:18 [PATCH 2 of 8] libxl: introduce libxl_set_relative_memory_target Stefano Stabellini
2010-08-31 17:25 ` Ian Jackson
2010-09-01 10:55   ` Stefano Stabellini
2010-09-01 18:01   ` Jeremy Fitzhardinge
2010-09-01 20:03     ` Dan Magenheimer
2010-09-01 21:17       ` Jeremy Fitzhardinge
2010-09-01 21:49         ` Dan Magenheimer [this message]
2010-09-02  9:15     ` Stefano Stabellini

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=7352ab89-2dd6-4d15-99cd-04867849f399@default \
    --to=dan.magenheimer@oracle.com \
    --cc=Ian.Jackson@eu.citrix.com \
    --cc=jeremy@goop.org \
    --cc=stefano.stabellini@eu.citrix.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.