Re: [Spice-devel] [PATCH x11spice v2 3/3] Add cache for SHM segments

2019-07-29 Thread Jeremy White
Hi Brendan, I've been running with this code for a while, and it works nicely. It's hard to quantify, but with my test case I get cache hit rates in the 90% range, and it does seem to improve overall performance. I've got two comments on the patch, inline below: On 7/16/19 9:16 PM, Brendan

Re: [Spice-devel] [spice-gtk 2/9] usb-redir: device error signal without device object

2019-07-29 Thread Yuri Benditovich
On Mon, Jul 29, 2019 at 6:32 PM Frediano Ziglio wrote: > > > > > On Mon, Jul 29, 2019 at 3:34 PM Frediano Ziglio wrote: > > > > > > > > > > > But this was always possible to print the error. > > > > > > No... I don't print the error, is up to the caller. > > > > But you're also the caller (in spi

Re: [Spice-devel] [spice-gtk 2/9] usb-redir: device error signal without device object

2019-07-29 Thread Frediano Ziglio
> > On Mon, Jul 29, 2019 at 3:34 PM Frediano Ziglio wrote: > > > > > > > > But this was always possible to print the error. > > > > No... I don't print the error, is up to the caller. > > But you're also the caller (in spice-options), aren't you? > The difference is that what I suggested comes t

Re: [Spice-devel] [PATCH spice-gtk v3 1/6] channel-display: Minimize the stream lag by ignoring the server one

2019-07-29 Thread Francois Gouget
Actually I found something I'm a bit unhappy with in this patch. So one goal is to make sure all streams are in sync. This is why the mmtime conversion is performed globally: if two frames from different streams have the same mmtime, the resulting client-side timestamps should be identical too

[Spice-devel] [PATCH spice-gtk] Use "#pragma once" instead of preprocessor guards

2019-07-29 Thread Frediano Ziglio
Guards currently are quite different in format. Unify updating to "#pragma once" syntax. This syntax is used by GTK tools too. To avoid possible regression public headers are left untouched. Signed-off-by: Frediano Ziglio --- src/bio-gio.h | 5 + src/channel-display-pri

Re: [Spice-devel] [spice-gtk 2/9] usb-redir: device error signal without device object

2019-07-29 Thread Yuri Benditovich
On Mon, Jul 29, 2019 at 3:34 PM Frediano Ziglio wrote: > > > > > But this was always possible to print the error. > > No... I don't print the error, is up to the caller. But you're also the caller (in spice-options), aren't you? The difference is that what I suggested comes to existing signal set

Re: [Spice-devel] [spice-gtk 2/9] usb-redir: device error signal without device object

2019-07-29 Thread Frediano Ziglio
> > But this was always possible to print the error. No... I don't print the error, is up to the caller. > IMO, the point is to raise it to remote-viewer GUI. If is a GUI the fprintf will be replaced by a MessageBox or similar, isn't it? > And this is simple with fake device and complicated ot

Re: [Spice-devel] [spice-gtk 2/9] usb-redir: device error signal without device object

2019-07-29 Thread Yuri Benditovich
But this was always possible to print the error. IMO, the point is to raise it to remote-viewer GUI. And this is simple with fake device and complicated other way. On Mon, Jul 29, 2019 at 2:14 PM Frediano Ziglio wrote: > > > On Mon, Jul 29, 2019 at 12:27 PM Frediano Ziglio wrote: > > > > > > > O

Re: [Spice-devel] [spice-gtk 2/9] usb-redir: device error signal without device object

2019-07-29 Thread Frediano Ziglio
> On Mon, Jul 29, 2019 at 12:27 PM Frediano Ziglio wrote: > > > > > On Mon, Jul 29, 2019 at 10:48 AM Frediano Ziglio > > > wrote: > > > > > > > > > > > > > > On Fri, Jul 26, 2019 at 7:14 PM Victor Toso > > > > > wrote: > > > > > > > > > > > > Hi, > > > > > > > > > > > > On Fri, Jul 26, 2019 at 0

Re: [Spice-devel] [spice-gtk 2/9] usb-redir: device error signal without device object

2019-07-29 Thread Yuri Benditovich
On Mon, Jul 29, 2019 at 12:27 PM Frediano Ziglio wrote: > > > On Mon, Jul 29, 2019 at 10:48 AM Frediano Ziglio wrote: > > > > > > > > > > > On Fri, Jul 26, 2019 at 7:14 PM Victor Toso > > > > wrote: > > > > > > > > > > Hi, > > > > > > > > > > On Fri, Jul 26, 2019 at 06:30:51PM +0300, Yuri Bendit

Re: [Spice-devel] [spice-gtk 2/9] usb-redir: device error signal without device object

2019-07-29 Thread Frediano Ziglio
> On Mon, Jul 29, 2019 at 10:48 AM Frediano Ziglio wrote: > > > > > > > > On Fri, Jul 26, 2019 at 7:14 PM Victor Toso > > > wrote: > > > > > > > > Hi, > > > > > > > > On Fri, Jul 26, 2019 at 06:30:51PM +0300, Yuri Benditovich wrote: > > > > > On Fri, Jul 26, 2019 at 2:09 PM Frediano Ziglio > > >

Re: [Spice-devel] [spice-gtk 1/9] usb-redir: define interfaces to support emulated devices

2019-07-29 Thread Yuri Benditovich
On Mon, Jul 29, 2019 at 11:03 AM Frediano Ziglio wrote: > > > > > On Fri, Jul 26, 2019 at 4:54 PM Frediano Ziglio wrote: > > > > > > > On Fri, Jul 26, 2019 at 11:43 AM Frediano Ziglio > > > > wrote: > > > > > > > > > > > > > > > > > On Thu, Jul 25, 2019 at 11:57 AM Frediano Ziglio > > > > > >

Re: [Spice-devel] [spice-gtk 2/9] usb-redir: device error signal without device object

2019-07-29 Thread Yuri Benditovich
On Mon, Jul 29, 2019 at 10:48 AM Frediano Ziglio wrote: > > > > > On Fri, Jul 26, 2019 at 7:14 PM Victor Toso wrote: > > > > > > Hi, > > > > > > On Fri, Jul 26, 2019 at 06:30:51PM +0300, Yuri Benditovich wrote: > > > > On Fri, Jul 26, 2019 at 2:09 PM Frediano Ziglio > > > > wrote: > > > > > > >

Re: [Spice-devel] [spice-gtk 1/9] usb-redir: define interfaces to support emulated devices

2019-07-29 Thread Frediano Ziglio
> > On Fri, Jul 26, 2019 at 4:54 PM Frediano Ziglio wrote: > > > > > On Fri, Jul 26, 2019 at 11:43 AM Frediano Ziglio > > > wrote: > > > > > > > > > > > > > > On Thu, Jul 25, 2019 at 11:57 AM Frediano Ziglio > > > > > wrote: > > > > > > > > > > > > > > > > > > > > SpiceUsbBackendDevice structur

Re: [Spice-devel] [spice-gtk 2/9] usb-redir: device error signal without device object

2019-07-29 Thread Frediano Ziglio
> > On Fri, Jul 26, 2019 at 7:14 PM Victor Toso wrote: > > > > Hi, > > > > On Fri, Jul 26, 2019 at 06:30:51PM +0300, Yuri Benditovich wrote: > > > On Fri, Jul 26, 2019 at 2:09 PM Frediano Ziglio > > > wrote: > > > > > > > > > > > > > > On Fri, Jul 26, 2019 at 10:09 AM Victor Toso > > > > > wrot