gitlab-ci: use qemu to run the full test suite

We currently allow the stage to fail, but at least,
we can make use of it.

Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>

Added quirk for Lenovo ThinkPad X200 Tablet.

Fixed horizontal scrolling on Logitech MX Master 2S/3

Logitech MX Master 2S and 3 by default use natural scrolling
for the horizontal scroll wheel, while the main wheel
uses traditional mode. This change inverts the default
direction of horizontal scrolling.

test: fix a coverity warning about an unused value

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tablet: Improve forced prox out behavior.

Some graphics tablets (most or all Wacom, for example) do not emit
proximity out events when the tablet pen goes out of range. To
compensate for this, libinput synthesizes proximity out events when no
events are received for a certain period of time. Unfortunately, on some
tablets, this is fairly failure prone when moving the pen slowly. As a
workaround, this patch causes libinput to avoid synthesizing proximity
out events when the pen is still in contact with the tablet pad, as
defined by the TABLET_TOOL_IN_CONTACT status.

middle-button: add space to middlebutton state debug message

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>`

test: Make udev_properties array fixed-size.

The udev_properties array is currently variable length, which causes the
tests to invoke undefined behavior on empty lists, as it attempts to
access the first array item to check if the key is NULL, which is an out
of bounds read and will fail when the struct alignment happens to line
up such that there is no padding after the list in the empty list case.

By making the udev_properties array 32 items long, it can encapsulate
every existing case, with only a fairly small amount of memory overhead,
and without requiring every single `TEST_DEVICE` call to initialize
`udev_properties`.

test: don't use debounced clicks for the middle button emulation click

This used to work under valgrind up to F30 but with the F31 beta something is
now a tad slower so it triggers the timeouts before the middle emulation kicks
in.

The middlebutton timeout is 50ms and the first debounce timeout is 30ms, so if
we're late by 20ms, well, there goes the timeout.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

doc/user: drop markdown source parsers

This is deprecated in sphinx 1.8 but we don't include any markdown sources
anyway, so let's just drop it.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

doc/user: update copyright for the sphinx docs

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

doc: update reference to the event codes header

The codes have been in input-event-codes.h for quite a few releases now.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gestures: where we have more fingers than slots, default to swipe

Fixes #360

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gestures: rename the inner/outer thresholds to min/max_move

inner/outer refer more to static thresholds when really what we have here is a
minimum movement before we look at the touch, and a maximum one after which
it's largely ignored.

Straight-up rename, no functional changes.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: only identify for pinch in a distinct pinch position

Previously, any lower finger spaced more than the vertical threshold apart
would be labelled as thumb. This causes some taps to be detected as
single-taps, particularly where the user's hand is at an angle that causes the
touches to be effectively vertical.

Restructure that condition so that we only go for a thumb if we're
distinctively apart, and we only *not* go for thumb if we're distinctively
close together.

Fixes #359

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: drop useless asserts()

Just above that line is an early return if either is NULL, so...

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Touchpad fix and trackpoint speed adjustment for Lenovo T490s

Same issue as #177

libinput-tool.c: remove unused includes

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

libinput-measure.c: remove unused includes

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

quirks.c: remove unused include

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

libinput-fuzz-to-zero.c: remove unused includes

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

libinput-fuzz-extract.c: remove unused includes

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

libinput-debug-events.c: remove unused includes

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

udev-seat.c: remove unused includes

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

path-seat.c: remove unused includes

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

evdev-mt-touchpad-buttons.c: remove unused includes

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

filter.h/c: remove unused includes

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

libinput-fuzz-override.c: remove unused includes

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

meson.build: bump to 1.14.900

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

timer.c: remove unused include <inttypes.h>

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

evdev-tablet-pad-leds.c: remove unused includes

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

evdev-tablet.c: remove unused include

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

evdev-mt-touchpad-gestures.c: remove unused include

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

evdev-mt-touchpad-edge-scroll.c: remove unused includes

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

evdev-mt-touchpad-tap.c: remove unused includes

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

evdev-mt-touchpad.h: remove unused include

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

evdev.c: remove unused include

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

filter-trackpoint.c: remove unused include

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

filter-touchpad-x230.c: remove unused includes

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

filter-touchpad.c: remove unused includes

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

filter-mouse.c: remove unused includes

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

filter-low-dpi.c: remove unused includes

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

filter-flat.c: remove unused includes

Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru>

udev: parse the EVDEV_ABS properties for a potential fuzz setting

Where a fuzz is defined in the 60-evdev.hwdb, we rely on a udev builtin to
set the kernel device to that fuzz value. Unfortunately that happens after our
program is called with this order of events:
1. 60-evdev.rules calls IMPORT(builtin) for the hwdb which sets the EVDEV_ABS_*
  properties. It also sets RUN{builtin}=keyboard but that's not invoked yet.
2. 90-libinput-fuzz-override.rules calls IMPORT{program} for our fuzz override
  bits. That sets the kernel fuzz value to 0 and sets the LIBINPUT_FUZZ_*
  propertie
3. The keyboard builtin is run once all the rules have been processed.

Our problem is that where the fuzz is set in a hwdb entry, the kernel fuzz is
still unset when we get to look at it, so we always end up with a fuzz of zero
for us and a nonzero kernel fuzz.

Work around this by checking the EVDEV_ABS property, extracting the fuzz from
there and re-printing that property without the fuzz. This way we ensure the
kernel remains at zero fuzz and we use the one from the hwdb instead.

Fixes #346

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Remove some usage of libinput-util.h

Positive side-effect - this exposed a bunch of missing #includes that got
pulled in by other headers before.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Split utility functions into separate source files

libinput-util.h is getting a bit of a catchall bucket and it includes things
like libinput-private.h which in turn includes libwacom. This makes
libinput-util.h less useful for bits that only need e.g. the string processing
utilities.

So let's split them all up in to separate files, to be used as-needed.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: do run the util tests under valgrind

These tests include string parsers, definitely want those to run under
valgrind to detect OOB reads and writes.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: flake8 fixes for the various python files

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: change python invocations to use /usr/bin/env python3

Let's be consistent here

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: install local quirks during libinput replay

The quirks for each device are listed in the recording but they may not apply
during libinput replay (e.g. for DMI matches). Work around this by writing out
the local-overrides.quirks file before initializing the devices. This way
we're guaranteed that the device is identical as on the reporter's machine.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: make an infinite loop more obvious in the code

Nothing sets "stop", so we might as well make it a while True

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: factor out a sanity check in libinput-replay

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Added magic trackpoint multiplier for Thinkpad 13 G2

Mark HP Elite x2 1013 G3 keyboard as external
Keyboard+touchpad is external:
http://h10032.www1.hp.com/ctg/Manual/c06077534 (page 12)

Tablet Mode Switch on HP Elite x2 1013 G3 is unreliable:
https://bugzilla.kernel.org/show_bug.cgi?id=204719
https://bugzilla.kernel.org/show_bug.cgi?id=204739

Keyboard+touchpad is external:
http://h10032.www1.hp.com/ctg/Manual/c06077534 (page 12)

libinput 1.14.1

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: don't allow gestures with a clickpad button down by a finger

Allowing gestures when holding a physical click enables tasks like
switching workspaces while dragging an icon, but this should only be
possible with a *thumb* holding down the clickpad, not fingers. This
commit restores the ability to hold down the clickpad with two or three
fingers to right- or middle-drag.

Fixes #339, #340

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: fix typo in debug-events

completion: add libinput(1) zsh completions

Fix double click button.

This fixes the double click button on the RollerMouse v3 (USB ID 0b33:1004).

libinput 1.14.0

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tablet: scale the available pressure range into the pressure thresholds

Pens that don't have a pressure offset (caused by a worn-out tip) still have
basic pressure thresholds to avoid tip events when we're still a bit away from
the tablet or barely touching it. That range is currently 5% of the pressure
for tip down, 1% for tip up.

This leaves us with 95% of the range and that needs to be scaled correctly,
otherwise the bottom 5% happen before a tip event and are inaccessible where
applications don't look at pressure before tip down.

Fixes #332

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: don't test at the 100 y range

Theoretically this shouldn't matter, but testing at the far end of the range
is bound to trigger some little issues eventually that should be triggered
explicitly, not by accident.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tablet: reduce the pressure range by the offset

Previously, the pressure range was calculated from the axis total range. A
device with a pressure offset making the bottom 10% inaccessible would lose
10% of that range as non-accessible. Due to the implementation, this affected
the upper range of the device, so the top N percent became unaccessible. Which
may be why no-one's noticed this yet.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tablet: make the pressure-offset inclusive of the axis minimum

The offset handling was inconsistent, stored as relative to the axis minimum
but used as absolute in some places. Fix this by always using the absolute
value including the minimum (i.e. no pressure offset means offset == minimum).

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: fix the pressure offset tests

Unsuprisingly, a normalized [0,1] value will always be between 0 and 1, so
bhis gave us a false positive. Check for the real values instead.

Those values aren't 100% correct because of a bug in the offset handling which
will be fixed in a follow-up commit. The difference is near enough that it
doesn't matter here anyway.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Add the ck_double_eq_tol() macros to the backwards compat headers

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tablet: add a comment explaining why we adjust the pressure offset downwards

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tablet: point the pressure offset log messages to the right URL

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tests: increase the timeout for the subprocess to receive the quit signal

On a very loaded machine, the process might not receive the quit signal
in a timely manner, and this introduce false positive results.

Add a longer timeout. This shouldn't interfere with the global time
spent in the tests, but will allow some loaded environment to pass
the tests.

Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>

gitlab-ci: force using docker format for the generated images

See https://gitlab.freedesktop.org/freedesktop/freedesktop/issues/178#note_203050

Some shared runners are not capable of understanding OCI format for
container images, and they are failing.

Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>

gitlab-ci: allow to run on unprivileged containers

This parameter is already included by default in ci-templates, but
we also need it in freebsd

Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>

gitlab CI: fetch the WAYLAND_WEB_TOKEN from a file

This way it can't leak into the meson testlog.txt during the other stages.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gitlab CI: replace the user:password with a netrc file

Gitlab supports masked tokens that get sanitized during log output but these
tokens are still in the environment. meson dumps the environment into
testlog.txt, resulting in our tokens leaking.

Avoid that leak by using a netrc file instead. The token value now refers to
the file name which is safe enough to leak into the test logs.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

evdev: always store user calibration matrix

In evdev_device_calibrate, the user matrix was not being stored when it
was the identity matrix. This resulted in
libinput_device_config_calibration_get_matrix not providing the correct
matrix. Instead of giving the identity matrix, the last non-identity
matrix set was given.

This just moves the storage of the user matrix in
evdev_device_calibrate to be above the identity matrix early return so
that it always get stored.

Signed-off-by: Brian Ashworth <bosrsf04@gmail.com>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

meson.build: drop explicit install:true from configure_file

meson implicitly sets install to whether install_dir is nonzero. Which means
it's superfluous anyway and removing it drops the meson warning:

WARNING: Project specifies a minimum meson_version '>= 0.41.0' but uses
features which were added in newer versions:
 * 0.50.0: {'install arg in configure_file'}

Fixes #334

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: record: fix two memory leaks

Found by coverity.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: record: fix segfault on exit

If we don't supply --with-libinput, the device is NULL so we can't unref it.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

libinput 1.13.902

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: mix tablet events into touch arbitration tests

We keep running into the proximity timeout for these tests, especially under
valgrind. To avoid this, manually intersperse the touch events with tablet
events.

Note that this manual loop would just work even without tablet events
because we no longer have a 10ms delay between touch events as enforced by
litest_touch_move_to. But let's do the right thing anyway.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

udev: Reproduce entire LIBINPUT_DEVICE_GROUP for paired ExpressKey Remote

In order for two devices to be in the same group, they need to share
identical LIBINPUT_DEVICE_GROUP attributes. The `wacom_handle_ekr` function
overwrites the VID/PID for an ExpressKey Remote, but the 'phys' path is
left unchanged. This only works if the EKR and the device we want to pair
it with are both direct sibings in the USB tree. It isn't always possible
to actually connect the devices like this, however. The Cintiq Pro 32 and
24, for instance, have multiple internal USB hubs and place the pen sensor
and the USB port for the EKR dongle behind different ones.

By copying the 'phys' path of the device we want to pair with, it is
possible to reproduce the entire LIBINPUT_DEVICE_GROUP and ensure that
the two devices actually end up paired in libinput.

Signed-off-by: Jason Gerecke <jason.gerecke@wacom.com>

Introduce Dell Latitude 5580 trackpoint multiplier

It is required, otherwise the trackpoint is too sensitive.

Tested with a Dell Latitude 5580.

Signed-off-by: Jacob Moroni <mail@jakemoroni.com>

tools: record: when running from the builddir, load the quirks correctly

Fixes #324

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: handle arguments correctly for the YAML verifier tool

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: skip the event tests for eventless recordings in the YAML verifier

When verifying a recording, let's skip those tests that require events but
don't have any.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

libinput 1.13.901

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Merge branch 'wip/advanced-thumb-detection-v2'

touchpad: stricter thumb detection if no pressure/size

touchpad: don't detect speed-based thumbs if there's already a thumb

doc/user: add documentation for the new thumb detection

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: revamp thumb detection

Instead of a simple yes/no/maybe for thumbs, have a more extensive state
machine that keeps track of the thumb. Since we only support one thumb anyway,
the tracking moves to the tp_dispatch struct.

Test case changes:
touchpad_clickfinger_3fg_tool_position:
  with better thumb detection we can now handle this properly and expect a
  right button (2fg) press for the test case
touchpad_thumb_no_doublethumb_with_timeout:
  two thumbs are now always two fingers, so let's switch to axis events here

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: make the touchpad size the only check for thumb detection

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: only run the speed finger tests when the touchpad has thumb detection

test: only run the speed tests for clickpads

We don't need speed detection for non-clickpads - the only reason to ever drop
a second finger on those is to either scroll or trigger a gesture. Unlike
clickpads, where a dropped finger may be a thumb to click.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: only log edge scroll state changes when the state actually changes

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: add a helper function for supressing a thumb

Only sets the state to YES at the moment, will do more in the future.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gestures: Improve thumb detection, allow one finger scroll

Check if there's a thumb if we have two touches. If one finger moves but
the thumb remains still, we assume that one is really a thumb. But if the
thumb moves while the finger is still, let's assume this is a 2-finger scroll.

Extracted from Matt Mayfield's thumb detection patchset

gestures: fast-track scroll/swipe detection when gestures are off

We can't detect pinch when gestures are off anyway, so we don't need to check
the finger distances.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

gestures: improvements to pinch detection

Only bias towards scrolling if the fingers are in the position past the
timeout.

gestures: improve scroll responsiveness for vertically aligned touches

Put some basic location checks in, if the fingers are next to each other and
vertically close, assume scroll over swipe.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>:

touchpad: basic thumb detection within gestures

When a touchpad has thumb detection enabled, avoid false-positive gestures
involving a resting thumb by using two thresholds: inner and outer.

While both touches remain inside their inner thresholds, remain in UNKNOWN
state to allow for accurate gesture detection even with no timeout.

If both touches move outside their inner thresholds, start a pinch or
swipe/scroll gesture according to direction, as usual.

If one touch moves outside its outer threshold while the other has not yet
exceeded its inner threshold, and thumb detection is enabled, then if one
touch is >20mm lower, mark it as a thumb and cancel the gesture.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>