i40evf: Drop i40evf_fire_sw_int as it is prone to races
authorAlexander Duyck <alexander.h.duyck@intel.com>
Tue, 14 Nov 2017 12:00:46 +0000 (07:00 -0500)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Wed, 10 Jan 2018 20:41:21 +0000 (12:41 -0800)
commita6cab7d7f9c8c1522444fee072c64dcff4392a3c
tree27374d408af2ad131d8f52b97da5b0e26480d6e7
parentf23735aa45f3a5cb301342a4f51be69f22d4aa85
i40evf: Drop i40evf_fire_sw_int as it is prone to races

Having the interrupts firing while we are polling causes extra overhead and
isn't needed for most systems out there. If an interrupt is lost us
experiencing a 2s latency spike before recovering is still not acceptable
and masks the issue. We are better off just identifying systems that lose
interrupts and instead enable workarounds for those systems.

To that end I am dropping the code that was strobing the interrupts as
there is a narrow window where having them enabled can actually cause
race issues anyway where a few stray packets might get misses if the
interrupt is re-enabled and fires before we call napi_complete.

Signed-off-by: Alexander Duyck <alexander.h.duyck@intel.com>
Tested-by: Andrew Bowers <andrewx.bowers@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/i40evf/i40evf_main.c