]> git.ipfire.org Git - thirdparty/linux.git/commit
gpio: cdev: use raw notifier for line state events
authorBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Tue, 11 Mar 2025 14:31:43 +0000 (15:31 +0100)
committerBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Thu, 13 Mar 2025 08:20:12 +0000 (09:20 +0100)
commitdcb73cbaaeb39c9fd00bf2e019f911725945e2fe
treeb1fb2414a1968a78f1d72d6d69379a803bd32909
parent0102fbf52b93e609fec0dab53b1fb4fe69113f5e
gpio: cdev: use raw notifier for line state events

We use a notifier to implement the mechanism of informing the user-space
about changes in GPIO line status. We register with the notifier when
the GPIO character device file is opened and unregister when the last
reference to the associated file descriptor is dropped.

Since commit fcc8b637c542 ("gpiolib: switch the line state notifier to
atomic") we use the atomic notifier variant. Atomic notifiers call
rcu_synchronize in atomic_notifier_chain_unregister() which caused a
significant performance regression in some circumstances, observed by
user-space when calling close() on the GPIO device file descriptor.

Replace the atomic notifier with the raw variant and provide
synchronization with a read-write spinlock.

Fixes: fcc8b637c542 ("gpiolib: switch the line state notifier to atomic")
Reported-by: David Jander <david@protonic.nl>
Closes: https://lore.kernel.org/all/20250311110034.53959031@erd003.prtnl/
Tested-by: David Jander <david@protonic.nl>
Tested-by: Kent Gibson <warthog618@gmail.com>
Link: https://lore.kernel.org/r/20250311-gpiolib-line-state-raw-notifier-v2-1-138374581e1e@linaro.org
Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
drivers/gpio/gpiolib-cdev.c
drivers/gpio/gpiolib.c
drivers/gpio/gpiolib.h