ixgbevf: rewrite watchdog task to function similar to igbvf

This patch cleans up the logic dealing with link down/up by breaking down the
link detection and up/down events into separate functions - similar to how these
events are handled in other drivers.

CC: Alexander Duyck <alexander.h.duyck@redhat.com>
Signed-off-by: Emil Tantilov <emil.s.tantilov@intel.com>
Tested-by: Phil Schmitt <phillip.j.schmitt@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>

2 files changed, 113 insertions, 75 deletions

diff --git a/drivers/net/ethernet/intel/ixgbevf/ixgbevf.h b/drivers/net/ethernet/intel/ixgbevf/ixgbevf.h
index 65c2aee2a083..a41ab370278f 100644
--- a/drivers/net/ethernet/intel/ixgbevf/ixgbevf.h
+++ b/drivers/net/ethernet/intel/ixgbevf/ixgbevf.h
@@ -436,6 +436,7 @@ struct ixgbevf_adapter {
         bool link_up;
 
         spinlock_t mbx_lock;
+        unsigned long last_reset;
 
         struct work_struct watchdog_task;
 };
diff --git a/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c b/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c
index 87f9f8686b6f..c1100654a4be 100644
--- a/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c
+++ b/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c
@@ -2231,6 +2231,8 @@ void ixgbevf_reset(struct ixgbevf_adapter *adapter)
                 memcpy(netdev->perm_addr, adapter->hw.mac.addr,
                        netdev->addr_len);
         }
+
+        adapter->last_reset = jiffies;
 }
 
 static int ixgbevf_acquire_msix_vectors(struct ixgbevf_adapter *adapter,
@@ -2684,7 +2686,8 @@ void ixgbevf_update_stats(struct ixgbevf_adapter *adapter)
         struct ixgbe_hw *hw = &adapter->hw;
         int i;
 
-        if (!adapter->link_up)
+        if (test_bit(__IXGBEVF_DOWN, &adapter->state) ||
+            test_bit(__IXGBEVF_RESETTING, &adapter->state))
                 return;
 
         UPDATE_VF_COUNTER_32bit(IXGBE_VFGPRC, adapter->stats.last_vfgprc,
@@ -2714,17 +2717,45 @@ void ixgbevf_update_stats(struct ixgbevf_adapter *adapter)
 static void ixgbevf_watchdog(unsigned long data)
 {
         struct ixgbevf_adapter *adapter = (struct ixgbevf_adapter *)data;
+
+        /* Do the reset outside of interrupt context */
+        schedule_work(&adapter->watchdog_task);
+}
+
+static void ixgbevf_reset_task(struct work_struct *work)
+{
+        struct ixgbevf_adapter *adapter;
+
+        adapter = container_of(work, struct ixgbevf_adapter, reset_task);
+
+        /* If we're already down or resetting, just bail */
+        if (test_bit(__IXGBEVF_DOWN, &adapter->state) ||
+            test_bit(__IXGBEVF_RESETTING, &adapter->state))
+                return;
+
+        adapter->tx_timeout_count++;
+
+        ixgbevf_reinit_locked(adapter);
+}
+
+/* ixgbevf_check_hang_subtask - check for hung queues and dropped interrupts
+ * @adapter - pointer to the device adapter structure
+ *
+ * This function serves two purposes.  First it strobes the interrupt lines
+ * in order to make certain interrupts are occurring.  Secondly it sets the
+ * bits needed to check for TX hangs.  As a result we should immediately
+ * determine if a hang has occurred.
+ */
+static void ixgbevf_check_hang_subtask(struct ixgbevf_adapter *adapter)
+{
         struct ixgbe_hw *hw = &adapter->hw;
         u32 eics = 0;
         int i;
 
-        /*
-         * Do the watchdog outside of interrupt context due to the lovely
-         * delays that some of the newer hardware requires
-         */
-
-        if (test_bit(__IXGBEVF_DOWN, &adapter->state))
-                goto watchdog_short_circuit;
+        /* If we're down or resetting, just bail */
+        if (test_bit(__IXGBEVF_DOWN, &adapter->state) ||
+            test_bit(__IXGBEVF_RESETTING, &adapter->state))
+                return;
 
         /* Force detection of hung controller */
         if (netif_carrier_ok(adapter->netdev)) {
@@ -2739,26 +2770,80 @@ static void ixgbevf_watchdog(unsigned long data)
                         eics |= 1 << i;
         }
 
+        /* Cause software interrupt to ensure rings are cleaned */
         IXGBE_WRITE_REG(hw, IXGBE_VTEICS, eics);
+}
 
-watchdog_short_circuit:
-        schedule_work(&adapter->watchdog_task);
+/**
+ * ixgbevf_watchdog_update_link - update the link status
+ * @adapter - pointer to the device adapter structure
+ **/
+static void ixgbevf_watchdog_update_link(struct ixgbevf_adapter *adapter)
+{
+        struct ixgbe_hw *hw = &adapter->hw;
+        u32 link_speed = adapter->link_speed;
+        bool link_up = adapter->link_up;
+        s32 err;
+
+        spin_lock_bh(&adapter->mbx_lock);
+
+        err = hw->mac.ops.check_link(hw, &link_speed, &link_up, false);
+
+        spin_unlock_bh(&adapter->mbx_lock);
+
+        /* if check for link returns error we will need to reset */
+        if (err && time_after(jiffies, adapter->last_reset + (10 * HZ))) {
+                schedule_work(&adapter->reset_task);
+                link_up = false;
+        }
+
+        adapter->link_up = link_up;
+        adapter->link_speed = link_speed;
 }
 
-static void ixgbevf_reset_task(struct work_struct *work)
+/**
+ * ixgbevf_watchdog_link_is_up - update netif_carrier status and
+ *                               print link up message
+ * @adapter - pointer to the device adapter structure
+ **/
+static void ixgbevf_watchdog_link_is_up(struct ixgbevf_adapter *adapter)
 {
-        struct ixgbevf_adapter *adapter;
-        adapter = container_of(work, struct ixgbevf_adapter, reset_task);
+        struct net_device *netdev = adapter->netdev;
 
-        /* If we're already down or resetting, just bail */
-        if (test_bit(__IXGBEVF_DOWN, &adapter->state) ||
-            test_bit(__IXGBEVF_REMOVING, &adapter->state) ||
-            test_bit(__IXGBEVF_RESETTING, &adapter->state))
+        /* only continue if link was previously down */
+        if (netif_carrier_ok(netdev))
                 return;
 
-        adapter->tx_timeout_count++;
+        dev_info(&adapter->pdev->dev, "NIC Link is Up %s\n",
+                 (adapter->link_speed == IXGBE_LINK_SPEED_10GB_FULL) ?
+                 "10 Gbps" :
+                 (adapter->link_speed == IXGBE_LINK_SPEED_1GB_FULL) ?
+                 "1 Gbps" :
+                 (adapter->link_speed == IXGBE_LINK_SPEED_100_FULL) ?
+                 "100 Mbps" :
+                 "unknown speed");
 
-        ixgbevf_reinit_locked(adapter);
+        netif_carrier_on(netdev);
+}
+
+/**
+ * ixgbevf_watchdog_link_is_down - update netif_carrier status and
+ *                                 print link down message
+ * @adapter - pointer to the adapter structure
+ **/
+static void ixgbevf_watchdog_link_is_down(struct ixgbevf_adapter *adapter)
+{