RE: CQ overrun with ib_send_bw

[Ralph Campbell <ralph.campbell-h88ZbnxC6KDQT0dZR+AlfA@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 1318 bytes --]

The patch is attached.

On Tue, 2010-08-17 at 04:36 -0700, Amir Ancel wrote:
> Hi Sean,
> 
> We've seen this issue as well.
> 
> Can you send the patch directly to us ?
> 
> Added Raz from my team which replaces Ido while he is OOO.
> 
> 
> Thanks,
> 
> Amir Ancel
> Performance Team Manager
> Mellanox Technologies
> 
> -----Original Message-----
> From: Tziporet Koren 
> Sent: Tuesday, August 17, 2010 2:19 PM
> To: Ralph Campbell; Hefty, Sean; Ido Shamay; Amir Ancel
> Cc: Sumeet Lahorani; linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> Subject: RE: CQ overrun with ib_send_bw
> 
> On 8/13/2010 10:21 PM, Ralph Campbell wrote:
> > On Fri, 2010-08-13 at 12:14 -0700, Hefty, Sean wrote:
> >>> I know there is a bug with "ib_send_bw -b" (bi-directional)
> >>> since it doesn't create a CQ that is large enough for all the
> >>> posted sends *and* receives.  I have tried several times to get the
> >>> following patch applied but I never got a reply and nothing was
> >>> done.
> >>
> >> Who's the maintainer of these tests?
> >
> > I believe it is:
> >
> > Ido Shamai <idos-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>
> >
> > git://git.openfabrics.org/~shamoya/perftest.git
> >
> >
> 
> Yes Ido is the maintainer, however he is on vacation till Sep.
> I add Amir that may help for now
> 
> Tziporet
> 


[-- Attachment #2: send_bw.patch --]
[-- Type: text/x-patch, Size: 491 bytes --]

diff --git a/send_bw.c b/send_bw.c
index ddd2b73..e3f644a 100644
--- a/send_bw.c
+++ b/send_bw.c
@@ -746,6 +746,8 @@ static struct pingpong_context *pp_init_ctx(struct ibv_device *ib_dev,
 	if (user_parm->use_mcg && !user_parm->servername) {
 		cq_rx_depth *= user_parm->num_of_clients_mcg;
 	}
+	if (user_parm->duplex)
+		cq_rx_depth += ctx->tx_depth;
 	ctx->cq = ibv_create_cq(ctx->context,cq_rx_depth, NULL, ctx->channel, 0);
 	if (!ctx->cq) {
 		fprintf(stderr, "Couldn't create CQ\n");