From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Dillow Subject: Re: [PATCH 13/14] IB/srp: Make transport layer retry count configurable Date: Thu, 27 Jun 2013 17:22:18 -0400 Message-ID: <1372368138.32164.40.camel@frustration.ornl.gov> References: <51B87501.4070005@acm.org> <51B8794F.6050003@acm.org> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <51B8794F.6050003-HInyCGIudOg@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Bart Van Assche Cc: Roland Dreier , Vu Pham , Sebastian Riemer , linux-rdma List-Id: linux-rdma@vger.kernel.org On Wed, 2013-06-12 at 15:36 +0200, Bart Van Assche wrote: > From: Vu Pham > > Allow the InfiniBand RC retry count to be configured by the user > as an option in the target login string. The transport layer > timeout in nanoseconds is computed as follows from the retry count: > > rc_timeout = rc_retry_count * 4 * 4096 * (1 << qp->timeout) > > The default value for tl_retry_count is changed from 7 into 2. I'm not sure you should change the default, but I've not thought about it much. It should probably also have a default set via module parameter similar to the indirect_command_descriptors, etc. Unless srp_daemon is patched, this won't be usable without that. I'm not tied to this position, though -- we should be moving in the direction of more control via the login string. Otherwise, Acked-by: David Dillow -- Dave Dillow National Center for Computational Science Oak Ridge National Laboratory (865) 241-6602 office -- 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