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 <nborisov@xxxxxxxx>
> ---
> 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.