decnet: use RCU to find network devices
authorstephen hemminger <shemminger@vyatta.com>
Tue, 10 Nov 2009 07:54:53 +0000 (07:54 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 11 Nov 2009 06:26:31 +0000 (22:26 -0800)
When showing device statistics use RCU rather than read_lock(&dev_base_lock)
Compile tested only.

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/decnet/dn_dev.c

index d82694d930b425b3237af25688cdf63071effc6c..b5ef237c8a74334c55692f5b535411aa1b580e84 100644 (file)
@@ -856,9 +856,7 @@ int dn_dev_bind_default(__le16 *addr)
        dev = dn_dev_get_default();
 last_chance:
        if (dev) {
-               read_lock(&dev_base_lock);
                rv = dn_dev_get_first(dev, addr);
-               read_unlock(&dev_base_lock);
                dev_put(dev);
                if (rv == 0 || dev == init_net.loopback_dev)
                        return rv;
@@ -1323,18 +1321,18 @@ static inline int is_dn_dev(struct net_device *dev)
 }
 
 static void *dn_dev_seq_start(struct seq_file *seq, loff_t *pos)
-       __acquires(&dev_base_lock)
+       __acquires(rcu)
 {
        int i;
        struct net_device *dev;
 
-       read_lock(&dev_base_lock);
+       rcu_read_lock();
 
        if (*pos == 0)
                return SEQ_START_TOKEN;
 
        i = 1;
-       for_each_netdev(&init_net, dev) {
+       for_each_netdev_rcu(&init_net, dev) {
                if (!is_dn_dev(dev))
                        continue;
 
@@ -1355,7 +1353,7 @@ static void *dn_dev_seq_next(struct seq_file *seq, void *v, loff_t *pos)
        if (v == SEQ_START_TOKEN)
                dev = net_device_entry(&init_net.dev_base_head);
 
-       for_each_netdev_continue(&init_net, dev) {
+       for_each_netdev_continue_rcu(&init_net, dev) {
                if (!is_dn_dev(dev))
                        continue;
 
@@ -1366,9 +1364,9 @@ static void *dn_dev_seq_next(struct seq_file *seq, void *v, loff_t *pos)
 }
 
 static void dn_dev_seq_stop(struct seq_file *seq, void *v)
-       __releases(&dev_base_lock)
+       __releases(rcu)
 {
-       read_unlock(&dev_base_lock);
+       rcu_read_unlock();
 }
 
 static char *dn_type2asc(char type)