virtio: fix the tcp/udp checksum offloads

Some vhost-backend calculates the wrong checksum in
case of tcp/udp offload when driver resets tcp/udp
checksum field to '0'.

Type: fix

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

diff --git a/src/vnet/devices/virtio/vhost_user_input.c b/src/vnet/devices/virtio/vhost_user_input.c
index 2cc192d..e2ad5cf 100644 (file)
--- a/src/vnet/devices/virtio/vhost_user_input.c
+++ b/src/vnet/devices/virtio/vhost_user_input.c
@@ -296,15 +296,11 @@ vhost_user_handle_rx_offload (vlib_buffer_t * b0, u8 * b0_data,
          tcp_header_t *tcp = (tcp_header_t *)
            (b0_data + vnet_buffer (b0)->l4_hdr_offset);
          l4_hdr_sz = tcp_header_bytes (tcp);
-         tcp->checksum = 0;
          b0->flags |= VNET_BUFFER_F_OFFLOAD_TCP_CKSUM;
        }
       else if (l4_proto == IP_PROTOCOL_UDP)
        {
-         udp_header_t *udp =
-           (udp_header_t *) (b0_data + vnet_buffer (b0)->l4_hdr_offset);
-         l4_hdr_sz = sizeof (*udp);
-         udp->checksum = 0;
+         l4_hdr_sz = sizeof (udp_header_t);
          b0->flags |= VNET_BUFFER_F_OFFLOAD_UDP_CKSUM;
        }
     }