[Spice-devel] guest sessions freeze

2012-01-23 Thread Josh Restivo
I've seen other mention of a similar problem over the past year but no resolution is ever given. I have two qemu-spice hosts at two different physical locations. When I connect to either of them from spicy or Christophe's osx port of vinagre, I frequently experience loss of input (keyboard/mouse).

Re: [Spice-devel] [PATCH spice-gtk] controller: use a controller listener abstraction

2012-01-23 Thread Hans de Goede
Looks good, ACK. Regards, Hans On 01/23/2012 02:36 PM, Marc-André Lureau wrote: Add a wrapper file for named pipe and socket listener, so we can release tarball with code compatible with windows and unix. --- gtk/controller/Makefile.am |8 +- gtk/controller/controller.v

[Spice-devel] [PATCH spice] More comparison between signed and unsigned integer expressions warning fixes

2012-01-23 Thread Hans de Goede
Turns out that more recent snapshots of gcc-4.7 emit even more of these, so the fixes which I've done before tagging the 0.10.1 release are not enough to build warning free with the latest gcc-4.7 snapshot . This fixes this. Signed-off-by: Hans de Goede --- client/cmd_line_parser.cpp |3 +--

[Spice-devel] developer assistance

2012-01-23 Thread William F Rotach
To spice-devel team: I am interested in assisting with the Spice project. One of the list of "todo"s, Single Device Multiple Monitors [ http://www.spice-space.org/page/Features/PCIMultipleOutput], looks interesting to me. I have examined some of the QXL code, and read up on the description of

[Spice-devel] Question about qemu spice memory use

2012-01-23 Thread Josh Bressers
Hello, I'm not on the list, so if you could CC me on replies it would be most appreciated. I tried to dig around for this bug but came up dry, I'm hoping someone here can point me in the right direction. I was playing Netflix video under windows XP (the host OS is Fedora 16), my qemu memory use

[Spice-devel] Announcing spice 0.10.1 aka the miniheader release

2012-01-23 Thread Hans de Goede
Hi All, I'm happy to announce spice-0.10.1 Major changes in 0.10.1 === * Mini header support * Add server API for injecting a client connection socket * Add Xinerama support to spicec * Many bugfixes / code cleanups * Requires spice-protocol >= 0.10.1 You can download it her

Re: [Spice-devel] [Spice-commits] configure.ac

2012-01-23 Thread Marc-André Lureau
So it seems that GLIB_WITH_NLS does much more checks to find out which combination of -lintl and -liconv is needed and changing LIBS accordingly, which will then correctly link the mentionned test and finally correctly set DATADIRNAME. I see several possibilities: 1) keeping AM_GLIB_GNU_GETTEXT 2)

Re: [Spice-devel] [Spice-commits] configure.ac

2012-01-23 Thread Marc-André Lureau
On Mon, Jan 23, 2012 at 1:24 PM, Daniel P. Berrange wrote: > Yeah, you really don't want to be calling both IT_PROG_INTLTOOL *and* > AM_GLIB_GNU_GETTEXT in the same configure.ac script. It causes a real > mess which prevents users to overriding the install locations with > make variables, or confi

[Spice-devel] [PATCH spice-gtk] controller: use a controller listener abstraction

2012-01-23 Thread Marc-André Lureau
Add a wrapper file for named pipe and socket listener, so we can release tarball with code compatible with windows and unix. --- gtk/controller/Makefile.am |8 +- gtk/controller/controller.vala | 25 +- gtk/controller/custom.vapi | 24 +

Re: [Spice-devel] [Qemu-devel] Vioserial of Windows guest OS on Qemu 0.15

2012-01-23 Thread Charles . Tsai-蔡清海-研究發展部
Vadim, Thanks. We will download the code from Yan's repository. -Original Message- From: Vadim Rozenfeld [mailto:vroze...@redhat.com] Sent: Friday, January 20, 2012 8:48 PM To: Charles.Tsai-蔡清海-研究發展部 Cc: Michael Roth; Stefan Hajnoczi; spice-devel@lists.freedesktop.org; Alex Huang-黃必賢-研

Re: [Spice-devel] Delay in Gimp when using qxl driver

