libinput 1.9.4
This commit is contained in:
Peter Hutterer
parent
c8c113962b
commit
3f8446aaf6
1
.gitignore
vendored
1
.gitignore
vendored
@ -12,3 +12,4 @@
|
|||||||
/libinput-1.9.1.tar.xz
|
/libinput-1.9.1.tar.xz
|
||||||
/libinput-1.9.2.tar.xz
|
/libinput-1.9.2.tar.xz
|
||||||
/libinput-1.9.3.tar.xz
|
/libinput-1.9.3.tar.xz
|
||||||
|
/libinput-1.9.4.tar.xz
|
||||||
|
|||||||
@ -1,163 +0,0 @@
|
|||||||
From 1c8636923b7d8245a5f55dfbfa191c0129ad4414 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Peter Hutterer <peter.hutterer@who-t.net>
|
|
||||||
Date: Fri, 8 Dec 2017 09:41:07 +1000
|
|
||||||
Subject: [PATCH libinput] fallback: send key events out immediately upon
|
|
||||||
receiving them
|
|
||||||
|
|
||||||
Commit db3b6fe5f7f8 "fallback: change to handle the state at EV_SYN time"
|
|
||||||
introduced regressions for two types of event sequences.
|
|
||||||
|
|
||||||
One is a kernel bug - some devices/drivers like the asus-wireless send a key
|
|
||||||
press + release within the same event frame which now cancels out and
|
|
||||||
disappears into the ether. This should be fixed in the kernel drivers but
|
|
||||||
there appear to be enough of them that we can't just pretend it's an outlier.
|
|
||||||
|
|
||||||
The second issue is a libinput bug. If we get two key events in the same frame
|
|
||||||
(e.g. shift + A) we update the state correctly but the events are sent in the
|
|
||||||
order of the event codes. KEY_A sorts before KEY_LEFTSHIFT and our shift + A
|
|
||||||
becomes A + shift.
|
|
||||||
|
|
||||||
Fix this by treating key events as before db3b6fe5f7f8 - by sending them out
|
|
||||||
as we get them.
|
|
||||||
|
|
||||||
https://bugs.freedesktop.org/show_bug.cgi?id=104030
|
|
||||||
|
|
||||||
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
||||||
---
|
|
||||||
src/evdev-fallback.c | 35 ++++++++++++++++----------------
|
|
||||||
test/test-keyboard.c | 56 ++++++++++++++++++++++++++++++++++++++++++++++++++++
|
|
||||||
2 files changed, 73 insertions(+), 18 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/src/evdev-fallback.c b/src/evdev-fallback.c
|
|
||||||
index 52a3bde1..0cd2497e 100644
|
|
||||||
--- a/src/evdev-fallback.c
|
|
||||||
+++ b/src/evdev-fallback.c
|
|
||||||
@@ -501,6 +501,22 @@ fallback_process_key(struct fallback_dispatch *dispatch,
|
|
||||||
}
|
|
||||||
|
|
||||||
hw_set_key_down(dispatch, e->code, e->value);
|
|
||||||
+
|
|
||||||
+ switch (type) {
|
|
||||||
+ case KEY_TYPE_NONE:
|
|
||||||
+ break;
|
|
||||||
+ case KEY_TYPE_KEY:
|
|
||||||
+ fallback_keyboard_notify_key(
|
|
||||||
+ dispatch,
|
|
||||||
+ device,
|
|
||||||
+ time,
|
|
||||||
+ e->code,
|
|
||||||
+ e->value ? LIBINPUT_KEY_STATE_PRESSED :
|
|
||||||
+ LIBINPUT_KEY_STATE_RELEASED);
|
|
||||||
+ break;
|
|
||||||
+ case KEY_TYPE_BUTTON:
|
|
||||||
+ break;
|
|
||||||
+ }
|
|
||||||
}
|
|
||||||
|
|
||||||
static void
|
|
||||||
@@ -834,27 +850,10 @@ fallback_handle_state(struct fallback_dispatch *dispatch,
|
|
||||||
if (dispatch->pending_event & EVDEV_KEY) {
|
|
||||||
bool want_debounce = false;
|
|
||||||
for (unsigned int code = 0; code <= KEY_MAX; code++) {
|
|
||||||
- bool new_state;
|
|
||||||
-
|
|
||||||
if (!hw_key_has_changed(dispatch, code))
|
|
||||||
continue;
|
|
||||||
|
|
||||||
- new_state = hw_is_key_down(dispatch, code);
|
|
||||||
-
|
|
||||||
- switch (get_key_type(code)) {
|
|
||||||
- case KEY_TYPE_NONE:
|
|
||||||
- break;
|
|
||||||
- case KEY_TYPE_KEY:
|
|
||||||
- fallback_keyboard_notify_key(
|
|
||||||
- dispatch,
|
|
||||||
- device,
|
|
||||||
- time,
|
|
||||||
- code,
|
|
||||||
- new_state ?
|
|
||||||
- LIBINPUT_KEY_STATE_PRESSED :
|
|
||||||
- LIBINPUT_KEY_STATE_RELEASED);
|
|
||||||
- break;
|
|
||||||
- case KEY_TYPE_BUTTON:
|
|
||||||
+ if (get_key_type(code) == KEY_TYPE_BUTTON) {
|
|
||||||
want_debounce = true;
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
diff --git a/test/test-keyboard.c b/test/test-keyboard.c
|
|
||||||
index dc2e630e..db836381 100644
|
|
||||||
--- a/test/test-keyboard.c
|
|
||||||
+++ b/test/test-keyboard.c
|
|
||||||
@@ -365,6 +365,61 @@ START_TEST(keyboard_no_buttons)
|
|
||||||
}
|
|
||||||
END_TEST
|
|
||||||
|
|
||||||
+START_TEST(keyboard_frame_order)
|
|
||||||
+{
|
|
||||||
+ struct litest_device *dev = litest_current_device();
|
|
||||||
+ struct libinput *li = dev->libinput;
|
|
||||||
+
|
|
||||||
+ if (!libevdev_has_event_code(dev->evdev, EV_KEY, KEY_A) ||
|
|
||||||
+ !libevdev_has_event_code(dev->evdev, EV_KEY, KEY_LEFTSHIFT))
|
|
||||||
+ return;
|
|
||||||
+
|
|
||||||
+ litest_drain_events(li);
|
|
||||||
+
|
|
||||||
+ litest_event(dev, EV_KEY, KEY_LEFTSHIFT, 1);
|
|
||||||
+ litest_event(dev, EV_KEY, KEY_A, 1);
|
|
||||||
+ litest_event(dev, EV_SYN, SYN_REPORT, 0);
|
|
||||||
+ libinput_dispatch(li);
|
|
||||||
+
|
|
||||||
+ litest_assert_key_event(li,
|
|
||||||
+ KEY_LEFTSHIFT,
|
|
||||||
+ LIBINPUT_KEY_STATE_PRESSED);
|
|
||||||
+ litest_assert_key_event(li, KEY_A, LIBINPUT_KEY_STATE_PRESSED);
|
|
||||||
+
|
|
||||||
+ litest_event(dev, EV_KEY, KEY_LEFTSHIFT, 0);
|
|
||||||
+ litest_event(dev, EV_KEY, KEY_A, 0);
|
|
||||||
+ litest_event(dev, EV_SYN, SYN_REPORT, 0);
|
|
||||||
+ libinput_dispatch(li);
|
|
||||||
+
|
|
||||||
+ litest_assert_key_event(li,
|
|
||||||
+ KEY_LEFTSHIFT,
|
|
||||||
+ LIBINPUT_KEY_STATE_RELEASED);
|
|
||||||
+ litest_assert_key_event(li, KEY_A, LIBINPUT_KEY_STATE_RELEASED);
|
|
||||||
+
|
|
||||||
+ litest_event(dev, EV_KEY, KEY_A, 1);
|
|
||||||
+ litest_event(dev, EV_KEY, KEY_LEFTSHIFT, 1);
|
|
||||||
+ litest_event(dev, EV_SYN, SYN_REPORT, 0);
|
|
||||||
+ libinput_dispatch(li);
|
|
||||||
+
|
|
||||||
+ litest_assert_key_event(li, KEY_A, LIBINPUT_KEY_STATE_PRESSED);
|
|
||||||
+ litest_assert_key_event(li,
|
|
||||||
+ KEY_LEFTSHIFT,
|
|
||||||
+ LIBINPUT_KEY_STATE_PRESSED);
|
|
||||||
+
|
|
||||||
+ litest_event(dev, EV_KEY, KEY_A, 0);
|
|
||||||
+ litest_event(dev, EV_KEY, KEY_LEFTSHIFT, 0);
|
|
||||||
+ litest_event(dev, EV_SYN, SYN_REPORT, 0);
|
|
||||||
+ libinput_dispatch(li);
|
|
||||||
+
|
|
||||||
+ litest_assert_key_event(li, KEY_A, LIBINPUT_KEY_STATE_RELEASED);
|
|
||||||
+ litest_assert_key_event(li,
|
|
||||||
+ KEY_LEFTSHIFT,
|
|
||||||
+ LIBINPUT_KEY_STATE_RELEASED);
|
|
||||||
+
|
|
||||||
+ libinput_dispatch(li);
|
|
||||||
+}
|
|
||||||
+END_TEST
|
|
||||||
+
|
|
||||||
START_TEST(keyboard_leds)
|
|
||||||
{
|
|
||||||
struct litest_device *dev = litest_current_device();
|
|
||||||
@@ -432,6 +487,7 @@ litest_setup_tests_keyboard(void)
|
|
||||||
litest_add("keyboard:time", keyboard_time_usec, LITEST_KEYS, LITEST_ANY);
|
|
||||||
|
|
||||||
litest_add("keyboard:events", keyboard_no_buttons, LITEST_KEYS, LITEST_ANY);
|
|
||||||
+ litest_add("keyboard:events", keyboard_frame_order, LITEST_KEYS, LITEST_ANY);
|
|
||||||
|
|
||||||
litest_add("keyboard:leds", keyboard_leds, LITEST_ANY, LITEST_ANY);
|
|
||||||
|
|
||||||
--
|
|
||||||
2.13.6
|
|
||||||
|
|
||||||
@ -4,8 +4,8 @@
|
|||||||
%global gitversion 58abea394
|
%global gitversion 58abea394
|
||||||
|
|
||||||
Name: libinput
|
Name: libinput
|
||||||
Version: 1.9.3
|
Version: 1.9.4
|
||||||
Release: 2%{?gitdate:.%{gitdate}git%{gitversion}}%{?dist}
|
Release: 1%{?gitdate:.%{gitdate}git%{gitversion}}%{?dist}
|
||||||
Summary: Input device library
|
Summary: Input device library
|
||||||
|
|
||||||
License: MIT
|
License: MIT
|
||||||
@ -19,7 +19,6 @@ Source0: http://www.freedesktop.org/software/libinput/libinput-%{version}
|
|||||||
%endif
|
%endif
|
||||||
|
|
||||||
Patch03: 0001-udev-add-integration-flag-for-the-Lenovo-Compact-Key.patch
|
Patch03: 0001-udev-add-integration-flag-for-the-Lenovo-Compact-Key.patch
|
||||||
Patch04: 0001-fallback-send-key-events-out-immediately-upon-receiv.patch
|
|
||||||
|
|
||||||
BuildRequires: git-core
|
BuildRequires: git-core
|
||||||
BuildRequires: gcc
|
BuildRequires: gcc
|
||||||
@ -122,6 +121,9 @@ git am -p1 %{patches} < /dev/null
|
|||||||
%{_mandir}/man1/libinput-measure-trackpoint-range.1*
|
%{_mandir}/man1/libinput-measure-trackpoint-range.1*
|
||||||
|
|
||||||
%changelog
|
%changelog
|
||||||
|
* Thu Dec 14 2017 Peter Hutterer <peter.hutterer@redhat.com> 1.9.4-1
|
||||||
|
- libinput 1.9.4
|
||||||
|
|
||||||
* Fri Dec 08 2017 Peter Hutterer <peter.hutterer@redhat.com> 1.9.3-2
|
* Fri Dec 08 2017 Peter Hutterer <peter.hutterer@redhat.com> 1.9.3-2
|
||||||
- Immediately post key events, don't wait for EV_SYN
|
- Immediately post key events, don't wait for EV_SYN
|
||||||
|
|
||||||
|
|||||||
2
sources
2
sources
@ -1 +1 @@
|
|||||||
SHA512 (libinput-1.9.3.tar.xz) = 33fb17724d4c2992bf31f9bb2e0825955468058e122ad420678b01ee293e1651d659fbae8873a7913f571243b3618032774cd350d2513eb73320d40aaa9953d1
|
SHA512 (libinput-1.9.4.tar.xz) = 302f9497ea9ffb2163c643e9ca2f0a773ea141f6fe0a3aa3d8e86eb11a5d11e75d858b1e679ebfd3f913c645beac059cfd356b37c4ea17a8853068f79a740a4b
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user