diff options
author | Leon Romanovsky <leonro@mellanox.com> | 2018-06-24 11:23:48 +0300 |
---|---|---|
committer | Jason Gunthorpe <jgg@mellanox.com> | 2018-06-26 14:37:26 -0600 |
commit | 3a2e791c9456aab38a727b2b2558c08210f59f03 (patch) | |
tree | 5b6141c4d92fdd7eed186c95cfb611b06f88a055 /drivers/infiniband/core | |
parent | RDMA/uverbs: Don't overwrite NULL pointer with ZERO_SIZE_PTR (diff) | |
download | linux-dev-3a2e791c9456aab38a727b2b2558c08210f59f03.tar.xz linux-dev-3a2e791c9456aab38a727b2b2558c08210f59f03.zip |
RDMA/umem: Don't check for a negative return value of dma_map_sg_attrs()
dma_map_sg_attrs() returns 0 on error and can't return a negative number
(ensured by BUG_ON), so don't check.
Signed-off-by: Leon Romanovsky <leonro@mellanox.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
Diffstat (limited to 'drivers/infiniband/core')
-rw-r--r-- | drivers/infiniband/core/umem.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/infiniband/core/umem.c b/drivers/infiniband/core/umem.c index 54ab6335c48d..498f59bb4989 100644 --- a/drivers/infiniband/core/umem.c +++ b/drivers/infiniband/core/umem.c @@ -206,7 +206,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, DMA_BIDIRECTIONAL, dma_attrs); - if (umem->nmap <= 0) { + if (!umem->nmap) { ret = -ENOMEM; goto out; } |