net/mlx5: provide send scheduling error statistics
[dpdk.git] / drivers / net / mlx5 / mlx5_stats.c
index 875dd10..e30542e 100644 (file)
  */
 
 #include <inttypes.h>
-#include <linux/sockios.h>
-#include <linux/ethtool.h>
 #include <stdint.h>
 #include <stdio.h>
+#include <unistd.h>
 
 #include <rte_ethdev_driver.h>
 #include <rte_common.h>
 #include <rte_malloc.h>
 
+#include <mlx5_common.h>
+
+#include "mlx5_defs.h"
 #include "mlx5.h"
 #include "mlx5_rxtx.h"
-#include "mlx5_defs.h"
-
-struct mlx5_counter_ctrl {
-       /* Name of the counter. */
-       char dpdk_name[RTE_ETH_XSTATS_NAME_SIZE];
-       /* Name of the counter on the device table. */
-       char ctr_name[RTE_ETH_XSTATS_NAME_SIZE];
-       uint32_t ib:1; /**< Nonzero for IB counters. */
-};
-
-static const struct mlx5_counter_ctrl mlx5_counters_init[] = {
-       {
-               .dpdk_name = "rx_port_unicast_bytes",
-               .ctr_name = "rx_vport_unicast_bytes",
-       },
-       {
-               .dpdk_name = "rx_port_multicast_bytes",
-               .ctr_name = "rx_vport_multicast_bytes",
-       },
-       {
-               .dpdk_name = "rx_port_broadcast_bytes",
-               .ctr_name = "rx_vport_broadcast_bytes",
-       },
-       {
-               .dpdk_name = "rx_port_unicast_packets",
-               .ctr_name = "rx_vport_unicast_packets",
-       },
-       {
-               .dpdk_name = "rx_port_multicast_packets",
-               .ctr_name = "rx_vport_multicast_packets",
-       },
-       {
-               .dpdk_name = "rx_port_broadcast_packets",
-               .ctr_name = "rx_vport_broadcast_packets",
-       },
-       {
-               .dpdk_name = "tx_port_unicast_bytes",
-               .ctr_name = "tx_vport_unicast_bytes",
-       },
-       {
-               .dpdk_name = "tx_port_multicast_bytes",
-               .ctr_name = "tx_vport_multicast_bytes",
-       },
-       {
-               .dpdk_name = "tx_port_broadcast_bytes",
-               .ctr_name = "tx_vport_broadcast_bytes",
-       },
-       {
-               .dpdk_name = "tx_port_unicast_packets",
-               .ctr_name = "tx_vport_unicast_packets",
-       },
-       {
-               .dpdk_name = "tx_port_multicast_packets",
-               .ctr_name = "tx_vport_multicast_packets",
-       },
-       {
-               .dpdk_name = "tx_port_broadcast_packets",
-               .ctr_name = "tx_vport_broadcast_packets",
-       },
-       {
-               .dpdk_name = "rx_wqe_err",
-               .ctr_name = "rx_wqe_err",
-       },
-       {
-               .dpdk_name = "rx_crc_errors_phy",
-               .ctr_name = "rx_crc_errors_phy",
-       },
-       {
-               .dpdk_name = "rx_in_range_len_errors_phy",
-               .ctr_name = "rx_in_range_len_errors_phy",
-       },
-       {
-               .dpdk_name = "rx_symbol_err_phy",
-               .ctr_name = "rx_symbol_err_phy",
-       },
-       {
-               .dpdk_name = "tx_errors_phy",
-               .ctr_name = "tx_errors_phy",
-       },
-       {
-               .dpdk_name = "rx_out_of_buffer",
-               .ctr_name = "out_of_buffer",
-               .ib = 1,
-       },
-       {
-               .dpdk_name = "tx_packets_phy",
-               .ctr_name = "tx_packets_phy",
-       },
-       {
-               .dpdk_name = "rx_packets_phy",
-               .ctr_name = "rx_packets_phy",
-       },
-       {
-               .dpdk_name = "tx_bytes_phy",
-               .ctr_name = "tx_bytes_phy",
-       },
-       {
-               .dpdk_name = "rx_bytes_phy",
-               .ctr_name = "rx_bytes_phy",
-       },
-};
-
-static const unsigned int xstats_n = RTE_DIM(mlx5_counters_init);
-
-/**
- * Read device counters table.
- *
- * @param dev
- *   Pointer to Ethernet device.
- * @param[out] stats
- *   Counters table output buffer.
- *
- * @return
- *   0 on success and stats is filled, negative errno value otherwise and
- *   rte_errno is set.
- */
-static int
-mlx5_read_dev_counters(struct rte_eth_dev *dev, uint64_t *stats)
-{
-       struct priv *priv = dev->data->dev_private;
-       struct mlx5_xstats_ctrl *xstats_ctrl = &priv->xstats_ctrl;
-       unsigned int i;
-       struct ifreq ifr;
-       unsigned int stats_sz = xstats_ctrl->stats_n * sizeof(uint64_t);
-       unsigned char et_stat_buf[sizeof(struct ethtool_stats) + stats_sz];
-       struct ethtool_stats *et_stats = (struct ethtool_stats *)et_stat_buf;
-       int ret;
-
-       et_stats->cmd = ETHTOOL_GSTATS;
-       et_stats->n_stats = xstats_ctrl->stats_n;
-       ifr.ifr_data = (caddr_t)et_stats;
-       ret = mlx5_ifreq(dev, SIOCETHTOOL, &ifr);
-       if (ret) {
-               DRV_LOG(WARNING,
-                       "port %u unable to read statistic values from device",
-                       dev->data->port_id);
-               return ret;
-       }
-       for (i = 0; i != xstats_n; ++i) {
-               if (mlx5_counters_init[i].ib) {
-                       FILE *file;
-                       MKSTR(path, "%s/ports/1/hw_counters/%s",
-                             priv->ibdev_path,
-                             mlx5_counters_init[i].ctr_name);
-
-                       file = fopen(path, "rb");
-                       if (file) {
-                               int n = fscanf(file, "%" SCNu64, &stats[i]);
-
-                               fclose(file);
-                               if (n != 1)
-                                       stats[i] = 0;
-                       }
-               } else {
-                       stats[i] = (uint64_t)
-                               et_stats->data[xstats_ctrl->dev_table_idx[i]];
-               }
-       }
-       return 0;
-}
-
-/**
- * Query the number of statistics provided by ETHTOOL.
- *
- * @param dev
- *   Pointer to Ethernet device.
- *
- * @return
- *   Number of statistics on success, negative errno value otherwise and
- *   rte_errno is set.
- */
-static int
-mlx5_ethtool_get_stats_n(struct rte_eth_dev *dev) {
-       struct ethtool_drvinfo drvinfo;
-       struct ifreq ifr;
-       int ret;
-
-       drvinfo.cmd = ETHTOOL_GDRVINFO;
-       ifr.ifr_data = (caddr_t)&drvinfo;
-       ret = mlx5_ifreq(dev, SIOCETHTOOL, &ifr);
-       if (ret) {
-               DRV_LOG(WARNING, "port %u unable to query number of statistics",
-                       dev->data->port_id);
-               return ret;
-       }
-       return drvinfo.n_stats;
-}
-
-/**
- * Init the structures to read device counters.
- *
- * @param dev
- *   Pointer to Ethernet device.
- */
-void
-mlx5_xstats_init(struct rte_eth_dev *dev)
-{
-       struct priv *priv = dev->data->dev_private;
-       struct mlx5_xstats_ctrl *xstats_ctrl = &priv->xstats_ctrl;
-       unsigned int i;
-       unsigned int j;
-       struct ifreq ifr;
-       struct ethtool_gstrings *strings = NULL;
-       unsigned int dev_stats_n;
-       unsigned int str_sz;
-       int ret;
-
-       ret = mlx5_ethtool_get_stats_n(dev);
-       if (ret < 0) {
-               DRV_LOG(WARNING, "port %u no extended statistics available",
-                       dev->data->port_id);
-               return;
-       }
-       dev_stats_n = ret;
-       xstats_ctrl->stats_n = dev_stats_n;
-       /* Allocate memory to grab stat names and values. */
-       str_sz = dev_stats_n * ETH_GSTRING_LEN;
-       strings = (struct ethtool_gstrings *)
-                 rte_malloc("xstats_strings",
-                            str_sz + sizeof(struct ethtool_gstrings), 0);
-       if (!strings) {
-               DRV_LOG(WARNING, "port %u unable to allocate memory for xstats",
-                    dev->data->port_id);
-               return;
-       }
-       strings->cmd = ETHTOOL_GSTRINGS;
-       strings->string_set = ETH_SS_STATS;
-       strings->len = dev_stats_n;
-       ifr.ifr_data = (caddr_t)strings;
-       ret = mlx5_ifreq(dev, SIOCETHTOOL, &ifr);
-       if (ret) {
-               DRV_LOG(WARNING, "port %u unable to get statistic names",
-                       dev->data->port_id);
-               goto free;
-       }
-       for (j = 0; j != xstats_n; ++j)
-               xstats_ctrl->dev_table_idx[j] = dev_stats_n;
-       for (i = 0; i != dev_stats_n; ++i) {
-               const char *curr_string = (const char *)
-                       &strings->data[i * ETH_GSTRING_LEN];
-
-               for (j = 0; j != xstats_n; ++j) {
-                       if (!strcmp(mlx5_counters_init[j].ctr_name,
-                                   curr_string)) {
-                               xstats_ctrl->dev_table_idx[j] = i;
-                               break;
-                       }
-               }
-       }
-       for (j = 0; j != xstats_n; ++j) {
-               if (mlx5_counters_init[j].ib)
-                       continue;
-               if (xstats_ctrl->dev_table_idx[j] >= dev_stats_n) {
-                       DRV_LOG(WARNING,
-                               "port %u counter \"%s\" is not recognized",
-                               dev->data->port_id,
-                               mlx5_counters_init[j].dpdk_name);
-                       goto free;
-               }
-       }
-       /* Copy to base at first time. */
-       assert(xstats_n <= MLX5_MAX_XSTATS);
-       ret = mlx5_read_dev_counters(dev, xstats_ctrl->base);
-       if (ret)
-               DRV_LOG(ERR, "port %u cannot read device counters: %s",
-                       dev->data->port_id, strerror(rte_errno));
-free:
-       rte_free(strings);
-}
 
 /**
  * DPDK callback to get extended device statistics.
@@ -303,29 +36,47 @@ int
 mlx5_xstats_get(struct rte_eth_dev *dev, struct rte_eth_xstat *stats,
                unsigned int n)
 {
-       struct priv *priv = dev->data->dev_private;
+       struct mlx5_priv *priv = dev->data->dev_private;
        unsigned int i;
        uint64_t counters[n];
+       struct mlx5_xstats_ctrl *xstats_ctrl = &priv->xstats_ctrl;
+       uint16_t mlx5_stats_n = xstats_ctrl->mlx5_stats_n;
 
-       if (n >= xstats_n && stats) {
-               struct mlx5_xstats_ctrl *xstats_ctrl = &priv->xstats_ctrl;
+       if (n >= mlx5_stats_n && stats) {
                int stats_n;
                int ret;
 
-               stats_n = mlx5_ethtool_get_stats_n(dev);
+               stats_n = mlx5_os_get_stats_n(dev);
                if (stats_n < 0)
                        return stats_n;
                if (xstats_ctrl->stats_n != stats_n)
-                       mlx5_xstats_init(dev);
-               ret = mlx5_read_dev_counters(dev, counters);
+                       mlx5_os_stats_init(dev);
+               ret = mlx5_os_read_dev_counters(dev, counters);
                if (ret)
                        return ret;
-               for (i = 0; i != xstats_n; ++i) {
+               for (i = 0; i != mlx5_stats_n; ++i) {
                        stats[i].id = i;
-                       stats[i].value = (counters[i] - xstats_ctrl->base[i]);
+                       if (xstats_ctrl->info[i].dev) {
+                               uint64_t wrap_n;
+                               uint64_t hw_stat = xstats_ctrl->hw_stats[i];
+
+                               stats[i].value = (counters[i] -
+                                                 xstats_ctrl->base[i]) &
+                                                 (uint64_t)UINT32_MAX;
+                               wrap_n = hw_stat >> 32;
+                               if (stats[i].value <
+                                           (hw_stat & (uint64_t)UINT32_MAX))
+                                       wrap_n++;
+                               stats[i].value |= (wrap_n) << 32;
+                               xstats_ctrl->hw_stats[i] = stats[i].value;
+                       } else {
+                               stats[i].value =
+                                       (counters[i] - xstats_ctrl->base[i]);
+                       }
                }
        }
-       return xstats_n;
+       mlx5_stats_n = mlx5_txpp_xstats_get(dev, stats, n, mlx5_stats_n);
+       return mlx5_stats_n;
 }
 
 /**
@@ -343,18 +94,22 @@ mlx5_xstats_get(struct rte_eth_dev *dev, struct rte_eth_xstat *stats,
 int
 mlx5_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats)
 {
-       struct priv *priv = dev->data->dev_private;
-       struct rte_eth_stats tmp = {0};
+       struct mlx5_priv *priv = dev->data->dev_private;
+       struct mlx5_stats_ctrl *stats_ctrl = &priv->stats_ctrl;
+       struct rte_eth_stats tmp;
        unsigned int i;
        unsigned int idx;
+       uint64_t wrap_n;
+       int ret;
 
+       memset(&tmp, 0, sizeof(tmp));
        /* Add software counters. */
        for (i = 0; (i != priv->rxqs_n); ++i) {
                struct mlx5_rxq_data *rxq = (*priv->rxqs)[i];
 
                if (rxq == NULL)
                        continue;
-               idx = rxq->stats.idx;
+               idx = rxq->idx;
                if (idx < RTE_ETHDEV_QUEUE_STAT_CNTRS) {
 #ifdef MLX5_PMD_SOFT_COUNTERS
                        tmp.q_ipackets[idx] += rxq->stats.ipackets;
@@ -375,13 +130,12 @@ mlx5_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats)
 
                if (txq == NULL)
                        continue;
-               idx = txq->stats.idx;
+               idx = txq->idx;
                if (idx < RTE_ETHDEV_QUEUE_STAT_CNTRS) {
 #ifdef MLX5_PMD_SOFT_COUNTERS
                        tmp.q_opackets[idx] += txq->stats.opackets;
                        tmp.q_obytes[idx] += txq->stats.obytes;
 #endif
-                       tmp.q_errors[idx] += txq->stats.oerrors;
                }
 #ifdef MLX5_PMD_SOFT_COUNTERS
                tmp.opackets += txq->stats.opackets;
@@ -389,6 +143,18 @@ mlx5_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats)
 #endif
                tmp.oerrors += txq->stats.oerrors;
        }
