app/testpmd: fix check without outer checksum
authorNithin Dabilpuram <ndabilpuram@marvell.com>
Mon, 16 Aug 2021 07:09:41 +0000 (12:39 +0530)
committerFerruh Yigit <ferruh.yigit@intel.com>
Wed, 25 Aug 2021 12:29:30 +0000 (14:29 +0200)
Donot use outer metadata when neither outer ip checksum nor
outer udp checksum is enabled. PMD's will ignore the
outer_l2_len and outer_l3_len in cases where none of
the outer checksum is enabled and hence only l2_len and
l3_len will be used to calculate the offsets for L2 or L3
header.

Fixes: 3c32113a1aac ("app/testpmd: fix IPv6 tunnel checksum")
Cc: stable@dpdk.org
Signed-off-by: Nithin Dabilpuram <ndabilpuram@marvell.com>
Acked-by: Xiaoyun Li <xiaoyun.li@intel.com>
app/test-pmd/csumonly.c

index 607c889..38cc256 100644 (file)
@@ -961,8 +961,7 @@ tunnel_update:
                            (tx_offloads &
                             DEV_TX_OFFLOAD_OUTER_IPV4_CKSUM) ||
                            (tx_offloads &
-                            DEV_TX_OFFLOAD_OUTER_UDP_CKSUM) ||
-                           (tx_ol_flags & PKT_TX_OUTER_IPV6)) {
+                            DEV_TX_OFFLOAD_OUTER_UDP_CKSUM)) {
                                m->outer_l2_len = info.outer_l2_len;
                                m->outer_l3_len = info.outer_l3_len;
                                m->l2_len = info.l2_len;