Skip to content

Commit

Permalink
app/testpmd: fix check without outer checksum
Browse files Browse the repository at this point in the history
[ upstream commit 4219d76 ]

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: 3c32113 ("app/testpmd: fix IPv6 tunnel checksum")

Signed-off-by: Nithin Dabilpuram <ndabilpuram@marvell.com>
Acked-by: Xiaoyun Li <xiaoyun.li@intel.com>
  • Loading branch information
nithind1988 authored and cpaelzer committed Nov 29, 2021
1 parent 92f8daa commit 82785de
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions app/test-pmd/csumonly.c
Expand Up @@ -927,8 +927,7 @@ pkt_burst_checksum_forward(struct fwd_stream *fs)
(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;
Expand Down

0 comments on commit 82785de

Please sign in to comment.