netvsc: Update driver to use ethtool_sprintf
authorAlexander Duyck <alexanderduyck@fb.com>
Wed, 17 Mar 2021 00:31:20 +0000 (17:31 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 17 Mar 2021 18:42:31 +0000 (11:42 -0700)
Replace instances of sprintf or memcpy with a pointer update with
ethtool_sprintf.

Signed-off-by: Alexander Duyck <alexanderduyck@fb.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/hyperv/netvsc_drv.c

index 15f262b..97b5c9b 100644 (file)
@@ -1612,34 +1612,23 @@ static void netvsc_get_strings(struct net_device *dev, u32 stringset, u8 *data)
 
        switch (stringset) {
        case ETH_SS_STATS:
-               for (i = 0; i < ARRAY_SIZE(netvsc_stats); i++) {
-                       memcpy(p, netvsc_stats[i].name, ETH_GSTRING_LEN);
-                       p += ETH_GSTRING_LEN;
-               }
+               for (i = 0; i < ARRAY_SIZE(netvsc_stats); i++)
+                       ethtool_sprintf(&p, netvsc_stats[i].name);
 
-               for (i = 0; i < ARRAY_SIZE(vf_stats); i++) {
-                       memcpy(p, vf_stats[i].name, ETH_GSTRING_LEN);
-                       p += ETH_GSTRING_LEN;
-               }
+               for (i = 0; i < ARRAY_SIZE(vf_stats); i++)
+                       ethtool_sprintf(&p, vf_stats[i].name);
 
                for (i = 0; i < nvdev->num_chn; i++) {
-                       sprintf(p, "tx_queue_%u_packets", i);
-                       p += ETH_GSTRING_LEN;
-                       sprintf(p, "tx_queue_%u_bytes", i);
-                       p += ETH_GSTRING_LEN;
-                       sprintf(p, "rx_queue_%u_packets", i);
-                       p += ETH_GSTRING_LEN;
-                       sprintf(p, "rx_queue_%u_bytes", i);
-                       p += ETH_GSTRING_LEN;
-                       sprintf(p, "rx_queue_%u_xdp_drop", i);
-                       p += ETH_GSTRING_LEN;
+                       ethtool_sprintf(&p, "tx_queue_%u_packets", i);
+                       ethtool_sprintf(&p, "tx_queue_%u_bytes", i);
+                       ethtool_sprintf(&p, "rx_queue_%u_packets", i);
+                       ethtool_sprintf(&p, "rx_queue_%u_bytes", i);
+                       ethtool_sprintf(&p, "rx_queue_%u_xdp_drop", i);
                }
 
                for_each_present_cpu(cpu) {
-                       for (i = 0; i < ARRAY_SIZE(pcpu_stats); i++) {
-                               sprintf(p, pcpu_stats[i].name, cpu);
-                               p += ETH_GSTRING_LEN;
-                       }
+                       for (i = 0; i < ARRAY_SIZE(pcpu_stats); i++)
+                               ethtool_sprintf(&p, pcpu_stats[i].name, cpu);
                }
 
                break;