From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [PATCH v0 RFC 0/2] xl/libxl support for PVUSB Date: Wed, 19 Nov 2014 16:32:27 -0500 Message-ID: <20141119213226.GA21408@laptop.dumpdata.com> References: <1407702234-22309-1-git-send-email-caobosimon@gmail.com> <5460E9D80200006600078038@soto.provo.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Simon Cao Cc: Lars Kurth , "xen-devel@lists.xensource.com" , Ian Campbell , George Dunlap , Chun Yan Liu , Ian Jackson List-Id: xen-devel@lists.xenproject.org On Sun, Nov 16, 2014 at 10:36:28AM +0800, Simon Cao wrote: > Hi, > = > I was working on the work. But I was busing preparing some job interviews > in the last three months, sorry for this long delay. I will update my > progress in a few days. OK, I put your name for this to be in Xen 4.6. Thanks! > = > Thanks! > = > Bo Cao > = > On Mon, Nov 10, 2014 at 4:37 PM, Chun Yan Liu wrote: > = > > Is there any progress on this work? I didn't see new version after this. > > Anyone knows the status? > > > > Thanks, > > Chunyan > > > > >>> On 8/11/2014 at 04:23 AM, in message > > <1407702234-22309-1-git-send-email-caobosimon@gmail.com>, Bo Cao > > wrote: > > > Finally I have a workable version xl/libxl support for PVUSB. Most of > > > its commands work property now, but there are still some probelm to be > > > solved. > > > Please take a loot and give me some advices. > > > > > > =3D=3D What have been implemented ? =3D=3D > > > I have implemented libxl functions for PVUSB in libxl_usb.c. It mainly > > > consists of two part: > > > usbctrl_add/remove/list and usb_add/remove/list in which usbctrl deno= te > > usb > > > controller in which > > > usd device can be plugged in. I don't use "ao_dev" in > > > libxl_deivce_usbctrl_add since we don't need to > > > execute hotplug script for usbctrl and without "ao_dev", adding defau= lt > > > usbctrl for usb device > > > would be easier. > > > > > > For the cammands to manipulate usb device such as "xl usb-attach" and= "xl > > > usb-detach", this patch now only > > > support to specify usb devices by their interface in sysfs. Using this > > > interface, we can read usb device > > > information through sysfs and bind/unbind usb device. (The support for > > > mapping the "lsusb" bus:addr to the > > > sysfs usb interface will come later). > > > > > > =3D=3D What needs to do next ? =3D=3D > > > There are two main problems to be solved. > > > > > > 1. PVUSB Options in VM Guest's Configuration File > > > The interface in VM Guest's configuration file to add usb device = is: > > > "usb=3D[interface=3D"1-1"]". > > > But the problem is now is that after the default usbctrl is added, the > > state > > > of usbctrl is "2", e,g, "XenbusStateInitWait", > > > waiting for xen-usbfront to connect. The xen-usbfront in VM Guest isn= 't > > > loaded. Therefore, "sysfs_intf_write" > > > will report error. Does anyone have any clue how to solve this? > > > > > > 2. sysfs_intf_write > > > In the process of "xl usb-attach domid intf=3D1-1", after writing > > "1-1" to > > > Xenstore entry, we need to > > > bind the controller of this usb device to usbback driver so that it c= an > > be > > > used by VM Guest. For exampele, > > > for usb device "1-1", it's controller interface maybe "1-1:1.0", and = we > > > write this value to "/sys/bus/usb/driver/usbback/bind". > > > But for some devices, they have two controllers, for example "1-1:1.0" > > and > > > "1-1:1.1". I think this means it has two functions, > > > such as usbhid and usb-storage. So in this case, we bind the two > > controller > > > to usbback? > > > > > > =3D=3D=3D=3D=3D=3D=3D=3D > > > There maybe some errors or bugs in the codes. Feel free to tell me. > > > > > > Cheers, > > > > > > - Simon > > > > > > --- > > > CC: George Dunlap > > > CC: Ian Jackson > > > CC: Ian Campbell > > > CC: Pasi K=E4rkk=E4inen > > > CC: Lars Kurth > > > > > > > > > > > > _______________________________________________ > > > Xen-devel mailing list > > > Xen-devel@lists.xen.org > > > http://lists.xen.org/xen-devel > > > > > > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xen.org > http://lists.xen.org/xen-devel