Commit b9f6ed2b authored by Alexander Duyck's avatar Alexander Duyck Committed by Jeff Kirsher

ixgbe: Drop unnecessary napi_schedule_prep and spare blank line from ixgbe_intr

This patch is a minor cleanup to address the unnecessary use of
napi_schedule_prep in ixgbe_intr and to also remove a blank line that is
not needed since it is separating a comment from the line it is explaining.
Signed-off-by: default avatarAlexander Duyck <alexander.h.duyck@intel.com>
Tested-by: default avatarStephen Ko <stephen.s.ko@intel.com>
Signed-off-by: default avatarJeff Kirsher <jeffrey.t.kirsher@intel.com>
parent 8f15486d
...@@ -2401,16 +2401,13 @@ static irqreturn_t ixgbe_intr(int irq, void *data) ...@@ -2401,16 +2401,13 @@ static irqreturn_t ixgbe_intr(int irq, void *data)
ixgbe_check_fan_failure(adapter, eicr); ixgbe_check_fan_failure(adapter, eicr);
if (napi_schedule_prep(&(q_vector->napi))) { /* would disable interrupts here but EIAM disabled it */
/* would disable interrupts here but EIAM disabled it */ napi_schedule(&q_vector->napi);
__napi_schedule(&(q_vector->napi));
}
/* /*
* re-enable link(maybe) and non-queue interrupts, no flush. * re-enable link(maybe) and non-queue interrupts, no flush.
* ixgbe_poll will re-enable the queue interrupts * ixgbe_poll will re-enable the queue interrupts
*/ */
if (!test_bit(__IXGBE_DOWN, &adapter->state)) if (!test_bit(__IXGBE_DOWN, &adapter->state))
ixgbe_irq_enable(adapter, false, false); ixgbe_irq_enable(adapter, false, false);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment