[PATCH] smc91x: more tweaks to help with RX overruns

Signed-off-by: Nicolas Pitre <nico@cam.org>

Index: linux-2.6/drivers/net/smc91x.c
===================================================================
This commit is contained in:
Nicolas Pitre 2005-04-12 16:26:40 -04:00 committed by Jeff Garzik
parent 8de901150f
commit ea9375607f
1 changed files with 13 additions and 13 deletions

View File

@ -129,7 +129,7 @@ MODULE_PARM_DESC(nowait, "set to 1 for no wait state");
/* /*
* Transmit timeout, default 5 seconds. * Transmit timeout, default 5 seconds.
*/ */
static int watchdog = 5000; static int watchdog = 1000;
module_param(watchdog, int, 0400); module_param(watchdog, int, 0400);
MODULE_PARM_DESC(watchdog, "transmit timeout in milliseconds"); MODULE_PARM_DESC(watchdog, "transmit timeout in milliseconds");
@ -660,15 +660,14 @@ static void smc_hardware_send_pkt(unsigned long data)
SMC_outw(((len & 1) ? (0x2000 | buf[len-1]) : 0), ioaddr, DATA_REG); SMC_outw(((len & 1) ? (0x2000 | buf[len-1]) : 0), ioaddr, DATA_REG);
/* /*
* If THROTTLE_TX_PKTS is set, we look at the TX_EMPTY flag * If THROTTLE_TX_PKTS is set, we stop the queue here. This will
* before queueing this packet for TX, and if it's clear then * have the effect of having at most one packet queued for TX
* we stop the queue here. This will have the effect of * in the chip's memory at all time.
* having at most 2 packets queued for TX in the chip's memory *
* at all time. If THROTTLE_TX_PKTS is not set then the queue * If THROTTLE_TX_PKTS is not set then the queue is stopped only
* is stopped only when memory allocation (MC_ALLOC) does not * when memory allocation (MC_ALLOC) does not succeed right away.
* succeed right away.
*/ */
if (THROTTLE_TX_PKTS && !(SMC_GET_INT() & IM_TX_EMPTY_INT)) if (THROTTLE_TX_PKTS)
netif_stop_queue(dev); netif_stop_queue(dev);
/* queue the packet for TX */ /* queue the packet for TX */
@ -1311,15 +1310,16 @@ static irqreturn_t smc_interrupt(int irq, void *dev_id, struct pt_regs *regs)
if (!status) if (!status)
break; break;
if (status & IM_RCV_INT) { if (status & IM_TX_INT) {
DBG(3, "%s: RX irq\n", dev->name); /* do this before RX as it will free memory quickly */
smc_rcv(dev);
} else if (status & IM_TX_INT) {
DBG(3, "%s: TX int\n", dev->name); DBG(3, "%s: TX int\n", dev->name);
smc_tx(dev); smc_tx(dev);
SMC_ACK_INT(IM_TX_INT); SMC_ACK_INT(IM_TX_INT);
if (THROTTLE_TX_PKTS) if (THROTTLE_TX_PKTS)
netif_wake_queue(dev); netif_wake_queue(dev);
} else if (status & IM_RCV_INT) {
DBG(3, "%s: RX irq\n", dev->name);
smc_rcv(dev);
} else if (status & IM_ALLOC_INT) { } else if (status & IM_ALLOC_INT) {
DBG(3, "%s: Allocation irq\n", dev->name); DBG(3, "%s: Allocation irq\n", dev->name);
tasklet_hi_schedule(&lp->tx_task); tasklet_hi_schedule(&lp->tx_task);