Re: [Spice-devel] Fail to create a printing channel

2012-03-22 Thread Charles . Tsai-蔡清海-研究發展部
Alon, In Windows, driver name is provided by the device driver. In this case , "com.redhat.print.0" is the device visible to application. Because device "com.redhat.print.0" is created by the Qemu which is in Linux, how the Linux passes the driver information to Windows So that Windows applicati

Re: [Spice-devel] Fail to create a printing channel

2012-03-22 Thread Charles . Tsai-蔡清海-研究發展部
Alon, I will put some traces in Qemu to see if I can find out where the code is stuck. -Original Message- From: Alon Levy [mailto:al...@redhat.com] Sent: Thursday, March 22, 2012 10:22 PM To: Charles.Tsai-蔡清海-研究發展部 Cc: spice-devel@lists.freedesktop.org Subject: Re: Fail to cre

Re: [Spice-devel] [PATCH spice-gtk] build-sys: move codegen & proto to spice-common

2012-03-22 Thread Alon Levy
On Thu, Mar 22, 2012 at 08:30:49PM +0100, Marc-André Lureau wrote: > On Thu, Mar 22, 2012 at 7:48 PM, Alon Levy wrote: > > Looks good to me, > > > > ACK. > > Notice that this patch makes use of branch wip2: > > http://cgit.freedesktop.org/spice/spice-common/log/?h=wip2 > > Have you also checked

[Spice-devel] spice client for windows

2012-03-22 Thread Frederic Bouchard
Hi, I would like to know if with the spice client for windows can run in full screen mode. Also if we can to save the configuration of the remote connection (like Microsoft client RDP which can save rdp session configuration.) Regards ___ Spice-devel m

[Spice-devel] [PATCH spice 3/3] Copy common/win/my_getopt-1.5/my_getopt client/windows

2012-03-22 Thread Marc-André Lureau
--- client/windows/my_getopt.cpp | 301 +++--- client/windows/my_getopt.h | 72 ++ 2 files changed, 352 insertions(+), 21 deletions(-) create mode 100644 client/windows/my_getopt.h diff --git a/client/windows/my_getopt.cpp b/client/windows/my_geto

[Spice-devel] [PATCH spice 0/3] Use spice-common submodule

2012-03-22 Thread Marc-André Lureau
Hi, spice-common contains the common/, python_modules/, spice_codegen.py, spice1.proto, spice.proto history of spice server until "before-split2" tag. On top of that, it includes changes that were only in spice-gtk, and changes to make it compatible with spice-gtk. The main change is the unified l

[Spice-devel] SSL connect problem

2012-03-22 Thread Anthony James
I'm having problems connecting to a spice virtual machine using SSL. I use the following command to connect: spicec -h localhost -p $PORT -s $SPORT --secure-channels all --host-subject "$HOSTSUBJECT" -w $PASSWD The error I receive is: Error: failed to connect w/SSL, ssl_error error:0001:lib

Re: [Spice-devel] [PATCH spice-gtk] build-sys: move codegen & proto to spice-common

2012-03-22 Thread Marc-André Lureau
On Thu, Mar 22, 2012 at 7:48 PM, Alon Levy wrote: > Looks good to me, > > ACK. Notice that this patch makes use of branch wip2: http://cgit.freedesktop.org/spice/spice-common/log/?h=wip2 Have you also checked it? I guess it should be reviewed too (or re-review for most of the patches)? Since I'

Re: [Spice-devel] [PATCH spice-gtk] build-sys: move codegen & proto to spice-common

2012-03-22 Thread Alon Levy
On Thu, Mar 22, 2012 at 06:58:43PM +0100, Marc-André Lureau wrote: > On Thu, Mar 22, 2012 at 6:30 PM, Alon Levy wrote: > > Actually, strike that - sorry for asking, not worth the trouble. > > yeah, in my opinion, build-system stuff isn't worth splitting... > > As long as you don't break the sacr

Re: [Spice-devel] [PATCH spice-gtk] build-sys: move codegen & proto to spice-common

2012-03-22 Thread Marc-André Lureau
On Thu, Mar 22, 2012 at 6:30 PM, Alon Levy wrote: > Actually, strike that - sorry for asking, not worth the trouble. yeah, in my opinion, build-system stuff isn't worth splitting... As long as you don't break the sacred rule of "make distcheck" and the builds with various options at each iterati

Re: [Spice-devel] Fail to create a printing channel

2012-03-22 Thread Alon Levy
On Thu, Mar 22, 2012 at 09:55:45PM +0800, Charles.Tsai-蔡清海-研究發展部 wrote: > Alon, > > I did all of what you mentioned before. Nothing is missed in my checklist. > Function "spice_server_char_device_add_interface" in fact is not called at > all. > Qemu somehow is blocked somewhere. For now, I still

Re: [Spice-devel] Fail to create a printing channel

