Signed-off-by: Jianming Qiao <kiki-g...@hotmail.com> --- drivers/net/ethernet/broadcom/bcmsysport.c | 81 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++--------------------- drivers/net/ethernet/broadcom/bcmsysport.h | 8 ++++++++ 2 files changed, 68 insertions(+), 21 deletions(-)
diff --git a/drivers/net/ethernet/broadcom/bcmsysport.c b/drivers/net/ethernet/broadcom/bcmsysport.c index 5274501..263512f 100644 --- a/drivers/net/ethernet/broadcom/bcmsysport.c +++ b/drivers/net/ethernet/broadcom/bcmsysport.c @@ -663,6 +663,7 @@ static unsigned int bcm_sysport_desc_rx(struct bcm_sysport_priv *priv, unsigned int budget) { struct net_device *ndev = priv->netdev; + struct bcmsysport_netstats *bsysport_netstats = priv->bsysport_netstats; unsigned int processed = 0, to_process; struct bcm_sysport_cb *cb; struct sk_buff *skb; @@ -763,8 +764,10 @@ static unsigned int bcm_sysport_desc_rx(struct bcm_sysport_priv *priv, } skb->protocol = eth_type_trans(skb, ndev); - ndev->stats.rx_packets++; - ndev->stats.rx_bytes += len; + u64_stats_update_begin(&bsysport_netstats->syncp); + bsysport_netstats->rx_packets++; + bsysport_netstats->rx_bytes += len; + u64_stats_update_end(&bsysport_netstats->syncp); napi_gro_receive(&priv->napi, skb); next: @@ -785,19 +788,26 @@ static void bcm_sysport_tx_reclaim_one(struct bcm_sysport_tx_ring *ring, { struct bcm_sysport_priv *priv = ring->priv; struct device *kdev = &priv->pdev->dev; + struct bcmsysport_netstats *bsysport_netstats = priv->bsysport_netstats; if (cb->skb) { + u64_stats_update_begin(&bsysport_netstats->syncp); ring->bytes += cb->skb->len; + u64_stats_update_end(&bsysport_netstats->syncp); *bytes_compl += cb->skb->len; dma_unmap_single(kdev, dma_unmap_addr(cb, dma_addr), dma_unmap_len(cb, dma_len), DMA_TO_DEVICE); + u64_stats_update_begin(&bsysport_netstats->syncp); ring->packets++; + u64_stats_update_end(&bsysport_netstats->syncp); (*pkts_compl)++; bcm_sysport_free_cb(cb); /* SKB fragment */ } else if (dma_unmap_addr(cb, dma_addr)) { + u64_stats_update_begin(&bsysport_netstats->syncp); ring->bytes += dma_unmap_len(cb, dma_len); + u64_stats_update_end(&bsysport_netstats->syncp); dma_unmap_page(kdev, dma_unmap_addr(cb, dma_addr), dma_unmap_len(cb, dma_len), DMA_TO_DEVICE); dma_unmap_addr_set(cb, dma_addr, 0); @@ -1671,23 +1681,6 @@ static int bcm_sysport_change_mac(struct net_device *dev, void *p) return 0; } -static struct net_device_stats *bcm_sysport_get_nstats(struct net_device *dev) -{ - struct bcm_sysport_priv *priv = netdev_priv(dev); - unsigned long tx_bytes = 0, tx_packets = 0; - struct bcm_sysport_tx_ring *ring; - unsigned int q; - - for (q = 0; q < dev->num_tx_queues; q++) { - ring = &priv->tx_rings[q]; - tx_bytes += ring->bytes; - tx_packets += ring->packets; - } - - dev->stats.tx_bytes = tx_bytes; - dev->stats.tx_packets = tx_packets; - return &dev->stats; -} static void bcm_sysport_netif_start(struct net_device *dev) { @@ -1923,6 +1916,49 @@ static int bcm_sysport_stop(struct net_device *dev) return 0; } +static int bcm_sysport_init(struct net_device *dev) +{ + struct bcm_sysport_priv *priv = netdev_priv(dev); + + priv->bsysport_netstats = kzalloc(sizeof(*priv->bsysport_netstats), + GFP_KERNEL); + if (!priv->bsysport_netstats) + return -ENOMEM; + + return 0; +} + +static void bcm_sysport_get_stats64(struct net_device *dev, + struct rtnl_link_stats64 *stats) +{ + struct bcm_sysport_priv *priv = netdev_priv(dev); + struct bcmsysport_netstats *bsysport_netstats = priv->bsysport_netstats; + struct bcm_sysport_tx_ring *ring; + unsigned int q; + u64 tx_packets = 0, tx_bytes = 0; + unsigned int start; + + netdev_stats_to_stats64(stats, &dev->stats); + + for (q = 0; q < dev->num_tx_queues; q++) { + ring = &priv->tx_rings[q]; + do { + start = u64_stats_fetch_begin_irq(&bsysport_netstats->syncp); + tx_bytes += ring->bytes; + tx_packets += ring->packets; + } while (u64_stats_fetch_retry_irq(&bsysport_netstats->syncp, start)); + } + + stats->tx_packets = tx_packets; + stats->tx_bytes = tx_bytes; + + do { + start = u64_stats_fetch_begin_irq(&bsysport_netstats->syncp); + stats->rx_packets = bsysport_netstats->rx_packets; + stats->rx_bytes = bsysport_netstats->rx_bytes; + } while (u64_stats_fetch_retry_irq(&bsysport_netstats->syncp, start)); +} + static const struct ethtool_ops bcm_sysport_ethtool_ops = { .get_drvinfo = bcm_sysport_get_drvinfo, .get_msglevel = bcm_sysport_get_msglvl, @@ -1942,6 +1978,7 @@ static int bcm_sysport_stop(struct net_device *dev) static const struct net_device_ops bcm_sysport_netdev_ops = { .ndo_start_xmit = bcm_sysport_xmit, .ndo_tx_timeout = bcm_sysport_tx_timeout, + .ndo_init = bcm_sysport_init, .ndo_open = bcm_sysport_open, .ndo_stop = bcm_sysport_stop, .ndo_set_features = bcm_sysport_set_features, @@ -1950,7 +1987,7 @@ static int bcm_sysport_stop(struct net_device *dev) #ifdef CONFIG_NET_POLL_CONTROLLER .ndo_poll_controller = bcm_sysport_poll_controller, #endif - .ndo_get_stats = bcm_sysport_get_nstats, + .ndo_get_stats64 = bcm_sysport_get_stats64, }; #define REV_FMT "v%2x.%02x" @@ -2126,10 +2163,12 @@ static int bcm_sysport_remove(struct platform_device *pdev) { struct net_device *dev = dev_get_drvdata(&pdev->dev); struct device_node *dn = pdev->dev.of_node; - + struct bcm_sysport_priv *priv = netdev_priv(dev); /* Not much to do, ndo_close has been called * and we use managed allocations */ + kfree(priv->bsysport_netstats); + priv->bsysport_netstats = NULL; unregister_netdev(dev); if (of_phy_is_fixed_link(dn)) of_phy_deregister_fixed_link(dn); diff --git a/drivers/net/ethernet/broadcom/bcmsysport.h b/drivers/net/ethernet/broadcom/bcmsysport.h index 77a51c1..29a7442 100644 --- a/drivers/net/ethernet/broadcom/bcmsysport.h +++ b/drivers/net/ethernet/broadcom/bcmsysport.h @@ -697,6 +697,12 @@ struct bcm_sysport_tx_ring { unsigned long bytes; /* bytes statistics */ }; +struct bcmsysport_netstats { + u64 rx_packets; + u64 rx_bytes; + struct u64_stats_sync syncp; +}; + /* Driver private structure */ struct bcm_sysport_priv { void __iomem *base; @@ -743,5 +749,7 @@ struct bcm_sysport_priv { /* Ethtool */ u32 msg_enable; + /* 64bit stats on 32bit Machine */ + struct bcmsysport_netstats *bsysport_netstats; }; #endif /* __BCM_SYSPORT_H */