On Fri, 2017-11-03 at 16:01 +0000, Marciniszyn, Mike wrote:
> > The RNR naks are happening because ALL post receives fail in a QP state of
> > reset.
> >
>
> Moving the post receives till after the init works:
>
> diff --git a/drivers/infiniband/ulp/srpt/ib_srpt.c b/drivers/infiniband/ulp/srpt/ib_srpt.c
> index 304855b..98b1b80 100644
> --- a/drivers/infiniband/ulp/srpt/ib_srpt.c
> +++ b/drivers/infiniband/ulp/srpt/ib_srpt.c
> @@ -1687,10 +1687,6 @@ static int srpt_create_ch_ib(struct srpt_rdma_ch *ch)
> goto err_destroy_cq;
> }
>
> - if (!sdev->use_srq)
> - for (i = 0; i < ch->rq_size; i++)
> - srpt_post_recv(sdev, ch, ch->ioctx_recv_ring[i]);
> -
> atomic_set(&ch->sq_wr_avail, qp_init->cap.max_send_wr);
>
> pr_debug("%s: max_cqe= %d max_sge= %d sq_size = %d cm_id= %p\n",
> @@ -1701,6 +1697,10 @@ static int srpt_create_ch_ib(struct srpt_rdma_ch *ch)
> if (ret)
> goto err_destroy_qp;
>
> + if (!sdev->use_srq)
> + for (i = 0; i < ch->rq_size; i++)
> + srpt_post_recv(sdev, ch, ch->ioctx_recv_ring[i]);
> +
> out:
> kfree(qp_init);
> return ret;
Thanks Mike for your work and this patch. I will test this patch further and
include it when I post the patches for fixing the behavior of the configfs
"use_srq" attribute.
Bart.��.n��������+%������w��{.n�����{���fk��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f
