Re: [PATCH v2] reorder: fix registration of dynamic field in mbuf

2023-03-16 Thread David Marchand
On Mon, Mar 13, 2023 at 2:05 PM Volodymyr Fialko wrote: > > 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 li

Re: [PATCH v2] reorder: fix registration of dynamic field in mbuf

2023-03-16 Thread David Marchand
On Wed, Mar 15, 2023 at 4:47 PM Pattan, Reshma wrote: > > -Original Message- > > From: David Marchand > > > A gentle reminder, please review and ack/comment. > > > Can we have this merged before RC3? > > > > I did not go into depth this time, the fix seems ok. > > Reshma, please review. >

RE: [PATCH v2] reorder: fix registration of dynamic field in mbuf

2023-03-15 Thread Pattan, Reshma
> -Original Message- > From: David Marchand > > A gentle reminder, please review and ack/comment. > > Can we have this merged before RC3? > > I did not go into depth this time, the fix seems ok. > Reshma, please review. > > Hi David, Unfortunately do not have a bandwidth to review th

Re: [PATCH v2] reorder: fix registration of dynamic field in mbuf

2023-03-15 Thread David Marchand
On Wed, Mar 15, 2023 at 4:29 PM Volodymyr Fialko wrote: > > Hi, > > A gentle reminder, please review and ack/comment. > Can we have this merged before RC3? I did not go into depth this time, the fix seems ok. Reshma, please review. -- David Marchand

RE: [PATCH v2] reorder: fix registration of dynamic field in mbuf

2023-03-15 Thread Volodymyr Fialko
lanukkaran ; Anoob Joseph > ; Volodymyr > Fialko ; 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_reorde

[PATCH v2] reorder: fix registration of dynamic field in mbuf

2023-03-13 Thread Volodymyr Fialko
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

[PATCH v2] reorder: fix registration of dynamic field in mbuf

2023-03-13 Thread Volodymyr Fialko
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