On 10/05/2016 03:27 PM, Maxime Coquelin wrote: >> @@ -903,7 +905,8 @@ virtio_recv_pkts(void *rx_queue, struct rte_mbuf >> **rx_pkts, uint16_t nb_pkts) >> rte_vlan_strip(rxm); >> >> /* Update offload features */ >> - if (virtio_rx_offload(rxm, hdr) < 0) { >> + if ((features & VIRTIO_NET_F_GUEST_CSUM) && > s/VIRTIO_NET_F_GUEST_CSUM/(1u << VIRTIO_NET_F_GUEST_CSUM)/
oooh good catch :) > And don't forget to update the test for LRO patch. yep > Except this, it sounds good. Thanks, I'll send a v3 soon. Olivier