Sorry. I totally missed this. I will look at it in the morning. Bob -----Original Message----- From: Shoaib Rao <rao.shoaib@xxxxxxxxxx> Sent: Sunday, September 12, 2021 7:50 PM To: Jason Gunthorpe <jgg@xxxxxxxx> Cc: Pearson, Robert B <robert.pearson2@xxxxxxx>; Zhu Yanjun <zyjzyj2000@xxxxxxxxx>; RDMA mailing list <linux-rdma@xxxxxxxxxxxxxxx> Subject: Re: [PATCH v3 0/1] RDMA/rxe: Bump up default maximum values used via uverbs On 8/6/21 6:49 AM, Jason Gunthorpe wrote: > On Wed, Aug 04, 2021 at 11:11:15PM -0700, Shoaib Rao wrote: >> Bob, >> >> Your third patch has an issue. >> >> In rxe_cq_post() >> >> >> addr = producer_addr(cq->queue, QUEUE_TYPE_TO_CLIENT); >> >> It should be >> >> addr = producer_addr(cq->queue, QUEUE_TYPE_FROM_CLIENT); >> >> After making this change, I have tested my patch and rping works. >> >> Bob can you please point me to the discussion which lead to the >> current changes, particularly the need for user barrier. >> >> Zhu can you apply Bob's 3 patches + the change above + my patch and >> report back. In my testing it works. > I'll expect Bob to resend > > [for-next,v2,3/3] RDMA/rxe: Add memory barriers to kernel queues > > Jason I have not seen a reply to this email thread. Has the issue been resolved and I missed it? Shoaib
