bridge: add RCU annotations to bridge port lookup
authorEric Dumazet <eric.dumazet@gmail.com>
Mon, 15 Nov 2010 06:38:14 +0000 (06:38 +0000)
committerDavid S. Miller <davem@davemloft.net>
Mon, 15 Nov 2010 19:13:18 +0000 (11:13 -0800)
br_port_get() renamed to br_port_get_rtnl() to make clear RTNL is held.

Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/bridge/br_if.c
net/bridge/br_netlink.c
net/bridge/br_notify.c
net/bridge/br_private.h

index 427f90a8ab7b9e457a074702ecbcd80def55846c..d9d1e2bac1d6efc826ed3ef826a91cd06bba78ca 100644 (file)
@@ -475,7 +475,7 @@ int br_del_if(struct net_bridge *br, struct net_device *dev)
 {
        struct net_bridge_port *p;
 
-       p = br_port_get(dev);
+       p = br_port_get_rtnl(dev);
        if (!p || p->br != br)
                return -EINVAL;
 
index e3de0a428f5d52c7a33393a74e7361a74aae370d..f8bf4c7f842c1714850ef21926035d44ee740bd4 100644 (file)
@@ -119,7 +119,7 @@ static int br_dump_ifinfo(struct sk_buff *skb, struct netlink_callback *cb)
 
        idx = 0;
        for_each_netdev(net, dev) {
-               struct net_bridge_port *port = br_port_get(dev);
+               struct net_bridge_port *port = br_port_get_rtnl(dev);
 
                /* not a bridge port */
                if (!port || idx < cb->args[0])
@@ -171,7 +171,7 @@ static int br_rtm_setlink(struct sk_buff *skb,  struct nlmsghdr *nlh, void *arg)
        if (!dev)
                return -ENODEV;
 
-       p = br_port_get(dev);
+       p = br_port_get_rtnl(dev);
        if (!p)
                return -EINVAL;
 
index ef2175c8b91d1d39bda3942b053fff3a902d4ac6..7d337c9b6082a5f3d23efa47823db5b2d99f7e5a 100644 (file)
@@ -37,10 +37,10 @@ static int br_device_event(struct notifier_block *unused, unsigned long event, v
        int err;
 
        /* not a port of a bridge */
-       if (!br_port_exists(dev))
+       p = br_port_get_rtnl(dev);
+       if (!p)
                return NOTIFY_DONE;
 
-       p = br_port_get(dev);
        br = p->br;
 
        switch (event) {
index 46e0bec1d7c5ca764313966f4acab25d6fc623cf..84aac7734bfc7e2c219680dc48d8c3ba75ca6924 100644 (file)
@@ -159,9 +159,10 @@ static inline struct net_bridge_port *br_port_get_rcu(const struct net_device *d
        return br_port_exists(dev) ? port : NULL;
 }
 
-static inline struct net_bridge_port *br_port_get(struct net_device *dev)
+static inline struct net_bridge_port *br_port_get_rtnl(struct net_device *dev)
 {
-       return br_port_exists(dev) ? dev->rx_handler_data : NULL;
+       return br_port_exists(dev) ?
+               rtnl_dereference(dev->rx_handler_data) : NULL;
 }
 
 struct br_cpu_netstats {