Re: [dpdk-dev] [PATCH 1/2] vhost: fix overflow on shadow used ring

2018-07-26 Thread Tiwei Bie
On Thu, Jul 26, 2018 at 09:37:20AM +0800, Tiwei Bie wrote: > The shadow used ring's size is the same as the vq's size, > so we shouldn't try more than "vq size" times. Besides, > the element pointed by avail->idx isn't available to the > device, so we will return error when try "vq size" times. >

Re: [dpdk-dev] [PATCH 1/2] vhost: fix overflow on shadow used ring

2018-07-26 Thread Jens Freimann
On Thu, Jul 26, 2018 at 09:37:20AM +0800, Tiwei Bie wrote: The shadow used ring's size is the same as the vq's size, so we shouldn't try more than "vq size" times. Besides, the element pointed by avail->idx isn't available to the device, so we will return error when try "vq size" times. Fixes: 2

[dpdk-dev] [PATCH 1/2] vhost: fix overflow on shadow used ring

2018-07-25 Thread Tiwei Bie
The shadow used ring's size is the same as the vq's size, so we shouldn't try more than "vq size" times. Besides, the element pointed by avail->idx isn't available to the device, so we will return error when try "vq size" times. Fixes: 24e4844048e1 ("vhost: unify Rx mergeable and non-mergeable pat