Re: [Spice-devel] Mouse failure happens when mouse hovers on some Apps like Kaspersky Anti-virus tool.

2012-08-16 Thread Arnon Gilboa
I guess you'll need to add a new agent msg for setting the mouse mode, plus small changes in agent client, as so far agent existence meant using client mouse. It will be nice if you'll post your patches for this. Arnon Naga Mohan Pothula wrote: Arnon, I tried //ret = _running =

[Spice-devel] [spice PATCH fix] display migration: restore destination state

2012-08-16 Thread Yonit Halperin
Restoring display channel from migration data. Not notifying client about changes that are artifacts of loading the vm. Remove legacy migration code. --- removed bad assert in display_channel_handle_migrate_data spice_assert(!display_channel-enable_jpeg); display_channel-enable_jpeg can be TRUE

Re: [Spice-devel] change the Monitor refresh rate of VM

2012-08-16 Thread Michael Tokarev
2012/8/15 flooding Controlled flooding2...@gmail.com mailto:flooding2...@gmail.com Hi all: I am now using spice now, but stucked by some problems. My vm is XP, and I can the monitor refresh rate! It is always 100HZ, and it it too high, How can I decrease it ? What

Re: [Spice-devel] change the Monitor refresh rate of VM

2012-08-16 Thread flooding Controlled
I just want to make sure that is could be changed or not. And, I think 100HZ is too high, normally 75~85HZ is OK for PC, and 60HZ for laptop. 2012/8/16 Michael Tokarev m...@tls.msk.ru 2012/8/15 flooding Controlled flooding2...@gmail.com mailto: flooding2...@gmail.com Hi all:

Re: [Spice-devel] change the Monitor refresh rate of VM

2012-08-16 Thread Michael Tokarev
On 16.08.2012 11:51, flooding Controlled wrote: I just want to make sure that is could be changed or not. And, I think 100HZ is too high, normally 75~85HZ is OK for PC, and 60HZ for laptop. Too high for what? The refresh rate is how often a (physical) video card sends information to a

Re: [Spice-devel] [PATCH spice-gtk 1/5] util-priv: factor out spice_make_scancode()

2012-08-16 Thread Hans de Goede
Looks good, ack. On 08/15/2012 09:59 PM, Marc-André Lureau wrote: Factor out the keyboard scancode manipulation function, to be reusable by newer code. --- gtk/channel-inputs.c | 22 -- gtk/spice-util-priv.h |1 + gtk/spice-util.c | 21 +

Re: [Spice-devel] [PATCH spice-gtk 2/5] inputs: add spice_inputs_key_press_and_release()

2012-08-16 Thread Hans de Goede
One small remark below, not sure if it is worth fixing though, ack either way. On 08/15/2012 09:59 PM, Marc-André Lureau wrote: If the server is capable of SPICE_INPUTS_CAP_SCANCODE, then we send can send a single message with key press and release, to avoid unwanted guest side key repeatition

Re: [Spice-devel] [PATCH spice-gtk 3/5] widget: give more context to send_key()

2012-08-16 Thread Hans de Goede
ACK. On 08/15/2012 09:59 PM, Marc-André Lureau wrote: - use a more explicit SendKeyType enum - if the key is a modifier key, we don't want to delay press event --- gtk/spice-widget.c | 21 + 1 file changed, 13 insertions(+), 8 deletions(-) diff --git

Re: [Spice-devel] [PATCH spice-gtk 4/5] widget: add keypress-delay property

2012-08-16 Thread Hans de Goede
ACK. On 08/15/2012 09:59 PM, Marc-André Lureau wrote: The delay before the press event is sent to the server if the key is kept pressed. If the key is released within that time, that delay is ignored and a single key-press-release event will be sent. --- gtk/spice-widget-priv.h |1 +

Re: [Spice-devel] [PATCH spice-gtk 5/5] widget: differentiate key press release from press only events

2012-08-16 Thread Christophe Fergeau
I haven't looked at the patch, but the log has some typos: On Wed, Aug 15, 2012 at 10:59:30PM +0300, Marc-André Lureau wrote: Until now, Spice clients only sent seperate key events for press and separate release. But this may result in unwanted key repeatition from guest VM repetition

Re: [Spice-devel] [PATCH spice-gtk 5/5] widget: differentiate key press release from press only events

