Skip to content

Commit

Permalink
RDMA/efa: Remove umem check on dereg MR flow
Browse files Browse the repository at this point in the history
EFA driver is not a kverbs provider, the check for MR umem is redundant.

Link: https://lore.kernel.org/r/20190826115350.21718-2-galpress@amazon.com
Reviewed-by: Firas JahJah <firasj@amazon.com>
Reviewed-by: Yossi Leybovich <sleybo@amazon.com>
Signed-off-by: Gal Pressman <galpress@amazon.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
  • Loading branch information
Gal Pressman authored and Jason Gunthorpe committed Aug 27, 2019
1 parent d8abe88 commit 958b681
Showing 1 changed file with 5 additions and 7 deletions.
12 changes: 5 additions & 7 deletions drivers/infiniband/hw/efa/efa_verbs.c
Original file line number Diff line number Diff line change
Expand Up @@ -1500,14 +1500,12 @@ int efa_dereg_mr(struct ib_mr *ibmr, struct ib_udata *udata)

ibdev_dbg(&dev->ibdev, "Deregister mr[%d]\n", ibmr->lkey);

if (mr->umem) {
params.l_key = mr->ibmr.lkey;
err = efa_com_dereg_mr(&dev->edev, &params);
if (err)
return err;
}
ib_umem_release(mr->umem);
params.l_key = mr->ibmr.lkey;
err = efa_com_dereg_mr(&dev->edev, &params);
if (err)
return err;

ib_umem_release(mr->umem);
kfree(mr);

return 0;
Expand Down

0 comments on commit 958b681

Please sign in to comment.