patch 'common/mlx5: fix error logging for queue modify' has been queued to stable release 25.11.1

[Kevin Traynor]

Hi,

FYI, your patch has been queued to stable release 25.11.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 03/31/26. 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. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.

Queued patches are on a temporary branch at:
https://github.com/kevintraynor/dpdk-stable

This queued commit can be viewed at:
https://github.com/kevintraynor/dpdk-stable/commit/9753847318e2b1bca899a86f8a440efe9f67a8ab

Thanks.

Kevin

---
>From 9753847318e2b1bca899a86f8a440efe9f67a8ab Mon Sep 17 00:00:00 2001
From: Dariusz Sosnowski <dsosnowski at nvidia.com>
Date: Tue, 24 Mar 2026 18:10:27 +0100
Subject: [PATCH] common/mlx5: fix error logging for queue modify

[ upstream commit 1855b0c11b27797431b1602156b8896341874bfd ]

Add missing verbose log of FW errors when modifying SQ and RQ.

Fixes: b0067860959d ("common/mlx5: update log for DevX general command failure")

Signed-off-by: Dariusz Sosnowski <dsosnowski at nvidia.com>
Acked-by: Viacheslav Ovsiienko <viacheslavo at 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
@@ -1673,11 +1673,9 @@ 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;
 }
 
@@ -2117,10 +2115,9 @@ mlx5_devx_cmd_modify_sq(struct mlx5_devx_obj *sq,
 	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.53.0

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2026-03-27 09:58:26.801976567 +0000
+++ 0022-common-mlx5-fix-error-logging-for-queue-modify.patch	2026-03-27 09:58:26.150625508 +0000
@@ -1 +1 @@
-From 1855b0c11b27797431b1602156b8896341874bfd Mon Sep 17 00:00:00 2001
+From 9753847318e2b1bca899a86f8a440efe9f67a8ab Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit 1855b0c11b27797431b1602156b8896341874bfd ]
+
@@ -9 +10,0 @@
-Cc: stable at dpdk.org