2012-08-16 Thread Hans de Goede
Hi, On 08/15/2012 09:59 PM, Marc-André Lureau wrote: Until now, Spice clients only sent seperate key events for press and release. But this may result in unwanted key repeatition from guest VM side. It seems OSes have various implementation. While MS Windows rely on hardware key repeats (which

Re: [Spice-devel] [Users] Spice console - Windows

2012-08-16 Thread Christophe Fergeau
On Wed, Aug 15, 2012 at 11:49:05PM +0300, Itamar Heim wrote: On 08/15/2012 11:15 PM, Ricardo Esteves wrote: Hi, I logged in to ovirt manager using IE9 (64 bits) and when I click the console icon of my VM nothing happens. virt-viewer-0.5.3_x64.exe

Re: [Spice-devel] QEMU 1.2 Test Day - August 16 2012

2012-08-16 Thread Stefan Hajnoczi
On Tue, Aug 14, 2012 at 2:37 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Thu, Aug 2, 2012 at 1:22 PM, Stefan Hajnoczi stefa...@gmail.com wrote: I have set up the QEMU 1.2 Testing wiki page and suggest August 16 as the Test Day: http://wiki.qemu.org/Planning/1.2/Testing QEMU 1.2 Test

Re: [Spice-devel] [Users] Spice console - Windows

2012-08-16 Thread David Jaša
CCing Peter Christophe Fergeau píše v Čt 16. 08. 2012 v 11:15 +0200: On Wed, Aug 15, 2012 at 11:49:05PM +0300, Itamar Heim wrote: On 08/15/2012 11:15 PM, Ricardo Esteves wrote: Hi, I logged in to ovirt manager using IE9 (64 bits) and when I click the console icon of my VM nothing happens.

Re: [Spice-devel] change the Monitor refresh rate of VM

2012-08-16 Thread Yaniv Kaul
It's just a number, I don't think we are doing anything with it. miniport/qxl.c: video_mode-Frequency = 100; And we don't do a lot with it later, I think. The real question is what the client refresh rate is, which I don't think we touch. Y. - Original Message - I just want to

Re: [Spice-devel] change the Monitor refresh rate of VM

2012-08-16 Thread David Jaša
flooding Controlled píše v Čt 16. 08. 2012 v 15:51 +0800: I just want to make sure that is could be changed or not. And, I think 100HZ is too high, normally 75~85HZ is OK for PC, and 60HZ for laptop. I think you can safely ignore it as long as you don't experience frequency-mismatch-caused

[Spice-devel] [PATCH][spice-html5] prevent out of sync modifier keys

2012-08-16 Thread Aric Stewart
Signed-off-by: Aric Stewart a...@codeweavers.com --- inputs.js | 86 + 1 files changed, 86 insertions(+), 0 deletions(-) diff --git a/inputs.js b/inputs.js index ce1d15e..57ee626 100644 --- a/inputs.js +++ b/inputs.js @@ -18,6 +18,15

[Spice-devel] status of my spice-html5 patches

2012-08-16 Thread Aric Stewart
Hi all, Just checking on the spice-html5 patches I sent a while ago. I see they have not gotten pushed and am wondering what i should do to help that process. thanks! -aric ___ Spice-devel mailing list Spice-devel@lists.freedesktop.org

Re: [Spice-devel] status of my spice-html5 patches

2012-08-16 Thread Alon Levy
On Thu, Aug 16, 2012 at 07:08:14AM -0500, Aric Stewart wrote: Hi all, Just checking on the spice-html5 patches I sent a while ago. I see they have not gotten pushed and am wondering what i should do to help that process. I can push them for you, or anyone else with commit access (like

Re: [Spice-devel] status of my spice-html5 patches

2012-08-16 Thread Aric Stewart
Hi Jeremy has told me that he does not have commit access. So if you would kindly push that would be greatly appreciated. thanks! -aric On 8/16/12 10:10 AM, Alon Levy wrote: On Thu, Aug 16, 2012 at 07:08:14AM -0500, Aric Stewart wrote: Hi all, Just checking on the spice-html5 patches I

Re: [Spice-devel] status of my spice-html5 patches

2012-08-16 Thread Aric Stewart
Makes sense, probably depends on how many people are out there reading the Javascript patches, my guess is it is just Jeremy and I. Jeremy is on vacation for the next 2 weeks. But I will tell him to more publicly ack the patches on the list. -aric On 8/16/12 10:28 AM, Alon Levy wrote: On

[Spice-devel] [PATCH spice-gtk 1/5] util-priv: factor out spice_make_scancode()

2012-08-16 Thread Marc-André Lureau
Factor out the keyboard scancode manipulation function, to be reusable by newer code. --- gtk/channel-inputs.c | 22 -- gtk/spice-util-priv.h |1 + gtk/spice-util.c | 21 + 3 files changed, 26 insertions(+), 18 deletions(-) diff --git

[Spice-devel] [PATCH spice-gtk 2/5] inputs: add spice_inputs_key_press_and_release()

2012-08-16 Thread Marc-André Lureau
If the server is capable of SPICE_INPUTS_CAP_SCANCODE, then we send can send a single message with key press and release, to avoid unwanted guest side key repeatition due to network jitter. If the server is not capable, spice-gtk will use some compatibility mode and send the existing DOWN and UP

[Spice-devel] [PATCH spice-gtk 3/5] widget: give more context to send_key()

2012-08-16 Thread Marc-André Lureau
- use a more explicit SendKeyType enum - if the key is a modifier key, we don't want to delay press event v2: fix compilation (remove down usage) --- gtk/spice-widget.c | 23 ++- 1 file changed, 14 insertions(+), 9 deletions(-) diff --git a/gtk/spice-widget.c

[Spice-devel] [PATCH spice-gtk 4/5] widget: add keypress-delay property

2012-08-16 Thread Marc-André Lureau
The delay before the press event is sent to the server if the key is kept pressed. If the key is released within that time, that delay is ignored and a single key-press-release event will be sent. --- gtk/spice-widget-priv.h |1 + gtk/spice-widget.c | 27 +++ 2

[Spice-devel] [PATCH spice-gtk 5/5] widget: differentiate key press release from press only events

2012-08-16 Thread Marc-André Lureau
Until now, Spice clients only sent separate key events for press and release. But this may result in unwanted key repetition from guest VM side. It seems OSes have various implementations. While MS Windows relies on hardware key repeats (which are several sequential press events), otoh, X11 uses

Re: [Spice-devel] [PATCH spice-gtk 1/5] util-priv: factor out spice_make_scancode()

2012-08-16 Thread Hans de Goede
Looks good now, ack series. On 08/16/2012 05:35 PM, Marc-André Lureau wrote: Factor out the keyboard scancode manipulation function, to be reusable by newer code. --- gtk/channel-inputs.c | 22 -- gtk/spice-util-priv.h |1 + gtk/spice-util.c | 21

Re: [Spice-devel] status of my spice-html5 patches

2012-08-16 Thread Alon Levy
On Thu, Aug 16, 2012 at 10:34:38AM -0500, Aric Stewart wrote: Makes sense, probably depends on how many people are out there reading the Javascript patches, my guess is it is just Jeremy and I. Jeremy is on vacation for the next 2 weeks. But I will tell him to more publicly ack the patches

[Spice-devel] (resend) [spice-html5] Add the scroll to the cursor offset

2012-08-16 Thread Aric Stewart
Signed-off-by: Aric Stewart a...@codeweavers.com --- spicemsg.js |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/spicemsg.js b/spicemsg.js index 41b001d..79c76bc 100644 --- a/spicemsg.js +++ b/spicemsg.js @@ -639,8 +639,8 @@ function SpiceMsgcMousePosition(sc, e)

[Spice-devel] [resend][spice-html5] work to improve keyboard keycode - scancode processing

2012-08-16 Thread Aric Stewart
Signed-off-by: Aric Stewart a...@codeweavers.com --- atKeynames.js | 183 +++ utils.js | 266 +++-- 2 files changed, 327 insertions(+), 122 deletions(-) create mode 100644 atKeynames.js diff --git

[Spice-devel] (resend)[spice-html5] prevent out of sync modifier keys

2012-08-16 Thread Aric Stewart
Signed-off-by: Aric Stewart a...@codeweavers.com --- inputs.js | 86 + 1 files changed, 86 insertions(+), 0 deletions(-) diff --git a/inputs.js b/inputs.js index ce1d15e..57ee626 100644 --- a/inputs.js +++ b/inputs.js @@ -18,6 +18,15

Re: [Spice-devel] status of my spice-html5 patches

2012-08-16 Thread Aric Stewart
Ok, I think i have finally gotten word wrapping correctly turned off (the source of my patch corruption) resent. -aric On 8/16/12 11:42 AM, Alon Levy wrote: On Thu, Aug 16, 2012 at 10:34:38AM -0500, Aric Stewart wrote: Makes sense, probably depends on how many people are out there reading

Re: [Spice-devel] spice-gtk usb-redirection

2012-08-16 Thread Jackson
ctrl endpoint request request_type value index length [data] And what's the main action and purpose of this cmd? thanks ___ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/spice-devel