From patchwork Wed Jan 23 01:40:41 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [net-next] drivers/net/ethernet/micrel/ks8851_mll: Implement basic statistics Date: Tue, 22 Jan 2013 15:40:41 -0000 From: "Choi, David" X-Patchwork-Id: 214708 Message-Id: To: "netdev@vger.kernel.org" Cc: "Doong, Ping" , "Choi, David" From: David J. Choi Summary of changes: add codes to collect basic statistical information about Ethernet packets. Signed-off-by: David J. Choi --- --- -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html --- net-next/drivers/net/ethernet/micrel/ks8851_mll.c.orig 2013-01-22 17:25:59.000000000 -0800 +++ net-next/drivers/net/ethernet/micrel/ks8851_mll.c 2013-01-22 17:27:57.000000000 -0800 @@ -798,10 +798,17 @@ static void ks_rcv(struct ks_net *ks, st skb_reserve(skb, 2); /* read data block including CRC 4 bytes */ ks_read_qmu(ks, (u16 *)skb->data, frame_hdr->len); - skb_put(skb, frame_hdr->len); + skb_put(skb, frame_hdr->len - 4); /*exclude size of CRC */ skb->protocol = eth_type_trans(skb, netdev); netif_rx(skb); + netdev->stats.rx_packets++; + netdev->stats.rx_bytes += (frame_hdr->len - 4); /*crc field*/ } else { + netdev->stats.rx_dropped++; + if ((frame_hdr->len >= RX_BUF_SIZE) || (frame_hdr->len == 0)) + netdev->stats.rx_length_errors++; + if (frame_hdr->sts & RXFSHR_RXFV) + netdev->stats.rx_frame_errors++; pr_err("%s: err:skb alloc\n", __func__); ks_wrreg16(ks, KS_RXQCR, (ks->rc_rxqcr | RXQCR_RRXEF)); if (skb) @@ -876,6 +883,8 @@ static irqreturn_t ks_irq(int irq, void pmecr &= ~PMECR_WKEVT_MASK; ks_wrreg16(ks, KS_PMECR, pmecr | PMECR_WKEVT_LINK); } + if (unlikely(status & IRQ_RXOI)) + ks->netdev->stats.rx_over_errors++; /* this should be the last in IRQ handler*/ ks_restore_cmd_reg(ks); @@ -1015,6 +1024,8 @@ static int ks_start_xmit(struct sk_buff if (likely(ks_tx_fifo_space(ks) >= skb->len + 12)) { ks_write_qmu(ks, skb->data, skb->len); + netdev->stats.tx_bytes += skb->len; + netdev->stats.tx_packets++; dev_kfree_skb(skb); } else retv = NETDEV_TX_BUSY;