virtio: vhost checksum problem for ipv6 65/26365/4
authorSteven Luong <sluong@cisco.com>
Sun, 5 Apr 2020 23:09:17 +0000 (16:09 -0700)
committerAndrew Yourtchenko <ayourtch@gmail.com>
Fri, 24 Apr 2020 09:11:33 +0000 (09:11 +0000)
When checksum is enable for IPv6, it erroneously set the flag
VNET_BUFFER_F_OFFLOAD_IP_CKSUM. That flag is meant for ip4 packets only.

Type: fix
Ticket: VPP-1857

Signed-off-by: Steven Luong <sluong@cisco.com>
Change-Id: Id03b2937bfa34e2a9b50a36aafe0700bad7fb95e
(cherry picked from commit f78294d8468f156e066e7e69aab3a1b285810c3a)

src/vnet/devices/virtio/vhost_user_input.c

index a371e57..4b52bd5 100644 (file)
@@ -275,14 +275,14 @@ vhost_user_handle_rx_offload (vlib_buffer_t * b0, u8 * b0_data,
       vnet_buffer (b0)->l4_hdr_offset = hdr->csum_start;
       b0->flags |= (VNET_BUFFER_F_L2_HDR_OFFSET_VALID |
                    VNET_BUFFER_F_L3_HDR_OFFSET_VALID |
-                   VNET_BUFFER_F_L4_HDR_OFFSET_VALID |
-                   VNET_BUFFER_F_OFFLOAD_IP_CKSUM);
+                   VNET_BUFFER_F_L4_HDR_OFFSET_VALID);
 
       if (PREDICT_TRUE (ethertype == ETHERNET_TYPE_IP4))
        {
          ip4_header_t *ip4 = (ip4_header_t *) (b0_data + l2hdr_sz);
          l4_proto = ip4->protocol;
-         b0->flags |= VNET_BUFFER_F_IS_IP4;
+         b0->flags |= (VNET_BUFFER_F_IS_IP4 |
+                       VNET_BUFFER_F_OFFLOAD_IP_CKSUM);
        }
       else if (PREDICT_TRUE (ethertype == ETHERNET_TYPE_IP6))
        {