Message ID | 20180710003501.13971-1-travisb@arista.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
Series | [net-next] macvlan: Change status when lower device goes down | expand |
From: Travis Brown <travisb@arista.com> Date: Tue, 10 Jul 2018 00:35:01 +0000 > Today macvlan ignores the notification when a lower device goes > administratively down, preventing the lack of connectivity from > bubbling up. > > Processing NETDEV_DOWN results in a macvlan state of LOWERLAYERDOWN > with NO-CARRIER which should be easy to interpret in userspace. > > 2: lower: <BROADCAST,MULTICAST> mtu 1500 qdisc mq state DOWN mode DEFAULT group default qlen 1000 > 3: macvlan@lower: <NO-CARRIER,BROADCAST,MULTICAST,UP,M-DOWN> mtu 1500 qdisc noqueue state LOWERLAYERDOWN mode DEFAULT group default qlen 1000 > > Signed-off-by: Suresh Krishnan <skrishnan@arista.com> > Signed-off-by: Travis Brown <travisb@arista.com> Seems reasonable, applied, thanks.
diff --git a/drivers/net/macvlan.c b/drivers/net/macvlan.c index adde8fc45588..6dcd715a9370 100644 --- a/drivers/net/macvlan.c +++ b/drivers/net/macvlan.c @@ -1647,6 +1647,7 @@ static int macvlan_device_event(struct notifier_block *unused, switch (event) { case NETDEV_UP: + case NETDEV_DOWN: case NETDEV_CHANGE: list_for_each_entry(vlan, &port->vlans, list) netif_stacked_transfer_operstate(vlan->lowerdev,