xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Andrew Cooper <andrew.cooper3@citrix.com>
Cc: suravee.suthikulpanit@amd.com, Aravind.Gopalakrishnan@amd.com,
	Jan Beulich <JBeulich@suse.com>,
	xen-devel <xen-devel@lists.xenproject.org>,
	boris.ostrovsky@oracle.com
Subject: Re: [RFH]: AMD CR intercept for lmsw/clts
Date: Fri, 15 Aug 2014 17:04:20 -0400	[thread overview]
Message-ID: <20140815210420.GA22144@laptop.dumpdata.com> (raw)
In-Reply-To: <53E1F69D.800@citrix.com>

On Wed, Aug 06, 2014 at 10:34:21AM +0100, Andrew Cooper wrote:
> On 05/08/2014 23:30, Mukesh Rathor wrote:
> > On Tue, 05 Aug 2014 14:00:25 +0100
> > Andrew Cooper <andrew.cooper3@citrix.com> wrote:
> >
> >> On 05/08/2014 13:11, Jan Beulich wrote:
> >>>>>> On 05.08.14 at 13:16, <andrew.cooper3@citrix.com> wrote:
> >>>> On 05/08/2014 08:46, Jan Beulich wrote:
> > ...
> >
> >> Despite the current limitations, I firmly believe that PVH should be
> >> HVM
> >> - device model, rather than PV + VMX/SVM.  
> > I think that might be a dangerous route to take, classifying upfront
> > whether it's that way or the other. Eg, if we say it's former, then
> > anyone adding any feature would not examine the best approach, but just
> > take hvm approach.
> 
> There are many PV-isms which already exist for HVM.  Saying "HVM -
> device model" does not preclude further PVism from being introduced and
> used.  It does however means that PV-aware HVM guests get equal
> opportunity at these improvements.  Fundamentally, having PVH closer to
> HVM than PV means fewer modifications required to turn a native kernel
> into a PVH kernel, which is a *very* good thing from the point of view
> of the kernel authors.

Right. I would like to stress that the x86 maintainers are excited
about this as it would remove the pvops that don't have clear
semantic.
> 
> But as I said, this is only my opinion.
> 
> >
> >> Fundamentally, the end goal of PVH needs deciding ASAP, and
> >> documenting, to help guide decisions like this.
> > I think it's decided somewhat. Evolve to one of three approaches: PV,
> > HVM, or alternate, picking the easiest and fastest. IMO, at the very
> > least, pvh should retain "guest modified" characteristic, that would be
> > good for xen future imho.
> 
> It clearly is not decided, or even semi-certain, by virtue of having
> this conversation.

HA!
> 
> There are currently many opinions (some of which certainly can't
> coexist, many which can), a lot of semi-baked code with many
> restrictions (and repeated breaking of PVH/PVHdom0 by making seemingly
> innocent code changes elsewhere), and no concrete plan of what PVH is or
> what it should be.
> 
> What needs to happen urgently is for someone to make a firm decision,
> and prepare a document for /docs/specs/pvh.  A document like that is not
> immutable in the future if hindsight shows otherwise, but it will
> provide solid guidance as to how to proceed in matters like this.

That could certainly be done but I think we are all tied in fixing
code and trying to get features in Xen 4.5 before the feature
freeze gates are shut.

It should be fairly easy as most of it is 'runs like HVM' with
some HVM-ism disabled (so point to Intel SDM and AMD). And then
going through the hypercalls and seeing which are enabled.

Then there is the business of the startup which is complex, but
fortunatly there is a Wiki page to rip:
http://wiki.xenproject.org/wiki/X86_Paravirtualised_Memory_Management

Andrew, that nice template you used for the migrationv2 - where can
one find it?

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

  reply	other threads:[~2014-08-15 21:04 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-05  1:33 [RFH]: AMD CR intercept for lmsw/clts Mukesh Rathor
2014-08-05  7:46 ` Jan Beulich
2014-08-05 11:16   ` Andrew Cooper
2014-08-05 12:11     ` Jan Beulich
2014-08-05 13:00       ` Andrew Cooper
2014-08-05 13:15         ` Jan Beulich
2014-08-05 22:30         ` Mukesh Rathor
2014-08-06  9:34           ` Andrew Cooper
2014-08-15 21:04             ` Konrad Rzeszutek Wilk [this message]
2014-08-15 21:48               ` Andrew Cooper
2014-08-05 22:22   ` Mukesh Rathor

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=20140815210420.GA22144@laptop.dumpdata.com \
    --to=konrad.wilk@oracle.com \
    --cc=Aravind.Gopalakrishnan@amd.com \
    --cc=JBeulich@suse.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=boris.ostrovsky@oracle.com \
    --cc=suravee.suthikulpanit@amd.com \
    --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).