Commit 4433f420 authored by Stephen Hemminger's avatar Stephen Hemminger Committed by David S. Miller

[BRIDGE]: handle speed detection after carrier changes

Speed of a interface may not be available until carrier
is detected in the case of autonegotiation. To get the correct value
we need to recheck speed after carrier event.  But the check needs to
be done in a context that is similar to normal ethtool interface (can sleep).

Also, delay check for 1ms to try avoid any carrier bounce transitions.
Signed-off-by: default avatarStephen Hemminger <shemminger@osdl.org>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 4505a3ef
...@@ -32,9 +32,8 @@ ...@@ -32,9 +32,8 @@
* ethtool, use ethtool_ops. Also, since driver might sleep need to * ethtool, use ethtool_ops. Also, since driver might sleep need to
* not be holding any locks. * not be holding any locks.
*/ */
static int br_initial_port_cost(struct net_device *dev) static int port_cost(struct net_device *dev)
{ {
struct ethtool_cmd ecmd = { ETHTOOL_GSET }; struct ethtool_cmd ecmd = { ETHTOOL_GSET };
struct ifreq ifr; struct ifreq ifr;
mm_segment_t old_fs; mm_segment_t old_fs;
...@@ -58,10 +57,6 @@ static int br_initial_port_cost(struct net_device *dev) ...@@ -58,10 +57,6 @@ static int br_initial_port_cost(struct net_device *dev)
return 2; return 2;
case SPEED_10: case SPEED_10:
return 100; return 100;
default:
pr_info("bridge: can't decode speed from %s: %d\n",
dev->name, ecmd.speed);
return 100;
} }
} }
...@@ -75,6 +70,35 @@ static int br_initial_port_cost(struct net_device *dev) ...@@ -75,6 +70,35 @@ static int br_initial_port_cost(struct net_device *dev)
return 100; /* assume old 10Mbps */ return 100; /* assume old 10Mbps */
} }
/*
* Check for port carrier transistions.
* Called from work queue to allow for calling functions that
* might sleep (such as speed check), and to debounce.
*/
static void port_carrier_check(void *arg)
{
struct net_bridge_port *p = arg;
rtnl_lock();
if (netif_carrier_ok(p->dev)) {
u32 cost = port_cost(p->dev);
spin_lock_bh(&p->br->lock);
if (p->state == BR_STATE_DISABLED) {
p->path_cost = cost;
br_stp_enable_port(p);
}
spin_unlock_bh(&p->br->lock);
} else {
spin_lock_bh(&p->br->lock);
if (p->state != BR_STATE_DISABLED)
br_stp_disable_port(p);
spin_unlock_bh(&p->br->lock);
}
rtnl_unlock();
}
static void destroy_nbp(struct net_bridge_port *p) static void destroy_nbp(struct net_bridge_port *p)
{ {
struct net_device *dev = p->dev; struct net_device *dev = p->dev;
...@@ -102,6 +126,9 @@ static void del_nbp(struct net_bridge_port *p) ...@@ -102,6 +126,9 @@ static void del_nbp(struct net_bridge_port *p)
dev->br_port = NULL; dev->br_port = NULL;
dev_set_promiscuity(dev, -1); dev_set_promiscuity(dev, -1);
cancel_delayed_work(&p->carrier_check);
flush_scheduled_work();
spin_lock_bh(&br->lock); spin_lock_bh(&br->lock);
br_stp_disable_port(p); br_stp_disable_port(p);
spin_unlock_bh(&br->lock); spin_unlock_bh(&br->lock);
...@@ -195,10 +222,9 @@ static int find_portno(struct net_bridge *br) ...@@ -195,10 +222,9 @@ static int find_portno(struct net_bridge *br)
return (index >= BR_MAX_PORTS) ? -EXFULL : index; return (index >= BR_MAX_PORTS) ? -EXFULL : index;
} }
/* called with RTNL */ /* called with RTNL but without bridge lock */
static struct net_bridge_port *new_nbp(struct net_bridge *br, static struct net_bridge_port *new_nbp(struct net_bridge *br,
struct net_device *dev, struct net_device *dev)
unsigned long cost)
{ {
int index; int index;
struct net_bridge_port *p; struct net_bridge_port *p;
...@@ -215,12 +241,13 @@ static struct net_bridge_port *new_nbp(struct net_bridge *br, ...@@ -215,12 +241,13 @@ static struct net_bridge_port *new_nbp(struct net_bridge *br,
p->br = br; p->br = br;
dev_hold(dev); dev_hold(dev);
p->dev = dev; p->dev = dev;
p->path_cost = cost; p->path_cost = port_cost(dev);
p->priority = 0x8000 >> BR_PORT_BITS; p->priority = 0x8000 >> BR_PORT_BITS;
dev->br_port = p; dev->br_port = p;
p->port_no = index; p->port_no = index;
br_init_port(p); br_init_port(p);
p->state = BR_STATE_DISABLED; p->state = BR_STATE_DISABLED;
INIT_WORK(&p->carrier_check, port_carrier_check, p);
kobject_init(&p->kobj); kobject_init(&p->kobj);
return p; return p;
...@@ -351,7 +378,7 @@ int br_add_if(struct net_bridge *br, struct net_device *dev) ...@@ -351,7 +378,7 @@ int br_add_if(struct net_bridge *br, struct net_device *dev)
if (dev->br_port != NULL) if (dev->br_port != NULL)
return -EBUSY; return -EBUSY;
if (IS_ERR(p = new_nbp(br, dev, br_initial_port_cost(dev)))) if (IS_ERR(p = new_nbp(br, dev)))
return PTR_ERR(p); return PTR_ERR(p);
if ((err = br_fdb_insert(br, p, dev->dev_addr))) if ((err = br_fdb_insert(br, p, dev->dev_addr)))
......
...@@ -52,17 +52,9 @@ static int br_device_event(struct notifier_block *unused, unsigned long event, v ...@@ -52,17 +52,9 @@ static int br_device_event(struct notifier_block *unused, unsigned long event, v
br_stp_recalculate_bridge_id(br); br_stp_recalculate_bridge_id(br);
break; break;
case NETDEV_CHANGE: /* device is up but carrier changed */ case NETDEV_CHANGE:
if (!(br->dev->flags & IFF_UP)) if (br->dev->flags & IFF_UP)
break; schedule_delayed_work(&p->carrier_check, BR_PORT_DEBOUNCE);
if (netif_carrier_ok(dev)) {
if (p->state == BR_STATE_DISABLED)
br_stp_enable_port(p);
} else {
if (p->state != BR_STATE_DISABLED)
br_stp_disable_port(p);
}
break; break;
case NETDEV_FEAT_CHANGE: case NETDEV_FEAT_CHANGE:
......
...@@ -27,6 +27,8 @@ ...@@ -27,6 +27,8 @@
#define BR_PORT_BITS 10 #define BR_PORT_BITS 10
#define BR_MAX_PORTS (1<<BR_PORT_BITS) #define BR_MAX_PORTS (1<<BR_PORT_BITS)
#define BR_PORT_DEBOUNCE (HZ/10)
typedef struct bridge_id bridge_id; typedef struct bridge_id bridge_id;
typedef struct mac_addr mac_addr; typedef struct mac_addr mac_addr;
typedef __u16 port_id; typedef __u16 port_id;
...@@ -78,6 +80,7 @@ struct net_bridge_port ...@@ -78,6 +80,7 @@ struct net_bridge_port
struct timer_list hold_timer; struct timer_list hold_timer;
struct timer_list message_age_timer; struct timer_list message_age_timer;
struct kobject kobj; struct kobject kobj;
struct work_struct carrier_check;
struct rcu_head rcu; struct rcu_head rcu;
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment