]> git.baikalelectronics.ru Git - kernel.git/commitdiff
r8169: remove no longer needed private rx/tx packet/byte counters
authorHeiner Kallweit <hkallweit1@gmail.com>
Thu, 29 Oct 2020 17:34:39 +0000 (18:34 +0100)
committerJakub Kicinski <kuba@kernel.org>
Sat, 31 Oct 2020 17:23:02 +0000 (10:23 -0700)
After switching to the net core rx/tx byte/packet counters we can
remove the now unused private version.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/realtek/r8169_main.c

index 0ef30ad8a4bf0a59cf728c41c352edaf448c96c4..b6c11aaa55b1d930167f80d73e861bb7ebfa81f7 100644 (file)
@@ -584,12 +584,6 @@ enum rtl_flag {
        RTL_FLAG_MAX
 };
 
-struct rtl8169_stats {
-       u64                     packets;
-       u64                     bytes;
-       struct u64_stats_sync   syncp;
-};
-
 struct rtl8169_private {
        void __iomem *mmio_addr;        /* memory map physical address */
        struct pci_dev *pci_dev;
@@ -600,8 +594,6 @@ struct rtl8169_private {
        u32 cur_rx; /* Index into the Rx descriptor buffer of next Rx pkt. */
        u32 cur_tx; /* Index into the Tx descriptor buffer of next Rx pkt. */
        u32 dirty_tx;
-       struct rtl8169_stats rx_stats;
-       struct rtl8169_stats tx_stats;
        struct TxDesc *TxDescArray;     /* 256-aligned Tx descriptor ring */
        struct RxDesc *RxDescArray;     /* 256-aligned Rx descriptor ring */
        dma_addr_t TxPhyAddr;
@@ -700,27 +692,6 @@ static bool rtl_supports_eee(struct rtl8169_private *tp)
               tp->mac_version != RTL_GIGA_MAC_VER_39;
 }
 
-static void rtl_get_priv_stats(struct rtl8169_stats *stats,
-                              u64 *pkts, u64 *bytes)
-{
-       unsigned int start;
-
-       do {
-               start = u64_stats_fetch_begin_irq(&stats->syncp);
-               *pkts = stats->packets;
-               *bytes = stats->bytes;
-       } while (u64_stats_fetch_retry_irq(&stats->syncp, start));
-}
-
-static void rtl_inc_priv_stats(struct rtl8169_stats *stats,
-                              u64 pkts, u64 bytes)
-{
-       u64_stats_update_begin(&stats->syncp);
-       stats->packets += pkts;
-       stats->bytes += bytes;
-       u64_stats_update_end(&stats->syncp);
-}
-
 static void rtl_read_mac_from_reg(struct rtl8169_private *tp, u8 *mac, int reg)
 {
        int i;
@@ -4416,9 +4387,7 @@ static void rtl_tx(struct net_device *dev, struct rtl8169_private *tp,
 
        if (tp->dirty_tx != dirty_tx) {
                netdev_completed_queue(dev, pkts_compl, bytes_compl);
-
                dev_sw_netstats_tx_add(dev, pkts_compl, bytes_compl);
-               rtl_inc_priv_stats(&tp->tx_stats, pkts_compl, bytes_compl);
 
                tp->dirty_tx = dirty_tx;
                /* Sync with rtl8169_start_xmit:
@@ -4541,7 +4510,6 @@ static int rtl_rx(struct net_device *dev, struct rtl8169_private *tp, u32 budget
                napi_gro_receive(&tp->napi, skb);
 
                dev_sw_netstats_rx_add(dev, pkt_size);
-               rtl_inc_priv_stats(&tp->rx_stats, 1, pkt_size);
 release_descriptor:
                rtl8169_mark_to_asic(desc);
        }
@@ -5345,8 +5313,6 @@ static int rtl_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
        }
 
        INIT_WORK(&tp->wk.work, rtl_task);
-       u64_stats_init(&tp->rx_stats.syncp);
-       u64_stats_init(&tp->tx_stats.syncp);
 
        rtl_init_mac_address(tp);