projects
/
linux.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
cf6a05c
)
RDMA/irdma: Remove extra ret variable in favor of existing err
author
Zhu Yanjun
<yanjun.zhu@linux.dev>
Wed, 4 Jan 2023 06:43:33 +0000
(
01:43
-0500)
committer
Leon 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
patch
|
blob
|
history
diff --git
a/drivers/infiniband/hw/irdma/verbs.c
b/drivers/infiniband/hw/irdma/verbs.c
index f6973ea55eda7eefc518231b997d3a9811286bee..f4674ecf9c8cc46085f3d80605828e60756a705e 100644
(file)
--- 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;
}