On Thu, Jan 18, 2018 at 10:36:01AM +0800, Yixian Liu wrote:
> diff --git a/providers/hns/hns_roce_u_verbs.c b/providers/hns/hns_roce_u_verbs.c
> index cde8568..7037a1c 100644
> +++ b/providers/hns/hns_roce_u_verbs.c
> @@ -276,6 +276,16 @@ struct ibv_cq *hns_roce_u_create_cq(struct ibv_context *context, int cqe,
>
> cmd.buf_addr = (uintptr_t) cq->buf.buf;
>
> + if (to_hr_dev(context->device)->hw_version != HNS_ROCE_HW_VER1) {
> + cq->set_ci_db = hns_roce_alloc_db(to_hr_ctx(context),
> + HNS_ROCE_CQ_TYPE_DB);
> + if (!cq->set_ci_db) {
> + fprintf(stderr, "alloc cq db buffer failed!\n");
> + goto err_buf;
> + }
> + cmd.db_addr = (uintptr_t) cq->set_ci_db;
Uhh.. why does the userspace already have the 'db_addr' member of
hns_roce_create_cq when the kernel doesn't?
What is going on here? How does forward and backwards compatibility of
the kABI work?
Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
