Commit 9a120bc5 authored by Michael Chan's avatar Michael Chan Committed by David S. Miller

bnx2: Allow phy reset to be skipped during chip reset.

Andy Gospodarek <andy@greyhouse.net> found that netconsole would
panic when resetting bnx2 devices.

>From Andy:
"The issue is the bnx2_set_link in bnx2_init_nic will print a link-status
message before we are fully initialized and ready to start polling.
Polling is currently disabled in this state, but since the
__LINK_STATE_RX_SCHED is overloaded to not only try and disable polling
but also to make the system aware there is something waiting to be
polled, we really have to fix this in drivers.

The problematic call is the one to netif_rx_complete as it tries to
remove an entry from the poll_list when there isn't one."

While this netconsole problem should be fixed separately, we really
should not reset the PHY when changing ring sizes, MTU, or other
similar settings.  The PHY reset causes several seconds of unnecessary
link disruptions.
Signed-off-by: default avatarMichael Chan <mchan@broadcom.com>
Acked-by: default avatarAndy Gospodarek <andy@greyhouse.net>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent f42a4449
...@@ -1875,7 +1875,7 @@ bnx2_setup_phy(struct bnx2 *bp, u8 port) ...@@ -1875,7 +1875,7 @@ bnx2_setup_phy(struct bnx2 *bp, u8 port)
} }
static int static int
bnx2_init_5709s_phy(struct bnx2 *bp) bnx2_init_5709s_phy(struct bnx2 *bp, int reset_phy)
{ {
u32 val; u32 val;
...@@ -1890,6 +1890,7 @@ bnx2_init_5709s_phy(struct bnx2 *bp) ...@@ -1890,6 +1890,7 @@ bnx2_init_5709s_phy(struct bnx2 *bp)
bnx2_write_phy(bp, MII_BNX2_AER_AER, MII_BNX2_AER_AER_AN_MMD); bnx2_write_phy(bp, MII_BNX2_AER_AER, MII_BNX2_AER_AER_AN_MMD);
bnx2_write_phy(bp, MII_BNX2_BLK_ADDR, MII_BNX2_BLK_ADDR_COMBO_IEEEB0); bnx2_write_phy(bp, MII_BNX2_BLK_ADDR, MII_BNX2_BLK_ADDR_COMBO_IEEEB0);
if (reset_phy)
bnx2_reset_phy(bp); bnx2_reset_phy(bp);
bnx2_write_phy(bp, MII_BNX2_BLK_ADDR, MII_BNX2_BLK_ADDR_SERDES_DIG); bnx2_write_phy(bp, MII_BNX2_BLK_ADDR, MII_BNX2_BLK_ADDR_SERDES_DIG);
...@@ -1924,10 +1925,11 @@ bnx2_init_5709s_phy(struct bnx2 *bp) ...@@ -1924,10 +1925,11 @@ bnx2_init_5709s_phy(struct bnx2 *bp)
} }
static int static int
bnx2_init_5708s_phy(struct bnx2 *bp) bnx2_init_5708s_phy(struct bnx2 *bp, int reset_phy)
{ {
u32 val; u32 val;
if (reset_phy)
bnx2_reset_phy(bp); bnx2_reset_phy(bp);
bp->mii_up1 = BCM5708S_UP1; bp->mii_up1 = BCM5708S_UP1;
...@@ -1981,8 +1983,9 @@ bnx2_init_5708s_phy(struct bnx2 *bp) ...@@ -1981,8 +1983,9 @@ bnx2_init_5708s_phy(struct bnx2 *bp)
} }
static int static int
bnx2_init_5706s_phy(struct bnx2 *bp) bnx2_init_5706s_phy(struct bnx2 *bp, int reset_phy)
{ {
if (reset_phy)
bnx2_reset_phy(bp); bnx2_reset_phy(bp);
bp->phy_flags &= ~BNX2_PHY_FLAG_PARALLEL_DETECT; bp->phy_flags &= ~BNX2_PHY_FLAG_PARALLEL_DETECT;
...@@ -2018,10 +2021,11 @@ bnx2_init_5706s_phy(struct bnx2 *bp) ...@@ -2018,10 +2021,11 @@ bnx2_init_5706s_phy(struct bnx2 *bp)
} }
static int static int
bnx2_init_copper_phy(struct bnx2 *bp) bnx2_init_copper_phy(struct bnx2 *bp, int reset_phy)
{ {
u32 val; u32 val;
if (reset_phy)
bnx2_reset_phy(bp); bnx2_reset_phy(bp);
if (bp->phy_flags & BNX2_PHY_FLAG_CRC_FIX) { if (bp->phy_flags & BNX2_PHY_FLAG_CRC_FIX) {
...@@ -2070,7 +2074,7 @@ bnx2_init_copper_phy(struct bnx2 *bp) ...@@ -2070,7 +2074,7 @@ bnx2_init_copper_phy(struct bnx2 *bp)
static int static int
bnx2_init_phy(struct bnx2 *bp) bnx2_init_phy(struct bnx2 *bp, int reset_phy)
{ {
u32 val; u32 val;
int rc = 0; int rc = 0;
...@@ -2096,14 +2100,14 @@ bnx2_init_phy(struct bnx2 *bp) ...@@ -2096,14 +2100,14 @@ bnx2_init_phy(struct bnx2 *bp)
if (bp->phy_flags & BNX2_PHY_FLAG_SERDES) { if (bp->phy_flags & BNX2_PHY_FLAG_SERDES) {
if (CHIP_NUM(bp) == CHIP_NUM_5706) if (CHIP_NUM(bp) == CHIP_NUM_5706)
rc = bnx2_init_5706s_phy(bp); rc = bnx2_init_5706s_phy(bp, reset_phy);
else if (CHIP_NUM(bp) == CHIP_NUM_5708) else if (CHIP_NUM(bp) == CHIP_NUM_5708)
rc = bnx2_init_5708s_phy(bp); rc = bnx2_init_5708s_phy(bp, reset_phy);
else if (CHIP_NUM(bp) == CHIP_NUM_5709) else if (CHIP_NUM(bp) == CHIP_NUM_5709)
rc = bnx2_init_5709s_phy(bp); rc = bnx2_init_5709s_phy(bp, reset_phy);
} }
else { else {
rc = bnx2_init_copper_phy(bp); rc = bnx2_init_copper_phy(bp, reset_phy);
} }
setup_phy: setup_phy:
...@@ -4873,7 +4877,7 @@ bnx2_reset_nic(struct bnx2 *bp, u32 reset_code) ...@@ -4873,7 +4877,7 @@ bnx2_reset_nic(struct bnx2 *bp, u32 reset_code)
} }
static int static int
bnx2_init_nic(struct bnx2 *bp) bnx2_init_nic(struct bnx2 *bp, int reset_phy)
{ {
int rc; int rc;
...@@ -4881,7 +4885,7 @@ bnx2_init_nic(struct bnx2 *bp) ...@@ -4881,7 +4885,7 @@ bnx2_init_nic(struct bnx2 *bp)
return rc; return rc;
spin_lock_bh(&bp->phy_lock); spin_lock_bh(&bp->phy_lock);
bnx2_init_phy(bp); bnx2_init_phy(bp, reset_phy);
bnx2_set_link(bp); bnx2_set_link(bp);
if (bp->phy_flags & BNX2_PHY_FLAG_REMOTE_PHY_CAP) if (bp->phy_flags & BNX2_PHY_FLAG_REMOTE_PHY_CAP)
bnx2_remote_phy_event(bp); bnx2_remote_phy_event(bp);
...@@ -5269,7 +5273,7 @@ bnx2_test_loopback(struct bnx2 *bp) ...@@ -5269,7 +5273,7 @@ bnx2_test_loopback(struct bnx2 *bp)
bnx2_reset_nic(bp, BNX2_DRV_MSG_CODE_RESET); bnx2_reset_nic(bp, BNX2_DRV_MSG_CODE_RESET);
spin_lock_bh(&bp->phy_lock); spin_lock_bh(&bp->phy_lock);
bnx2_init_phy(bp); bnx2_init_phy(bp, 1);
spin_unlock_bh(&bp->phy_lock); spin_unlock_bh(&bp->phy_lock);
if (bnx2_run_loopback(bp, BNX2_MAC_LOOPBACK)) if (bnx2_run_loopback(bp, BNX2_MAC_LOOPBACK))
rc |= BNX2_MAC_LOOPBACK_FAILED; rc |= BNX2_MAC_LOOPBACK_FAILED;
...@@ -5659,7 +5663,7 @@ bnx2_open(struct net_device *dev) ...@@ -5659,7 +5663,7 @@ bnx2_open(struct net_device *dev)
return rc; return rc;
} }
rc = bnx2_init_nic(bp); rc = bnx2_init_nic(bp, 1);
if (rc) { if (rc) {
bnx2_napi_disable(bp); bnx2_napi_disable(bp);
...@@ -5691,7 +5695,7 @@ bnx2_open(struct net_device *dev) ...@@ -5691,7 +5695,7 @@ bnx2_open(struct net_device *dev)
bnx2_setup_int_mode(bp, 1); bnx2_setup_int_mode(bp, 1);
rc = bnx2_init_nic(bp); rc = bnx2_init_nic(bp, 0);
if (!rc) if (!rc)
rc = bnx2_request_irq(bp); rc = bnx2_request_irq(bp);
...@@ -5727,7 +5731,7 @@ bnx2_reset_task(struct work_struct *work) ...@@ -5727,7 +5731,7 @@ bnx2_reset_task(struct work_struct *work)
bp->in_reset_task = 1; bp->in_reset_task = 1;
bnx2_netif_stop(bp); bnx2_netif_stop(bp);
bnx2_init_nic(bp); bnx2_init_nic(bp, 1);
atomic_set(&bp->intr_sem, 1); atomic_set(&bp->intr_sem, 1);
bnx2_netif_start(bp); bnx2_netif_start(bp);
...@@ -6421,7 +6425,7 @@ bnx2_set_coalesce(struct net_device *dev, struct ethtool_coalesce *coal) ...@@ -6421,7 +6425,7 @@ bnx2_set_coalesce(struct net_device *dev, struct ethtool_coalesce *coal)
if (netif_running(bp->dev)) { if (netif_running(bp->dev)) {
bnx2_netif_stop(bp); bnx2_netif_stop(bp);
bnx2_init_nic(bp); bnx2_init_nic(bp, 0);
bnx2_netif_start(bp); bnx2_netif_start(bp);
} }
...@@ -6464,7 +6468,7 @@ bnx2_change_ring_size(struct bnx2 *bp, u32 rx, u32 tx) ...@@ -6464,7 +6468,7 @@ bnx2_change_ring_size(struct bnx2 *bp, u32 rx, u32 tx)
rc = bnx2_alloc_mem(bp); rc = bnx2_alloc_mem(bp);
if (rc) if (rc)
return rc; return rc;
bnx2_init_nic(bp); bnx2_init_nic(bp, 0);
bnx2_netif_start(bp); bnx2_netif_start(bp);
} }
return 0; return 0;
...@@ -6732,7 +6736,7 @@ bnx2_self_test(struct net_device *dev, struct ethtool_test *etest, u64 *buf) ...@@ -6732,7 +6736,7 @@ bnx2_self_test(struct net_device *dev, struct ethtool_test *etest, u64 *buf)
bnx2_reset_chip(bp, BNX2_DRV_MSG_CODE_RESET); bnx2_reset_chip(bp, BNX2_DRV_MSG_CODE_RESET);
} }
else { else {
bnx2_init_nic(bp); bnx2_init_nic(bp, 1);
bnx2_netif_start(bp); bnx2_netif_start(bp);
} }
...@@ -7619,7 +7623,7 @@ bnx2_resume(struct pci_dev *pdev) ...@@ -7619,7 +7623,7 @@ bnx2_resume(struct pci_dev *pdev)
bnx2_set_power_state(bp, PCI_D0); bnx2_set_power_state(bp, PCI_D0);
netif_device_attach(dev); netif_device_attach(dev);
bnx2_init_nic(bp); bnx2_init_nic(bp, 1);
bnx2_netif_start(bp); bnx2_netif_start(bp);
return 0; return 0;
} }
......
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