Re: [dpdk-dev] [PATCH v2 06/10] net/virtio: fix queue setup consistency

2018-02-09 Thread Maxime Coquelin
..@redhat.com; Thomas Monjalon ; sta...@dpdk.org Subject: Re: [dpdk-dev] [PATCH v2 06/10] net/virtio: fix queue setup consistency Hi, It's in my short plans, but unfortunately some other high priority tasks were inserted before. Honnestly, I'm not sure I'll be able to make it f

Re: [dpdk-dev] [PATCH v2 06/10] net/virtio: fix queue setup consistency

2018-02-09 Thread Maxime Coquelin
8 PM To: Yao, Lei A Cc: dev@dpdk.org; y...@fridaylinux.org; maxime.coque...@redhat.com; Thomas Monjalon ; sta...@dpdk.org Subject: Re: [dpdk-dev] [PATCH v2 06/10] net/virtio: fix queue setup consistency Hi Lei, It's on my todo list, I'll check this as soon as possible. Olivier On Thu, Feb 0

Re: [dpdk-dev] [PATCH v2 06/10] net/virtio: fix queue setup consistency

2018-02-08 Thread Wang, Zhihong
> sta...@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 06/10] net/virtio: fix queue setup > consistency > > Hi, > > It's in my short plans, but unfortunately some other high priority tasks > were inserted before. Honnestly, I'm not sure I'll be able to mak

Re: [dpdk-dev] [PATCH v2 06/10] net/virtio: fix queue setup consistency

2018-02-07 Thread Olivier Matz
: Yao, Lei A > > Cc: dev@dpdk.org; y...@fridaylinux.org; maxime.coque...@redhat.com; > > Thomas Monjalon ; sta...@dpdk.org > > Subject: Re: [dpdk-dev] [PATCH v2 06/10] net/virtio: fix queue setup > > consistency > > > > Hi Lei, > > > > It's on my

Re: [dpdk-dev] [PATCH v2 06/10] net/virtio: fix queue setup consistency

2018-02-07 Thread Xu, Qian Q
c: dev@dpdk.org; y...@fridaylinux.org; maxime.coque...@redhat.com; > Thomas Monjalon ; sta...@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 06/10] net/virtio: fix queue setup > consistency > > Hi Lei, > > It's on my todo list, I'll check this as soon as possible. >

Re: [dpdk-dev] [PATCH v2 06/10] net/virtio: fix queue setup consistency

2018-02-01 Thread Olivier Matz
> > -Original Message- > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Olivier Matz > > Sent: Thursday, September 7, 2017 8:14 PM > > To: dev@dpdk.org; y...@fridaylinux.org; maxime.coque...@redhat.com > > Cc: step...@networkplumber.org; sta...@dpdk

Re: [dpdk-dev] [PATCH v2 06/10] net/virtio: fix queue setup consistency

2018-01-31 Thread Yao, Lei A
ect: [dpdk-dev] [PATCH v2 06/10] net/virtio: fix queue setup consistency > > In rx/tx queue setup functions, some code is executed only if > use_simple_rxtx == 1. The value of this variable can change depending on > the offload flags or sse support. If Rx queue setup is called befo

Re: [dpdk-dev] [PATCH v2 06/10] net/virtio: fix queue setup consistency

2017-12-07 Thread Tiwei Bie
Hi Olivier, On Thu, Dec 07, 2017 at 03:14:44PM +0100, Olivier MATZ wrote: > On Wed, Dec 06, 2017 at 01:25:29PM +0800, Tiwei Bie wrote: > > Hi Maxime and Olivier: > > > > On Thu, Sep 07, 2017 at 02:13:43PM +0200, Olivier Matz wrote: > > [...] > > > diff --git a/drivers/net/virtio/virtio_ethdev.c

Re: [dpdk-dev] [PATCH v2 06/10] net/virtio: fix queue setup consistency

2017-12-07 Thread Olivier MATZ
Hi Tiwei, On Wed, Dec 06, 2017 at 01:25:29PM +0800, Tiwei Bie wrote: > Hi Maxime and Olivier: > > On Thu, Sep 07, 2017 at 02:13:43PM +0200, Olivier Matz wrote: > [...] > > diff --git a/drivers/net/virtio/virtio_ethdev.c > > b/drivers/net/virtio/virtio_ethdev.c > > index 8eee3ff80..c7888f103 1006

Re: [dpdk-dev] [PATCH v2 06/10] net/virtio: fix queue setup consistency

2017-12-05 Thread Tiwei Bie
Hi Maxime and Olivier: On Thu, Sep 07, 2017 at 02:13:43PM +0200, Olivier Matz wrote: [...] > diff --git a/drivers/net/virtio/virtio_ethdev.c > b/drivers/net/virtio/virtio_ethdev.c > index 8eee3ff80..c7888f103 100644 > --- a/drivers/net/virtio/virtio_ethdev.c > +++ b/drivers/net/virtio/virtio_ethd

[dpdk-dev] [PATCH v2 06/10] net/virtio: fix queue setup consistency

2017-09-07 Thread Olivier Matz
In rx/tx queue setup functions, some code is executed only if use_simple_rxtx == 1. The value of this variable can change depending on the offload flags or sse support. If Rx queue setup is called before Tx queue setup, it can result in an invalid configuration: - dev_configure is called: use_simp