Re: [Spice-devel] Remote control delay when I access to windows7 kvm virtual machine with spice

2017-02-20 Thread Pavel Grunt
On Tue, 2017-02-21 at 15:10 +0800, John Y. wrote: > spice-server is 0.12.4-20 > > I did not add the spice agent. > Can spice agent provide better VGA performance?  It will give you the arbitrary resolution feature (guest will change its resolution to match the remote-viewer's window size) > I th

Re: [Spice-devel] Remote control delay when I access to windows7 kvm virtual machine with spice

2017-02-20 Thread John Y.
spice-server is 0.12.4-20 I did not add the spice agent. Can spice agent provide better VGA performance? I though it just provide data transferring. I had install spice guest tools. Regards, John 2017-02-21 14:29 GMT+08:00 Pavel Grunt : > Hello John, > > On Tue, 2017-02-21 at 10:28 +0800, Joh

Re: [Spice-devel] Remote control delay when I access to windows7 kvm virtual machine with spice

2017-02-20 Thread Pavel Grunt
Hello John, On Tue, 2017-02-21 at 10:28 +0800, John Y. wrote: > I had a kvm virtual machine with windows 7 which is created by > qemu2.5 and spice-server-1.8.0. do you mean spice-server 0.12.8 ? > I used spice to remote access the vm, but I felled lag sometime. > I opened the notepad and typed so

[Spice-devel] Remote control delay when I access to windows7 kvm virtual machine with spice

2017-02-20 Thread John Y.
I had a kvm virtual machine with windows 7 which is created by qemu2.5 and spice-server-1.8.0. I used spice to remote access the vm, but I felled lag sometime. I opened the notepad and typed some words, but words displayed after 0.5-1 seconds. I can fell the delay obviously. The vm is on a centos

Re: [Spice-devel] [PATCH v2 spice-gtk 1/2] authentication: Handle failed SASL authentication separately

2017-02-20 Thread Christophe de Dinechin
> On 19 Feb 2017, at 15:47, Snir Sheriber wrote: > > Remove handling with failures in the SASL authentication > process to separate function > --- > src/spice-channel.c | 44 +++- > 1 file changed, 27 insertions(+), 17 deletions(-) > > diff --git a/src/spi

Re: [Spice-devel] [PATCH] Don't close all but one display during reboot.

2017-02-20 Thread Christophe de Dinechin
Is it possible to make the max number of monitors something persistent (e.g. set / get that using libvirt), so that the behavior would be consistent between a first boot and a reboot? Christophe > On 20 Feb 2017, at 15:49, Jonathon Jongsma wrote: > > On Thu, 2017-02-02 at 10:30 -0600, Jonatho

Re: [Spice-devel] [PATCH] Don't close all but one display during reboot.

2017-02-20 Thread Jonathon Jongsma
On Thu, 2017-02-02 at 10:30 -0600, Jonathon Jongsma wrote: > On Thu, 2017-02-02 at 05:29 -0500, Frediano Ziglio wrote: > > > > > > When a guest is rebooted, the QXL driver gets unloaded at some > > > point in > > > the reboot process. When the driver is unloaded, the spice server > > > sets a > >

[Spice-devel] [spice-gtk v1] gtk-session: Use GWeakRef

2017-02-20 Thread Victor Toso
From: Victor Toso The custom WeakRef structure was introduced in 9baba9fd89 (2012) to fix rhbz#743773. Since glib 2.32, GWeakRef was introduced and it behaves similarly to our WeakRef. Moving to GWeakRef to remove some code which exists in glib. Note that I'm keeping to utility functions: - get

Re: [Spice-devel] [PATCH spice-server 3/3] Add message counters to statistics

2017-02-20 Thread Frediano Ziglio
Didn't forget to merge this. It's just that not every time you call send_item a message is sent (just remembered before merging). Frediano > > Acked-by: Jonathon Jongsma > > > On Tue, 2017-02-14 at 15:51 +, Frediano Ziglio wrote: > > Signed-off-by: Frediano Ziglio > > --- > >  server/re

Re: [Spice-devel] [PATCH spice-gtk v5 6/6] usb-device-widget: Migrate to GtkGrid from GtkBox

2017-02-20 Thread Pavel Grunt
Hi Daniel, On Mon, 2017-02-20 at 13:04 +, Daniel P. Berrange wrote: > On Mon, Feb 20, 2017 at 12:44:52PM +0100, Victor Toso wrote: > > Hi, > > > > On Fri, Feb 17, 2017 at 11:24:55AM +0100, Pavel Grunt wrote: > > > GtkVBox is deprecated since Gtk 3.2, GtkBox is going to be > > > deprecated. Ju

Re: [Spice-devel] [spice v8] dcc: handle preferred video codec message

2017-02-20 Thread Victor Toso
Hi, On Mon, Feb 20, 2017 at 12:16:33PM +0100, Christophe de Dinechin wrote: > > > On 9 Feb 2017, at 09:21, Victor Toso wrote: > > > > [0] SPICE_MSGC_DISPLAY_PREFERRED_VIDEO_CODEC_TYPE > > > > This message provides a list of video codecs based on client's order > > of preference. > > > > We dup

Re: [Spice-devel] [PATCH spice-gtk v5 6/6] usb-device-widget: Migrate to GtkGrid from GtkBox

2017-02-20 Thread Daniel P. Berrange
On Mon, Feb 20, 2017 at 12:44:52PM +0100, Victor Toso wrote: > Hi, > > On Fri, Feb 17, 2017 at 11:24:55AM +0100, Pavel Grunt wrote: > > GtkVBox is deprecated since Gtk 3.2, GtkBox is going to be > > deprecated. Just use the GtkGrid and GtkContainer api. > > Sure > > > --- > > GtkContainer is an

Re: [Spice-devel] [PATCH spice-gtk v5 6/6] usb-device-widget: Migrate to GtkGrid from GtkBox

2017-02-20 Thread Pavel Grunt
Hi, On Mon, 2017-02-20 at 12:44 +0100, Victor Toso wrote: > Hi, > > On Fri, Feb 17, 2017 at 11:24:55AM +0100, Pavel Grunt wrote: > > GtkVBox is deprecated since Gtk 3.2, GtkBox is going to be > > deprecated. Just use the GtkGrid and GtkContainer api. > > Sure > > > --- > > GtkContainer is an ab

Re: [Spice-devel] [spice v8] dcc: handle preferred video codec message

2017-02-20 Thread Frediano Ziglio
> > [0] SPICE_MSGC_DISPLAY_PREFERRED_VIDEO_CODEC_TYPE > > This message provides a list of video codecs based on client's order > of preference. > > We duplicate the video codecs array from reds.c and sort it using the > order of codecs as reference. > > This message will not change an ongoing s

Re: [Spice-devel] [PATCH spice-gtk v5 6/6] usb-device-widget: Migrate to GtkGrid from GtkBox

2017-02-20 Thread Victor Toso
Hi, On Fri, Feb 17, 2017 at 11:24:55AM +0100, Pavel Grunt wrote: > GtkVBox is deprecated since Gtk 3.2, GtkBox is going to be > deprecated. Just use the GtkGrid and GtkContainer api. Sure > --- > GtkContainer is an abstract class, so let's use the Grid with the > vertical orientation At first I

Re: [Spice-devel] [spice v8] dcc: handle preferred video codec message

2017-02-20 Thread Christophe de Dinechin
> On 9 Feb 2017, at 09:21, Victor Toso wrote: > > [0] SPICE_MSGC_DISPLAY_PREFERRED_VIDEO_CODEC_TYPE > > This message provides a list of video codecs based on client's order > of preference. > > We duplicate the video codecs array from reds.c and sort it using the > order of codecs as reference

Re: [Spice-devel] [vdagent-linux v8 0/4]

2017-02-20 Thread Victor Toso
Hi, This is now pushed as: commit 99d9d3583143aef7143ec986cebe2980fdeeb776 vdagentd: Do endian swapping. commit 3903ebd5935014cb13bfe697bc8c717ac737455d vdagentd: Adjust size checks commit 72acb448f76d07c1d3b74538bdedcde3d8f0d0f4 vdagentd: Add missing size checks commit bf7cb2353455e1e7f289dc5

Re: [Spice-devel] [spice v8] dcc: handle preferred video codec message

2017-02-20 Thread Christophe de Dinechin
> On 10 Feb 2017, at 14:14, Victor Toso wrote: > > Hi, > > On Fri, Feb 10, 2017 at 07:51:05AM -0500, Frediano Ziglio wrote: >>> [0] SPICE_MSGC_DISPLAY_PREFERRED_VIDEO_CODEC_TYPE >>> >>> This message provides a list of video codecs based on client's order >>> of preference. >>> >>> We duplicat

Re: [Spice-devel] bool or gboolean

2017-02-20 Thread Victor Toso
Hi, On Thu, Feb 16, 2017 at 01:24:04PM +0100, Christophe de Dinechin wrote: > > > On 16 Feb 2017, at 13:17, Pavel Grunt wrote: > > > > On Wed, 2017-02-15 at 11:39 -0500, Frediano Ziglio wrote: > >> Hi, > >> question was raised recently on the ML and IRC. > >> > >> Some time ago we decided to u

Re: [Spice-devel] [spice v8] dcc: handle preferred video codec message

2017-02-20 Thread Victor Toso
Hi, On Fri, Feb 10, 2017 at 02:14:11PM +0100, Victor Toso wrote: > Hi, > > On Fri, Feb 10, 2017 at 07:51:05AM -0500, Frediano Ziglio wrote: > > > [0] SPICE_MSGC_DISPLAY_PREFERRED_VIDEO_CODEC_TYPE > > > > > > This message provides a list of video codecs based on client's order > > > of preference

Re: [Spice-devel] [PATCH spice-gtk] gtk-session: Set value directly

2017-02-20 Thread Victor Toso
Hi, On Fri, Feb 17, 2017 at 01:44:55PM +0100, Pavel Grunt wrote: > Spotted by coverity Including the coverity message instead would be more interesting IMHO. Acked-by: Victor Toso > --- > src/spice-gtk-session.c | 6 ++ > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/src

Re: [Spice-devel] Spice protocol behind a Firewall

2017-02-20 Thread Victor Toso
Hi, On Sun, Feb 19, 2017 at 11:48:44AM +0200, Uri Lublin wrote: > On 02/19/2017 08:07 AM, Oscar Segarra wrote: > > Hi, > > > > First of all, I'd like to say that I'm not sure enough I'm writing to > > the correct mailing list, I have not been able to find a common users > > mailing list. > > > >