From: Zhu Yanjun Date: Wed, 4 Jan 2023 06:43:33 +0000 (-0500) Subject: RDMA/irdma: Remove extra ret variable in favor of existing err X-Git-Tag: v6.3-rc1~102^2~54 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=bd99ede8ef2dc03e29a181b755ba4f78da2644e6;p=thirdparty%2Fkernel%2Flinux.git RDMA/irdma: Remove extra ret variable in favor of existing err In the function irdma_reg_user_mr, err and ret exist. Actually, one variable err is enough. Signed-off-by: Zhu Yanjun Link: https://lore.kernel.org/r/20230104064333.660344-1-yanjun.zhu@intel.com Signed-off-by: Leon Romanovsky --- diff --git a/drivers/infiniband/hw/irdma/verbs.c b/drivers/infiniband/hw/irdma/verbs.c index f6973ea55eda7..f4674ecf9c8cc 100644 --- a/drivers/infiniband/hw/irdma/verbs.c +++ b/drivers/infiniband/hw/irdma/verbs.c @@ -2771,7 +2771,6 @@ static struct ib_mr *irdma_reg_user_mr(struct ib_pd *pd, u64 start, u64 len, bool use_pbles = false; unsigned long flags; int err = -EINVAL; - int ret; if (len > iwdev->rf->sc_dev.hw_attrs.max_mr_size) return ERR_PTR(-EINVAL); @@ -2871,9 +2870,9 @@ static struct ib_mr *irdma_reg_user_mr(struct ib_pd *pd, u64 start, u64 len, goto error; if (use_pbles) { - ret = irdma_check_mr_contiguous(palloc, + err = irdma_check_mr_contiguous(palloc, iwmr->page_size); - if (ret) { + if (err) { irdma_free_pble(iwdev->rf->pble_rsrc, palloc); iwpbl->pbl_allocated = false; }