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

Acked-by: Arthur Kiyanovski <akiyano@amazon.com>
Signed-off-by: Alexander Duyck <alexanderduyck@fb.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/amazon/ena/ena_ethtool.c

index d6cc7aa..2fe7cce 100644 (file)
@@ -251,10 +251,10 @@ static void ena_queue_strings(struct ena_adapter *adapter, u8 **data)
                for (j = 0; j < ENA_STATS_ARRAY_TX; j++) {
                        ena_stats = &ena_stats_tx_strings[j];
 
-                       snprintf(*data, ETH_GSTRING_LEN,
-                                "queue_%u_%s_%s", i,
-                                is_xdp ? "xdp_tx" : "tx", ena_stats->name);
-                       (*data) += ETH_GSTRING_LEN;
+                       ethtool_sprintf(data,
+                                       "queue_%u_%s_%s", i,
+                                       is_xdp ? "xdp_tx" : "tx",
+                                       ena_stats->name);
                }
 
                if (!is_xdp) {
@@ -264,9 +264,9 @@ static void ena_queue_strings(struct ena_adapter *adapter, u8 **data)
                        for (j = 0; j < ENA_STATS_ARRAY_RX; j++) {
                                ena_stats = &ena_stats_rx_strings[j];
 
-                               snprintf(*data, ETH_GSTRING_LEN,
-                                        "queue_%u_rx_%s", i, ena_stats->name);
-                               (*data) += ETH_GSTRING_LEN;
+                               ethtool_sprintf(data,
+                                               "queue_%u_rx_%s", i,
+                                               ena_stats->name);
                        }
                }
        }
@@ -280,9 +280,8 @@ static void ena_com_dev_strings(u8 **data)
        for (i = 0; i < ENA_STATS_ARRAY_ENA_COM; i++) {
                ena_stats = &ena_stats_ena_com_strings[i];
 
-               snprintf(*data, ETH_GSTRING_LEN,
-                        "ena_admin_q_%s", ena_stats->name);
-               (*data) += ETH_GSTRING_LEN;
+               ethtool_sprintf(data,
+                               "ena_admin_q_%s", ena_stats->name);
        }
 }
 
@@ -295,15 +294,13 @@ static void ena_get_strings(struct ena_adapter *adapter,
 
        for (i = 0; i < ENA_STATS_ARRAY_GLOBAL; i++) {
                ena_stats = &ena_stats_global_strings[i];
-               memcpy(data, ena_stats->name, ETH_GSTRING_LEN);
-               data += ETH_GSTRING_LEN;
+               ethtool_sprintf(&data, ena_stats->name);
        }
 
        if (eni_stats_needed) {
                for (i = 0; i < ENA_STATS_ARRAY_ENI(adapter); i++) {
                        ena_stats = &ena_stats_eni_strings[i];
-                       memcpy(data, ena_stats->name, ETH_GSTRING_LEN);
-                       data += ETH_GSTRING_LEN;
+                       ethtool_sprintf(&data, ena_stats->name);
                }
        }