ixgbe: Correct Adaptive Interrupt Moderation so that it will change values
authorAlexander Duyck <alexander.h.duyck@intel.com>
Wed, 8 Feb 2012 07:49:43 +0000 (07:49 +0000)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Tue, 13 Mar 2012 03:29:40 +0000 (20:29 -0700)
This change corrects an issue in which Adaptive Interrupt Moderation was
not changing values due to the fact that we were performing an and
operation on the resultant value that was causing the value to never change
from the default 20K interrupts per second.

Signed-off-by: Alexander Duyck <alexander.h.duyck@intel.com>
Tested-by: Stephen Ko <stephen.s.ko@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/ixgbe/ixgbe_main.c

index fc3c33a..bad9c05 100644 (file)
@@ -1834,7 +1834,7 @@ void ixgbe_write_eitr(struct ixgbe_q_vector *q_vector)
        struct ixgbe_adapter *adapter = q_vector->adapter;
        struct ixgbe_hw *hw = &adapter->hw;
        int v_idx = q_vector->v_idx;
-       u32 itr_reg = q_vector->itr;
+       u32 itr_reg = q_vector->itr & IXGBE_MAX_EITR;
 
        switch (adapter->hw.mac.type) {
        case ixgbe_mac_82598EB:
@@ -1886,7 +1886,7 @@ static void ixgbe_set_itr(struct ixgbe_q_vector *q_vector)
                          ((9 * new_itr) + q_vector->itr);
 
                /* save the algorithm value here */
-               q_vector->itr = new_itr & IXGBE_MAX_EITR;
+               q_vector->itr = new_itr;
 
                ixgbe_write_eitr(q_vector);
        }