From 9daec33938676c7439eecefe4c29d0891dd386ba Mon Sep 17 00:00:00 2001 From: Yu Watanabe Date: Tue, 18 Jul 2023 15:55:12 +0900 Subject: [PATCH] udev-util: drop udev_queue_init() from shared It is only used in libudev, let's move it. --- src/libudev/libudev-queue.c | 13 ++++++++----- src/shared/udev-util.c | 14 -------------- src/shared/udev-util.h | 1 - 3 files changed, 8 insertions(+), 20 deletions(-) diff --git a/src/libudev/libudev-queue.c b/src/libudev/libudev-queue.c index 7c5bb5500c6..0af99e5481b 100644 --- a/src/libudev/libudev-queue.c +++ b/src/libudev/libudev-queue.c @@ -192,18 +192,21 @@ _public_ struct udev_list_entry *udev_queue_get_queued_list_entry(struct udev_qu * Returns: a file descriptor to watch for a queue to become empty. */ _public_ int udev_queue_get_fd(struct udev_queue *udev_queue) { - int r; + _cleanup_close_ int fd = -EBADF; assert_return(udev_queue, -EINVAL); if (udev_queue->fd >= 0) return udev_queue->fd; - r = udev_queue_init(); - if (r < 0) - return r; + fd = inotify_init1(IN_CLOEXEC); + if (fd < 0) + return -errno; - return udev_queue->fd = r; + if (inotify_add_watch(fd, "/run/udev" , IN_DELETE) < 0) + return -errno; + + return udev_queue->fd = TAKE_FD(fd); } /** diff --git a/src/shared/udev-util.c b/src/shared/udev-util.c index 3e81234dc1f..fddf096d76a 100644 --- a/src/shared/udev-util.c +++ b/src/shared/udev-util.c @@ -2,7 +2,6 @@ #include #include -#include #include #include "alloc-util.h" @@ -578,19 +577,6 @@ int udev_queue_is_empty(void) { (errno == ENOENT ? true : -errno) : false; } -int udev_queue_init(void) { - _cleanup_close_ int fd = -EBADF; - - fd = inotify_init1(IN_CLOEXEC); - if (fd < 0) - return -errno; - - if (inotify_add_watch(fd, "/run/udev" , IN_DELETE) < 0) - return -errno; - - return TAKE_FD(fd); -} - bool udev_available(void) { static int cache = -1; diff --git a/src/shared/udev-util.h b/src/shared/udev-util.h index 175fa0b6f08..05a0849934b 100644 --- a/src/shared/udev-util.h +++ b/src/shared/udev-util.h @@ -53,7 +53,6 @@ int udev_resolve_subsys_kernel(const char *string, char *result, size_t maxsize, bool devpath_conflict(const char *a, const char *b); int udev_queue_is_empty(void); -int udev_queue_init(void); bool udev_available(void);