+       ret = mlx5_os_read_dev_stat(priv, "out_of_buffer", &tmp.imissed);
+       if (ret == 0) {
+               tmp.imissed = (tmp.imissed - stats_ctrl->imissed_base) &
+                                (uint64_t)UINT32_MAX;
+               wrap_n = stats_ctrl->imissed >> 32;
+               if (tmp.imissed < (stats_ctrl->imissed & (uint64_t)UINT32_MAX))
+                       wrap_n++;
+               tmp.imissed |= (wrap_n) << 32;
+               stats_ctrl->imissed = tmp.imissed;
+       } else {
+               tmp.imissed = stats_ctrl->imissed;
+       }
 #ifndef MLX5_PMD_SOFT_COUNTERS
        /* FIXME: retrieve and add hardware counters. */
 #endif
@@ -401,31 +167,36 @@ mlx5_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats)
  *
  * @param dev
  *   Pointer to Ethernet device structure.
+ *
+ * @return
+ *   always 0 on success and stats is reset
  */
-void
+int
 mlx5_stats_reset(struct rte_eth_dev *dev)
 {
-       struct priv *priv = dev->data->dev_private;
+       struct mlx5_priv *priv = dev->data->dev_private;
+       struct mlx5_stats_ctrl *stats_ctrl = &priv->stats_ctrl;
        unsigned int i;
-       unsigned int idx;
 
        for (i = 0; (i != priv->rxqs_n); ++i) {
                if ((*priv->rxqs)[i] == NULL)
                        continue;
-               idx = (*priv->rxqs)[i]->stats.idx;
-               (*priv->rxqs)[i]->stats =
-                       (struct mlx5_rxq_stats){ .idx = idx };
+               memset(&(*priv->rxqs)[i]->stats, 0,
+                      sizeof(struct mlx5_rxq_stats));
        }
        for (i = 0; (i != priv->txqs_n); ++i) {
                if ((*priv->txqs)[i] == NULL)
                        continue;
-               idx = (*priv->txqs)[i]->stats.idx;
-               (*priv->txqs)[i]->stats =
-                       (struct mlx5_txq_stats){ .idx = idx };
+               memset(&(*priv->txqs)[i]->stats, 0,
+                      sizeof(struct mlx5_txq_stats));
        }
+       mlx5_os_read_dev_stat(priv, "out_of_buffer", &stats_ctrl->imissed_base);
+       stats_ctrl->imissed = 0;
 #ifndef MLX5_PMD_SOFT_COUNTERS
        /* FIXME: reset hardware counters. */
 #endif
+
+       return 0;
 }
 
 /**
@@ -433,34 +204,42 @@ mlx5_stats_reset(struct rte_eth_dev *dev)
  *
  * @param dev
  *   Pointer to Ethernet device structure.
+ *
+ * @return
+ *   0 on success and stats is reset, negative errno value otherwise and
+ *   rte_errno is set.
  */
