Hi, On Thu, Jul 11, 2019 at 03:15:12PM +0300, Yuri Benditovich wrote: > I have some patches ready with significant simplification of > usb-device-manager and there are even more possibilities, if we can > move to libusb >= 1.21. Cool > Do we have some objections against 1.21 (currently min. version > is 1.16)? That's already the version in RHEL 7. I don't think it'd be a problem to make it a requirement. > > Thanks, > Yuri > > > > On Thu, Jul 11, 2019 at 2:45 PM Victor Toso <victortoso@xxxxxxxxxx> wrote: > > > > Hi, > > > > On Wed, Jul 10, 2019 at 03:22:38PM +0200, Victor Toso wrote: > > > Hi, > > > > > > On Wed, Jul 10, 2019 at 10:49:21AM +0100, Frediano Ziglio wrote: > > > > This layer communicates with libusb and libusbredir and > > > > provides the API for all the operations related to USB > > > > redirection. All other modules of spice-gtk communicate > > > > only with usb backend instead of calling libusb and > > > > usbredirhost directly. > > > > This is prerequisite of further implementation of > > > > cd-sharing via USB redirection > > > > > > Looks good. I plan to do further test tomorrow morning and push, > > > if no one else complains. > > > > > > Thanks and sorry for taking quite long time to merge this. > > > > Tested with windows 10 and Fedora 29 guests. Seems to work fine. > > Debug is a bit verbose but should be fine till we get CD-ROM > > redirection patches in. I plan to work on some cleanup on > > usbredir stack as discussed earlier but I don't want that or > > minor changes to prevent further work on CD-ROM redir feature. > > > > I'll be pushing this now. > > > > Cheers, > > Victor > > > > > > > > Cheers, > > > Victor > > > > > > > > > > > Changes from v2: rebased on master > > > > > > > > Changes from v1: cosmetic changes per v1 review > > > > Added a rebased patch (3) from Frediano Ziglio > > > > For the increment see > > > > https://gitlab.freedesktop.org/yuri_benditovich/spice-gtk/commits/backend2-increment > > > > > > > > Yuri Benditovich (3): > > > > usb-redir: isolate usage of libusb and usbredirhost > > > > usb-redirection: do not duplicate USB device properties > > > > usb-device-manager: Define _SpiceUsbDevice instead of > > > > SpiceUsbDeviceInfo > > > > > > > > src/channel-usbredir-priv.h | 12 +- > > > > src/channel-usbredir.c | 236 ++---------- > > > > src/meson.build | 2 + > > > > src/usb-backend.c | 657 ++++++++++++++++++++++++++++++++++ > > > > src/usb-backend.h | 102 ++++++ > > > > src/usb-device-manager-priv.h | 1 - > > > > src/usb-device-manager.c | 397 +++++++------------- > > > > src/win-usb-dev.c | 103 ++---- > > > > src/win-usb-dev.h | 5 +- > > > > 9 files changed, 967 insertions(+), 548 deletions(-) > > > > create mode 100644 src/usb-backend.c > > > > create mode 100644 src/usb-backend.h > > > > > > > > -- > > > > 2.20.1 > > > > > > > > _______________________________________________ > > > > Spice-devel mailing list > > > > Spice-devel@xxxxxxxxxxxxxxxxxxxxx > > > > https://lists.freedesktop.org/mailman/listinfo/spice-devel > > > >
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel