All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ian Campbell <ian.campbell@citrix.com>
To: Jan Beulich <JBeulich@suse.com>
Cc: Jun Nakajima <jun.nakajima@intel.com>, Tim Deegan <tim@xen.org>,
	Kevin Tian <kevin.tian@intel.com>, Keir Fraser <keir@xen.org>,
	Eddie Dong <eddie.dong@intel.com>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
	George Dunlap <george.dunlap@eu.citrix.com>,
	Andrew Cooper <andrew.cooper3@citrix.com>,
	Ian Jackson <ian.jackson@eu.citrix.com>,
	Don Slutz <don.slutz@gmail.com>, Don Slutz <dslutz@verizon.com>,
	xen-devel@lists.xen.org,
	Aravind Gopalakrishnan <Aravind.Gopalakrishnan@amd.com>,
	Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>,
	BorisOstrovsky <boris.ostrovsky@oracle.com>
Subject: Re: [PATCH v11 6/9] xen: Add ring 3 vmware_port support
Date: Fri, 5 Jun 2015 11:54:02 +0100	[thread overview]
Message-ID: <1433501642.7108.211.camel@citrix.com> (raw)
In-Reply-To: <557188840200007800081342@mail.emea.novell.com>

On Fri, 2015-06-05 at 10:31 +0100, Jan Beulich wrote:
> > I'm talking about cost-benefits analysis.  What's the benefit of
> > accepting this patch, and is it worth the cost?
> 
> The basic idea of allowing guests originally having got installed on
> VMware to continue their lives on Xen is certainly something worth
> accepting some cost. It's really hard to judge whether in the case
> here things go too far (and that would equally apply to the hand
> crafted instruction decoding done in earlier versions of this series).

I can see the benefit in having a guest which was installed on vmware be
able to boot and work on Xen.

But AIUI this userspace vmware port thing is not needed for that basic
use case but instead goes farther and enables advanced features like
clip boards integration, which TBH I think we could consider living
without (especially considering the costs discussed here).

It would be really useful to see a comprehensive list of exactly what
guest ring3 access to the vmware port actually enables i.e. a list of
specific features which require it.

Ian.

  reply	other threads:[~2015-06-05 10:54 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-22 15:50 [PATCH v11 0/9] Xen VMware tools support Don Slutz
2015-05-22 15:50 ` [PATCH v11 1/9] tools: Add vga=vmware Don Slutz
2015-05-22 15:50 ` [PATCH v11 2/9] xen: Add support for VMware cpuid leaves Don Slutz
2015-05-22 15:50 ` [PATCH v11 3/9] tools: Add vmware_hwver support Don Slutz
2015-06-03 14:53   ` George Dunlap
2015-06-04 15:15     ` Ian Campbell
2015-06-04 15:46       ` Don Slutz
2015-06-04 15:17   ` Ian Campbell
2015-06-04 15:59     ` Don Slutz
2015-05-22 15:50 ` [PATCH v11 4/9] vmware: Add VMware provided include file Don Slutz
2015-05-22 15:50 ` [PATCH v11 5/9] xen: Add vmware_port support Don Slutz
2015-06-05  9:52   ` Jan Beulich
2015-06-05 13:18     ` Don Slutz
2015-05-22 15:50 ` [PATCH v11 6/9] xen: Add ring 3 " Don Slutz
2015-06-03 15:26   ` George Dunlap
2015-06-03 15:58     ` Andrew Cooper
2015-06-03 16:23       ` George Dunlap
2015-06-03 16:40         ` Andrew Cooper
2015-06-03 17:00           ` George Dunlap
2015-06-03 16:41         ` Don Slutz
2015-06-03 16:58           ` George Dunlap
2015-06-04 12:37             ` Don Slutz
2015-06-04 14:14               ` George Dunlap
2015-06-04 16:17                 ` Don Slutz
2015-06-03 16:36       ` Don Slutz
2015-06-03 16:50         ` George Dunlap
2015-06-05  9:31           ` Jan Beulich
2015-06-05 10:54             ` Ian Campbell [this message]
2015-06-11 22:10               ` Don Slutz
2015-06-12  6:25                 ` Jan Beulich
2015-06-12 12:52                   ` Don Slutz
2015-06-23 16:14   ` Jan Beulich
2015-06-26 14:54     ` Don Slutz
2015-05-22 15:50 ` [PATCH v11 7/9] tools: Add " Don Slutz
2015-06-03 17:06   ` George Dunlap
2015-06-04 15:49     ` Ian Campbell
2015-06-04 16:09       ` Don Slutz
2015-06-04 15:20   ` Ian Campbell
2015-05-22 15:50 ` [PATCH v11 8/9] Add IOREQ_TYPE_VMWARE_PORT Don Slutz
2015-06-03 17:09   ` George Dunlap
2015-06-04 11:28     ` Don Slutz
2015-06-05  9:35       ` Jan Beulich
2015-06-05 10:03         ` Paul Durrant
2015-06-08 10:05       ` George Dunlap
2015-06-11 21:51         ` Don Slutz
2015-05-22 15:50 ` [PATCH v11 9/9] Add xentrace to vmware_port Don Slutz
2015-06-04 11:20   ` George Dunlap
2015-06-04 12:31     ` Don Slutz

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=1433501642.7108.211.camel@citrix.com \
    --to=ian.campbell@citrix.com \
    --cc=Aravind.Gopalakrishnan@amd.com \
    --cc=JBeulich@suse.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=boris.ostrovsky@oracle.com \
    --cc=don.slutz@gmail.com \
    --cc=dslutz@verizon.com \
    --cc=eddie.dong@intel.com \
    --cc=george.dunlap@eu.citrix.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=jun.nakajima@intel.com \
    --cc=keir@xen.org \
    --cc=kevin.tian@intel.com \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=suravee.suthikulpanit@amd.com \
    --cc=tim@xen.org \
    --cc=xen-devel@lists.xen.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 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.