2012-01-23 Thread Yaniv Kaul
On 01/23/2012 12:26 PM, Dominique Rodrigues wrote: Le 23/01/2012 11:41, Alon Levy a écrit : On Sun, Jan 22, 2012 at 11:05:59PM +0100, Dominique Rodrigues wrote: Le 22/01/2012 22:36, Alon Levy a écrit : On Sun, Jan 22, 2012 at 11:03:47PM +0200, Alon Levy wrote: On Sun, Jan 22, 2012 at 06:33:59

Re: [Spice-devel] [Spice-commits] configure.ac

2012-01-23 Thread Daniel P. Berrange
On Mon, Jan 23, 2012 at 01:19:51PM +0100, Christophe Fergeau wrote: > On Mon, Jan 23, 2012 at 03:17:08AM -0800, Marc-André Lureau wrote: > > configure.ac |8 > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > > New commits: > > commit c5cf7fb3bc4b48ee10c8128d4f2670f870fedad8 >

[Spice-devel] [ALREADY PUSHED my bad 6/6] server/red_channel: red_peer_handle_incoming: comment on null check

2012-01-23 Thread Alon Levy
Signed-off-by: Alon Levy --- server/red_channel.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/server/red_channel.c b/server/red_channel.c index d2c9e6d..ec02018 100644 --- a/server/red_channel.c +++ b/server/red_channel.c @@ -162,6 +162,8 @@ static void red_peer_hand

[Spice-devel] [ALREADY PUSHED my bad 5/6] server/tests/test_empty_success: fix warning on bzero

2012-01-23 Thread Alon Levy
Signed-off-by: Alon Levy --- server/tests/test_empty_success.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/server/tests/test_empty_success.c b/server/tests/test_empty_success.c index 3641eb7..31aae85 100644 --- a/server/tests/test_empty_success.c +++ b/server/tests/t

[Spice-devel] [ALREADY PUSHED my bad 4/6] docs/Spice_for_newbies.odt: some updates - added copy paste and wan mention, removed vdi_port

2012-01-23 Thread Alon Levy
--- docs/Spice_for_newbies.odt | Bin 32028 -> 40824 bytes 1 files changed, 0 insertions(+), 0 deletions(-) [removed binary diff] -- 1.7.8.4 ___ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo

[Spice-devel] [ALREADY PUSHED my bad 3/6] server/red_channel: avoid segfault if stream == NULL

