Re: [PATCH 0/7] More btrfs_path auto cleaning

Daniel Vacek
2025-04-03T08:01:30Z

On Wed, 2 Apr 2025 at 01:18, David Sterba <dsterba@suse.com> wrote:
>
> A few more conversions to automatic freeing of btrfs_path, same
> separation as last time, first patch with the trivial ones and separated
> when there are goto/return conversion.
>
> David Sterba (7):
> btrfs: do more trivial BTRFS_PATH_AUTO_FREE conversions
> btrfs: use BTRFS_PATH_AUTO_FREE in may_destroy_subvol()
> btrfs: use BTRFS_PATH_AUTO_FREE in btrfs_set_inode_index_count()
> btrfs: use BTRFS_PATH_AUTO_FREE in can_nocow_extent()
> btrfs: use BTRFS_PATH_AUTO_FREE in btrfs_encoded_read_inline()
> btrfs: use BTRFS_PATH_AUTO_FREE in btrfs_del_inode_extref()
> btrfs: use BTRFS_PATH_AUTO_FREE in btrfs_insert_inode_extref()

With what David mentioned for 1/7 the series looks good to me.

Reviewed-by: Daniel Vacek <neelx@suse.com>

--nX

>
> fs/btrfs/block-group.c | 3 +-
> fs/btrfs/fiemap.c | 3 +-
> fs/btrfs/file-item.c | 3 +-
> fs/btrfs/file.c | 3 +-
> fs/btrfs/free-space-cache.c | 3 +-
> fs/btrfs/inode-item.c | 31 ++++-----
> fs/btrfs/inode.c | 123 ++++++++++++++----------------------
> 7 files changed, 65 insertions(+), 104 deletions(-)
>
> --
> 2.48.1
>
>