patch 'test/mbuf: fix test with mbuf debug enabled' has been queued to stable release 22.11.2
Xueming Li
xuemingl at nvidia.com
Sun Apr 9 17:24:47 CEST 2023
Hi,
FYI, your patch has been queued to stable release 22.11.2
Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 04/11/23. So please
shout if anyone has objections.
Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.
Queued patches are on a temporary branch at:
https://git.dpdk.org/dpdk-stable/log/?h=22.11-staging
This queued commit can be viewed at:
https://git.dpdk.org/dpdk-stable/log/?h=22.11-staging/commit/29cdc2b88f04b2c459705a24274de32d53001fcf
Thanks.
Xueming Li <xuemingl at nvidia.com>
---
>From 29cdc2b88f04b2c459705a24274de32d53001fcf Mon Sep 17 00:00:00 2001
From: Pavel Ivashchenko <pivashchenko at nfware.com>
Date: Thu, 16 Mar 2023 22:14:56 +0300
Subject: [PATCH] test/mbuf: fix test with mbuf debug enabled
Cc: Xueming Li <xuemingl at nvidia.com>
[ upstream commit ce5440e0350df101443aa4d0e96bea0a06ef9364 ]
How to reproduce:
1. Define RTE_LIBRTE_MBUF_DEBUG
2. MALLOC_PERTURB_=178 DPDK_TEST=mbuf_autotest gdb --args
obj-x86_64-linux-gnu/app/test/dpdk-test
--file-prefix=mbuf_autotest
PANIC in rte_mbuf_sanity_check():
bad pkt_len
...
#6 0x00007ffff7d3d4cc in rte_mbuf_sanity_check
(m=m at entry=0x17f8c3400, is_header=is_header at entry=1)
at ../lib/mbuf/rte_mbuf.c:384
#7 0x0000555555653d57 in rte_pktmbuf_free (m=0x17f8c3400)
at ../lib/mbuf/rte_mbuf.h:1385
#8 0x000055555565c7a6 in test_nb_segs_and_next_reset ()
at ../app/test/test_mbuf.c:2752
#9 test_mbuf () at ../app/test/test_mbuf.c:2967
...
(gdb) frame 6
#6 0x00007ffff7d3d4cc in rte_mbuf_sanity_check
(m=m at entry=0x17f8c3400, is_header=is_header at entry=1)
at ../lib/mbuf/rte_mbuf.c:384
384 rte_panic("%s\n", reason);
(gdb) p/d m->pkt_len
$4 = 1500
Fixes: efc6f9104c80 ("mbuf: fix reset on mbuf free")
Signed-off-by: Pavel Ivashchenko <pivashchenko at nfware.com>
Acked-by: Olivier Matz <olivier.matz at 6wind.com>
---
app/test/test_mbuf.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/app/test/test_mbuf.c b/app/test/test_mbuf.c
index 6cbb03b0af..1f4dd62466 100644
--- a/app/test/test_mbuf.c
+++ b/app/test/test_mbuf.c
@@ -2744,6 +2744,7 @@ test_nb_segs_and_next_reset(void)
/* split m0 chain in two, between m1 and m2 */
m0->nb_segs = 2;
+ m0->pkt_len -= m2->data_len;
m1->next = NULL;
m2->nb_segs = 1;
--
2.25.1
---
Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- - 2023-04-09 21:45:41.296890800 +0800
+++ 0099-test-mbuf-fix-test-with-mbuf-debug-enabled.patch 2023-04-09 21:45:38.719042200 +0800
@@ -1 +1 @@
-From ce5440e0350df101443aa4d0e96bea0a06ef9364 Mon Sep 17 00:00:00 2001
+From 29cdc2b88f04b2c459705a24274de32d53001fcf Mon Sep 17 00:00:00 2001
@@ -4,0 +5,3 @@
+Cc: Xueming Li <xuemingl at nvidia.com>
+
+[ upstream commit ce5440e0350df101443aa4d0e96bea0a06ef9364 ]
@@ -36 +38,0 @@
-Cc: stable at dpdk.org
@@ -45 +47 @@
-index 81a6632d11..8d8d3b9386 100644
+index 6cbb03b0af..1f4dd62466 100644
More information about the stable
mailing list