Commit 2365dd3c authored by Anand Jain's avatar Anand Jain Committed by Chris Mason

btrfs: undo sysfs when open_ctree() fails

reproducer:
mkfs.btrfs -f /dev/sdb &&\
mount /dev/sdb /btrfs &&\
btrfs dev add -f /dev/sdc /btrfs &&\
umount /btrfs &&\
wipefs -a /dev/sdc &&\
mount -o degraded /dev/sdb /btrfs
//above mount fails so try with RO
mount -o degraded,ro /dev/sdb /btrfs

------
sysfs: cannot create duplicate filename '/fs/btrfs/3f48c79e-5ed0-4e87-b189-86e749e503f4'
::

dump_stack+0x49/0x5e
warn_slowpath_common+0x87/0xb0
warn_slowpath_fmt+0x41/0x50
strlcat+0x69/0x80
sysfs_warn_dup+0x87/0xa0
sysfs_add_one+0x40/0x50
create_dir+0x76/0xc0
sysfs_create_dir_ns+0x7a/0xc0
kobject_add_internal+0xad/0x220
kobject_add_varg+0x38/0x60
kobject_init_and_add+0x53/0x70
mutex_lock+0x11/0x40
__free_pages+0x25/0x30
free_pages+0x41/0x50
selinux_sb_copy_data+0x14e/0x1e0
mount_fs+0x3e/0x1a0
vfs_kern_mount+0x71/0x120
do_mount+0x3f7/0x980
SyS_mount+0x8b/0xe0
system_call_fastpath+0x16/0x1b
------

further 'modprobe -r btrfs' fails as well
Signed-off-by: default avatarAnand Jain <anand.jain@oracle.com>
Signed-off-by: default avatarJosef Bacik <jbacik@fb.com>
Signed-off-by: default avatarChris Mason <clm@fb.com>
parent f74b86d8
...@@ -2745,13 +2745,13 @@ int open_ctree(struct super_block *sb, ...@@ -2745,13 +2745,13 @@ int open_ctree(struct super_block *sb,
ret = btrfs_init_space_info(fs_info); ret = btrfs_init_space_info(fs_info);
if (ret) { if (ret) {
printk(KERN_ERR "BTRFS: Failed to initial space info: %d\n", ret); printk(KERN_ERR "BTRFS: Failed to initial space info: %d\n", ret);
goto fail_block_groups; goto fail_sysfs;
} }
ret = btrfs_read_block_groups(extent_root); ret = btrfs_read_block_groups(extent_root);
if (ret) { if (ret) {
printk(KERN_ERR "BTRFS: Failed to read block groups: %d\n", ret); printk(KERN_ERR "BTRFS: Failed to read block groups: %d\n", ret);
goto fail_block_groups; goto fail_sysfs;
} }
fs_info->num_tolerated_disk_barrier_failures = fs_info->num_tolerated_disk_barrier_failures =
btrfs_calc_num_tolerated_disk_barrier_failures(fs_info); btrfs_calc_num_tolerated_disk_barrier_failures(fs_info);
...@@ -2760,13 +2760,13 @@ int open_ctree(struct super_block *sb, ...@@ -2760,13 +2760,13 @@ int open_ctree(struct super_block *sb,
!(sb->s_flags & MS_RDONLY)) { !(sb->s_flags & MS_RDONLY)) {
printk(KERN_WARNING "BTRFS: " printk(KERN_WARNING "BTRFS: "
"too many missing devices, writeable mount is not allowed\n"); "too many missing devices, writeable mount is not allowed\n");
goto fail_block_groups; goto fail_sysfs;
} }
fs_info->cleaner_kthread = kthread_run(cleaner_kthread, tree_root, fs_info->cleaner_kthread = kthread_run(cleaner_kthread, tree_root,
"btrfs-cleaner"); "btrfs-cleaner");
if (IS_ERR(fs_info->cleaner_kthread)) if (IS_ERR(fs_info->cleaner_kthread))
goto fail_block_groups; goto fail_sysfs;
fs_info->transaction_kthread = kthread_run(transaction_kthread, fs_info->transaction_kthread = kthread_run(transaction_kthread,
tree_root, tree_root,
...@@ -2948,6 +2948,9 @@ int open_ctree(struct super_block *sb, ...@@ -2948,6 +2948,9 @@ int open_ctree(struct super_block *sb,
*/ */
filemap_write_and_wait(fs_info->btree_inode->i_mapping); filemap_write_and_wait(fs_info->btree_inode->i_mapping);
fail_sysfs:
btrfs_sysfs_remove_one(fs_info);
fail_block_groups: fail_block_groups:
btrfs_put_block_group_cache(fs_info); btrfs_put_block_group_cache(fs_info);
btrfs_free_block_groups(fs_info); btrfs_free_block_groups(fs_info);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment