Re: [E1000-devel] Possible bug in the i40evf driver

2017-08-02 Thread Alexander Duyck
On Tue, Aug 1, 2017 at 8:29 AM, Codrut Grosu wrote: >Hi, > > We believe there might be a bug in the i40evf driver. > > We think that there is race issue between i40evf_reset_task and i40evf_down > / i40evf_open. > The reason is that the functions napi_enable /

Re: [E1000-devel] Possible bug in the i40evf driver

2017-08-02 Thread Fujinaka, Todd
Our developer thinks the i40evf driver is guarded against the race issue as you mentioned. The main thing is in i40evf driver, the check/set "while (test_and_set_bit(__I40EVF_IN_CRITICAL_TASK, >crit_section))" is present ahead of the call of i40evf_napi_disable_all(adapter) in i40evf_reset