[dpdk-dev] [PATCH v2 1/2] librte_ether: ensure not overwrite device data in mp app

2016-09-26 Thread Kerlin, MarcinX
Hi Thomas, > -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Friday, September 23, 2016 4:13 PM > To: Kerlin, MarcinX > Cc: Pattan, Reshma ; dev at dpdk.org; De Lara > Guarch, Pablo > Subject: Re: [dpdk-dev] [PATCH v2 1/2]

[dpdk-dev] [PATCH v2 1/2] librte_ether: ensure not overwrite device data in mp app

2016-09-23 Thread Thomas Monjalon
2016-09-22 14:11, Kerlin, MarcinX: > Hi Reshma, > > From: Pattan, Reshma > > > > Hi Marcin, > > > > > /** > > > * @internal > > > + * Returns a shared device data slot specified by the unique identifier > > > name. > > > + * > > > + * @paramname > > > + * The pointer to the Unique i

[dpdk-dev] [PATCH v2 1/2] librte_ether: ensure not overwrite device data in mp app

2016-09-22 Thread Kerlin, MarcinX
Hi, > -Original Message- > From: Pattan, Reshma > Sent: Tuesday, September 20, 2016 6:48 PM > To: Kerlin, MarcinX ; dev at dpdk.org > Cc: thomas.monjalon at 6wind.com; De Lara Guarch, Pablo > ; Kerlin, MarcinX > > Subject: RE: [dpdk-dev] [PATCH v2 1/2] librte_eth

[dpdk-dev] [PATCH v2 1/2] librte_ether: ensure not overwrite device data in mp app

2016-09-22 Thread Kerlin, MarcinX
Hi Reshma, > -Original Message- > From: Pattan, Reshma > Sent: Tuesday, September 20, 2016 6:14 PM > To: Kerlin, MarcinX ; dev at dpdk.org > Cc: thomas.monjalon at 6wind.com; De Lara Guarch, Pablo > ; Kerlin, MarcinX > > Subject: RE: [dpdk-dev] [PATCH v2 1/2]

[dpdk-dev] [PATCH v2 1/2] librte_ether: ensure not overwrite device data in mp app

2016-09-20 Thread Pattan, Reshma
Hi, > + > + if (dev_data_id == RTE_MAX_ETHPORTS) { > + RTE_PMD_DEBUG_TRACE("Reached maximum number of > Ethernet ports by all " > + "the processes\n"); > + return NULL; > + } > + > Can the log message be changed to ("Cannot allocate mor

[dpdk-dev] [PATCH v2 1/2] librte_ether: ensure not overwrite device data in mp app

2016-09-20 Thread Marcin Kerlin
Added prevention not overwrite device data in array rte_eth_dev_data[]. Secondary process appends in the first free place rather than at the beginning. This behavior prevents overwriting devices of primary process by secondary process. Signed-off-by: Marcin Kerlin --- lib/librte_ether/rte_ethdev

[dpdk-dev] [PATCH v2 1/2] librte_ether: ensure not overwrite device data in mp app

2016-09-20 Thread Pattan, Reshma
Hi Marcin, > /** > * @internal > + * Returns a shared device data slot specified by the unique identifier name. > + * > + * @paramname > + * The pointer to the Unique identifier name for each shared Ethernet > +device > + * between multiple processes. > + * @return > + * - The pointer