xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: George Dunlap <george.dunlap@eu.citrix.com>
Cc: "Xen-devel@lists.xensource.com" <Xen-devel@lists.xensource.com>,
	Keir Fraser <keir.xen@gmail.com>,
	Ian Campbell <Ian.Campbell@citrix.com>,
	Stefano Stabellini <Stefano.Stabellini@eu.citrix.com>
Subject: Re: [HYBRID]: status update...
Date: Thu, 2 Aug 2012 10:24:31 -0400	[thread overview]
Message-ID: <20120802142431.GG16749@phenom.dumpdata.com> (raw)
In-Reply-To: <501A4E0C.1090509@eu.citrix.com>

On Thu, Aug 02, 2012 at 10:53:16AM +0100, George Dunlap wrote:
> On 01/08/12 23:34, Mukesh Rathor wrote:
> >On Wed, 1 Aug 2012 16:25:01 +0100
> >George Dunlap <George.Dunlap@eu.citrix.com> wrote:
> >
> >>I hope this isn't bikeshedding; but I don't like "Hybrid" as a name
> >>for this feature, mainly for "marketing" reasons.  I think it will
> >>probably give people the wrong idea about what the technology does.
> >>PV domains is one of Xen's really distinct advantages -- much simpler
> >>interface, lighter-weight (no qemu, legacy boot), &c &c.  As I
> >>understand it, the mode you've been calling "hybrid" still has all of
> >>these advantages -- it just uses some of the HVM hardware extensions
> >>to make the interface even simpler / faster.  I'm afraid "hybrid" may
> >>be seen as, "Even Xen has had to give up on PV."
> >>
> >>Can I suggest something like "PVH" instead?  That (at least to me)
> >>makes it clear that PV domains are still fully PV, but just use some
> >>HVM extensions.
> >>
> >>Thoughts?
> >Hi George,
> >
> >We gave some thought looking for name. I figured pure PV will be around for
> >a while at least. So there's PV on one side and HVM on the other, hybrid
> >somewhere in between.
> I understand the idea, but I think it's not very accurate.  I would
> call Stefano's "PVHVM" stuff hybrid -- it has the legacy boot and
> emulated devices, but uses the PV interfaces for event delivery
> extensively.  The mode you're working on is too far towards the "PV"
> side to be called "hybrid".  (And as we've seen, the term has
> already confused people, who interpreted it as basically PVHVM.)
> >
> >The issue with PV in HVM is that it limits PV to HVM container only. The
> >vision I had was that hybrid, a PV ops kernel that is somewhere in between
> >PV and HVM, could be configured with options. So, one could run hybrid
> >with say EPT off (altho, won't be supported this anymore). But generic name
> >like hybrid allows it in future to be flexible, instead of confined to a
> >specific. I suppose a PV guest could just be started with various options.
> In general, I think "PV" should mean, "Doesn't use legacy boot,
> doesn't need emulated devices".  So I don't think "PVH" places any
> limitations on what particular subset of HVM hardware you use.  For
> things that specifically depend on knowing whether guest PTs are
> using mfns or gpfns, I think we should have checks for specific
> things -- for instance, "xen_mm_translate()" or something like that.


I like that.. We currently have 'if (feature(AUTOTRANSLATE)' .. blah
blah sprinkled around.

If we altered it to be 'if (xen_mm_translate())' and replaced a bunch
of 'if (xen_pv_domain())' with that it should make it easier. It
might even make the 'xen_hybrid_domain()' not needed at all.

This is good - it would also allow to remove some of the 'xen_hvm_domain()'
with it.

> 
> Also, don't confuse EPT (which is Intel-specific) with HAP (which is
> the generic term for either EPT or RVI); and don't confuse either of
> those with what is called "translate" mode.  Translate mode (where
> Xen translates the guest PTs from gpfns to mfns) can be done either
> with HAP or with shadow; and given the performance issues HAP has
> with certain workloads, we need to make sure that the HVM container
> mode can use both.
> 
> >As for name in code, 'pvh' was confusing, as PVHVM is now routinely used to
> >refer to HVM with PV drivers. 'hpv' for HVM/hybrid PV, well, thats a certain
> >virus ;). So I just used hybrid in the code to refer to PV guest that runs
> >in HVM container. I suppose I could change the flag to pv_in_hvm or
> >something.
> But is "pvhvm" ever actually used in the code?  If not, it's not a problem.
> 
> Actually, perhaps it would be better in any case, rather than having
> checks for "pvh" mode, to have checks for specific things -- e.g.,
> is translation on or off (i.e., are running in classic PV mode, or
> with HAP)?  I'm not sure the other things you're doing with HVM, but
> it should be possible to come up with a descriptive name to use in
> the code for those options, rather than limiting to a specific mode.
> 
> In ancient days, there used to be options, both within Xen and
> within the classic Xen kernel, to run a PV guest in fully-translated
> mode (i.e., the guest PTs contained gpfns, not mfns), and
> "shadow_translate" was a mode used across guest types (both PV and
> HVM) to determine whether this was the case or not.

