linux-kernel - Re: Spurious touchpad events with closed LID

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20170626170312.GB4965@dtor-ws>
Date:   Mon, 26 Jun 2017 10:03:12 -0700
From:   Dmitry Torokhov <dmitry.torokhov@...il.com>
To:     Pali Rohár <pali.rohar@...il.com>
Cc:     linux-kernel@...r.kernel.org, linux-input@...r.kernel.org
Subject: Re: Spurious touchpad events with closed LID

On Mon, Jun 26, 2017 at 06:54:53PM +0200, Pali Rohár wrote:
> Hi!
> 
> When I'm using dock with external input devices (keyboard + mouse) and 
> LID is closed, I'm getting spurious touchpad events and random mouse 
> clicks and movements.
> 
> It is because top part of LID is above touchpad and probably generates 
> touch pushes.
> 
> Year (or two?) when I had conversation with ALPS people I was told that 
> Windows driver is automatically turning touchpad off when ACPI LID close 
> event is received (and similarly turn touchpad on).
> 
> Maybe Linux should do similar thing? Random movement or touchpad clicks 
> is really annoying. But I'm not sure if kernel or userspace should do 
> this job... What do you think?

It is a matter of policy (deciding when device is "usable") and this
should be controlled from userspace. Kernel should provide necessary
knobs for it though. For a long time I was saying that it should be done
at device core level, but I do not think we will ever get there.

On ChromeOS input devices export "inhibit" attribute that basically
overrides open/close count and prevents delivery of events to userspace,
and power management driver controls this attribute together with wake
up and others. This allows us to implement policies like "the touchpad
should only be active and a wakeup source while the device is in laptop
mode, but not in tablet or tent mode, or when lid is closed", "disable
keyboard in tablet mode or when list is closed", etc.

Drivers can also implement inhibit/uninhibit methods that can put
inhibited devices into lower power mode. I am not too happy with the
implementation (I'd like to see if we could merge inhibit/uninhibit and
open/close), that is why I haven't pushed it upstream yet.

Thanks.

-- 
Dmitry