2012-01-23 Thread Alon Levy
--- server/red_channel.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/server/red_channel.c b/server/red_channel.c index e526179..d2c9e6d 100644 --- a/server/red_channel.c +++ b/server/red_channel.c @@ -162,6 +162,10 @@ static void red_peer_handle_incoming(RedsStream

[Spice-devel] [ALREADY PUSHED my bad 2/6] client/foreign_menu: arm build fix

2012-01-23 Thread Alon Levy
--- client/foreign_menu.cpp |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/client/foreign_menu.cpp b/client/foreign_menu.cpp index d1df49d..faef4c2 100644 --- a/client/foreign_menu.cpp +++ b/client/foreign_menu.cpp @@ -30,7 +30,7 @@ #ifdef WIN32 #define PIPE_NAME "Sp

[Spice-devel] [ALREADY PUSHED my bad 1/6] import existing odt docs

2012-01-23 Thread Alon Levy
--- docs/Spice_for_newbies.odt | Bin 0 -> 32028 bytes docs/Spice_protocol.odt| Bin 0 -> 44052 bytes docs/Spice_style.odt | Bin 0 -> 19045 bytes docs/Spice_user_manual.odt | Bin 0 -> 31959 bytes docs/Vd_interfaces.odt | Bin 0 -> 21577 bytes 5 files changed, 0 insertions(+),

[Spice-devel] [ALREADY PUSHED my bad 0/6]

2012-01-23 Thread Alon Levy
Stuff pushed with Dan McGees patches accidentally. Reviewed by Hans post push (hence the last patch commenting on the third patch). Alon Levy (6): import existing odt docs client/foreign_menu: arm build fix server/red_channel: avoid segfault if stream == NULL docs/Spice_for_newbies.odt: so

Re: [Spice-devel] [Spice-commits] configure.ac

2012-01-23 Thread Christophe Fergeau
On Mon, Jan 23, 2012 at 03:17:08AM -0800, Marc-André Lureau wrote: > configure.ac |8 > 1 file changed, 4 insertions(+), 4 deletions(-) > > New commits: > commit c5cf7fb3bc4b48ee10c8128d4f2670f870fedad8 > Author: Marc-Andr?? Lureau > Date: Mon Jan 23 00:54:42 2012 +0100 > > b

[Spice-devel] [PATCH vdagent-win] vdagent: fix cursor position sync on multimon rhbz#757819

2012-01-23 Thread Arnon Gilboa
On any change of the display settings, driven by client message or guest user, normalize all display positions to non-negative coordinates, and update total width and height of the virtual desktop. The bug was due to wrong handling of (non-primary) displays positioned on negative coordinates. ---

Re: [Spice-devel] Delay in Gimp when using qxl driver

2012-01-23 Thread Dominique Rodrigues
Le 23/01/2012 11:41, Alon Levy a écrit : On Sun, Jan 22, 2012 at 11:05:59PM +0100, Dominique Rodrigues wrote: Le 22/01/2012 22:36, Alon Levy a écrit : On Sun, Jan 22, 2012 at 11:03:47PM +0200, Alon Levy wrote: On Sun, Jan 22, 2012 at 06:33:59PM +0100, Dominique Rodrigues wrote: Le 22/01/2012

Re: [Spice-devel] [PATCH 0/7] Code cleanup and cross-platform changes

2012-01-23 Thread Alon Levy
On Thu, Jan 19, 2012 at 02:09:57PM -0600, Dan McGee wrote: > The goal of these 8 patches is to lay the stage for spice to run on more > platforms- other Linux distributions as well as Solaris/Illumos. > > Some patches are simple cleanups with no functional changes; these were done > while attempti

Re: [Spice-devel] Delay in Gimp when using qxl driver

2012-01-23 Thread Alon Levy
On Sun, Jan 22, 2012 at 11:05:59PM +0100, Dominique Rodrigues wrote: > Le 22/01/2012 22:36, Alon Levy a écrit : > >On Sun, Jan 22, 2012 at 11:03:47PM +0200, Alon Levy wrote: > >>On Sun, Jan 22, 2012 at 06:33:59PM +0100, Dominique Rodrigues wrote: > >>>Le 22/01/2012 18:28, Alon Levy a écrit : >

Re: [Spice-devel] [PATCH spice-gtk v2 2/2] Add command line options for setting the cache size and the glz window size

2012-01-23 Thread Hans de Goede
Hi, One small nitpick, otherwise it looks good. On 01/22/2012 09:12 AM, Yonit Halperin wrote: This options will help us tune and find the optimal values. --- gtk/spice-option.c | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/gtk/spice-option.c b/gtk/spice-o

Re: [Spice-devel] [PATCH spice-gtk v2 1/2] Change the setting of the images cache size and the glz window size

2012-01-23 Thread Hans de Goede
Hi, I've various remarks, see my comments inline. On 01/22/2012 09:12 AM, Yonit Halperin wrote: Set the default sizes to be the same as in the old linux spice client. cache_size=20M pixels (instead of 32M), window_size=8M pixels for a 64MB dev ram (instead of 16M pixels). --- gtk/channel-disp

[Spice-devel] [PATCH v2 murmur 2/2] replace lookup3 with MurmurHash3

2012-01-23 Thread Alon Levy
See http://code.google.com/p/smhasher/wiki/MurmurHash3 Performance quotes from there are 2.5 times what lookup3 can do, for 32 bit variant, which is what we use: Lookup3_x86_32 - 1234 mb/sec Lookup3_x64_32 - 1265 mb/sec MurmurHash3_x86_32 - 3105 mb/sec New files are released to th

[Spice-devel] [PATCH v2 murmur 1/2] qxl_image: silence two warnings because of unused hash

2012-01-23 Thread Alon Levy
--- src/qxl_image.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/src/qxl_image.c b/src/qxl_image.c index 0ffb031..4988cbf 100644 --- a/src/qxl_image.c +++ b/src/qxl_image.c @@ -136,6 +136,8 @@ qxl_image_create (qxl_screen_t *qxl, const uint8_t *data, data += y *

Re: [Spice-devel] [PATCH] replace lookup3 with MurmurHash3

2012-01-23 Thread Alon Levy
On Mon, Jan 23, 2012 at 09:52:59AM +0200, Alon Levy wrote: > See http://code.google.com/p/smhasher/wiki/MurmurHash3 > > Performance quotes from there are 2.5 times what lookup3 can do, for > 32 bit variant, which is what we use: > > Lookup3_x86_32 - 1234 mb/sec > Lookup3_x64_32 - 1265