fedora core 6 1.2949 + vserver 2.2.0
[linux-2.6.git] / drivers / net / epic100.c
index 771bfc6..3a6a83d 100644 (file)
 
        Information and updates available at
        http://www.scyld.com/network/epic100.html
+       [this link no longer provides anything useful -jgarzik]
 
        ---------------------------------------------------------------------
-       
-       Linux kernel-specific changes:
-       
-       LK1.1.2 (jgarzik):
-       * Merge becker version 1.09 (4/08/2000)
-
-       LK1.1.3:
-       * Major bugfix to 1.09 driver (Francis Romieu)
-       
-       LK1.1.4 (jgarzik):
-       * Merge becker test version 1.09 (5/29/2000)
-
-       LK1.1.5:
-       * Fix locking (jgarzik)
-       * Limit 83c175 probe to ethernet-class PCI devices (rgooch)
-
-       LK1.1.6:
-       * Merge becker version 1.11
-       * Move pci_enable_device before any PCI BAR len checks
-
-       LK1.1.7:
-       * { fill me in }
-
-       LK1.1.8:
-       * ethtool driver info support (jgarzik)
-
-       LK1.1.9:
-       * ethtool media get/set support (jgarzik)
-
-       LK1.1.10:
-       * revert MII transceiver init change (jgarzik)
-
-       LK1.1.11:
-       * implement ETHTOOL_[GS]SET, _NWAY_RST, _[GS]MSGLVL, _GLINK (jgarzik)
-       * replace some MII-related magic numbers with constants
-
-       LK1.1.12:
-       * fix power-up sequence
-
-       LK1.1.13:
-       * revert version 1.1.12, power-up sequence "fix"
-
-       LK1.1.14 (Kryzsztof Halasa):
-       * fix spurious bad initializations
-       * pound phy a la SMSC's app note on the subject
-       
-       AC1.1.14ac
-       * fix power up/down for ethtool that broke in 1.11
 
 */
 
 #define DRV_NAME        "epic100"
-#define DRV_VERSION     "1.11+LK1.1.14+AC1.1.14"
-#define DRV_RELDATE     "June 2, 2004"
+#define DRV_VERSION     "2.1"
+#define DRV_RELDATE     "Sept 11, 2006"
 
 /* The user-configurable values.
    These may be modified when a driver module is loaded.*/
 
 static int debug = 1;                  /* 1 normal messages, 0 quiet .. 7 verbose. */
-/* Maximum events (Rx packets, etc.) to handle at each interrupt. */
-static int max_interrupt_work = 32;
 
 /* Used to pass the full-duplex flag, etc. */
 #define MAX_UNITS 8            /* More are supported, limit only on options */
@@ -99,9 +50,9 @@ static int rx_copybreak;
    Making the Tx ring too large decreases the effectiveness of channel
    bonding and packet priority.
    There are no ill effects from too-large receive rings. */
-#define TX_RING_SIZE   16
-#define TX_QUEUE_LEN   10              /* Limit ring entries actually used.  */
-#define RX_RING_SIZE   32
+#define TX_RING_SIZE   256
+#define TX_QUEUE_LEN   240             /* Limit ring entries actually used.  */
+#define RX_RING_SIZE   256
 #define TX_TOTAL_SIZE  TX_RING_SIZE*sizeof(struct epic_tx_desc)
 #define RX_TOTAL_SIZE  RX_RING_SIZE*sizeof(struct epic_rx_desc)
 
@@ -116,7 +67,6 @@ static int rx_copybreak;
 #define TX_FIFO_THRESH 256
 #define RX_FIFO_THRESH 1               /* 0-3, 0==32, 64,96, or 3==128 bytes  */
 
-#include <linux/config.h>
 #include <linux/module.h>
 #include <linux/kernel.h>
 #include <linux/string.h>
@@ -135,7 +85,7 @@ static int rx_copybreak;
 #include <linux/ethtool.h>
 #include <linux/mii.h>
 #include <linux/crc32.h>
-#include <asm/bitops.h>
+#include <linux/bitops.h>
 #include <asm/io.h>
 #include <asm/uaccess.h>
 
@@ -151,13 +101,11 @@ MODULE_AUTHOR("Donald Becker <becker@scyld.com>");
 MODULE_DESCRIPTION("SMC 83c170 EPIC series Ethernet driver");
 MODULE_LICENSE("GPL");
 
-MODULE_PARM(debug, "i");
-MODULE_PARM(max_interrupt_work, "i");
-MODULE_PARM(rx_copybreak, "i");
-MODULE_PARM(options, "1-" __MODULE_STRING(MAX_UNITS) "i");
-MODULE_PARM(full_duplex, "1-" __MODULE_STRING(MAX_UNITS) "i");
+module_param(debug, int, 0);
+module_param(rx_copybreak, int, 0);
+module_param_array(options, int, NULL, 0);
+module_param_array(full_duplex, int, NULL, 0);
 MODULE_PARM_DESC(debug, "EPIC/100 debug level (0-5)");
-MODULE_PARM_DESC(max_interrupt_work, "EPIC/100 maximum events handled per interrupt");
 MODULE_PARM_DESC(options, "EPIC/100: Bits 0-3: media type, bit 4: full duplex");
 MODULE_PARM_DESC(rx_copybreak, "EPIC/100 copy breakpoint for copy-only-tiny-frames");
 MODULE_PARM_DESC(full_duplex, "EPIC/100 full duplex setting(s) (1)");
@@ -195,23 +143,10 @@ IVc. Errata
 */
 
 
-enum pci_id_flags_bits {
-        /* Set PCI command register bits before calling probe1(). */
-        PCI_USES_IO=1, PCI_USES_MEM=2, PCI_USES_MASTER=4,
-        /* Read and map the single following PCI BAR. */
-        PCI_ADDR0=0<<4, PCI_ADDR1=1<<4, PCI_ADDR2=2<<4, PCI_ADDR3=3<<4,
-        PCI_ADDR_64BITS=0x100, PCI_NO_ACPI_WAKE=0x200, PCI_NO_MIN_LATENCY=0x400,
-};
-
 enum chip_capability_flags { MII_PWRDWN=1, TYPE2_INTR=2, NO_MII=4 };
 
 #define EPIC_TOTAL_SIZE 0x100
 #define USE_IO_OPS 1
