> -----Original Message-----
> From: Thomas Monjalon [mailto:tho...@monjalon.net]
> Sent: Thursday, November 8, 2018 5:26 AM
> To: Stojaczyk, Dariusz <dariusz.stojac...@intel.com>
> Cc: dev@dpdk.org; gaetan.ri...@6wind.com; Zhang, Qi Z
> <qi.z.zh...@intel.com>; Guo, Jia <jia....@intel.com>
> Subject: Re: [PATCH] devargs: do not replace already inserted devargs
>
> 08/11/2018 12:25, Stojaczyk, Dariusz:
> > From: Thomas Monjalon [mailto:tho...@monjalon.net]
> > >
> > > 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 <tho...@monjalon.net>
> >
> > Tested-by: Darek Stojaczyk <dariusz.stojac...@intel.com>
>
> Is it fixing any use case?
Tested-by: Qi Zhang <qi.z.zh...@intel.com>
The patch also fix below scenario
attach net_af_packet,iface=enp50s0f0
attach net_af_packet,iface=enp50s0f0 (failed as expected)
detach net_af_packet (failed)
Thanks!
Qi
>
