test/mbuf: fix mbuf_autotest retest fail
Checks
Commit Message
Retest "mbuf_autotest" will fail because the mbuf pool was
not freed after previous test successful done.
This patch fixes it.
Fixes: efc6f9104c80 ("mbuf: fix reset on mbuf free")
Cc: stable@dpdk.org
Signed-off-by: Jie Hai <haijie1@huawei.com>
---
app/test/test_mbuf.c | 1 +
1 file changed, 1 insertion(+)
Comments
On Tue, Jan 31, 2023 at 3:52 AM Jie Hai <haijie1@huawei.com> wrote:
>
> Retest "mbuf_autotest" will fail because the mbuf pool was
> not freed after previous test successful done.
> This patch fixes it.
Good catch.
>
> Fixes: efc6f9104c80 ("mbuf: fix reset on mbuf free")
> Cc: stable@dpdk.org
>
> Signed-off-by: Jie Hai <haijie1@huawei.com>
Reviewed-by: David Marchand <david.marchand@redhat.com>
On Tue, Feb 7, 2023 at 2:21 PM David Marchand <david.marchand@redhat.com> wrote:
> On Tue, Jan 31, 2023 at 3:52 AM Jie Hai <haijie1@huawei.com> wrote:
> >
> > Retest "mbuf_autotest" will fail because the mbuf pool was
> > not freed after previous test successful done.
> > This patch fixes it.
> >
> > Fixes: efc6f9104c80 ("mbuf: fix reset on mbuf free")
> > Cc: stable@dpdk.org
> >
> > Signed-off-by: Jie Hai <haijie1@huawei.com>
> Reviewed-by: David Marchand <david.marchand@redhat.com>
Applied, thanks.
@@ -2764,6 +2764,7 @@ test_nb_segs_and_next_reset(void)
m2->nb_segs != 1 || m2->next != NULL)
GOTO_FAIL("nb_segs or next was not reset properly");
+ rte_mempool_free(pool);
return 0;
fail: