Skip to content

Commit b96ab16

Browse files
fdmananakdave
authored andcommitted
btrfs: rename the functions to init and release an extent io tree
These functions are exported so they should have a 'btrfs_' prefix by convention, to make it clear they are btrfs specific and to avoid collisions with functions from elsewhere in the kernel. So add a 'btrfs_' prefix to their name to make it clear they are from btrfs. Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com> Signed-off-by: Filipe Manana <fdmanana@suse.com> Reviewed-by: David Sterba <dsterba@suse.com> Signed-off-by: David Sterba <dsterba@suse.com>
1 parent 3c0f3ed commit b96ab16

File tree

10 files changed

+33
-32
lines changed

10 files changed

+33
-32
lines changed

fs/btrfs/disk-io.c

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -691,10 +691,10 @@ static void __setup_root(struct btrfs_root *root, struct btrfs_fs_info *fs_info,
691691
btrfs_set_root_last_log_commit(root, 0);
692692
root->anon_dev = 0;
693693
if (!dummy) {
694-
extent_io_tree_init(fs_info, &root->dirty_log_pages,
695-
IO_TREE_ROOT_DIRTY_LOG_PAGES);
696-
extent_io_tree_init(fs_info, &root->log_csum_range,
697-
IO_TREE_LOG_CSUM_RANGE);
694+
btrfs_extent_io_tree_init(fs_info, &root->dirty_log_pages,
695+
IO_TREE_ROOT_DIRTY_LOG_PAGES);
696+
btrfs_extent_io_tree_init(fs_info, &root->log_csum_range,
697+
IO_TREE_LOG_CSUM_RANGE);
698698
}
699699

700700
spin_lock_init(&root->root_item_lock);
@@ -1920,8 +1920,8 @@ static int btrfs_init_btree_inode(struct super_block *sb)
19201920
inode->i_mapping->a_ops = &btree_aops;
19211921
mapping_set_gfp_mask(inode->i_mapping, GFP_NOFS);
19221922

1923-
extent_io_tree_init(fs_info, &BTRFS_I(inode)->io_tree,
1924-
IO_TREE_BTREE_INODE_IO);
1923+
btrfs_extent_io_tree_init(fs_info, &BTRFS_I(inode)->io_tree,
1924+
IO_TREE_BTREE_INODE_IO);
19251925
extent_map_tree_init(&BTRFS_I(inode)->extent_tree);
19261926

19271927
BTRFS_I(inode)->root = btrfs_grab_root(fs_info->tree_root);
@@ -2855,8 +2855,8 @@ void btrfs_init_fs_info(struct btrfs_fs_info *fs_info)
28552855
rwlock_init(&fs_info->block_group_cache_lock);
28562856
fs_info->block_group_cache_tree = RB_ROOT_CACHED;
28572857

2858-
extent_io_tree_init(fs_info, &fs_info->excluded_extents,
2859-
IO_TREE_FS_EXCLUDED_EXTENTS);
2858+
btrfs_extent_io_tree_init(fs_info, &fs_info->excluded_extents,
2859+
IO_TREE_FS_EXCLUDED_EXTENTS);
28602860

28612861
mutex_init(&fs_info->ordered_operations_mutex);
28622862
mutex_init(&fs_info->tree_log_mutex);

fs/btrfs/extent-io-tree.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -95,8 +95,8 @@ const struct btrfs_fs_info *btrfs_extent_io_tree_to_fs_info(const struct extent_
9595
return tree->fs_info;
9696
}
9797

98-
void extent_io_tree_init(struct btrfs_fs_info *fs_info,
99-
struct extent_io_tree *tree, unsigned int owner)
98+
void btrfs_extent_io_tree_init(struct btrfs_fs_info *fs_info,
99+
struct extent_io_tree *tree, unsigned int owner)
100100
{
101101
tree->state = RB_ROOT;
102102
spin_lock_init(&tree->lock);
@@ -111,7 +111,7 @@ void extent_io_tree_init(struct btrfs_fs_info *fs_info,
111111
* aren't any waiters on any extent state record (EXTENT_LOCK_BITS are never
112112
* set on any extent state when calling this function).
113113
*/
114-
void extent_io_tree_release(struct extent_io_tree *tree)
114+
void btrfs_extent_io_tree_release(struct extent_io_tree *tree)
115115
{
116116
struct rb_root root;
117117
struct extent_state *state;

fs/btrfs/extent-io-tree.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -137,9 +137,9 @@ struct extent_state {
137137
const struct btrfs_inode *btrfs_extent_io_tree_to_inode(const struct extent_io_tree *tree);
138138
const struct btrfs_fs_info *btrfs_extent_io_tree_to_fs_info(const struct extent_io_tree *tree);
139139

140-
void extent_io_tree_init(struct btrfs_fs_info *fs_info,
141-
struct extent_io_tree *tree, unsigned int owner);
142-
void extent_io_tree_release(struct extent_io_tree *tree);
140+
void btrfs_extent_io_tree_init(struct btrfs_fs_info *fs_info,
141+
struct extent_io_tree *tree, unsigned int owner);
142+
void btrfs_extent_io_tree_release(struct extent_io_tree *tree);
143143
int btrfs_lock_extent_bits(struct extent_io_tree *tree, u64 start, u64 end, u32 bits,
144144
struct extent_state **cached);
145145
bool btrfs_try_lock_extent_bits(struct extent_io_tree *tree, u64 start, u64 end,

fs/btrfs/inode.c

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3807,7 +3807,8 @@ static int btrfs_init_file_extent_tree(struct btrfs_inode *inode)
38073807
if (!inode->file_extent_tree)
38083808
return -ENOMEM;
38093809

3810-
extent_io_tree_init(fs_info, inode->file_extent_tree, IO_TREE_INODE_FILE_EXTENT);
3810+
btrfs_extent_io_tree_init(fs_info, inode->file_extent_tree,
3811+
IO_TREE_INODE_FILE_EXTENT);
38113812
/* Lockdep class is set only for the file extent tree. */
38123813
lockdep_set_class(&inode->file_extent_tree->lock, &file_extent_tree_class);
38133814

@@ -7733,7 +7734,7 @@ struct inode *btrfs_alloc_inode(struct super_block *sb)
77337734
extent_map_tree_init(&ei->extent_tree);
77347735

77357736
/* This io tree sets the valid inode. */
7736-
extent_io_tree_init(fs_info, &ei->io_tree, IO_TREE_INODE_IO);
7737+
btrfs_extent_io_tree_init(fs_info, &ei->io_tree, IO_TREE_INODE_IO);
77377738
ei->io_tree.inode = ei;
77387739

77397740
ei->file_extent_tree = NULL;

fs/btrfs/relocation.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3860,7 +3860,7 @@ static struct reloc_control *alloc_reloc_control(struct btrfs_fs_info *fs_info)
38603860
btrfs_backref_init_cache(fs_info, &rc->backref_cache, true);
38613861
rc->reloc_root_tree.rb_root = RB_ROOT;
38623862
spin_lock_init(&rc->reloc_root_tree.lock);
3863-
extent_io_tree_init(fs_info, &rc->processed_blocks, IO_TREE_RELOC_BLOCKS);
3863+
btrfs_extent_io_tree_init(fs_info, &rc->processed_blocks, IO_TREE_RELOC_BLOCKS);
38643864
return rc;
38653865
}
38663866

fs/btrfs/tests/btrfs-tests.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ struct btrfs_device *btrfs_alloc_dummy_device(struct btrfs_fs_info *fs_info)
102102
if (!dev)
103103
return ERR_PTR(-ENOMEM);
104104

105-
extent_io_tree_init(fs_info, &dev->alloc_state, 0);
105+
btrfs_extent_io_tree_init(fs_info, &dev->alloc_state, 0);
106106
INIT_LIST_HEAD(&dev->dev_list);
107107
list_add(&dev->dev_list, &fs_info->fs_devices->devices);
108108

@@ -111,7 +111,7 @@ struct btrfs_device *btrfs_alloc_dummy_device(struct btrfs_fs_info *fs_info)
111111

112112
static void btrfs_free_dummy_device(struct btrfs_device *dev)
113113
{
114-
extent_io_tree_release(&dev->alloc_state);
114+
btrfs_extent_io_tree_release(&dev->alloc_state);
115115
kfree(dev);
116116
}
117117

fs/btrfs/tests/extent-io-tests.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ static int test_find_delalloc(u32 sectorsize, u32 nodesize)
148148
* Passing NULL as we don't have fs_info but tracepoints are not used
149149
* at this point
150150
*/
151-
extent_io_tree_init(NULL, tmp, IO_TREE_SELFTEST);
151+
btrfs_extent_io_tree_init(NULL, tmp, IO_TREE_SELFTEST);
152152

153153
/*
154154
* First go through and create and mark all of our pages dirty, we pin
@@ -563,7 +563,7 @@ static int test_find_first_clear_extent_bit(void)
563563

564564
test_msg("running find_first_clear_extent_bit test");
565565

566-
extent_io_tree_init(NULL, &tree, IO_TREE_SELFTEST);
566+
btrfs_extent_io_tree_init(NULL, &tree, IO_TREE_SELFTEST);
567567

568568
/* Test correct handling of empty tree */
569569
btrfs_find_first_clear_extent_bit(&tree, 0, &start, &end, CHUNK_TRIMMED);

fs/btrfs/transaction.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,7 @@ static noinline void switch_commit_roots(struct btrfs_trans_handle *trans)
197197
list_del_init(&root->dirty_list);
198198
free_extent_buffer(root->commit_root);
199199
root->commit_root = btrfs_root_node(root);
200-
extent_io_tree_release(&root->dirty_log_pages);
200+
btrfs_extent_io_tree_release(&root->dirty_log_pages);
201201
btrfs_qgroup_clean_swapped_blocks(root);
202202
}
203203

@@ -383,10 +383,10 @@ static noinline int join_transaction(struct btrfs_fs_info *fs_info,
383383
INIT_LIST_HEAD(&cur_trans->deleted_bgs);
384384
spin_lock_init(&cur_trans->dropped_roots_lock);
385385
list_add_tail(&cur_trans->list, &fs_info->trans_list);
386-
extent_io_tree_init(fs_info, &cur_trans->dirty_pages,
387-
IO_TREE_TRANS_DIRTY_PAGES);
388-
extent_io_tree_init(fs_info, &cur_trans->pinned_extents,
389-
IO_TREE_FS_PINNED_EXTENTS);
386+
btrfs_extent_io_tree_init(fs_info, &cur_trans->dirty_pages,
387+
IO_TREE_TRANS_DIRTY_PAGES);
388+
btrfs_extent_io_tree_init(fs_info, &cur_trans->pinned_extents,
389+
IO_TREE_FS_PINNED_EXTENTS);
390390
btrfs_set_fs_generation(fs_info, fs_info->generation + 1);
391391
cur_trans->transid = fs_info->generation;
392392
fs_info->running_transaction = cur_trans;
@@ -1265,7 +1265,7 @@ static int btrfs_write_and_wait_transaction(struct btrfs_trans_handle *trans)
12651265
blk_finish_plug(&plug);
12661266
ret2 = btrfs_wait_extents(fs_info, dirty_pages);
12671267

1268-
extent_io_tree_release(&trans->transaction->dirty_pages);
1268+
btrfs_extent_io_tree_release(&trans->transaction->dirty_pages);
12691269

12701270
if (ret)
12711271
return ret;

fs/btrfs/tree-log.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3251,8 +3251,8 @@ static void free_log_tree(struct btrfs_trans_handle *trans,
32513251
}
32523252
}
32533253

3254-
extent_io_tree_release(&log->dirty_log_pages);
3255-
extent_io_tree_release(&log->log_csum_range);
3254+
btrfs_extent_io_tree_release(&log->dirty_log_pages);
3255+
btrfs_extent_io_tree_release(&log->log_csum_range);
32563256

32573257
btrfs_put_root(log);
32583258
}

fs/btrfs/volumes.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -404,7 +404,7 @@ static void btrfs_free_device(struct btrfs_device *device)
404404
{
405405
WARN_ON(!list_empty(&device->post_commit_list));
406406
rcu_string_free(device->name);
407-
extent_io_tree_release(&device->alloc_state);
407+
btrfs_extent_io_tree_release(&device->alloc_state);
408408
btrfs_destroy_dev_zone_info(device);
409409
kfree(device);
410410
}
@@ -1225,7 +1225,7 @@ static void btrfs_close_one_device(struct btrfs_device *device)
12251225

12261226
device->fs_info = NULL;
12271227
atomic_set(&device->dev_stats_ccnt, 0);
1228-
extent_io_tree_release(&device->alloc_state);
1228+
btrfs_extent_io_tree_release(&device->alloc_state);
12291229

12301230
/*
12311231
* Reset the flush error record. We might have a transient flush error
@@ -6952,7 +6952,7 @@ struct btrfs_device *btrfs_alloc_device(struct btrfs_fs_info *fs_info,
69526952

69536953
atomic_set(&dev->dev_stats_ccnt, 0);
69546954
btrfs_device_data_ordered_init(dev);
6955-
extent_io_tree_init(fs_info, &dev->alloc_state, IO_TREE_DEVICE_ALLOC_STATE);
6955+
btrfs_extent_io_tree_init(fs_info, &dev->alloc_state, IO_TREE_DEVICE_ALLOC_STATE);
69566956

69576957
if (devid)
69586958
tmp = *devid;

0 commit comments

Comments
 (0)