[Bridge] Re: BUG: when using 'brctl stp'
Stephen Hemminger
shemminger at linux-foundation.org
Tue Aug 14 06:11:05 PDT 2007
Bridge locking for /sys/class/net/br0/bridge/stp_enabled
was wrong. Another bug in bridge utilities makes it such that
this interface, meant it wasn't being used. The locking needs to be removed
from set_stp_state(), the lock is already acquired down in
br_stp_start()/br_stp_stop.
Signed-off-by: Stephen Hemminger <shemminger at linux-foundation.org>
--- a/net/bridge/br_sysfs_br.c 2007-07-16 14:24:18.000000000 +0100
+++ b/net/bridge/br_sysfs_br.c 2007-08-14 13:44:23.000000000 +0100
@@ -150,9 +150,7 @@ static ssize_t show_stp_state(struct dev
static void set_stp_state(struct net_bridge *br, unsigned long val)
{
rtnl_lock();
- spin_unlock_bh(&br->lock);
br_stp_set_enabled(br, val);
- spin_lock_bh(&br->lock);
rtnl_unlock();
}
More information about the Bridge
mailing list