From mboxrd@z Thu Jan 1 00:00:00 1970 From: Or Gerlitz Subject: Re: [PATCH for 3.8 v3, resend 0/3] IB/SRP patches for kernel 3.8 Date: Thu, 7 Feb 2013 11:41:22 +0200 Message-ID: <511376C2.6050100@mellanox.com> References: <510BDCAA.204@acm.org> <5111327F.6050402@acm.org> <5112049B.8030406@acm.org> <5112CE60.2030607@mellanox.com> <51136E74.9090209@acm.org> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <51136E74.9090209-HInyCGIudOg@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Bart Van Assche Cc: Vu Pham , Or Gerlitz , David Dillow , Roland Dreier , "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Oren Duer , Sagi Grimberg List-Id: linux-rdma@vger.kernel.org On 07/02/2013 11:05, Bart Van Assche wrote: > On 02/06/13 22:42, Vu Pham wrote: >> Conclusion: >> 1. disable the port/path long enough >35 minutes, we have dangling scsi >> host. >> 2. enable the port within 30 minute, scsi host re-establish connection, >> path re-instate and then scsi_host was removed (no entry in sysfs) >> >> I attached a log here to show what happened above. > > Hello Vu, > > I found the following in the attached logs: > > [ ... ] > Feb 6 19:24:25 vsa30 kernel: scsi host10: ib_srp: SRP reset_host called > [ ... ] > Feb 6 19:25:28 vsa30 kernel: scsi host10: SRP abort called > [ ... ] > > It is easy to see in patch 3/3 that srp_reset_host() invokes > srp_reconnect_target() unconditionally and that that last function > kills all outstanding requests via srp_reset_req(). So to me the above > output means that the attached logs were generated by a kernel missing > at least patch 3/3. This means that the above conclusions are invalid. Hi Bart, Please calm down, we truly think your work is great step in the right direction. We will double check the environment and the logs provided to you. On the half side of the glass, I think Vu also saw things that work better with these patches (BTW - if the fourth patch that Vu used "save & restore host_scribble during error handling" is also needed, maybe you add it to this series, so they are reviewed/accepted together). We will set with you online session in the coming 2-3 working days to build the kernel and conduct the test together. Or. -- 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