diff options
| author |   Steven Luong <sluong@cisco.com> | 2020-04-05 16:09:17 -0700 |
|---|---|---|
| committer | Steven Luong <sluong@cisco.com> | 2020-04-05 16:33:51 -0700 |
| commit | f78294d8468f156e066e7e69aab3a1b285810c3a (patch) | |
| tree | e055e54a715ff48489ddcb88d7098be560408bed | |
| parent | 597d4df6cf8b96e645e21447974cd82c3285aee0 (diff) | |
virtio: vhost checksum problem for ipv6
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
| -rw-r--r-- | src/vnet/devices/virtio/vhost_user_input.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/vnet/devices/virtio/vhost_user_input.c b/src/vnet/devices/virtio/vhost_user_input.c index ee02b3b86ee..e1e9dc48a24 100644 --- a/src/vnet/devices/virtio/vhost_user_input.c +++ b/src/vnet/devices/virtio/vhost_user_input.c @@ -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)) { |