net/mlx5: standardize on negative errno values
[dpdk.git] / drivers / net / mlx5 / mlx5_stats.c
index 2427585..06e9a1f 100644 (file)
@@ -1,40 +1,15 @@
-/*-
- *   BSD LICENSE
- *
- *   Copyright 2015 6WIND S.A.
- *   Copyright 2015 Mellanox.
- *
- *   Redistribution and use in source and binary forms, with or without
- *   modification, are permitted provided that the following conditions
- *   are met:
- *
- *     * Redistributions of source code must retain the above copyright
- *       notice, this list of conditions and the following disclaimer.
- *     * Redistributions in binary form must reproduce the above copyright
- *       notice, this list of conditions and the following disclaimer in
- *       the documentation and/or other materials provided with the
- *       distribution.
- *     * Neither the name of 6WIND S.A. nor the names of its
- *       contributors may be used to endorse or promote products derived
- *       from this software without specific prior written permission.
- *
- *   THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- *   "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- *   LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- *   A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- *   OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- *   SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- *   LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- *   DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- *   THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- *   (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- *   OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+/* SPDX-License-Identifier: BSD-3-Clause
+ * Copyright 2015 6WIND S.A.
+ * Copyright 2015 Mellanox.
  */
 
+#include <inttypes.h>
 #include <linux/sockios.h>
 #include <linux/ethtool.h>
+#include <stdint.h>
+#include <stdio.h>
 
-#include <rte_ethdev.h>
+#include <rte_ethdev_driver.h>
 #include <rte_common.h>
 #include <rte_malloc.h>
 
