[0/3] btrfs: minor block group cleanups

Message

Anand Jain June 3, 2020, 10:10 a.m. UTC

Minor cleanup patches.
1/3 cleansup the return value of btrfs_return_cluster_to_free_space().
2/3 and 3/3 together cleanups block group reference count.

2/3 and 3/3 are related. However 1/3 is independent to the rest of the
patches in this set.

Anand Jain (3):
  btrfs: fix btrfs_return_cluster_to_free_space() return
  btrfs: rename btrfs_block_group::count
  btrfs: use helper btrfs_get_block_group

 fs/btrfs/block-group.c      |  8 ++++----
 fs/btrfs/block-group.h      |  2 +-
 fs/btrfs/free-space-cache.c | 17 +++++++----------
 fs/btrfs/free-space-cache.h |  2 +-
 4 files changed, 13 insertions(+), 16 deletions(-)

Comments

David Sterba June 4, 2020, 4:56 p.m. UTC | #1

On Wed, Jun 03, 2020 at 06:10:17PM +0800, Anand Jain wrote:
> Minor cleanup patches.
> 1/3 cleansup the return value of btrfs_return_cluster_to_free_space().
> 2/3 and 3/3 together cleanups block group reference count.
> 
> 2/3 and 3/3 are related. However 1/3 is independent to the rest of the
> patches in this set.
> 
> Anand Jain (3):
>   btrfs: fix btrfs_return_cluster_to_free_space() return
>   btrfs: rename btrfs_block_group::count
>   btrfs: use helper btrfs_get_block_group

1 and 3 applied but I don't recommend to spend time on cleaning up
free-space-cache.[ch] as it's going to be phased out.