RDMA/qedr: Improve error logs for rdma_alloc_tid error return
authorPrabhakar Kushwaha <pkushwaha@marvell.com>
Thu, 29 Jul 2021 15:17:32 +0000 (18:17 +0300)
committerJason Gunthorpe <jgg@nvidia.com>
Fri, 30 Jul 2021 14:27:34 +0000 (11:27 -0300)
Use -EINVAL return type to identify whether error is returned because of
"Out of MR resources" or any other error types.

Link: https://lore.kernel.org/r/20210729151732.30995-2-pkushwaha@marvell.com
Signed-off-by: Shai Malin <smalin@marvell.com>
Signed-off-by: Ariel Elior <aelior@marvell.com>
Signed-off-by: Prabhakar Kushwaha <pkushwaha@marvell.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
drivers/infiniband/hw/qedr/verbs.c

index fdc47ef..b72ef24 100644 (file)
@@ -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;
        }