> -----Original Message----- > From: Shijith Thotton [mailto:[email protected]] > Sent: Monday, October 23, 2017 8:28 PM > To: Yigit, Ferruh <[email protected]> > Cc: Wu, Jingjing <[email protected]>; Gregory Etelson <[email protected]>; > Harish Patil <[email protected]>; Thomas Monjalon > <[email protected]>; [email protected]; [email protected]; Tan, Jianfeng > <[email protected]>; George Prekas <[email protected]>; Gonzalez > Monroy, Sergio <[email protected]> > Subject: Re: [PATCH] igb_uio: remove device reset in open > > On Fri, Oct 20, 2017 at 09:57:38AM -0700, Ferruh Yigit wrote: > > On 10/20/2017 9:55 AM, Ferruh Yigit wrote: > > > Remove device reset during application start, the reset for > > > application exit still there. > > > > > > Reset in open removed because of following comments: > > > 1- Device reset not completed when VF driver loaded, which cause VF PMD > > > initialization error. > > > Adding delay can solve the issue but will increase driver load time. > > > > > > 2- Reset will be issues all devices unconditionally, not very efficient > > > way. > > > > > > Fixes: b58eedfc7dd5 ("igb_uio: issue FLR during open and release of > > > device file") > > > Cc: [email protected] > > > > > > Signed-off-by: Ferruh Yigit <[email protected]> > > > > Hi Jingjing, Shijith, Gregory, Harish, > > > > Can you please test this on top of current master (which has already > > Jingjin's > > fix) ? > > Hi Ferruh, > > Tested for LiquidIO cards and it works. > > Thanks, > Shijith
Hi, Ferruh For i40e, it's OK too. Thanks Jingjing
