On Tue, Jun 04, 2013 at 11:49:50AM -0700, sean.he...@intel.com wrote:

> +struct ibv_qp_open_attr {
> +.in +8
> +uint32_t          comp_mask;  /* Identifies valid fields */

Since the user is expected to set comp_mask we should clearly document
what it should be set to.

> +uint32_t          qp_num;     /* QP number */
> +void             *qp_context; /* Associated context of the QP */

Would be nice to clarify what this value is too. I took a quick peak
and it wasn't clear to me. The XRC example does not use it.

Is it a user defined cookie? If so can we call it a cookie instead?

Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to