[EXT] Re: [PATCH] reorder: fix registration of dynamic field in mbuf
Volodymyr Fialko
vfialko at marvell.com
Mon Mar 13 14:08:40 CET 2023
> All rte_reorder_buffer objects need to go through rte_reorder_init().
> You can check rte_reorder_init() return code.
Hi David,
I agree with all comments, however there is one catch with locks.
Both reorder lib and mbuf dynamic field are using `rte_mcfg_tailq` read/write lock for synchronization, so
I moved alloc+init before lock+insert to avoid deadlocking in v2. Let me know if you see any issues with this approach.
More information about the dev
mailing list