Improve the readability of the loop removing tx-queue from unused
CPUs/rx-queues in __netif_set_xps_queue. The change should only be
cosmetic.

Signed-off-by: Antoine Tenart <aten...@kernel.org>
---
 net/core/dev.c | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/net/core/dev.c b/net/core/dev.c
index 4d39938417c4..052797ca65f6 100644
--- a/net/core/dev.c
+++ b/net/core/dev.c
@@ -2786,13 +2786,16 @@ int __netif_set_xps_queue(struct net_device *dev, const 
unsigned long *mask,
 
        /* removes tx-queue from unused CPUs/rx-queues */
        for (j = 0; j < dev_maps->nr_ids; j++) {
-               for (i = tc, tci = j * dev_maps->num_tc; i--; tci++)
-                       active |= remove_xps_queue(dev_maps, tci, index);
-               if (!netif_attr_test_mask(j, mask, dev_maps->nr_ids) ||
-                   !netif_attr_test_online(j, online_mask, dev_maps->nr_ids))
-                       active |= remove_xps_queue(dev_maps, tci, index);
-               for (i = dev_maps->num_tc - tc, tci++; --i; tci++)
+               tci = j * dev_maps->num_tc;
+
+               for (i = 0; i < dev_maps->num_tc; i++, tci++) {
+                       if (i == tc &&
+                           netif_attr_test_mask(j, mask, dev_maps->nr_ids) &&
+                           netif_attr_test_online(j, online_mask, 
dev_maps->nr_ids))
+                               continue;
+
                        active |= remove_xps_queue(dev_maps, tci, index);
+               }
        }
 
        /* free map if not active */
-- 
2.29.2

Reply via email to