Btrfs: remove unnecessary ->s_umount in cleaner_kthread()
authorMiao Xie <miaox@cn.fujitsu.com>
Tue, 14 May 2013 10:20:40 +0000 (10:20 +0000)
committerJosef Bacik <jbacik@fusionio.com>
Fri, 14 Jun 2013 15:29:32 +0000 (11:29 -0400)
commitd027824564c5fcee19109530b87c94c9908e910a
tree9ee6f6b7c9eaeee8494536061f868e732d607198
parent3c64a1aba7cfcb04f79e76f859b3d66660275d59
Btrfs: remove unnecessary ->s_umount in cleaner_kthread()

In order to avoid the R/O remount, we acquired ->s_umount lock during
we deleted the dead snapshots and subvolumes. But it is unnecessary,
because we have cleaner_mutex.

We use cleaner_mutex to protect the process of the dead snapshots/subvolumes
deletion. And when we remount the fs to be R/O, we also acquire this mutex to
do cleanup after we change the status of the fs. That is this lock can serialize
the above operations, the cleaner can be aware of the status of the fs, and if
the cleaner is deleting the dead snapshots/subvolumes, the remount task will
wait for it. So it is safe to remove ->s_umount in cleaner_kthread().

Cc: David Sterba <dsterba@suse.cz>
Signed-off-by: Miao Xie <miaox@cn.fujitsu.com>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
fs/btrfs/disk-io.c