btrfs: call functions that overwrite their root parameter with fs_info
authorJeff Mahoney <jeffm@suse.com>
Tue, 21 Jun 2016 14:40:19 +0000 (10:40 -0400)
committerDavid Sterba <dsterba@suse.com>
Tue, 6 Dec 2016 15:06:57 +0000 (16:06 +0100)
There are 11 functions that accept a root parameter and immediately
overwrite it.  We can pass those an fs_info pointer instead.

Signed-off-by: Jeff Mahoney <jeffm@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/ctree.h
fs/btrfs/disk-io.c
fs/btrfs/extent-tree.c
fs/btrfs/file-item.c
fs/btrfs/free-space-cache.c
fs/btrfs/free-space-cache.h
fs/btrfs/transaction.c
fs/btrfs/tree-log.c
fs/btrfs/volumes.c
fs/btrfs/volumes.h

index 1b25a460eceae0b2c142c1a711860e3197496c07..51cd757461c4f52b6b1c2104e51706e92aa682c8 100644 (file)
@@ -2639,7 +2639,7 @@ int btrfs_setup_space_cache(struct btrfs_trans_handle *trans,
                            struct btrfs_root *root);
 int btrfs_extent_readonly(struct btrfs_root *root, u64 bytenr);
 int btrfs_free_block_groups(struct btrfs_fs_info *info);
