mount: try lazy unmount if normal one fails
authorRafał Miłecki <rafal@milecki.pl>
Wed, 21 Feb 2018 12:32:16 +0000 (13:32 +0100)
committerRafał Miłecki <rafal@milecki.pl>
Mon, 26 Feb 2018 05:11:58 +0000 (06:11 +0100)
If umount call fails (e.g. because of some application accessing old
path) the only chance of getting mount entry cleaned is receiving autofs
expire packet. This isn't fully reliable:
1) Sometimes expire can happen before closing all handles. In such case
   a second unmount try will fail as well and there won't be a next one.
2) Expire happens after specified timeout. If device reappears quickly
   (e.g. due to being replugged or USB controller restart) it may be
   worth having old mount cleaned up earlier for better user experience.

This problem can be easily handled by using MNT_DETACH (lazy) umount as
a fallback.

Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
mount.c

diff --git a/mount.c b/mount.c
index ddcd5b303fb9766191adbd3147debce0ced71881..3994149bf12a4b4d3a46170f8009948b1b5ad63b 100644 (file)
--- a/mount.c
+++ b/mount.c
@@ -604,6 +604,7 @@ static int mount_dev_del(struct mount *mount)
                log_printf("device %s has disappeared ... unmounting %s\n", mount->dev, tmp);
                if (umount(tmp)) {
                        err = -errno;
+                       umount2(tmp, MNT_DETACH);
                }
                rmdir(tmp);
                mount_dump_uci_state();