From: Zhu Yanjun <yanjun....@windriver.com>
According to the suggestion from Rustad, Mark D, this behavior perhaps
is more related to the copper phy. But to make fiber phy more robust,
to all the interfaces as a slave interface, the link_speed and link_up
is synchronized.
Signed-off-by: Zhu Yanjun <yanjun....@windriver.com>
---
drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c
b/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c
index 1bb6056..ce47639 100644
--- a/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c
+++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c
@@ -6441,10 +6441,12 @@ static void ixgbe_watchdog_link_is_up(struct
ixgbe_adapter *adapter)
* a bonding driver in 802.3ad mode. When X540 NIC acts as an
* independent interface, it is not necessary to synchronize link_up
* and link_speed.
- * In the end, not continue if (X540 NIC && SLAVE && link_speed UNKNOWN)
+ * According to the suggestion from Rustad, Mark D, this behavior
+ * perhaps is related to the copper phy. To make fiber phy more robust,
+ * To all the interfaces as a slave, the link_speed is checked.
+ * In the end, not continue if (SLAVE && link_speed UNKNOWN)
*/
- if ((hw->mac.type == ixgbe_mac_X540) &&
- (netdev->flags & IFF_SLAVE))
+ if (netdev->flags & IFF_SLAVE)
if (link_speed == IXGBE_LINK_SPEED_UNKNOWN)
return;
--
1.7.9.5
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
