net/mlx5e: Present SW stats when state is not opened
authorEran Ben Elisha <eranbe@mellanox.com>
Tue, 29 May 2018 07:54:47 +0000 (10:54 +0300)
committerSaeed Mahameed <saeedm@mellanox.com>
Fri, 1 Jun 2018 23:48:14 +0000 (16:48 -0700)
The driver can present all SW stats even when the state not opened.
Fixed get strings, count and stats to support it.

In addition, fix tc2txq to hold a static mapping which doesn't depend on
the amount of open channels, and cannot have the same value on two
different cells  while moving between configurations.
Example:
- OOB 16 channels
- Change to 2 channels, 8 TCs
- tc2txq[15][0] == tc2txq[1][7] == 15
This will cause multiple appearances of the same TX index in statistics
output.

Fixes: 76c3810bade3 ("net/mlx5e: Avoid reset netdev stats on configuration changes")
Signed-off-by: Eran Ben Elisha <eranbe@mellanox.com>
Signed-off-by: Saeed Mahameed <saeedm@mellanox.com>
drivers/net/ethernet/mellanox/mlx5/core/en_main.c
drivers/net/ethernet/mellanox/mlx5/core/en_stats.c

index 9b19863b059d2aeb20c4637bd246f70abfc25ed1..adc55de6d4f445da9ca67e93e446fd7d76bc94c7 100644 (file)
@@ -2633,15 +2633,21 @@ static void mlx5e_netdev_set_tcs(struct net_device *netdev)
                netdev_set_tc_queue(netdev, tc, nch, 0);
 }
 
                netdev_set_tc_queue(netdev, tc, nch, 0);
 }
 
-static void mlx5e_build_channels_tx_maps(struct mlx5e_priv *priv)
+static void mlx5e_build_tc2txq_maps(struct mlx5e_priv *priv)
 {
 {
-       struct mlx5e_channel *c;
-       struct mlx5e_txqsq *sq;
+       int max_nch = priv->profile->max_nch(priv->mdev);
        int i, tc;
 
        int i, tc;
 
-       for (i = 0; i < priv->profile->max_nch(priv->mdev); i++)
+       for (i = 0; i < max_nch; i++)
                for (tc = 0; tc < priv->profile->max_tc; tc++)
                for (tc = 0; tc < priv->profile->max_tc; tc++)
-                       priv->channel_tc2txq[i][tc] = i + tc * priv->channels.num;
+                       priv->channel_tc2txq[i][tc] = i + tc * max_nch;
+}
+
+static void mlx5e_build_tx2sq_maps(struct mlx5e_priv *priv)
+{
+       struct mlx5e_channel *c;
+       struct mlx5e_txqsq *sq;
+       int i, tc;
 
        for (i = 0; i < priv->channels.num; i++) {
                c = priv->channels.c[i];
 
        for (i = 0; i < priv->channels.num; i++) {
                c = priv->channels.c[i];
@@ -2661,7 +2667,7 @@ void mlx5e_activate_priv_channels(struct mlx5e_priv *priv)
        netif_set_real_num_tx_queues(netdev, num_txqs);
        netif_set_real_num_rx_queues(netdev, priv->channels.num);
 
        netif_set_real_num_tx_queues(netdev, num_txqs);
        netif_set_real_num_rx_queues(netdev, priv->channels.num);
 
-       mlx5e_build_channels_tx_maps(priv);
+       mlx5e_build_tx2sq_maps(priv);
        mlx5e_activate_channels(&priv->channels);
        write_lock(&priv->stats_lock);
        priv->channels_active = true;
        mlx5e_activate_channels(&priv->channels);
        write_lock(&priv->stats_lock);
        priv->channels_active = true;
@@ -4446,6 +4452,7 @@ static void mlx5e_nic_init(struct mlx5_core_dev *mdev,
        if (err)
                mlx5_core_err(mdev, "TLS initialization failed, %d\n", err);
        mlx5e_build_nic_netdev(netdev);
        if (err)
                mlx5_core_err(mdev, "TLS initialization failed, %d\n", err);
        mlx5e_build_nic_netdev(netdev);
+       mlx5e_build_tc2txq_maps(priv);
        mlx5e_vxlan_init(priv);
 }
 
        mlx5e_vxlan_init(priv);
 }
 
index 776b4d68e15601369eb36ae6cb364f3c6e21a7da..3b2aed43f660f3626d36cf51ddcde0e51aa30f75 100644 (file)
@@ -1161,9 +1161,6 @@ static int mlx5e_grp_channels_get_num_stats(struct mlx5e_priv *priv)
 {
        int max_nch = priv->profile->max_nch(priv->mdev);
 
 {
        int max_nch = priv->profile->max_nch(priv->mdev);
 
-       if (!test_bit(MLX5E_STATE_OPENED, &priv->state))
-               return 0;
-
        return (NUM_RQ_STATS * max_nch) +
               (NUM_CH_STATS * max_nch) +
               (NUM_SQ_STATS * max_nch * priv->max_opened_tc);
        return (NUM_RQ_STATS * max_nch) +
               (NUM_CH_STATS * max_nch) +
               (NUM_SQ_STATS * max_nch * priv->max_opened_tc);
@@ -1175,9 +1172,6 @@ static int mlx5e_grp_channels_fill_strings(struct mlx5e_priv *priv, u8 *data,
        int max_nch = priv->profile->max_nch(priv->mdev);
        int i, j, tc;
 
        int max_nch = priv->profile->max_nch(priv->mdev);
        int i, j, tc;
 
-       if (!test_bit(MLX5E_STATE_OPENED, &priv->state))
-               return idx;
-
        for (i = 0; i < max_nch; i++)
                for (j = 0; j < NUM_CH_STATS; j++)
                        sprintf(data + (idx++) * ETH_GSTRING_LEN,
        for (i = 0; i < max_nch; i++)
                for (j = 0; j < NUM_CH_STATS; j++)
                        sprintf(data + (idx++) * ETH_GSTRING_LEN,
@@ -1187,7 +1181,6 @@ static int mlx5e_grp_channels_fill_strings(struct mlx5e_priv *priv, u8 *data,
                for (j = 0; j < NUM_RQ_STATS; j++)
                        sprintf(data + (idx++) * ETH_GSTRING_LEN, rq_stats_desc[j].format, i);
 
                for (j = 0; j < NUM_RQ_STATS; j++)
                        sprintf(data + (idx++) * ETH_GSTRING_LEN, rq_stats_desc[j].format, i);
 
-       /* priv->channel_tc2txq[i][tc] is valid only when device is open */
        for (tc = 0; tc < priv->max_opened_tc; tc++)
                for (i = 0; i < max_nch; i++)
                        for (j = 0; j < NUM_SQ_STATS; j++)
        for (tc = 0; tc < priv->max_opened_tc; tc++)
                for (i = 0; i < max_nch; i++)
                        for (j = 0; j < NUM_SQ_STATS; j++)
@@ -1204,9 +1197,6 @@ static int mlx5e_grp_channels_fill_stats(struct mlx5e_priv *priv, u64 *data,
        int max_nch = priv->profile->max_nch(priv->mdev);
        int i, j, tc;
 
        int max_nch = priv->profile->max_nch(priv->mdev);
        int i, j, tc;
 
-       if (!test_bit(MLX5E_STATE_OPENED, &priv->state))
-               return idx;
-
        for (i = 0; i < max_nch; i++)
                for (j = 0; j < NUM_CH_STATS; j++)
                        data[idx++] =
        for (i = 0; i < max_nch; i++)
                for (j = 0; j < NUM_CH_STATS; j++)
                        data[idx++] =