commit a75ad876401a700127ebf234fc422e76fcd57b4c
author Steven Luong <sluong@cisco.com> Tue Aug 06 21:51:34 2019 -0700
committer Damjan Marion <dmarion@me.com> Mon Aug 19 09:38:41 2019 +0000
tree 73d0a4d425dd7672cc2f58ad490c053b371da6e7
parent 74eba446b1e71f7fe5bac29cf30d4363a9b8b0cd

devices: skip checksum calculation if guest supports checksum offload

Set VNET_HW_INTERFACE_FLAG_SUPPORTS_TX_L4_CKSUM_OFFLOAD for the interface
to skip checksum calculation if guest supports checksum offload.

Type: fix

Signed-off-by: Steven Luong <sluong@cisco.com>
Change-Id: Ie933c3462394f07580ef7f2bec1d2eb3b075bd0c

src/vnet/devices/virtio/vhost_user.c

diff --git a/src/vnet/devices/virtio/vhost_user.c b/src/vnet/devices/virtio/vhost_user.c
index 788b348..c06f78c 100644
--- a/src/vnet/devices/virtio/vhost_user.c
+++ b/src/vnet/devices/virtio/vhost_user.c
@@ -498,10 +498,14 @@
       ASSERT (vui->virtio_net_hdr_sz < VLIB_BUFFER_PRE_DATA_SIZE);
       vnet_hw_interface_t *hw = vnet_get_hw_interface (vnm, vui->hw_if_index);
       if (vui->enable_gso &&
-	  (vui->features & (1ULL << FEAT_VIRTIO_NET_F_GUEST_CSUM)))
-	hw->flags |= VNET_HW_INTERFACE_FLAG_SUPPORTS_GSO;
+	  ((vui->features & FEATURE_VIRTIO_NET_F_HOST_GUEST_TSO_FEATURE_BITS)
+	   == FEATURE_VIRTIO_NET_F_HOST_GUEST_TSO_FEATURE_BITS))
+	hw->flags |=
+	  (VNET_HW_INTERFACE_FLAG_SUPPORTS_GSO |
+	   VNET_HW_INTERFACE_FLAG_SUPPORTS_TX_L4_CKSUM_OFFLOAD);
       else
-	hw->flags &= ~VNET_HW_INTERFACE_FLAG_SUPPORTS_GSO;
+	hw->flags &= ~(VNET_HW_INTERFACE_FLAG_SUPPORTS_GSO |
+		       VNET_HW_INTERFACE_FLAG_SUPPORTS_TX_L4_CKSUM_OFFLOAD);
       vnet_hw_interface_set_flags (vnm, vui->hw_if_index, 0);
       vui->is_ready = 0;
       vhost_user_update_iface_state (vui);