net,rcu: convert call_rcu(xps_dev_maps_release) to kfree_rcu()
authorLai Jiangshan <laijs@cn.fujitsu.com>
Fri, 18 Mar 2011 04:02:47 +0000 (12:02 +0800)
committerPaul E. McKenney <paulmck@linux.vnet.ibm.com>
Sun, 8 May 2011 05:51:04 +0000 (22:51 -0700)
The rcu callback xps_dev_maps_release() just calls a kfree(),
so we use kfree_rcu() instead of the call_rcu(xps_dev_maps_release).

Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
Acked-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Reviewed-by: Josh Triplett <josh@joshtriplett.org>
net/core/net-sysfs.c

index 48ffc216c86fcd69fb8e29ea499bcb0b074568aa..80b2aad3b73d4dda35fb59b8090e9d3c06450ea5 100644 (file)
@@ -891,14 +891,6 @@ static ssize_t show_xps_map(struct netdev_queue *queue,
        return len;
 }
 
-static void xps_dev_maps_release(struct rcu_head *rcu)
-{
-       struct xps_dev_maps *dev_maps =
-           container_of(rcu, struct xps_dev_maps, rcu);
-
-       kfree(dev_maps);
-}
-
 static DEFINE_MUTEX(xps_map_mutex);
 #define xmap_dereference(P)            \
        rcu_dereference_protected((P), lockdep_is_held(&xps_map_mutex))
@@ -1008,7 +1000,7 @@ static ssize_t store_xps_map(struct netdev_queue *queue,
        }
 
        if (dev_maps)
-               call_rcu(&dev_maps->rcu, xps_dev_maps_release);
+               kfree_rcu(dev_maps, rcu);
 
        netdev_queue_numa_node_write(queue, (numa_node >= 0) ? numa_node :
                                            NUMA_NO_NODE);
@@ -1080,7 +1072,7 @@ static void netdev_queue_release(struct kobject *kobj)
 
                if (!nonempty) {
                        RCU_INIT_POINTER(dev->xps_maps, NULL);
-                       call_rcu(&dev_maps->rcu, xps_dev_maps_release);
+                       kfree_rcu(dev_maps, rcu);
                }
        }