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]>
