xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Mukesh Rathor <mukesh.rathor@oracle.com>
To: Tim Deegan <tim@xen.org>
Cc: George Dunlap <george.dunlap@eu.citrix.com>,
	xen-devel <xen-devel@lists.xenproject.org>,
	keir.xen@gmail.com, Jan Beulich <JBeulich@suse.com>
Subject: Re: [RFC 0 PATCH 3/3] PVH dom0: construct_dom0 changes
Date: Wed, 9 Oct 2013 15:31:20 -0700	[thread overview]
Message-ID: <20131009153120.1528e37b@mantra.us.oracle.com> (raw)
In-Reply-To: <20131009175040.GD40780@ocelot.phlegethon.org>

On Wed, 9 Oct 2013 18:50:40 +0100
Tim Deegan <tim@xen.org> wrote:

> At 08:30 -0400 on 08 Oct (1381221032), Konrad Rzeszutek Wilk wrote:
> > George Dunlap <george.dunlap@eu.citrix.com> wrote:
> > >If it were just a question of cleaning up those bits, I could
> > >probably have another draft posted sometime this week.  But if
> > >we're stepping back and looking at whether this is the right
> > >approach, or whether something like Tim has suggested -- basically
> > >making PVH to be HVM minus 
> > >qemu plus a handful of hypercalls, and most of the changes in the
> > >domain 
> > >builder rather than in Xen -- that will take a bit longer,
> > >particularly
> > >
> > >because it would probably mean me having to understand and modify
> > >the Linux side of things as well.  At this point I'm not really
> > >sure what the best approach is going forward.
> > 
> > Arrg.  I am not really sure how to express myself here but from the
> > start Mukesh has asking for assistance and review of ideas and
> > design of this and gotten it and acted on it. Now after two years
> > of going this path folks are suggesting a new design?
> 
> Sorry for not making the suggestion sooner -- it honestly hadn't
> occurred to me.  I had read a number of revisions of the PVH Xen
> patches (and many discussions of what the new type of guests should
> be called) before thinking that there didn't need to be a new kind of
> guest at all.

The orig series didn't have a new guest type, but I was sorta compelled
to create one. From guest perspective, it's in PV mode with auto
translate. It appears this is now losing sight of the orig goal. The
main problem being solved was 64bit PV syscall overhead. The best way to
solve the sys call overhead was to run in HVM container. I discussed
briefly with Ian P and then Keir on the sidelines of xen summit. Then I
did code walk with Steffano and Ian C at hackathon last year. Looked
like we were all on same page.

Still, if something could be done better, I'm in favor. But, lets
get something done asap so as to not get left behind. Everyone will
have a different opinion on the best approach, and unless some
compromises are made things can just drag on..... 

> FWIW, I don't think this would be a complete redesign. AFAICT the
> guest kernel changes would stay as they are, and most of the
> toolstack changes too.  Some of the Xen changes would stay
> (implementation of setvcpuinfo, for example) and some would just go
> away.

Good idea, but keep in mind that dom0 will need most of the xen 
changes for coming in PVH mode. So, there might not be enough to make
it worthwhile.

thanks
Mukesh

  reply	other threads:[~2013-10-09 22:31 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-25 21:03 [RFC 0 PATCH 0/3]: PVH dom0 construction Mukesh Rathor
2013-09-25 21:03 ` [RFC 0 PATCH 1/3] PVH dom0: create domctl_memory_mapping() function Mukesh Rathor
2013-09-26  7:03   ` Jan Beulich
2013-09-25 21:03 ` [RFC 0 PATCH 2/3] PVH dom0: move some pv specific code to static functions Mukesh Rathor
2013-09-26  7:21   ` Jan Beulich
2013-09-26 23:32     ` Mukesh Rathor
2013-09-25 21:03 ` [RFC 0 PATCH 3/3] PVH dom0: construct_dom0 changes Mukesh Rathor
2013-09-26  8:02   ` Jan Beulich
2013-09-27  0:17     ` Mukesh Rathor
2013-09-27  6:54       ` Jan Beulich
2013-10-03  0:53         ` Mukesh Rathor
2013-10-04  6:53           ` Jan Beulich
2013-10-04 13:35             ` Konrad Rzeszutek Wilk
2013-10-04 14:05               ` Jan Beulich
2013-10-04 16:02                 ` Konrad Rzeszutek Wilk
2013-10-04 16:07                   ` Jan Beulich
2013-10-04 20:59                     ` Konrad Rzeszutek Wilk
2013-10-05  1:06                       ` Mukesh Rathor
2013-10-07  7:12                         ` Jan Beulich
2013-10-08  0:58             ` Mukesh Rathor
2013-10-08  7:51               ` Jan Beulich
2013-10-08  8:03                 ` Jan Beulich
2013-10-08  9:39                   ` George Dunlap
2013-10-08  9:57                     ` Jan Beulich
2013-10-08 10:01                       ` George Dunlap
2013-10-08 10:19                         ` Lars Kurth
2013-10-08 12:30                     ` Konrad Rzeszutek Wilk
2013-10-09 13:02                       ` George Dunlap
2013-10-09 13:13                         ` Andrew Cooper
2013-10-09 13:16                           ` George Dunlap
2013-10-09 14:37                             ` Andrew Cooper
2013-10-09 17:50                       ` Tim Deegan
2013-10-09 22:31                         ` Mukesh Rathor [this message]
2013-09-27  1:55     ` Mukesh Rathor
2013-09-27  7:01       ` Jan Beulich
2013-09-27 23:03         ` Mukesh Rathor
2013-09-30  6:56           ` Jan Beulich
2013-10-08  0:52             ` Mukesh Rathor
2013-10-08  7:43               ` Jan Beulich
2013-10-09 21:59                 ` 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=20131009153120.1528e37b@mantra.us.oracle.com \
    --to=mukesh.rathor@oracle.com \
    --cc=JBeulich@suse.com \
    --cc=george.dunlap@eu.citrix.com \
    --cc=keir.xen@gmail.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).