Re: [dpdk-dev] [PATCH] devargs: do not replace already inserted devargs

2018-11-11 Thread Thomas Monjalon
08/11/2018 18:12, Zhang, Qi Z: > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > 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

Re: [dpdk-dev] [PATCH] devargs: do not replace already inserted devargs

2018-11-08 Thread Zhang, Qi Z
> -Original Message- > From: Thomas Monjalon [mailto:tho...@monjalon.net] > Sent: Thursday, November 8, 2018 5:26 AM > To: Stojaczyk, Dariusz > Cc: dev@dpdk.org; gaetan.ri...@6wind.com; Zhang, Qi Z > ; Guo, Jia > Subject: Re: [PATCH] devargs: do not replace already inserted devargs >

Re: [dpdk-dev] [PATCH] devargs: do not replace already inserted devargs

2018-11-08 Thread Slava Ovsiienko
> -Original Message- > From: dev On Behalf Of Stojaczyk, Dariusz > Sent: Thursday, November 8, 2018 13:25 > To: Thomas Monjalon ; dev@dpdk.org > Cc: gaetan.ri...@6wind.com; Zhang, Qi Z ; Guo, Jia > > Subject: Re: [dpdk-dev] [PATCH] devargs: do not replace alrea

Re: [dpdk-dev] [PATCH] devargs: do not replace already inserted devargs

2018-11-08 Thread Thomas Monjalon
08/11/2018 13:35, Stojaczyk, Dariusz: > From: Thomas Monjalon [mailto:tho...@monjalon.net] > > 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

Re: [dpdk-dev] [PATCH] devargs: do not replace already inserted devargs

2018-11-08 Thread Stojaczyk, Dariusz
> -Original Message- > From: Thomas Monjalon [mailto:tho...@monjalon.net] > Sent: Thursday, November 8, 2018 1:26 PM > To: Stojaczyk, Dariusz > Cc: dev@dpdk.org; gaetan.ri...@6wind.com; Zhang, Qi Z > ; Guo, Jia > Subject: Re: [PATCH] devargs: do not replace already inserted devargs > >

Re: [dpdk-dev] [PATCH] devargs: do not replace already inserted devargs

2018-11-08 Thread Thomas Monjalon
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.

Re: [dpdk-dev] [PATCH] devargs: do not replace already inserted devargs

2018-11-08 Thread Stojaczyk, Dariusz
> -Original Message- > From: Thomas Monjalon [mailto:tho...@monjalon.net] > Sent: Thursday, November 8, 2018 12:21 AM > To: dev@dpdk.org > Cc: gaetan.ri...@6wind.com; Zhang, Qi Z ; Guo, Jia > ; Stojaczyk, Dariusz > Subject: [PATCH] devargs: do not replace already inserted devargs > > Th

[dpdk-dev] [PATCH] devargs: do not replace already inserted devargs

2018-11-07 Thread Thomas Monjalon
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