From: Yu Watanabe <watanabe.yu+github@gmail.com>
Date: Sun, 12 Sep 2021 06:49:22 +0000 (+0900)
Subject: udev-watch: retry to save watch handle with random delay
X-Git-Tag: v250-rc1~686^2
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=2d3af41f0e837390b734253f5c4a99a9f33c53e3;p=thirdparty%2Fsystemd.git

udev-watch: retry to save watch handle with random delay

Also, remove the watch handle if we cannot save it.
---

diff --git a/src/udev/udev-watch.c b/src/udev/udev-watch.c
index 6032dfb4f6b..87c4f09928b 100644
--- a/src/udev/udev-watch.c
+++ b/src/udev/udev-watch.c
@@ -12,8 +12,13 @@
 #include "dirent-util.h"
 #include "fs-util.h"
 #include "parse-util.h"
+#include "random-util.h"
 #include "udev-watch.h"
 
+#define SAVE_WATCH_HANDLE_MAX_RETRIES  128
+#define MAX_RANDOM_DELAY (100 * USEC_PER_MSEC)
+#define MIN_RANDOM_DELAY ( 10 * USEC_PER_MSEC)
+
 int udev_watch_restore(int inotify_fd) {
         struct dirent *ent;
         DIR *dir;
@@ -93,11 +98,43 @@ int udev_watch_begin(int inotify_fd, sd_device *dev) {
                 return ignore ? 0 : r;
         }
 
-        r = device_set_watch_handle(dev, wd);
-        if (r < 0)
-                return log_device_warning_errno(dev, r, "Failed to save watch handle in /run/udev/watch: %m");
+        for (unsigned i = 0; i < SAVE_WATCH_HANDLE_MAX_RETRIES; i++) {
+                if (i > 0) {
+                        usec_t delay = MIN_RANDOM_DELAY + random_u64_range(MAX_RANDOM_DELAY - MIN_RANDOM_DELAY);
+
+                        /* When the same handle is reused for different device node, we may fail to
+                         * save the watch handle with -EEXIST. Let's consider the case of two workers A
+                         * and B do the following:
+                         *
+                         * 1. A calls inotify_rm_watch()
+                         * 2. B calls inotify_add_watch()
+                         * 3. B calls device_set_watch_handle()
+                         * 4. A calls device_set_watch_handle(-1)
+                         *
+                         * At step 3, the old symlinks to save the watch handle still exist. So,
+                         * device_set_watch_handle() fails with -EEXIST. */
+
+                        log_device_debug_errno(dev, r,
+                                               "Failed to save watch handle '%i' for %s in "
+                                               "/run/udev/watch, retrying in after %s: %m",
+                                               wd, devnode, FORMAT_TIMESPAN(delay, USEC_PER_MSEC));
+                        (void) usleep(delay);
+                }
 
-        return 0;
+                r = device_set_watch_handle(dev, wd);
+                if (r >= 0)
+                        return 0;
+                if (r != -EEXIST)
+                        break;
+        }
+
+        log_device_warning_errno(dev, r,
+                                 "Failed to save watch handle '%i' for %s in /run/udev/watch: %m",
+                                 wd, devnode);
+
+        (void) inotify_rm_watch(inotify_fd, wd);
+
+        return r;
 }
 
 int udev_watch_end(int inotify_fd, sd_device *dev) {