Re: [dpdk-dev] [PATCH v2] vhost: add sanity check when operating the split ring

2021-10-21 Thread Maxime Coquelin
On 10/15/21 11:25, Li Feng wrote: On Fri, Oct 15, 2021 at 4:52 PM Maxime Coquelin wrote: The title is too vague, I would put something like: vhost: add sanity check on inflight last index On 10/14/21 14:40, Li Feng wrote: The idx in rte_vhost_set_last_inflight_io_split is from the fronte

Re: [dpdk-dev] [PATCH v2] vhost: add sanity check when operating the split ring

2021-10-15 Thread Li Feng
On Fri, Oct 15, 2021 at 4:52 PM Maxime Coquelin wrote: > > The title is too vague, I would put something like: > > vhost: add sanity check on inflight last index > > On 10/14/21 14:40, Li Feng wrote: > > The idx in rte_vhost_set_last_inflight_io_split is from the frontend > > s/idx/index/ > > > dr

Re: [dpdk-dev] [PATCH v2] vhost: add sanity check when operating the split ring

2021-10-15 Thread Maxime Coquelin
The title is too vague, I would put something like: vhost: add sanity check on inflight last index On 10/14/21 14:40, Li Feng wrote: The idx in rte_vhost_set_last_inflight_io_split is from the frontend s/idx/index/ driver, check if it's in the virtqueue range. Fixes: bb0c2de9602b ("vhost:

[dpdk-dev] [PATCH v2] vhost: add sanity check when operating the split ring

2021-10-14 Thread Li Feng
The idx in rte_vhost_set_last_inflight_io_split is from the frontend driver, check if it's in the virtqueue range. Fixes: bb0c2de9602b ("vhost: add APIs to operate inflight ring") Cc: sta...@dpdk.org Signed-off-by: Li Feng --- lib/vhost/vhost.c | 3 +++ 1 file changed, 3 insertions(+) diff --g