From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sebastian Riemer Subject: Re: [PATCH 04/14] IB/srp: Skip host settle delay Date: Thu, 13 Jun 2013 11:53:37 +0200 Message-ID: <51B996A1.6080604@profitbricks.com> References: <51B87501.4070005@acm.org> <51B87689.8030806@acm.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <51B87689.8030806-HInyCGIudOg@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Bart Van Assche Cc: Roland Dreier , David Dillow , Vu Pham , linux-rdma , Christoph Hellwig List-Id: linux-rdma@vger.kernel.org On 12.06.2013 15:24, Bart Van Assche wrote: > The SRP initiator implements host reset by reconnecting to the SRP > target. That means that communication with the target is possible > as soon as host reset finished. Hence skip the host settle delay. > > Signed-off-by: Bart Van Assche > Cc: Roland Dreier > Cc: David Dillow > Cc: Vu Pham > Cc: Sebastian Riemer > --- > drivers/infiniband/ulp/srp/ib_srp.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/infiniband/ulp/srp/ib_srp.c b/drivers/infiniband/ulp/srp/ib_srp.c > index fb37b47..be12780 100644 > --- a/drivers/infiniband/ulp/srp/ib_srp.c > +++ b/drivers/infiniband/ulp/srp/ib_srp.c > @@ -1949,6 +1949,7 @@ static struct scsi_host_template srp_template = { > .eh_abort_handler = srp_abort, > .eh_device_reset_handler = srp_reset_device, > .eh_host_reset_handler = srp_reset_host, > + .skip_settle_delay = true, > .sg_tablesize = SRP_DEF_SG_TABLESIZE, > .can_queue = SRP_CMD_SQ_SIZE, > .this_id = -1, > Signed-off-by: Sebastian Riemer Acked-by: Sebastian Riemer Tested-by: Sebastian Riemer Reviewed-by: Sebastian Riemer Reviewed-by: Christoph Hellwig Choose something, I totally agree. Adding Christoph in CC as he has reviewed this as well. Cheers, Sebastian -- 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