Commit 0fb7a0aa authored by Stephen Hemminger's avatar Stephen Hemminger Committed by Greg Kroah-Hartman

sky2: reliable recovery

This adds working recovery from transmit timeouts.  Previous code
didn't do enough to truly reset chip.

It is a backport of the 2.6.21 code.
Signed-off-by: default avatarStephen Hemminger <shemminger@linux-foundation.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent a63b596e
...@@ -1802,38 +1802,22 @@ static void sky2_tx_timeout(struct net_device *dev) ...@@ -1802,38 +1802,22 @@ static void sky2_tx_timeout(struct net_device *dev)
{ {
struct sky2_port *sky2 = netdev_priv(dev); struct sky2_port *sky2 = netdev_priv(dev);
struct sky2_hw *hw = sky2->hw; struct sky2_hw *hw = sky2->hw;
unsigned txq = txqaddr[sky2->port]; unsigned port = sky2->port;
u16 report, done;
if (netif_msg_timer(sky2)) if (netif_msg_timer(sky2))
printk(KERN_ERR PFX "%s: tx timeout\n", dev->name); printk(KERN_ERR PFX "%s: tx timeout\n", dev->name);
report = sky2_read16(hw, sky2->port == 0 ? STAT_TXA1_RIDX : STAT_TXA2_RIDX); /* Get information for bug report :-) */
done = sky2_read16(hw, Q_ADDR(txq, Q_DONE)); printk(KERN_INFO PFX "%s: transmit ring %u .. %u report=%u done=%u\n",
dev->name, sky2->tx_cons, sky2->tx_prod,
printk(KERN_DEBUG PFX "%s: transmit ring %u .. %u report=%u done=%u\n", sky2_read16(hw, port == 0 ? STAT_TXA1_RIDX : STAT_TXA2_RIDX),
dev->name, sky2_read16(hw, Q_ADDR(txqaddr[sky2->port], Q_DONE)));
sky2->tx_cons, sky2->tx_prod, report, done);
if (report != done) {
printk(KERN_INFO PFX "status burst pending (irq moderation?)\n");
sky2_write8(hw, STAT_TX_TIMER_CTRL, TIM_STOP); printk(KERN_INFO PFX "gmac control %#x status %#x\n",
sky2_write8(hw, STAT_TX_TIMER_CTRL, TIM_START); gma_read16(hw, port, GM_GP_CTRL), gma_read16(hw, port, GM_GP_STAT));
} else if (report != sky2->tx_cons) {
printk(KERN_INFO PFX "status report lost?\n");
sky2_tx_complete(sky2, report);
} else {
printk(KERN_INFO PFX "hardware hung? flushing\n");
sky2_write32(hw, Q_ADDR(txq, Q_CSR), BMU_STOP); /* can't restart safely under softirq */
sky2_write32(hw, Y2_QADDR(txq, PREF_UNIT_CTRL), PREF_UNIT_RST_SET); schedule_work(&hw->restart_work);
sky2_tx_complete(sky2, sky2->tx_prod);
sky2_qset(hw, txq);
sky2_prefetch_init(hw, txq, sky2->tx_le_map, TX_RING_SIZE - 1);
}
} }
static int sky2_change_mtu(struct net_device *dev, int new_mtu) static int sky2_change_mtu(struct net_device *dev, int new_mtu)
...@@ -2565,6 +2549,49 @@ static int sky2_reset(struct sky2_hw *hw) ...@@ -2565,6 +2549,49 @@ static int sky2_reset(struct sky2_hw *hw)
return 0; return 0;
} }
static void sky2_restart(struct work_struct *work)
{
struct sky2_hw *hw = container_of(work, struct sky2_hw, restart_work);
struct net_device *dev;
int i, err;
dev_dbg(&hw->pdev->dev, "restarting\n");
del_timer_sync(&hw->idle_timer);
rtnl_lock();
sky2_write32(hw, B0_IMSK, 0);
sky2_read32(hw, B0_IMSK);
netif_poll_disable(hw->dev[0]);
for (i = 0; i < hw->ports; i++) {
dev = hw->dev[i];
if (netif_running(dev))
sky2_down(dev);
}
sky2_reset(hw);
sky2_write32(hw, B0_IMSK, Y2_IS_BASE);
netif_poll_enable(hw->dev[0]);
for (i = 0; i < hw->ports; i++) {
dev = hw->dev[i];
if (netif_running(dev)) {
err = sky2_up(dev);
if (err) {
printk(KERN_INFO PFX "%s: could not restart %d\n",
dev->name, err);
dev_close(dev);
}
}
}
sky2_idle_start(hw);
rtnl_unlock();
}
static u32 sky2_supported_modes(const struct sky2_hw *hw) static u32 sky2_supported_modes(const struct sky2_hw *hw)
{ {
if (sky2_is_copper(hw)) { if (sky2_is_copper(hw)) {
...@@ -3508,6 +3535,8 @@ static int __devinit sky2_probe(struct pci_dev *pdev, ...@@ -3508,6 +3535,8 @@ static int __devinit sky2_probe(struct pci_dev *pdev,
} }
setup_timer(&hw->idle_timer, sky2_idle, (unsigned long) hw); setup_timer(&hw->idle_timer, sky2_idle, (unsigned long) hw);
INIT_WORK(&hw->restart_work, sky2_restart);
sky2_idle_start(hw); sky2_idle_start(hw);
pci_set_drvdata(pdev, hw); pci_set_drvdata(pdev, hw);
......
...@@ -1898,6 +1898,7 @@ struct sky2_hw { ...@@ -1898,6 +1898,7 @@ struct sky2_hw {
dma_addr_t st_dma; dma_addr_t st_dma;
struct timer_list idle_timer; struct timer_list idle_timer;
struct work_struct restart_work;
int msi; int msi;
wait_queue_head_t msi_wait; wait_queue_head_t msi_wait;
}; };
......
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