From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id EA8F41B10D for ; Wed, 21 Nov 2018 17:50:58 +0100 (CET) Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.phx2.redhat.com [10.5.11.16]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 57BBA81F0B; Wed, 21 Nov 2018 16:50:58 +0000 (UTC) Received: from ktraynor.remote.csb (unknown [10.36.118.7]) by smtp.corp.redhat.com (Postfix) with ESMTP id 665AC5C21E; Wed, 21 Nov 2018 16:50:57 +0000 (UTC) From: Kevin Traynor To: Shahaf Shuler Cc: dpdk stable Date: Wed, 21 Nov 2018 16:48:14 +0000 Message-Id: <20181121164828.32249-60-ktraynor@redhat.com> In-Reply-To: <20181121164828.32249-1-ktraynor@redhat.com> References: <20181121164828.32249-1-ktraynor@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Scanned-By: MIMEDefang 2.79 on 10.5.11.16 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.27]); Wed, 21 Nov 2018 16:50:58 +0000 (UTC) Subject: [dpdk-stable] patch 'net/mlx5: support missing counter in extended statistics' has been queued to stable release 18.08.1 X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 21 Nov 2018 16:50:59 -0000 Hi, FYI, your patch has been queued to stable release 18.08.1 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 11/27/18. So please shout if anyone has objections. Also note that after the patch there's a diff of the upstream commit vs the patch applied to the branch. If the code is different (ie: not only metadata diffs), due for example to a change in context or macro names, please double check it. Thanks. Kevin Traynor --- >>From 866f2720a1735d213f08108fc8092fa13d7154bc Mon Sep 17 00:00:00 2001 From: Shahaf Shuler Date: Mon, 17 Sep 2018 12:46:34 +0300 Subject: [PATCH] net/mlx5: support missing counter in extended statistics [ upstream commit 1a611fdaf6ec3d860d7b0f39cdd8b754ef51ebc3 ] The current code would fail if one of the counters DPDK counters was not found on the device counters. As representors and PF port has different counters the both cannot work together. Addressing this issue by making the counter init more flexible to contain all the counter found and skipping the error. Signed-off-by: Shahaf Shuler --- drivers/net/mlx5/mlx5.h | 11 ++++++ drivers/net/mlx5/mlx5_stats.c | 67 +++++++++++++++-------------------- 2 files changed, 40 insertions(+), 38 deletions(-) diff --git a/drivers/net/mlx5/mlx5.h b/drivers/net/mlx5/mlx5.h index 35a196e76..b4ba147ab 100644 --- a/drivers/net/mlx5/mlx5.h +++ b/drivers/net/mlx5/mlx5.h @@ -72,10 +72,21 @@ struct mlx5_shared_data { extern struct mlx5_shared_data *mlx5_shared_data; +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. */ +}; + struct mlx5_xstats_ctrl { /* Number of device stats. */ uint16_t stats_n; + /* Number of device stats identified by PMD. */ + uint16_t mlx5_stats_n; /* Index in the device counters table. */ uint16_t dev_table_idx[MLX5_MAX_XSTATS]; uint64_t base[MLX5_MAX_XSTATS]; + struct mlx5_counter_ctrl info[MLX5_MAX_XSTATS]; }; diff --git a/drivers/net/mlx5/mlx5_stats.c b/drivers/net/mlx5/mlx5_stats.c index ed5b51bb2..9b89b9bfd 100644 --- a/drivers/net/mlx5/mlx5_stats.c +++ b/drivers/net/mlx5/mlx5_stats.c @@ -18,12 +18,4 @@ #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[] = { { @@ -154,10 +146,10 @@ mlx5_read_dev_counters(struct rte_eth_dev *dev, uint64_t *stats) return ret; } - for (i = 0; i != xstats_n; ++i) { - if (mlx5_counters_init[i].ib) { + for (i = 0; i != xstats_ctrl->mlx5_stats_n; ++i) { + if (xstats_ctrl->info[i].ib) { FILE *file; MKSTR(path, "%s/ports/1/hw_counters/%s", priv->ibdev_path, - mlx5_counters_init[i].ctr_name); + xstats_ctrl->info[i].ctr_name); file = fopen(path, "rb"); @@ -223,4 +215,6 @@ mlx5_xstats_init(struct rte_eth_dev *dev) int ret; + /* So that it won't aggregate for each init. */ + xstats_ctrl->mlx5_stats_n = 0; ret = mlx5_ethtool_get_stats_n(dev); if (ret < 0) { @@ -250,6 +244,4 @@ mlx5_xstats_init(struct rte_eth_dev *dev) 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 *) @@ -259,23 +251,23 @@ mlx5_xstats_init(struct rte_eth_dev *dev) if (!strcmp(mlx5_counters_init[j].ctr_name, curr_string)) { - xstats_ctrl->dev_table_idx[j] = i; + unsigned int idx = xstats_ctrl->mlx5_stats_n++; + + xstats_ctrl->dev_table_idx[idx] = i; + xstats_ctrl->info[idx] = mlx5_counters_init[j]; 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; + /* Add IB counters. */ + for (i = 0; i != xstats_n; ++i) { + if (mlx5_counters_init[i].ib) { + unsigned int idx = xstats_ctrl->mlx5_stats_n++; + + xstats_ctrl->info[idx] = mlx5_counters_init[i]; } } + assert(xstats_ctrl->mlx5_stats_n <= MLX5_MAX_XSTATS); xstats_ctrl->stats_n = dev_stats_n; /* Copy to base at first time. */ - assert(xstats_n <= MLX5_MAX_XSTATS); ret = mlx5_read_dev_counters(dev, xstats_ctrl->base); if (ret) @@ -307,9 +299,8 @@ mlx5_xstats_get(struct rte_eth_dev *dev, struct rte_eth_xstat *stats, 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 (!priv->xstats_ctrl.stats_n) - return 0; - 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; @@ -323,10 +314,10 @@ mlx5_xstats_get(struct rte_eth_dev *dev, struct rte_eth_xstat *stats, 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]); } } - return xstats_n; + return mlx5_stats_n; } @@ -444,5 +435,5 @@ mlx5_xstats_reset(struct rte_eth_dev *dev) 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; @@ -483,17 +474,17 @@ mlx5_xstats_get_names(struct rte_eth_dev *dev __rte_unused, struct rte_eth_xstat_name *xstats_names, unsigned int n) { - struct priv *priv = dev->data->dev_private; unsigned int i; + struct 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 (!priv->xstats_ctrl.stats_n) - return 0; - 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; + return mlx5_xstats_n; } -- 2.19.0 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2018-11-21 16:44:32.596694936 +0000 +++ 0060-net-mlx5-support-missing-counter-in-extended-statist.patch 2018-11-21 16:44:30.000000000 +0000 @@ -1,8 +1,10 @@ -From 1a611fdaf6ec3d860d7b0f39cdd8b754ef51ebc3 Mon Sep 17 00:00:00 2001 +From 866f2720a1735d213f08108fc8092fa13d7154bc Mon Sep 17 00:00:00 2001 From: Shahaf Shuler Date: Mon, 17 Sep 2018 12:46:34 +0300 Subject: [PATCH] net/mlx5: support missing counter in extended statistics +[ upstream commit 1a611fdaf6ec3d860d7b0f39cdd8b754ef51ebc3 ] + The current code would fail if one of the counters DPDK counters was not found on the device counters. @@ -12,8 +14,6 @@ Addressing this issue by making the counter init more flexible to contain all the counter found and skipping the error. -Cc: stable@dpdk.org - Signed-off-by: Shahaf Shuler --- drivers/net/mlx5/mlx5.h | 11 ++++++ @@ -21,10 +21,10 @@ 2 files changed, 40 insertions(+), 38 deletions(-) diff --git a/drivers/net/mlx5/mlx5.h b/drivers/net/mlx5/mlx5.h -index 4122e546c..ab242980a 100644 +index 35a196e76..b4ba147ab 100644 --- a/drivers/net/mlx5/mlx5.h +++ b/drivers/net/mlx5/mlx5.h -@@ -73,10 +73,21 @@ struct mlx5_shared_data { +@@ -72,10 +72,21 @@ struct mlx5_shared_data { extern struct mlx5_shared_data *mlx5_shared_data; +struct mlx5_counter_ctrl {