@@ -47,6 +22,7 @@ struct mlx5_counter_ctrl {
        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[] = {
@@ -121,6 +97,23 @@ static const struct mlx5_counter_ctrl mlx5_counters_init[] = {
        {
                .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",
        },
 };
 
@@ -129,39 +122,54 @@ static const unsigned int xstats_n = RTE_DIM(mlx5_counters_init);
 /**
  * Read device counters table.
  *
- * @param priv
- *   Pointer to private structure.
+ * @param dev
+ *   Pointer to Ethernet device.
  * @param[out] stats
  *   Counters table output buffer.
  *
  * @return
- *   0 on success and stats is filled, negative on error.
+ *   0 on success and stats is filled, negative errno value otherwise and
+ *   rte_errno is set.
  */
 static int
-priv_read_dev_counters(struct priv *priv, uint64_t *stats)
+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;
-       if (priv_ifreq(priv, SIOCETHTOOL, &ifr) != 0) {
+       ret = mlx5_ifreq(dev, SIOCETHTOOL, &ifr);
+       if (ret) {
                WARN("unable to read statistic values from device");
-               return -1;
+               return ret;
        }
        for (i = 0; i != xstats_n; ++i) {
-               if (priv_is_ib_cntr(mlx5_counters_init[i].ctr_name))
-                       priv_get_cntr_sysfs(priv,
-                                           mlx5_counters_init[i].ctr_name,
-                                           &stats[i]);
-               else
+               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;
 }
@@ -169,22 +177,25 @@ priv_read_dev_counters(struct priv *priv, uint64_t *stats)
 /**
  * Query the number of statistics provided by ETHTOOL.
  *
- * @param priv
- *   Pointer to private structure.
+ * @param dev
+ *   Pointer to Ethernet device.
  *
  * @return
- *   Number of statistics on success, -1 on error.
+ *   Number of statistics on success, negative errno value otherwise and
+ *   rte_errno is set.
  */
 static int
-priv_ethtool_get_stats_n(struct priv *priv) {
+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;
-       if (priv_ifreq(priv, SIOCETHTOOL, &ifr) != 0) {
+       ret = mlx5_ifreq(dev, SIOCETHTOOL, &ifr);
+       if (ret) {
                WARN("unable to query number of statistics");
-               return -1;
+               return ret;
        }
        return drvinfo.n_stats;
 }
@@ -192,12 +203,13 @@ priv_ethtool_get_stats_n(struct priv *priv) {
 /**
  * Init the structures to read device counters.
  *
- * @param priv
- *   Pointer to private structure.
+ * @param dev
+ *   Pointer to Ethernet device.
  */
 void
-priv_xstats_init(struct priv *priv)
+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;
@@ -205,12 +217,14 @@ priv_xstats_init(struct priv *priv)
        struct ethtool_gstrings *strings = NULL;
        unsigned int dev_stats_n;
        unsigned int str_sz;
+       int ret;
 
-       dev_stats_n = priv_ethtool_get_stats_n(priv);
-       if (dev_stats_n < 1) {
+       ret = mlx5_ethtool_get_stats_n(dev);
+       if (ret < 0) {
                WARN("no extended statistics available");
                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;
@@ -225,7 +239,8 @@ priv_xstats_init(struct priv *priv)
        strings->string_set = ETH_SS_STATS;
        strings->len = dev_stats_n;
        ifr.ifr_data = (caddr_t)strings;
-       if (priv_ifreq(priv, SIOCETHTOOL, &ifr) != 0) {
+       ret = mlx5_ifreq(dev, SIOCETHTOOL, &ifr);
+       if (ret) {
                WARN("unable to get statistic names");
                goto free;
        }
@@ -244,7 +259,7 @@ priv_xstats_init(struct priv *priv)
                }
        }
        for (j = 0; j != xstats_n; ++j) {
-               if (priv_is_ib_cntr(mlx5_counters_init[j].ctr_name))
+               if (mlx5_counters_init[j].ib)
                        continue;
                if (xstats_ctrl->dev_table_idx[j] >= dev_stats_n) {
                        WARN("counter \"%s\" is not recognized",
@@ -254,60 +269,56 @@ priv_xstats_init(struct priv *priv)
        }
        /* Copy to base at first time. */
        assert(xstats_n <= MLX5_MAX_XSTATS);
-       priv_read_dev_counters(priv, xstats_ctrl->base);
+       ret = mlx5_read_dev_counters(dev, xstats_ctrl->base);
+       if (ret)
+               ERROR("cannot read device counters: %s", strerror(rte_errno));
 free:
        rte_free(strings);
 }
 
 /**
- * Get device extended statistics.
+ * DPDK callback to get extended device statistics.
  *
- * @param priv
- *   Pointer to private structure.
+ * @param dev
+ *   Pointer to Ethernet device.
  * @param[out] stats
  *   Pointer to rte extended stats table.
+ * @param n
+ *   The size of the stats table.
  *
  * @return
  *   Number of extended stats on success and stats is filled,
- *   negative on error.
+ *   negative on error and rte_errno is set.
  */
-static int
-priv_xstats_get(struct priv *priv, struct rte_eth_xstat *stats)
+int
+mlx5_xstats_get(struct rte_eth_dev *dev, struct rte_eth_xstat *stats,
+               unsigned int n)
 {
-       struct mlx5_xstats_ctrl *xstats_ctrl = &priv->xstats_ctrl;
+       struct priv *priv = dev->data->dev_private;
        unsigned int i;
-       unsigned int n = xstats_n;
        uint64_t counters[n];
 
-       if (priv_read_dev_counters(priv, counters) < 0)
-               return -1;
-       for (i = 0; i != xstats_n; ++i) {
-               stats[i].id = i;
-               stats[i].value = (counters[i] - xstats_ctrl->base[i]);
+       if (n >= xstats_n && stats) {
+               struct mlx5_xstats_ctrl *xstats_ctrl = &priv->xstats_ctrl;
+               int stats_n;
+               int ret;
+
+               stats_n = mlx5_ethtool_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);
+               if (ret)
+                       return ret;
+               for (i = 0; i != xstats_n; ++i) {
+                       stats[i].id = i;
+                       stats[i].value = (counters[i] - xstats_ctrl->base[i]);
+               }
        }
        return n;
 }
 
-/**
- * Reset device extended statistics.
- *
- * @param priv
- *   Pointer to private structure.
- */
-static void
-priv_xstats_reset(struct priv *priv)
-{
-       struct mlx5_xstats_ctrl *xstats_ctrl = &priv->xstats_ctrl;
-       unsigned int i;
-       unsigned int n = xstats_n;
-       uint64_t counters[n];
-
-       if (priv_read_dev_counters(priv, counters) < 0)
-               return;
-       for (i = 0; i != n; ++i)
-               xstats_ctrl->base[i] = counters[i];
-}
-
 /**
  * DPDK callback to get device statistics.
  *
@@ -315,16 +326,19 @@ priv_xstats_reset(struct priv *priv)
  *   Pointer to Ethernet device structure.
  * @param[out] stats
  *   Stats structure output buffer.
+ *
+ * @return
+ *   0 on success and stats is filled, negative errno value otherwise and
+ *   rte_errno is set.
  */
 int
 mlx5_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats)
 {
-       struct priv *priv = mlx5_get_priv(dev);
+       struct priv *priv = dev->data->dev_private;
        struct rte_eth_stats tmp = {0};
        unsigned int i;
        unsigned int idx;
 
-       priv_lock(priv);
        /* Add software counters. */
        for (i = 0; (i != priv->rxqs_n); ++i) {
                struct mlx5_rxq_data *rxq = (*priv->rxqs)[i];
@@ -370,7 +384,6 @@ mlx5_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats)
        /* FIXME: retrieve and add hardware counters. */
 #endif
        *stats = tmp;
-       priv_unlock(priv);
        return 0;
 }
 
@@ -387,7 +400,6 @@ mlx5_stats_reset(struct rte_eth_dev *dev)
        unsigned int i;
        unsigned int idx;
 
-       priv_lock(priv);
        for (i = 0; (i != priv->rxqs_n); ++i) {
                if ((*priv->rxqs)[i] == NULL)
                        continue;
@@ -405,45 +417,6 @@ mlx5_stats_reset(struct rte_eth_dev *dev)
 #ifndef MLX5_PMD_SOFT_COUNTERS
        /* FIXME: reset hardware counters. */
 #endif
-       priv_unlock(priv);
-}
-
-/**
- * DPDK callback to get extended device statistics.
- *
- * @param dev
- *   Pointer to Ethernet device structure.
- * @param[out] stats
- *   Stats table output buffer.
- * @param n
- *   The size of the stats table.
- *
- * @return
- *   Number of xstats on success, negative on failure.
- */
-int
-mlx5_xstats_get(struct rte_eth_dev *dev,
-               struct rte_eth_xstat *stats, unsigned int n)
-{
-       struct priv *priv = mlx5_get_priv(dev);
-       int ret = xstats_n;
-
-       if (n >= xstats_n && stats) {
-               struct mlx5_xstats_ctrl *xstats_ctrl = &priv->xstats_ctrl;
-               int stats_n;
-
-               priv_lock(priv);
-               stats_n = priv_ethtool_get_stats_n(priv);
-               if (stats_n < 0) {
-                       priv_unlock(priv);
-                       return -1;
-               }
-               if (xstats_ctrl->stats_n != stats_n)
-                       priv_xstats_init(priv);
-               ret = priv_xstats_get(priv, stats);
-               priv_unlock(priv);
-       }
-       return ret;
 }
 
 /**
@@ -455,19 +428,30 @@ mlx5_xstats_get(struct rte_eth_dev *dev,
 void
 mlx5_xstats_reset(struct rte_eth_dev *dev)
 {
-       struct priv *priv = mlx5_get_priv(dev);
+       struct 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;
+       uint64_t counters[n];
+       int ret;
 
-       priv_lock(priv);
-       stats_n = priv_ethtool_get_stats_n(priv);
-       if (stats_n < 0)
-               goto unlock;
+       stats_n = mlx5_ethtool_get_stats_n(dev);
+       if (stats_n < 0) {
+               ERROR("%p cannot get stats: %s", (void *)dev,
+                     strerror(-stats_n));
+               return;
+       }
        if (xstats_ctrl->stats_n != stats_n)
-               priv_xstats_init(priv);
-       priv_xstats_reset(priv);
-unlock:
-       priv_unlock(priv);
+               mlx5_xstats_init(dev);
+       ret = mlx5_read_dev_counters(dev, counters);
+       if (ret) {
+               ERROR("%p cannot read device counters: %s", (void *)dev,
+                     strerror(rte_errno));
+               return;
+       }
+       for (i = 0; i != n; ++i)
+               xstats_ctrl->base[i] = counters[i];
 }
 
 /**
@@ -484,21 +468,18 @@ unlock:
  *   Number of xstats names.
  */
 int
-mlx5_xstats_get_names(struct rte_eth_dev *dev,
+mlx5_xstats_get_names(struct rte_eth_dev *dev __rte_unused,
                struct rte_eth_xstat_name *xstats_names, unsigned int n)
 {
-       struct priv *priv = mlx5_get_priv(dev);
        unsigned int i;
 
        if (n >= xstats_n && xstats_names) {
-               priv_lock(priv);
                for (i = 0; i != xstats_n; ++i) {
                        strncpy(xstats_names[i].name,
                                mlx5_counters_init[i].dpdk_name,
                                RTE_ETH_XSTATS_NAME_SIZE);
                        xstats_names[i].name[RTE_ETH_XSTATS_NAME_SIZE - 1] = 0;
                }
-               priv_unlock(priv);
        }
        return xstats_n;
 }