Re: [dpdk-dev] [PATCH v2] vhost: fix packed ring zero-copy

2020-02-24 Thread Maxime Coquelin
On 2/24/20 11:32 AM, David Marchand wrote: > On Mon, Feb 24, 2020 at 9:28 AM Maxime Coquelin > wrote: >> >> Hi David & Thomas, >> >> On 2/24/20 4:14 PM, Marvin Liu wrote: >>> Available buffer ID should be stored in the zmbuf in the packed-ring >>> dequeue path. There's no guarantee that local q

Re: [dpdk-dev] [PATCH v2] vhost: fix packed ring zero-copy

2020-02-24 Thread David Marchand
On Mon, Feb 24, 2020 at 9:28 AM Maxime Coquelin wrote: > > Hi David & Thomas, > > On 2/24/20 4:14 PM, Marvin Liu wrote: > > Available buffer ID should be stored in the zmbuf in the packed-ring > > dequeue path. There's no guarantee that local queue avail index is > > equal to buffer ID. > > > > Fi

Re: [dpdk-dev] [PATCH v2] vhost: fix packed ring zero-copy

2020-02-24 Thread Maxime Coquelin
Hi David & Thomas, On 2/24/20 4:14 PM, Marvin Liu wrote: > Available buffer ID should be stored in the zmbuf in the packed-ring > dequeue path. There's no guarantee that local queue avail index is > equal to buffer ID. > > Fixes: d1eafb532268 ("vhost: add packed ring zcopy batch and single dequeu

[dpdk-dev] [PATCH v2] vhost: fix packed ring zero-copy

2020-02-23 Thread Marvin Liu
Available buffer ID should be stored in the zmbuf in the packed-ring dequeue path. There's no guarantee that local queue avail index is equal to buffer ID. Fixes: d1eafb532268 ("vhost: add packed ring zcopy batch and single dequeue") Cc: sta...@dpdk.org Signed-off-by: Marvin Liu Reported-by: Yin