Re: [PATCH v5 1/3] btrfs: allow btrfs_truncate_block() to fallback to nocow for data space reservation

[Anand Jain <anand.jain@xxxxxxxxxx>]

Looks good.
Just a nit below.
Reviewed-by: Anand Jain <anand.jain@xxxxxxxxxx>

> +	if (ret) {
> +		if (!only_release_metadata)
> +			btrfs_delalloc_release_space(inode, data_reserved,
> +					block_start, blocksize, true);
> +		else
> +			btrfs_delalloc_release_metadata(BTRFS_I(inode),
> +					blocksize, true);
> +	}

Nit: (May be it can be fixed during integration so no need to send 
another revision.) here the ! operation inside if can be removed.

if (only_release_metadata)
   btrfs_delalloc_release_metadata()
else
   btrfs_delalloc_release_space()

Thanks, Anand