]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
device: skip deserialization of device units when udevd is not running 8675/head
authorFranck Bui <fbui@suse.com>
Tue, 17 Apr 2018 13:12:06 +0000 (15:12 +0200)
committerFranck Bui <fbui@suse.com>
Fri, 20 Apr 2018 15:49:28 +0000 (17:49 +0200)
Do not try to party initialize a device during deserialization if it's not
known by udev (anymore) and therefore hasn't been seen during device
enumeration.

The device unit in this case has not been initialized properly and setting it
in the "plugged" state can be confusing.

Actually this happens during every boots when PID switches to the new rootfs:
PID is reexecuted and enumerates devices but since udev is not running, the
list of enumerated devices is empty.

src/core/device.c

index aafc97ffd4d3ccd46cc5e2abf9e892e2d4d4b410..12dccdb975c769abda651a7c12022e641eaa9bf6 100644 (file)
@@ -173,6 +173,18 @@ static int device_deserialize_item(Unit *u, const char *key, const char *value,
         assert(value);
         assert(fds);
 
+        /* The device was known at the time units were serialized but it's not
+         * anymore at the time units are deserialized. This happens when PID1 is
+         * re-executed after having switched to the new rootfs: devices were
+         * enumerated but udevd wasn't running yet thus the list of devices
+         * (handled by systemd) to initialize was empty. In such case we wait
+         * for the device events to be re-triggered by udev so device units are
+         * properly re-initialized. */
+        if (d->found == DEVICE_NOT_FOUND) {
+                assert(d->sysfs == NULL);
+                return 0;
+        }
+
         if (streq(key, "state")) {
                 DeviceState state;