]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
iavf: Fix displaying queue statistics shown by ethtool
authorJedrzej Jagielski <jedrzej.jagielski@intel.com>
Fri, 17 Sep 2021 08:52:52 +0000 (08:52 +0000)
committerTony Nguyen <anthony.l.nguyen@intel.com>
Tue, 30 Nov 2021 16:56:07 +0000 (08:56 -0800)
Driver provided too many lines as an output to ethtool -S command.
Return actual length of string set of ethtool stats. Instead of predefined
maximal value use the actual value on netdev, iterate over active queues.
Without this patch, ethtool -S report would produce additional
erroneous lines of queues that are not configured.

Signed-off-by: Witold Fijalkowski <witoldx.fijalkowski@intel.com>
Signed-off-by: Przemyslaw Patynowski <przemyslawx.patynowski@intel.com>
Signed-off-by: Mateusz Palczewski <mateusz.palczewski@intel.com>
Signed-off-by: Jedrzej Jagielski <jedrzej.jagielski@intel.com>
Tested-by: Konrad Jankowski <konrad0.jankowski@intel.com>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
drivers/net/ethernet/intel/iavf/iavf_ethtool.c

index f0e8b5adfecc735128669ff624ed87ca60aa1278..1d01d6be8a863036c70ec1a2ac5a32d0d630d603 100644 (file)
@@ -331,9 +331,16 @@ static int iavf_get_link_ksettings(struct net_device *netdev,
  **/
 static int iavf_get_sset_count(struct net_device *netdev, int sset)
 {
+       /* Report the maximum number queues, even if not every queue is
+        * currently configured. Since allocation of queues is in pairs,
+        * use netdev->real_num_tx_queues * 2. The real_num_tx_queues is set
+        * at device creation and never changes.
+        */
+
        if (sset == ETH_SS_STATS)
                return IAVF_STATS_LEN +
-                       (IAVF_QUEUE_STATS_LEN * 2 * IAVF_MAX_REQ_QUEUES);
+                       (IAVF_QUEUE_STATS_LEN * 2 *
+                        netdev->real_num_tx_queues);
        else if (sset == ETH_SS_PRIV_FLAGS)
                return IAVF_PRIV_FLAGS_STR_LEN;
        else
@@ -360,17 +367,18 @@ static void iavf_get_ethtool_stats(struct net_device *netdev,
        iavf_add_ethtool_stats(&data, adapter, iavf_gstrings_stats);
 
        rcu_read_lock();
-       for (i = 0; i < IAVF_MAX_REQ_QUEUES; i++) {
+       /* As num_active_queues describe both tx and rx queues, we can use
+        * it to iterate over rings' stats.
+        */
+       for (i = 0; i < adapter->num_active_queues; i++) {
                struct iavf_ring *ring;
 
-               /* Avoid accessing un-allocated queues */
-               ring = (i < adapter->num_active_queues ?
-                       &adapter->tx_rings[i] : NULL);
+               /* Tx rings stats */
+               ring = &adapter->tx_rings[i];
                iavf_add_queue_stats(&data, ring);
 
-               /* Avoid accessing un-allocated queues */
-               ring = (i < adapter->num_active_queues ?
-                       &adapter->rx_rings[i] : NULL);
+               /* Rx rings stats */
+               ring = &adapter->rx_rings[i];
                iavf_add_queue_stats(&data, ring);
        }
        rcu_read_unlock();
@@ -407,10 +415,10 @@ static void iavf_get_stat_strings(struct net_device *netdev, u8 *data)
 
        iavf_add_stat_strings(&data, iavf_gstrings_stats);
 
-       /* Queues are always allocated in pairs, so we just use num_tx_queues
-        * for both Tx and Rx queues.
+       /* Queues are always allocated in pairs, so we just use
+        * real_num_tx_queues for both Tx and Rx queues.
         */
-       for (i = 0; i < netdev->num_tx_queues; i++) {
+       for (i = 0; i < netdev->real_num_tx_queues; i++) {
                iavf_add_stat_strings(&data, iavf_gstrings_queue_stats,
                                      "tx", i);
                iavf_add_stat_strings(&data, iavf_gstrings_queue_stats,