Revert "block: mount_action: handle mount/umount deps"
authorRafał Miłecki <rafal@milecki.pl>
Thu, 2 Apr 2020 12:12:12 +0000 (14:12 +0200)
committerRafał Miłecki <rafal@milecki.pl>
Sat, 2 May 2020 10:59:34 +0000 (12:59 +0200)
commit0b93429329e9f403cf4fc45043b73e3a8d4e8e0d
treeb2078be5b3bf0cf359ff44494140c5f5bd4ff352
parentc80f7002114f65dc40895c9064c077d9631b8f6e
Revert "block: mount_action: handle mount/umount deps"

This reverts commit 32c3126b2f0464106d74317336b6aef1d7d5f82f.

Internally stored list of devices is implemented using struct vlist_tree
and keeps devices sorted by their mount target paths. This DOESN'T mean
that all preceding entires of a given device are its parent devices.
Making such an assumption and mounting all preceding devices results in
unwanted mounts in most cases.

For example having devices like:
/dev/sda1 (mount target: /mnt/foo1)
/dev/sda2 (mount target: /mnt/foo2)
/dev/sdb1 (mount target: /mnt/bar1)
/dev/sdb2 (mount target: /mnt/bar2)
will result in devices vlist_tree having entries sorted like:
/dev/sdb1 (bar1), /dev/sdb2 (bar2), /dev/sda1 (foo1), /dev/sda2 (foo2)

Using autofs and accessing /mnt/foo2 would result in mounting all 4
partitions and spinning unneeded /dev/sdb.

Cc: Yousong Zhou <yszhou4tech@gmail.com>
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
block.c