Hi, A gentle reminder, please review and ack/comment. Can we have this merged before RC3?
> -----Original Message----- > From: Volodymyr Fialko <[email protected]> > Sent: Monday, March 13, 2023 2:04 PM > To: [email protected]; Reshma Pattan <[email protected]>; David Marchand > <[email protected]>; Andrew Rybchenko <[email protected]> > Cc: Jerin Jacob Kollanukkaran <[email protected]>; Anoob Joseph > <[email protected]>; Volodymyr > Fialko <[email protected]>; [email protected] > 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: [email protected] > > Signed-off-by: Volodymyr Fialko <[email protected]> > --- > lib/reorder/rte_reorder.c | 94 +++++++++++++++++++++++++-------------- > lib/reorder/rte_reorder.h | 1 + > 2 files changed, 61 insertions(+), 34 deletions(-) <snip>
