RETRACTED [PATCH v4] btrfs: Turn delayed_nodes_tree into an XArray

[Gabriel Niebler <gniebler@suse.com>]

Disregard this patch, please, it has a syntax error and won't even 
compile...

Am 19.04.22 um 15:37 schrieb Gabriel Niebler:
> … in the btrfs_root struct and adjust all usages of this object to use the
> XArray API, because it is notionally easier to use and unserstand, as it
> provides array semantics, and also takes care of locking for us, further
> simplifying the code.
> 
> Signed-off-by: Gabriel Niebler <gniebler@suse.com>

<snip>

> @@ -128,36 +128,29 @@ static struct btrfs_delayed_node *btrfs_get_or_create_delayed_node(
>   	u64 ino = btrfs_ino(btrfs_inode);
>   	int ret;
>   
> -again:
> -	node = btrfs_get_delayed_node(btrfs_inode);
> -	if (node)
> -		return node;
> -
> -	node = kmem_cache_zalloc(delayed_node_cache, GFP_NOFS);
> -	if (!node)
> -		return ERR_PTR(-ENOMEM);
> -	btrfs_init_delayed_node(node, root, ino);
> +        do {
> +		node = btrfs_get_delayed_node(btrfs_inode);
> +		if (node)
> +			return node;
>   
> -	/* cached in the btrfs inode and can be accessed */
> -	refcount_set(&node->refs, 2);
> +		node = kmem_cache_zalloc(delayed_node_cache, GFP_NOFS);
> +		if (!node)
> +			return ERR_PTR(-ENOMEM);
> +		btrfs_init_delayed_node(node, root, ino);
>   
> -	ret = radix_tree_preload(GFP_NOFS);
> -	if (ret) {
> -		kmem_cache_free(delayed_node_cache, node);
> -		return ERR_PTR(ret);
> -	}
> +		/* cached in the btrfs inode and can be accessed */
> +		refcount_set(&node->refs, 2);
>   
> -	spin_lock(&root->inode_lock);
> -	ret = radix_tree_insert(&root->delayed_nodes_tree, ino, node);
> -	if (ret == -EEXIST) {
> -		spin_unlock(&root->inode_lock);
> -		kmem_cache_free(delayed_node_cache, node);
> -		radix_tree_preload_end();
> -		goto again;
> -	}
> +		spin_lock(&root->inode_lock);
> +		ret = xa_insert(&root->delayed_nodes, ino, node, GFP_NOFS);
> +		if (ret) {
> +			spin_unlock(&root->inode_lock);
> +			kmem_cache_free(delayed_node_cache, node);
> +			if (ret != -EBUSY)
> +				return ERR_PTR(ret);
> +	} while (ret);

Here it is: missing "}"!

<snip>

Resubmission pending, sorry for the inconvenience