efcf6cdbc8
Update to qemu 3.1.0-rc5. Is almost certainly the last rc, so should be same as 3.1.0 final. Putting into devel project 'early' because of SLE and Leap needs, not to get into Factory early. Look for the final 3.1 within a week. OBS-URL: https://build.opensuse.org/request/show/655897 OBS-URL: https://build.opensuse.org/package/show/Virtualization/qemu?expand=0&rev=433
56 lines
2.0 KiB
Diff
56 lines
2.0 KiB
Diff
From 18166e0ec877624187ecfe11070bd748ea0b0a74 Mon Sep 17 00:00:00 2001
|
|
From: Chunyan Liu <cyliu@suse.com>
|
|
Date: Thu, 3 Mar 2016 16:48:17 +0800
|
|
Subject: [PATCH] Fix tigervnc long press issue
|
|
|
|
Using xen tools 'xl vncviewer' with tigervnc (default on SLE-12),
|
|
found that: the display of the guest is unexpected while keep
|
|
pressing a key. We expect the same character multiple times, but
|
|
it prints only one time. This happens on a PV guest in text mode.
|
|
|
|
After debugging, found that tigervnc sends repeated key down events
|
|
in this case, to differentiate from user pressing the same key many
|
|
times. Vnc server only prints the character when it finally receives
|
|
key up event.
|
|
|
|
To solve this issue, this patch tries to add additional key up event
|
|
before the next repeated key down event (if the key is not a control
|
|
key).
|
|
|
|
[CYL: BSC#882405]
|
|
Signed-off-by: Chunyan Liu <cyliu@suse.com>
|
|
---
|
|
ui/vnc.c | 19 +++++++++++++++++++
|
|
1 file changed, 19 insertions(+)
|
|
|
|
diff --git a/ui/vnc.c b/ui/vnc.c
|
|
index 0c1b477425..1c98bb91c7 100644
|
|
--- a/ui/vnc.c
|
|
+++ b/ui/vnc.c
|
|
@@ -1824,6 +1824,25 @@ static void do_key_event(VncState *vs, int down, int keycode, int sym)
|
|
if (down)
|
|
vs->modifiers_state[keycode] ^= 1;
|
|
break;
|
|
+ default:
|
|
+ if (qemu_console_is_graphic(NULL)) {
|
|
+ /* record key 'down' info. Some client like tigervnc
|
|
+ * will send key down repeatedly if user pressing a
|
|
+ * a key for long time. In this case, we should add
|
|
+ * additional key up event before repeated key down,
|
|
+ * so that it can display the key multiple times.
|
|
+ */
|
|
+ if (down) {
|
|
+ if (vs->modifiers_state[keycode]) {
|
|
+ /* add a key up event */
|
|
+ do_key_event(vs, 0, keycode, sym);
|
|
+ }
|
|
+ vs->modifiers_state[keycode] = 1;
|
|
+ } else {
|
|
+ vs->modifiers_state[keycode] = 0;
|
|
+ }
|
|
+ }
|
|
+ break;
|
|
}
|
|
|
|
/* Turn off the lock state sync logic if the client support the led
|