[PATCH v3 00/13] gpiolib: add an ioctl() for monitoring line status changes

From: Bartosz Golaszewski
Date: Thu Dec 19 2019 - 12:15:38 EST


From: Bartosz Golaszewski <bgolaszewski@xxxxxxxxxxxx>

When discussing the recent user-space changes with Kent and while working
on dbus API for libgpiod I noticed that we really don't have any way of
keeping the line info synchronized between the kernel and user-space
processes. We can of course periodically re-read the line information or
even do it every time we want to read a property but this isn't optimal.

This series adds a new ioctl() that allows user-space to set up a watch on
the GPIO chardev file-descriptor which can then be polled for events
emitted by the kernel when the line is requested, released or its status
changed. This of course doesn't require the line to be requested. Multiple
user-space processes can watch the same lines.

This series also includes a variety of minor tweaks & fixes for problems
discovered during development. For instance it addresses a race-condition
in current line event fifo.

v1: https://lkml.org/lkml/2019/11/27/327

v1 -> v2:
- rework the main patch of the series: re-use the existing file-descriptor
associated with an open character device
- add a patch adding a debug message when the line event kfifo is full and
we're discarding another event
- rework the locking mechanism for lineevent kfifo: reuse the spinlock
from the waitqueue structure
- other minor changes

v2 -> v3:
- added patches providing new implementation for some kfifo macros
- fixed a regression in the patch reworking the line event fifo: reading
multiple events is now still possible
- reworked the structure for new ioctl: it's now padded such that there
be no alignment issues if running a 64-bit kernel on 32-bit userspace
- fixed a bug where one process could disable the status watch of another
- use kstrtoul() instead of atoi() in gpio-watch for string validation

Bartosz Golaszewski (13):
gpiolib: use 'unsigned int' instead of 'unsigned' in gpio_set_config()
gpiolib: have a single place of calling set_config()
gpiolib: convert the type of hwnum to unsigned int in
gpiochip_get_desc()
gpiolib: use gpiochip_get_desc() in linehandle_create()
gpiolib: use gpiochip_get_desc() in lineevent_create()
gpiolib: use gpiochip_get_desc() in gpio_ioctl()
kfifo: provide noirqsave variants of spinlocked in and out helpers
kfifo: provide kfifo_is_empty_spinlocked()
gpiolib: rework the locking mechanism for lineevent kfifo
gpiolib: emit a debug message when adding events to a full kfifo
gpiolib: provide a dedicated function for setting lineinfo
gpiolib: add new ioctl() for monitoring changes in line info
tools: gpio: implement gpio-watch

drivers/gpio/gpiolib.c | 402 +++++++++++++++++++++++++++---------
drivers/gpio/gpiolib.h | 4 +-
include/linux/gpio/driver.h | 3 +-
include/linux/kfifo.h | 73 +++++++
include/uapi/linux/gpio.h | 24 +++
tools/gpio/.gitignore | 1 +
tools/gpio/Build | 1 +
tools/gpio/Makefile | 11 +-
tools/gpio/gpio-watch | Bin 0 -> 26528 bytes
tools/gpio/gpio-watch.c | 99 +++++++++
10 files changed, 514 insertions(+), 104 deletions(-)
create mode 100755 tools/gpio/gpio-watch
create mode 100644 tools/gpio/gpio-watch.c

--
2.23.0