livibverbs: Add support for XRC SRQs

XRC support requires the use of a new type of SRQ.

XRC shared receive queues: xrc srq's are similar to normal
srq's, except that they are bound to an xrcd, rather
than to a protection domain.  Based on the current spec
and implementation, they are only usable with xrc qps.  To
support xrc srq's, we define a new srq_init_attr structure
to include an srq type and other needed information.

The kernel ABI is also updated to allow creating extended
SRQs.

Signed-off-by: Sean Hefty <sean.hefty@intel.com>

diff --git a/include/infiniband/driver.h b/include/infiniband/driver.h
index cdd917a02641cf2957acb98debc47e6b373f2eff..30aa57387f41a5e2a7c814f3181d0076eac07252 100644 (file)
--- a/include/infiniband/driver.h
+++ b/include/infiniband/driver.h
@@ -64,6 +64,23 @@ struct verbs_xrcd {
        uint32_t                handle;
 };
 
+enum verbs_srq_mask {
+       VERBS_SRQ_TYPE          = 1 << 0,
+       VERBS_SRQ_XRCD          = 1 << 1,
+       VERBS_SRQ_CQ            = 1 << 2,
+       VERBS_SRQ_NUM           = 1 << 3,
+       VERBS_SRQ_RESERVED      = 1 << 4
+};
+
+struct verbs_srq {
+       struct ibv_srq          srq;
+       uint32_t                comp_mask;
+       enum ibv_srq_type       srq_type;
+       struct verbs_xrcd      *xrcd;
+       struct ibv_cq          *cq;
+       uint32_t                srq_num;
+};
+
 typedef struct ibv_device *(*ibv_driver_init_func)(const char *uverbs_sys_path,
                                                   int abi_version);
 
@@ -116,6 +133,10 @@ int ibv_cmd_create_srq(struct ibv_pd *pd,
                       struct ibv_srq *srq, struct ibv_srq_init_attr *attr,
                       struct ibv_create_srq *cmd, size_t cmd_size,
                       struct ibv_create_srq_resp *resp, size_t resp_size);
