Re: [dpdk-dev] [PATCH v1] vhost: set zmbufs to NULL when freed

2020-07-20 Thread Fu, Patrick
V2 patch sent with revised title > -Original Message- > From: Maxime Coquelin > Sent: Monday, July 20, 2020 11:52 PM > To: Fu, Patrick ; dev@dpdk.org; Xia, Chenbo > > Cc: Wang, Yinan > Subject: Re: [PATCH v1] vhost: set zmbufs to NULL when freed > > Do you agree with below title? > vho

Re: [dpdk-dev] [PATCH v1] vhost: set zmbufs to NULL when freed

2020-07-20 Thread Ferruh Yigit
On 7/20/2020 4:51 PM, Maxime Coquelin wrote: > Do you agree with below title? > vhost: fix double-free with zero-copy Existing commit dropped from next-net and patchwork status updated. > > Thanks, > Maxime > > On 7/15/20 5:58 PM, patrick...@intel.com wrote: >> From: Patrick Fu >> >> zmbufs sh

Re: [dpdk-dev] [PATCH v1] vhost: set zmbufs to NULL when freed

2020-07-20 Thread Maxime Coquelin
Do you agree with below title? vhost: fix double-free with zero-copy Thanks, Maxime On 7/15/20 5:58 PM, patrick...@intel.com wrote: > From: Patrick Fu > > zmbufs should be set to NULL when getting freed to avoid double free on > the same buffer pointer > > Fixes: b0a985d1f340 ("vhost: add dequ

Re: [dpdk-dev] [PATCH v1] vhost: set zmbufs to NULL when freed

2020-07-17 Thread Ferruh Yigit
On 7/17/2020 4:19 AM, Xia, Chenbo wrote: > >> -Original Message- >> From: Fu, Patrick >> Sent: Wednesday, July 15, 2020 11:59 PM >> To: dev@dpdk.org; maxime.coque...@redhat.com; Xia, Chenbo >> >> Cc: Wang, Yinan ; Fu, Patrick >> Subject: [PATCH v1] vhost: set zmbufs to NULL when freed >

Re: [dpdk-dev] [PATCH v1] vhost: set zmbufs to NULL when freed

2020-07-16 Thread Xia, Chenbo
> -Original Message- > From: Fu, Patrick > Sent: Wednesday, July 15, 2020 11:59 PM > To: dev@dpdk.org; maxime.coque...@redhat.com; Xia, Chenbo > > Cc: Wang, Yinan ; Fu, Patrick > Subject: [PATCH v1] vhost: set zmbufs to NULL when freed > > From: Patrick Fu > > zmbufs should be set t

Re: [dpdk-dev] [PATCH v1] vhost: set zmbufs to NULL when freed

2020-07-16 Thread Xiao, QimaiX
dev] [PATCH v1] vhost: set zmbufs to NULL when freed > > From: Patrick Fu > > zmbufs should be set to NULL when getting freed to avoid double free on > the same buffer pointer > > Fixes: b0a985d1f340 ("vhost: add dequeue zero copy") > > Signed-off-by: Pat

[dpdk-dev] [PATCH v1] vhost: set zmbufs to NULL when freed

2020-07-15 Thread patrick . fu
From: Patrick Fu zmbufs should be set to NULL when getting freed to avoid double free on the same buffer pointer Fixes: b0a985d1f340 ("vhost: add dequeue zero copy") Signed-off-by: Patrick Fu --- lib/librte_vhost/vhost_user.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/librte_vhos