Re: [Spice-devel] Performance

2013-03-13 Thread Han Pilmeyer
On 13/03/2013 11:39, Christophe Fergeau wrote: On Tue, Mar 12, 2013 at 01:17:26PM +0100, Han Pilmeyer wrote: I thought it might be nice to chime in with my results. I'm running nearly the same versions, except that I'm still using the 0.30 version of the spice-guest-tools. I just tested my net

Re: [Spice-devel] [PATCH 1/2][spice-html5] added sendCtrlAltDel method.

2013-03-13 Thread Jeremy White
Hi, If you're not going to respond to my comments: http://lists.freedesktop.org/archives/spice-devel/2013-March/012529.html I'm certainly not going to commit anything you send. Cheers, Jeremy ___ Spice-devel mailing list Spice-devel@lists.freedesktop

Re: [Spice-devel] German keymap in Windows 7

2013-03-13 Thread Jens Voskuhl
O.K., so i would try a new version of virt-viewer when it is available. Jens 2013/3/13 David Jaša > Hi, > > Jens Voskuhl píše v St 13. 03. 2013 v 18:08 +0100: > > Problem is solved. > > sort of, given that remote-viewer is _the_ client that should be used > and old spice-client-win is being dep

Re: [Spice-devel] German keymap in Windows 7

2013-03-13 Thread David Jaša
Hi, Jens Voskuhl píše v St 13. 03. 2013 v 18:08 +0100: > Problem is solved. sort of, given that remote-viewer is _the_ client that should be used and old spice-client-win is being deprecated... > I think it was an issue in the spice client (virt-viewer for windows > version 0.5.5). The older sp

Re: [Spice-devel] German keymap in Windows 7

2013-03-13 Thread Jens Voskuhl
Problem is solved. I think it was an issue in the spice client (virt-viewer for windows version 0.5.5). The older spice-client-win (version 0.6.3) works for me (qwertz input/output). My vm is windows 7 SP1 german (x32). Also my client is windows 7 SP1 german (x64). Input language is german in all c

Re: [Spice-devel] German keymap in Windows 7

2013-03-13 Thread David Jaša
Hi, In addition to Alon's questions, Alon Levy píše v St 13. 03. 2013 v 12:16 -0400: > > > > Hi, > > > > > > i`m running Windows 7 Prof. 32-bit under KVM/QEMU. I`m using spice > > togehter with qxl. Guest drivers are all installed. > > I can succesfully connect to the guest client and everythi

Re: [Spice-devel] German keymap in Windows 7

2013-03-13 Thread Alon Levy
> > Hi, > > > i`m running Windows 7 Prof. 32-bit under KVM/QEMU. I`m using spice > togehter with qxl. Guest drivers are all installed. > I can succesfully connect to the guest client and everything works > fine. But my keyboard is not using qwertz. The input language > in the Windows 7 guest is

[Spice-devel] [PATCH 2/2][spice-html5] added on succeess event

2013-03-13 Thread Amos Benari
- Forwarded Message - From: "Amos Benari" To: spice-devel@lists.freedesktop.org Cc: "Amos Benari" Sent: Wednesday, March 13, 2013 3:31:53 PM Subject: [PATCH 2/2] added on succeess event --- main.js |1 + spiceconn.js |8 2 files changed, 9 insertions(+), 0 deleti

[Spice-devel] [PATCH 1/2][spice-html5] added sendCtrlAltDel method.

2013-03-13 Thread Amos Benari
- Forwarded Message - From: "Amos Benari" To: spice-devel@lists.freedesktop.org Cc: "Amos Benari" Sent: Wednesday, March 13, 2013 3:30:54 PM Subject: [PATCH 1/2] added sendCtrlAltDel method. --- inputs.js | 20 1 files changed, 20 insertions(+), 0 deletions(-)

[Spice-devel] German keymap in Windows 7

2013-03-13 Thread Jens Voskuhl
Hi, i`m running Windows 7 Prof. 32-bit under KVM/QEMU. I`m using spice togehter with qxl. Guest drivers are all installed. I can succesfully connect to the guest client and everything works fine. But my keyboard is not using qwertz. The input language in the Windows 7 guest is DE. When i type z it