+int ibv_cmd_create_srq_ex(struct ibv_pd *pd,
+                         struct ibv_srq *srq, struct ibv_srq_init_attr_ex *attr_ex,
+                         struct ibv_create_xsrq *cmd, size_t cmd_size,
+                         struct ibv_create_srq_resp *resp, size_t resp_size);
 int ibv_cmd_modify_srq(struct ibv_srq *srq,
                       struct ibv_srq_attr *srq_attr,
                       int srq_attr_mask,
@@ -160,4 +181,10 @@ const char *ibv_get_sysfs_path(void);
 int ibv_read_sysfs_file(const char *dir, const char *file,
                        char *buf, size_t size);
 
+static inline uint32_t verbs_get_srq_num(struct ibv_srq *srq)
+{
+       struct verbs_srq *vsrq = container_of(srq, struct verbs_srq, srq);
+       return (vsrq->comp_mask & VERBS_SRQ_NUM) ? vsrq->srq_num : 0;
+}
+
 #endif /* INFINIBAND_DRIVER_H */

diff --git a/include/infiniband/kern-abi.h b/include/infiniband/kern-abi.h
index d7c673f43bed28ec9214bbf9da22ec7b678e4b19..3d72fa738c964c1cedee6a3f26cfb921b9c03031 100644 (file)
--- a/include/infiniband/kern-abi.h
+++ b/include/infiniband/kern-abi.h
@@ -88,6 +88,7 @@ enum {
        IB_USER_VERBS_CMD_POST_SRQ_RECV,
        IB_USER_VERBS_CMD_OPEN_XRCD,
        IB_USER_VERBS_CMD_CLOSE_XRCD,
+       IB_USER_VERBS_CMD_CREATE_XSRQ
 };
 
 /*
@@ -730,11 +731,28 @@ struct ibv_create_srq {
        __u64 driver_data[0];
 };
 
+struct ibv_create_xsrq {
+       __u32 command;
+       __u16 in_words;
+       __u16 out_words;
+       __u64 response;
+       __u64 user_handle;
+       __u32 srq_type;
+       __u32 pd_handle;
+       __u32 max_wr;
+       __u32 max_sge;
+       __u32 srq_limit;
+       __u32 reserved;
+       __u32 xrcd_handle;
+       __u32 cq_handle;
+       __u64 driver_data[0];
+};
+
 struct ibv_create_srq_resp {
        __u32 srq_handle;
        __u32 max_wr;
        __u32 max_sge;
-       __u32 reserved;
+       __u32 srqn;
 };
 
 struct ibv_modify_srq {
@@ -829,6 +847,7 @@ enum {
        IB_USER_VERBS_CMD_CREATE_COMP_CHANNEL_V2 = -1,
        IB_USER_VERBS_CMD_OPEN_XRCD_V2 = -1,
        IB_USER_VERBS_CMD_CLOSE_XRCD_V2 = -1,
+       IB_USER_VERBS_CMD_CREATE_XSRQ_V2 = -1
 };
 
 struct ibv_destroy_cq_v1 {

diff --git a/include/infiniband/verbs.h b/include/infiniband/verbs.h
index 08750e7835923ce3da8ea49fd49678bb23856269..c75b239c8900d7a6f93c1faff812a3d2a95d67ce 100644 (file)
--- a/include/infiniband/verbs.h
+++ b/include/infiniband/verbs.h
@@ -414,6 +414,28 @@ struct ibv_srq_init_attr {
        struct ibv_srq_attr     attr;
 };
 
+enum ibv_srq_type {
+       IBV_SRQT_BASIC,
+       IBV_SRQT_XRC
+};
+
+enum ibv_srq_init_attr_mask {
+       IBV_SRQ_INIT_ATTR_TYPE          = 1 << 0,
+       IBV_SRQ_INIT_ATTR_XRCD          = 1 << 1,
+       IBV_SRQ_INIT_ATTR_CQ            = 1 << 2,
+       IBV_SRQ_INIT_ATTR_RESERVED      = 1 << 3
+};
+
+struct ibv_srq_init_attr_ex {
+       void                   *srq_context;
+       struct ibv_srq_attr     attr;
+
+       uint64_t                comp_mask;
+       enum ibv_srq_type       srq_type;
+       struct ibv_xrcd        *xrcd;
+       struct ibv_cq          *cq;
+};
+
 enum ibv_qp_type {
        IBV_QPT_RC = 2,
        IBV_QPT_UC,
@@ -736,6 +758,9 @@ struct verbs_context {
        int (*drv_new_func1) ();        new corresponding provider call of func1
        int (*lib_new_func1) ();        New library call func1
        */
+       uint32_t                (*get_srq_num)(struct ibv_srq *srq);
+       struct ibv_srq *        (*create_srq_ex)(struct ibv_pd *pd,
+                                                struct ibv_srq_init_attr_ex *srq_init_attr_ex);
        struct ibv_xrcd *       (*open_xrcd)(struct ibv_context *context,
                                             int fd, int oflags);
        int                     (*close_xrcd)(struct ibv_xrcd *xrcd);
@@ -1021,6 +1046,17 @@ static inline int ibv_req_notify_cq(struct ibv_cq *cq, int solicited_only)
 struct ibv_srq *ibv_create_srq(struct ibv_pd *pd,
                               struct ibv_srq_init_attr *srq_init_attr);
 
+static inline struct ibv_srq *
+ibv_create_srq_ex(struct ibv_pd *pd, struct ibv_srq_init_attr_ex *srq_init_attr_ex)
+{
+       struct verbs_context *vctx = verbs_get_ctx_op(pd->context, create_srq_ex);
+       if (!vctx) {
+               errno = ENOSYS;
+               return NULL;
+       }
+       return vctx->create_srq_ex(pd, srq_init_attr_ex);
+}
+
 /**
  * ibv_modify_srq - Modifies the attributes for the specified SRQ.
  * @srq: The SRQ to modify.
@@ -1045,6 +1081,16 @@ int ibv_modify_srq(struct ibv_srq *srq,
  */
 int ibv_query_srq(struct ibv_srq *srq, struct ibv_srq_attr *srq_attr);
 
+static inline uint32_t ibv_get_srq_num(struct ibv_srq *srq)
+{
+       struct verbs_context *vctx = verbs_get_ctx_op(pd->context, get_srq_num);
+       if (!vctx) {
+               errno = ENOSYS;
+               return 0;
+       }
+       return vctx->get_srq_num(srq);
+}
+
 /**
  * ibv_destroy_srq - Destroys the specified SRQ.
  * @srq: The SRQ to destroy.

diff --git a/src/cmd.c b/src/cmd.c
index e6caf5b27635b9e74cc3a172db97cb539e785e4b..cbdcfdbdf03d692760243feac3c3e1d60b1f9acc 100644 (file)
--- a/src/cmd.c
+++ b/src/cmd.c
@@ -481,6 +481,69 @@ int ibv_cmd_create_srq(struct ibv_pd *pd,
        return 0;
 }
 
+int ibv_cmd_create_srq_ex(struct ibv_pd *pd,
+                         struct verbs_srq *srq, struct ibv_srq_init_attr_ex *attr_ex,
+                         struct ibv_create_xsrq *cmd, size_t cmd_size,
+                         struct ibv_create_srq_resp *resp, size_t resp_size)
+{
+       struct verbs_xrcd *vxrcd = NULL;
+
+       IBV_INIT_CMD_RESP(cmd, cmd_size, CREATE_XSRQ, resp, resp_size);
+
+       if (attr_ex->comp_mask >= IBV_SRQ_INIT_ATTR_RESERVED)
+               return ENOSYS;
+
+       cmd->user_handle = (uintptr_t) srq;
+       cmd->pd_handle   = pd->handle;
+       cmd->max_wr      = attr_ex->attr.max_wr;
+       cmd->max_sge     = attr_ex->attr.max_sge;
+       cmd->srq_limit   = attr_ex->attr.srq_limit;
+
+       cmd->srq_type = (attr_ex->comp_mask & IBV_SRQ_INIT_ATTR_TYPE) ?
+                       attr_ex->srq_type : IBV_SRQT_BASIC;
+       if (attr_ex->comp_mask & IBV_SRQ_INIT_ATTR_XRCD) {
+               vxrcd = container_of(attr_ex->xrcd, struct verbs_xrcd, xrcd);
+               cmd->xrcd_handle = vxrcd->handle;
+       }
+       if (attr_ex->comp_mask & IBV_SRQ_INIT_ATTR_CQ)
+               cmd->cq_handle   = attr_ex->cq->handle;
+
+       if (write(pd->context->cmd_fd, cmd, cmd_size) != cmd_size)
+               return errno;
+
+       VALGRIND_MAKE_MEM_DEFINED(resp, resp_size);
+
+       srq->handle           = resp->srq_handle;
+       srq->context          = pd->context;
+       srq->srq_context      = srq_init_attr_ex->srq_context;
+       srq->pd               = pd;
+       srq->events_completed = 0;
+       pthread_mutex_init(&srq->mutex, NULL);
+       pthread_cond_init(&srq->cond, NULL);
+
+       srq->comp_mask |= IBV_SRQ_INIT_ATTR_TYPE;
+       srq->srq_type = (srq_init_attr_ex->comp_mask & IBV_SRQ_INIT_ATTR_TYPE) ?
+                       srq_init_attr_ex->srq_type : IBV_SRQT_BASIC;
+       if (srq->srq_type == IBV_SRQT_XRC) {
+               srq->comp_mask |= IBV_SRQ_NUM;
+               srq->srq_num = resp->srqn;
+       }
+       if (srq_init_attr_ex->comp_mask & IBV_SRQ_INIT_ATTR_XRCD) {
+               srq->comp_mask |= IBV_SRQ_XRCD;
+               srq->xrcd = vxrcd;
+       }
+       if (srq_init_attr_ex->comp_mask & IBV_SRQ_INIT_ATTR_CQ) {
+               srq->comp_mask |= IBV_SRQ_CQ;
+               srq->cq = srq_init_attr_ex->cq;
+       }
+
+       attr_ex->attr.max_wr = resp->max_wr;
+       attr_ex->attr.max_sge = resp->max_sge;
+
+       return 0;
+}
+
+
 static int ibv_cmd_modify_srq_v3(struct ibv_srq *srq,
                                 struct ibv_srq_attr *srq_attr,
                                 int srq_attr_mask,

diff --git a/src/libibverbs.map b/src/libibverbs.map
index 9a15f3fb80945215b40c1c865d9a35490d54aa66..c8ec645964eccc50e6fb298d6df944d914d9f875 100644 (file)
--- a/src/libibverbs.map
+++ b/src/libibverbs.map
@@ -100,5 +100,6 @@ IBVERBS_1.1 {
                
                ibv_cmd_open_xrcd;
                ibv_cmd_close_xrcd;
+               ibv_cmd_create_srq_ex;
                
 } IBVERBS_1.0;