diff options
| author | George Wilkie <gwilkie@vyatta.att-mail.com> | 2019-02-20 09:19:11 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2019-02-22 20:40:23 +0100 |
| commit | 8c7a77267eec81dd81af8412f29e50c0b1082548 (patch) | |
| tree | ea0abef5c43ee2f5d52c8d09e7ef32fa8620d7d7 | |
| parent | r8152: Fix an error on RTL8153-BD MAC Address Passthrough support (diff) | |
| download | linux-8c7a77267eec81dd81af8412f29e50c0b1082548.tar.xz linux-8c7a77267eec81dd81af8412f29e50c0b1082548.zip | |
team: use operstate consistently for linkup
When a port is added to a team, its initial state is derived
from netif_carrier_ok rather than netif_oper_up.
If it is carrier up but operationally down at the time of being
added, the port state.linkup will be set prematurely.
port state.linkup should be set consistently using
netif_oper_up rather than netif_carrier_ok.
Fixes: f1d22a1e0595 ("team: account for oper state")
Signed-off-by: George Wilkie <gwilkie@vyatta.att-mail.com>
Acked-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
| -rw-r--r-- | drivers/net/team/team.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/team/team.c b/drivers/net/team/team.c index 958f1cf67282..6ce3f666d142 100644 --- a/drivers/net/team/team.c +++ b/drivers/net/team/team.c @@ -1256,7 +1256,7 @@ static int team_port_add(struct team *team, struct net_device *port_dev, list_add_tail_rcu(&port->list, &team->port_list); team_port_enable(team, port); __team_compute_features(team); - __team_port_change_port_added(port, !!netif_carrier_ok(port_dev)); + __team_port_change_port_added(port, !!netif_oper_up(port_dev)); __team_options_change_check(team); netdev_info(dev, "Port device %s added\n", portname); @@ -2915,7 +2915,7 @@ static int team_device_event(struct notifier_block *unused, switch (event) { case NETDEV_UP: - if (netif_carrier_ok(dev)) + if (netif_oper_up(dev)) team_port_change_check(port, true); break; case NETDEV_DOWN: |