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??!
next prev parent 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 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.