[PATCH] bonding: bond_check_dev_link() needs to check for netif_running()
From: Petri Gynther <hidden>
Date: 2009-08-26 20:35:06
Subsystem:
bonding driver, networking drivers, the rest · Maintainers:
Jay Vosburgh, Andrew Lunn, "David S. Miller", Eric Dumazet, Jakub Kicinski, Paolo Abeni, Linus Torvalds
bond_check_dev_link() needs to check for netif_running() before checking the link status of a slave. This needs to be done because some interfaces keep reporting netif_carrier_ok() == TRUE even after "ifconfig down". Signed-off-by: Petri Gynther <redacted> --- drivers/net/bonding/bond_main.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c
index aa1be1f..0835cbe 100644
--- a/drivers/net/bonding/bond_main.c
+++ b/drivers/net/bonding/bond_main.c@@ -695,6 +695,9 @@ static int bond_check_dev_link(struct bonding *bond, struct ifreq ifr; struct mii_ioctl_data *mii; + if (!netif_running(slave_dev)) + return 0; + if (bond->params.use_carrier) return netif_carrier_ok(slave_dev) ? BMSR_LSTATUS : 0;
--
1.5.4.3