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

2018-09-27 Thread Christophe Fergeau
On Fri, Sep 21, 2018 at 08:10:08PM +0200, Christophe Fergeau wrote: > > All your SPICE_DEBUG calls include a __FUNCTION__, but it seems > SPICE_DEBUG already does that for you? > > #define SPICE_DEBUG(fmt, ...) \ > do {

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

2018-09-25 Thread Christophe Fergeau
On Mon, Sep 24, 2018 at 06:56:59PM +0300, Yuri Benditovich wrote: > On Mon, Sep 24, 2018 at 4:32 PM Christophe Fergeau > wrote: > > > On Sat, Sep 22, 2018 at 12:21:04AM +0300, Yuri Benditovich wrote: > > > On Sat, Sep 22, 2018 at 12:14 AM, Yuri Benditovich < > > > yuri.benditov...@daynix.com> wro

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

2018-09-24 Thread Yuri Benditovich
On Mon, Sep 24, 2018 at 4:32 PM Christophe Fergeau wrote: > On Sat, Sep 22, 2018 at 12:21:04AM +0300, Yuri Benditovich wrote: > > On Sat, Sep 22, 2018 at 12:14 AM, Yuri Benditovich < > > yuri.benditov...@daynix.com> wrote: > > > > > Hi > > > > > > On Fri, Sep 21, 2018 at 9:10 PM, Christophe Ferge

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

2018-09-24 Thread Christophe Fergeau
On Sat, Sep 22, 2018 at 12:21:04AM +0300, Yuri Benditovich wrote: > On Sat, Sep 22, 2018 at 12:14 AM, Yuri Benditovich < > yuri.benditov...@daynix.com> wrote: > > > Hi > > > > On Fri, Sep 21, 2018 at 9:10 PM, Christophe Fergeau > > wrote: > > > >> On Mon, Sep 17, 2018 at 04:22:50PM +0300, Yuri Be

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

2018-09-21 Thread Yuri Benditovich
On Sat, Sep 22, 2018 at 12:14 AM, Yuri Benditovich < yuri.benditov...@daynix.com> wrote: > Hi > > On Fri, Sep 21, 2018 at 9:10 PM, Christophe Fergeau > wrote: > >> On Mon, Sep 17, 2018 at 04:22:50PM +0300, Yuri Benditovich wrote: >> > Changes from v3: >> > Single commit with all new files split t

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

2018-09-21 Thread Yuri Benditovich
Hi On Fri, Sep 21, 2018 at 9:10 PM, Christophe Fergeau wrote: > On Mon, Sep 17, 2018 at 04:22:50PM +0300, Yuri Benditovich wrote: > > Changes from v3: > > Single commit with all new files split to 12 patches > > to make patch review easier > > What you did is not splitting at all, adding one new

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

2018-09-21 Thread Christophe Fergeau
On Mon, Sep 17, 2018 at 04:22:50PM +0300, Yuri Benditovich wrote: > Changes from v3: > Single commit with all new files split to 12 patches > to make patch review easier What you did is not splitting at all, adding one new file per commit, and then all the modifications to the existing code in a f

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

2018-09-20 Thread Yuri Benditovich
On Thu, Sep 20, 2018 at 2:48 PM, Gerd Hoffmann wrote: > Hi, > > > If we consider the nbd PoC and the solution Daynix sent (spice-gtk and > > emulation) I personally prefer the Daynix solution and as Yuri said > already > > the glue code required for the nbd is bigger than the emulation code. >

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

2018-09-20 Thread Gerd Hoffmann
Hi, > If we consider the nbd PoC and the solution Daynix sent (spice-gtk and > emulation) I personally prefer the Daynix solution and as Yuri said already > the glue code required for the nbd is bigger than the emulation code. Oh. Fair enough. I certainly didn't expect that the nbd glue is mo

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

2018-09-20 Thread Frediano Ziglio
> Hi, > > > During my talk at KVM forum, I mentionned that an interesting project > > idea/heack would be to try to reuse emulated USB devices in qemu, and > > abstract them so they could speech usbredir. I think that would be > > really neat, because not only we could share USB emulation code fro

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

2018-09-20 Thread Gerd Hoffmann
Hi, > During my talk at KVM forum, I mentionned that an interesting project > idea/heack would be to try to reuse emulated USB devices in qemu, and > abstract them so they could speech usbredir. I think that would be > really neat, because not only we could share USB emulation code from > qemu w

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

2018-09-20 Thread Gerd Hoffmann
Hi, > > Probably the bare minimum needed to get things going. > > Which guests have you tested with this? > > Several Windows + several Linuxes. What you recommend to add to the check > list? I'd suggest adding at least FreeBSD (most popular from the *BSDs I think) to regular tests, maybe NetB

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

