diff options
author | Al Viro <viro@zeniv.linux.org.uk> | 2009-05-05 23:48:50 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2009-06-11 21:36:07 -0400 |
commit | 443b94baaa16771e98b29ca7c24f1e305738ffca (patch) | |
tree | cf498a25d012330928e0315222c46ea90c200530 | |
parent | 5af7926ff33b68b3ba46531471c6e0564b285efc (diff) | |
download | op-kernel-dev-443b94baaa16771e98b29ca7c24f1e305738ffca.zip op-kernel-dev-443b94baaa16771e98b29ca7c24f1e305738ffca.tar.gz |
Make sure that all callers of remount hold s_umount exclusive
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
-rw-r--r-- | fs/super.c | 5 |
1 files changed, 3 insertions, 2 deletions
@@ -579,7 +579,7 @@ static void do_emergency_remount(struct work_struct *work) list_for_each_entry(sb, &super_blocks, s_list) { sb->s_count++; spin_unlock(&sb_lock); - down_read(&sb->s_umount); + down_write(&sb->s_umount); if (sb->s_root && sb->s_bdev && !(sb->s_flags & MS_RDONLY)) { /* * ->remount_fs needs lock_kernel(). @@ -590,7 +590,8 @@ static void do_emergency_remount(struct work_struct *work) do_remount_sb(sb, MS_RDONLY, NULL, 1); unlock_kernel(); } - drop_super(sb); + up_write(&sb->s_umount); + put_super(sb); spin_lock(&sb_lock); } spin_unlock(&sb_lock); |