Commit 986e0aeb authored by Michael Chan's avatar Michael Chan Committed by David S. Miller

[TG3]: Remove reset during MAC address changes.

The reset was added a while back so that ASF could re-init whatever
MAC address it wanted to use after the MAC address was changed.
Instead of resetting, we can just keep MAC address 1 unchanged during
MAC address changes if MAC address 1 is different from MAC address 0.

This fixes 2 problems:

1. Bonding calls set_mac_address in contexts that cannot sleep.
It no longer sleeps with the chip reset removed.

2. When ASF shares the same MAC address as the NIC, it needs to
always do that even when the MAC address is changed. 
Signed-off-by: default avatarMichael Chan <mchan@broadcom.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent a85feb8c
...@@ -5934,7 +5934,7 @@ static int tg3_load_tso_firmware(struct tg3 *tp) ...@@ -5934,7 +5934,7 @@ static int tg3_load_tso_firmware(struct tg3 *tp)
/* tp->lock is held. */ /* tp->lock is held. */
static void __tg3_set_mac_addr(struct tg3 *tp) static void __tg3_set_mac_addr(struct tg3 *tp, int skip_mac_1)
{ {
u32 addr_high, addr_low; u32 addr_high, addr_low;
int i; int i;
...@@ -5946,6 +5946,8 @@ static void __tg3_set_mac_addr(struct tg3 *tp) ...@@ -5946,6 +5946,8 @@ static void __tg3_set_mac_addr(struct tg3 *tp)
(tp->dev->dev_addr[4] << 8) | (tp->dev->dev_addr[4] << 8) |
(tp->dev->dev_addr[5] << 0)); (tp->dev->dev_addr[5] << 0));
for (i = 0; i < 4; i++) { for (i = 0; i < 4; i++) {
if (i == 1 && skip_mac_1)
continue;
tw32(MAC_ADDR_0_HIGH + (i * 8), addr_high); tw32(MAC_ADDR_0_HIGH + (i * 8), addr_high);
tw32(MAC_ADDR_0_LOW + (i * 8), addr_low); tw32(MAC_ADDR_0_LOW + (i * 8), addr_low);
} }
...@@ -5972,7 +5974,7 @@ static int tg3_set_mac_addr(struct net_device *dev, void *p) ...@@ -5972,7 +5974,7 @@ static int tg3_set_mac_addr(struct net_device *dev, void *p)
{ {
struct tg3 *tp = netdev_priv(dev); struct tg3 *tp = netdev_priv(dev);
struct sockaddr *addr = p; struct sockaddr *addr = p;
int err = 0; int err = 0, skip_mac_1 = 0;
if (!is_valid_ether_addr(addr->sa_data)) if (!is_valid_ether_addr(addr->sa_data))
return -EINVAL; return -EINVAL;
...@@ -5983,22 +5985,21 @@ static int tg3_set_mac_addr(struct net_device *dev, void *p) ...@@ -5983,22 +5985,21 @@ static int tg3_set_mac_addr(struct net_device *dev, void *p)
return 0; return 0;
if (tp->tg3_flags & TG3_FLAG_ENABLE_ASF) { if (tp->tg3_flags & TG3_FLAG_ENABLE_ASF) {
/* Reset chip so that ASF can re-init any MAC addresses it u32 addr0_high, addr0_low, addr1_high, addr1_low;
* needs.
*/
tg3_netif_stop(tp);
tg3_full_lock(tp, 1);
tg3_halt(tp, RESET_KIND_SHUTDOWN, 1); addr0_high = tr32(MAC_ADDR_0_HIGH);
err = tg3_restart_hw(tp, 0); addr0_low = tr32(MAC_ADDR_0_LOW);
if (!err) addr1_high = tr32(MAC_ADDR_1_HIGH);
tg3_netif_start(tp); addr1_low = tr32(MAC_ADDR_1_LOW);
tg3_full_unlock(tp);
} else { /* Skip MAC addr 1 if ASF is using it. */
spin_lock_bh(&tp->lock); if ((addr0_high != addr1_high || addr0_low != addr1_low) &&
__tg3_set_mac_addr(tp); !(addr1_high == 0 && addr1_low == 0))
spin_unlock_bh(&tp->lock); skip_mac_1 = 1;
} }
spin_lock_bh(&tp->lock);
__tg3_set_mac_addr(tp, skip_mac_1);
spin_unlock_bh(&tp->lock);
return err; return err;
} }
...@@ -6315,7 +6316,7 @@ static int tg3_reset_hw(struct tg3 *tp, int reset_phy) ...@@ -6315,7 +6316,7 @@ static int tg3_reset_hw(struct tg3 *tp, int reset_phy)
tp->rx_jumbo_ptr); tp->rx_jumbo_ptr);
/* Initialize MAC address and backoff seed. */ /* Initialize MAC address and backoff seed. */
__tg3_set_mac_addr(tp); __tg3_set_mac_addr(tp, 0);
/* MTU + ethernet header + FCS + optional VLAN tag */ /* MTU + ethernet header + FCS + optional VLAN tag */
tw32(MAC_RX_MTU_SIZE, tp->dev->mtu + ETH_HLEN + 8); tw32(MAC_RX_MTU_SIZE, tp->dev->mtu + ETH_HLEN + 8);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment