]> git.ipfire.org Git - thirdparty/util-linux.git/commitdiff
lib/loopdev: open loop control device read-only if possible
authorThomas Weißschuh <thomas.weissschuh@linutronix.de>
Wed, 16 Apr 2025 09:55:31 +0000 (11:55 +0200)
committerThomas Weißschuh <thomas.weissschuh@linutronix.de>
Wed, 16 Apr 2025 09:55:31 +0000 (11:55 +0200)
Opening the control device read-write to trigger udev events is a hack.
It works around a kernel bug which fails to send uevents directly from the
kernel by triggering a full udev reprobe.
While it works to trigger the events, it is problematic.
The udev reprobe will asynchronously recreate all partition devices which
requires an ungly locking and synchronization scheme after losetup has
completed to guarantee the usability of the partition devices.

The kernel has been fixed to properly emit these uevents in
commit 7ed2a771b5fb ("loop: properly send KOBJ_CHANGED uevent for disk device").

Drop the workaround.

This is a partial revert of
commit ced1142d6f25 ("losetup: cleanup device node modes").

Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
lib/loopdev.c

index f2931552c665c374245a660a472c489c1ab7b116..65efeb9ea1fc92683ff697db78f592f4115ea6f9 100644 (file)
@@ -1418,11 +1418,7 @@ int loopcxt_setup_device(struct loopdev_cxt *lc)
        do {
                errno = 0;
 
-               /* For the ioctls, it's enough to use O_RDONLY for a read-only
-                * loop device, but udevd monitor devices by inotify, and udevd
-                * needs IN_CLOSE_WRITE event to trigger probing of the new device.
-                */
-               dev_fd = __loopcxt_get_fd(lc, O_RDWR);
+               dev_fd = __loopcxt_get_fd(lc, mode);
                if (dev_fd >= 0 || lc->control_ok == 0)
                        break;
                if (errno != EACCES && errno != ENOENT)