[v2,02/30] btrfs-progs: stop accessing ->free_space_root directly

Message ID	c03b48d0184936184009c4f90f0816e5f0a35827.1636575146.git.josef@toxicpanda.com (mailing list archive)
State	New, archived
Headers	show Return-Path: <linux-btrfs-owner@kernel.org> From: Josef Bacik <josef@toxicpanda.com> To: linux-btrfs@vger.kernel.org, kernel-team@fb.com Subject: [PATCH v2 02/30] btrfs-progs: stop accessing ->free_space_root directly Date: Wed, 10 Nov 2021 15:14:14 -0500 Message-Id: <c03b48d0184936184009c4f90f0816e5f0a35827.1636575146.git.josef@toxicpanda.com> In-Reply-To: <cover.1636575146.git.josef@toxicpanda.com> References: <cover.1636575146.git.josef@toxicpanda.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk
Series	btrfs-progs: extent tree v2 support, global roots \| expand [v2,00/30] btrfs-progs: extent tree v2 support, global roots [v2,01/30] btrfs-progs: stop accessing ->extent_root directly [v2,02/30] btrfs-progs: stop accessing ->free_space_root directly [v2,03/30] btrfs-progs: track csum, extent, and free space trees in a rb tree [v2,04/30] btrfs-progs: check: make reinit work per found root item [v2,05/30] btrfs-progs: check: check the global roots for uptodate root nodes [v2,06/30] btrfs-progs: check: check all of the csum roots [v2,07/30] btrfs-progs: check: fill csum root from all extent roots [v2,08/30] btrfs-progs: common: search all extent roots for marking used space [v2,09/30] btrfs-progs: common: allow users to select extent-tree-v2 option [v2,10/30] btrfs-progs: add definitions for the block group tree [v2,11/30] btrfs-progs: add on disk pointers to global tree ids [v2,12/30] btrfs-progs: add support for loading the block group root [v2,13/30] btrfs-progs: add print support for the block group tree [v2,14/30] btrfs-progs: mkfs: use the btrfs_block_group_root helper [v2,15/30] btrfs-progs: check-lowmem: use the btrfs_block_group_root helper [v2,16/30] btrfs-progs: handle no bg item in extent tree for free space tree [v2,17/30] btrfs-progs: mkfs: add support for the block group tree [v2,18/30] btrfs-progs: check: add block group tree support [v2,19/30] btrfs-progs: qgroup-verify: scan extents based on block groups [v2,20/30] btrfs-progs: check: make free space tree validation extent tree v2 aware [v2,21/30] btrfs-progs: check: add helper to reinit the root based on a key [v2,22/30] btrfs-progs: check: handle the block group tree properly [v2,23/30] btrfs-progs: load the number of global roots into the fs_info [v2,24/30] btrfs-progs: handle the per-block group global root id [v2,25/30] btrfs-progs: add a btrfs_delete_and_free_root helper [v2,26/30] btrfs-progs: make btrfs_clear_free_space_tree extent tree v2 aware [v2,27/30] btrfs-progs: make btrfs_create_tree take a key for the root key [v2,28/30] btrfs-progs: mkfs: set chunk_item_objectid properly for extent tree v2 [v2,29/30] btrfs-progs: mkfs: create the global root's [v2,30/30] btrfs-progs: check: don't do the root item check for extent tree v2