-void
+int
 mlx5_xstats_reset(struct rte_eth_dev *dev)
 {
-       struct priv *priv = dev->data->dev_private;
+       struct mlx5_priv *priv = dev->data->dev_private;
        struct mlx5_xstats_ctrl *xstats_ctrl = &priv->xstats_ctrl;
        int stats_n;
        unsigned int i;
-       unsigned int n = xstats_n;
+       unsigned int n = xstats_ctrl->mlx5_stats_n;
        uint64_t counters[n];
        int ret;
 
-       stats_n = mlx5_ethtool_get_stats_n(dev);
+       stats_n = mlx5_os_get_stats_n(dev);
        if (stats_n < 0) {
                DRV_LOG(ERR, "port %u cannot get stats: %s", dev->data->port_id,
                        strerror(-stats_n));
-               return;
+               return stats_n;
        }
        if (xstats_ctrl->stats_n != stats_n)
-               mlx5_xstats_init(dev);
-       ret = mlx5_read_dev_counters(dev, counters);
+               mlx5_os_stats_init(dev);
+       ret = mlx5_os_read_dev_counters(dev, counters);
        if (ret) {
                DRV_LOG(ERR, "port %u cannot read device counters: %s",
                        dev->data->port_id, strerror(rte_errno));
-               return;
+               return ret;
        }
-       for (i = 0; i != n; ++i)
+       for (i = 0; i != n; ++i) {
                xstats_ctrl->base[i] = counters[i];
+               xstats_ctrl->hw_stats[i] = 0;
+       }
+       mlx5_txpp_xstats_reset(dev);
+       return 0;
 }
 
 /**
@@ -477,18 +256,23 @@ mlx5_xstats_reset(struct rte_eth_dev *dev)
  *   Number of xstats names.
  */
 int
