Hi Anatoly,

Could you please review and provide suggestions if any.

Thanks,
Mingjin

> -----Original Message-----
> From: Ling, WeiX <weix.l...@intel.com>
> Sent: Tuesday, September 12, 2023 5:08 PM
> To: Ye, MingjinX <mingjinx...@intel.com>; dev@dpdk.org
> Cc: Yang, Qiming <qiming.y...@intel.com>; Zhou, YidingX
> <yidingx.z...@intel.com>; Ye, MingjinX <mingjinx...@intel.com>;
> sta...@dpdk.org; Burakov, Anatoly <anatoly.bura...@intel.com>
> Subject: RE: [PATCH] bus/vdev: fix devargs memory leak
> 
> > -----Original Message-----
> > From: Mingjin Ye <mingjinx...@intel.com>
> > Sent: Friday, September 1, 2023 3:24 PM
> > To: dev@dpdk.org
> > Cc: Yang, Qiming <qiming.y...@intel.com>; Zhou, YidingX
> > <yidingx.z...@intel.com>; Ye, MingjinX <mingjinx...@intel.com>;
> > sta...@dpdk.org; Burakov, Anatoly <anatoly.bura...@intel.com>
> > Subject: [PATCH] bus/vdev: fix devargs memory leak
> >
> > When a device is created by a secondary process, an empty devargs is
> > temporarily generated and bound to it. This causes the device to not
> > be associated with the correct devargs, and the empty devargs are not
> > released when the resource is freed.
> >
> > This patch fixes the issue by matching the devargs when inserting a
> > device in secondary process.
> >
> > Fixes: dda987315ca2 ("vdev: make virtual bus use its device struct")
> > Fixes: a16040453968 ("eal: extract vdev infra")
> > Cc: sta...@dpdk.org
> >
> > Signed-off-by: Mingjin Ye <mingjinx...@intel.com>
> > ---
> 
> Tested-by: Wei Ling <weix.l...@intel.com>

Reply via email to