aboutsummaryrefslogtreecommitdiff
path: root/tools/usb
diff options
context:
space:
mode:
authorJason Gerecke2021-10-15 10:28:02 +0800
committerJiri Kosina2021-10-27 10:13:57 +0200
commitb7644592bd0d78cf7aba377124c2d3082607685b (patch)
tree4607774c98b5cf55f28c8fb263d345931978cb41 /tools/usb
parent42d43c92fc577dca59ed74aec7868abec8d6ca6e (diff)
HID: wacom: Shrink critical section in `wacom_add_shared_data`
The size of the critical section in this function appears to be larger than necessary. The `wacom_udev_list_lock` exists to ensure that one interface cannot begin checking if a shared object exists while a second interface is doing the same (otherwise both could determine that no object exists yet and create their own independent objects rather than sharing just one). It should be safe for the critical section to end once a fresly-allocated shared object would be found by other threads (i.e., once it has been added to `wacom_udev_list`, which is looped over by `wacom_get_hdev_data`). This commit is a necessary pre-requisite for a later change to swap the use of `devm_add_action` with `devm_add_action_or_reset`, which would otherwise deadlock in its error case. Signed-off-by: Jason Gerecke <jason.gerecke@wacom.com> Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'tools/usb')
0 files changed, 0 insertions, 0 deletions