[PATCH 2.4] via-rhine 1.18-rc1: Fix Rhine-I regression

Roger Luethi (rl@hellgate.ch)
Fri, 4 Jul 2003 15:13:17 +0200


This is a MIME-formatted message. If you see this text it means that your
E-mail software does not support MIME-formatted messages.

--=_courier-23760-1057324539-0001-2
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

This patch addresses a minor regression reported by Rhine-I users (leading
to occasional Tx timeouts).

I also merged some cosmetic changes (including spelling fix from 2.5), but
with the recent interrupt handling changes 2.4 and 2.5 versions of
via-rhine seem to have diverged for good.

Roger

--=_courier-23760-1057324539-0001-2
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment; filename="via-rhine-1.18.diff"

--- linux-2.4/drivers/net/via-rhine.c.org 2003-06-24 22:29:27.000000000 +0200
+++ linux-2.4/drivers/net/via-rhine.c 2003-07-04 15:00:51.000000000 +0200
@@ -2,6 +2,8 @@
/*
Written 1998-2001 by Donald Becker.

+ Current Maintainer: Roger Luethi <rl@hellgate.ch>
+
This software may be used and distributed according to the terms of
the GNU General Public License (GPL), incorporated herein by reference.
Drivers based on or derived from this code fall under the GPL and must
@@ -9,8 +11,9 @@
a complete program and may only be used when the entire operating
system is licensed under the GPL.

- This driver is designed for the VIA VT86C100A Rhine-I.
- It also works with the 6102 Rhine-II, and 6105/6105M Rhine-III.
+ This driver is designed for the VIA VT86C100A Rhine-I.
+ It also works with the Rhine-II (6102) and Rhine-III (6105/6105L/6105LOM
+ and management NIC 6105M).

The author may be reached as becker@scyld.com, or C/O
Scyld Computing Corporation
@@ -115,11 +118,15 @@
- Force flushing for PCI posted writes
- More reset code changes

+ LK1.1.18 (Roger Luethi)
+ - No filtering multicast in promisc mode (Edward Peng)
+ - Fix for Rhine-I Tx timeouts
+
*/

#define DRV_NAME "via-rhine"
-#define DRV_VERSION "1.1.17"
-#define DRV_RELDATE "March-1-2003"
+#define DRV_VERSION "1.1.18"
+#define DRV_RELDATE "July-4-2003"


/* A few user-configurable values.
@@ -139,7 +146,7 @@
Both 'options[]' and 'full_duplex[]' should exist for driver
interoperability.
The media type is usually passed in 'options[]'.
- The default is autonegotation for speed and duplex.
+ The default is autonegotiation for speed and duplex.
This should rarely be overridden.
Use option values 0x10/0x20 for 10Mbps, 0x100,0x200 for 100Mbps.
Use option values 0x10 and 0x100 for forcing half duplex fixed speed.
@@ -386,17 +393,17 @@
{ "VIA VT6102 Rhine-II", RHINE_IOTYPE, 256,
CanHaveMII | HasWOL },
{ "VIA VT6105 Rhine-III", RHINE_IOTYPE, 256,
- CanHaveMII | HasWOL },
+ CanHaveMII | HasWOL },
{ "VIA VT6105M Rhine-III", RHINE_IOTYPE, 256,
- CanHaveMII | HasWOL },
+ CanHaveMII | HasWOL },
};

static struct pci_device_id via_rhine_pci_tbl[] __devinitdata =
{
{0x1106, 0x3043, PCI_ANY_ID, PCI_ANY_ID, 0, 0, VT86C100A},
{0x1106, 0x3065, PCI_ANY_ID, PCI_ANY_ID, 0, 0, VT6102},
- {0x1106, 0x3106, PCI_ANY_ID, PCI_ANY_ID, 0, 0, VT6105},
- {0x1106, 0x3053, PCI_ANY_ID, PCI_ANY_ID, 0, 0, VT6105M},
+ {0x1106, 0x3106, PCI_ANY_ID, PCI_ANY_ID, 0, 0, VT6105}, /* 6105{,L,LOM} */
+ {0x1106, 0x3053, PCI_ANY_ID, PCI_ANY_ID, 0, 0, VT6105M},
{0,} /* terminate list */
};
MODULE_DEVICE_TABLE(pci, via_rhine_pci_tbl);
@@ -441,7 +448,7 @@
IntrRxWakeUp=0x8000,
IntrNormalSummary=0x0003, IntrAbnormalSummary=0xC260,
IntrTxDescRace=0x080000, /* mapped from IntrStatus2 */
- IntrTxErrSummary=0x082210,
+ IntrTxErrSummary=0x082218,
};

/* The Rx and Tx buffer descriptors. */
@@ -1264,7 +1271,7 @@

if (skb->len < ETH_ZLEN) {
skb = skb_padto(skb, ETH_ZLEN);
- if(skb == NULL)
+ if (skb == NULL)
return 0;
}

@@ -1650,11 +1657,18 @@
printk(KERN_INFO "%s: Tx descriptor write-back race.\n",
dev->name);
}
- if (intr_status & ( IntrTxAborted | IntrTxUnderrun | IntrTxDescRace ))
+ if ((intr_status & IntrTxError) && ~( IntrTxAborted | IntrTxUnderrun |
+ IntrTxDescRace )) {
+ if (debug > 2)
+ printk(KERN_INFO "%s: Unspecified error.\n",
+ dev->name);
+ }
+ if (intr_status & ( IntrTxAborted | IntrTxUnderrun | IntrTxDescRace |
+ IntrTxError ))
via_rhine_restart_tx(dev);

if (intr_status & ~( IntrLinkChange | IntrStatsMax | IntrTxUnderrun |
- IntrTxError | IntrTxAborted | IntrNormalSummary |
+ IntrTxError | IntrTxAborted | IntrNormalSummary |
IntrTxDescRace )) {
if (debug > 1)
printk(KERN_ERR "%s: Something Wicked happened! %8.8x.\n",

--=_courier-23760-1057324539-0001-2--