From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Choi, David" Subject: RE: [PATCH net-next] drivers/net/ethernet/micrel/ks8851_mll: Implement basic statistics Date: Wed, 23 Jan 2013 19:36:11 +0000 Message-ID: References: <1358968194.2107.37.camel@joe-AO722> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT Cc: "netdev@vger.kernel.org" , "Doong, Ping" , "davem@davemloft.net" , "bhutchings@solarflare.com" To: Joe Perches Return-path: Received: from p01c12o149.mxlogic.net ([208.65.145.72]:49133 "EHLO p01c12o149.mxlogic.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751388Ab3AWTgR convert rfc822-to-8bit (ORCPT ); Wed, 23 Jan 2013 14:36:17 -0500 In-Reply-To: <1358968194.2107.37.camel@joe-AO722> Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: From: David J. Choi Summary of changes: add codes to collect basic statistical information about Ethernet packets. Signed-off-by: David J. Choi --- --- 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-23 11:28:45.000000000 -0800 @@ -793,19 +793,34 @@ static void ks_rcv(struct ks_net *ks, st frame_hdr = ks->frame_head_info; while (ks->frame_cnt--) { skb = netdev_alloc_skb(netdev, frame_hdr->len + 16); - if (likely(skb && (frame_hdr->sts & RXFSHR_RXFV) && + if (unlikely(!skb)) { + /* discard the packet from the device */ + ks_wrreg16(ks, KS_RXQCR, ks->rc_rxqcr | RXQCR_RRXEF); + netdev->stats.rx_dropped++; + } else if (likely((frame_hdr->sts & RXFSHR_RXFV) && (frame_hdr->len < RX_BUF_SIZE) && frame_hdr->len)) { 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); + + /* exclude the size of CRC */ + skb_put(skb, frame_hdr->len - 4); skb->protocol = eth_type_trans(skb, netdev); netif_rx(skb); + netdev->stats.rx_packets++; + + /* crc field */ + netdev->stats.rx_bytes += frame_hdr->len - 4; } else { - pr_err("%s: err:skb alloc\n", __func__); - ks_wrreg16(ks, KS_RXQCR, (ks->rc_rxqcr | RXQCR_RRXEF)); - if (skb) - dev_kfree_skb_irq(skb); + /* discard the packet from the device */ + ks_wrreg16(ks, KS_RXQCR, ks->rc_rxqcr | RXQCR_RRXEF); + netdev->stats.rx_dropped++; + if (frame_hdr->len >= RX_BUF_SIZE || !frame_hdr->len) + netdev->stats.rx_length_errors++; + if (!(frame_hdr->sts & RXFSHR_RXFV)) + netdev->stats.rx_frame_errors++; + + dev_kfree_skb_irq(skb); } frame_hdr++; } @@ -876,6 +891,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 +1032,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; ---