diff options
author | Lennart Poettering <lennart@poettering.net> | 2019-12-20 18:37:24 +0100 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2019-12-20 18:37:24 +0100 |
commit | 4ca8072fd6195c0d8e1c0da24db6b67a97070260 (patch) | |
tree | 38f426ec4f3b80abc6c23218b2aab7c88379e1d5 /src/shutdown | |
parent | umount: check LO_FLAGS_AUTOCLEAR after LOOP_CLR_FD claimed success (diff) | |
download | systemd-4ca8072fd6195c0d8e1c0da24db6b67a97070260.tar.xz systemd-4ca8072fd6195c0d8e1c0da24db6b67a97070260.zip |
umount: when we fail to detach a loopback device, set the auto-clear flag
We might get lucky and this cleans up things later on automatically for
us.
Diffstat (limited to 'src/shutdown')
-rw-r--r-- | src/shutdown/umount.c | 25 |
1 files changed, 24 insertions, 1 deletions
diff --git a/src/shutdown/umount.c b/src/shutdown/umount.c index cc86f4180e..8a5e80eeaa 100644 --- a/src/shutdown/umount.c +++ b/src/shutdown/umount.c @@ -340,7 +340,30 @@ static int delete_loopback(const char *device) { if (errno == ENXIO) /* Nothing bound, didn't do anything */ return 0; - return -errno; + if (errno != EBUSY) + return log_debug_errno(errno, "Failed to clear loopback device %s: %m", device); + + if (ioctl(fd, LOOP_GET_STATUS64, &info) < 0) { + if (errno == ENXIO) /* What? Suddenly detached after all? That's fine by us then. */ + return 1; + + log_debug_errno(errno, "Failed to invoke LOOP_GET_STATUS64 on loopback device %s, ignoring: %m", device); + return -EBUSY; /* propagate original error */ + } + + if (FLAGS_SET(info.lo_flags, LO_FLAGS_AUTOCLEAR)) /* someone else already set LO_FLAGS_AUTOCLEAR for us? fine by us */ + return -EBUSY; /* propagate original error */ + + info.lo_flags |= LO_FLAGS_AUTOCLEAR; + if (ioctl(fd, LOOP_SET_STATUS64, &info) < 0) { + if (errno == ENXIO) /* Suddenly detached after all? Fine by us */ + return 1; + + log_debug_errno(errno, "Failed to set LO_FLAGS_AUTOCLEAR flag for loop device %s, ignoring: %m", device); + } else + log_debug("Successfully set LO_FLAGS_AUTOCLEAR flag for loop device %s.", device); + + return -EBUSY; } if (ioctl(fd, LOOP_GET_STATUS64, &info) < 0) { |