Commit bd99ede8 authored by Zhu Yanjun's avatar Zhu Yanjun Committed by Leon Romanovsky
Browse files

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: default avatarZhu Yanjun <yanjun.zhu@linux.dev>
Link: https://lore.kernel.org/r/20230104064333.660344-1-yanjun.zhu@intel.com


Signed-off-by: default avatarLeon Romanovsky <leon@kernel.org>
parent cf6a05c8

drivers/infiniband/hw/irdma/verbs.c

+2 −3
Original line number Diff line number Diff line
@@ -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;

			}