Hi.
I noticed minor issue:

> @@ -564,6 +597,16 @@ struct ibv_srq {
>       pthread_mutex_t         mutex;
>       pthread_cond_t          cond;
>       uint32_t                events_completed;
> +
> +     /* Following fields only available if device supports extensions */
> +     enum ibv_srq_type       srq_type;
> +     union {
> +             struct {
> +                     uint32_t         srq_num;
> +                     struct ibv_xrcd *xrcd;
> +                     struct ibv_cq   *cq;
> +             } xrc;
> +     } ext;
>  };

Should the following struct be used instead of the suggested struct above? 

        struct {
                struct ibv_xrcd *xrcd;
                struct ibv_cq   *cq;
                uint32_t         srq_num;
        } xrc;

(attributes were rearranged according to their size).


Thanks
Dotan
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to