diff --git a/kernel-shared/ctree.h b/kernel-shared/ctree.h index 23750156..c263a3bb 100644 --- a/kernel-shared/ctree.h +++ b/kernel-shared/ctree.h @@ -1163,7 +1163,7 @@ struct btrfs_fs_info { struct btrfs_root *dev_root; struct btrfs_root *_csum_root; struct btrfs_root *quota_root; - struct btrfs_root *free_space_root; + struct btrfs_root *_free_space_root; struct btrfs_root *uuid_root; struct rb_root fs_root_tree; diff --git a/kernel-shared/disk-io.c b/kernel-shared/disk-io.c index 76cf7567..4a43e102 100644 --- a/kernel-shared/disk-io.c +++ b/kernel-shared/disk-io.c @@ -791,7 +791,7 @@ struct btrfs_root *btrfs_read_fs_root(struct btrfs_fs_info *fs_info, return fs_info->quota_enabled ? fs_info->quota_root : ERR_PTR(-ENOENT); if (location->objectid == BTRFS_FREE_SPACE_TREE_OBJECTID) - return fs_info->free_space_root ? fs_info->free_space_root : + return fs_info->_free_space_root ? fs_info->_free_space_root : ERR_PTR(-ENOENT); BUG_ON(location->objectid == BTRFS_TREE_RELOC_OBJECTID); @@ -821,7 +821,7 @@ void btrfs_free_fs_info(struct btrfs_fs_info *fs_info) free(fs_info->chunk_root); free(fs_info->dev_root); free(fs_info->_csum_root); - free(fs_info->free_space_root); + free(fs_info->_free_space_root); free(fs_info->uuid_root); free(fs_info->super_copy); free(fs_info->log_root_tree); @@ -842,14 +842,14 @@ struct btrfs_fs_info *btrfs_new_fs_info(int writable, u64 sb_bytenr) fs_info->dev_root = calloc(1, sizeof(struct btrfs_root)); fs_info->_csum_root = calloc(1, sizeof(struct btrfs_root)); fs_info->quota_root = calloc(1, sizeof(struct btrfs_root)); - fs_info->free_space_root = calloc(1, sizeof(struct btrfs_root)); + fs_info->_free_space_root = calloc(1, sizeof(struct btrfs_root)); fs_info->uuid_root = calloc(1, sizeof(struct btrfs_root)); fs_info->super_copy = calloc(1, BTRFS_SUPER_INFO_SIZE); if (!fs_info->tree_root || !fs_info->_extent_root || !fs_info->chunk_root || !fs_info->dev_root || !fs_info->_csum_root || !fs_info->quota_root || - !fs_info->free_space_root || !fs_info->uuid_root || + !fs_info->_free_space_root || !fs_info->uuid_root || !fs_info->super_copy) goto free_all; @@ -1046,17 +1046,17 @@ int btrfs_setup_all_roots(struct btrfs_fs_info *fs_info, u64 root_tree_bytenr, if (btrfs_fs_compat_ro(fs_info, FREE_SPACE_TREE)) { ret = find_and_setup_root(root, fs_info, BTRFS_FREE_SPACE_TREE_OBJECTID, - fs_info->free_space_root); + fs_info->_free_space_root); if (ret) { - free(fs_info->free_space_root); - fs_info->free_space_root = NULL; + free(fs_info->_free_space_root); + fs_info->_free_space_root = NULL; printk("Couldn't read free space tree\n"); return -EIO; } - fs_info->free_space_root->track_dirty = 1; + fs_info->_free_space_root->track_dirty = 1; } else { - free(fs_info->free_space_root); - fs_info->free_space_root = NULL; + free(fs_info->_free_space_root); + fs_info->_free_space_root = NULL; } ret = find_and_setup_log_root(root, fs_info, sb); @@ -1095,8 +1095,8 @@ int btrfs_setup_all_roots(struct btrfs_fs_info *fs_info, u64 root_tree_bytenr, void btrfs_release_all_roots(struct btrfs_fs_info *fs_info) { - if (fs_info->free_space_root) - free_extent_buffer(fs_info->free_space_root->node); + if (fs_info->_free_space_root) + free_extent_buffer(fs_info->_free_space_root->node); if (fs_info->quota_root) free_extent_buffer(fs_info->quota_root->node); if (fs_info->_csum_root) diff --git a/kernel-shared/free-space-tree.c b/kernel-shared/free-space-tree.c index 0434733d..70b3d62a 100644 --- a/kernel-shared/free-space-tree.c +++ b/kernel-shared/free-space-tree.c @@ -25,13 +25,19 @@ #include "kernel-lib/bitops.h" #include "common/internal.h" +static struct btrfs_root *btrfs_free_space_root(struct btrfs_fs_info *fs_info, + struct btrfs_block_group *block_group) +{ + return fs_info->_free_space_root; +} + static struct btrfs_free_space_info * search_free_space_info(struct btrfs_trans_handle *trans, struct btrfs_fs_info *fs_info, struct btrfs_block_group *block_group, struct btrfs_path *path, int cow) { - struct btrfs_root *root = fs_info->free_space_root; + struct btrfs_root *root = btrfs_free_space_root(fs_info, block_group); struct btrfs_key key; int ret; @@ -103,7 +109,8 @@ static int add_new_free_space_info(struct btrfs_trans_handle *trans, struct btrfs_block_group *block_group, struct btrfs_path *path) { - struct btrfs_root *root = trans->fs_info->free_space_root; + struct btrfs_fs_info *fs_info = trans->fs_info; + struct btrfs_root *root = btrfs_free_space_root(fs_info, block_group); struct btrfs_free_space_info *info; struct btrfs_key key; struct extent_buffer *leaf; @@ -179,7 +186,7 @@ static int convert_free_space_to_bitmaps(struct btrfs_trans_handle *trans, struct btrfs_path *path) { struct btrfs_fs_info *fs_info = trans->fs_info; - struct btrfs_root *root = fs_info->free_space_root; + struct btrfs_root *root = btrfs_free_space_root(fs_info, block_group); struct btrfs_free_space_info *info; struct btrfs_key key, found_key; struct extent_buffer *leaf; @@ -318,7 +325,7 @@ static int convert_free_space_to_extents(struct btrfs_trans_handle *trans, struct btrfs_path *path) { struct btrfs_fs_info *fs_info = trans->fs_info; - struct btrfs_root *root = fs_info->free_space_root; + struct btrfs_root *root = btrfs_free_space_root(fs_info, block_group); struct btrfs_free_space_info *info; struct btrfs_key key, found_key; struct extent_buffer *leaf; @@ -558,7 +565,8 @@ static int modify_free_space_bitmap(struct btrfs_trans_handle *trans, struct btrfs_path *path, u64 start, u64 size, int remove) { - struct btrfs_root *root = trans->fs_info->free_space_root; + struct btrfs_fs_info *fs_info = trans->fs_info; + struct btrfs_root *root = btrfs_free_space_root(fs_info, block_group); struct btrfs_key key; u64 end = start + size; u64 cur_start, cur_size; @@ -671,7 +679,8 @@ static int remove_free_space_extent(struct btrfs_trans_handle *trans, struct btrfs_path *path, u64 start, u64 size) { - struct btrfs_root *root = trans->fs_info->free_space_root; + struct btrfs_fs_info *fs_info = trans->fs_info; + struct btrfs_root *root = btrfs_free_space_root(fs_info, block_group); struct btrfs_key key; u64 found_start, found_end; u64 end = start + size; @@ -811,7 +820,8 @@ static int add_free_space_extent(struct btrfs_trans_handle *trans, struct btrfs_path *path, u64 start, u64 size) { - struct btrfs_root *root = trans->fs_info->free_space_root; + struct btrfs_fs_info *fs_info = trans->fs_info; + struct btrfs_root *root = btrfs_free_space_root(fs_info, block_group); struct btrfs_key key, new_key; u64 found_start, found_end; u64 end = start + size; @@ -1107,7 +1117,8 @@ out: int remove_block_group_free_space(struct btrfs_trans_handle *trans, struct btrfs_block_group *block_group) { - struct btrfs_root *root = trans->fs_info->free_space_root; + struct btrfs_fs_info *fs_info = trans->fs_info; + struct btrfs_root *root = btrfs_free_space_root(fs_info, block_group); struct btrfs_path *path; struct btrfs_key key, found_key; struct extent_buffer *leaf; @@ -1215,7 +1226,7 @@ int btrfs_clear_free_space_tree(struct btrfs_fs_info *fs_info) { struct btrfs_trans_handle *trans; struct btrfs_root *tree_root = fs_info->tree_root; - struct btrfs_root *free_space_root = fs_info->free_space_root; + struct btrfs_root *free_space_root = btrfs_free_space_root(fs_info, NULL); int ret; u64 features; @@ -1227,7 +1238,7 @@ int btrfs_clear_free_space_tree(struct btrfs_fs_info *fs_info) features &= ~(BTRFS_FEATURE_COMPAT_RO_FREE_SPACE_TREE_VALID | BTRFS_FEATURE_COMPAT_RO_FREE_SPACE_TREE); btrfs_set_super_compat_ro_flags(fs_info->super_copy, features); - fs_info->free_space_root = NULL; + fs_info->_free_space_root = NULL; ret = clear_free_space_tree(trans, free_space_root); if (ret) @@ -1263,7 +1274,7 @@ static int load_free_space_bitmaps(struct btrfs_fs_info *fs_info, u32 expected_extent_count, int *errors) { - struct btrfs_root *root = fs_info->free_space_root; + struct btrfs_root *root = btrfs_free_space_root(fs_info, block_group); struct btrfs_key key; int prev_bit = 0, bit; u64 extent_start = 0; @@ -1343,7 +1354,7 @@ static int load_free_space_extents(struct btrfs_fs_info *fs_info, u32 expected_extent_count, int *errors) { - struct btrfs_root *root = fs_info->free_space_root; + struct btrfs_root *root = btrfs_free_space_root(fs_info, block_group); struct btrfs_key key, prev_key; int have_prev = 0; u64 start, end; @@ -1463,7 +1474,7 @@ int btrfs_create_free_space_tree(struct btrfs_fs_info *fs_info) ret = PTR_ERR(free_space_root); goto abort; } - fs_info->free_space_root = free_space_root; + fs_info->_free_space_root = free_space_root; add_root_to_dirty_list(free_space_root); do { diff --git a/mkfs/main.c b/mkfs/main.c index 9647f12a..16f9ba19 100644 --- a/mkfs/main.c +++ b/mkfs/main.c @@ -289,8 +289,8 @@ static int recow_roots(struct btrfs_trans_handle *trans, ret = __recow_root(trans, csum_root); if (ret) return ret; - if (info->free_space_root) { - ret = __recow_root(trans, info->free_space_root); + if (info->_free_space_root) { + ret = __recow_root(trans, info->_free_space_root); if (ret) return ret; }

[v2,02/30] btrfs-progs: stop accessing ->free_space_root directly

Commit Message

Patch