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@vger.kernel.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 <i...@dev.mellanox.co.il>
> >
> > 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
>
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");
