Re: [dpdk-dev] [PATCH 3/4] drivers/net: do not allocate rte_eth_dev_data privately

2018-03-11 Thread Tan, Jianfeng
On 3/7/2018 2:10 PM, Matan Azrad wrote: Yes, we are removing all private rte_eth_dev_data. If I missed some device, welcome to point out. net/ring What is about next PCI device? net/cxgbe Will change these two in next version. Thank you, Matan.

Re: [dpdk-dev] [PATCH 3/4] drivers/net: do not allocate rte_eth_dev_data privately

2018-03-06 Thread Matan Azrad
2018 2:08 PM > > > To: Tan, Jianfeng; Yigit, Ferruh > > > Cc: Richardson, Bruce; Ananyev, Konstantin; Thomas Monjalon; > > > maxime.coque...@redhat.com; Burakov, Anatoly; dev@dpdk.org > > > Subject: RE: [dpdk-dev] [PATCH 3/4] drivers/net: do not allocate > >

Re: [dpdk-dev] [PATCH 3/4] drivers/net: do not allocate rte_eth_dev_data privately

2018-03-06 Thread Matan Azrad
v, Konstantin; Thomas Monjalon; > > maxime.coque...@redhat.com; Burakov, Anatoly; dev@dpdk.org > > Subject: RE: [dpdk-dev] [PATCH 3/4] drivers/net: do not allocate > > rte_eth_dev_data privately > > > > Hi Jianfeng > > > > Please see a comment bel

Re: [dpdk-dev] [PATCH 3/4] drivers/net: do not allocate rte_eth_dev_data privately

2018-03-06 Thread Tan, Jianfeng
t; Subject: RE: [dpdk-dev] [PATCH 3/4] drivers/net: do not allocate > rte_eth_dev_data privately > > Hi Jianfeng > > Please see a comment below. > > > From: Jianfeng Tan, Sent: Sunday, March 4, 2018 5:30 PM > > We introduced private rte_eth_dev_data to allow v

Re: [dpdk-dev] [PATCH 3/4] drivers/net: do not allocate rte_eth_dev_data privately

2018-03-05 Thread Matan Azrad
Hi Jianfeng Please see a comment below. > From: Jianfeng Tan, Sent: Sunday, March 4, 2018 5:30 PM > We introduced private rte_eth_dev_data to allow vdev to be created both in > primary process and secondary process(es). This is not friendly to multi- > process model, for example, it leads to port

[dpdk-dev] [PATCH 3/4] drivers/net: do not allocate rte_eth_dev_data privately

2018-03-04 Thread Jianfeng Tan
We introduced private rte_eth_dev_data to allow vdev to be created both in primary process and secondary process(es). This is not friendly to multi-process model, for example, it leads to port id contention issue if two processes both find the data entry is free. And to get stats of primary vdev i