xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: George Dunlap <george.dunlap@eu.citrix.com>
To: Ian Campbell <Ian.Campbell@citrix.com>
Cc: Anthony Perard <anthony.perard@citrix.com>,
	"sstanisi@cbnco.com" <sstanisi@cbnco.com>,
	Roger Pau Monne <roger.pau@citrix.com>,
	Ian Jackson <Ian.Jackson@eu.citrix.com>,
	"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
Subject: Re: [PATCH v6 1/2] libxl: Introduce functions to add and remove USB devices to an HVM guest
Date: Wed, 24 Apr 2013 16:45:24 +0100	[thread overview]
Message-ID: <5177FE14.4010800@eu.citrix.com> (raw)
In-Reply-To: <1366811466.20256.363.camel@zakaz.uk.xensource.com>

On 24/04/13 14:51, Ian Campbell wrote:
> On Wed, 2013-04-24 at 14:32 +0100, George Dunlap wrote:
>
>> There's also the translation of "AUTO" protocol into PV or HVM, and
> This made me wonder, how is libxl_device_usb_protocol different from the
> type of the domain? Can you (or is the intention) use PV with an HVM
> domain? I suppose DEVICEMODEL is HVM only?

The intention is to allow HVM guests to use either DEVICEMODEL or PV as 
protocols.

> Is "protocol" really the right word for this? I'd half expect it to mean
> USB 1.0 vs 2.0 vs 3.0. For NICS we call this Enum libxl_nic_type. FWIW

But we're already using 'type' for the device type. :-)

I think for nics it makes sense to call it a 'type', as for NICs we 
refer to the *emulated device* as a NIC, or the PV device as a NIC, 
which is then (virtually) plugged into a bridge somewhere.  But in this 
case I don't think it makes sense, as it's the actual host device you 
care about, and the way the guest talks to it is either via PV or qemu.  
"Protocol" may not be the very best option, but at least it gives you 
the idea of a conduit.

  -George

  reply	other threads:[~2013-04-24 15:45 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-19 15:59 [PATCH v6 1/2] libxl: Introduce functions to add and remove USB devices to an HVM guest George Dunlap
2013-04-19 15:59 ` [PATCH v6 2/2] xl: Add commands for usb hot-plug George Dunlap
2013-04-24 12:45   ` Ian Campbell
2013-04-25 10:16     ` George Dunlap
2013-04-25 11:38       ` Ian Campbell
2013-04-25 11:57         ` George Dunlap
2013-04-25 12:04           ` George Dunlap
2013-04-25 12:08           ` Ian Campbell
2013-04-25 12:14             ` George Dunlap
2013-04-25 12:21               ` George Dunlap
2013-04-25 12:45                 ` Ian Campbell
2013-04-25 12:47               ` Ian Campbell
2013-04-25 13:42           ` Pasi Kärkkäinen
2013-04-25 13:48             ` George Dunlap
2013-04-24 11:56 ` [PATCH v6 1/2] libxl: Introduce functions to add and remove USB devices to an HVM guest Ian Campbell
2013-04-24 12:48   ` George Dunlap
2013-04-24 12:53     ` Ian Campbell
2013-04-24 13:37       ` George Dunlap
2013-04-24 13:53         ` Ian Campbell
2013-04-24 12:38 ` Ian Campbell
2013-04-24 13:32   ` George Dunlap
2013-04-24 13:47     ` Ian Campbell
2013-04-24 13:51     ` Ian Campbell
2013-04-24 15:45       ` George Dunlap [this message]
2013-04-24 15:51         ` Ian Campbell
2013-04-24 17:49           ` Pasi Kärkkäinen
2013-04-25  7:44             ` Ian Campbell
2013-04-25  7:54               ` Pasi Kärkkäinen
2013-04-25  9:56               ` George Dunlap
2013-04-25 10:17                 ` Pasi Kärkkäinen
2013-04-25 14:19 ` Anthony PERARD
2013-04-25 14:31   ` George Dunlap

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=5177FE14.4010800@eu.citrix.com \
    --to=george.dunlap@eu.citrix.com \
    --cc=Ian.Campbell@citrix.com \
    --cc=Ian.Jackson@eu.citrix.com \
    --cc=anthony.perard@citrix.com \
    --cc=roger.pau@citrix.com \
    --cc=sstanisi@cbnco.com \
    --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 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).