ixgbe: fix EICR write in ixgbe_msix_other
authorJacob Keller <jacob.e.keller@intel.com>
Sat, 2 Mar 2013 07:51:42 +0000 (07:51 +0000)
committerBen Hutchings <ben@decadent.org.uk>
Mon, 13 May 2013 14:02:26 +0000 (15:02 +0100)
commitded197648dfd25c4c9b1a1a7ab60d82256da82a2
treeffc1b2a88bbf481bb6ab1610f0dad34771bd553f
parent7210dd24ee9e4dbb8f16cfe11d1d4319909bfe34
ixgbe: fix EICR write in ixgbe_msix_other

commit d87d830720a1446403ed38bfc2da268be0d356d1 upstream.

Previously, the ixgbe_msix_other was writing the full 32bits of the set
interrupts, instead of only the ones which the ixgbe_msix_other is
handling. This resulted in a loss of performance when the X540's PPS feature is
enabled due to sometimes clearing queue interrupts which resulted in the driver
not getting the interrupt for cleaning the q_vector rings often enough. The fix
is to simply mask the lower 16bits off so that this handler does not write them
in the EICR, which causes them to remain high and be properly handled by the
clean_rings interrupt routine as normal.

Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>
Tested-by: Phil Schmitt <phillip.j.schmitt@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
drivers/net/ethernet/intel/ixgbe/ixgbe_main.c