diff options
| author | Kay Sievers <kay@vrfy.org> | 2014-06-03 16:49:38 +0200 |
|---|---|---|
| committer | Kay Sievers <kay@vrfy.org> | 2014-06-03 16:49:38 +0200 |
| commit | e918a1b5a94f270186dca59156354acd2a596494 (patch) | |
| tree | 4b3d32fe3e05d729b803fc2e24efa765cba1686c | |
| parent | e5999b46792b859043a62431d426a280f26a16d7 (diff) | |
udev: exclude device-mapper from block device ownership event locking
| -rw-r--r-- | src/udev/udevd.c | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/src/udev/udevd.c b/src/udev/udevd.c index 819ea3bb2..6c0510497 100644 --- a/src/udev/udevd.c +++ b/src/udev/udevd.c @@ -290,7 +290,19 @@ static void worker_new(struct event *event) * acquired the lock, the external process will block until * udev has finished its event handling. */ - if (streq_ptr("block", udev_device_get_subsystem(dev))) { + + /* + * <kabi_> since we make check - device seems unused - we try + * ioctl to deactivate - and device is found to be opened + * <kay> sure, you try to take a write lock + * <kay> if you get it udev is out + * <kay> if you can't get it, udev is busy + * <kabi_> we cannot deactivate openned device (as it is in-use) + * <kay> maybe we should just exclude dm from that thing entirely + * <kabi_> IMHO this sounds like a good plan for this moment + */ + if (streq_ptr("block", udev_device_get_subsystem(dev)) && + !startswith("dm-", udev_device_get_sysname(dev))) { struct udev_device *d = dev; if (streq_ptr("partition", udev_device_get_devtype(d))) |