net: hns3: fix __QUEUE_STATE_STACK_XOFF not cleared issue

When change MTU or other operations, which just calling .reset_notify
to do HNAE3_DOWN_CLIENT and HNAE3_UP_CLIENT, then
the netdev_tx_reset_queue() in the hns3_clear_all_ring() will be
ignored. So the dev_watchdog() may misdiagnose a TX timeout.

This patch separates netdev_tx_reset_queue() from
hns3_clear_all_ring(), and unifies hns3_clear_all_ring() and
hns3_force_clear_all_ring into one, since they are doing
similar things.

Fixes: 47dd920badb4 ("net: hns3: delay ring buffer clearing during reset")
Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3_enet.c b/drivers/net/ethernet/hisilicon/hns3/hns3_enet.c
index b23652cd29221162d2f257691d3d9200e22f2ee9..a5d46f0a50ccfc42a6629c8bfda6069af3837d72 100644 (file)
--- a/drivers/net/ethernet/hisilicon/hns3/hns3_enet.c
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3_enet.c
@@ -28,8 +28,7 @@
 #define hns3_set_field(origin, shift, val)     ((origin) |= ((val) << (shift)))
 #define hns3_tx_bd_count(S)    DIV_ROUND_UP(S, HNS3_MAX_BD_SIZE)
 
-static void hns3_clear_all_ring(struct hnae3_handle *h);
-static void hns3_force_clear_all_ring(struct hnae3_handle *h);
+static void hns3_clear_all_ring(struct hnae3_handle *h, bool force);
 static void hns3_remove_hw_addr(struct net_device *netdev);
 
 static const char hns3_driver_name[] = "hns3";
@@ -463,6 +462,20 @@ static int hns3_nic_net_open(struct net_device *netdev)
        return 0;
 }
 
+static void hns3_reset_tx_queue(struct hnae3_handle *h)
+{
+       struct net_device *ndev = h->kinfo.netdev;
+       struct hns3_nic_priv *priv = netdev_priv(ndev);
+       struct netdev_queue *dev_queue;
+       u32 i;
+
+       for (i = 0; i < h->kinfo.num_tqps; i++) {
+               dev_queue = netdev_get_tx_queue(ndev,
+                                               priv->ring_data[i].queue_index);
+               netdev_tx_reset_queue(dev_queue);
+       }
+}
+
 static void hns3_nic_net_down(struct net_device *netdev)
 {
        struct hns3_nic_priv *priv = netdev_priv(netdev);
@@ -493,7 +506,9 @@ static void hns3_nic_net_down(struct net_device *netdev)
         * to disable the ring through firmware when downing the netdev.
         */
        if (!hns3_nic_resetting(netdev))
-               hns3_clear_all_ring(priv->ae_handle);
+               hns3_clear_all_ring(priv->ae_handle, false);
+
+       hns3_reset_tx_queue(priv->ae_handle);
 }
 
 static int hns3_nic_net_stop(struct net_device *netdev)
@@ -3921,7 +3936,7 @@ static void hns3_client_uninit(struct hnae3_handle *handle, bool reset)
 
        hns3_del_all_fd_rules(netdev, true);
 
-       hns3_force_clear_all_ring(handle);
+       hns3_clear_all_ring(handle, true);
 
        hns3_nic_uninit_vector_data(priv);
 
@@ -4090,43 +4105,26 @@ static void hns3_force_clear_rx_ring(struct hns3_enet_ring *ring)
        }
 }
 
-static void hns3_force_clear_all_ring(struct hnae3_handle *h)
-{
-       struct net_device *ndev = h->kinfo.netdev;
-       struct hns3_nic_priv *priv = netdev_priv(ndev);
-       struct hns3_enet_ring *ring;
-       u32 i;
-
-       for (i = 0; i < h->kinfo.num_tqps; i++) {
-               ring = priv->ring_data[i].ring;
-               hns3_clear_tx_ring(ring);
-
-               ring = priv->ring_data[i + h->kinfo.num_tqps].ring;
-               hns3_force_clear_rx_ring(ring);
-       }
-}
-
-static void hns3_clear_all_ring(struct hnae3_handle *h)
+static void hns3_clear_all_ring(struct hnae3_handle *h, bool force)
 {
        struct net_device *ndev = h->kinfo.netdev;
        struct hns3_nic_priv *priv = netdev_priv(ndev);
        u32 i;
 
        for (i = 0; i < h->kinfo.num_tqps; i++) {
-               struct netdev_queue *dev_queue;
                struct hns3_enet_ring *ring;
 
                ring = priv->ring_data[i].ring;
                hns3_clear_tx_ring(ring);
-               dev_queue = netdev_get_tx_queue(ndev,
-                                               priv->ring_data[i].queue_index);
-               netdev_tx_reset_queue(dev_queue);
 
                ring = priv->ring_data[i + h->kinfo.num_tqps].ring;
                /* Continue to clear other rings even if clearing some
                 * rings failed.
                 */
-               hns3_clear_rx_ring(ring);
+               if (force)
+                       hns3_force_clear_rx_ring(ring);
+               else
+                       hns3_clear_rx_ring(ring);
        }
 }
 
@@ -4331,8 +4329,8 @@ static int hns3_reset_notify_uninit_enet(struct hnae3_handle *handle)
                return 0;
        }
 
-       hns3_clear_all_ring(handle);
-       hns3_force_clear_all_ring(handle);
+       hns3_clear_all_ring(handle, true);
+       hns3_reset_tx_queue(priv->ae_handle);
 
        hns3_nic_uninit_vector_data(priv);