ixgbe: fix slow load times on 82598 nics
authorDon Skidmore <donald.c.skidmore@intel.com>
Tue, 27 Jan 2009 04:57:17 +0000 (20:57 -0800)
committerDavid S. Miller <davem@davemloft.net>
Tue, 27 Jan 2009 04:57:17 +0000 (20:57 -0800)
Load times for NICs that use i2c to communicate with the phy were taking
~4.5 sec per port.  This fix first checks to see if the link is already
up before calling get_link_capabilities, since if it is we don't need
query the phy for link state.

Signed-off-by: Don Skidmore <donald.c.skidmore@intel.com>
Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ixgbe/ixgbe_main.c

index f7b592eff68e3a864acafd1490318d31cb887474..18d4353afa65352492add64d94ee34b70c00bc83 100644 (file)
@@ -3901,16 +3901,27 @@ static void ixgbe_netpoll(struct net_device *netdev)
  **/
 static int ixgbe_link_config(struct ixgbe_hw *hw)
 {
-       u32 autoneg = IXGBE_LINK_SPEED_10GB_FULL;
+       u32 autoneg;
+       bool link_up = false;
+       u32 ret = IXGBE_ERR_LINK_SETUP;
 
-       /* must always autoneg for both 1G and 10G link */
-       hw->mac.autoneg = true;
+       if (hw->mac.ops.check_link)
+               ret = hw->mac.ops.check_link(hw, &autoneg, &link_up, false);
 
-       if ((hw->mac.type == ixgbe_mac_82598EB) &&
-           (hw->phy.media_type == ixgbe_media_type_copper))
-               autoneg = IXGBE_LINK_SPEED_82598_AUTONEG;
+       if (ret || !link_up)
+               goto link_cfg_out;
 
-       return hw->mac.ops.setup_link_speed(hw, autoneg, true, true);
+       if (hw->mac.ops.get_link_capabilities)
+               ret = hw->mac.ops.get_link_capabilities(hw, &autoneg,
+                                                       &hw->mac.autoneg);
+       if (ret)
+               goto link_cfg_out;
+
+       if (hw->mac.ops.setup_link_speed)
+               ret = hw->mac.ops.setup_link_speed(hw, autoneg, true, true);
+
+link_cfg_out:
+       return ret;
 }
 
 static const struct net_device_ops ixgbe_netdev_ops = {