On Mon, Oct 20, 2014 at 8:36 AM, Minh Duc Tran
<MinhDuc.Tran@xxxxxxxxxx> > ---
a/drivers/infiniband/ulp/iser/iser_verbs.c
> +++ b/drivers/infiniband/ulp/iser/iser_verbs.c
> @@ -114,6 +114,9 @@ static int iser_create_device_ib_res(struct iser_device *device)
> if (IS_ERR(device->pd))
> goto pd_err;
>
> + max_cqe = (dev_attr->max_cqe < ISER_MAX_CQ_LEN) ?
> + dev_attr->max_cqe : ISER_MAX_CQ_LEN;
> +
If I was the ocrdma maintainer I would say load and clear: NO, please.
Your current offering supports 32 CQs per device, and this means that
on 32 core node you will be able to run only iSER, no other ULP.
Generally, this approach is wrong, causing bad user experience and we
will not do that.
> for (i = 0; i < device->comps_used; i++) {
> struct iser_comp *comp = &device->comps[i];
>
> @@ -122,7 +125,7 @@ static int iser_create_device_ib_res(struct iser_device *device)
> iser_cq_callback,
> iser_cq_event_callback,
> (void *)comp,
> - ISER_MAX_CQ_LEN, i);
> + max_cqe, i);
> if (IS_ERR(comp->cq)) {
> comp->cq = NULL;
> goto cq_err;
> @@ -426,6 +429,7 @@ void iser_free_fastreg_pool(struct ib_conn *ib_conn)
> static int iser_create_ib_conn_res(struct ib_conn *ib_conn)
> {
> struct iser_device *device;
> + struct ib_device_attr *dev_attr;
> struct ib_qp_init_attr init_attr;
> int ret = -ENOMEM;
> int index, min_index = 0;
> @@ -433,6 +437,7 @@ static int iser_create_ib_conn_res(struct ib_conn *ib_conn)
> BUG_ON(ib_conn->device == NULL);
>
> device = ib_conn->device;
> + dev_attr = &device->dev_attr;
>
> memset(&init_attr, 0, sizeof init_attr);
>
> @@ -461,7 +466,9 @@ static int iser_create_ib_conn_res(struct ib_conn *ib_conn)
> init_attr.cap.max_send_wr = ISER_QP_SIG_MAX_REQ_DTOS + 1;
> init_attr.create_flags |= IB_QP_CREATE_SIGNATURE_EN;
> } else {
> - init_attr.cap.max_send_wr = ISER_QP_MAX_REQ_DTOS + 1;
> + init_attr.cap.max_send_wr =
> + (dev_attr->max_qp_wr < ISER_QP_MAX_REQ_DTOS) ?
> + dev_attr->max_qp_wr : ISER_QP_MAX_REQ_DTOS;
> }
>
> ret = rdma_create_qp(ib_conn->cma_id, device->pd, &init_attr);
> --
> 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
--
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
