Re: [dpdk-dev] [PATCH 1/5] net/virtio: fix vector Rx break caused by rxq flushing

2017-12-08 Thread Kevin Traynor
On 12/08/2017 02:30 AM, Tiwei Bie wrote: > On Thu, Dec 07, 2017 at 04:00:57PM +, Kevin Traynor wrote: >> On 12/07/2017 05:30 AM, Tiwei Bie wrote: >>> The vector Rx will be broken if backend has consumed all >>> the descs in the avail ring before the device is started. >>> Because in current imp

Re: [dpdk-dev] [PATCH 1/5] net/virtio: fix vector Rx break caused by rxq flushing

2017-12-07 Thread Tiwei Bie
On Thu, Dec 07, 2017 at 04:00:57PM +, Kevin Traynor wrote: > On 12/07/2017 05:30 AM, Tiwei Bie wrote: > > The vector Rx will be broken if backend has consumed all > > the descs in the avail ring before the device is started. > > Because in current implementation, vector Rx will return > > immed

Re: [dpdk-dev] [PATCH 1/5] net/virtio: fix vector Rx break caused by rxq flushing

2017-12-07 Thread Fischetti, Antonio
cember 7, 2017 9:30 AM > To: Maxime Coquelin ; Bie, Tiwei > ; y...@fridaylinux.org; dev@dpdk.org > Cc: sta...@dpdk.org > Subject: Re: [dpdk-dev] [PATCH 1/5] net/virtio: fix vector Rx break > caused by rxq flushing > > Thanks Tiwei for working on this, I'll give it a try so

Re: [dpdk-dev] [PATCH 1/5] net/virtio: fix vector Rx break caused by rxq flushing

2017-12-07 Thread Kevin Traynor
On 12/07/2017 05:30 AM, Tiwei Bie wrote: > The vector Rx will be broken if backend has consumed all > the descs in the avail ring before the device is started. > Because in current implementation, vector Rx will return > immediately without refilling the avail ring if the used > ring is empty. So w

Re: [dpdk-dev] [PATCH 1/5] net/virtio: fix vector Rx break caused by rxq flushing

2017-12-07 Thread Fischetti, Antonio
Thanks Tiwei for working on this, I'll give it a try soon. Antonio > -Original Message- > From: Maxime Coquelin [mailto:maxime.coque...@redhat.com] > Sent: Thursday, December 7, 2017 9:15 AM > To: Bie, Tiwei ; y...@fridaylinux.org; dev@dpdk.org > Cc: Fischetti, Antonio ; sta...@dpdk.org >

Re: [dpdk-dev] [PATCH 1/5] net/virtio: fix vector Rx break caused by rxq flushing

2017-12-07 Thread Maxime Coquelin
On 12/07/2017 06:30 AM, Tiwei Bie wrote: The vector Rx will be broken if backend has consumed all the descs in the avail ring before the device is started. Because in current implementation, vector Rx will return immediately without refilling the avail ring if the used ring is empty. So we have

[dpdk-dev] [PATCH 1/5] net/virtio: fix vector Rx break caused by rxq flushing

2017-12-06 Thread Tiwei Bie
The vector Rx will be broken if backend has consumed all the descs in the avail ring before the device is started. Because in current implementation, vector Rx will return immediately without refilling the avail ring if the used ring is empty. So we have to refill the avail ring after flushing the