X-Git-Url: https://iankelling.org/git/?a=blobdiff_plain;f=mount-latest-subvol;h=d7be613c763d122d0259b5a434ca4a161f599627;hb=0310db0c0589445a132e77f0b7a8fbb5e1ae3082;hp=d39b8823f89a1da652c035f320a513d1cf13dc45;hpb=7a82762b44f4732f6b075a0cb5d8b258f5022d12;p=distro-setup

diff --git a/mount-latest-subvol b/mount-latest-subvol
index d39b882..d7be613 100644
--- a/mount-latest-subvol
+++ b/mount-latest-subvol
@@ -81,6 +81,31 @@ fstab() {
         tu /etc/fstab <<<"$l"
     done
 }
+kill-dir() {
+    found_pids=false
+    sig=${1:-TERM}
+    if pids=$(timeout 4 lsof -t $dir); then
+        found_pids=true
+        timeout 4 lsof -w $dir
+        kill -$sig $pids
+    fi
+    # fuser will find open sockets that lsof won't, for example from gpg-agent.
+    # note: -v shows kernel processes, which then doesn't return true when we want
+    if timeout 4 fuser -m $dir &>/dev/null; then
+        found_pids=true
+        fuser -$sig -mvk $dir
+    fi
+    if $found_pids; then
+        sleep .5
+        return 0
+    fi
+    return 1
+}
+
+force=false
+if [[ $1 == -f ]]; then
+    force=true
+fi
 
 ret=0
 
@@ -92,10 +117,14 @@ EOF
 
 shopt -s nullglob
 
+# ssh and probably some other things care about parent directory
+# ownership, and ssh doesn\'t allow any group writable parent
+# directories, so we are forced to use a directory structure similar
+# to home directories
 f=(/mnt/root/btrbk/q.*)
 if [[ -e $f ]]; then
     fstab <<EOF
-$first_root_crypt  /q  btrfs  noatime,subvol=q  0 0
+$first_root_crypt  /q  btrfs  noatime,subvol=q,gid=1000  0 0
 /q/p  /p  none  bind  0 0
 EOF
 fi
@@ -163,18 +192,8 @@ for vol in q a o i; do
             if e umount -R $dir; then
                 unmounted+=($dir)
             else
-                if pids=$(timeout 4 lsof -t $dir); then
-                    timeout 4 lsof -w $dir
-                    kill $pids
-                fi
-
-                # fuser will find open sockets that lsof won't, for example from gpg-agent.
-                # note: -v shows kernel processes, which then doesn't return true when we want
-                if timeout 4 fuser -m $dir &>/dev/null; then
-                    fuser -TERM -mvk $dir
-                fi
+                kill-dir || kill-dir INT || kill-dir HUP || ! $force || kill-dir KILL ||:
 
-                sleep .5
                 if e umount -R $dir; then
                     unmounted+=($dir)
                 else