[dpdk-dev] [PATCH v2 09/12] virtio: add Rx checksum offload support

2016-10-12 Thread Yuanhan Liu
On Wed, Oct 05, 2016 at 03:27:47PM +0200, Maxime Coquelin wrote: > >/* 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)/ Th

[dpdk-dev] [PATCH v2 09/12] virtio: add Rx checksum offload support

2016-10-12 Thread Olivier MATZ
On 10/12/2016 03:02 PM, Yuanhan Liu wrote: > On Wed, Oct 05, 2016 at 03:27:47PM +0200, Maxime Coquelin wrote: >>> /* Update offload features */ >>> - if (virtio_rx_offload(rxm, hdr) < 0) { >>> + if ((features & VIRTIO_NET_F_GUEST_CSUM) && >> s/VIRTIO_NE

[dpdk-dev] [PATCH v2 09/12] virtio: add Rx checksum offload support

2016-10-11 Thread Olivier MATZ
On 10/11/2016 04:36 PM, Maxime Coquelin wrote: > > > On 10/11/2016 04:29 PM, Olivier MATZ wrote: >> >> >> On 10/11/2016 04:04 PM, Maxime Coquelin wrote: +/* Optionally fill offload information in structure */ +static int +virtio_rx_offload(struct rte_mbuf *m, struct virtio_net_hdr

[dpdk-dev] [PATCH v2 09/12] virtio: add Rx checksum offload support

2016-10-11 Thread Maxime Coquelin
On 10/11/2016 04:29 PM, Olivier MATZ wrote: > > > On 10/11/2016 04:04 PM, Maxime Coquelin wrote: >>> +/* Optionally fill offload information in structure */ >>> +static int >>> +virtio_rx_offload(struct rte_mbuf *m, struct virtio_net_hdr *hdr) >>> +{ >>> +struct rte_net_hdr_lens hdr_lens; >>>

[dpdk-dev] [PATCH v2 09/12] virtio: add Rx checksum offload support

2016-10-11 Thread Olivier MATZ
On 10/11/2016 04:04 PM, Maxime Coquelin wrote: >> +/* Optionally fill offload information in structure */ >> +static int >> +virtio_rx_offload(struct rte_mbuf *m, struct virtio_net_hdr *hdr) >> +{ >> +struct rte_net_hdr_lens hdr_lens; >> +uint32_t hdrlen, ptype; >> +int l4_supported =

[dpdk-dev] [PATCH v2 09/12] virtio: add Rx checksum offload support

2016-10-11 Thread Maxime Coquelin
On 10/03/2016 11:00 AM, Olivier Matz wrote: > Signed-off-by: Olivier Matz > --- > drivers/net/virtio/virtio_ethdev.c | 14 > drivers/net/virtio/virtio_ethdev.h | 2 +- > drivers/net/virtio/virtio_rxtx.c | 69 > ++ > drivers/net/virtio/virtqueue.h

[dpdk-dev] [PATCH v2 09/12] virtio: add Rx checksum offload support

2016-10-05 Thread Olivier Matz
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,

[dpdk-dev] [PATCH v2 09/12] virtio: add Rx checksum offload support

2016-10-05 Thread Maxime Coquelin
Hi Olivier, On 10/05/2016 01:56 PM, Olivier Matz wrote: > Hi Maxime, > > On 10/03/2016 02:51 PM, Maxime Coquelin wrote: >>> --- a/drivers/net/virtio/virtio_rxtx.c >>> +++ b/drivers/net/virtio/virtio_rxtx.c >>> @@ -50,6 +50,7 @@ >>> #include >>> #include >>> #include >>> +#include >>> >>> #

[dpdk-dev] [PATCH v2 09/12] virtio: add Rx checksum offload support

2016-10-05 Thread Olivier Matz
Hi Maxime, On 10/03/2016 02:51 PM, Maxime Coquelin wrote: >> --- a/drivers/net/virtio/virtio_rxtx.c >> +++ b/drivers/net/virtio/virtio_rxtx.c >> @@ -50,6 +50,7 @@ >> #include >> #include >> #include >> +#include >> >> #include "virtio_logs.h" >> #include "virtio_ethdev.h" >> @@ -627,6 +62

[dpdk-dev] [PATCH v2 09/12] virtio: add Rx checksum offload support

2016-10-03 Thread Maxime Coquelin
Hi Olivier, On 10/03/2016 11:00 AM, Olivier Matz wrote: > Signed-off-by: Olivier Matz > --- > drivers/net/virtio/virtio_ethdev.c | 14 > drivers/net/virtio/virtio_ethdev.h | 2 +- > drivers/net/virtio/virtio_rxtx.c | 69 > ++ > drivers/net/virtio

[dpdk-dev] [PATCH v2 09/12] virtio: add Rx checksum offload support

2016-10-03 Thread Olivier Matz
Signed-off-by: Olivier Matz --- drivers/net/virtio/virtio_ethdev.c | 14 drivers/net/virtio/virtio_ethdev.h | 2 +- drivers/net/virtio/virtio_rxtx.c | 69 ++ drivers/net/virtio/virtqueue.h | 1 + 4 files changed, 78 insertions(+), 8 deletions(-