[PATCH] Input: cyapa - do not call input_device_enabled from power mode handler
dmitry.torokhov at gmail.com
Fri Dec 11 03:31:16 EST 2020
On Fri, Dec 11, 2020 at 09:22:44AM +0100, Marek Szyprowski wrote:
> On 11.12.2020 08:09, Dmitry Torokhov wrote:
> > Input device's user counter is supposed to be accessed only while holding
> > input->mutex. Commit d69f0a43c677 ("Input: use input_device_enabled()")
> > recently switched cyapa to using the dedicated API and it uncovered the
> > fact that cyapa driver violated this constraint.
> > This patch removes checks whether the input device is open when clearing
> > device queues when changing device's power mode as there is no harm in
> > sending input events through closed input device - the events will simply
> > be dropped by the input core.
> > Note that there are more places in cyapa driver that call
> > input_device_enabled() without holding input->mutex, those are left
> > unfixed for now.
> > Reported-by: Marek Szyprowski <m.szyprowski at samsung.com>
> > Signed-off-by: Dmitry Torokhov <dmitry.torokhov at gmail.com>
> > ---
> > Marek, could you please try this one?
> The warning is still there:
Ah, yes, we are hitting another instance right after setting power mode.
I need to think more how to handle that one.
More information about the linux-arm-kernel