-mlx5_xstats_get_names(struct rte_eth_dev *dev __rte_unused,
+mlx5_xstats_get_names(struct rte_eth_dev *dev,
                      struct rte_eth_xstat_name *xstats_names, unsigned int n)
 {
        unsigned int i;
+       struct mlx5_priv *priv = dev->data->dev_private;
+       struct mlx5_xstats_ctrl *xstats_ctrl = &priv->xstats_ctrl;
+       unsigned int mlx5_xstats_n = xstats_ctrl->mlx5_stats_n;
 
-       if (n >= xstats_n && xstats_names) {
-               for (i = 0; i != xstats_n; ++i) {
+       if (n >= mlx5_xstats_n && xstats_names) {
+               for (i = 0; i != mlx5_xstats_n; ++i) {
                        strncpy(xstats_names[i].name,
-                               mlx5_counters_init[i].dpdk_name,
+                               xstats_ctrl->info[i].dpdk_name,
                                RTE_ETH_XSTATS_NAME_SIZE);
                        xstats_names[i].name[RTE_ETH_XSTATS_NAME_SIZE - 1] = 0;
                }
        }
-       return xstats_n;
+       mlx5_xstats_n = mlx5_txpp_xstats_get_names(dev, xstats_names,
+                                                  n, mlx5_xstats_n);
+       return mlx5_xstats_n;
 }