[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Patch V1 2/3] xen/usb: add capability for passing thro
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [Patch V1 2/3] xen/usb: add capability for passing through isoc jobs to host devices |
Date: |
Wed, 09 Sep 2015 14:01:00 +0200 |
Hi,
> > So, the signaling needs to be different. The host adapter needs to
> > signal somehow that it can handle async iso packets. One way would be
> > to flag this per usb bus, another one per usb packet. Also all xen
> > naming and the xen inlude should go away. BTW: does this build without
> > xen-devel installed?
>
> Okay, I'll try to make it more generic. I think the async iso capability
> should be a bus attribute.
Makes sense.
> > Can we get rid of the callbacks? By filling the USBPacket iovec with
> > the iso request chunks for example?
>
> Difficult. One iso request chunk could require multiple iovec entries.
Why multiple small iovecs instead of one big iovec?
usb_host_req_complete_iso_xen() returns a single status for the whole
USBPacket anyway ...
> The RFC version tried to avoid the callbacks and there you didn't like
> exposing the additional structures.
-ENOPATCH.
cheers,
Gerd
Re: [Qemu-devel] [Xen-devel] [Patch V1 2/3] xen/usb: add capability for passing through isoc jobs to host devices, Konrad Rzeszutek Wilk, 2015/09/15
[Qemu-devel] [Patch V1 3/3] xen: add pvUSB backend, Juergen Gross, 2015/09/03