From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:32963) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Uv1v2-0002ZB-CY for qemu-devel@nongnu.org; Fri, 05 Jul 2013 05:03:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Uv1v1-0000ly-19 for qemu-devel@nongnu.org; Fri, 05 Jul 2013 05:03:08 -0400 Received: from mx3-phx2.redhat.com ([209.132.183.24]:50844) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Uv1v0-0000ka-QN for qemu-devel@nongnu.org; Fri, 05 Jul 2013 05:03:06 -0400 Date: Fri, 5 Jul 2013 05:02:56 -0400 (EDT) From: Paolo Bonzini Message-ID: <2039015654.6402905.1373014976350.JavaMail.root@redhat.com> In-Reply-To: <51D686F6.8020606@m2r.biz> References: <1372932359-4625-1-git-send-email-fabio.fantoni@m2r.biz> <51D54B8B.9010703@m2r.biz> <20130704103224.GT7483@zion.uk.xensource.com> <51D564CB.9090602@m2r.biz> <51D5AE39.2010606@suse.de> <51D5D676.2080901@redhat.com> <51D686F6.8020606@m2r.biz> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [Xen-devel] [PATCH] libxl: Spice usbredirection support for upstream qemu List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Fabio Fantoni Cc: xen-devel@lists.xensource.com, Wei Liu , Ian Campbell , Stefano Stabellini , Ian Jackson , qemu-devel@nongnu.org, Hans de Goede , spice-devel@lists.freedesktop.org, Andreas =?utf-8?Q?F=C3=A4rber?= ----- Messaggio originale ----- > Da: "Fabio Fantoni" > A: "Paolo Bonzini" > Cc: "Andreas F=C3=A4rber" , "Stefano Stabellini" , > xen-devel@lists.xensource.com, "Wei Liu" , "Ian Camp= bell" , "Ian > Jackson" , qemu-devel@nongnu.org, "Hans de Goe= de" , > spice-devel@lists.freedesktop.org > Inviato: Venerd=C3=AC, 5 luglio 2013 10:42:30 > Oggetto: Re: [Xen-devel] [PATCH] libxl: Spice usbredirection support for = upstream qemu >=20 > Il 04/07/2013 22:09, Paolo Bonzini ha scritto: > > Il 04/07/2013 19:17, Andreas F=C3=A4rber ha scritto: > >> Also if at some point you try to use, e.g., q35 rather than i440fx > >> machine, it might have EHCI by default and the snippet would be > >> instantiating a duplicate one. I.e., the three levels of dependency > >> could be separated more clearly: PCI, USB, redirection. > > At least you could start specifying the USB desired version (1.1, 2.0 > > with companion controllers, 3.0) as an extension of the "usb=3D1" that = is > > already supported. > > > > Paolo >=20 > Thanks and sorry, I found only now that is working also with only > -device usb-ehci,id=3Dusb. > I'll post a new patch version. With only the EHCI you might have problems redirecting USB 1.1 devices. The EHCI+UHCI magic is the right thing to do for USB 2.0 support on x86. What Andreas and I were suggesting was to split the patch in two parts: first adding support for USB 2.0/3.0, and only then adding redirection (perhaps with the possibility to customize how many redirected devices to support). Paolo