[NET] drivers/net: statistics cleanup #1 -- save memory and shrink code

[safe/jmp/linux-2.6] / drivers / net / znet.c

diff --git a/drivers/net/znet.c b/drivers/net/znet.c
index dcd4e1b..43712c7 100644 (file)
--- a/drivers/net/znet.c
+++ b/drivers/net/znet.c
@@ -128,7 +128,6 @@ MODULE_LICENSE("GPL");
 
 struct znet_private {
        int rx_dma, tx_dma;
-       struct net_device_stats stats;
        spinlock_t lock;
        short sia_base, sia_size, io_size;
        struct i82593_conf_block i593_init;
@@ -161,7 +160,6 @@ static int  znet_send_packet(struct sk_buff *skb, struct net_device *dev);
 static irqreturn_t znet_interrupt(int irq, void *dev_id);
 static void    znet_rx(struct net_device *dev);
 static int     znet_close(struct net_device *dev);
-static struct net_device_stats *net_get_stats(struct net_device *dev);
 static void hardware_init(struct net_device *dev);
 static void update_stop_hit(short ioaddr, unsigned short rx_stop_offset);
 static void znet_tx_timeout (struct net_device *dev);
@@ -445,7 +443,6 @@ static int __init znet_probe (void)
        dev->open = &znet_open;
        dev->hard_start_xmit = &znet_send_packet;
        dev->stop = &znet_close;
-       dev->get_stats  = net_get_stats;
        dev->set_multicast_list = &znet_set_multicast_list;
        dev->tx_timeout = znet_tx_timeout;
        dev->watchdog_timeo = TX_TIMEOUT;
@@ -564,7 +561,7 @@ static int znet_send_packet(struct sk_buff *skb, struct net_device *dev)
                ushort *tx_link = znet->tx_cur - 1;
                ushort rnd_len = (length + 1)>>1;
 
-               znet->stats.tx_bytes+=length;
+               dev->stats.tx_bytes+=length;
 
                if (znet->tx_cur >= znet->tx_end)
                  znet->tx_cur = znet->tx_start;
@@ -639,20 +636,20 @@ static irqreturn_t znet_interrupt(int irq, void *dev_id)
                        tx_status = inw(ioaddr);
                        /* It's undocumented, but tx_status seems to match the i82586. */
                        if (tx_status & TX_OK) {
-                               znet->stats.tx_packets++;
-                               znet->stats.collisions += tx_status & TX_NCOL_MASK;
+                               dev->stats.tx_packets++;
+                               dev->stats.collisions += tx_status & TX_NCOL_MASK;
                        } else {
                                if (tx_status & (TX_LOST_CTS | TX_LOST_CRS))
-                                       znet->stats.tx_carrier_errors++;
+                                       dev->stats.tx_carrier_errors++;
                                if (tx_status & TX_UND_RUN)
-                                       znet->stats.tx_fifo_errors++;
+                                       dev->stats.tx_fifo_errors++;
                                if (!(tx_status & TX_HRT_BEAT))
-                                       znet->stats.tx_heartbeat_errors++;
+                                       dev->stats.tx_heartbeat_errors++;
                                if (tx_status & TX_MAX_COL)
-                                       znet->stats.tx_aborted_errors++;
+                                       dev->stats.tx_aborted_errors++;
                                /* ...and the catch-all. */
                                if ((tx_status | (TX_LOST_CRS | TX_LOST_CTS | TX_UND_RUN | TX_HRT_BEAT | TX_MAX_COL)) != (TX_LOST_CRS | TX_LOST_CTS | TX_UND_RUN | TX_HRT_BEAT | TX_MAX_COL))
-                                       znet->stats.tx_errors++;
+                                       dev->stats.tx_errors++;
 
                                /* Transceiver may be stuck if cable
                                 * was removed while emiting a
@@ -748,19 +745,19 @@ static void znet_rx(struct net_device *dev)
                                 this_rfp_ptr[-3]<<1);
                /* Once again we must assume that the i82586 docs apply. */
                if ( ! (status & RX_RCV_OK)) { /* There was an error. */
-                       znet->stats.rx_errors++;
-                       if (status & RX_CRC_ERR) znet->stats.rx_crc_errors++;
-                       if (status & RX_ALG_ERR) znet->stats.rx_frame_errors++;
+                       dev->stats.rx_errors++;
+                       if (status & RX_CRC_ERR) dev->stats.rx_crc_errors++;
+                       if (status & RX_ALG_ERR) dev->stats.rx_frame_errors++;
 #if 0
-                       if (status & 0x0200) znet->stats.rx_over_errors++; /* Wrong. */
-                       if (status & 0x0100) znet->stats.rx_fifo_errors++;
+                       if (status & 0x0200) dev->stats.rx_over_errors++; /* Wrong. */
+                       if (status & 0x0100) dev->stats.rx_fifo_errors++;
 #else
                        /* maz : Wild guess... */
-                       if (status & RX_OVRRUN) znet->stats.rx_over_errors++;
+                       if (status & RX_OVRRUN) dev->stats.rx_over_errors++;
 #endif
-                       if (status & RX_SRT_FRM) znet->stats.rx_length_errors++;
+                       if (status & RX_SRT_FRM) dev->stats.rx_length_errors++;
                } else if (pkt_len > 1536) {
-                       znet->stats.rx_length_errors++;
+                       dev->stats.rx_length_errors++;
                } else {
                        /* Malloc up new buffer. */
                        struct sk_buff *skb;
@@ -769,7 +766,7 @@ static void znet_rx(struct net_device *dev)
                        if (skb == NULL) {
                                if (znet_debug)
                                  printk(KERN_WARNING "%s: Memory squeeze, dropping packet.\n", dev->name);
-                               znet->stats.rx_dropped++;
+                               dev->stats.rx_dropped++;
                                break;
                        }
 
@@ -789,8 +786,8 @@ static void znet_rx(struct net_device *dev)
                  skb->protocol=eth_type_trans(skb,dev);
                  netif_rx(skb);
                  dev->last_rx = jiffies;
-                 znet->stats.rx_packets++;
-                 znet->stats.rx_bytes += pkt_len;
+                 dev->stats.rx_packets++;
+                 dev->stats.rx_bytes += pkt_len;
                }
                znet->rx_cur = this_rfp_ptr;
                if (znet->rx_cur >= znet->rx_end)
@@ -827,15 +824,6 @@ static int znet_close(struct net_device *dev)
        return 0;
 }
 
-/* Get the current statistics. This may be called with the card open or
-   closed. */
-static struct net_device_stats *net_get_stats(struct net_device *dev)
-{
-       struct znet_private *znet = dev->priv;
-
-       return &znet->stats;
-}
-
 static void show_dma(struct net_device *dev)
 {
        short ioaddr = dev->base_addr;