2018-09-20 Thread Marc-André Lureau
Hi On Thu, Sep 20, 2018 at 12:37 PM Yuri Benditovich wrote: > > Hi > > On Thu, Sep 20, 2018 at 8:59 AM, Gerd Hoffmann wrote: >> >> Hi, >> >> > spice-server changes were backward-incompatible and were not accepted >> >> Why they are not backward compatible? > > > Possible, Marc Andre can answer

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

2018-09-20 Thread Yuri Benditovich
Hi On Thu, Sep 20, 2018 at 8:59 AM, Gerd Hoffmann wrote: > Hi, > > > spice-server changes were backward-incompatible and were not accepted > > Why they are not backward compatible? > Possible, Marc Andre can answer. He was involved at time of presentation of 2 solutions and did not raise any

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

2018-09-19 Thread Vasiliy Tolstov
чт, 20 сент. 2018 г. в 8:59, Gerd Hoffmann : > > Hi, > > > spice-server changes were backward-incompatible and were not accepted > > Why they are not backward compatible? > > > usb-storage is just a header processing and holder of units > > Yes, bulk-only transport isn't that difficuilt to handle

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

2018-09-19 Thread Gerd Hoffmann
Hi, > spice-server changes were backward-incompatible and were not accepted Why they are not backward compatible? > usb-storage is just a header processing and holder of units Yes, bulk-only transport isn't that difficuilt to handle. > scsi source is 2K lines, similar to nbd server Probably

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

2018-09-19 Thread Yuri Benditovich
On Wed, Sep 19, 2018 at 3:49 PM, Gerd Hoffmann wrote: > Hi, > > > > Can you summarize the reasons to discard the approach? > > > > > > > I will try. > > > > The POC of nbd-based cd sharing was on the table. It needed some > > unclear rework to avoid breaking ABI with mainstream spice-server. >

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

2018-09-19 Thread Gerd Hoffmann
Hi, > > Can you summarize the reasons to discard the approach? > > > > I will try. > > The POC of nbd-based cd sharing was on the table. It needed some > unclear rework to avoid breaking ABI with mainstream spice-server. --verbose please. > The solution requires updates in qemu and spice on

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

2018-09-19 Thread Yuri Benditovich
On Wed, Sep 19, 2018 at 7:41 AM, Gerd Hoffmann wrote: > Hi, > > > > Hmm, so it isn't only usb emulation, but also scsi emulation duplicated > > > in spice-client. Why this route? Wouldn't it be simpler to run nbd > > > over spice-chardev, then hook up the iso as blockdev in qemu? Which > > >

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

2018-09-18 Thread Gerd Hoffmann
Hi, > > Hmm, so it isn't only usb emulation, but also scsi emulation duplicated > > in spice-client. Why this route? Wouldn't it be simpler to run nbd > > over spice-chardev, then hook up the iso as blockdev in qemu? Which > > you then can attach to any cdrom emulation (ide/sata/scsi/usb) in

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

2018-09-18 Thread Yuri Benditovich
On Tue, Sep 18, 2018 at 7:04 PM, Yuri Benditovich < yuri.benditov...@daynix.com> wrote: > > > On Tue, Sep 18, 2018 at 1:52 PM, Marc-André Lureau < > marcandre.lur...@gmail.com> wrote: > >> Hi >> >> On Mon, Sep 17, 2018 at 5:24 PM Yuri Benditovich >> wrote: >> > >> > This series of patches adds ab

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

2018-09-18 Thread Yuri Benditovich
On Tue, Sep 18, 2018 at 1:52 PM, Marc-André Lureau < marcandre.lur...@gmail.com> wrote: > Hi > > On Mon, Sep 17, 2018 at 5:24 PM Yuri Benditovich > wrote: > > > > This series of patches adds ability to share ISO files or > > local CD devices with guest system using USB redirection > > by emulatio

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

2018-09-18 Thread Frediano Ziglio
> On Tue, Sep 18, 2018 at 02:52:27PM +0400, Marc-André Lureau wrote: > > Hi > > > > On Mon, Sep 17, 2018 at 5:24 PM Yuri Benditovich > > wrote: > > > > > > This series of patches adds ability to share ISO files or > > > local CD devices with guest system using USB redirection > > > by emulation o

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

2018-09-18 Thread Gerd Hoffmann
On Tue, Sep 18, 2018 at 02:52:27PM +0400, Marc-André Lureau wrote: > Hi > > On Mon, Sep 17, 2018 at 5:24 PM Yuri Benditovich > 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. >

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

2018-09-18 Thread Marc-André Lureau
Hi On Mon, Sep 17, 2018 at 5:24 PM Yuri Benditovich 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

[Spice-devel] [spice-gtk v4 00/13] CD sharing feature

2018-09-17 Thread Yuri Benditovich
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. 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 selecte