net: macsec: Use helper functions to update stats
authorLi Zetao <lizetao1@huawei.com>
Thu, 10 Aug 2023 08:56:41 +0000 (16:56 +0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 14 Aug 2023 07:06:24 +0000 (08:06 +0100)
Use the helper functions dev_sw_netstats_rx_add() and
dev_sw_netstats_tx_add() to update stats, which helps to
provide code readability.

Signed-off-by: Li Zetao <lizetao1@huawei.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/macsec.c

index 144ec75..ae60817 100644 (file)
@@ -518,14 +518,8 @@ static void macsec_count_tx(struct sk_buff *skb, struct macsec_tx_sc *tx_sc,
 
 static void count_tx(struct net_device *dev, int ret, int len)
 {
-       if (likely(ret == NET_XMIT_SUCCESS || ret == NET_XMIT_CN)) {
-               struct pcpu_sw_netstats *stats = this_cpu_ptr(dev->tstats);
-
-               u64_stats_update_begin(&stats->syncp);
-               u64_stats_inc(&stats->tx_packets);
-               u64_stats_add(&stats->tx_bytes, len);
-               u64_stats_update_end(&stats->syncp);
-       }
+       if (likely(ret == NET_XMIT_SUCCESS || ret == NET_XMIT_CN))
+               dev_sw_netstats_tx_add(dev, 1, len);
 }
 
 static void macsec_encrypt_done(void *data, int err)
@@ -827,12 +821,7 @@ static void macsec_finalize_skb(struct sk_buff *skb, u8 icv_len, u8 hdr_len)
 
 static void count_rx(struct net_device *dev, int len)
 {
-       struct pcpu_sw_netstats *stats = this_cpu_ptr(dev->tstats);
-
-       u64_stats_update_begin(&stats->syncp);
-       u64_stats_inc(&stats->rx_packets);
-       u64_stats_add(&stats->rx_bytes, len);
-       u64_stats_update_end(&stats->syncp);
+       dev_sw_netstats_rx_add(dev, len);
 }
 
 static void macsec_decrypt_done(void *data, int err)