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

2024-04-30 Thread Kumar Kartikeya Dwivedi
On Tue, 30 Apr 2024 at 12:43, Benjamin Tissoires 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

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

2024-04-30 Thread Benjamin Tissoires
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: Benjami