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

We now have struct net_device_stats embedded in struct net_device,
and the default ->get_stats() hook does the obvious thing for us.

Run through drivers/net/* and remove the driver-local storage of
statistics, and driver-local ->get_stats() hook where applicable.

This was just the low-hanging fruit in drivers/net; plenty more drivers
remain to be updated.

[ Resolved conflicts with napi_struct changes and fix sunqe build
  regression... -DaveM ]

Signed-off-by: Jeff Garzik <jeff@garzik.org>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 19 insertions, 31 deletions

diff --git a/drivers/net/znet.c b/drivers/net/znet.c
index dcd4e1b136b5..43712c7b9ecf 100644
--- 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;