Re: [PATCH 2/2] testing/radix-tree/maple: Hack around kfree_rcu not existing
From: Lorenzo Stoakes
Date: Sun Aug 10 2025 - 10:24:45 EST
On Fri, Jul 18, 2025 at 06:21:37PM +0100, Pedro Falcato wrote:
> liburcu doesn't have kfree_rcu (or anything similar). Despite that, we
> can hack around it in a trivial fashion, by adding a wrapper.
>
> This wrapper only works for maple_nodes, and not anything else (due to
> us not being able to know rcu_head offsets in any way), and thus we take
> advantage of the type checking to avoid future silent breakage.
>
> Signed-off-by: Pedro Falcato <pfalcato@xxxxxxx>
So, patch 1/2 of this series has broken the VMA tests for the same reason.
Let's fix this by putting it in a shared place.
The fix enclosed works for me (assuming the patch is reverted).
Cheers, Lorenzo
> ---
> tools/testing/radix-tree/maple.c | 17 +++++++++++++++++
> 1 file changed, 17 insertions(+)
>
> diff --git a/tools/testing/radix-tree/maple.c b/tools/testing/radix-tree/maple.c
> index 172700fb7784..bfdc93c36cf9 100644
> --- a/tools/testing/radix-tree/maple.c
> +++ b/tools/testing/radix-tree/maple.c
> @@ -23,6 +23,23 @@
> #define MODULE_LICENSE(x)
> #define dump_stack() assert(0)
>
> +
> +#include <linux/maple_tree.h>
> +
> +static void free_node(struct rcu_head *head)
> +{
> + struct maple_node *node = container_of(head, struct maple_node, rcu);
> +
> + free(node);
> +}
> +
> +static void kfree_rcu_node(struct maple_node *node)
> +{
> + call_rcu(&node->rcu, free_node);
> +}
> +
> +#define kfree_rcu(ptr, memb) kfree_rcu_node(ptr)
> +
> #include "../../../lib/maple_tree.c"
> #include "../../../lib/test_maple_tree.c"
>
> --
> 2.50.1
>
>
----8<----