[dpdk-dev] [PATCH v4 2/8] lib/librte_ether: defind RX/TX lock mode

2016-06-14 Thread Ananyev, Konstantin
Hi Wenzhuo, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 3. I thought the plan was to introduce a locking in all > > > > > > > > appropriate control path functions (dev_start/dev_stop etc.) > > > > > > > > Without that locking version of RX/TX seems a bit useless. > > > > > > >

[dpdk-dev] [PATCH v4 2/8] lib/librte_ether: defind RX/TX lock mode

2016-06-14 Thread Lu, Wenzhuo
Hi Konstantin, > -Original Message- > From: Ananyev, Konstantin > Sent: Monday, June 13, 2016 6:48 PM > To: Lu, Wenzhuo; Tao, Zhe; dev at dpdk.org > Cc: Richardson, Bruce; Chen, Jing D; Liang, Cunming; Wu, Jingjing; Zhang, > Helin > Subject: RE: [PATCH v4 2/8] lib/librte_ether: defind RX/

[dpdk-dev] [PATCH v4 2/8] lib/librte_ether: defind RX/TX lock mode

2016-06-13 Thread Ananyev, Konstantin
Hi Wenzhuo, > > > > > > > > > > > > > > > > > > > > > > 3. I thought the plan was to introduce a locking in all > > > > > > appropriate control path functions (dev_start/dev_stop etc.) > > > > > > Without that locking version of RX/TX seems a bit useless. > > > > > > Yes, I understand that you do

[dpdk-dev] [PATCH v4 2/8] lib/librte_ether: defind RX/TX lock mode

2016-06-13 Thread Lu, Wenzhuo
Hi Konstantin, > -Original Message- > From: Ananyev, Konstantin > Sent: Monday, June 13, 2016 7:17 AM > To: Lu, Wenzhuo; Tao, Zhe; dev at dpdk.org > Cc: Richardson, Bruce; Chen, Jing D; Liang, Cunming; Wu, Jingjing; Zhang, > Helin > Subject: RE: [PATCH v4 2/8] lib/librte_ether: defind RX/T

[dpdk-dev] [PATCH v4 2/8] lib/librte_ether: defind RX/TX lock mode

2016-06-12 Thread Ananyev, Konstantin
Hi Wenzhuo, > > Hi Konstantin, > > > > -Original Message- > > From: Ananyev, Konstantin > > Sent: Wednesday, June 8, 2016 5:20 PM > > To: Lu, Wenzhuo; Tao, Zhe; dev at dpdk.org > > Cc: Richardson, Bruce; Chen, Jing D; Liang, Cunming; Wu, Jingjing; Zhang, > > Helin > > Subject: RE: [PAT

[dpdk-dev] [PATCH v4 2/8] lib/librte_ether: defind RX/TX lock mode

2016-06-12 Thread Lu, Wenzhuo
Hi Konstantin, > -Original Message- > From: Ananyev, Konstantin > Sent: Wednesday, June 8, 2016 5:20 PM > To: Lu, Wenzhuo; Tao, Zhe; dev at dpdk.org > Cc: Richardson, Bruce; Chen, Jing D; Liang, Cunming; Wu, Jingjing; Zhang, > Helin > Subject: RE: [PATCH v4 2/8] lib/librte_ether: defind

[dpdk-dev] [PATCH v4 2/8] lib/librte_ether: defind RX/TX lock mode

2016-06-08 Thread Ananyev, Konstantin
> > Hi Konstantin, > > > > -Original Message- > > From: Ananyev, Konstantin > > Sent: Tuesday, June 7, 2016 5:59 PM > > To: Tao, Zhe; dev at dpdk.org > > Cc: Lu, Wenzhuo; Richardson, Bruce; Chen, Jing D; Liang, Cunming; Wu, > > Jingjing; > > Zhang, Helin > > Subject: RE: [PATCH v4 2/8

[dpdk-dev] [PATCH v4 2/8] lib/librte_ether: defind RX/TX lock mode

2016-06-08 Thread Lu, Wenzhuo
Hi Konstantin, > -Original Message- > From: Ananyev, Konstantin > Sent: Tuesday, June 7, 2016 5:59 PM > To: Tao, Zhe; dev at dpdk.org > Cc: Lu, Wenzhuo; Richardson, Bruce; Chen, Jing D; Liang, Cunming; Wu, > Jingjing; > Zhang, Helin > Subject: RE: [PATCH v4 2/8] lib/librte_ether: defind

[dpdk-dev] [PATCH v4 2/8] lib/librte_ether: defind RX/TX lock mode

2016-06-07 Thread Zhe Tao
Define lock mode for RX/TX queue. Because when resetting the device we want the resetting thread to get the lock of the RX/TX queue to make sure the RX/TX is stopped. Using next ABI macro for this ABI change as it has too much impact. 7 APIs and 1 global variable are impacted. Signed-off-by: Wenz

[dpdk-dev] [PATCH v4 2/8] lib/librte_ether: defind RX/TX lock mode

2016-06-07 Thread Ananyev, Konstantin
Hi Zhe & Wenzhuo, Please find my comments below. BTW, for clarification - is that patch for 16.11? I believe it's too late to introduce such significant change in 16.07. Thanks Konstantin > Define lock mode for RX/TX queue. Because when resetting > the device we want the resetting thread to get