net: fix IPv6 checksum with TSO

[dpdk.git] / lib / librte_net / rte_net.h

diff --git a/lib/librte_net/rte_net.h b/lib/librte_net/rte_net.h
index 7088584..94b06d9 100644 (file)
--- a/lib/librte_net/rte_net.h
+++ b/lib/librte_net/rte_net.h
@@ -51,7 +51,8 @@ struct rte_net_hdr_lens {
  *   Protocol that follows IPv6 header.
  *   -1 if an error occurs during mbuf parsing.
  */
-int __rte_experimental
+__rte_experimental
+int
 rte_net_skip_ip6_ext(uint16_t proto, const struct rte_mbuf *m, uint32_t *off,
        int *frag);
 
@@ -112,27 +113,24 @@ uint32_t rte_net_get_ptype(const struct rte_mbuf *m,
 static inline int
 rte_net_intel_cksum_flags_prepare(struct rte_mbuf *m, uint64_t ol_flags)
 {
-       struct rte_ipv4_hdr *ipv4_hdr;
+       /* Initialise ipv4_hdr to avoid false positive compiler warnings. */
+       struct rte_ipv4_hdr *ipv4_hdr = NULL;
        struct rte_ipv6_hdr *ipv6_hdr;
        struct rte_tcp_hdr *tcp_hdr;
        struct rte_udp_hdr *udp_hdr;
        uint64_t inner_l3_offset = m->l2_len;
 
-#ifdef RTE_LIBRTE_ETHDEV_DEBUG
        /*
         * Does packet set any of available offloads?
         * Mainly it is required to avoid fragmented headers check if
         * no offloads are requested.
         */
-       if (!(ol_flags & PKT_TX_OFFLOAD_MASK))
+       if (!(ol_flags & (PKT_TX_IP_CKSUM | PKT_TX_L4_MASK | PKT_TX_TCP_SEG)))
                return 0;
-#endif
 
-       if ((ol_flags & PKT_TX_OUTER_IP_CKSUM) ||
-               (ol_flags & PKT_TX_OUTER_IPV6))
+       if (ol_flags & (PKT_TX_OUTER_IPV4 | PKT_TX_OUTER_IPV6))
                inner_l3_offset += m->outer_l2_len + m->outer_l3_len;
 
-#ifdef RTE_LIBRTE_ETHDEV_DEBUG
        /*
         * Check if headers are fragmented.
         * The check could be less strict depending on which offloads are
@@ -141,7 +139,6 @@ rte_net_intel_cksum_flags_prepare(struct rte_mbuf *m, uint64_t ol_flags)
        if (unlikely(rte_pktmbuf_data_len(m) <
                     inner_l3_offset + m->l3_len + m->l4_len))
                return -ENOTSUP;
-#endif
 
        if (ol_flags & PKT_TX_IPV4) {
                ipv4_hdr = rte_pktmbuf_mtod_offset(m, struct rte_ipv4_hdr *,
@@ -151,7 +148,7 @@ rte_net_intel_cksum_flags_prepare(struct rte_mbuf *m, uint64_t ol_flags)
                        ipv4_hdr->hdr_checksum = 0;
        }
 
-       if ((ol_flags & PKT_TX_UDP_CKSUM) == PKT_TX_UDP_CKSUM) {
+       if ((ol_flags & PKT_TX_L4_MASK) == PKT_TX_UDP_CKSUM) {
                if (ol_flags & PKT_TX_IPV4) {
                        udp_hdr = (struct rte_udp_hdr *)((char *)ipv4_hdr +
                                        m->l3_len);
@@ -167,7 +164,7 @@ rte_net_intel_cksum_flags_prepare(struct rte_mbuf *m, uint64_t ol_flags)
                        udp_hdr->dgram_cksum = rte_ipv6_phdr_cksum(ipv6_hdr,
                                        ol_flags);
                }
-       } else if ((ol_flags & PKT_TX_TCP_CKSUM) ||
+       } else if ((ol_flags & PKT_TX_L4_MASK) == PKT_TX_TCP_CKSUM ||
                        (ol_flags & PKT_TX_TCP_SEG)) {
                if (ol_flags & PKT_TX_IPV4) {
                        /* non-TSO tcp or TSO */