Re: [PATCH 00/17] Freespace tree big fs_info cleanup

[David Sterba]

On Thu, May 10, 2018 at 03:44:39PM +0300, Nikolay Borisov wrote:
> Here is a series which cleans _all_ freespace tree functions from a redundant
> fs_info argument since they already take either a transaction or a 
> block_group_cache structure. Both of those structures contain a reference to 
> fs info and can be used instead of an additional parameter. This is needed 
> since I will be pulling some of the freespace tree code into btrfs-progs in 
> implementing check/rebuild functionality for the freespace tree. So better 
> have
> this sooner rather than later. 
> 
> This series should bring no functional changes but just in case it passed the 
> btrfs' selftests as well as a full xfstest run. 
> 
> Nikolay Borisov (17):
>   btrfs: Make btrfs_init_dummy_trans initialize trans' fs_info field
>   btrfs: Remove fs_info argument from add_block_group_free_space
>   btrfs: Remove fs_info argument from __add_block_group_free_space
>   btrfs: Remove fs_info argument from __add_to_free_space_tree
>   btrfs: Remove fs_info parameter from add_new_free_space_info
>   btrfs: Remove fs_info argument from add_new_free_space
>   btrfs: Remove fs_info parameter from remove_block_group_free_space
>   btrfs: Remove fs_info argument from convert_free_space_to_bitmaps
>   btrfs: Remove fs_info parameter from convert_free_space_to_extents
>   btrfs: Remove fs_info argument from update_free_space_extent_count
>   btrfs: Remove fs_info argument from modify_free_space_bitmap
>   btrfs: Remove fs_info argument from add_free_space_extent
>   btrfs: Remove fs_info argument from remove_free_space_extent
>   btrfs: Remove fs_info argument from __remove_from_free_space_tree
>   btrfs: Remove fs_info argument from remove_from_free_space_tree
>   btrfs: Remove fs_info argument from add_to_free_space_tree
>   btrfs: Remove fs_info argument from populate_free_space_tree

All
Reviewed-by: David Sterba 

and added to misc-next, thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 00/17] Freespace tree big fs_info cleanup

[Nikolay Borisov]

Here is a series which cleans _all_ freespace tree functions from a redundant
fs_info argument since they already take either a transaction or a 
block_group_cache structure. Both of those structures contain a reference to 
fs info and can be used instead of an additional parameter. This is needed 
since I will be pulling some of the freespace tree code into btrfs-progs in 
implementing check/rebuild functionality for the freespace tree. So better have
this sooner rather than later. 

This series should bring no functional changes but just in case it passed the 
btrfs' selftests as well as a full xfstest run. 

Nikolay Borisov (17):
  btrfs: Make btrfs_init_dummy_trans initialize trans' fs_info field
  btrfs: Remove fs_info argument from add_block_group_free_space
  btrfs: Remove fs_info argument from __add_block_group_free_space
  btrfs: Remove fs_info argument from __add_to_free_space_tree
  btrfs: Remove fs_info parameter from add_new_free_space_info
  btrfs: Remove fs_info argument from add_new_free_space
  btrfs: Remove fs_info parameter from remove_block_group_free_space
  btrfs: Remove fs_info argument from convert_free_space_to_bitmaps
  btrfs: Remove fs_info parameter from convert_free_space_to_extents
  btrfs: Remove fs_info argument from update_free_space_extent_count
  btrfs: Remove fs_info argument from modify_free_space_bitmap
  btrfs: Remove fs_info argument from add_free_space_extent
  btrfs: Remove fs_info argument from remove_free_space_extent
  btrfs: Remove fs_info argument from __remove_from_free_space_tree
  btrfs: Remove fs_info argument from remove_from_free_space_tree
  btrfs: Remove fs_info argument from add_to_free_space_tree
  btrfs: Remove fs_info argument from populate_free_space_tree

 fs/btrfs/ctree.h   |   2 +-
 fs/btrfs/extent-tree.c |  25 ---
 fs/btrfs/free-space-tree.c | 115 ++---
 fs/btrfs/free-space-tree.h |   8 ---
 fs/btrfs/tests/btrfs-tests.c   |   4 +-
 fs/btrfs/tests/btrfs-tests.h   |   3 +-
 fs/btrfs/tests/free-space-tree-tests.c |  55 
 fs/btrfs/tests/qgroup-tests.c  |  12 ++--
 8 files changed, 99 insertions(+), 125 deletions(-)

-- 
2.7.4

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html