From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bart Van Assche Subject: Re: [PATCH v2 0/3] new ib_drain_qp() API Date: Tue, 9 Feb 2016 13:03:07 -0800 Message-ID: <56BA540B.4040405@sandisk.com> References: <010901d16375$1a023210$4e069630$@opengridcomputing.com> <011601d1637b$8c01a3e0$a404eba0$@opengridcomputing.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <011601d1637b$8c01a3e0$a404eba0$@opengridcomputing.com> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Steve Wise , linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Cc: bart.vanassche-XdAiOPVOjttBDgjK7y7TUQ@public.gmane.org, Sagi Grimberg , 'Christoph Hellwig' , Chuck Lever List-Id: linux-rdma@vger.kernel.org On 02/09/2016 12:50 PM, Steve Wise wrote: >>> This series creates a new helper API for draining a queue pair. It is a >>> rework of an original patch created by Christoph Hellwig as part of the CQ >>> API rework and was dropped to be resubmitted by me with iw_cxgb4 support. >> >> I finally got SRP running to test this series and it isn't working. :( Anyway, there will be v3 posted once I fix it. >> >> Steve. > > The problem is SRP creates its SQ CQ with IB_POLL_DIRECT context, so the CQ is never armed for interrupts, and SRP polls the CQ > directly. So the drain logic won't work. > > I propose ib_drain_qp() takes a new parameter that tells it which queues to drain. Then the SRP code will only drain the RQ (which > is what it is doing prior to this series). Hello Steve, How about creating three functions - one that drains the receive queue, one that drains the send queue and a third function that drains both ? The latter function then can call the two former functions. And since only one of these three functions will have a user in your patch series (the function that drains the RQ), how about only introducing only that function now and to wait with introducing the two other functions until these have a user ? 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