-int btrfs_read_block_groups(struct btrfs_root *root);
+int btrfs_read_block_groups(struct btrfs_fs_info *info);
 int btrfs_can_relocate(struct btrfs_root *root, u64 bytenr);
 int btrfs_make_block_group(struct btrfs_trans_handle *trans,
                           struct btrfs_root *root, u64 bytes_used,
@@ -3055,7 +3055,7 @@ int btrfs_find_name_in_ext_backref(struct btrfs_path *path,
 /* file-item.c */
 struct btrfs_dio_private;
 int btrfs_del_csums(struct btrfs_trans_handle *trans,
-                   struct btrfs_root *root, u64 bytenr, u64 len);
+                   struct btrfs_fs_info *fs_info, u64 bytenr, u64 len);
 int btrfs_lookup_bio_sums(struct btrfs_root *root, struct inode *inode,
                          struct bio *bio, u32 *dst);
 int btrfs_lookup_bio_sums_dio(struct btrfs_root *root, struct inode *inode,
index 811662cce97732af769946d58a57a471022187de..92c2aea5118b90c6e8e22457b71df837a42b61c0 100644 (file)
@@ -2937,7 +2937,7 @@ int open_ctree(struct super_block *sb,
        read_extent_buffer(chunk_root->node, fs_info->chunk_tree_uuid,
           btrfs_header_chunk_tree_uuid(chunk_root->node), BTRFS_UUID_SIZE);
 
-       ret = btrfs_read_chunk_tree(chunk_root);
+       ret = btrfs_read_chunk_tree(fs_info);
        if (ret) {
                btrfs_err(fs_info, "failed to read chunk tree: %d", ret);
                goto fail_tree_roots;
@@ -3038,7 +3038,7 @@ retry_root_backup:
                goto fail_sysfs;
        }
 
-       ret = btrfs_read_block_groups(fs_info->extent_root);
+       ret = btrfs_read_block_groups(fs_info);
        if (ret) {
                btrfs_err(fs_info, "failed to read block groups: %d", ret);
                goto fail_sysfs;
index 0d80136206b19a13072821c46a1a60017c244a76..13ef5d53943bc2ff85c8e111fe8c460d53db1488 100644 (file)
@@ -2414,7 +2414,7 @@ static int run_one_delayed_ref(struct btrfs_trans_handle *trans,
                        btrfs_pin_extent(root, node->bytenr,
                                         node->num_bytes, 1);
                        if (head->is_data) {
-                               ret = btrfs_del_csums(trans, root,
+                               ret = btrfs_del_csums(trans, root->fs_info,
                                                      node->bytenr,
                                                      node->num_bytes);
                        }
@@ -3622,7 +3622,8 @@ again:
 
                if (cache->disk_cache_state == BTRFS_DC_SETUP) {
                        cache->io_ctl.inode = NULL;
-                       ret = btrfs_write_out_cache(root, trans, cache, path);
+                       ret = btrfs_write_out_cache(root->fs_info, trans,
+                                                   cache, path);
                        if (ret == 0 && cache->io_ctl.inode) {
                                num_started++;
                                should_put = 0;
@@ -3774,7 +3775,8 @@ int btrfs_write_dirty_block_groups(struct btrfs_trans_handle *trans,
 
                if (!ret && cache->disk_cache_state == BTRFS_DC_SETUP) {
                        cache->io_ctl.inode = NULL;
-                       ret = btrfs_write_out_cache(root, trans, cache, path);
+                       ret = btrfs_write_out_cache(root->fs_info, trans,
+                                                   cache, path);
                        if (ret == 0 && cache->io_ctl.inode) {
                                num_started++;
                                should_put = 0;
@@ -7068,7 +7070,7 @@ static int __btrfs_free_extent(struct btrfs_trans_handle *trans,
                btrfs_release_path(path);
 
                if (is_data) {
-                       ret = btrfs_del_csums(trans, root, bytenr, num_bytes);
+                       ret = btrfs_del_csums(trans, info, bytenr, num_bytes);
                        if (ret) {
                                btrfs_abort_transaction(trans, ret);
                                goto out;
@@ -9925,12 +9927,12 @@ btrfs_create_block_group_cache(struct btrfs_root *root, u64 start, u64 size)
        return cache;
 }
 
-int btrfs_read_block_groups(struct btrfs_root *root)
+int btrfs_read_block_groups(struct btrfs_fs_info *info)
 {
+       struct btrfs_root *root = info->extent_root;
        struct btrfs_path *path;
        int ret;
        struct btrfs_block_group_cache *cache;
-       struct btrfs_fs_info *info = root->fs_info;
        struct btrfs_space_info *space_info;
        struct btrfs_key key;
        struct btrfs_key found_key;
@@ -9943,7 +9945,6 @@ int btrfs_read_block_groups(struct btrfs_root *root)
        feature = btrfs_super_incompat_flags(info->super_copy);
        mixed = !!(feature & BTRFS_FEATURE_INCOMPAT_MIXED_GROUPS);
 
-       root = info->extent_root;
        key.objectid = 0;
        key.offset = 0;
        key.type = BTRFS_BLOCK_GROUP_ITEM_KEY;
@@ -10733,7 +10734,7 @@ void btrfs_delete_unused_bgs(struct btrfs_fs_info *fs_info)
                 * Btrfs_remove_chunk will abort the transaction if things go
                 * horribly wrong.
                 */
-               ret = btrfs_remove_chunk(trans, root,
+               ret = btrfs_remove_chunk(trans, fs_info,
                                         block_group->key.objectid);
 
                if (ret) {
index 5e74178ba9d994c45a8c3efcc0afe9bb4dc42245..41c1145cbce1293b47f23472e3f6d8ca20086e99 100644 (file)
@@ -590,8 +590,9 @@ static noinline void truncate_one_csum(struct btrfs_root *root,
  * range of bytes.
  */
 int btrfs_del_csums(struct btrfs_trans_handle *trans,
-                   struct btrfs_root *root, u64 bytenr, u64 len)
+                   struct btrfs_fs_info *fs_info, u64 bytenr, u64 len)
 {
+       struct btrfs_root *root = fs_info->csum_root;
        struct btrfs_path *path;
        struct btrfs_key key;
        u64 end_byte = bytenr + len;
@@ -601,8 +602,6 @@ int btrfs_del_csums(struct btrfs_trans_handle *trans,
        u16 csum_size = btrfs_super_csum_size(root->fs_info->super_copy);
        int blocksize_bits = root->fs_info->sb->s_blocksize_bits;
 
-       root = root->fs_info->csum_root;
-
        path = btrfs_alloc_path();
        if (!path)
                return -ENOMEM;
index c698dccb3757338968af9b7e9f815dceab4d6e08..a538133aec63f8c26d386e4e2acd56ee36de7410 100644 (file)
@@ -1350,17 +1350,16 @@ out_nospc:
        goto out;
 }
 
-int btrfs_write_out_cache(struct btrfs_root *root,
+int btrfs_write_out_cache(struct btrfs_fs_info *fs_info,
                          struct btrfs_trans_handle *trans,
                          struct btrfs_block_group_cache *block_group,
                          struct btrfs_path *path)
 {
+       struct btrfs_root *root = fs_info->tree_root;
        struct btrfs_free_space_ctl *ctl = block_group->free_space_ctl;
        struct inode *inode;
        int ret = 0;
 
-       root = root->fs_info->tree_root;
-
        spin_lock(&block_group->lock);
        if (block_group->disk_cache_state < BTRFS_DC_SETUP) {
                spin_unlock(&block_group->lock);
index 363fdd955e5d5a1823e3660365a7b40a010bfea6..70e7a7cc15f3ef9b68a6f3eaf02f4e91203b1fd1 100644 (file)
@@ -72,7 +72,7 @@ int btrfs_wait_cache_io(struct btrfs_root *root,
                        struct btrfs_block_group_cache *block_group,
                        struct btrfs_io_ctl *io_ctl,
                        struct btrfs_path *path, u64 offset);
-int btrfs_write_out_cache(struct btrfs_root *root,
+int btrfs_write_out_cache(struct btrfs_fs_info *fs_info,
                          struct btrfs_trans_handle *trans,
                          struct btrfs_block_group_cache *block_group,
                          struct btrfs_path *path);
index 9517de0e668cc9f52664a00964912eec39ddef19..88be1f812391248578ea70fec67930b7cd8f3459 100644 (file)
@@ -1219,10 +1219,9 @@ void btrfs_add_dead_root(struct btrfs_root *root)
  * update all the cowonly tree roots on disk
  */
 static noinline int commit_fs_roots(struct btrfs_trans_handle *trans,
-                                   struct btrfs_root *root)
+                                   struct btrfs_fs_info *fs_info)
 {
        struct btrfs_root *gang[8];
-       struct btrfs_fs_info *fs_info = root->fs_info;
        int i;
        int ret;
        int err = 0;
@@ -1236,7 +1235,7 @@ static noinline int commit_fs_roots(struct btrfs_trans_handle *trans,
                if (ret == 0)
                        break;
                for (i = 0; i < ret; i++) {
-                       root = gang[i];
+                       struct btrfs_root *root = gang[i];
                        radix_tree_tag_clear(&fs_info->fs_roots_radix,
                                        (unsigned long)root->root_key.objectid,
                                        BTRFS_ROOT_TRANS_TAG);
@@ -1343,7 +1342,7 @@ static int qgroup_account_snapshot(struct btrfs_trans_handle *trans,
         */
        mutex_lock(&fs_info->tree_log_mutex);
 
-       ret = commit_fs_roots(trans, src);
+       ret = commit_fs_roots(trans, fs_info);
        if (ret)
                goto out;
        ret = btrfs_qgroup_prepare_account_extents(trans, fs_info);
@@ -2132,7 +2131,7 @@ int btrfs_commit_transaction(struct btrfs_trans_handle *trans,
         */
        mutex_lock(&root->fs_info->tree_log_mutex);
 
-       ret = commit_fs_roots(trans, root);
+       ret = commit_fs_roots(trans, root->fs_info);
        if (ret) {
                mutex_unlock(&root->fs_info->tree_log_mutex);
                mutex_unlock(&root->fs_info->reloc_mutex);
index e0478f51cf16942aa26b2b1eae03a5a6cd7ca88d..72bd398f6c66987f8b94907b12dc74841a840764 100644 (file)
@@ -797,9 +797,9 @@ static noinline int replay_one_extent(struct btrfs_trans_handle *trans,
                                                list);
                                if (!ret)
                                        ret = btrfs_del_csums(trans,
-                                                     root->fs_info->csum_root,
-                                                     sums->bytenr,
-                                                     sums->len);
+                                                             root->fs_info,
+                                                             sums->bytenr,
+                                                             sums->len);
                                if (!ret)
                                        ret = btrfs_csum_file_blocks(trans,
                                                root->fs_info->csum_root,
index 1886b94f13aca32479feb254c7cdca9489d81b70..17ab4b6b1fd902dfc70650a718aa8c755aee0d47 100644 (file)
@@ -1667,9 +1667,10 @@ error:
  * the btrfs_device struct should be fully filled in
  */
 static int btrfs_add_device(struct btrfs_trans_handle *trans,
-                           struct btrfs_root *root,
+                           struct btrfs_fs_info *fs_info,
                            struct btrfs_device *device)
 {
+       struct btrfs_root *root = fs_info->chunk_root;
        int ret;
        struct btrfs_path *path;
        struct btrfs_dev_item *dev_item;
@@ -1677,8 +1678,6 @@ static int btrfs_add_device(struct btrfs_trans_handle *trans,
        struct btrfs_key key;
        unsigned long ptr;
 
-       root = root->fs_info->chunk_root;
-
        path = btrfs_alloc_path();
        if (!path)
                return -ENOMEM;
@@ -1737,16 +1736,15 @@ static void update_dev_time(char *path_name)
        filp_close(filp, NULL);
 }
 
-static int btrfs_rm_dev_item(struct btrfs_root *root,
+static int btrfs_rm_dev_item(struct btrfs_fs_info *fs_info,
                             struct btrfs_device *device)
 {
+       struct btrfs_root *root = fs_info->chunk_root;
        int ret;
        struct btrfs_path *path;
        struct btrfs_key key;
        struct btrfs_trans_handle *trans;
 
-       root = root->fs_info->chunk_root;
-
        path = btrfs_alloc_path();
        if (!path)
                return -ENOMEM;
@@ -1909,7 +1907,7 @@ int btrfs_rm_device(struct btrfs_root *root, char *device_path, u64 devid)
         * counter although write_all_supers() is not locked out. This
         * could give a filesystem state which requires a degraded mount.
         */
-       ret = btrfs_rm_dev_item(root->fs_info->chunk_root, device);
+       ret = btrfs_rm_dev_item(root->fs_info, device);
        if (ret)
                goto error_undo;
 
@@ -2241,8 +2239,9 @@ static int btrfs_prepare_sprout(struct btrfs_root *root)
  * Store the expected generation for seed devices in device items.
  */
 static int btrfs_finish_sprout(struct btrfs_trans_handle *trans,
-                              struct btrfs_root *root)
+                              struct btrfs_fs_info *fs_info)
 {
+       struct btrfs_root *root = fs_info->chunk_root;
        struct btrfs_path *path;
        struct extent_buffer *leaf;
        struct btrfs_dev_item *dev_item;
@@ -2257,7 +2256,6 @@ static int btrfs_finish_sprout(struct btrfs_trans_handle *trans,
        if (!path)
                return -ENOMEM;
 
-       root = root->fs_info->chunk_root;
        key.objectid = BTRFS_DEV_ITEMS_OBJECTID;
        key.offset = 0;
        key.type = BTRFS_DEV_ITEM_KEY;
@@ -2452,7 +2450,7 @@ int btrfs_init_new_device(struct btrfs_root *root, char *device_path)
                }
        }
 
-       ret = btrfs_add_device(trans, root, device);
+       ret = btrfs_add_device(trans, root->fs_info, device);
        if (ret) {
                btrfs_abort_transaction(trans, ret);
                goto error_trans;
@@ -2461,7 +2459,7 @@ int btrfs_init_new_device(struct btrfs_root *root, char *device_path)
        if (seeding_dev) {
                char fsid_buf[BTRFS_UUID_UNPARSED_SIZE];
 
-               ret = btrfs_finish_sprout(trans, root);
+               ret = btrfs_finish_sprout(trans, root->fs_info);
                if (ret) {
                        btrfs_abort_transaction(trans, ret);
                        goto error_trans;
@@ -2716,14 +2714,14 @@ int btrfs_grow_device(struct btrfs_trans_handle *trans,
 }
 
 static int btrfs_free_chunk(struct btrfs_trans_handle *trans,
-                           struct btrfs_root *root, u64 chunk_objectid,
+                           struct btrfs_fs_info *fs_info, u64 chunk_objectid,
                            u64 chunk_offset)
 {
+       struct btrfs_root *root = fs_info->chunk_root;
        int ret;
        struct btrfs_path *path;
        struct btrfs_key key;
 
-       root = root->fs_info->chunk_root;
        path = btrfs_alloc_path();
        if (!path)
                return -ENOMEM;
@@ -2800,8 +2798,9 @@ static int btrfs_del_sys_chunk(struct btrfs_root *root, u64 chunk_objectid, u64
 }
 
 int btrfs_remove_chunk(struct btrfs_trans_handle *trans,
-                      struct btrfs_root *root, u64 chunk_offset)
+                      struct btrfs_fs_info *fs_info, u64 chunk_offset)
 {
+       struct btrfs_root *root = fs_info->chunk_root;
        struct extent_map_tree *em_tree;
        struct extent_map *em;
        struct btrfs_root *extent_root = root->fs_info->extent_root;
@@ -2811,9 +2810,7 @@ int btrfs_remove_chunk(struct btrfs_trans_handle *trans,
        int i, ret = 0;
        struct btrfs_fs_devices *fs_devices = root->fs_info->fs_devices;
 
-       /* Just in case */
-       root = root->fs_info->chunk_root;
-       em_tree = &root->fs_info->mapping_tree.map_tree;
+       em_tree = &fs_info->mapping_tree.map_tree;
 
        read_lock(&em_tree->lock);
        em = lookup_extent_mapping(em_tree, chunk_offset, 1);
@@ -2875,7 +2872,8 @@ int btrfs_remove_chunk(struct btrfs_trans_handle *trans,
        }
        mutex_unlock(&fs_devices->device_list_mutex);
 
-       ret = btrfs_free_chunk(trans, root, chunk_objectid, chunk_offset);
+       ret = btrfs_free_chunk(trans, root->fs_info, chunk_objectid,
+                              chunk_offset);
        if (ret) {
                btrfs_abort_transaction(trans, ret);
                goto out;
@@ -2903,15 +2901,13 @@ out:
        return ret;
 }
 
-static int btrfs_relocate_chunk(struct btrfs_root *root, u64 chunk_offset)
+static int btrfs_relocate_chunk(struct btrfs_fs_info *fs_info, u64 chunk_offset)
 {
-       struct btrfs_root *extent_root;
+       struct btrfs_root *root = fs_info->chunk_root;
+       struct btrfs_root *extent_root = fs_info->extent_root;
        struct btrfs_trans_handle *trans;
        int ret;
 
-       root = root->fs_info->chunk_root;
-       extent_root = root->fs_info->extent_root;
-
        /*
         * Prevent races with automatic removal of unused block groups.
         * After we relocate and before we remove the chunk with offset
@@ -2949,7 +2945,7 @@ static int btrfs_relocate_chunk(struct btrfs_root *root, u64 chunk_offset)
         * step two, delete the device extents and the
         * chunk tree entries
         */
-       ret = btrfs_remove_chunk(trans, root, chunk_offset);
+       ret = btrfs_remove_chunk(trans, fs_info, chunk_offset);
        btrfs_end_transaction(trans, extent_root);
        return ret;
 }
@@ -3003,7 +2999,7 @@ again:
                btrfs_release_path(path);
 
                if (chunk_type & BTRFS_BLOCK_GROUP_SYSTEM) {
-                       ret = btrfs_relocate_chunk(chunk_root,
+                       ret = btrfs_relocate_chunk(root->fs_info,
                                                   found_key.offset);
                        if (ret == -ENOSPC)
                                failed++;
@@ -3669,8 +3665,7 @@ again:
                        chunk_reserved = 1;
                }
 
-               ret = btrfs_relocate_chunk(chunk_root,
-                                          found_key.offset);
+               ret = btrfs_relocate_chunk(fs_info, found_key.offset);
                mutex_unlock(&fs_info->delete_unused_bgs_mutex);
                if (ret && ret != -ENOSPC)
                        goto error;
@@ -4439,7 +4434,7 @@ again:
                chunk_offset = btrfs_dev_extent_chunk_offset(l, dev_extent);
                btrfs_release_path(path);
 
-               ret = btrfs_relocate_chunk(root, chunk_offset);
+               ret = btrfs_relocate_chunk(root->fs_info, chunk_offset);
                mutex_unlock(&root->fs_info->delete_unused_bgs_mutex);
                if (ret && ret != -ENOSPC)
                        goto done;
@@ -6785,8 +6780,9 @@ out_short_read:
        return -EIO;
 }
 
-int btrfs_read_chunk_tree(struct btrfs_root *root)
+int btrfs_read_chunk_tree(struct btrfs_fs_info *fs_info)
 {
+       struct btrfs_root *root = fs_info->chunk_root;
        struct btrfs_path *path;
        struct extent_buffer *leaf;
        struct btrfs_key key;
@@ -6795,8 +6791,6 @@ int btrfs_read_chunk_tree(struct btrfs_root *root)
        int slot;
        u64 total_dev = 0;
 
-       root = root->fs_info->chunk_root;
-
        path = btrfs_alloc_path();
        if (!path)
                return -ENOMEM;
index 9029a313492294a7eb51249e40cef291e5eea596..416574f9dfc277015ab2900e41f755fc0227091c 100644 (file)
@@ -407,7 +407,7 @@ int btrfs_rmap_block(struct btrfs_fs_info *fs_info,
                     u64 chunk_start, u64 physical, u64 devid,
                     u64 **logical, int *naddrs, int *stripe_len);
 int btrfs_read_sys_array(struct btrfs_root *root);
-int btrfs_read_chunk_tree(struct btrfs_root *root);
+int btrfs_read_chunk_tree(struct btrfs_fs_info *fs_info);
 int btrfs_alloc_chunk(struct btrfs_trans_handle *trans,
                      struct btrfs_root *extent_root, u64 type);
 void btrfs_mapping_init(struct btrfs_mapping_tree *tree);
@@ -483,7 +483,7 @@ int btrfs_finish_chunk_alloc(struct btrfs_trans_handle *trans,
                                struct btrfs_root *extent_root,
                                u64 chunk_offset, u64 chunk_size);
 int btrfs_remove_chunk(struct btrfs_trans_handle *trans,
-                      struct btrfs_root *root, u64 chunk_offset);
+                      struct btrfs_fs_info *fs_info, u64 chunk_offset);
 
 static inline int btrfs_dev_stats_dirty(struct btrfs_device *dev)
 {