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