[PATCH net 1/2] net: restore call to netdev_queue_numa_node_write when resetting XPS

[Sabrina Dubroca]

Before commit 80d19669ecd3 ("net: Refactor XPS for CPUs and Rx queues"),
netif_reset_xps_queues() did netdev_queue_numa_node_write() for all the
queues being reset. Now, this is only done when the "active" variable in
clean_xps_maps() is false, ie when on all the CPUs, there's no active
XPS mapping left.

Fixes: 80d19669ecd3 ("net: Refactor XPS for CPUs and Rx queues")
Signed-off-by: Sabrina Dubroca <s...@queasysnail.net>
---
 net/core/dev.c | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/net/core/dev.c b/net/core/dev.c
index ddc551f24ba2..32a63f4c3a92 100644
--- a/net/core/dev.c
+++ b/net/core/dev.c
@@ -2187,17 +2187,19 @@ static void clean_xps_maps(struct net_device *dev, 
const unsigned long *mask,
                active |= remove_xps_queue_cpu(dev, dev_maps, j, offset,
                                               count);
        if (!active) {
-               if (is_rxqs_map) {
+               if (is_rxqs_map)
                        RCU_INIT_POINTER(dev->xps_rxqs_map, NULL);
-               } else {
+               else
                        RCU_INIT_POINTER(dev->xps_cpus_map, NULL);
+               kfree_rcu(dev_maps, rcu);
+       }
 
-                       for (i = offset + (count - 1); count--; i--)
-                               netdev_queue_numa_node_write(
-                                       netdev_get_tx_queue(dev, i),
-                                                       NUMA_NO_NODE);
+       if (!is_rxqs_map) {
+               for (i = offset + (count - 1); count--; i--) {
+                       netdev_queue_numa_node_write(
+                               netdev_get_tx_queue(dev, i),
+                               NUMA_NO_NODE);
                }
-               kfree_rcu(dev_maps, rcu);
        }
 }
 
-- 
2.19.2