mbox series

[0/2] lib/reorder: fix drain/free issues

Message ID 20230107151939.2540148-1-vfialko@marvell.com (mailing list archive)
Headers
Series lib/reorder: fix drain/free issues |

Message

Volodymyr Fialko Jan. 7, 2023, 3:19 p.m. UTC 
  This patch address issues with reorder drain/free,
discovered with enabled `RTE_LIBRTE_MEMPOOL_DEBUG`.

Volodymyr Fialko (2):
  reorder: invalidate buf from ready queue in drain
  test/reorder: fix double free of drained buffers

 app/test/test_reorder.c   | 2 ++
 lib/reorder/rte_reorder.c | 1 +
 2 files changed, 3 insertions(+)

Comments

David Marchand Jan. 26, 2023, 3:47 p.m. UTC | #1 
Hi Reshma,

On Sat, Jan 7, 2023 at 4:20 PM Volodymyr Fialko <vfialko@marvell.com> wrote:
>
> This patch address issues with reorder drain/free,
> discovered with enabled `RTE_LIBRTE_MEMPOOL_DEBUG`.
>
> Volodymyr Fialko (2):
>   reorder: invalidate buf from ready queue in drain
>   test/reorder: fix double free of drained buffers
>
>  app/test/test_reorder.c   | 2 ++
>  lib/reorder/rte_reorder.c | 1 +
>  2 files changed, 3 insertions(+)

Please review, thanks.
Volodymyr Fialko Feb. 6, 2023, 3:47 p.m. UTC | #2 
Hi All,

Kind reminder to all maintainers, please review and ack/comment.

> This patch address issues with reorder drain/free, discovered with enabled
> `RTE_LIBRTE_MEMPOOL_DEBUG`.
Thomas Monjalon Feb. 7, 2023, 11:18 a.m. UTC | #3 
Pïng for review

26/01/2023 16:47, David Marchand:
> Hi Reshma,
> 
> On Sat, Jan 7, 2023 at 4:20 PM Volodymyr Fialko <vfialko@marvell.com> wrote:
> >
> > This patch address issues with reorder drain/free,
> > discovered with enabled `RTE_LIBRTE_MEMPOOL_DEBUG`.
> >
> > Volodymyr Fialko (2):
> >   reorder: invalidate buf from ready queue in drain
> >   test/reorder: fix double free of drained buffers
> >
> >  app/test/test_reorder.c   | 2 ++
> >  lib/reorder/rte_reorder.c | 1 +
> >  2 files changed, 3 insertions(+)
> 
> Please review, thanks.
Thomas Monjalon Feb. 19, 2023, 11:19 p.m. UTC | #4 
07/01/2023 16:19, Volodymyr Fialko:
> This patch address issues with reorder drain/free,
> discovered with enabled `RTE_LIBRTE_MEMPOOL_DEBUG`.
> 
> Volodymyr Fialko (2):
>   reorder: invalidate buf from ready queue in drain
>   test/reorder: fix double free of drained buffers

Applied, thanks.