2012-03-22 Thread Charles . Tsai-蔡清海-研究發展部
Alon, I did all of what you mentioned before. Nothing is missed in my checklist. Function "spice_server_char_device_add_interface" in fact is not called at all. Qemu somehow is blocked somewhere. For now, I still do not know the root cause. -Original Message- From: Alon Levy [mailto:al..

Re: [Spice-devel] Fail to create a printing channel

2012-03-22 Thread Alon Levy
On Thu, Mar 22, 2012 at 07:34:38PM +0800, Charles.Tsai-蔡清海-研究發展部 wrote: > Alon, Or anyone else on the list.. > > I am running the following script to create the printing channel based > on the virtio device framework. > When the VM is launched, it seems to block somewhere in Qemu. >

Re: [Spice-devel] [PATCH spice-gtk] RFC: always release shm of primary surfaces

2012-03-22 Thread Yonit Halperin
On 03/22/2012 02:02 PM, Marc-André Lureau wrote: On Thu, Mar 22, 2012 at 8:17 AM, Yonit Halperin wrote: Just notice that it is not only for the primary surface, but for every surface. Only if it has the SPICE_SURFACE_FLAGS_PRIMARY, if I read correctly. sorry, missed that. _

Re: [Spice-devel] [PATCH 0/5] 32 bit fixes

2012-03-22 Thread Alon Levy
On Thu, Mar 22, 2012 at 11:21:23AM +0100, Gerd Hoffmann wrote: > On 03/21/12 17:00, Alon Levy wrote: > > With those patches and a single patch to qemu I'll send soon F17-Alpha-i686 > > boots and loads qxl driver using qemu-system-i386 and spice-server built > > with -m32. > > I think this should g

Re: [Spice-devel] [PATCH spice-gtk] RFC: always release shm of primary surfaces

2012-03-22 Thread Marc-André Lureau
On Thu, Mar 22, 2012 at 8:17 AM, Yonit Halperin wrote: > Just notice that it is not only for the primary surface, but for every > surface. Only if it has the SPICE_SURFACE_FLAGS_PRIMARY, if I read correctly. -- Marc-André Lureau ___ Spice-devel mailin

[Spice-devel] Fail to create a printing channel

2012-03-22 Thread Charles . Tsai-蔡清海-研究發展部
Alon, I am running the following script to create the printing channel based on the virtio device framework. When the VM is launched, it seems to block somewhere in Qemu. If I remove the following two lines, the VM can be launched without any problem.

Re: [Spice-devel] [PATCH 0/5] 32 bit fixes

2012-03-22 Thread Gerd Hoffmann
On 03/21/12 17:00, Alon Levy wrote: > With those patches and a single patch to qemu I'll send soon F17-Alpha-i686 > boots and loads qxl driver using qemu-system-i386 and spice-server built > with -m32. I think this should get spice-server going on arm too, right? Acked-by: Gerd Hoffmann cheers,

Re: [Spice-devel] [PATCH] ui/spice-display: use uintptr_t when casting qxl physical addresses

2012-03-22 Thread Hans de Goede
Good one, ACK. Acked-by: Hans de Goede On 03/21/2012 05:17 PM, Alon Levy wrote: The current intptr_t casts are a problem when the address's highest bit is 1, and it is cast to a intptr_t and then to uint64_t, such as at: surface.mem= (intptr_t)ssd->buf; This causes the sign bit

Re: [Spice-devel] [PATCH spice-gtk] RFC: always release shm of primary surfaces

2012-03-22 Thread Hans de Goede
ACK! On 03/21/2012 07:53 PM, Marc-André Lureau wrote: Note: I can't reproduce the leak, even after dozen of resolution switch. Always delete shared memory segment of primary surfaces when destroying its canvas. --- gtk/channel-display.c |6 +- 1 files changed, 1 insertions(+), 5 dele

Re: [Spice-devel] [PATCH 0/5] 32 bit fixes

2012-03-22 Thread Hans de Goede
Looks good, ACK series. And thanks for looking into this, it is about time we start supporting 32 bits :) On 03/21/2012 05:00 PM, Alon Levy wrote: With those patches and a single patch to qemu I'll send soon F17-Alpha-i686 boots and loads qxl driver using qemu-system-i386 and spice-server built

Re: [Spice-devel] [PATCH spice-gtk 1/2] controller: handle SEND_CAD message as a property

2012-03-22 Thread Hans de Goede
Looks good, ACK series. On 03/21/2012 11:07 PM, Marc-André Lureau wrote: --- gtk/controller/controller.vala |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/gtk/controller/controller.vala b/gtk/controller/controller.vala index 12b5b8d..b083f4e 100644 --- a/gtk/con

Re: [Spice-devel] [PATCH 1/2] server/tests/test_display_base: add Command

2012-03-22 Thread Hans de Goede
Looks good, Ack series. On 03/21/2012 11:16 PM, Alon Levy wrote: add parameters to the commands used for display tests. --- server/tests/test_display_base.c | 110 ++ server/tests/test_display_base.h | 36 + 2 files changed, 101 insertions(+

Re: [Spice-devel] [PATCH spice-gtk] RFC: always release shm of primary surfaces

2012-03-22 Thread Yonit Halperin
On 03/21/2012 08:53 PM, Marc-André Lureau wrote: Note: I can't reproduce the leak, even after dozen of resolution switch. Always delete shared memory segment of primary surfaces when destroying its canvas. Ack. Tested. Just notice that it is not only for the primary surface, but for every sur