[Spice-devel] [spice-gtk PATCH v2] spicy.c: Don't attempt to connect automatically if hostname and port are not provided

2012-08-24 Thread riegamaths
From: Dunrong Huang This patch will fix following error: $ spicy --spice-debug .. (spicy:21981): GSpice-DEBUG: spice-session.c:1618 new main channel, switching (spicy:21981): GSpice-DEBUG: spice-gtk-session.c:811 Changing main channel from (nil) to 0x8534b0 (spicy:21981): GSpice-DEBUG: spicy

Re: [Spice-devel] [spice-gtk PATCH] spicy.c: Don't attempt to connect automatically if hostname and port are not provided

2012-08-24 Thread Dunrong Huang
Thanks for your review. 2012/8/24 Marc-André Lureau : > Hi, > > On Fri, Aug 24, 2012 at 11:48 AM, wrote: >> From: Dunrong Huang >> >> This patch will fix following error: >> $ spicy --spice-debug >> .. >> (spicy:21981): GSpice-DEBUG: spice-session.c:1618 new main channel, switching >> (spic

[Spice-devel] [PATCH 2/3] qxl-virtio: adds command logging

2012-08-24 Thread Erlon Cruz
Now spice commands sent from applications can be logged in guest side Signed-off-by: Erlon R. Cruz Signed-off-by: Fabiano Fidêncio Signed-off-by: Rafael F. Santos --- src/Makefile.am |9 ++- src/qxl.h|3 + src/qxl_logger.c | 254 +++

[Spice-devel] [PATCH 1/3] qxl-virtio: add compilation support for qxl-virtio and for debugging stuffs

2012-08-24 Thread Erlon Cruz
Signed-off-by: Erlon R. Cruz Signed-off-by: Fabiano Fidêncio Signed-off-by: Rafael F. Santos --- Makefile.am |5 - configure.ac| 34 ++ src/Makefile.am | 34 -- 3 files changed, 70 insertions(+), 3 deletions(-)

[Spice-devel] Xf86 driver video for QEMU qxl-virtio device

2012-08-24 Thread Erlon Cruz
The following series of patches implements support for QEMU qxl-virtio device, please review! ___ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/spice-devel

Re: [Spice-devel] [spice-gtk PATCH] spicy.c: Don't attempt to connect automatically if hostname and port are not provided

2012-08-24 Thread Marc-André Lureau
Hi, On Fri, Aug 24, 2012 at 11:48 AM, wrote: > From: Dunrong Huang > > This patch will fix following error: > $ spicy --spice-debug > .. > (spicy:21981): GSpice-DEBUG: spice-session.c:1618 new main channel, switching > (spicy:21981): GSpice-DEBUG: spice-gtk-session.c:811 Changing main chann

[Spice-devel] [PATCH spice-gtk 6/6] spice-session: make spice_session_channel_open_host() async again

2012-08-24 Thread Marc-André Lureau
The previous patches (introducing GSocketClient & GProxy usage) turned the code into only sync calls. By using async() variants of resolve & connect functions, make the open host connection asynchronous again. --- gtk/spice-session.c | 195 +--- 1 f

[Spice-devel] [PATCH spice-gtk 5/6] Add SpiceProxy object

2012-08-24 Thread Marc-André Lureau
Add a simple object to handle the SPICE_PROXY values. It's not clear to me whether each GIO user needs to handle the proxy configuration, or if there is a more global mechanism (via g_network_address_parse_uri()) Also, the parsing is currently very limited and only support basic HTTP proxy URI. I

[Spice-devel] [PATCH spice-gtk 4/6] session: allow to connect via HTTP CONNECT proxy

2012-08-24 Thread Marc-André Lureau
Allow to connect to a Spice server via a HTTP proxy with CONNECT method. spice-gtk will use the SPICE_PROXY environment variable, which can currently only have the following syntax: [http://]hostname[:port] This is paving the way to more proxies support (socks4/socks5). This code is now entirely

[Spice-devel] [PATCH spice-gtk 3/6] spice-channel: remove unnecessary g_socket_close()

2012-08-24 Thread Marc-André Lureau
--- gtk/spice-channel.c | 1 - 1 file changed, 1 deletion(-) diff --git a/gtk/spice-channel.c b/gtk/spice-channel.c index 37198f5..eb113db 100644 --- a/gtk/spice-channel.c +++ b/gtk/spice-channel.c @@ -2379,7 +2379,6 @@ static void channel_reset(SpiceChannel *channel, gboolean migrating) }

[Spice-devel] [PATCH spice-gtk 2/6] spice-channel: plug a small memory leak

2012-08-24 Thread Marc-André Lureau
--- gtk/spice-channel.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/gtk/spice-channel.c b/gtk/spice-channel.c index 54d406d..37198f5 100644 --- a/gtk/spice-channel.c +++ b/gtk/spice-channel.c @@ -1346,6 +1346,7 @@ static gboolean spice_channel_perform_auth_sasl(SpiceChannel *channel)

[Spice-devel] [PATCH spice-gtk 1/6] Add wocky HTTP proxy

2012-08-24 Thread Marc-André Lureau
Courtesy of Nicolas Dufresne It might make sense to include this proxy in glib/gio, but it is still missing some features according to its author, namely SSL and perhaps better CRLF. --- gtk/Makefile.am| 2 + gtk/spice-session.c| 3 + gtk/wocky-http-proxy.c | 429

[Spice-devel] [PATCH spice-gtk 0/6] Add simple HTTP CONNECT proxy support

2012-08-24 Thread Marc-André Lureau
Hi, The following series allows to connect to a Spice server via a HTTP proxy, using the CONNECT method. It can be tested with a Squid proxy, (which allows CONNECT on various ports) by setting the environment variable SPICE_PROXY=proxy_host:proxy_port (it doesn't support any other parameters atm,

[Spice-devel] [spice-gtk PATCH] spicy.c: Don't attempt to connect automatically if hostname and port are not provided

2012-08-24 Thread riegamaths
From: Dunrong Huang This patch will fix following error: $ spicy --spice-debug .. (spicy:21981): GSpice-DEBUG: spice-session.c:1618 new main channel, switching (spicy:21981): GSpice-DEBUG: spice-gtk-session.c:811 Changing main channel from (nil) to 0x8534b0 (spicy:21981): GSpice-DEBUG: spicy

Re: [Spice-devel] Passed trough video card screen access

2012-08-24 Thread Alon Levy
- Original Message - > > That would work, but would not use most of the spice protocol. > > Most of the SPICE protocol is not needed. Main objective for Windows > SPICE-server is pass only framebuffer to SPICE-client, other > features can be recieved from a hypervisor connection (sound,

Re: [Spice-devel] Render support, spice-gtk, v2

2012-08-24 Thread Hans de Goede
Series looks good, ack series. On 08/24/2012 03:02 AM, Søren Sandmann Pedersen wrote: ssia ___ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/spice-devel _

Re: [Spice-devel] Render patches, spice, v2

2012-08-24 Thread Hans de Goede
Hi, Series looks good, except for patch 3, you need to add the equivalent of this spice-gtk change --- a/gtk/channel-display.c +++ b/gtk/channel-display.c @@ -681,6 +681,7 @@ static void spice_display_channel_reset_capabilities(SpiceChannel *channel) { spice_channel_set_capability(SPICE_CH

Re: [Spice-devel] spice-common render patches, v2

2012-08-24 Thread Hans de Goede
Series looks good, ack. On 08/24/2012 03:00 AM, Søren Sandmann Pedersen wrote: ssia ___ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/spice-devel

Re: [Spice-devel] spice-protocol patches for Render support, v2

2012-08-24 Thread Hans de Goede
Series looks good, ack. On 08/24/2012 02:59 AM, Søren Sandmann Pedersen wrote: ssia ___ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/spice-devel