From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bart Van Assche Subject: Re: [PATCH 1/3] IB: new common API for draining queues Date: Thu, 11 Feb 2016 14:59:20 -0800 Message-ID: <56BD1248.80805@sandisk.com> References: <3e7261d1436d33320223d365974ff38945f0d558.1455230646.git.swise@chelsio.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <3e7261d1436d33320223d365974ff38945f0d558.1455230646.git.swise-ut6Up61K2wZBDgjK7y7TUQ@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Steve Wise , linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-rdma@vger.kernel.org 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. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html