Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755252AbbKWV2c (ORCPT ); Mon, 23 Nov 2015 16:28:32 -0500 Received: from quartz.orcorp.ca ([184.70.90.242]:53408 "EHLO quartz.orcorp.ca" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751270AbbKWV2a (ORCPT ); Mon, 23 Nov 2015 16:28:30 -0500 Date: Mon, 23 Nov 2015 14:28:22 -0700 From: Jason Gunthorpe To: Bart Van Assche Cc: Christoph Hellwig , linux-rdma@vger.kernel.org, sagig@dev.mellanox.co.il, axboe@fb.com, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/9] IB: add a proper completion queue abstraction Message-ID: <20151123212822.GE6062@obsidianresearch.com> References: <1447422410-20891-1-git-send-email-hch@lst.de> <1447422410-20891-3-git-send-email-hch@lst.de> <20151113182513.GB21808@obsidianresearch.com> <564640C4.3000603@sandisk.com> <20151113220636.GA32133@obsidianresearch.com> <20151114071344.GE27738@lst.de> <20151123203712.GB5640@obsidianresearch.com> <56537F59.4080708@sandisk.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <56537F59.4080708@sandisk.com> User-Agent: Mutt/1.5.23 (2014-03-12) X-Broken-Reverse-DNS: no host name found for IP address 10.0.0.160 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1076 Lines: 25 On Mon, Nov 23, 2015 at 01:04:25PM -0800, Bart Van Assche wrote: > Considerable time ago the send queue in the SRP initiator driver was > modified from signaled to non-signaled to reduce the number of interrupts > triggered by the SRP initiator driver. The SRP initiator driver polls the > send queue every time before a SCSI command is sent to the target. I think > this is a pattern that is also useful for other ULP's so I'm not convinced > that ib_process_cq_direct() should be deprecated :-) As I explained, that is a fine idea, but I can't see how SRP is able to correctly do sendq flow control without spinning on the poll, which it does not do. I'm guessing SRP is trying to drive sendq flow control from the recv side, like NFS was. This is wrong and should not be part of the common API. Does that make sense? Jason -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/