Bug 392754

Summary:	After I wake up a touch screen via double tap, its power management stops working until some input device is used
Product:	[Plasma] kwin	Reporter:	Patrick Silva <bugseforuns>
Component:	platform-drm	Assignee:	KWin default assignee <kwin-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	Flags:	mgraesslin: Wayland+ mgraesslin: X11- mgraesslin: ReviewRequest+
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Neon
OS:	Linux
URL:	https://phabricator.kde.org/D12668
Latest Commit:	https://commits.kde.org/kwin/69afe4d266ffbde364913d31fbcbc00769b4b390	Version Fixed/Implemented In:	5.12.6
Sentry Crash Report:

Description Patrick Silva 2018-04-05 12:07:57 UTC

reproducible on neon dev unstable

start wayland session
wait your system to reach screen power management timeout
after your screen to turn off, wake up it doing double tap
do NOT use any input device (touch screen, touchpad, mouse, keyboard) and wait for screen power management timeout again:
your screen remains on when the timeout is reached
move your cursor, press any key or tap your screen, wait for the timeout again: now your screen turns off

Comment 1 Martin Flöser 2018-05-02 18:54:11 UTC

Possible patch at https://phabricator.kde.org/D12668

Any chance to test? The issue and the fix look straight forward to me.

Comment 2 Martin Flöser 2018-05-20 13:40:52 UTC

Git commit 69afe4d266ffbde364913d31fbcbc00769b4b390 by Martin Flöser.
Committed on 20/05/2018 at 13:38.
Pushed by graesslin into branch 'Plasma/5.12'.

Update seat's timestamp after waking up screen through double tap

Summary:
The drm platform has a special handling for touch events while the
screen is in dpms. All touch events are filtered out, but a double
tap wakes up the screen.

Due to the filtering out of the events the idle timers were not reset.
So if you only double tapped, the screen turned on, but not off again.

This change updates the timestamp after double tap, so that the idle
timers are restarted. All other events are still filtered out, to not
have "fake" events (e.g. smart phone in pocket) wake up the device.
FIXED-IN: 5.12.6

Test Plan: only compile tested

Reviewers: #kwin, #plasma

Subscribers: kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D12668

M  +4    -1    plugins/platforms/drm/drm_inputeventfilter.cpp

https://commits.kde.org/kwin/69afe4d266ffbde364913d31fbcbc00769b4b390