Revert "bridge: Use hlist_for_each_entry_rcu() in br_multicast_add_router()"
authorDavid S. Miller <davem@davemloft.net>
Tue, 27 Apr 2010 23:49:58 +0000 (16:49 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 27 Apr 2010 23:49:58 +0000 (16:49 -0700)
This reverts commit ff65e8275f6c96a5eda57493bd84c4555decf7b3.

As explained by Stephen Hemminger, the traversal doesn't require
RCU handling as we hold a lock.

The list addition et al. calls, on the other hand, do.

Signed-off-by: David S. Miller <davem@davemloft.net>
net/bridge/br_multicast.c

index e29c9b738796741a30cecd80558012068859f443..fcba313f18944437055f11d25af4fcd3dfb460e0 100644 (file)
@@ -1045,7 +1045,7 @@ static void br_multicast_add_router(struct net_bridge *br,
        struct net_bridge_port *p;
        struct hlist_node *n, *last = NULL;
 
-       hlist_for_each_entry_rcu(p, n, &br->router_list, rlist) {
+       hlist_for_each_entry(p, n, &br->router_list, rlist) {
                if ((unsigned long) port >= (unsigned long) p) {
                        hlist_add_before_rcu(n, &port->rlist);
                        return;