Re: [dpdk-dev] [PATCH v1] vhost: add vq status check in async enqueue poll

2020-07-20 Thread Fu, Patrick
V2 patch sent with revised title > -Original Message- > From: Maxime Coquelin > Sent: Monday, July 20, 2020 11:58 PM > To: Fu, Patrick ; dev@dpdk.org; Xia, Chenbo > > Subject: Re: [PATCH v1] vhost: add vq status check in async enqueue poll > > What about this commit title? > vhost: fix

Re: [dpdk-dev] [PATCH v1] vhost: add vq status check in async enqueue poll

2020-07-20 Thread Ferruh Yigit
On 7/20/2020 4:57 PM, Maxime Coquelin wrote: > What about this commit title? > vhost: fix missing virtqueue status check in async path Existing commit dropped from next-net and patchwork status updated. > > On 7/16/20 5:30 PM, patrick...@intel.com wrote: >> From: Patrick Fu >> >> Vring should n

Re: [dpdk-dev] [PATCH v1] vhost: add vq status check in async enqueue poll

2020-07-20 Thread Maxime Coquelin
What about this commit title? vhost: fix missing virtqueue status check in async path On 7/16/20 5:30 PM, patrick...@intel.com wrote: > From: Patrick Fu > > Vring should not be touched if vq is disabled. This patch adds the vq > status check in async enqueue polling to avoid accessing to a disab

Re: [dpdk-dev] [PATCH v1] vhost: add vq status check in async enqueue poll

2020-07-17 Thread Ferruh Yigit
On 7/17/2020 9:09 AM, Xia, Chenbo wrote: > >> -Original Message- >> From: Fu, Patrick >> Sent: Thursday, July 16, 2020 11:30 PM >> To: dev@dpdk.org; maxime.coque...@redhat.com; Xia, Chenbo >> >> Cc: Fu, Patrick >> Subject: [PATCH v1] vhost: add vq status check in async enqueue poll >> >

Re: [dpdk-dev] [PATCH v1] vhost: add vq status check in async enqueue poll

2020-07-17 Thread Xia, Chenbo
> -Original Message- > From: Fu, Patrick > Sent: Thursday, July 16, 2020 11:30 PM > To: dev@dpdk.org; maxime.coque...@redhat.com; Xia, Chenbo > > Cc: Fu, Patrick > Subject: [PATCH v1] vhost: add vq status check in async enqueue poll > > From: Patrick Fu > > Vring should not be touch

[dpdk-dev] [PATCH v1] vhost: add vq status check in async enqueue poll

2020-07-16 Thread patrick . fu
From: Patrick Fu Vring should not be touched if vq is disabled. This patch adds the vq status check in async enqueue polling to avoid accessing to a disabled queue. Fixes: cd6760da1076 ("vhost: introduce async enqueue for split ring") Signed-off-by: Patrick Fu --- lib/librte_vhost/virtio_net.