[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, 28 insertions, 49 deletions

diff --git a/drivers/net/smc911x.c b/drivers/net/smc911x.c
index 5f03e44ad135..c0276c04dece 100644
--- a/drivers/net/smc911x.c
+++ b/drivers/net/smc911x.c
@@ -115,13 +115,6 @@ struct smc911x_local {
          */
         struct sk_buff *pending_tx_skb;
 
-        /*
-         * these are things that the kernel wants me to keep, so users
-         * can find out semi-useless statistics of how well the card is
-         * performing
-         */
-        struct net_device_stats stats;
-
         /* version/revision of the SMC911x chip */
         u16 version;
         u16 revision;
@@ -315,8 +308,8 @@ static void smc911x_reset(struct net_device *dev)
         if (lp->pending_tx_skb != NULL) {
                 dev_kfree_skb (lp->pending_tx_skb);
                 lp->pending_tx_skb = NULL;
-                lp->stats.tx_errors++;
-                lp->stats.tx_aborted_errors++;
+                dev->stats.tx_errors++;
+                dev->stats.tx_aborted_errors++;
         }
 }
 
@@ -449,14 +442,14 @@ static inline void	 smc911x_rcv(struct net_device *dev)
         pkt_len = (status & RX_STS_PKT_LEN_) >> 16;
         if (status & RX_STS_ES_) {
                 /* Deal with a bad packet */
-                lp->stats.rx_errors++;
+                dev->stats.rx_errors++;
                 if (status & RX_STS_CRC_ERR_)
-                        lp->stats.rx_crc_errors++;
+                        dev->stats.rx_crc_errors++;
                 else {
                         if (status & RX_STS_LEN_ERR_)
-                                lp->stats.rx_length_errors++;
+                                dev->stats.rx_length_errors++;
                         if (status & RX_STS_MCAST_)
-                                lp->stats.multicast++;
+                                dev->stats.multicast++;
                 }
                 /* Remove the bad packet data from the RX FIFO */
                 smc911x_drop_pkt(dev);
@@ -467,7 +460,7 @@ static inline void	 smc911x_rcv(struct net_device *dev)
                 if (unlikely(skb == NULL)) {
                         PRINTK( "%s: Low memory, rcvd packet dropped.\n",
                                 dev->name);
-                        lp->stats.rx_dropped++;
+                        dev->stats.rx_dropped++;
                         smc911x_drop_pkt(dev);
                         return;
                 }
@@ -503,8 +496,8 @@ static inline void	 smc911x_rcv(struct net_device *dev)
                 dev->last_rx = jiffies;
                 skb->protocol = eth_type_trans(skb, dev);
                 netif_rx(skb);
-                lp->stats.rx_packets++;
-                lp->stats.rx_bytes += pkt_len-4;
+                dev->stats.rx_packets++;
+                dev->stats.rx_bytes += pkt_len-4;
 #endif
         }
 }
@@ -616,8 +609,8 @@ static int smc911x_hard_start_xmit(struct sk_buff *skb, struct net_device *dev)
                 printk("%s: No Tx free space %d < %d\n",
                         dev->name, free, skb->len);
                 lp->pending_tx_skb = NULL;
-                lp->stats.tx_errors++;
-                lp->stats.tx_dropped++;
+                dev->stats.tx_errors++;
+                dev->stats.tx_dropped++;
                 dev_kfree_skb(skb);
                 return 0;
         }
@@ -667,8 +660,8 @@ static void smc911x_tx(struct net_device *dev)
                         dev->name,
                         (SMC_GET_TX_FIFO_INF() & TX_FIFO_INF_TSUSED_) >> 16);
                 tx_status = SMC_GET_TX_STS_FIFO();
-                lp->stats.tx_packets++;
-                lp->stats.tx_bytes+=tx_status>>16;
+                dev->stats.tx_packets++;
+                dev->stats.tx_bytes+=tx_status>>16;
                 DBG(SMC_DEBUG_TX, "%s: Tx FIFO tag 0x%04x status 0x%04x\n",
                         dev->name, (tx_status & 0xffff0000) >> 16,
                         tx_status & 0x0000ffff);
