Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-29 Thread Wei Liu
On Thu, Sep 17, 2015 at 10:54:23AM +0100, George Dunlap wrote: [...] > > Hi, George, > > > > I'm still confused about the expected look concerning PV/EMU type handling > > in > > this patch series. > > > > In earlier version, we tried to extract common things in libxl_usb.c and put > > pvusb

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-17 Thread Chun Yan Liu
>>> On 9/9/2015 at 12:52 AM, in message <55ef1244@citrix.com>, George Dunlap wrote: > On 09/08/2015 03:17 PM, Ian Campbell wrote: > > On Mon, 2015-08-10 at 18:35 +0800, Chunyan Liu wrote: > > > > Sorry for the delay, between 4.6 freeze crunch, conference and

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-17 Thread Chun Yan Liu
>>> On 9/9/2015 at 12:52 AM, in message <55ef1244@citrix.com>, George Dunlap wrote: > On 09/08/2015 03:17 PM, Ian Campbell wrote: > > On Mon, 2015-08-10 at 18:35 +0800, Chunyan Liu wrote: > > > > Sorry for the delay, between 4.6 freeze crunch, conference and

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-17 Thread George Dunlap
On 09/17/2015 09:19 AM, Chun Yan Liu wrote: > > On 9/9/2015 at 12:52 AM, in message <55ef1244@citrix.com>, George Dunlap > wrote: >> On 09/08/2015 03:17 PM, Ian Campbell wrote: >>> On Mon, 2015-08-10 at 18:35 +0800, Chunyan Liu wrote: >>> >>> Sorry

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-17 Thread Chun Yan Liu
; Juergen Gross writes ("Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb > API"): > >> On 09/14/2015 12:36 PM, George Dunlap wrote: > >> > Anyone want to look into the Linux source code to find out how big it > >> > will allow busnum / devnum to g

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-15 Thread Chun Yan Liu
>>> On 9/11/2015 at 09:26 PM, in message <1441978018.3549.33.ca...@citrix.com>, >>> Ian Campbell wrote: > On Thu, 2015-09-10 at 23:42 -0600, Chun Yan Liu wrote: > > > > > Do these fields have any particular size requirements arising from e.g. > the > > > USB

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-14 Thread Juergen Gross
On 09/14/2015 12:36 PM, George Dunlap wrote: On Mon, Sep 14, 2015 at 4:48 AM, Juergen Gross wrote: On 09/11/2015 04:41 PM, Ian Campbell wrote: On Fri, 2015-09-11 at 16:18 +0200, Juergen Gross wrote: On 09/11/2015 04:09 PM, Ian Campbell wrote: On Fri, 2015-09-11 at 15:55

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-14 Thread Juergen Gross
On 09/14/2015 01:12 PM, Ian Jackson wrote: Juergen Gross writes ("Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API"): On 09/14/2015 12:36 PM, George Dunlap wrote: Anyone want to look into the Linux source code to find out how big it will allow busnum / devnum to grow? driver

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-14 Thread George Dunlap
On Mon, Sep 14, 2015 at 4:48 AM, Juergen Gross wrote: > On 09/11/2015 04:41 PM, Ian Campbell wrote: >> >> On Fri, 2015-09-11 at 16:18 +0200, Juergen Gross wrote: >>> >>> On 09/11/2015 04:09 PM, Ian Campbell wrote: On Fri, 2015-09-11 at 15:55 +0200, Juergen Gross wrote:

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-14 Thread Ian Jackson
Juergen Gross writes ("Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API"): > On 09/14/2015 12:36 PM, George Dunlap wrote: > > Anyone want to look into the Linux source code to find out how big it > > will allow busnum / devnum to grow? > > drivers/usb/core/

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-14 Thread George Dunlap
On Mon, Sep 14, 2015 at 12:12 PM, Ian Jackson <ian.jack...@eu.citrix.com> wrote: > Juergen Gross writes ("Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API"): >> On 09/14/2015 12:36 PM, George Dunlap wrote: >> > Anyone want to look into the Lin

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-13 Thread Juergen Gross
On 09/11/2015 04:41 PM, Ian Campbell wrote: On Fri, 2015-09-11 at 16:18 +0200, Juergen Gross wrote: On 09/11/2015 04:09 PM, Ian Campbell wrote: On Fri, 2015-09-11 at 15:55 +0200, Juergen Gross wrote: On 09/11/2015 03:26 PM, Ian Campbell wrote: On Thu, 2015-09-10 at 23:42 -0600, Chun Yan Liu

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-11 Thread Ian Campbell
On Fri, 2015-09-11 at 15:55 +0200, Juergen Gross wrote: > On 09/11/2015 03:26 PM, Ian Campbell wrote: > > On Thu, 2015-09-10 at 23:42 -0600, Chun Yan Liu wrote: > > > > > > > Do these fields have any particular size requirements arising from > > > > e.g. the > > > > USB spec or from possible dom0

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-11 Thread Ian Campbell
On Thu, 2015-09-10 at 23:42 -0600, Chun Yan Liu wrote: > > > Do these fields have any particular size requirements arising from e.g. the > > USB spec or from possible dom0 implementations? > > > > If they have a well defined fixed size from a USB spec then maybe we > > could > > use the

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-11 Thread Juergen Gross
On 09/11/2015 04:09 PM, Ian Campbell wrote: On Fri, 2015-09-11 at 15:55 +0200, Juergen Gross wrote: On 09/11/2015 03:26 PM, Ian Campbell wrote: On Thu, 2015-09-10 at 23:42 -0600, Chun Yan Liu wrote: Do these fields have any particular size requirements arising from e.g. the USB spec or from

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-11 Thread Ian Campbell
On Fri, 2015-09-11 at 16:18 +0200, Juergen Gross wrote: > On 09/11/2015 04:09 PM, Ian Campbell wrote: > > On Fri, 2015-09-11 at 15:55 +0200, Juergen Gross wrote: > > > On 09/11/2015 03:26 PM, Ian Campbell wrote: > > > > On Thu, 2015-09-10 at 23:42 -0600, Chun Yan Liu wrote: > > > > > > > > > > >

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-11 Thread Ian Jackson
Most of what you say is right, I think, but: Ian Campbell writes ("Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API"): > On Fri, 2015-09-11 at 16:18 +0200, Juergen Gross wrote: > > In theory I think up to 7 cascaded > > hubs are possible, but I don't think the resu

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-10 Thread Chun Yan Liu
>>> On 9/8/2015 at 10:17 PM, in message <1441721852.24450.120.ca...@citrix.com>, Ian Campbell wrote: > On Mon, 2015-08-10 at 18:35 +0800, Chunyan Liu wrote: > > Sorry for the delay, between 4.6 freeze crunch, conference and vacation > I've been a bit swamped. >

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-09 Thread Chun Yan Liu
>>> On 9/9/2015 at 12:52 AM, in message <55ef1244@citrix.com>, George Dunlap wrote: > On 09/08/2015 03:17 PM, Ian Campbell wrote: > > On Mon, 2015-08-10 at 18:35 +0800, Chunyan Liu wrote: > > > > Sorry for the delay, between 4.6 freeze crunch, conference and

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-08 Thread Ian Campbell
On Mon, 2015-08-10 at 18:35 +0800, Chunyan Liu wrote: Sorry for the delay, between 4.6 freeze crunch, conference and vacation I've been a bit swamped. I'm just going to comment on the APIs (mainly public libxl.h and .idl) in this pass. > diff --git a/tools/libxl/libxl.h b/tools/libxl/libxl.h >

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-09-08 Thread George Dunlap
On 09/08/2015 03:17 PM, Ian Campbell wrote: > On Mon, 2015-08-10 at 18:35 +0800, Chunyan Liu wrote: > > Sorry for the delay, between 4.6 freeze crunch, conference and vacation > I've been a bit swamped. > > I'm just going to comment on the APIs (mainly public libxl.h and .idl) in > this pass. >

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-08-17 Thread Chun Yan Liu
On 8/13/2015 at 05:09 PM, in message 20150813090938.gi7...@zion.uk.xensource.com, Wei Liu wei.l...@citrix.com wrote: On Tue, Aug 11, 2015 at 08:24:01PM -0600, Chun Yan Liu wrote: On 8/11/2015 at 07:27 PM, in message 2015082702.gf7...@zion.uk.xensource.com, Wei Liu

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-08-13 Thread Wei Liu
On Tue, Aug 11, 2015 at 08:24:01PM -0600, Chun Yan Liu wrote: On 8/11/2015 at 07:27 PM, in message 2015082702.gf7...@zion.uk.xensource.com, Wei Liu wei.l...@citrix.com wrote: On Mon, Aug 10, 2015 at 06:35:24PM +0800, Chunyan Liu wrote: Add pvusb APIs, including: -

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-08-13 Thread Chun Yan Liu
On 8/13/2015 at 05:09 PM, in message 20150813090938.gi7...@zion.uk.xensource.com, Wei Liu wei.l...@citrix.com wrote: On Tue, Aug 11, 2015 at 08:24:01PM -0600, Chun Yan Liu wrote: On 8/11/2015 at 07:27 PM, in message 2015082702.gf7...@zion.uk.xensource.com, Wei Liu

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-08-11 Thread Wei Liu
On Mon, Aug 10, 2015 at 06:35:24PM +0800, Chunyan Liu wrote: Add pvusb APIs, including: - attach/detach (create/destroy) virtual usb controller. - attach/detach usb device - list usb controller and usb devices - some other helper functions Signed-off-by: Chunyan Liu cy...@suse.com

Re: [Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-08-11 Thread Chun Yan Liu
On 8/11/2015 at 07:27 PM, in message 2015082702.gf7...@zion.uk.xensource.com, Wei Liu wei.l...@citrix.com wrote: On Mon, Aug 10, 2015 at 06:35:24PM +0800, Chunyan Liu wrote: Add pvusb APIs, including: - attach/detach (create/destroy) virtual usb controller. - attach/detach usb

[Xen-devel] [PATCH V6 3/7] libxl: add pvusb API

2015-08-10 Thread Chunyan Liu
Add pvusb APIs, including: - attach/detach (create/destroy) virtual usb controller. - attach/detach usb device - list usb controller and usb devices - some other helper functions Signed-off-by: Chunyan Liu cy...@suse.com Signed-off-by: Simon Cao caobosi...@gmail.com --- changes: - Address