Re: [dpdk-dev] [PATCH v2 19/25] drivers/net: check process type in close operation

2020-09-28 Thread Thomas Monjalon
28/09/2020 20:51, Stephen Hemminger: > On Mon, 28 Sep 2020 01:42:43 +0200 > Thomas Monjalon wrote: > > > The secondary processes are not allowed to release shared resources. > > Only process-private ressources should be freed in a secondary process. > > Most of the time, there is no process-priva

Re: [dpdk-dev] [PATCH v2 19/25] drivers/net: check process type in close operation

2020-09-28 Thread Ajit Khaparde
On Mon, Sep 28, 2020 at 2:56 AM Sachin Saxena (OSS) wrote: > > For dpaa, dpaa2, enetc and pfe > > Reviewed-by: Sachin Saxena > > On 28-Sep-20 5:12 AM, Thomas Monjalon wrote: > > The secondary processes are not allowed to release shared resources. > Only process-private ressources should be freed

Re: [dpdk-dev] [PATCH v2 19/25] drivers/net: check process type in close operation

2020-09-28 Thread Stephen Hemminger
On Mon, 28 Sep 2020 01:42:43 +0200 Thomas Monjalon wrote: > The secondary processes are not allowed to release shared resources. > Only process-private ressources should be freed in a secondary process. > Most of the time, there is no process-private ressource, > so the close operation is just fo

Re: [dpdk-dev] [PATCH v2 19/25] drivers/net: check process type in close operation

2020-09-28 Thread Sachin Saxena (OSS)
For dpaa, dpaa2, enetc and pfe Reviewed-by: Sachin Saxena On 28-Sep-20 5:12 AM, Thomas Monjalon wrote: The secondary processes are not allowed to release shared resources. Only process-private ressources should be freed in a secondary process. Most of the time, there is no process-private resso

Re: [dpdk-dev] [PATCH v2 19/25] drivers/net: check process type in close operation

2020-09-27 Thread Xu, Rosen
Hi, > -Original Message- > From: Thomas Monjalon > Sent: Monday, September 28, 2020 7:43 > To: dev@dpdk.org > Cc: Yigit, Ferruh ; arybche...@solarflare.com; > Loftus, Ciara ; Zhang, Qi Z ; > Shepard Siegel ; Ed Czeck > ; John Miller ; > Steven Webster ; Matt Peters > ; Ajit Khaparde > ; S

Re: [dpdk-dev] [PATCH v2 19/25] drivers/net: check process type in close operation

2020-09-27 Thread Thomas Monjalon
> The secondary processes are not allowed to release shared resources. > Only process-private ressources should be freed in a secondary process. > Most of the time, there is no process-private ressource, > so the close operation is just forbidden in a secondary process. Missed a (double) typo in t

[dpdk-dev] [PATCH v2 19/25] drivers/net: check process type in close operation

2020-09-27 Thread Thomas Monjalon
The secondary processes are not allowed to release shared resources. Only process-private ressources should be freed in a secondary process. Most of the time, there is no process-private ressource, so the close operation is just forbidden in a secondary process. After adding proper check in the po