From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 2BBBFA034F; Mon, 30 Aug 2021 16:27:18 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A4391410E2; Mon, 30 Aug 2021 16:27:17 +0200 (CEST) Received: from shelob.oktetlabs.ru (shelob.oktetlabs.ru [91.220.146.113]) by mails.dpdk.org (Postfix) with ESMTP id D1FEB40F35 for ; Mon, 30 Aug 2021 16:27:15 +0200 (CEST) Received: from localhost.localdomain (unknown [5.144.120.37]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by shelob.oktetlabs.ru (Postfix) with ESMTPSA id 3F3247F504; Mon, 30 Aug 2021 17:27:15 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 shelob.oktetlabs.ru 3F3247F504 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=oktetlabs.ru; s=default; t=1630333635; bh=BC2lo0UQv5qywXBvbbywVIVr5rI4bXNG/i1LmK/WF4w=; h=From:To:Cc:Subject:Date; b=p7UaWYWZHmc/1Gke948ZcLWiQlbykFf+UGIxlR9FwrjSJ8qoNiUHfLxSt8JdP28dv huG44GfWxG+d0zAgKCATR5CwAvAwVLWAaZIgGEFlrs2S1fr84EK7sBrKuf5oCqxECX 23yHXEBPP6BCpZO7V6Pq4Hf5u7ePKgFGdDxUDTIk= From: Ivan Malov To: dev@dpdk.org Cc: Andrew Rybchenko , Maxime Coquelin , Chenbo Xia Date: Mon, 30 Aug 2021 17:26:55 +0300 Message-Id: <20210830142655.18373-1-ivan.malov@oktetlabs.ru> X-Mailer: git-send-email 2.20.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-dev] [PATCH] net/virtio: handle Tx checksums correctly for tunnel packets X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Tx prepare method calls rte_net_intel_cksum_prepare(), which handles tunnel packets correctly, but Tx burst path does not take tunnel presence into account when computing the offsets. Signed-off-by: Ivan Malov Reviewed-by: Andrew Rybchenko --- drivers/net/virtio/virtqueue.h | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/net/virtio/virtqueue.h b/drivers/net/virtio/virtqueue.h index 03957b2bd0..b83ff32efb 100644 --- a/drivers/net/virtio/virtqueue.h +++ b/drivers/net/virtio/virtqueue.h @@ -620,19 +620,21 @@ static inline void virtqueue_xmit_offload(struct virtio_net_hdr *hdr, struct rte_mbuf *cookie) { uint64_t csum_l4 = cookie->ol_flags & PKT_TX_L4_MASK; + uint16_t o_l23_len = (cookie->ol_flags & PKT_TX_TUNNEL_MASK) ? + cookie->outer_l2_len + cookie->outer_l3_len : 0; if (cookie->ol_flags & PKT_TX_TCP_SEG) csum_l4 |= PKT_TX_TCP_CKSUM; switch (csum_l4) { case PKT_TX_UDP_CKSUM: - hdr->csum_start = cookie->l2_len + cookie->l3_len; + hdr->csum_start = o_l23_len + cookie->l2_len + cookie->l3_len; hdr->csum_offset = offsetof(struct rte_udp_hdr, dgram_cksum); hdr->flags = VIRTIO_NET_HDR_F_NEEDS_CSUM; break; case PKT_TX_TCP_CKSUM: - hdr->csum_start = cookie->l2_len + cookie->l3_len; + hdr->csum_start = o_l23_len + cookie->l2_len + cookie->l3_len; hdr->csum_offset = offsetof(struct rte_tcp_hdr, cksum); hdr->flags = VIRTIO_NET_HDR_F_NEEDS_CSUM; break; @@ -650,7 +652,8 @@ virtqueue_xmit_offload(struct virtio_net_hdr *hdr, struct rte_mbuf *cookie) VIRTIO_NET_HDR_GSO_TCPV6 : VIRTIO_NET_HDR_GSO_TCPV4; hdr->gso_size = cookie->tso_segsz; - hdr->hdr_len = cookie->l2_len + cookie->l3_len + cookie->l4_len; + hdr->hdr_len = o_l23_len + cookie->l2_len + cookie->l3_len + + cookie->l4_len; } else { ASSIGN_UNLESS_EQUAL(hdr->gso_type, 0); ASSIGN_UNLESS_EQUAL(hdr->gso_size, 0); -- 2.20.1