[PATCH xserver 2/2 v2] xwayland: add a server sync before repeating keys

Olivier Fourdan ofourdan at redhat.com
Tue Mar 8 15:23:35 UTC 2016


Key repeat is handled by the X server, but input events need to be
processed and forwarded by the Wayland compositor first.

Make sure the Wayland compositor is actually processing events, to
avoid repeating keys in xwayland while the Wayland compositor cannot
deal with input events for whatever reason, thus not dispatching key
release events, leading to fictious repeat keys while the use has
already released the key.

Bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=762618
Signed-off-by: Olivier Fourdan <ofourdan at redhat.com>
---

 v2: A slightly less hack-y-ish version of the patch
     Use xwl_seat to store the pending sync, set up callback fpr keyboard
     in seat_handle_capabilities() and add a call to _dispatch_pending()
     in the callback function to make sure we didn't miss a reply

 hw/xwayland/xwayland-input.c | 46 ++++++++++++++++++++++++++++++++++++++++++--
 hw/xwayland/xwayland.h       |  2 ++
 2 files changed, 46 insertions(+), 2 deletions(-)

diff --git a/hw/xwayland/xwayland-input.c b/hw/xwayland/xwayland-input.c
index f9e3255..afe2e27 100644
--- a/hw/xwayland/xwayland-input.c
+++ b/hw/xwayland/xwayland-input.c
@@ -41,7 +41,7 @@
         (miPointerPtr)dixLookupPrivate(&(dev)->devPrivates, miPointerPrivKey): \
         (miPointerPtr)dixLookupPrivate(&(GetMaster(dev, MASTER_POINTER))->devPrivates, miPointerPrivKey))
 
-static void
+	static void
 xwl_pointer_control(DeviceIntPtr device, PtrCtrl *ctrl)
 {
     /* Nothing to do, dix handles all settings */
@@ -527,6 +527,43 @@ keyboard_handle_modifiers(void *data, struct wl_keyboard *keyboard,
 }
 
 static void
+sync_callback(void *data, struct wl_callback *callback, uint32_t serial)
+{
+    struct xwl_seat *xwl_seat = data;
+
+    xwl_seat->sync_pending = FALSE;
+    wl_callback_destroy(callback);
+}
+
+static const struct wl_callback_listener sync_listener = {
+   sync_callback
+};
+
+static Bool
+keyboard_check_repeat (DeviceIntPtr dev, XkbSrvInfoPtr xkbi, unsigned key)
+{
+    struct xwl_seat *xwl_seat = dev->public.devicePrivate;
+    struct xwl_screen *xwl_screen = xwl_seat->xwl_screen;
+    struct wl_callback *callback;
+
+    /* Make sure we didn't miss a possible reply from the compositor */
+    wl_display_dispatch_pending (xwl_screen->display);
+
+    if (!xwl_seat->sync_pending) {
+        callback = wl_display_sync (xwl_screen->display);
+        wl_callback_add_listener(callback, &sync_listener, xwl_seat);
+        xwl_seat->sync_pending = TRUE;
+
+        return TRUE;
+    }
+
+    ErrorF("Key repeat discarded, Wayland compositor doesn't seem to "
+           "be processing events fast enough!\n");
+
+    return FALSE;
+}
+
+static void
 keyboard_handle_repeat_info (void *data, struct wl_keyboard *keyboard,
                              int32_t rate, int32_t delay)
 {
@@ -716,6 +753,7 @@ seat_handle_capabilities(void *data, struct wl_seat *seat,
                          enum wl_seat_capability caps)
 {
     struct xwl_seat *xwl_seat = data;
+    DeviceIntPtr master;
 
     if (caps & WL_SEAT_CAPABILITY_POINTER && xwl_seat->wl_pointer == NULL) {
         xwl_seat->wl_pointer = wl_seat_get_pointer(seat);
@@ -748,12 +786,16 @@ seat_handle_capabilities(void *data, struct wl_seat *seat,
             ActivateDevice(xwl_seat->keyboard, TRUE);
         }
         EnableDevice(xwl_seat->keyboard, TRUE);
+        master = GetMaster(xwl_seat->keyboard, MASTER_KEYBOARD);
+        master->key->xkbInfo->checkRepeat = keyboard_check_repeat;
     } else if (!(caps & WL_SEAT_CAPABILITY_KEYBOARD) && xwl_seat->wl_keyboard) {
         wl_keyboard_release(xwl_seat->wl_keyboard);
         xwl_seat->wl_keyboard = NULL;
 
-        if (xwl_seat->keyboard)
+        if (xwl_seat->keyboard) {
+            xwl_seat->keyboard->key->xkbInfo->checkRepeat = NULL;
             DisableDevice(xwl_seat->keyboard, TRUE);
+        }
     }
 
     if (caps & WL_SEAT_CAPABILITY_TOUCH && xwl_seat->wl_touch == NULL) {
diff --git a/hw/xwayland/xwayland.h b/hw/xwayland/xwayland.h
index 2a54183..aaf3b40 100644
--- a/hw/xwayland/xwayland.h
+++ b/hw/xwayland/xwayland.h
@@ -138,6 +138,8 @@ struct xwl_seat {
     size_t keymap_size;
     char *keymap;
     struct wl_surface *keyboard_focus;
+
+    Bool sync_pending;
 };
 
 struct xwl_output {
-- 
2.5.0



More information about the xorg-devel mailing list