Re: [dpdk-dev] [PATCH] igb_uio: revert open and release operations

2017-10-24 Thread Ferruh Yigit
On 10/17/2017 1:14 PM, Ferruh Yigit wrote: > This reverts commit 6b9ed026a8704b9e5ee5da7997617ef7cc82e114. > This reverts commit 5f6ff30dc5075c49069d684bab229aef7ff0fdc3. > This reverts commit b58eedfc7dd57eef6d12e2c654a52c834f36084a. > > There were bug reports about terminated application may lea

Re: [dpdk-dev] [PATCH] igb_uio: revert open and release operations

2017-10-20 Thread Ferruh Yigit
On 10/20/2017 8:26 AM, Tan, Jianfeng wrote: > Hi Ferruh & Harish, > > > On 10/20/2017 9:15 AM, Ferruh Yigit wrote: >> On 10/19/2017 3:43 PM, Patil, Harish wrote: >>> -Original Message- >>> From: Harish Patil >>> Date: Tuesday, October 17, 2017 at 9:50 PM >>> To: Thomas Monjalon , Ferruh

Re: [dpdk-dev] [PATCH] igb_uio: revert open and release operations

2017-10-20 Thread Tan, Jianfeng
Hi Ferruh & Harish, On 10/20/2017 9:15 AM, Ferruh Yigit wrote: On 10/19/2017 3:43 PM, Patil, Harish wrote: -Original Message- From: Harish Patil Date: Tuesday, October 17, 2017 at 9:50 PM To: Thomas Monjalon , Ferruh Yigit Cc: "dev@dpdk.org" , Jianfeng Tan , Jingjing Wu , "Thotton, S

Re: [dpdk-dev] [PATCH] igb_uio: revert open and release operations

2017-10-19 Thread Ferruh Yigit
On 10/19/2017 3:43 PM, Patil, Harish wrote: > -Original Message- > From: Harish Patil > Date: Tuesday, October 17, 2017 at 9:50 PM > To: Thomas Monjalon , Ferruh Yigit > > Cc: "dev@dpdk.org" , Jianfeng Tan , > Jingjing Wu , "Thotton, Shijith" > , Gregory Etelson , George > Prekas , "sta..

Re: [dpdk-dev] [PATCH] igb_uio: revert open and release operations

2017-10-19 Thread Patil, Harish
-Original Message- From: Harish Patil Date: Tuesday, October 17, 2017 at 9:50 PM To: Thomas Monjalon , Ferruh Yigit Cc: "dev@dpdk.org" , Jianfeng Tan , Jingjing Wu , "Thotton, Shijith" , Gregory Etelson , George Prekas , "sta...@dpdk.org" Subject: Re: [PATCH] igb_uio: revert open and rel

Re: [dpdk-dev] [PATCH] igb_uio: revert open and release operations

2017-10-18 Thread Ferruh Yigit
On 10/17/2017 11:27 PM, Shijith Thotton wrote: > On Tue, Oct 17, 2017 at 09:14:36PM +0100, Ferruh Yigit wrote: >> This reverts commit 6b9ed026a8704b9e5ee5da7997617ef7cc82e114. >> This reverts commit 5f6ff30dc5075c49069d684bab229aef7ff0fdc3. >> This reverts commit b58eedfc7dd57eef6d12e2c654a52c834f3

Re: [dpdk-dev] [PATCH] igb_uio: revert open and release operations

2017-10-17 Thread Shijith Thotton
On Tue, Oct 17, 2017 at 09:14:36PM +0100, Ferruh Yigit wrote: > This reverts commit 6b9ed026a8704b9e5ee5da7997617ef7cc82e114. > This reverts commit 5f6ff30dc5075c49069d684bab229aef7ff0fdc3. > This reverts commit b58eedfc7dd57eef6d12e2c654a52c834f36084a. > > There were bug reports about terminated

Re: [dpdk-dev] [PATCH] igb_uio: revert open and release operations

2017-10-17 Thread Patil, Harish
-Original Message- From: Thomas Monjalon Date: Tuesday, October 17, 2017 at 1:33 PM To: Ferruh Yigit , Harish Patil Cc: "dev@dpdk.org" , Jianfeng Tan , Jingjing Wu , "Thotton, Shijith" , Gregory Etelson , George Prekas , "sta...@dpdk.org" Subject: Re: [PATCH] igb_uio: revert open and r

Re: [dpdk-dev] [PATCH] igb_uio: revert open and release operations

2017-10-17 Thread Wu, Jingjing
Hi, Ferruh This one is generic, we can keep it. Commit: 8cacf78469a7 ("net/i40e: fix VF initialization error") Thanks Jingjing > -Original Message- > From: Yigit, Ferruh > Sent: Wednesday, October 18, 2017 4:15 AM > To: Thomas Monjalon ; Yigit, Ferruh > > Cc: dev@dpdk.org; Tan, Jianfen

Re: [dpdk-dev] [PATCH] igb_uio: revert open and release operations

2017-10-17 Thread Thomas Monjalon
17/10/2017 22:14, Ferruh Yigit: > There were bug reports about terminated application may leave device in > undesired state: > http://dpdk.org/ml/archives/dev/2016-November/049745.html > http://dpdk.org/ml/archives/dev/2016-November/050932.html > > And a proposal to fix: > http://dpdk.org/ml/archi

[dpdk-dev] [PATCH] igb_uio: revert open and release operations

2017-10-17 Thread Ferruh Yigit
This reverts commit 6b9ed026a8704b9e5ee5da7997617ef7cc82e114. This reverts commit 5f6ff30dc5075c49069d684bab229aef7ff0fdc3. This reverts commit b58eedfc7dd57eef6d12e2c654a52c834f36084a. There were bug reports about terminated application may leave device in undesired state: http://dpdk.org/ml/arch

Re: [dpdk-dev] [PATCH] igb_uio: revert open and release operations

2017-10-13 Thread Thomas Monjalon
13/10/2017 23:11, Thomas Monjalon: > 13/10/2017 23:05, Ferruh Yigit: > > On 10/13/2017 3:51 PM, Thomas Monjalon wrote: > > > Some VF drivers cannot work with igb_uio because of the > > > reset done in these functions. > > > > > > First bug report: > > > http://dpdk.org/ml/archives/dev/2017-Septe

Re: [dpdk-dev] [PATCH] igb_uio: revert open and release operations

2017-10-13 Thread Thomas Monjalon
13/10/2017 23:05, Ferruh Yigit: > On 10/13/2017 3:51 PM, Thomas Monjalon wrote: > > Some VF drivers cannot work with igb_uio because of the > > reset done in these functions. > > > > First bug report: > > http://dpdk.org/ml/archives/dev/2017-September/075236.html > > > > A partial reset was t

Re: [dpdk-dev] [PATCH] igb_uio: revert open and release operations

2017-10-13 Thread Ferruh Yigit
On 10/13/2017 3:51 PM, Thomas Monjalon wrote: > Some VF drivers cannot work with igb_uio because of the > reset done in these functions. > > First bug report: > http://dpdk.org/ml/archives/dev/2017-September/075236.html > > A partial reset was tried: > http://dpdk.org/patch/28940 > >

[dpdk-dev] [PATCH] igb_uio: revert open and release operations

2017-10-13 Thread Thomas Monjalon
Some VF drivers cannot work with igb_uio because of the reset done in these functions. First bug report: http://dpdk.org/ml/archives/dev/2017-September/075236.html A partial reset was tried: http://dpdk.org/patch/28940 Second bug report after a partial revert trial: http: