Commit 9ad3d6f7 authored by Emil Tantilov's avatar Emil Tantilov Committed by Jeff Kirsher

ixgbevf: minor cleanups for ixgbevf_set_itr()

adapter->rx_itr_setting is not a mask so check it with == instead of &
do not default to 12K interrupts in ixgbevf_set_itr()

There should be no functional effect from these changes.
Signed-off-by: default avatarEmil Tantilov <emil.s.tantilov@intel.com>
Tested-by: default avatarPhil Schmitt <phillip.j.schmitt@intel.com>
Signed-off-by: default avatarJeff Kirsher <jeffrey.t.kirsher@intel.com>
parent d0f71aff
...@@ -1047,7 +1047,7 @@ static int ixgbevf_poll(struct napi_struct *napi, int budget) ...@@ -1047,7 +1047,7 @@ static int ixgbevf_poll(struct napi_struct *napi, int budget)
return budget; return budget;
/* all work done, exit the polling mode */ /* all work done, exit the polling mode */
napi_complete_done(napi, work_done); napi_complete_done(napi, work_done);
if (adapter->rx_itr_setting & 1) if (adapter->rx_itr_setting == 1)
ixgbevf_set_itr(q_vector); ixgbevf_set_itr(q_vector);
if (!test_bit(__IXGBEVF_DOWN, &adapter->state) && if (!test_bit(__IXGBEVF_DOWN, &adapter->state) &&
!test_bit(__IXGBEVF_REMOVING, &adapter->state)) !test_bit(__IXGBEVF_REMOVING, &adapter->state))
...@@ -1250,9 +1250,10 @@ static void ixgbevf_set_itr(struct ixgbevf_q_vector *q_vector) ...@@ -1250,9 +1250,10 @@ static void ixgbevf_set_itr(struct ixgbevf_q_vector *q_vector)
new_itr = IXGBE_20K_ITR; new_itr = IXGBE_20K_ITR;
break; break;
case bulk_latency: case bulk_latency:
default:
new_itr = IXGBE_12K_ITR; new_itr = IXGBE_12K_ITR;
break; break;
default:
break;
} }
if (new_itr != q_vector->itr) { if (new_itr != q_vector->itr) {
......
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