ip_frag: remove remaining IP checkum offload flag
authorSunil Kumar Kori <skori@marvell.com>
Tue, 16 Jul 2019 06:23:28 +0000 (11:53 +0530)
committerThomas Monjalon <thomas@monjalon.net>
Tue, 16 Jul 2019 11:23:04 +0000 (13:23 +0200)
Currently PKT_TX_IP_CKSUM is being set into mbuf->ol_flags during
fragmentation operation implicitly by the library. Because of this,
application is forced to use checksum offload whether it is supported
by platform or not.

Also documentation does not provide any expected value of ol_flags in
returned fragmented mbufs so application will never come to know that which
offloads are enabled. So transmission may be failed for the platforms which
does not support checksum offload.

So removing mentioned flag from the library.

Mentioned change is part of http://patches.dpdk.org/patch/53475.
Changes for reassembly operation is already accepted. This patch set
implements the similar change for fragmentation operation.

Fixes: e29fc44370c2 ("ip_frag: remove IP checkum offload flag")

Signed-off-by: Sunil Kumar Kori <skori@marvell.com>
lib/librte_ip_frag/rte_ipv4_fragmentation.c

index 221d74e..9e9f986 100644 (file)
@@ -173,7 +173,6 @@ rte_ipv4_fragment_packet(struct rte_mbuf *pkt_in,
                fragment_offset = (uint16_t)(fragment_offset +
                    out_pkt->pkt_len - sizeof(struct rte_ipv4_hdr));
 
-               out_pkt->ol_flags |= PKT_TX_IP_CKSUM;
                out_pkt->l3_len = sizeof(struct rte_ipv4_hdr);
 
                /* Write the fragment to the output list */