public inbox for dev@dpdk.org
 help / color / mirror / Atom feed
* [PATCH 0/2] net/mlx5: small debugging improvements
@ 2026-03-24 17:10 Dariusz Sosnowski
  2026-03-24 17:10 ` [PATCH 1/2] common/mlx5: fix error logging for queue modify Dariusz Sosnowski
                   ` (2 more replies)
  0 siblings, 3 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
  Cc: dev, Raslan Darawsheh

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


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [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

end of thread, other threads:[~2026-03-25 14:29 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
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 ` [PATCH 0/2] net/mlx5: small debugging improvements Raslan Darawsheh

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox