Re: [spice-gtk v4 00/13] CD sharing feature

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On Tue, Sep 18, 2018 at 7:04 PM, Yuri Benditovich <yuri.benditovich@xxxxxxxxxx> wrote:


On Tue, Sep 18, 2018 at 1:52 PM, Marc-André Lureau <marcandre.lureau@xxxxxxxxx> wrote:
Hi

On Mon, Sep 17, 2018 at 5:24 PM Yuri Benditovich
<yuri.benditovich@xxxxxxxxxx> wrote:
>
> This series of patches adds ability to share ISO files or
> local CD devices with guest system using USB redirection
> by emulation of USB CD device.

and DVD, I guess.

How much of this "new" emulation code overlaps with existing USB code in QEMU ?

AFAIU, no such overlapping. QEMU works with guest usbredir, spice-gtk with host (libusb) and
parser (cd)
 

Gerd, as QEMU USB storage maintainer, your help reviewing this code is
very much welcome!

> For more information see commit comment of patch 13.
>
> Currently both new usb device widget and old one
> present in the code and the old one can be selected
> by configuration. If required, old usb device widget
> will be removed in separate commit
>
> Changes from v3:
> Single commit with all new files split to 12 patches
> to make patch review easier
>

It would help review iterations (at least for CI) if you could hold
your WIP tree on gitlab.freedesktop.org

 


thanks

> Yuri Benditovich (13):
>   cd-sharing: add cd-device header file
>   cd-sharing: implement cd-device for Linux
>   cd-sharing: implement cd-device for Windows
>   cd-sharing: add header file with SCSI definitions
>   cd-sharing: introduce header file for SCSI device parameters
>   cd-sharing: add interface definition for SCSI device
>   cd-sharing: add implementation of SCSI device
>   cd-sharing: added interface file for USB mass storage device
>   cd-sharing: added implementation of USB mass storage device
>   cd-charing: added interface definition for USB backend device
>   cd-sharing: added implementation of USB backend
>   cd-sharing: implementation of new USB widget
>   cd-sharing: Implementation of CD sharing feature
>
configure.ac                  |   27 +
>  src/Makefile.am               |   15 +-
>  src/cd-device-linux.c         |  137 ++
>  src/cd-device-win.c           |  193 +++
>  src/cd-device.h               |   40 +
>  src/cd-scsi-dev-params.h      |   52 +
>  src/cd-scsi.c                 | 2765 +++++++++++++++++++++++++++++++++++++++++
>  src/cd-scsi.h                 |  122 ++
>  src/cd-usb-bulk-msd.c         |  555 +++++++++
>  src/cd-usb-bulk-msd.h         |  134 ++


>  src/channel-usbredir-priv.h   |    9 +-
>  src/channel-usbredir.c        |  267 ++--
>  src/map-file                  |    9 +
>  src/scsi-constants.h          |  324 +++++
>  src/spice-option.c            |   15 +
>  src/usb-backend-common.c      | 2015 ++++++++++++++++++++++++++++++
>  src/usb-backend.h             |  112 ++
>  src/usb-device-manager-priv.h |    5 +-
>  src/usb-device-manager.c      |  743 +++++++----
>  src/usb-device-manager.h      |  107 +-
>  src/usb-device-redir-widget.c | 2065 ++++++++++++++++++++++++++++++
>  src/usb-device-widget.c       |    5 +
>  src/usbutil.c                 |   52 +-
>  src/usbutil.h                 |    5 +-
>  src/win-usb-dev.c             |   66 +-
>  src/win-usb-dev.h             |    2 +
>  26 files changed, 9321 insertions(+), 520 deletions(-)
>  create mode 100644 src/cd-device-linux.c
>  create mode 100644 src/cd-device-win.c
>  create mode 100644 src/cd-device.h
>  create mode 100644 src/cd-scsi-dev-params.h
>  create mode 100644 src/cd-scsi.c
>  create mode 100644 src/cd-scsi.h
>  create mode 100644 src/cd-usb-bulk-msd.c
>  create mode 100644 src/cd-usb-bulk-msd.h
>  create mode 100644 src/scsi-constants.h
>  create mode 100644 src/usb-backend-common.c
>  create mode 100644 src/usb-backend.h
>  create mode 100644 src/usb-device-redir-widget.c
>
> --
> 2.9.4
>
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/spice-devel



--
Marc-André Lureau


_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]