readjust the execution conditions for dev_watchdog() and reduce
the tab indentation of the code.

Cc: Jamal Hadi Salim <jhs@mojatatu.com>
Cc: Cong Wang <xiyou.wangcong@gmail.com>
Cc: Jiri Pirko <jiri@resnulli.us>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Eric Dumazet <edumazet@google.com>
Cc: Jakub Kicinski <kuba@kernel.org>
Cc: Paolo Abeni <pabeni@redhat.com>
Cc: Simon Horman <horms@kernel.org>
Signed-off-by: Tonghao Zhang <tonghao@bamaicloud.com>
---
 net/sched/sch_generic.c | 82 ++++++++++++++++++++---------------------
 1 file changed, 40 insertions(+), 42 deletions(-)

diff --git a/net/sched/sch_generic.c b/net/sched/sch_generic.c
index 16afb834fe4a..1b905cc05520 100644
--- a/net/sched/sch_generic.c
+++ b/net/sched/sch_generic.c
@@ -500,51 +500,49 @@ static void dev_watchdog(struct timer_list *t)
 	bool release = true;
 
 	spin_lock(&dev->tx_global_lock);
-	if (!qdisc_tx_is_noop(dev)) {
-		if (netif_device_present(dev) &&
-		    netif_running(dev) &&
-		    netif_carrier_ok(dev)) {
-			unsigned int timedout_ms = 0;
-			unsigned int i;
-			unsigned long trans_start;
-			unsigned long oldest_start = jiffies;
-
-			for (i = 0; i < dev->num_tx_queues; i++) {
-				struct netdev_queue *txq;
-
-				txq = netdev_get_tx_queue(dev, i);
-				if (!netif_xmit_stopped(txq))
-					continue;
-
-				/* Paired with WRITE_ONCE() + smp_mb...() in
-				 * netdev_tx_sent_queue() and netif_tx_stop_queue().
-				 */
-				smp_mb();
-				trans_start = READ_ONCE(txq->trans_start);
-
-				if (time_after(jiffies, trans_start + dev->watchdog_timeo)) {
-					timedout_ms = jiffies_to_msecs(jiffies - trans_start);
-					atomic_long_inc(&txq->trans_timeout);
-					break;
-				}
-				if (time_after(oldest_start, trans_start))
-					oldest_start = trans_start;
-			}
+	if (!qdisc_tx_is_noop(dev) &&
+	    netif_device_present(dev) &&
+	    netif_running(dev) && netif_carrier_ok(dev)) {
+		unsigned int timedout_ms = 0;
+		unsigned int i;
+		unsigned long trans_start;
+		unsigned long oldest_start = jiffies;
+
+		for (i = 0; i < dev->num_tx_queues; i++) {
+			struct netdev_queue *txq;
+
+			txq = netdev_get_tx_queue(dev, i);
+			if (!netif_xmit_stopped(txq))
+				continue;
+
+			/* Paired with WRITE_ONCE() + smp_mb...() in
+			 * netdev_tx_sent_queue() and netif_tx_stop_queue().
+			 */
+			smp_mb();
+			trans_start = READ_ONCE(txq->trans_start);
 
-			if (unlikely(timedout_ms)) {
-				trace_net_dev_xmit_timeout(dev, i);
-				netdev_crit(dev, "NETDEV WATCHDOG: CPU: %d: transmit queue %u timed out %u ms\n",
-					    raw_smp_processor_id(),
-					    i, timedout_ms);
-				netif_freeze_queues(dev);
-				dev->netdev_ops->ndo_tx_timeout(dev, i);
-				netif_unfreeze_queues(dev);
+			if (time_after(jiffies, trans_start + dev->watchdog_timeo)) {
+				timedout_ms = jiffies_to_msecs(jiffies - trans_start);
+				atomic_long_inc(&txq->trans_timeout);
+				break;
 			}
-			if (!mod_timer(&dev->watchdog_timer,
-				       round_jiffies(oldest_start +
-						     dev->watchdog_timeo)))
-				release = false;
+			if (time_after(oldest_start, trans_start))
+				oldest_start = trans_start;
+		}
+
+		if (unlikely(timedout_ms)) {
+			trace_net_dev_xmit_timeout(dev, i);
+			netdev_crit(dev, "NETDEV WATCHDOG: CPU: %d: transmit queue %u timed out %u ms\n",
+				    raw_smp_processor_id(),
+				    i, timedout_ms);
+			netif_freeze_queues(dev);
+			dev->netdev_ops->ndo_tx_timeout(dev, i);
+			netif_unfreeze_queues(dev);
 		}
+		if (!mod_timer(&dev->watchdog_timer,
+			       round_jiffies(oldest_start +
+					     dev->watchdog_timeo)))
+			release = false;
 	}
 	spin_unlock(&dev->tx_global_lock);
 
-- 
2.34.1