btrfs: Remove 'tree' argument from read_extent_buffer_pages
authorNikolay Borisov <nborisov@suse.com>
Wed, 10 Apr 2019 14:24:40 +0000 (17:24 +0300)
committerDavid Sterba <dsterba@suse.com>
Mon, 29 Apr 2019 17:02:46 +0000 (19:02 +0200)
This function always uses the btree inode's io_tree. Stop taking the
tree as a function argument and instead access it internally from
read_extent_buffer_pages. No functional changes.

Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/disk-io.c
fs/btrfs/extent_io.c
fs/btrfs/extent_io.h

index d41511a05f2eef3c5f5049591230f679dd6b4009..62983ff0c3e175623d2206527cc365877f4cf1cf 100644 (file)
@@ -465,8 +465,7 @@ static int btree_read_extent_buffer_pages(struct extent_buffer *eb,
        io_tree = &BTRFS_I(fs_info->btree_inode)->io_tree;
        while (1) {
                clear_bit(EXTENT_BUFFER_CORRUPT, &eb->bflags);
-               ret = read_extent_buffer_pages(io_tree, eb, WAIT_COMPLETE,
-                                              mirror_num);
+               ret = read_extent_buffer_pages(eb, WAIT_COMPLETE, mirror_num);
                if (!ret) {
                        if (verify_parent_transid(io_tree, eb,
                                                   parent_transid, 0))
@@ -1034,15 +1033,13 @@ static const struct address_space_operations btree_aops = {
 void readahead_tree_block(struct btrfs_fs_info *fs_info, u64 bytenr)
 {
        struct extent_buffer *buf = NULL;
-       struct inode *btree_inode = fs_info->btree_inode;
        int ret;
 
        buf = btrfs_find_create_tree_block(fs_info, bytenr);
        if (IS_ERR(buf))
                return;
 
-       ret = read_extent_buffer_pages(&BTRFS_I(btree_inode)->io_tree, buf,
-                       WAIT_NONE, 0);
+       ret = read_extent_buffer_pages(buf, WAIT_NONE, 0);
        if (ret < 0)
                free_extent_buffer_stale(buf);
        else
@@ -1053,8 +1050,6 @@ int reada_tree_block_flagged(struct btrfs_fs_info *fs_info, u64 bytenr,
                         int mirror_num, struct extent_buffer **eb)
 {
        struct extent_buffer *buf = NULL;
-       struct inode *btree_inode = fs_info->btree_inode;
-       struct extent_io_tree *io_tree = &BTRFS_I(btree_inode)->io_tree;
        int ret;
 
        buf = btrfs_find_create_tree_block(fs_info, bytenr);
@@ -1063,8 +1058,7 @@ int reada_tree_block_flagged(struct btrfs_fs_info *fs_info, u64 bytenr,
 
        set_bit(EXTENT_BUFFER_READAHEAD, &buf->bflags);
 
-       ret = read_extent_buffer_pages(io_tree, buf, WAIT_PAGE_LOCK,
-                                      mirror_num);
+       ret = read_extent_buffer_pages(buf, WAIT_PAGE_LOCK, mirror_num);
        if (ret) {
                free_extent_buffer_stale(buf);
                return ret;
index 828708f6510c37372550006027a639dc3dcbbdc6..048d6c6fe7b9bd815d47ee3ed3df26fa7ccad6b2 100644 (file)
@@ -5330,8 +5330,7 @@ void set_extent_buffer_uptodate(struct extent_buffer *eb)
        }
 }
 
-int read_extent_buffer_pages(struct extent_io_tree *tree,
-                            struct extent_buffer *eb, int wait, int mirror_num)
+int read_extent_buffer_pages(struct extent_buffer *eb, int wait, int mirror_num)
 {
        int i;
        struct page *page;
@@ -5343,6 +5342,7 @@ int read_extent_buffer_pages(struct extent_io_tree *tree,
        unsigned long num_reads = 0;
        struct bio *bio = NULL;
        unsigned long bio_flags = 0;
+       struct extent_io_tree *tree = &BTRFS_I(eb->fs_info->btree_inode)->io_tree;
 
        if (test_bit(EXTENT_BUFFER_UPTODATE, &eb->bflags))
                return 0;
index abafb48947efd1110e7ddff368b3a90c5ab3c69d..37240e03c4e3935b31a724f7a2a8e81f83b1a7d9 100644 (file)
@@ -433,8 +433,7 @@ void free_extent_buffer_stale(struct extent_buffer *eb);
 #define WAIT_NONE      0
 #define WAIT_COMPLETE  1
 #define WAIT_PAGE_LOCK 2
-int read_extent_buffer_pages(struct extent_io_tree *tree,
-                            struct extent_buffer *eb, int wait,
+int read_extent_buffer_pages(struct extent_buffer *eb, int wait,
                             int mirror_num);
 void wait_on_extent_buffer_writeback(struct extent_buffer *eb);