[dpdk-dev] [PATCH v2 4/7] vhost: add dequeue zero copy

2016-10-10 Thread Maxime Coquelin
hould be updated. Maxime > > -Original Message- > From: Maxime Coquelin [mailto:maxime.coquelin at redhat.com] > Sent: Monday, October 10, 2016 11:14 AM > To: Xu, Qian Q ; Yuanhan Liu linux.intel.com> > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 4/7

[dpdk-dev] [PATCH v2 4/7] vhost: add dequeue zero copy

2016-10-10 Thread Maxime Coquelin
Hi Xu, On 10/10/2016 12:12 PM, Xu, Qian Q wrote: > Good to know. I will try v3. BTW, on the master branch, seems vhost PMD is > broken. When we run --vdev 'eth_vhost0,', then it will report the error > that the driver is not supported. V16.07 is OK, but I haven't got time to do > git bisec

[dpdk-dev] [PATCH v2 4/7] vhost: add dequeue zero copy

2016-10-10 Thread Xu, Qian Q
nhan Liu Cc: dev at dpdk.org Subject: Re: [dpdk-dev] [PATCH v2 4/7] vhost: add dequeue zero copy Hi Xu, On 10/10/2016 12:12 PM, Xu, Qian Q wrote: > Good to know. I will try v3. BTW, on the master branch, seems vhost PMD is > broken. When we run --vdev 'eth_vhost0,', then it will

[dpdk-dev] [PATCH v2 4/7] vhost: add dequeue zero copy

2016-10-10 Thread Xu, Qian Q
k-dev] [PATCH v2 4/7] vhost: add dequeue zero copy Hi Xu, On 10/10/2016 12:12 PM, Xu, Qian Q wrote: > Good to know. I will try v3. BTW, on the master branch, seems vhost PMD is > broken. When we run --vdev 'eth_vhost0,', then it will report the error > that the driver is not

[dpdk-dev] [PATCH v2 4/7] vhost: add dequeue zero copy

2016-10-10 Thread Xu, Qian Q
om: Yuanhan Liu [mailto:yuanhan@linux.intel.com] Sent: Sunday, October 9, 2016 3:03 AM To: Xu, Qian Q Cc: dev at dpdk.org; Maxime Coquelin Subject: Re: [dpdk-dev] [PATCH v2 4/7] vhost: add dequeue zero copy On Thu, Oct 06, 2016 at 02:37:27PM +, Xu, Qian Q wrote: > this function copy_desc_to_m

[dpdk-dev] [PATCH v2 4/7] vhost: add dequeue zero copy

2016-10-09 Thread Yuanhan Liu
On Thu, Oct 06, 2016 at 02:37:27PM +, Xu, Qian Q wrote: > this function copy_desc_to_mbuf has changed on the dpdk-next-virtio repo. > Based on current dpdk-next-virtio repo, the commit ID is as below: > commit b4f7b43cd9d3b6413f41221051d03a23bc5f5fbe > Author: Zhiyong Yang > Date: Thu Sep

[dpdk-dev] [PATCH v2 4/7] vhost: add dequeue zero copy

2016-10-06 Thread Xu, Qian Q
See the bottom. -Original Message- From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Yuanhan Liu Sent: Friday, September 23, 2016 5:13 AM To: dev at dpdk.org Cc: Maxime Coquelin ; Yuanhan Liu Subject: [dpdk-dev] [PATCH v2 4/7] vhost: add dequeue zero copy The basic idea of dequeue

[dpdk-dev] [PATCH v2 4/7] vhost: add dequeue zero copy

2016-09-26 Thread Maxime Coquelin
On 09/23/2016 06:13 AM, Yuanhan Liu wrote: > The basic idea of dequeue zero copy is, instead of copying data from > the desc buf, here we let the mbuf reference the desc buf addr directly. > > Doing so, however, has one major issue: we can't update the used ring > at the end of rte_vhost_dequeue_

[dpdk-dev] [PATCH v2 4/7] vhost: add dequeue zero copy

2016-09-23 Thread Yuanhan Liu
The basic idea of dequeue zero copy is, instead of copying data from the desc buf, here we let the mbuf reference the desc buf addr directly. Doing so, however, has one major issue: we can't update the used ring at the end of rte_vhost_dequeue_burst. Because we don't do the copy here, an update of