diff options
| author |   Eli Cohen <eli@dev.mellanox.co.il> | 2013-10-31 15:26:32 +0200 |
|---|---|---|
| committer |   Roland Dreier <roland@purestorage.com> | 2013-11-15 10:25:32 -0800 |
| commit | 1c636f801615bdfc9b1d46904e8258c7a025670b (patch) | |
| tree | 600ff4b245e9287489c6c4b8a3a132f5adae5eb9 /drivers/infiniband/core/uverbs_cmd.c | |
| parent | IB/core: Add Cisco usNIC rdma node and transport types (diff) | |
| download | linux-dev-1c636f801615bdfc9b1d46904e8258c7a025670b.tar.xz linux-dev-1c636f801615bdfc9b1d46904e8258c7a025670b.zip | |
IB/core: Encorce MR access rights rules on kernel consumers
Enforce the rule that when requesting remote write or atomic permissions, local
write must be indicated as well. See IB spec 11.2.8.2.
Spotted by: Hagay Abramovsky <hagaya@mellanox.com>
Signed-off-by: Eli Cohen <eli@mellanox.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
Diffstat (limited to 'drivers/infiniband/core/uverbs_cmd.c')
| -rw-r--r-- | drivers/infiniband/core/uverbs_cmd.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/drivers/infiniband/core/uverbs_cmd.c b/drivers/infiniband/core/uverbs_cmd.c index 5bb2a82d52e7..7f671b7b8bac 100644 --- a/drivers/infiniband/core/uverbs_cmd.c +++ b/drivers/infiniband/core/uverbs_cmd.c @@ -939,13 +939,9 @@ ssize_t ib_uverbs_reg_mr(struct ib_uverbs_file *file, if ((cmd.start & ~PAGE_MASK) != (cmd.hca_va & ~PAGE_MASK)) return -EINVAL; - /* - * Local write permission is required if remote write or - * remote atomic permission is also requested. - */ - if (cmd.access_flags & (IB_ACCESS_REMOTE_ATOMIC | IB_ACCESS_REMOTE_WRITE) && - !(cmd.access_flags & IB_ACCESS_LOCAL_WRITE)) - return -EINVAL; + ret = ib_check_mr_access(cmd.access_flags); + if (ret) + return ret; uobj = kmalloc(sizeof *uobj, GFP_KERNEL); if (!uobj) |