RDMA/irdma: Remove extra ret variable in favor of existing err
authorZhu Yanjun <yanjun.zhu@linux.dev>
Wed, 4 Jan 2023 06:43:33 +0000 (01:43 -0500)
committerLeon Romanovsky <leon@kernel.org>
Wed, 4 Jan 2023 06:59:02 +0000 (08:59 +0200)
In the function irdma_reg_user_mr, err and ret exist. Actually,
one variable err is enough.

Signed-off-by: Zhu Yanjun <yanjun.zhu@linux.dev>
Link: https://lore.kernel.org/r/20230104064333.660344-1-yanjun.zhu@intel.com
Signed-off-by: Leon Romanovsky <leon@kernel.org>
drivers/infiniband/hw/irdma/verbs.c

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