btrfs: make btrfs_free_reserved_data_space_noquota take btrfs_fs_info
authorNikolay Borisov <nborisov@suse.com>
Wed, 3 Jun 2020 05:55:38 +0000 (08:55 +0300)
committerDavid Sterba <dsterba@suse.com>
Mon, 27 Jul 2020 10:55:35 +0000 (12:55 +0200)
No point in taking an inode only to get btrfs_fs_info from it, instead
take btrfs_fs_info directly.

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/delalloc-space.c
fs/btrfs/delalloc-space.h
fs/btrfs/inode.c
fs/btrfs/relocation.c

index d3c43e4..fc7496c 100644 (file)
@@ -255,7 +255,7 @@ int btrfs_check_data_free_space(struct inode *inode,
        /* Use new btrfs_qgroup_reserve_data to reserve precious data space. */
        ret = btrfs_qgroup_reserve_data(BTRFS_I(inode), reserved, start, len);
        if (ret < 0)
-               btrfs_free_reserved_data_space_noquota(inode, len);
+               btrfs_free_reserved_data_space_noquota(fs_info, len);
        else
                ret = 0;
        return ret;
@@ -269,10 +269,9 @@ int btrfs_check_data_free_space(struct inode *inode,
  * which we can't sleep and is sure it won't affect qgroup reserved space.
  * Like clear_bit_hook().
  */
-void btrfs_free_reserved_data_space_noquota(struct inode *inode,
+void btrfs_free_reserved_data_space_noquota(struct btrfs_fs_info *fs_info,
                                            u64 len)
 {
-       struct btrfs_fs_info *fs_info = btrfs_sb(inode->i_sb);
        struct btrfs_space_info *data_sinfo;
 
        ASSERT(IS_ALIGNED(len, fs_info->sectorsize));
@@ -300,7 +299,7 @@ void btrfs_free_reserved_data_space(struct inode *inode,
              round_down(start, root->fs_info->sectorsize);
        start = round_down(start, root->fs_info->sectorsize);
 
-       btrfs_free_reserved_data_space_noquota(inode, len);
+       btrfs_free_reserved_data_space_noquota(root->fs_info, len);
        btrfs_qgroup_free_data(BTRFS_I(inode), reserved, start, len);
 }
 
index fe8c6aa..bf3beec 100644 (file)
@@ -13,7 +13,7 @@ void btrfs_free_reserved_data_space(struct inode *inode,
 void btrfs_delalloc_release_space(struct inode *inode,
                                  struct extent_changeset *reserved,
                                  u64 start, u64 len, bool qgroup_free);
-void btrfs_free_reserved_data_space_noquota(struct inode *inode,
+void btrfs_free_reserved_data_space_noquota(struct btrfs_fs_info *fs_info,
                                            u64 len);
 void btrfs_delalloc_release_metadata(struct btrfs_inode *inode, u64 num_bytes,
                                     bool qgroup_free);
index 0d711e5..f700e38 100644 (file)
@@ -2080,9 +2080,7 @@ void btrfs_clear_delalloc_extent(struct inode *vfs_inode,
                if (root->root_key.objectid != BTRFS_DATA_RELOC_TREE_OBJECTID &&
                    do_list && !(state->state & EXTENT_NORESERVE) &&
                    (*bits & EXTENT_CLEAR_DATA_RESV))
-                       btrfs_free_reserved_data_space_noquota(
-                                       &inode->vfs_inode,
-                                       len);
+                       btrfs_free_reserved_data_space_noquota(fs_info, len);
 
                percpu_counter_add_batch(&fs_info->delalloc_bytes, -len,
                                         fs_info->delalloc_batch);
@@ -7312,7 +7310,7 @@ static int btrfs_get_blocks_direct_write(struct extent_map **map,
                         * use the existing or preallocated extent, so does not
                         * need to adjust btrfs_space_info's bytes_may_use.
                         */
-                       btrfs_free_reserved_data_space_noquota(inode, len);
+                       btrfs_free_reserved_data_space_noquota(fs_info, len);
                        goto skip_cow;
                }
        }
index 73300c7..4a2959e 100644 (file)
@@ -2613,7 +2613,7 @@ int prealloc_file_extent_cluster(struct inode *inode,
        inode_unlock(inode);
 
        if (cur_offset < prealloc_end)
-               btrfs_free_reserved_data_space_noquota(inode,
+               btrfs_free_reserved_data_space_noquota(btrfs_sb(inode->i_sb),
                                               prealloc_end + 1 - cur_offset);
        return ret;
 }