> -----Original Message----- > From: dev <[email protected]> On Behalf Of Stojaczyk, Dariusz > Sent: Thursday, November 8, 2018 13:25 > To: Thomas Monjalon <[email protected]>; [email protected] > Cc: [email protected]; Zhang, Qi Z <[email protected]>; Guo, Jia > <[email protected]> > Subject: Re: [dpdk-dev] [PATCH] devargs: do not replace already inserted > devargs > > > > -----Original Message----- > > From: Thomas Monjalon [mailto:[email protected]] > > Sent: Thursday, November 8, 2018 12:21 AM > > To: [email protected] > > Cc: [email protected]; Zhang, Qi Z <[email protected]>; Guo, > > Jia <[email protected]>; Stojaczyk, Dariusz > > <[email protected]> > > Subject: [PATCH] devargs: do not replace already inserted devargs > > > > The devargs of a device can be replaced by a newly allocated one when > > trying to probe again the same device (multi-process or multi-ports > > scenarios). This is breaking some pointer references. > > > > It can be avoided by copying the new content, freeing the new devargs, > > and returning the already inserted pointer. > > > > Signed-off-by: Thomas Monjalon <[email protected]> > > --- > > Tested-by: Darek Stojaczyk <[email protected]>
Attaching the representors caused segmentation fault. The scenario: - setup with SRIOV enabled, PF plus N VF - switchdev mode enabled (have representor for E-Switch ports) - run testpmd specifying PF only, one device probed and one port is created - trying to attach representor with port attach command - segmentation fault occurs within rte_eth_devargs_parse () The patch fixes this issue. Tested-by: Viacheslav Ovsiienko <[email protected]>
