* [PATCH 1/2] qed: Use accurate error num in qed_cxt_dynamic_ilt_alloc
@ 2021-07-29 15:17 Prabhakar Kushwaha
2021-07-29 15:17 ` [PATCH 2/2] qedr: Improve error logs for rdma_alloc_tid error return Prabhakar Kushwaha
2021-07-30 14:33 ` [PATCH 1/2] qed: Use accurate error num in qed_cxt_dynamic_ilt_alloc Jason Gunthorpe
0 siblings, 2 replies; 3+ messages in thread
From: Prabhakar Kushwaha @ 2021-07-29 15:17 UTC (permalink / raw)
To: linux-rdma, dledford, jgg, mkalderon
Cc: davem, kuba, smalin, aelior, pkushwaha, prabhakar.pkin, malin1024
To have more accurate error return type use -EOPNOTSUPP instead
of -EINVAL.
Signed-off-by: Shai Malin <smalin@marvell.com>
Signed-off-by: Ariel Elior <aelior@marvell.com>
Signed-off-by: Prabhakar Kushwaha <pkushwaha@marvell.com>
---
drivers/net/ethernet/qlogic/qed/qed_cxt.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/ethernet/qlogic/qed/qed_cxt.c b/drivers/net/ethernet/qlogic/qed/qed_cxt.c
index 5a0a3cbcc1c1..cb0f2a3a1ac9 100644
--- a/drivers/net/ethernet/qlogic/qed/qed_cxt.c
+++ b/drivers/net/ethernet/qlogic/qed/qed_cxt.c
@@ -2226,8 +2226,8 @@ qed_cxt_dynamic_ilt_alloc(struct qed_hwfn *p_hwfn,
p_blk = &p_cli->pf_blks[CDUT_SEG_BLK(QED_CXT_ROCE_TID_SEG)];
break;
default:
- DP_NOTICE(p_hwfn, "-EINVALID elem type = %d", elem_type);
- return -EINVAL;
+ DP_NOTICE(p_hwfn, "-EOPNOTSUPP elem type = %d", elem_type);
+ return -EOPNOTSUPP;
}
/* Calculate line in ilt */
--
2.24.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH 2/2] qedr: Improve error logs for rdma_alloc_tid error return
2021-07-29 15:17 [PATCH 1/2] qed: Use accurate error num in qed_cxt_dynamic_ilt_alloc Prabhakar Kushwaha
@ 2021-07-29 15:17 ` Prabhakar Kushwaha
2021-07-30 14:33 ` [PATCH 1/2] qed: Use accurate error num in qed_cxt_dynamic_ilt_alloc Jason Gunthorpe
1 sibling, 0 replies; 3+ messages in thread
From: Prabhakar Kushwaha @ 2021-07-29 15:17 UTC (permalink / raw)
To: linux-rdma, dledford, jgg, mkalderon
Cc: davem, kuba, smalin, aelior, pkushwaha, prabhakar.pkin, malin1024
Use -EINVAL return type to identify whether error is returned
because of "Out of MR resources" or any other error types.
Signed-off-by: Shai Malin <smalin@marvell.com>
Signed-off-by: Ariel Elior <aelior@marvell.com>
Signed-off-by: Prabhakar Kushwaha <pkushwaha@marvell.com>
---
drivers/infiniband/hw/qedr/verbs.c | 18 +++++++++++++++---
1 file changed, 15 insertions(+), 3 deletions(-)
diff --git a/drivers/infiniband/hw/qedr/verbs.c b/drivers/infiniband/hw/qedr/verbs.c
index fdc47ef7d861..b72ef24db657 100644
--- a/drivers/infiniband/hw/qedr/verbs.c
+++ b/drivers/infiniband/hw/qedr/verbs.c
@@ -2996,7 +2996,11 @@ struct ib_mr *qedr_reg_user_mr(struct ib_pd *ibpd, u64 start, u64 len,
rc = dev->ops->rdma_alloc_tid(dev->rdma_ctx, &mr->hw_mr.itid);
if (rc) {
- DP_ERR(dev, "roce alloc tid returned an error %d\n", rc);
+ if (rc == -EINVAL)
+ DP_ERR(dev, "Out of MR resources\n");
+ else
+ DP_ERR(dev, "roce alloc tid returned error %d\n", rc);
+
goto err1;
}
@@ -3091,7 +3095,11 @@ static struct qedr_mr *__qedr_alloc_mr(struct ib_pd *ibpd,
rc = dev->ops->rdma_alloc_tid(dev->rdma_ctx, &mr->hw_mr.itid);
if (rc) {
- DP_ERR(dev, "roce alloc tid returned an error %d\n", rc);
+ if (rc == -EINVAL)
+ DP_ERR(dev, "Out of MR resources\n");
+ else
+ DP_ERR(dev, "roce alloc tid returned error %d\n", rc);
+
goto err0;
}
@@ -3221,7 +3229,11 @@ struct ib_mr *qedr_get_dma_mr(struct ib_pd *ibpd, int acc)
rc = dev->ops->rdma_alloc_tid(dev->rdma_ctx, &mr->hw_mr.itid);
if (rc) {
- DP_ERR(dev, "roce alloc tid returned an error %d\n", rc);
+ if (rc == -EINVAL)
+ DP_ERR(dev, "Out of MR resources\n");
+ else
+ DP_ERR(dev, "roce alloc tid returned error %d\n", rc);
+
goto err1;
}
--
2.24.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 1/2] qed: Use accurate error num in qed_cxt_dynamic_ilt_alloc
2021-07-29 15:17 [PATCH 1/2] qed: Use accurate error num in qed_cxt_dynamic_ilt_alloc Prabhakar Kushwaha
2021-07-29 15:17 ` [PATCH 2/2] qedr: Improve error logs for rdma_alloc_tid error return Prabhakar Kushwaha
@ 2021-07-30 14:33 ` Jason Gunthorpe
1 sibling, 0 replies; 3+ messages in thread
From: Jason Gunthorpe @ 2021-07-30 14:33 UTC (permalink / raw)
To: Prabhakar Kushwaha
Cc: linux-rdma, dledford, mkalderon, davem, kuba, smalin, aelior,
prabhakar.pkin, malin1024
On Thu, Jul 29, 2021 at 06:17:31PM +0300, Prabhakar Kushwaha wrote:
> To have more accurate error return type use -EOPNOTSUPP instead
> of -EINVAL.
>
> Signed-off-by: Shai Malin <smalin@marvell.com>
> Signed-off-by: Ariel Elior <aelior@marvell.com>
> Signed-off-by: Prabhakar Kushwaha <pkushwaha@marvell.com>
> ---
> drivers/net/ethernet/qlogic/qed/qed_cxt.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Both applied to for-next, thanks
Jason
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-07-30 14:33 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-29 15:17 [PATCH 1/2] qed: Use accurate error num in qed_cxt_dynamic_ilt_alloc Prabhakar Kushwaha
2021-07-29 15:17 ` [PATCH 2/2] qedr: Improve error logs for rdma_alloc_tid error return Prabhakar Kushwaha
2021-07-30 14:33 ` [PATCH 1/2] qed: Use accurate error num in qed_cxt_dynamic_ilt_alloc Jason Gunthorpe
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.