From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:58312) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1agvgF-0004vi-44 for qemu-devel@nongnu.org; Fri, 18 Mar 2016 10:47:11 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1agvgB-0006Js-95 for qemu-devel@nongnu.org; Fri, 18 Mar 2016 10:47:11 -0400 Received: from mx2.suse.de ([195.135.220.15]:60413) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1agvgB-0006J8-2C for qemu-devel@nongnu.org; Fri, 18 Mar 2016 10:47:07 -0400 References: <1457623170-30896-1-git-send-email-jgross@suse.com> <1458305544.6882.50.camel@redhat.com> From: Juergen Gross Message-ID: <56EC14E8.40103@suse.com> Date: Fri, 18 Mar 2016 15:47:04 +0100 MIME-Version: 1.0 In-Reply-To: <1458305544.6882.50.camel@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 0/2] usb, xen: add pvUSB backend List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Gerd Hoffmann Cc: xen-devel@lists.xensource.com, qemu-devel@nongnu.org, konrad.wilk@oracle.com, stefano.stabellini@eu.citrix.com On 18/03/16 13:52, Gerd Hoffmann wrote: > On Do, 2016-03-10 at 16:19 +0100, Juergen Gross wrote: >> This series adds a Xen pvUSB backend driver to qemu. USB devices >> connected to the host can be passed through to a Xen guest. The >> devices are specified via Xenstore. Access to the devices is done >> via host-libusb.c > >> I've tested the backend with various USB devices (memory sticks, >> keyboard, ...). > > Patches look sane to me. > > Have you tested both virtual and physical devices? Given how it is > written devices such as the virtual usb tablet should work just fine > too. I tested with physical devices only. TBH, I don't think a virtual device would work, given how the to be used device is selected (driver="usb-host", hostbus, hostport). > I can take that through the usb queue, but I'd like to see someone from > xen have a look at this too. Reviews anyone? Awesome, thanks for the thumbs up! Juergen From mboxrd@z Thu Jan 1 00:00:00 1970 From: Juergen Gross Subject: Re: [PATCH v2 0/2] usb, xen: add pvUSB backend Date: Fri, 18 Mar 2016 15:47:04 +0100 Message-ID: <56EC14E8.40103@suse.com> References: <1457623170-30896-1-git-send-email-jgross@suse.com> <1458305544.6882.50.camel@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <1458305544.6882.50.camel@redhat.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: Gerd Hoffmann Cc: xen-devel@lists.xensource.com, qemu-devel@nongnu.org, stefano.stabellini@eu.citrix.com List-Id: xen-devel@lists.xenproject.org T24gMTgvMDMvMTYgMTM6NTIsIEdlcmQgSG9mZm1hbm4gd3JvdGU6Cj4gT24gRG8sIDIwMTYtMDMt MTAgYXQgMTY6MTkgKzAxMDAsIEp1ZXJnZW4gR3Jvc3Mgd3JvdGU6Cj4+IFRoaXMgc2VyaWVzIGFk ZHMgYSBYZW4gcHZVU0IgYmFja2VuZCBkcml2ZXIgdG8gcWVtdS4gVVNCIGRldmljZXMKPj4gY29u bmVjdGVkIHRvIHRoZSBob3N0IGNhbiBiZSBwYXNzZWQgdGhyb3VnaCB0byBhIFhlbiBndWVzdC4g VGhlCj4+IGRldmljZXMgYXJlIHNwZWNpZmllZCB2aWEgWGVuc3RvcmUuIEFjY2VzcyB0byB0aGUg ZGV2aWNlcyBpcyBkb25lCj4+IHZpYSBob3N0LWxpYnVzYi5jCj4gCj4+IEkndmUgdGVzdGVkIHRo ZSBiYWNrZW5kIHdpdGggdmFyaW91cyBVU0IgZGV2aWNlcyAobWVtb3J5IHN0aWNrcywKPj4ga2V5 Ym9hcmQsIC4uLikuCj4gCj4gUGF0Y2hlcyBsb29rIHNhbmUgdG8gbWUuCj4gCj4gSGF2ZSB5b3Ug dGVzdGVkIGJvdGggdmlydHVhbCBhbmQgcGh5c2ljYWwgZGV2aWNlcz8gIEdpdmVuIGhvdyBpdCBp cwo+IHdyaXR0ZW4gZGV2aWNlcyBzdWNoIGFzIHRoZSB2aXJ0dWFsIHVzYiB0YWJsZXQgc2hvdWxk IHdvcmsganVzdCBmaW5lCj4gdG9vLgoKSSB0ZXN0ZWQgd2l0aCBwaHlzaWNhbCBkZXZpY2VzIG9u bHkuCgpUQkgsIEkgZG9uJ3QgdGhpbmsgYSB2aXJ0dWFsIGRldmljZSB3b3VsZCB3b3JrLCBnaXZl biBob3cgdGhlIHRvIGJlIHVzZWQKZGV2aWNlIGlzIHNlbGVjdGVkIChkcml2ZXI9InVzYi1ob3N0 IiwgaG9zdGJ1cywgaG9zdHBvcnQpLgoKPiBJIGNhbiB0YWtlIHRoYXQgdGhyb3VnaCB0aGUgdXNi IHF1ZXVlLCBidXQgSSdkIGxpa2UgdG8gc2VlIHNvbWVvbmUgZnJvbQo+IHhlbiBoYXZlIGEgbG9v ayBhdCB0aGlzIHRvby4gIFJldmlld3MgYW55b25lPwoKQXdlc29tZSwgdGhhbmtzIGZvciB0aGUg dGh1bWJzIHVwIQoKCkp1ZXJnZW4KCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fClhlbi1kZXZlbCBtYWlsaW5nIGxpc3QKWGVuLWRldmVsQGxpc3RzLnhlbi5v cmcKaHR0cDovL2xpc3RzLnhlbi5vcmcveGVuLWRldmVsCg==