RDMA/hns: Fix an error code in hns_roce_set_user_sq_size()

This function is supposed to return negative kernel error codes but here
it returns CMD_RST_PRC_EBUSY (2).  The error code eventually gets passed
to IS_ERR() and since it's not an error pointer it leads to an Oops in
hns_roce_v1_rsv_lp_qp()

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>

diff --git a/drivers/infiniband/hw/hns/hns_roce_hw_v2.c b/drivers/infiniband/hw/hns/hns_roce_hw_v2.c
index edd62b4dc0a03027e0ea3c4f685b970a71f0c2a6..7f2da5e10e67483a4995b0fc9bda2c2384f28bd6 100644 (file)
--- a/drivers/infiniband/hw/hns/hns_roce_hw_v2.c
+++ b/drivers/infiniband/hw/hns/hns_roce_hw_v2.c
@@ -1098,7 +1098,7 @@ static int hns_roce_cmq_send(struct hns_roce_dev *hr_dev,
        if (ret == CMD_RST_PRC_SUCCESS)
                return 0;
        if (ret == CMD_RST_PRC_EBUSY)
-               return ret;
+               return -EBUSY;
 
        ret = __hns_roce_cmq_send(hr_dev, desc, num);
        if (ret) {
@@ -1106,7 +1106,7 @@ static int hns_roce_cmq_send(struct hns_roce_dev *hr_dev,
                if (retval == CMD_RST_PRC_SUCCESS)
                        return 0;
                else if (retval == CMD_RST_PRC_EBUSY)
-                       return retval;
+                       return -EBUSY;
        }
 
        return ret;