[Spice-devel] Spice client issues on Fedora14 Live.

2011-10-19 Thread Naga Mohan Pothula
Hi, I'm facing two strange problems with Spice client 0.9 on Fedora 14 Live CD which boot parameters are set to load Fed14 live into RAM to run faster. 1.) keyboard/Mouse doesn't function at logon screen when guest image is started from shutdown/restart       This issue happens with windows gues

Re: [Spice-devel] [master PATCH 2/2] server: netstat: send random data not zeros

2011-10-19 Thread Frédéric Grelot
> > That may be overkill, but you could also "scramble" it by xoring it > > with a randomly generated sequence (and send the seed to the > > client to allow him to reproduce the sequence). The only overhead > > would thus be the seed. > > By the way, that could be applied to any message, not necess

Re: [Spice-devel] [master PATCH 2/2] server: netstat: send random data not zeros

2011-10-19 Thread Yaniv Kaul
On 10/19/2011 01:42 PM, Frédéric Grelot wrote: That seems like a good idea. Note that if the first image is not compressed, we may have a similar problem with WAN accelerators. Also maybe it is better to always compress it. This random buffer was intended as a quick fix. In the long run, we woul

Re: [Spice-devel] [PATCH] qxl: fix guest cursor tracking

2011-10-19 Thread Gerd Hoffmann
On 10/18/11 18:58, Yonit Halperin wrote: > (1) If the guest cursor command is empty, don't reload it after migration. > (2) Cleaning the guest cursor when it is released by > the spice server. In addition, explicitly reset the > cursor in spice upon destroying the primary surface > (was

Re: [Spice-devel] [master PATCH 1/2] server: netstat: modify network bandwidth calculation

2011-10-19 Thread Uri Lublin
On 10/18/2011 11:53 PM, Yaniv Kaul wrote: On 18/10/2011 19:20, Uri Lublin wrote: Currently spice-server network bandwidth estimation is: send an empty ping packet to the client (and ignore it) ("warmup") Useless. 'warmup' of what exactly? The TCP MSS and everything else is already set by the T

Re: [Spice-devel] [master PATCH 2/2] server: netstat: send random data not zeros

2011-10-19 Thread Frédéric Grelot
> That seems like a good idea. > Note that if the first image is not compressed, we may have a similar > problem > with WAN accelerators. Also maybe it is better to always compress it. > > This random buffer was intended as a quick fix. > In the long run, we would like to measure network statisti

Re: [Spice-devel] [master PATCH 2/2] server: netstat: send random data not zeros

2011-10-19 Thread Uri Lublin
On 10/18/2011 11:50 PM, Yaniv Kaul wrote: On 18/10/2011 19:20, Uri Lublin wrote: WAN accelerators may compress the zeros which results with a wrong bandwidth calculation. Is this really the right approach? Why not calculate the latency based on the time it takes to send the first image? That i

Re: [Spice-devel] [PATCH 1/4] gtk: warn if argument is invalid instead of crash

2011-10-19 Thread Alon Levy
On Wed, Oct 19, 2011 at 11:29:27AM +0200, Marc-André Lureau wrote: Series Reviewed-by: Alon Levy > --- > gtk/spice-widget.c |3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) > > diff --git a/gtk/spice-widget.c b/gtk/spice-widget.c > index afe030d..fa7f675 100644 > --- a/gtk/spice-

[Spice-devel] [PATCH 4/4] do not segfault if link message header size is set to 0

2011-10-19 Thread Marc-André Lureau
https://bugs.freedesktop.org/show_bug.cgi?id=41988 --- gtk/spice-channel.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/gtk/spice-channel.c b/gtk/spice-channel.c index 5bc244f..23c16e7 100644 --- a/gtk/spice-channel.c +++ b/gtk/spice-channel.c @@ -976,6 +976,11 @@ s

[Spice-devel] [PATCH 3/4] Add SPICE_NOGRAB environment variable for debugging

2011-10-19 Thread Marc-André Lureau
--- gtk/spice-widget.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/gtk/spice-widget.c b/gtk/spice-widget.c index fa7f675..5f8b301 100644 --- a/gtk/spice-widget.c +++ b/gtk/spice-widget.c @@ -406,6 +406,9 @@ static void try_keyboard_grab(SpiceDisplay *display)

[Spice-devel] [PATCH 2/4] gtk: channel-event emit an enum

2011-10-19 Thread Marc-André Lureau
This doesn't break ABI, as int and enum are compatible at glib level. --- gtk/spice-channel.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/gtk/spice-channel.c b/gtk/spice-channel.c index 3a17204..5bc244f 100644 --- a/gtk/spice-channel.c +++ b/gtk/spice-channel.c @@ -

[Spice-devel] [PATCH 1/4] gtk: warn if argument is invalid instead of crash

2011-10-19 Thread Marc-André Lureau
--- gtk/spice-widget.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/gtk/spice-widget.c b/gtk/spice-widget.c index afe030d..fa7f675 100644 --- a/gtk/spice-widget.c +++ b/gtk/spice-widget.c @@ -675,6 +675,7 @@ static gboolean draw_event(GtkWidget *widget, cairo_t *cr)

Re: [Spice-devel] [master PATCH 2/2] server: netstat: send random data not zeros

2011-10-19 Thread Alon Levy
On Tue, Oct 18, 2011 at 11:50:37PM +0200, Yaniv Kaul wrote: > On 18/10/2011 19:20, Uri Lublin wrote: > >WAN accelerators may compress the zeros which results with a > >wrong bandwidth calculation. > > Is this really the right approach? Why not calculate the latency > based on the time it takes to

Re: [Spice-devel] [master PATCH 2/2] server: netstat: send random data not zeros

2011-10-19 Thread Alon Levy
On Tue, Oct 18, 2011 at 11:50:37PM +0200, Yaniv Kaul wrote: > On 18/10/2011 19:20, Uri Lublin wrote: > >WAN accelerators may compress the zeros which results with a > >wrong bandwidth calculation. > > Is this really the right approach? Why not calculate the latency > based on the time it takes to