Re: [Spice-devel] 回复: spice client migrate to Android & IOS plateform

2013-03-13 Thread shohyanglim
Yes, it's experimental only and had been created and maintained by me, but no updating for a long time BR. Rozen.Lin. On Fri, 2013-03-08 at 13:17 +0100, Attila Sukosd wrote: > There has been some efforts to have an Android spice client: > > > http://code.google.com/p/spice-client-android/ >

Re: [Spice-devel] [spice-xpi PATCHv2 01/12] Split nsPluginInstance::connect a bit

2013-03-13 Thread Marc-André Lureau
Except the g_message vs g_critical, ack the series. -- Marc-André Lureau ___ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/spice-devel

Re: [Spice-devel] [spice-xpi PATCHv2 02/12] Use gio to write trust store

2013-03-13 Thread Marc-André Lureau
On Wed, Mar 13, 2013 at 11:15 AM, Christophe Fergeau wrote: > +tmp_file = g_file_new_tmp("trustore.pem-XX", &iostream, NULL); > +if (tmp_file == NULL) { > +g_message("Couldn't create truststore"); > +return false; > } > -else > -{ > -g_critical("cou

Re: [Spice-devel] spice-guest-tools 0.52 and winxp doesn't boot anymore

2013-03-13 Thread Gianluca Cecchi
On Wed, Mar 13, 2013 at 10:01 AM, Christophe Fergeau wrote: > > If the network driver is the culprit, just uninstall it and forget about > that issue, it's generally not that important, you'll get network without > it, just a bit less performant. > Actually the network adapter is virtio so I do ne

Re: [Spice-devel] Performance

2013-03-13 Thread Christophe Fergeau
On Tue, Mar 12, 2013 at 01:17:26PM +0100, Han Pilmeyer wrote: > I thought it might be nice to chime in with my results. I'm running > nearly the same versions, except that I'm still using the 0.30 > version of the spice-guest-tools. > > I just tested my network speed at the KVM host and in my Wind

[Spice-devel] [spice-xpi PATCHv2 02/12] Use gio to write trust store

2013-03-13 Thread Christophe Fergeau
This has the big advantage of being portable, in addition to removing some code. Big disadvantage is that this adds a dependency on gio. --- SpiceXPI/src/plugin/plugin.cpp | 69 ++ SpiceXPI/src/plugin/plugin.h | 2 ++ configure.ac | 2 +

Re: [Spice-devel] [spice-xpi 08/12] Use glib functions to spawn/watch client

2013-03-13 Thread Christophe Fergeau
On Tue, Mar 12, 2013 at 01:16:20PM +0100, Marc-André Lureau wrote: > On Tue, Mar 12, 2013 at 12:23 PM, Christophe Fergeau > wrote: > > +/* FIXME: we are not in the main thread!! */ > > +fake_this->m_plugin->OnSpiceClientExit(status); > > That scares me, but it looks like you didn't introd

[Spice-devel] [spice-xpi PATCHv2 12/12] Add an abstract controller base class

2013-03-13 Thread Christophe Fergeau
It contains all the generic code, then platform-specific controller implementations will inherit from it and overload the non-generic parts abstract methods. --- v2: - adjust to the changes in other patches, in particular the split of GetClientPaths, which should fix a compilation issue that

[Spice-devel] [spice-xpi PATCHv2 11/12] Use g_usleep in SpiceController::connect(int retries)

2013-03-13 Thread Christophe Fergeau
g_usleep is more portable than sleep() --- SpiceXPI/src/plugin/controller-unix.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/SpiceXPI/src/plugin/controller-unix.cpp b/SpiceXPI/src/plugin/controller-unix.cpp index 16f7033..04257e9 100644 --- a/SpiceXPI/src/plugin/controll

[Spice-devel] [spice-xpi PATCHv2 10/12] Add helper function to get client paths

2013-03-13 Thread Christophe Fergeau
The SPICE client paths will be different on Windows and Unix so add a helper function returning the paths to use to spawn the clients. --- v2: - use g_strdupv rather than manually duping everything - split GetClientPaths to GetClientPath and GetFallbackClientPath SpiceXPI/src/plugin/controlle

[Spice-devel] [spice-xpi PATCHv2 09/12] Add SpiceController::SetupControllerPipe

2013-03-13 Thread Christophe Fergeau
Communication with the client is done differently on Windows and Linux (named pipe VS unix socket), so move this platform-specific setup to a distinct method. --- SpiceXPI/src/plugin/controller-unix.cpp | 15 +++ SpiceXPI/src/plugin/controller.h| 2 ++ 2 files changed, 13 inse

[Spice-devel] [spice-xpi PATCHv2 08/12] Use glib functions to spawn/watch client

2013-03-13 Thread Christophe Fergeau
This removes quite a lot of OS-specific code. --- v2 - use g_thread_new instead of g_thread_create SpiceXPI/src/plugin/controller-unix.cpp | 143 +--- SpiceXPI/src/plugin/controller.h| 7 +- configure.ac| 2 +- 3 files changed

[Spice-devel] [spice-xpi PATCHv2 07/12] Move StartClient() to the controller class

2013-03-13 Thread Christophe Fergeau
This is platform specific, and is related to starting the receiver of controller messages, so let's move it with the rest of the platform-specific code. --- SpiceXPI/src/plugin/controller-unix.cpp | 106 +- SpiceXPI/src/plugin/controller.h| 13 +++- SpiceXPI/sr

[Spice-devel] [spice-xpi PATCHv2 06/12] Mark Controller::Connect() as private

2013-03-13 Thread Christophe Fergeau
It's only used internally, external code calls Connect(int retries) --- SpiceXPI/src/plugin/controller.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/SpiceXPI/src/plugin/controller.h b/SpiceXPI/src/plugin/controller.h index dbcc424..001d2b3 100644 --- a/SpiceXPI/src/plugin/c

[Spice-devel] [spice-xpi PATCHv2 05/12] Remove unused controller constructor

2013-03-13 Thread Christophe Fergeau
--- SpiceXPI/src/plugin/controller-unix.cpp | 6 -- SpiceXPI/src/plugin/controller.h| 1 - 2 files changed, 7 deletions(-) diff --git a/SpiceXPI/src/plugin/controller-unix.cpp b/SpiceXPI/src/plugin/controller-unix.cpp index 2499cae..0d8f0f8 100644 --- a/SpiceXPI/src/plugin/controller

[Spice-devel] [spice-xpi PATCHv2 04/12] Update copyright notices

2013-03-13 Thread Christophe Fergeau
--- v2: - extend copyright range from 2009-2011 to 2009-2013 rather than adding a new line SpiceXPI/src/plugin/controller-unix.cpp | 3 ++- SpiceXPI/src/plugin/controller.h| 3 ++- SpiceXPI/src/plugin/plugin.cpp | 3 ++- 3 files changed, 6 insertions(+), 3 deletions(-) dif

[Spice-devel] [spice-xpi PATCHv2 03/12] Rename controller.cpp to controller-unix.cpp

2013-03-13 Thread Christophe Fergeau
It mostly contain low-level controller/plugin communication code, and this is mainly non-portable code, let's rename this file to make this explicit. We will then add a controller.cpp file containing the generic code, and a controller-win.cpp file with the Windows specific code. --- SpiceXPI/src/p

[Spice-devel] [spice-xpi PATCHv2 01/12] Split nsPluginInstance::connect a bit

2013-03-13 Thread Christophe Fergeau
This makes it more readable, and moves OS-specific code to separate functions. --- v2: - only call g_type_init() on older glib - removed extra space SpiceXPI/src/plugin/plugin.cpp | 170 +++-- SpiceXPI/src/plugin/plugin.h | 3 + 2 files changed, 99 inse

Re: [Spice-devel] spice-guest-tools 0.52 and winxp doesn't boot anymore

2013-03-13 Thread Christophe Fergeau
Hey Gianluca On Tue, Mar 12, 2013 at 09:54:56PM +0100, Gianluca Cecchi wrote: > On Mon, Mar 11, 2013 at 10:00 AM, Christophe Fergeau wrote: > > > There are various issues with this release, could you try to disable the > > balloon service (blnsvr.exe)? > > If this does not help, can you try remov