Re: [lustre-devel] [PATCH 1/4] staging: lustre: o2iblnd: limit cap.max_send_wr for MLX5 (fwd)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> James,
>   I’m pretty sure this port to upstream is flawed.  See below

Yep. I missed that line. Thanks for review. Greg you can drop this one. I 
will submit a updated version soon. This patch series is not 
order dependent so the other patches are safe to land.
 
> On 5/9/16, 7:53 AM, "lustre-devel on behalf of James Simmons" <lustre-devel-bounces@xxxxxxxxxxxxxxxx on behalf of jsimmons@xxxxxxxxxxxxx> wrote:
> 
>>From: Dmitry Eremin <dmitry.eremin@xxxxxxxxx>
>>
>>Decrease cap.max_send_wr until it is accepted by rdma_create_qp()
>>
>>Signed-off-by: Dmitry Eremin <dmitry.eremin@xxxxxxxxx>
>>Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-7124
>>Reviewed-on: http://review.whamcloud.com/18347
>>Reviewed-by: Olaf Weber <olaf@xxxxxxx>
>>Reviewed-by: Doug Oucharek <doug.s.oucharek@xxxxxxxxx>
>>Reviewed-by: Oleg Drokin <oleg.drokin@xxxxxxxxx>
>>Signed-off-by: James Simmons <jsimmons@xxxxxxxxxxxxx>
>>---
>> .../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c    |   11 ++++++++++-
>> 1 files changed, 10 insertions(+), 1 deletions(-)
>>
>>diff --git a/drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c b/drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c
>>index d99b4fa..bc179a2 100644
>>--- a/drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c
>>+++ b/drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c
>>@@ -768,7 +768,12 @@ kib_conn_t *kiblnd_create_conn(kib_peer_t *peer, struct rdma_cm_id *cmid,
>> 
>>      conn->ibc_sched = sched;
>> 
>> -	rc = rdma_create_qp(cmid, conn->ibc_hdev->ibh_pd, init_qp_attr);
>> +	do {
>> +		rc = rdma_create_qp(cmid, conn->ibc_hdev->ibh_pd, init_qp_attr);
>> +		if (!rc || init_qp_attr->cap.max_send_wr < 16)
>> +			break;
> 
> I think the loop here is missing code to decrease the max_send_wr value.  In current master it’s
> 
> 		init_qp_attr->cap.max_send_wr -= init_qp_attr->cap.max_send_wr / 4;
> 
>>+	} while (rc);
>>+
>> 	if (rc) {
>> 		CERROR("Can't create QP: %d, send_wr: %d, recv_wr: %d\n",
>> 		       rc, init_qp_attr->cap.max_send_wr,
>>@@ -776,6 +781,10 @@ kib_conn_t *kiblnd_create_conn(kib_peer_t *peer, struct rdma_cm_id *cmid,
>> 		goto failed_2;
>> 	}
>> 
>>+	if (init_qp_attr->cap.max_send_wr != IBLND_SEND_WRS(conn))
>>+		CDEBUG(D_NET, "original send wr %d, created with %d\n",
>>+		       IBLND_SEND_WRS(conn), init_qp_attr->cap.max_send_wr);
>>+
>> 	LIBCFS_FREE(init_qp_attr, sizeof(*init_qp_attr));
>> 
>> 	/* 1 ref for caller and each rxmsg */
>> -- 
>> 1.7.1
>> 
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux