Re: [PATCH] btrfs-progs: remove data extents from the free space tree

[Qu Wenruo <quwenruo.btrfs@gmx.com>]

On 2021/10/6 02:21, Josef Bacik wrote:
> Dave reported a failure of mkfs-test 009 with the free space tree
> enabled by default.  This is because 009 pre-populates the file system
> with a given directory, and for some reason our data allocation path
> isn't the same as in the kernel.

Mind to share the exact difference between btrfs-progs and kernel?

Just hope we don't miss anything.

Thanks,
Qu

>  Fix this by making sure when we
> allocate a data extent we remove the space from the free space tree, and
> with this our mkfs tests now pass.
>
> Issue: #410
> Signed-off-by: Josef Bacik <josef@toxicpanda.com>
> ---
>   kernel-shared/extent-tree.c | 6 ++++++
>   1 file changed, 6 insertions(+)
>
> diff --git a/kernel-shared/extent-tree.c b/kernel-shared/extent-tree.c
> index 4b79874d..90733d55 100644
> --- a/kernel-shared/extent-tree.c
> +++ b/kernel-shared/extent-tree.c
> @@ -3483,6 +3483,12 @@ static int __btrfs_record_file_extent(struct btrfs_trans_handle *trans,
>   		} else if (ret != -EEXIST) {
>   			goto fail;
>   		}
> +
> +		ret = remove_from_free_space_tree(trans, disk_bytenr,
> +						  num_bytes);
> +		if (ret)
> +			goto fail;
> +
>   		btrfs_run_delayed_refs(trans, -1);
>   		extent_bytenr = disk_bytenr;
>   		extent_num_bytes = num_bytes;
>