-#ifdef USE_IO_OPS
-#define EPIC_IOTYPE PCI_USES_MASTER|PCI_USES_IO|PCI_ADDR0
-#else
-#define EPIC_IOTYPE PCI_USES_MASTER|PCI_USES_MEM|PCI_ADDR1
-#endif
 
 typedef enum {
        SMSC_83C170_0,
@@ -222,20 +157,15 @@ typedef enum {
 
 struct epic_chip_info {
        const char *name;
-       enum pci_id_flags_bits pci_flags;
-        int io_size;                            /* Needed for I/O region check or ioremap(). */
         int drv_flags;                          /* Driver use, intended as capability flags. */
 };
 
 
 /* indexed by chip_t */
-static struct epic_chip_info pci_id_tbl[] = {
-       { "SMSC EPIC/100 83c170",
-        EPIC_IOTYPE, EPIC_TOTAL_SIZE, TYPE2_INTR | NO_MII | MII_PWRDWN },
-       { "SMSC EPIC/100 83c170",
-        EPIC_IOTYPE, EPIC_TOTAL_SIZE, TYPE2_INTR },
-       { "SMSC EPIC/C 83c175",
-        EPIC_IOTYPE, EPIC_TOTAL_SIZE, TYPE2_INTR | MII_PWRDWN },
+static const struct epic_chip_info pci_id_tbl[] = {
+       { "SMSC EPIC/100 83c170",       TYPE2_INTR | NO_MII | MII_PWRDWN },
+       { "SMSC EPIC/100 83c170",       TYPE2_INTR },
+       { "SMSC EPIC/C 83c175",         TYPE2_INTR | MII_PWRDWN },
 };
 
 
@@ -248,7 +178,7 @@ static struct pci_device_id epic_pci_tbl[] = {
 };
 MODULE_DEVICE_TABLE (pci, epic_pci_tbl);
 
-       
+
 #ifndef USE_IO_OPS
 #undef inb
 #undef inw
@@ -289,7 +219,13 @@ enum CommandBits {
        StopTxDMA=0x20, StopRxDMA=0x40, RestartTx=0x80,
 };
 
-static u16 media2miictl[16] = {
+#define EpicRemoved    0xffffffff      /* Chip failed or removed (CardBus) */
+
+#define EpicNapiEvent  (TxEmpty | TxDone | \
+                        RxDone | RxStarted | RxEarlyWarn | RxOverflow | RxFull)
+#define EpicNormalEvent        (0x0000ffff & ~EpicNapiEvent)
+
+static const u16 media2miictl[16] = {
        0, 0x0C00, 0x0C00, 0x2000,  0x0100, 0x2100, 0, 0,
        0, 0, 0, 0,  0, 0, 0, 0 };
 
@@ -327,9 +263,12 @@ struct epic_private {
 
        /* Ring pointers. */
        spinlock_t lock;                                /* Group with Tx control cache line. */
+       spinlock_t napi_lock;
+       unsigned int reschedule_in_poll;
        unsigned int cur_tx, dirty_tx;
 
        unsigned int cur_rx, dirty_rx;
+       u32 irq_mask;
        unsigned int rx_buf_sz;                         /* Based on MTU+slack. */
 
        struct pci_dev *pci_dev;                        /* PCI bus location. */
@@ -356,15 +295,16 @@ static void epic_timer(unsigned long data);
 static void epic_tx_timeout(struct net_device *dev);
 static void epic_init_ring(struct net_device *dev);
 static int epic_start_xmit(struct sk_buff *skb, struct net_device *dev);
-static int epic_rx(struct net_device *dev);
-static irqreturn_t epic_interrupt(int irq, void *dev_instance, struct pt_regs *regs);
+static int epic_rx(struct net_device *dev, int budget);
+static int epic_poll(struct net_device *dev, int *budget);
+static irqreturn_t epic_interrupt(int irq, void *dev_instance);
 static int netdev_ioctl(struct net_device *dev, struct ifreq *rq, int cmd);
-static struct ethtool_ops netdev_ethtool_ops;
+static const struct ethtool_ops netdev_ethtool_ops;
 static int epic_close(struct net_device *dev);
 static struct net_device_stats *epic_get_stats(struct net_device *dev);
 static void set_rx_mode(struct net_device *dev);
 
-\f
+
 
 static int __devinit epic_init_one (struct pci_dev *pdev,
                                    const struct pci_device_id *ent)
@@ -375,7 +315,7 @@ static int __devinit epic_init_one (struct pci_dev *pdev,
        int irq;
        struct net_device *dev;
        struct epic_private *ep;
-       int i, option = 0, duplex = 0;
+       int i, ret, option = 0, duplex = 0;
        void *ring_space;
        dma_addr_t ring_dma;
 
@@ -386,40 +326,44 @@ static int __devinit epic_init_one (struct pci_dev *pdev,
                printk (KERN_INFO "%s" KERN_INFO "%s" KERN_INFO "%s",
                        version, version2, version3);
 #endif
-       
+
        card_idx++;
-       
-       i = pci_enable_device(pdev);
-       if (i)
-               return i;
+
+       ret = pci_enable_device(pdev);
+       if (ret)
+               goto out;
        irq = pdev->irq;
 
-       if (pci_resource_len(pdev, 0) < pci_id_tbl[chip_idx].io_size) {
-               printk (KERN_ERR "card %d: no PCI region space\n", card_idx);
-               return -ENODEV;
+       if (pci_resource_len(pdev, 0) < EPIC_TOTAL_SIZE) {
+               dev_err(&pdev->dev, "no PCI region space\n");
+               ret = -ENODEV;
+               goto err_out_disable;
        }
-       
+
        pci_set_master(pdev);
 
+       ret = pci_request_regions(pdev, DRV_NAME);
+       if (ret < 0)
+               goto err_out_disable;
+
+       ret = -ENOMEM;
+
        dev = alloc_etherdev(sizeof (*ep));
        if (!dev) {
-               printk (KERN_ERR "card %d: no memory for eth device\n", card_idx);
-               return -ENOMEM;
+               dev_err(&pdev->dev, "no memory for eth device\n");
+               goto err_out_free_res;
        }
        SET_MODULE_OWNER(dev);
        SET_NETDEV_DEV(dev, &pdev->dev);
 
-       if (pci_request_regions(pdev, DRV_NAME))
-               goto err_out_free_netdev;
-
 #ifdef USE_IO_OPS
        ioaddr = pci_resource_start (pdev, 0);
 #else
        ioaddr = pci_resource_start (pdev, 1);
        ioaddr = (long) ioremap (ioaddr, pci_resource_len (pdev, 1));
        if (!ioaddr) {
-               printk (KERN_ERR DRV_NAME " %d: ioremap failed\n", card_idx);
-               goto err_out_free_res;
+               dev_err(&pdev->dev, "ioremap failed\n");
+               goto err_out_free_netdev;
        }
 #endif
 
@@ -456,7 +400,9 @@ static int __devinit epic_init_one (struct pci_dev *pdev,
        dev->base_addr = ioaddr;
        dev->irq = irq;
 
-       spin_lock_init (&ep->lock);
+       spin_lock_init(&ep->lock);
+       spin_lock_init(&ep->napi_lock);
+       ep->reschedule_in_poll = 0;
 
        /* Bring the chip out of low-power mode. */
        outl(0x4200, ioaddr + GENCTL);
@@ -476,8 +422,7 @@ static int __devinit epic_init_one (struct pci_dev *pdev,
                ((u16 *)dev->dev_addr)[i] = le16_to_cpu(inw(ioaddr + LAN0 + i*4));
 
        if (debug > 2) {
-               printk(KERN_DEBUG DRV_NAME "(%s): EEPROM contents\n",
-                      pci_name(pdev));
+               dev_printk(KERN_DEBUG, &pdev->dev, "EEPROM contents:\n");
                for (i = 0; i < 64; i++)
                        printk(" %4.4x%s", read_eeprom(ioaddr, i),
                                   i % 16 == 15 ? "\n" : "");
@@ -486,6 +431,9 @@ static int __devinit epic_init_one (struct pci_dev *pdev,
        ep->pci_dev = pdev;
        ep->chip_id = chip_idx;
        ep->chip_flags = pci_id_tbl[chip_idx].drv_flags;
+       ep->irq_mask =
+               (ep->chip_flags & TYPE2_INTR ?  PCIBusErr175 : PCIBusErr170)
+                | CntFull | TxUnderrun | EpicNapiEvent;
 
        /* Find the connected MII xcvrs.
           Doing this in open() would allow detecting external xcvrs later, but
@@ -496,21 +444,23 @@ static int __devinit epic_init_one (struct pci_dev *pdev,
                        int mii_status = mdio_read(dev, phy, MII_BMSR);
                        if (mii_status != 0xffff  &&  mii_status != 0x0000) {
                                ep->phys[phy_idx++] = phy;
-                               printk(KERN_INFO DRV_NAME "(%s): MII transceiver #%d control "
-                                          "%4.4x status %4.4x.\n",
-                                          pci_name(pdev), phy, mdio_read(dev, phy, 0), mii_status);
+                               dev_info(&pdev->dev,
+                                       "MII transceiver #%d control "
+                                       "%4.4x status %4.4x.\n",
+                                       phy, mdio_read(dev, phy, 0), mii_status);
                        }
                }
                ep->mii_phy_cnt = phy_idx;
                if (phy_idx != 0) {
                        phy = ep->phys[0];
                        ep->mii.advertising = mdio_read(dev, phy, MII_ADVERTISE);
-                       printk(KERN_INFO DRV_NAME "(%s): Autonegotiation advertising %4.4x link "
+                       dev_info(&pdev->dev,
+                               "Autonegotiation advertising %4.4x link "
                                   "partner %4.4x.\n",
-                                  pci_name(pdev), ep->mii.advertising, mdio_read(dev, phy, 5));
+                                  ep->mii.advertising, mdio_read(dev, phy, 5));
                } else if ( ! (ep->chip_flags & NO_MII)) {
-                       printk(KERN_WARNING DRV_NAME "(%s): ***WARNING***: No MII transceiver found!\n",
-                              pci_name(pdev));
+                       dev_warn(&pdev->dev,
+                               "***WARNING***: No MII transceiver found!\n");
                        /* Use the known PHY address of the EPII. */
                        ep->phys[0] = 3;
                }
@@ -525,8 +475,7 @@ static int __devinit epic_init_one (struct pci_dev *pdev,
        /* The lower four bits are the media type. */
        if (duplex) {
                ep->mii.force_media = ep->mii.full_duplex = 1;
-               printk(KERN_INFO DRV_NAME "(%s):  Forced full duplex operation requested.\n",
-                      pci_name(pdev));
+               dev_info(&pdev->dev, "Forced full duplex requested.\n");
        }
        dev->if_port = ep->default_port = option;
 
@@ -540,10 +489,12 @@ static int __devinit epic_init_one (struct pci_dev *pdev,
        dev->ethtool_ops = &netdev_ethtool_ops;
        dev->watchdog_timeo = TX_TIMEOUT;
        dev->tx_timeout = &epic_tx_timeout;
+       dev->poll = epic_poll;
+       dev->weight = 64;
 
-       i = register_netdev(dev);
-       if (i)
-               goto err_out_unmap_tx;
+       ret = register_netdev(dev);
+       if (ret < 0)
+               goto err_out_unmap_rx;
 
        printk(KERN_INFO "%s: %s at %#lx, IRQ %d, ",
                   dev->name, pci_id_tbl[chip_idx].name, ioaddr, dev->irq);
@@ -551,21 +502,26 @@ static int __devinit epic_init_one (struct pci_dev *pdev,
                printk("%2.2x:", dev->dev_addr[i]);
        printk("%2.2x.\n", dev->dev_addr[i]);
 
-       return 0;
+out:
+       return ret;
 
+err_out_unmap_rx:
+       pci_free_consistent(pdev, RX_TOTAL_SIZE, ep->rx_ring, ep->rx_ring_dma);
 err_out_unmap_tx:
        pci_free_consistent(pdev, TX_TOTAL_SIZE, ep->tx_ring, ep->tx_ring_dma);
 err_out_iounmap:
 #ifndef USE_IO_OPS
        iounmap(ioaddr);
-err_out_free_res:
-#endif
-       pci_release_regions(pdev);
 err_out_free_netdev:
+#endif
        free_netdev(dev);
-       return -ENODEV;
+err_out_free_res:
+       pci_release_regions(pdev);
+err_out_disable:
+       pci_disable_device(pdev);
+       goto out;
 }
-\f
+
 /* Serial EEPROM section. */
 
 /*  EEPROM_Ctrl bits. */
@@ -589,6 +545,38 @@ err_out_free_netdev:
 #define EE_READ256_CMD (6 << 8)
 #define EE_ERASE_CMD   (7 << 6)
 
+static void epic_disable_int(struct net_device *dev, struct epic_private *ep)
+{
+       long ioaddr = dev->base_addr;
+
+       outl(0x00000000, ioaddr + INTMASK);
+}
+
+static inline void __epic_pci_commit(long ioaddr)
+{
+#ifndef USE_IO_OPS
+       inl(ioaddr + INTMASK);
+#endif
+}
+
+static inline void epic_napi_irq_off(struct net_device *dev,
+                                    struct epic_private *ep)
+{
+       long ioaddr = dev->base_addr;
+
+       outl(ep->irq_mask & ~EpicNapiEvent, ioaddr + INTMASK);
+       __epic_pci_commit(ioaddr);
+}
+
+static inline void epic_napi_irq_on(struct net_device *dev,
+                                   struct epic_private *ep)
+{
+       long ioaddr = dev->base_addr;
+
+       /* No need to commit possible posted write */
+       outl(ep->irq_mask | EpicNapiEvent, ioaddr + INTMASK);
+}
+
 static int __devinit read_eeprom(long ioaddr, int location)
 {
        int i;
@@ -655,7 +643,7 @@ static void mdio_write(struct net_device *dev, int phy_id, int loc, int value)
 
        outw(value, ioaddr + MIIData);
        outl((phy_id << 9) | (loc << 4) | MII_WRITEOP, ioaddr + MIICtrl);
-       for (i = 10000; i > 0; i--) { 
+       for (i = 10000; i > 0; i--) {
                barrier();
                if ((inl(ioaddr + MIICtrl) & MII_WRITEOP) == 0)
                        break;
@@ -663,7 +651,7 @@ static void mdio_write(struct net_device *dev, int phy_id, int loc, int value)
        return;
 }
 
-\f
+
 static int epic_open(struct net_device *dev)
 {
        struct epic_private *ep = dev->priv;
@@ -674,7 +662,7 @@ static int epic_open(struct net_device *dev)
        /* Soft reset the chip. */
        outl(0x4001, ioaddr + GENCTL);
 
-       if ((retval = request_irq(dev->irq, &epic_interrupt, SA_SHIRQ, dev->name, dev)))
+       if ((retval = request_irq(dev->irq, &epic_interrupt, IRQF_SHARED, dev->name, dev)))
                return retval;
 
        epic_init_ring(dev);
@@ -706,7 +694,7 @@ static int epic_open(struct net_device *dev)
 #endif
 
        udelay(20); /* Looks like EPII needs that if you want reliable RX init. FIXME: pci posting bug? */
-       
+
        for (i = 0; i < 3; i++)
                outl(cpu_to_le16(((u16*)dev->dev_addr)[i]), ioaddr + LAN0 + i*4);
 
@@ -749,9 +737,8 @@ static int epic_open(struct net_device *dev)
 
        /* Enable interrupts by setting the interrupt mask. */
        outl((ep->chip_flags & TYPE2_INTR ? PCIBusErr175 : PCIBusErr170)
-                | CntFull | TxUnderrun | TxDone | TxEmpty
-                | RxError | RxOverflow | RxFull | RxHeader | RxDone,
-                ioaddr + INTMASK);
+                | CntFull | TxUnderrun
+                | RxError | RxHeader | EpicNapiEvent, ioaddr + INTMASK);
 
        if (debug > 1)
                printk(KERN_DEBUG "%s: epic_open() ioaddr %lx IRQ %d status %4.4x "
@@ -778,7 +765,7 @@ static void epic_pause(struct net_device *dev)
        struct epic_private *ep = dev->priv;
 
        netif_stop_queue (dev);
-       
+
        /* Disable interrupts by clearing the interrupt mask. */
        outl(0x00000000, ioaddr + INTMASK);
        /* Stop the chip's Tx and Rx DMA processes. */
@@ -792,7 +779,7 @@ static void epic_pause(struct net_device *dev)
        }
 
        /* Remove the packets on the Rx queue. */
-       epic_rx(dev);
+       epic_rx(dev, RX_RING_SIZE);
 }
 
 static void epic_restart(struct net_device *dev)
@@ -838,9 +825,9 @@ static void epic_restart(struct net_device *dev)
 
        /* Enable interrupts by setting the interrupt mask. */
        outl((ep->chip_flags & TYPE2_INTR ? PCIBusErr175 : PCIBusErr170)
-                | CntFull | TxUnderrun | TxDone | TxEmpty
-                | RxError | RxOverflow | RxFull | RxHeader | RxDone,
-                ioaddr + INTMASK);
+                | CntFull | TxUnderrun
+                | RxError | RxHeader | EpicNapiEvent, ioaddr + INTMASK);
+
        printk(KERN_DEBUG "%s: epic_restart() done, cmd status %4.4x, ctl %4.4x"
                   " interrupt %4.4x.\n",
                   dev->name, (int)inl(ioaddr + COMMAND), (int)inl(ioaddr + GENCTL),
@@ -926,7 +913,6 @@ static void epic_init_ring(struct net_device *dev)
        int i;
 
        ep->tx_full = 0;
-       ep->lock = (spinlock_t) SPIN_LOCK_UNLOCKED;
        ep->dirty_tx = ep->cur_tx = 0;
        ep->cur_rx = ep->dirty_rx = 0;
        ep->rx_buf_sz = (dev->mtu <= 1500 ? PKT_BUF_SZ : dev->mtu + 32);
@@ -935,7 +921,7 @@ static void epic_init_ring(struct net_device *dev)
        for (i = 0; i < RX_RING_SIZE; i++) {
                ep->rx_ring[i].rxstatus = 0;
                ep->rx_ring[i].buflength = cpu_to_le32(ep->rx_buf_sz);
-               ep->rx_ring[i].next = ep->rx_ring_dma + 
+               ep->rx_ring[i].next = ep->rx_ring_dma +
                                      (i+1)*sizeof(struct epic_rx_desc);
                ep->rx_skbuff[i] = NULL;
        }
@@ -950,8 +936,8 @@ static void epic_init_ring(struct net_device *dev)
                        break;
                skb->dev = dev;                 /* Mark as being used by this device. */
                skb_reserve(skb, 2);    /* 16 byte align the IP header. */
-               ep->rx_ring[i].bufaddr = pci_map_single(ep->pci_dev, 
-                       skb->tail, ep->rx_buf_sz, PCI_DMA_FROMDEVICE);
+               ep->rx_ring[i].bufaddr = pci_map_single(ep->pci_dev,
+                       skb->data, ep->rx_buf_sz, PCI_DMA_FROMDEVICE);
                ep->rx_ring[i].rxstatus = cpu_to_le32(DescOwn);
        }
        ep->dirty_rx = (unsigned int)(i - RX_RING_SIZE);
@@ -961,7 +947,7 @@ static void epic_init_ring(struct net_device *dev)
        for (i = 0; i < TX_RING_SIZE; i++) {
                ep->tx_skbuff[i] = NULL;
                ep->tx_ring[i].txstatus = 0x0000;
-               ep->tx_ring[i].next = ep->tx_ring_dma + 
+               ep->tx_ring[i].next = ep->tx_ring_dma +
                        (i+1)*sizeof(struct epic_tx_desc);
        }
        ep->tx_ring[i-1].next = ep->tx_ring_dma;
@@ -974,12 +960,9 @@ static int epic_start_xmit(struct sk_buff *skb, struct net_device *dev)
        int entry, free_count;
        u32 ctrl_word;
        unsigned long flags;
-       
-       if (skb->len < ETH_ZLEN) {
-               skb = skb_padto(skb, ETH_ZLEN);
-               if (skb == NULL)
-                       return 0;
-       }
+
+       if (skb_padto(skb, ETH_ZLEN))
+               return 0;
 
        /* Caution: the write order is important here, set the field with the
           "ownership" bit last. */
@@ -990,7 +973,7 @@ static int epic_start_xmit(struct sk_buff *skb, struct net_device *dev)
        entry = ep->cur_tx % TX_RING_SIZE;
 
        ep->tx_skbuff[entry] = skb;
-       ep->tx_ring[entry].bufaddr = pci_map_single(ep->pci_dev, skb->data, 
+       ep->tx_ring[entry].bufaddr = pci_map_single(ep->pci_dev, skb->data,
                                                    skb->len, PCI_DMA_TODEVICE);
        if (free_count < TX_QUEUE_LEN/2) {/* Typical path */
                ctrl_word = cpu_to_le32(0x100000); /* No interrupt */
@@ -1026,142 +1009,148 @@ static int epic_start_xmit(struct sk_buff *skb, struct net_device *dev)
        return 0;
 }
 
+static void epic_tx_error(struct net_device *dev, struct epic_private *ep,
+                         int status)
+{
+       struct net_device_stats *stats = &ep->stats;
+
+#ifndef final_version
+       /* There was an major error, log it. */
+       if (debug > 1)
+               printk(KERN_DEBUG "%s: Transmit error, Tx status %8.8x.\n",
+                      dev->name, status);
+#endif
+       stats->tx_errors++;
+       if (status & 0x1050)
+               stats->tx_aborted_errors++;
+       if (status & 0x0008)
+               stats->tx_carrier_errors++;
+       if (status & 0x0040)
+               stats->tx_window_errors++;
+       if (status & 0x0010)
+               stats->tx_fifo_errors++;
+}
+
+static void epic_tx(struct net_device *dev, struct epic_private *ep)
+{
+       unsigned int dirty_tx, cur_tx;
+
+       /*
+        * Note: if this lock becomes a problem we can narrow the locked
+        * region at the cost of occasionally grabbing the lock more times.
+        */
+       cur_tx = ep->cur_tx;
+       for (dirty_tx = ep->dirty_tx; cur_tx - dirty_tx > 0; dirty_tx++) {
+               struct sk_buff *skb;
+               int entry = dirty_tx % TX_RING_SIZE;
+               int txstatus = le32_to_cpu(ep->tx_ring[entry].txstatus);
+
+               if (txstatus & DescOwn)
+                       break;  /* It still hasn't been Txed */
+
+               if (likely(txstatus & 0x0001)) {
+                       ep->stats.collisions += (txstatus >> 8) & 15;
+                       ep->stats.tx_packets++;
+                       ep->stats.tx_bytes += ep->tx_skbuff[entry]->len;
+               } else
+                       epic_tx_error(dev, ep, txstatus);
+
+               /* Free the original skb. */
+               skb = ep->tx_skbuff[entry];
+               pci_unmap_single(ep->pci_dev, ep->tx_ring[entry].bufaddr,
+                                skb->len, PCI_DMA_TODEVICE);
+               dev_kfree_skb_irq(skb);
+               ep->tx_skbuff[entry] = NULL;
+       }
+
+#ifndef final_version
+       if (cur_tx - dirty_tx > TX_RING_SIZE) {
+               printk(KERN_WARNING
+                      "%s: Out-of-sync dirty pointer, %d vs. %d, full=%d.\n",
+                      dev->name, dirty_tx, cur_tx, ep->tx_full);
+               dirty_tx += TX_RING_SIZE;
+       }
+#endif
+       ep->dirty_tx = dirty_tx;
+       if (ep->tx_full && cur_tx - dirty_tx < TX_QUEUE_LEN - 4) {
+               /* The ring is no longer full, allow new TX entries. */
+               ep->tx_full = 0;
+               netif_wake_queue(dev);
+       }
+}
+
 /* The interrupt handler does all of the Rx thread work and cleans up
    after the Tx thread. */
-static irqreturn_t epic_interrupt(int irq, void *dev_instance, struct pt_regs *regs)
+static irqreturn_t epic_interrupt(int irq, void *dev_instance)
 {
        struct net_device *dev = dev_instance;
        struct epic_private *ep = dev->priv;
        long ioaddr = dev->base_addr;
-       int status, boguscnt = max_interrupt_work;
        unsigned int handled = 0;
+       int status;
 
-       do {
-               status = inl(ioaddr + INTSTAT);
-               /* Acknowledge all of the current interrupt sources ASAP. */
-               outl(status & 0x00007fff, ioaddr + INTSTAT);
+       status = inl(ioaddr + INTSTAT);
+       /* Acknowledge all of the current interrupt sources ASAP. */
+       outl(status & EpicNormalEvent, ioaddr + INTSTAT);
 
-               if (debug > 4)
-                       printk(KERN_DEBUG "%s: Interrupt, status=%#8.8x new "
-                                  "intstat=%#8.8x.\n",
-                                  dev->name, status, (int)inl(ioaddr + INTSTAT));
+       if (debug > 4) {
+               printk(KERN_DEBUG "%s: Interrupt, status=%#8.8x new "
+                                  "intstat=%#8.8x.\n", dev->name, status,
+                                  (int)inl(ioaddr + INTSTAT));
+       }
 
-               if ((status & IntrSummary) == 0)
-                       break;
-               handled = 1;
-
-               if (status & (RxDone | RxStarted | RxEarlyWarn | RxOverflow))
-                       epic_rx(dev);
-
-               if (status & (TxEmpty | TxDone)) {
-                       unsigned int dirty_tx, cur_tx;
-
-                       /* Note: if this lock becomes a problem we can narrow the locked
-                          region at the cost of occasionally grabbing the lock more
-                          times. */
-                       spin_lock(&ep->lock);
-                       cur_tx = ep->cur_tx;
-                       dirty_tx = ep->dirty_tx;
-                       for (; cur_tx - dirty_tx > 0; dirty_tx++) {
-                               struct sk_buff *skb;
-                               int entry = dirty_tx % TX_RING_SIZE;
-                               int txstatus = le32_to_cpu(ep->tx_ring[entry].txstatus);
-
-                               if (txstatus & DescOwn)
-                                       break;                  /* It still hasn't been Txed */
-
-                               if ( ! (txstatus & 0x0001)) {
-                                       /* There was an major error, log it. */
-#ifndef final_version
-                                       if (debug > 1)
-                                               printk(KERN_DEBUG "%s: Transmit error, Tx status %8.8x.\n",
-                                                          dev->name, txstatus);
-#endif
-                                       ep->stats.tx_errors++;
-                                       if (txstatus & 0x1050) ep->stats.tx_aborted_errors++;
-                                       if (txstatus & 0x0008) ep->stats.tx_carrier_errors++;
-                                       if (txstatus & 0x0040) ep->stats.tx_window_errors++;
-                                       if (txstatus & 0x0010) ep->stats.tx_fifo_errors++;
-                               } else {
-                                       ep->stats.collisions += (txstatus >> 8) & 15;
-                                       ep->stats.tx_packets++;
-                                       ep->stats.tx_bytes += ep->tx_skbuff[entry]->len;
-                               }
-
-                               /* Free the original skb. */
-                               skb = ep->tx_skbuff[entry];
-                               pci_unmap_single(ep->pci_dev, ep->tx_ring[entry].bufaddr, 
-                                                skb->len, PCI_DMA_TODEVICE);
-                               dev_kfree_skb_irq(skb);
-                               ep->tx_skbuff[entry] = NULL;
-                       }
+       if ((status & IntrSummary) == 0)
+               goto out;
 
-#ifndef final_version
-                       if (cur_tx - dirty_tx > TX_RING_SIZE) {
-                               printk(KERN_WARNING "%s: Out-of-sync dirty pointer, %d vs. %d, full=%d.\n",
-                                          dev->name, dirty_tx, cur_tx, ep->tx_full);
-                               dirty_tx += TX_RING_SIZE;
-                       }
-#endif
-                       ep->dirty_tx = dirty_tx;
-                       if (ep->tx_full
-                               && cur_tx - dirty_tx < TX_QUEUE_LEN - 4) {
-                               /* The ring is no longer full, allow new TX entries. */
-                               ep->tx_full = 0;
-                               spin_unlock(&ep->lock);
-                               netif_wake_queue(dev);
-                       } else
-                               spin_unlock(&ep->lock);
-               }
+       handled = 1;
 
-               /* Check uncommon events all at once. */
-               if (status & (CntFull | TxUnderrun | RxOverflow | RxFull |
-                                         PCIBusErr170 | PCIBusErr175)) {
-                       if (status == 0xffffffff) /* Chip failed or removed (CardBus). */
-                               break;
-                       /* Always update the error counts to avoid overhead later. */
-                       ep->stats.rx_missed_errors += inb(ioaddr + MPCNT);
-                       ep->stats.rx_frame_errors += inb(ioaddr + ALICNT);
-                       ep->stats.rx_crc_errors += inb(ioaddr + CRCCNT);
-
-                       if (status & TxUnderrun) { /* Tx FIFO underflow. */
-                               ep->stats.tx_fifo_errors++;
-                               outl(ep->tx_threshold += 128, ioaddr + TxThresh);
-                               /* Restart the transmit process. */
-                               outl(RestartTx, ioaddr + COMMAND);
-                       }
-                       if (status & RxOverflow) {              /* Missed a Rx frame. */
-                               ep->stats.rx_errors++;
-                       }
-                       if (status & (RxOverflow | RxFull))
-                               outw(RxQueued, ioaddr + COMMAND);
-                       if (status & PCIBusErr170) {
-                               printk(KERN_ERR "%s: PCI Bus Error!  EPIC status %4.4x.\n",
-                                          dev->name, status);
-                               epic_pause(dev);
-                               epic_restart(dev);
-                       }
-                       /* Clear all error sources. */
-                       outl(status & 0x7f18, ioaddr + INTSTAT);
+       if ((status & EpicNapiEvent) && !ep->reschedule_in_poll) {
+               spin_lock(&ep->napi_lock);
+               if (netif_rx_schedule_prep(dev)) {
+                       epic_napi_irq_off(dev, ep);
+                       __netif_rx_schedule(dev);
+               } else
+                       ep->reschedule_in_poll++;
+               spin_unlock(&ep->napi_lock);
+       }
+       status &= ~EpicNapiEvent;
+
+       /* Check uncommon events all at once. */
+       if (status & (CntFull | TxUnderrun | PCIBusErr170 | PCIBusErr175)) {
+               if (status == EpicRemoved)
+                       goto out;
+
+               /* Always update the error counts to avoid overhead later. */
+               ep->stats.rx_missed_errors += inb(ioaddr + MPCNT);
+               ep->stats.rx_frame_errors += inb(ioaddr + ALICNT);
+               ep->stats.rx_crc_errors += inb(ioaddr + CRCCNT);
+
+               if (status & TxUnderrun) { /* Tx FIFO underflow. */
+                       ep->stats.tx_fifo_errors++;
+                       outl(ep->tx_threshold += 128, ioaddr + TxThresh);
+                       /* Restart the transmit process. */
+                       outl(RestartTx, ioaddr + COMMAND);
                }
-               if (--boguscnt < 0) {
-                       printk(KERN_ERR "%s: Too much work at interrupt, "
-                                  "IntrStatus=0x%8.8x.\n",
-                                  dev->name, status);
-                       /* Clear all interrupt sources. */
-                       outl(0x0001ffff, ioaddr + INTSTAT);
-                       break;
+               if (status & PCIBusErr170) {
+                       printk(KERN_ERR "%s: PCI Bus Error! status %4.4x.\n",
+                                        dev->name, status);
+                       epic_pause(dev);
+                       epic_restart(dev);
                }
-       } while (1);
+               /* Clear all error sources. */
+               outl(status & 0x7f18, ioaddr + INTSTAT);
+       }
 
-       if (debug > 3)
-               printk(KERN_DEBUG "%s: exiting interrupt, intr_status=%#4.4x.\n",
-                          dev->name, status);
+out:
+       if (debug > 3) {
+               printk(KERN_DEBUG "%s: exit interrupt, intr_status=%#4.4x.\n",
+                                  dev->name, status);
+       }
 
        return IRQ_RETVAL(handled);
 }
 
-static int epic_rx(struct net_device *dev)
+static int epic_rx(struct net_device *dev, int budget)
 {
        struct epic_private *ep = dev->priv;
        int entry = ep->cur_rx % RX_RING_SIZE;
@@ -1171,6 +1160,10 @@ static int epic_rx(struct net_device *dev)
        if (debug > 4)
                printk(KERN_DEBUG " In epic_rx(), entry %d %8.8x.\n", entry,
                           ep->rx_ring[entry].rxstatus);
+
+       if (rx_work_limit > budget)
+               rx_work_limit = budget;
+
        /* If we own the next entry, it's a new packet. Send it up. */
        while ((ep->rx_ring[entry].rxstatus & cpu_to_le32(DescOwn)) == 0) {
                int status = le32_to_cpu(ep->rx_ring[entry].rxstatus);
@@ -1212,21 +1205,21 @@ static int epic_rx(struct net_device *dev)
                                                            ep->rx_ring[entry].bufaddr,
                                                            ep->rx_buf_sz,
                                                            PCI_DMA_FROMDEVICE);
-                               eth_copy_and_sum(skb, ep->rx_skbuff[entry]->tail, pkt_len, 0);
+                               eth_copy_and_sum(skb, ep->rx_skbuff[entry]->data, pkt_len, 0);
                                skb_put(skb, pkt_len);
                                pci_dma_sync_single_for_device(ep->pci_dev,
                                                               ep->rx_ring[entry].bufaddr,
                                                               ep->rx_buf_sz,
                                                               PCI_DMA_FROMDEVICE);
                        } else {
-                               pci_unmap_single(ep->pci_dev, 
-                                       ep->rx_ring[entry].bufaddr, 
+                               pci_unmap_single(ep->pci_dev,
+                                       ep->rx_ring[entry].bufaddr,
                                        ep->rx_buf_sz, PCI_DMA_FROMDEVICE);
                                skb_put(skb = ep->rx_skbuff[entry], pkt_len);
                                ep->rx_skbuff[entry] = NULL;
                        }
                        skb->protocol = eth_type_trans(skb, dev);
-                       netif_rx(skb);
+                       netif_receive_skb(skb);
                        dev->last_rx = jiffies;
                        ep->stats.rx_packets++;
                        ep->stats.rx_bytes += pkt_len;
@@ -1245,8 +1238,8 @@ static int epic_rx(struct net_device *dev)
                                break;
                        skb->dev = dev;                 /* Mark as being used by this device. */
                        skb_reserve(skb, 2);    /* Align IP on 16 byte boundaries */
-                       ep->rx_ring[entry].bufaddr = pci_map_single(ep->pci_dev, 
-                               skb->tail, ep->rx_buf_sz, PCI_DMA_FROMDEVICE);
+                       ep->rx_ring[entry].bufaddr = pci_map_single(ep->pci_dev,
+                               skb->data, ep->rx_buf_sz, PCI_DMA_FROMDEVICE);
                        work_done++;
                }
                ep->rx_ring[entry].rxstatus = cpu_to_le32(DescOwn);
@@ -1254,6 +1247,65 @@ static int epic_rx(struct net_device *dev)
        return work_done;
 }
 
+static void epic_rx_err(struct net_device *dev, struct epic_private *ep)
+{
+       long ioaddr = dev->base_addr;
+       int status;
+
+       status = inl(ioaddr + INTSTAT);
+
+       if (status == EpicRemoved)
+               return;
+       if (status & RxOverflow)        /* Missed a Rx frame. */
+               ep->stats.rx_errors++;
+       if (status & (RxOverflow | RxFull))
+               outw(RxQueued, ioaddr + COMMAND);
+}
+
+static int epic_poll(struct net_device *dev, int *budget)
+{
+       struct epic_private *ep = dev->priv;
+       int work_done = 0, orig_budget;
+       long ioaddr = dev->base_addr;
+
+       orig_budget = (*budget > dev->quota) ? dev->quota : *budget;
+
+rx_action:
+
+       epic_tx(dev, ep);
+
+       work_done += epic_rx(dev, *budget);
+
+       epic_rx_err(dev, ep);
+
+       *budget -= work_done;
+       dev->quota -= work_done;
+
+       if (netif_running(dev) && (work_done < orig_budget)) {
+               unsigned long flags;
+               int more;
+
+               /* A bit baroque but it avoids a (space hungry) spin_unlock */
+
+               spin_lock_irqsave(&ep->napi_lock, flags);
+
+               more = ep->reschedule_in_poll;
+               if (!more) {
+                       __netif_rx_complete(dev);
+                       outl(EpicNapiEvent, ioaddr + INTSTAT);
+                       epic_napi_irq_on(dev, ep);
+               } else
+                       ep->reschedule_in_poll--;
+
+               spin_unlock_irqrestore(&ep->napi_lock, flags);
+
+               if (more)
+                       goto rx_action;
+       }
+
+       return (work_done >= orig_budget);
+}
+
 static int epic_close(struct net_device *dev)
 {
        long ioaddr = dev->base_addr;
@@ -1268,9 +1320,13 @@ static int epic_close(struct net_device *dev)
                           dev->name, (int)inl(ioaddr + INTSTAT));
 
        del_timer_sync(&ep->timer);
-       epic_pause(dev);
+
+       epic_disable_int(dev, ep);
+
        free_irq(dev->irq, dev);
 
+       epic_pause(dev);
+
        /* Free all the skbuffs in the Rx queue. */
        for (i = 0; i < RX_RING_SIZE; i++) {
                skb = ep->rx_skbuff[i];
@@ -1278,7 +1334,7 @@ static int epic_close(struct net_device *dev)
                ep->rx_ring[i].rxstatus = 0;            /* Not owned by Epic chip. */
                ep->rx_ring[i].buflength = 0;
                if (skb) {
-                       pci_unmap_single(ep->pci_dev, ep->rx_ring[i].bufaddr, 
+                       pci_unmap_single(ep->pci_dev, ep->rx_ring[i].bufaddr,
                                         ep->rx_buf_sz, PCI_DMA_FROMDEVICE);
                        dev_kfree_skb(skb);
                }
@@ -1289,7 +1345,7 @@ static int epic_close(struct net_device *dev)
                ep->tx_skbuff[i] = NULL;
                if (!skb)
                        continue;
-               pci_unmap_single(ep->pci_dev, ep->tx_ring[i].bufaddr, 
+               pci_unmap_single(ep->pci_dev, ep->tx_ring[i].bufaddr,
                                 skb->len, PCI_DMA_TODEVICE);
                dev_kfree_skb(skb);
        }
@@ -1330,7 +1386,6 @@ static void set_rx_mode(struct net_device *dev)
        if (dev->flags & IFF_PROMISC) {                 /* Set promiscuous. */
                outl(0x002C, ioaddr + RxCtrl);
                /* Unconditionally log net taps. */
-               printk(KERN_INFO "%s: Promiscuous mode enabled.\n", dev->name);
                memset(mc_filter, 0xff, sizeof(mc_filter));
        } else if ((dev->mc_count > 0)  ||  (dev->flags & IFF_ALLMULTI)) {
                /* There is apparently a chip bug, so the multicast filter
@@ -1437,7 +1492,7 @@ static void ethtool_complete(struct net_device *dev)
        }
 }
 
-static struct ethtool_ops netdev_ethtool_ops = {
+static const struct ethtool_ops netdev_ethtool_ops = {
        .get_drvinfo            = netdev_get_drvinfo,
        .get_settings           = netdev_get_settings,
        .set_settings           = netdev_set_settings,
@@ -1482,7 +1537,7 @@ static void __devexit epic_remove_one (struct pci_dev *pdev)
 {
        struct net_device *dev = pci_get_drvdata(pdev);
        struct epic_private *ep = dev->priv;
-       
+
        pci_free_consistent(pdev, TX_TOTAL_SIZE, ep->tx_ring, ep->tx_ring_dma);
        pci_free_consistent(pdev, RX_TOTAL_SIZE, ep->rx_ring, ep->rx_ring_dma);
        unregister_netdev(dev);
@@ -1491,6 +1546,7 @@ static void __devexit epic_remove_one (struct pci_dev *pdev)
 #endif
        pci_release_regions(pdev);
        free_netdev(dev);
+       pci_disable_device(pdev);
        pci_set_drvdata(pdev, NULL);
        /* pci_power_off(pdev, -1); */
 }
@@ -1498,7 +1554,7 @@ static void __devexit epic_remove_one (struct pci_dev *pdev)
 
 #ifdef CONFIG_PM
 
-static int epic_suspend (struct pci_dev *pdev, u32 state)
+static int epic_suspend (struct pci_dev *pdev, pm_message_t state)
 {
        struct net_device *dev = pci_get_drvdata(pdev);
        long ioaddr = dev->base_addr;
@@ -1547,7 +1603,7 @@ static int __init epic_init (void)
                version, version2, version3);
 #endif
 
-       return pci_module_init (&epic_driver);
+       return pci_register_driver(&epic_driver);
 }