app/testpmd: do not change IP addrs in checksum engine
authorOlivier Matz <olivier.matz@6wind.com>
Wed, 12 Oct 2016 15:39:47 +0000 (17:39 +0200)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Wed, 12 Oct 2016 16:37:15 +0000 (18:37 +0200)
commit03d17e4d0179de035d7d636cb949a67327fee591
tree594f181c7545454e30628ec9325da818d7b0518a
parent4c3ea50880c90a1681c7f4c12a558cff37d01f03
app/testpmd: do not change IP addrs in checksum engine

The csum forward engine was updated to change the IP addresses in the
packet data in
commit 51f694dd40f5 ("app/testpmd: rework checksum forward engine")

This was done to ensure that the checksum is correctly reprocessed when
using hardware checksum offload. But the functions
process_inner_cksums() and process_outer_cksums() already reset the
checksum field to 0, so this is not necessary.

Moreover, this makes the engine more complex than needed, and prevents
to easily use it to forward traffic (like iperf) as it modifies the
packets.

This patch drops this behavior.

Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
app/test-pmd/csumonly.c