Hi,
A gentle reminder, please review and ack/comment.
Can we have this merged before RC3?
> -----Original Message-----
> From: Volodymyr Fialko <vfia...@marvell.com>
> Sent: Monday, March 13, 2023 2:04 PM
> To: dev@dpdk.org; Reshma Pattan <reshma.pat...@intel.com>; David Marchand
> <david.march...@redhat.com>; Andrew Rybchenko <andrew.rybche...@oktetlabs.ru>
> Cc: Jerin Jacob Kollanukkaran <jer...@marvell.com>; Anoob Joseph
> <ano...@marvell.com>; Volodymyr
> Fialko <vfia...@marvell.com>; sta...@dpdk.org
> Subject: [PATCH v2] reorder: fix registration of dynamic field in mbuf
>
> It's possible to initialize reorder buffer with user allocated memory via
> rte_reorder_init() function. In such case rte_reorder_create() is not
> required and reorder dynamic field in
> rte_mbuf will not be registered.
>
> Both reorder lib and mbuf dynamic field are using `rte_mcfg_tailq` read/write
> lock for synchronization, to
> avoid deadlocking move reorder buffer initialization before queue insertion.
>
> Fixes: 01f3496695b5 ("reorder: switch sequence number to dynamic mbuf field")
> Cc: sta...@dpdk.org
>
> Signed-off-by: Volodymyr Fialko <vfia...@marvell.com>
> ---
> lib/reorder/rte_reorder.c | 94 +++++++++++++++++++++++++--------------
> lib/reorder/rte_reorder.h | 1 +
> 2 files changed, 61 insertions(+), 34 deletions(-)
<snip>
