Hi Anatoly,

can you please take a look at this patch.

Thanks,
Mingjin

> -----Original Message-----
> From: Ye, MingjinX <[email protected]>
> Sent: Friday, September 15, 2023 4:56 PM
> To: Burakov, Anatoly <[email protected]>; [email protected]
> Cc: Yang, Qiming <[email protected]>; Zhou, YidingX
> <[email protected]>; [email protected]; Ling, WeiX
> <[email protected]>
> Subject: RE: [PATCH] bus/vdev: fix devargs memory leak
> 
> Hi Anatoly,
> 
> Could you please review and provide suggestions if any.
> 
> Thanks,
> Mingjin
> 
> > -----Original Message-----
> > From: Ling, WeiX <[email protected]>
> > Sent: Tuesday, September 12, 2023 5:08 PM
> > To: Ye, MingjinX <[email protected]>; [email protected]
> > Cc: Yang, Qiming <[email protected]>; Zhou, YidingX
> > <[email protected]>; Ye, MingjinX <[email protected]>;
> > [email protected]; Burakov, Anatoly <[email protected]>
> > Subject: RE: [PATCH] bus/vdev: fix devargs memory leak
> >
> > > -----Original Message-----
> > > From: Mingjin Ye <[email protected]>
> > > Sent: Friday, September 1, 2023 3:24 PM
> > > To: [email protected]
> > > Cc: Yang, Qiming <[email protected]>; Zhou, YidingX
> > > <[email protected]>; Ye, MingjinX <[email protected]>;
> > > [email protected]; Burakov, Anatoly <[email protected]>
> > > 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: [email protected]
> > >
> > > Signed-off-by: Mingjin Ye <[email protected]>
> > > ---
> >
> > Tested-by: Wei Ling <[email protected]>

Reply via email to