| To: | tsbogend@xxxxxxxxxxxxxxxx, jgarzik@xxxxxxxxx, netdev@xxxxxxxxxxx |
|---|---|
| Subject: | [PATCH 4/5] 2.6.6-bk8 pcnet32 fix boundary comparison bug. |
| From: | Don Fry <brazilnut@xxxxxxxxxx> |
| Date: | Fri, 21 May 2004 13:31:02 -0700 (PDT) |
| Sender: | netdev-bounce@xxxxxxxxxxx |
This patch fixes a boundary condition. It is possible to get an
interrupt and have the transmit queue completely full. Without this
the driver will leak skb's and pci maps (if the arch supports them).
--- linux-2.6.6-bk8/drivers/net/pro.pcnet32.c Fri May 21 09:59:18 2004
+++ linux-2.6.6-bk8/drivers/net/pcnet32.c Fri May 21 10:00:10 2004
@@ -1833,7 +1833,7 @@
}
delta = (lp->cur_tx - dirty_tx) & (TX_RING_MOD_MASK + TX_RING_SIZE);
- if (delta >= TX_RING_SIZE) {
+ if (delta > TX_RING_SIZE) {
if (netif_msg_drv(lp))
printk(KERN_ERR "%s: out-of-sync dirty pointer, %d vs. %d,
full=%d.\n",
dev->name, dirty_tx, lp->cur_tx, lp->tx_full);
--
Don Fry
brazilnut@xxxxxxxxxx
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [PATCH 3/5] 2.6.6-bk8 pcnet32 correct 79C976 variant string., Don Fry |
|---|---|
| Next by Date: | [PATCH 5/5] 2.6.6-bk8 pcnet32 remove timer and complexity., Don Fry |
| Previous by Thread: | [PATCH 3/5] 2.6.6-bk8 pcnet32 correct 79C976 variant string., Don Fry |
| Next by Thread: | [PATCH 5/5] 2.6.6-bk8 pcnet32 remove timer and complexity., Don Fry |
| Indexes: | [Date] [Thread] [Top] [All Lists] |