Re: [PATCH] HID: wacom: Make use of the helper function devm_add_action_or_reset()

From: Jiri Kosina
Date: Wed Oct 27 2021 - 04:15:10 EST


On Tue, 26 Oct 2021, Jason Gerecke wrote:

> Following up on this. I took a second look at the shared struct, and
> believe that things should work fine during initialization and
> steady-state. There are, however, opportunities for e.g. one
> device/thread to be removed and set e.g. `shared->touch = NULL` while a
> second device/thread is attempting to send an event out of that device.
> This is going to be very rare and only on disconnect, which is probably
> why we've never received reports of real-world issues.
>
> This shared issue is present with or without the changes by Cai and
> myself. I would ask that these two patches be merged

Now applied. Thanks,

--
Jiri Kosina
SUSE Labs