[PATCH v1] net/mlx5: fix secondary process query stats segfault
Rongwei Liu
rongweil at nvidia.com
Mon Jan 22 08:44:05 CET 2024
The "outer_of_buffer" counter is owned by the primary process devx
object and it is pointer by pointer in mlx5_priv structure. Actually,
there are 4 levels' pointers in this piece of code.
The secondary process can't access this part directly since it belongs
to another process' heap.
Return ENOTSUP as workaround.
Signed-off-by: Rongwei Liu <rongweil at nvidia.com>
Acked-by: Matan Azrad <matan at nvidia.com>
Fixes: 750e48c7d ("common/mlx5: add DevX commands for queue counters")
Cc: matan at nvidia.com
Cc: stable at dpdk.org
---
drivers/net/mlx5/linux/mlx5_os.c | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/drivers/net/mlx5/linux/mlx5_os.c b/drivers/net/mlx5/linux/mlx5_os.c
index 8682899735..dc854ebb64 100644
--- a/drivers/net/mlx5/linux/mlx5_os.c
+++ b/drivers/net/mlx5/linux/mlx5_os.c
@@ -3192,9 +3192,15 @@ mlx5_os_read_dev_stat(struct mlx5_priv *priv, const char *ctr_name,
if (priv->sh) {
if (priv->q_counters != NULL &&
- strcmp(ctr_name, "out_of_buffer") == 0)
+ strcmp(ctr_name, "out_of_buffer") == 0) {
+ if (rte_eal_process_type() == RTE_PROC_SECONDARY) {
+ DRV_LOG(WARNING, "Devx out_of_buffer counter is not supported in the secondary process");
+ rte_errno = ENOTSUP;
+ return 1;
+ }
return mlx5_devx_cmd_queue_counter_query
(priv->q_counters, 0, (uint32_t *)stat);
+ }
MKSTR(path, "%s/ports/%d/hw_counters/%s",
priv->sh->ibdev_path,
priv->dev_port,
--
2.27.0
More information about the stable
mailing list