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 /include/rdma | |
| parent | 180771a3707a4c0577cbf4f830c754dbabfdfccb (diff) | |
| download | lwn-1c636f801615bdfc9b1d46904e8258c7a025670b.tar.gz lwn-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 'include/rdma')
| -rw-r--r-- | include/rdma/ib_verbs.h | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/include/rdma/ib_verbs.h b/include/rdma/ib_verbs.h index 60354d53948e..68c053d0d629 100644 --- a/include/rdma/ib_verbs.h +++ b/include/rdma/ib_verbs.h @@ -2386,4 +2386,17 @@ struct ib_flow *ib_create_flow(struct ib_qp *qp, struct ib_flow_attr *flow_attr, int domain); int ib_destroy_flow(struct ib_flow *flow_id); +static inline int ib_check_mr_access(int flags) +{ + /* + * Local write permission is required if remote write or + * remote atomic permission is also requested. + */ + if (flags & (IB_ACCESS_REMOTE_ATOMIC | IB_ACCESS_REMOTE_WRITE) && + !(flags & IB_ACCESS_LOCAL_WRITE)) + return -EINVAL; + + return 0; +} + #endif /* IB_VERBS_H */ |