Re: [PATCH V1 libibverbs 1/8] Add ibv_poll_cq_ex verb

Jason Gunthorpe
2016-02-24T19:02:30Z

On Wed, Feb 24, 2016 at 11:41:57AM +0200, Yishai Hadas wrote:

> +enum {
> + IBV_WC_EX_WITH_8BIT_FIELDS = IBV_WC_EX_WITH_SL |
> + IBV_WC_EX_WITH_DLID_PATH_BITS
> +};
> +
> +struct ibv_wc_ex {
> + uint64_t wr_id;
> + /* wc_flags is a combination of ibv_wc_flags_ex flags. The IBV_WC_EX_WITH_XXX
> + * flags dynamically define the valid fields in buffer[0].
> + */
> + uint64_t wc_flags;
> + uint32_t status;
> + uint32_t opcode;
> + uint32_t vendor_err;
> + uint32_t reserved;
> + uint8_t buffer[0];
> +};

Um, maybe you should give an example of how on earth anyone is
supposed to use this, all of this looks like a *really bad idea* to
me.

Jason