xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Mukesh Rathor <mukesh.rathor@oracle.com>
To: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Cc: "Ian Campbell" <Ian.Campbell@citrix.com>,
	george.dunlap@eu.citrix.com,
	"Ian Jackson" <ian.jackson@eu.citrix.com>,
	tim@xen.org, "David Vrabel" <david.vrabel@citrix.com>,
	jbeulich@suse.com, xen-devel@lists.xenproject.org,
	"Roger Pau Monné" <roger.pau@citrix.com>
Subject: Re: Is: PVH - how to solve maxmem != memory scenario? Was:Re: [PATCH] libxl: create PVH guests with max memory assigned
Date: Tue, 5 Aug 2014 14:36:22 -0700	[thread overview]
Message-ID: <20140805143622.191b6103@mantra.us.oracle.com> (raw)
In-Reply-To: <20140805141844.GG13057@laptop.dumpdata.com>

On Tue, 5 Aug 2014 10:18:44 -0400
Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> wrote:

> On Tue, Aug 05, 2014 at 01:08:22PM +0200, Roger Pau Monné wrote:
> > On 05/08/14 11:34, David Vrabel wrote:
> > > On 05/08/14 09:55, Ian Campbell wrote:
> > >> On Thu, 2014-07-17 at 13:02 +0200, Roger Pau Monne wrote:
> > >>
> > >> Sorry for the delay replying, this somehow slipped through my

........

> It is evolving :-)
> > 
> > I've always thought of PVH as PVHVM without a device model, so IMHO
> > it would make more sense to use PoD rather than the PV ballooning
> > approach, but I would like to hear opinions from others before
> > taking a stab into implementing it.
> 
> Lets rope Mukesh, Tim, George and Jan in here.
> 
> Mukesh's feeling was that it is an PV.
> 
> I believe George is the opinion of 'HVM' without the device model.
> 
> In the past I  was thinking that since it is from the PV it would
> be more of that (PV) without the P2M and M2P. And the memory
> management (so E820) would follow the PV paths and do the proper
> ballooning/decreasing.
> 
> However I think it was you (David) who suggested that we just
> setup the E820 properly in the toolstack/hypervisor and have it
> match the hypervisors' P2M. That I believe is what Roger's patch
> was aiming at.
> 
> A bit of past history:
> Mukesh's initial patches (v3, see
> https://lkml.org/lkml/2012/10/17/553, and
> https://lkml.org/lkml/2013/12/12/627, for new hypercall) took the
> path that the PV guest will act as PV. And it will do the proper
> hypercalls to expand/contract the Xen's P2M to balloon out and in.
> However the only reason for this was to match the P2M (assuming it
> was flat and up to nr_pages) to E820 (which would be discontingous)
> and setup the correct EPT entries in the hypervisor.
> 
> My personal opinion is that the easiest path is the best.

Agree. My opinion, that I had expressed to Roger few weeks ago when we
initially talked of which approach to take here, is we should pick
the best path in terms of performance and simplicity. My intention always 
has been to examine both PV and HVM, and pick the best, or if a third
option makes sense, go that way - that IMO is the whole point of PVH.

thanks,
Mukesh


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

      parent reply	other threads:[~2014-08-05 21:36 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-17 11:02 [PATCH] libxl: create PVH guests with max memory assigned Roger Pau Monne
2014-07-18 16:49 ` Konrad Rzeszutek Wilk
2014-07-18 17:00   ` Roger Pau Monné
2014-07-18 17:11     ` Andrew Cooper
2014-07-21 10:16       ` Ian Campbell
2014-07-18 20:53     ` Konrad Rzeszutek Wilk
2014-08-05  8:57       ` Ian Campbell
2014-07-18 17:19   ` Olaf Hering
2014-07-18 19:33     ` Konrad Rzeszutek Wilk
2014-08-01 15:34 ` Roger Pau Monné
2014-08-04 18:44   ` Konrad Rzeszutek Wilk
2014-08-05  8:55 ` Ian Campbell
2014-08-05  9:34   ` David Vrabel
2014-08-05 11:08     ` Roger Pau Monné
2014-08-05 14:06       ` Ian Campbell
2014-08-05 14:10       ` George Dunlap
2014-08-05 21:22         ` Mukesh Rathor
2014-08-05 14:18       ` Is: PVH - how to solve maxmem != memory scenario? Was:Re: " Konrad Rzeszutek Wilk
2014-08-05 14:36         ` Jan Beulich
2014-08-05 14:48           ` Konrad Rzeszutek Wilk
2014-08-05 15:12             ` Jan Beulich
2014-08-05 15:41               ` Konrad Rzeszutek Wilk
2014-08-05 15:05         ` David Vrabel
2014-08-05 15:40           ` Konrad Rzeszutek Wilk
2014-08-05 15:51             ` Jan Beulich
2014-08-05 15:56               ` Konrad Rzeszutek Wilk
2014-08-05 16:07                 ` Jan Beulich
2014-08-05 19:45           ` Tim Deegan
2014-08-05 21:36         ` Mukesh Rathor [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=20140805143622.191b6103@mantra.us.oracle.com \
    --to=mukesh.rathor@oracle.com \
    --cc=Ian.Campbell@citrix.com \
    --cc=david.vrabel@citrix.com \
    --cc=george.dunlap@eu.citrix.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=jbeulich@suse.com \
    --cc=konrad.wilk@oracle.com \
    --cc=roger.pau@citrix.com \
    --cc=tim@xen.org \
    --cc=xen-devel@lists.xenproject.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).