diff options
author | Gal Pressman <galpress@amazon.com> | 2019-08-26 07:53:49 -0400 |
---|---|---|
committer | Jason Gunthorpe <jgg@mellanox.com> | 2019-08-27 12:01:14 -0400 |
commit | 958b6813f0c077c45a36d0a10b5bdcd27216eabe (patch) | |
tree | 15b76a23146c8fa6d0a4fdca74cd16671a791dc1 /drivers/infiniband/hw | |
parent | d8abe88450beb96a66e434323eb6ab737654b840 (diff) |
RDMA/efa: Remove umem check on dereg MR flow
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>
Diffstat (limited to 'drivers/infiniband/hw')
-rw-r--r-- | drivers/infiniband/hw/efa/efa_verbs.c | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/drivers/infiniband/hw/efa/efa_verbs.c b/drivers/infiniband/hw/efa/efa_verbs.c index 70851bd7f801..1e23c621a419 100644 --- a/drivers/infiniband/hw/efa/efa_verbs.c +++ b/drivers/infiniband/hw/efa/efa_verbs.c | |||
@@ -1500,14 +1500,12 @@ int efa_dereg_mr(struct ib_mr *ibmr, struct ib_udata *udata) | |||
1500 | 1500 | ||
1501 | ibdev_dbg(&dev->ibdev, "Deregister mr[%d]\n", ibmr->lkey); | 1501 | ibdev_dbg(&dev->ibdev, "Deregister mr[%d]\n", ibmr->lkey); |
1502 | 1502 | ||
1503 | if (mr->umem) { | 1503 | params.l_key = mr->ibmr.lkey; |
1504 | params.l_key = mr->ibmr.lkey; | 1504 | err = efa_com_dereg_mr(&dev->edev, ¶ms); |
1505 | err = efa_com_dereg_mr(&dev->edev, ¶ms); | 1505 | if (err) |
1506 | if (err) | 1506 | return err; |
1507 | return err; | ||
1508 | } | ||
1509 | ib_umem_release(mr->umem); | ||
1510 | 1507 | ||
1508 | ib_umem_release(mr->umem); | ||
1511 | kfree(mr); | 1509 | kfree(mr); |
1512 | 1510 | ||
1513 | return 0; | 1511 | return 0; |