Re: [PATCH bpf-next v3 1/3] bpf: do not walk twice the map on free

From: Kumar Kartikeya Dwivedi
Date: Tue Apr 30 2024 - 08:38:01 EST


On Tue, 30 Apr 2024 at 12:43, Benjamin Tissoires <bentiss@xxxxxxxxxx> wrote:
>
> If someone stores both a timer and a workqueue in a map, on free we
> would walk it twice.
> Add a check in array_map_free_timers_wq and free the timers
> and workqueues if they are present.
>
> Fixes: 246331e3f1ea ("bpf: allow struct bpf_wq to be embedded in arraymaps and hashmaps")
> Signed-off-by: Benjamin Tissoires <bentiss@xxxxxxxxxx>
>
> ---
>
> no changes in v3
>
> no changes in v2
> ---

Acked-by: Kumar Kartikeya Dwivedi <memxor@xxxxxxxxx>

> [...]