* [PATCH 1/2] common/mlx5: fix error logging for queue modify
2026-03-24 17:10 [PATCH 0/2] net/mlx5: small debugging improvements Dariusz Sosnowski
@ 2026-03-24 17:10 ` Dariusz Sosnowski
2026-03-24 17:10 ` [PATCH 2/2] net/mlx5: report share group and queue ID Dariusz Sosnowski
2026-03-25 14:28 ` [PATCH 0/2] net/mlx5: small debugging improvements Raslan Darawsheh
2 siblings, 0 replies; 4+ messages in thread
From: Dariusz Sosnowski @ 2026-03-24 17:10 UTC (permalink / raw)
To: Viacheslav Ovsiienko, Bing Zhao, Ori Kam, Suanming Mou,
Matan Azrad, Gregory Etelson
Cc: dev, Raslan Darawsheh, stable
Add missing verbose log of FW errors when modifying SQ and RQ.
Fixes: b0067860959d ("common/mlx5: update log for DevX general command failure")
Cc: getelson@nvidia.com
Cc: stable@dpdk.org
Signed-off-by: Dariusz Sosnowski <dsosnowski@nvidia.com>
Acked-by: Viacheslav Ovsiienko <viacheslavo@nvidia.com>
---
drivers/common/mlx5/mlx5_devx_cmds.c | 19 ++++++++-----------
1 file changed, 8 insertions(+), 11 deletions(-)
diff --git a/drivers/common/mlx5/mlx5_devx_cmds.c b/drivers/common/mlx5/mlx5_devx_cmds.c
index d12ebf8487..d46aa045df 100644
--- a/drivers/common/mlx5/mlx5_devx_cmds.c
+++ b/drivers/common/mlx5/mlx5_devx_cmds.c
@@ -1672,13 +1672,11 @@ mlx5_devx_cmd_modify_rq(struct mlx5_devx_obj *rq,
}
ret = mlx5_glue->devx_obj_modify(rq->obj, in, sizeof(in),
out, sizeof(out));
-
- if (ret) {
- DRV_LOG(ERR, "Failed to modify RQ using DevX");
- rte_errno = errno;
- return -errno;
+ if (ret || MLX5_FW_STATUS(out)) {
+ DEVX_DRV_LOG(ERR, out, "RQ modify", "rq_id", rq->id);
+ return MLX5_DEVX_ERR_RC(ret);
}
- return ret;
+ return 0;
}
/*
@@ -2116,12 +2114,11 @@ mlx5_devx_cmd_modify_sq(struct mlx5_devx_obj *sq,
MLX5_SET(sqc, sq_ctx, hairpin_peer_vhca, sq_attr->hairpin_peer_vhca);
ret = mlx5_glue->devx_obj_modify(sq->obj, in, sizeof(in),
out, sizeof(out));
- if (ret) {
- DRV_LOG(ERR, "Failed to modify SQ using DevX");
- rte_errno = errno;
- return -rte_errno;
+ if (ret || MLX5_FW_STATUS(out)) {
+ DEVX_DRV_LOG(ERR, out, "SQ modify", "sq_id", sq->id);
+ return MLX5_DEVX_ERR_RC(ret);
}
- return ret;
+ return 0;
}
/*
--
2.47.3
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH 2/2] net/mlx5: report share group and queue ID
2026-03-24 17:10 [PATCH 0/2] net/mlx5: small debugging improvements Dariusz Sosnowski
2026-03-24 17:10 ` [PATCH 1/2] common/mlx5: fix error logging for queue modify Dariusz Sosnowski
@ 2026-03-24 17:10 ` Dariusz Sosnowski
2026-03-25 14:28 ` [PATCH 0/2] net/mlx5: small debugging improvements Raslan Darawsheh
2 siblings, 0 replies; 4+ messages in thread
From: Dariusz Sosnowski @ 2026-03-24 17:10 UTC (permalink / raw)
To: Viacheslav Ovsiienko, Bing Zhao, Ori Kam, Suanming Mou,
Matan Azrad, Xueming Li
Cc: dev, Raslan Darawsheh, stable
Add missing share group and queue index to Rx queue info
reported through rte_eth_rx_queue_info_get().
Fixes: 09c2555303be ("net/mlx5: support shared Rx queue")
Cc: xuemingl@nvidia.com
Cc: stable@dpdk.org
Signed-off-by: Dariusz Sosnowski <dsosnowski@nvidia.com>
Acked-by: Viacheslav Ovsiienko <viacheslavo@nvidia.com>
---
drivers/net/mlx5/mlx5_rx.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/drivers/net/mlx5/mlx5_rx.c b/drivers/net/mlx5/mlx5_rx.c
index bc0470e6af..1f818f3c83 100644
--- a/drivers/net/mlx5/mlx5_rx.c
+++ b/drivers/net/mlx5/mlx5_rx.c
@@ -190,6 +190,10 @@ mlx5_rxq_info_get(struct rte_eth_dev *dev, uint16_t rx_queue_id,
RTE_BIT32(rxq->elts_n);
qinfo->avail_thresh = rxq_priv ?
mlx5_rxq_lwm_to_percentage(rxq_priv) : 0;
+ if (rxq_ctrl != NULL) {
+ qinfo->conf.share_group = rxq_ctrl->share_group;
+ qinfo->conf.share_qid = rxq_ctrl->share_qid;
+ }
}
/**
--
2.47.3
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH 0/2] net/mlx5: small debugging improvements
2026-03-24 17:10 [PATCH 0/2] net/mlx5: small debugging improvements Dariusz Sosnowski
2026-03-24 17:10 ` [PATCH 1/2] common/mlx5: fix error logging for queue modify Dariusz Sosnowski
2026-03-24 17:10 ` [PATCH 2/2] net/mlx5: report share group and queue ID Dariusz Sosnowski
@ 2026-03-25 14:28 ` Raslan Darawsheh
2 siblings, 0 replies; 4+ messages in thread
From: Raslan Darawsheh @ 2026-03-25 14:28 UTC (permalink / raw)
To: Dariusz Sosnowski, Viacheslav Ovsiienko, Bing Zhao, Ori Kam,
Suanming Mou, Matan Azrad
Cc: dev
Hi,
On 24/03/2026 7:10 PM, Dariusz Sosnowski wrote:
> Small debugging improvements for mlx5 PMD:
>
> - Fix error logs generated on failure to modify SQ and RQ,
> so that verbose FW error is printed.
> - Add reporting share group and share queue ID
> in mlx5 implementation of rte_eth_rx_queue_info_get().
>
> Dariusz Sosnowski (2):
> common/mlx5: fix error logging for queue modify
> net/mlx5: report share group and queue ID
>
> drivers/common/mlx5/mlx5_devx_cmds.c | 19 ++++++++-----------
> drivers/net/mlx5/mlx5_rx.c | 4 ++++
> 2 files changed, 12 insertions(+), 11 deletions(-)
>
> --
> 2.47.3
>
Series applied to next-net-mlx,
Kindest regards
Raslan Darawsheh
^ permalink raw reply [flat|nested] 4+ messages in thread