Btrfs: Wait for uuid-tree rebuild task on remount read-only
authorStefan Behrens <sbehrens@giantdisaster.de>
Fri, 11 Oct 2013 15:14:58 +0000 (17:14 +0200)
committerChris Mason <chris.mason@fusionio.com>
Tue, 12 Nov 2013 03:01:18 +0000 (22:01 -0500)
If the user remounts the filesystem read-only while the uuid-tree
scan and rebuild task is still running (this happens once after the
filesystem was mounted with an old kernel, or when forced with the
mount options), the remount should wait on the tasks completion
before setting the filesystem read-only. Otherwise the background
task continues to write to the filesystem which is apparently not
what users expect.

The reproducer:

TEST_DEV=/dev/sdzzzzz1
TEST_MNT=/mnt
mkfs.btrfs -f $TEST_DEV
mount $TEST_DEV $TEST_MNT
for i in `seq 50000`; do btrfs subvolume create ${TEST_MNT}/$i; done
umount $TEST_MNT
mount $TEST_DEV $TEST_MNT -o rescan_uuid_tree
sleep 1
ps -elf | fgrep '[btrfs-uuid]' | grep -v grep
mount $TEST_DEV $TEST_MNT -o ro,remount
ps -elf | fgrep '[btrfs-uuid]' | grep -v grep
sleep 1
umount $TEST_MNT

Signed-off-by: Stefan Behrens <sbehrens@giantdisaster.de>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
fs/btrfs/super.c

index 1f62c1ccab1fef0a8780df61337444c4b51aa326..02f552f11829148e8f681a05dd288bf7331f5bc2 100644 (file)
@@ -1330,6 +1330,12 @@ static int btrfs_remount(struct super_block *sb, int *flags, char *data)
                 * this also happens on 'umount -rf' or on shutdown, when
                 * the filesystem is busy.
                 */
+
+               /* wait for the uuid_scan task to finish */
+               down(&fs_info->uuid_tree_rescan_sem);
+               /* avoid complains from lockdep et al. */
+               up(&fs_info->uuid_tree_rescan_sem);
+
                sb->s_flags |= MS_RDONLY;
 
                btrfs_dev_replace_suspend_for_unmount(fs_info);