@@ -676,22 +669,22 @@ static void smc911x_tx(struct net_device *dev)
                  * full-duplex mode */
                 if ((tx_status & TX_STS_ES_) && !(lp->ctl_rfduplx &&
                     !(tx_status & 0x00000306))) {
-                        lp->stats.tx_errors++;
+                        dev->stats.tx_errors++;
                 }
                 if (tx_status & TX_STS_MANY_COLL_) {
-                        lp->stats.collisions+=16;
-                        lp->stats.tx_aborted_errors++;
+                        dev->stats.collisions+=16;
+                        dev->stats.tx_aborted_errors++;
                 } else {
-                        lp->stats.collisions+=(tx_status & TX_STS_COLL_CNT_) >> 3;
+                        dev->stats.collisions+=(tx_status & TX_STS_COLL_CNT_) >> 3;
                 }
                 /* carrier error only has meaning for half-duplex communication */
                 if ((tx_status & (TX_STS_LOC_ | TX_STS_NO_CARR_)) &&
                     !lp->ctl_rfduplx) {
-                        lp->stats.tx_carrier_errors++;
+                        dev->stats.tx_carrier_errors++;
                 }
                 if (tx_status & TX_STS_LATE_COLL_) {
-                        lp->stats.collisions++;
-                        lp->stats.tx_aborted_errors++;
+                        dev->stats.collisions++;
+                        dev->stats.tx_aborted_errors++;
                 }
         }
 }
@@ -1121,11 +1114,11 @@ static irqreturn_t smc911x_interrupt(int irq, void *dev_id)
                 /* Handle various error conditions */
                 if (status & INT_STS_RXE_) {
                         SMC_ACK_INT(INT_STS_RXE_);
-                        lp->stats.rx_errors++;
+                        dev->stats.rx_errors++;
                 }
                 if (status & INT_STS_RXDFH_INT_) {
                         SMC_ACK_INT(INT_STS_RXDFH_INT_);
-                        lp->stats.rx_dropped+=SMC_GET_RX_DROP();
+                        dev->stats.rx_dropped+=SMC_GET_RX_DROP();
                  }
                 /* Undocumented interrupt-what is the right thing to do here? */
                 if (status & INT_STS_RXDF_INT_) {
@@ -1140,8 +1133,8 @@ static irqreturn_t smc911x_interrupt(int irq, void *dev_id)
                                 cr &= ~MAC_CR_RXEN_;
                                 SMC_SET_MAC_CR(cr);
                                 DBG(SMC_DEBUG_RX, "%s: RX overrun\n", dev->name);
-                                lp->stats.rx_errors++;
-                                lp->stats.rx_fifo_errors++;
+                                dev->stats.rx_errors++;
+                                dev->stats.rx_fifo_errors++;
                         }
                         SMC_ACK_INT(INT_STS_RDFL_);
                 }
@@ -1152,8 +1145,8 @@ static irqreturn_t smc911x_interrupt(int irq, void *dev_id)
                                 SMC_SET_MAC_CR(cr);
                                 rx_overrun=1;
                                 DBG(SMC_DEBUG_RX, "%s: RX overrun\n", dev->name);
-                                lp->stats.rx_errors++;
-                                lp->stats.rx_fifo_errors++;
+                                dev->stats.rx_errors++;
+                                dev->stats.rx_fifo_errors++;
                         }
                         SMC_ACK_INT(INT_STS_RDFO_);
                 }
@@ -1307,8 +1300,8 @@ smc911x_rx_dma_irq(int dma, void *data)
         dev->last_rx = jiffies;
         skb->protocol = eth_type_trans(skb, dev);
         netif_rx(skb);
-        lp->stats.rx_packets++;
-        lp->stats.rx_bytes += skb->len;
+        dev->stats.rx_packets++;
+        dev->stats.rx_bytes += skb->len;
 
         spin_lock_irqsave(&lp->lock, flags);
         pkts = (SMC_GET_RX_FIFO_INF() & RX_FIFO_INF_RXSUSED_) >> 16;
@@ -1568,19 +1561,6 @@ static int smc911x_close(struct net_device *dev)
 }
 
 /*
- * Get the current statistics.
- * This may be called with the card open or closed.
- */
-static struct net_device_stats *smc911x_query_statistics(struct net_device *dev)
-{
-        struct smc911x_local *lp = netdev_priv(dev);
-        DBG(SMC_DEBUG_FUNC, "%s: --> %s\n", dev->name, __FUNCTION__);
-
-
-        return &lp->stats;
-}
-
-/*
  * Ethtool support
  */
 static int
@@ -2056,7 +2036,6 @@ static int __init smc911x_probe(struct net_device *dev, unsigned long ioaddr)
         dev->hard_start_xmit = smc911x_hard_start_xmit;
         dev->tx_timeout = smc911x_timeout;
         dev->watchdog_timeo = msecs_to_jiffies(watchdog);
-        dev->get_stats = smc911x_query_statistics;
         dev->set_multicast_list = smc911x_set_multicast_list;
         dev->ethtool_ops = &smc911x_ethtool_ops;
 #ifdef CONFIG_NET_POLL_CONTROLLER