dom0_shadow=true .. And strangly enought it looks to actually boot
the pvops kernel (dom0) without much issues. Wow. It must be faking it
I think - no bugs??!

  reply	other threads:[~2012-08-02 14:24 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-27  1:17 [HYBRID]: status update Mukesh Rathor
2012-06-27  2:10 ` Mukesh Rathor
2012-06-28  8:44 ` Ian Campbell
2012-07-03 19:24   ` Mukesh Rathor
2012-07-04 10:01     ` Stefano Stabellini
2012-07-04 23:13       ` Mukesh Rathor
2012-07-05 14:05         ` Stefano Stabellini
2012-08-01 15:25 ` George Dunlap
2012-08-01 15:25   ` Konrad Rzeszutek Wilk
2012-08-01 15:59     ` George Dunlap
2012-08-01 16:05       ` Konrad Rzeszutek Wilk
2012-08-01 16:21         ` George Dunlap
2012-08-01 16:51           ` Sander Eikelenboom
2012-08-01 16:55             ` Konrad Rzeszutek Wilk
2012-08-01 16:53           ` HYBRID naming [Was: Re: [HYBRID]: status update...] Matt Wilson
2012-08-02 11:41             ` George Dunlap
2012-08-04  1:06               ` Pasi Kärkkäinen
2012-08-01 16:05       ` [HYBRID]: status update Andrew Cooper
2012-08-01 16:08       ` Ian Campbell
2012-08-01 16:02   ` Sander Eikelenboom
2012-08-01 22:34   ` Mukesh Rathor
2012-08-02  9:53     ` George Dunlap
2012-08-02 14:24       ` Konrad Rzeszutek Wilk [this message]
2012-08-13 22:14       ` Mukesh Rathor
2012-08-14 10:44         ` Stefano Stabellini
2012-08-14 17:38           ` Mukesh Rathor
2012-08-14 17:42             ` Stefano Stabellini
2012-08-14 17:51               ` Mukesh Rathor
2012-08-15  8:19                 ` Jan Beulich
2012-08-15  9:42                   ` Ian Campbell
2012-08-15  9:54                     ` Jan Beulich
  -- strict thread matches above, loose matches on Subject: below --
2012-05-11  1:02 [HYBRID] : " Mukesh Rathor
2012-05-11  1:40 ` Konrad Rzeszutek Wilk
2012-05-11  8:03   ` Ian Campbell
2012-05-11  6:14 ` Pasi Kärkkäinen

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=20120802142431.GG16749@phenom.dumpdata.com \
    --to=konrad.wilk@oracle.com \
    --cc=Ian.Campbell@citrix.com \
    --cc=Stefano.Stabellini@eu.citrix.com \
    --cc=Xen-devel@lists.xensource.com \
    --cc=george.dunlap@eu.citrix.com \
    --cc=keir.xen@gmail.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 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).