On Mon, Mar 25, 2019 at 02:31:26PM +0200, Nikolay Borisov wrote:
> Rather than hijacking the existing defines let's just define new bits,
> with more descriptive names. Instead of using yet more (currently at 18)
> bits for the new flags, use the fact those flags will be specific to
> the device allocation tree so define them using existing EXTENT_* flags.
> 
> Signed-off-by: Nikolay Borisov <nbori...@suse.com>
> ---
>  fs/btrfs/disk-io.c   | 2 +-
>  fs/btrfs/extent_io.h | 4 ++++
>  2 files changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c
> index 80f0787eb278..81183694f8ea 100644
> --- a/fs/btrfs/disk-io.c
> +++ b/fs/btrfs/disk-io.c
> @@ -4055,8 +4055,8 @@ void close_ctree(struct btrfs_fs_info *fs_info)
>               btrfsic_unmount(fs_info->fs_devices);
>  #endif
>  
> -     btrfs_close_devices(fs_info->fs_devices);
>       btrfs_mapping_tree_free(&fs_info->mapping_tree);
> +     btrfs_close_devices(fs_info->fs_devices);

Have you squashed this change to the wrong commit? In v2 there was a
separate patch but mentioning the following one ("replace pending/pinned
chunks lists with io tree"). I'm going to drop this hunk and use the
separate patch as it makes it more clear.

Reply via email to