btrfs: make btrfs_set_extent_delalloc take btrfs_inode
authorNikolay Borisov <nborisov@suse.com>
Wed, 3 Jun 2020 05:55:35 +0000 (08:55 +0300)
committerDavid Sterba <dsterba@suse.com>
Mon, 27 Jul 2020 10:55:35 +0000 (12:55 +0200)
Preparation to make btrfs_dirty_pages take btrfs_inode as parameter.

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/ctree.h
fs/btrfs/file.c
fs/btrfs/inode.c
fs/btrfs/reflink.c
fs/btrfs/relocation.c
fs/btrfs/tests/inode-tests.c

index 65e0630..794706f 100644 (file)
@@ -2926,7 +2926,7 @@ int btrfs_truncate_inode_items(struct btrfs_trans_handle *trans,
 
 int btrfs_start_delalloc_snapshot(struct btrfs_root *root);
 int btrfs_start_delalloc_roots(struct btrfs_fs_info *fs_info, int nr);
-int btrfs_set_extent_delalloc(struct inode *inode, u64 start, u64 end,
+int btrfs_set_extent_delalloc(struct btrfs_inode *inode, u64 start, u64 end,
                              unsigned int extra_bits,
                              struct extent_state **cached_state);
 int btrfs_create_subvol_root(struct btrfs_trans_handle *trans,
index 760ddc1..b6c9921 100644 (file)
@@ -546,7 +546,7 @@ int btrfs_dirty_pages(struct inode *inode, struct page **pages,
                }
        }
 
-       err = btrfs_set_extent_delalloc(inode, start_pos, end_of_last_block,
+       err = btrfs_set_extent_delalloc(BTRFS_I(inode), start_pos, end_of_last_block,
                                        extra_bits, cached);
        if (err)
                return err;
index 5abb6d0..0d711e5 100644 (file)
@@ -2260,13 +2260,13 @@ static noinline int add_pending_csums(struct btrfs_trans_handle *trans,
        return 0;
 }
 
-int btrfs_set_extent_delalloc(struct inode *inode, u64 start, u64 end,
+int btrfs_set_extent_delalloc(struct btrfs_inode *inode, u64 start, u64 end,
                              unsigned int extra_bits,
                              struct extent_state **cached_state)
 {
        WARN_ON(PAGE_ALIGNED(end));
-       return set_extent_delalloc(&BTRFS_I(inode)->io_tree, start, end,
-                                  extra_bits, cached_state);
+       return set_extent_delalloc(&inode->io_tree, start, end, extra_bits,
+                                  cached_state);
 }
 
 /* see btrfs_writepage_start_hook for details on why this is required */
@@ -2363,7 +2363,7 @@ again:
                goto again;
        }
 
-       ret = btrfs_set_extent_delalloc(inode, page_start, page_end, 0,
+       ret = btrfs_set_extent_delalloc(BTRFS_I(inode), page_start, page_end, 0,
                                        &cached_state);
        if (ret)
                goto out_reserved;
@@ -4581,7 +4581,7 @@ again:
                         EXTENT_DELALLOC | EXTENT_DO_ACCOUNTING | EXTENT_DEFRAG,
                         0, 0, &cached_state);
 
-       ret = btrfs_set_extent_delalloc(inode, block_start, block_end, 0,
+       ret = btrfs_set_extent_delalloc(BTRFS_I(inode), block_start, block_end, 0,
                                        &cached_state);
        if (ret) {
                unlock_extent_cached(io_tree, block_start, block_end,
@@ -8296,7 +8296,7 @@ again:
                          EXTENT_DELALLOC | EXTENT_DO_ACCOUNTING |
                          EXTENT_DEFRAG, 0, 0, &cached_state);
 
-       ret2 = btrfs_set_extent_delalloc(inode, page_start, end, 0,
+       ret2 = btrfs_set_extent_delalloc(BTRFS_I(inode), page_start, end, 0,
                                        &cached_state);
        if (ret2) {
                unlock_extent_cached(io_tree, page_start, page_end,
index 040009d..fe3e05b 100644 (file)
@@ -84,7 +84,8 @@ static int copy_inline_to_page(struct inode *inode,
        clear_extent_bit(&BTRFS_I(inode)->io_tree, file_offset, range_end,
                         EXTENT_DELALLOC | EXTENT_DO_ACCOUNTING | EXTENT_DEFRAG,
                         0, 0, NULL);
-       ret = btrfs_set_extent_delalloc(inode, file_offset, range_end, 0, NULL);
+       ret = btrfs_set_extent_delalloc(BTRFS_I(inode), file_offset, range_end,
+                                       0, NULL);
        if (ret)
                goto out_unlock;
 
index e3f3e2e..73300c7 100644 (file)
@@ -2754,8 +2754,8 @@ static int relocate_file_extent_cluster(struct inode *inode,
                        nr++;
                }
 
-               ret = btrfs_set_extent_delalloc(inode, page_start, page_end, 0,
-                                               NULL);
+               ret = btrfs_set_extent_delalloc(BTRFS_I(inode), page_start,
+                                               page_end, 0, NULL);
                if (ret) {
                        unlock_page(page);
                        put_page(page);
index 24a8c71..894a63a 100644 (file)
@@ -954,8 +954,8 @@ static int test_extent_accounting(u32 sectorsize, u32 nodesize)
        btrfs_test_inode_set_ops(inode);
 
        /* [BTRFS_MAX_EXTENT_SIZE] */
-       ret = btrfs_set_extent_delalloc(inode, 0, BTRFS_MAX_EXTENT_SIZE - 1, 0,
-                                       NULL);
+       ret = btrfs_set_extent_delalloc(BTRFS_I(inode), 0,
+                                       BTRFS_MAX_EXTENT_SIZE - 1, 0, NULL);
        if (ret) {
                test_err("btrfs_set_extent_delalloc returned %d", ret);
                goto out;
@@ -968,7 +968,7 @@ static int test_extent_accounting(u32 sectorsize, u32 nodesize)
        }
 
        /* [BTRFS_MAX_EXTENT_SIZE][sectorsize] */
-       ret = btrfs_set_extent_delalloc(inode, BTRFS_MAX_EXTENT_SIZE,
+       ret = btrfs_set_extent_delalloc(BTRFS_I(inode), BTRFS_MAX_EXTENT_SIZE,
                                        BTRFS_MAX_EXTENT_SIZE + sectorsize - 1,
                                        0, NULL);
        if (ret) {
@@ -999,7 +999,7 @@ static int test_extent_accounting(u32 sectorsize, u32 nodesize)
        }
 
        /* [BTRFS_MAX_EXTENT_SIZE][sectorsize] */
-       ret = btrfs_set_extent_delalloc(inode, BTRFS_MAX_EXTENT_SIZE >> 1,
+       ret = btrfs_set_extent_delalloc(BTRFS_I(inode), BTRFS_MAX_EXTENT_SIZE >> 1,
                                        (BTRFS_MAX_EXTENT_SIZE >> 1)
                                        + sectorsize - 1,
                                        0, NULL);
@@ -1017,7 +1017,7 @@ static int test_extent_accounting(u32 sectorsize, u32 nodesize)
        /*
         * [BTRFS_MAX_EXTENT_SIZE+sectorsize][sectorsize HOLE][BTRFS_MAX_EXTENT_SIZE+sectorsize]
         */
-       ret = btrfs_set_extent_delalloc(inode,
+       ret = btrfs_set_extent_delalloc(BTRFS_I(inode),
                        BTRFS_MAX_EXTENT_SIZE + 2 * sectorsize,
                        (BTRFS_MAX_EXTENT_SIZE << 1) + 3 * sectorsize - 1,
                        0, NULL);
@@ -1035,7 +1035,7 @@ static int test_extent_accounting(u32 sectorsize, u32 nodesize)
        /*
        * [BTRFS_MAX_EXTENT_SIZE+sectorsize][sectorsize][BTRFS_MAX_EXTENT_SIZE+sectorsize]
        */
-       ret = btrfs_set_extent_delalloc(inode,
+       ret = btrfs_set_extent_delalloc(BTRFS_I(inode),
                        BTRFS_MAX_EXTENT_SIZE + sectorsize,
                        BTRFS_MAX_EXTENT_SIZE + 2 * sectorsize - 1, 0, NULL);
        if (ret) {
@@ -1069,7 +1069,7 @@ static int test_extent_accounting(u32 sectorsize, u32 nodesize)
         * Refill the hole again just for good measure, because I thought it
         * might fail and I'd rather satisfy my paranoia at this point.
         */
-       ret = btrfs_set_extent_delalloc(inode,
+       ret = btrfs_set_extent_delalloc(BTRFS_I(inode),
                        BTRFS_MAX_EXTENT_SIZE + sectorsize,
                        BTRFS_MAX_EXTENT_SIZE + 2 * sectorsize - 1, 0, NULL);
        if (ret) {