> -----Original Message----- > From: linux-rdma-owner@xxxxxxxxxxxxxxx [mailto:linux-rdma-owner@xxxxxxxxxxxxxxx] On Behalf Of Bart Van Assche > Sent: Thursday, February 11, 2016 4:59 PM > To: Steve Wise; linux-rdma@xxxxxxxxxxxxxxx > Subject: Re: [PATCH 1/3] IB: new common API for draining queues > > On 02/05/2016 01:13 PM, Steve Wise wrote: > > +static void wait_for_drain(struct ib_cq *cq, struct completion *c) > > +{ > > + if (cq->poll_ctx == IB_POLL_DIRECT) > > + do > > + ib_process_cq_direct(cq, 1024); > > + while (!wait_for_completion_timeout(c, msecs_to_jiffies(100))); > > + else > > + wait_for_completion(c); > > +} > > Hello Steve, > > Have you verified this patch with checkpatch ? I expect that checkpatch > will report that six braces are missing from this function. > > Thanks, > > Bart. I'll run it through. Thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html