On Sun, 2012-01-15 at 09:28 +0000, Bart Van Assche wrote: > On Sat, Jan 14, 2012 at 10:10 PM, David Dillow <dillowda@xxxxxxxx> wrote: > > Patch 17 (srp_transport: Add transport layer recovery support) doesn't > > apply with 'git am'. I haven't investigated why yet, so it may be quite > > simple. > > Hmm, that's strange. As far as I can see what arrived on the > linux-scsi mailing list is identical to the patches I prepared, and > I've just checked that these apply cleanly on top of Linus' tree. By > the way, this patch series is available on github too: > http://github.com/bvanassche/linux/commits/srp-ha/ I suspect it was the local Exchange server substituting spaces for the tabs; git am was upset with the white space damage. If you can put a URL in the cover letter, that will be quite helpful for future reviews. I'm a bit worried about the lack of comments from others about your changes to scsi_transport_srp.*. We really should see some ACKs from the SCSI guys, and while I would think they should go via James Bottomley rather than Roland, I don't think that going through the InfiniBand tree will be a real issue if James is OK with them. However, given the lack of any comments from that side, I wonder if it makes any sense to contain the changes to the initiator for now, and then migrate the common functionality to the transport layer via the SCSI tree as we can get the right people's attention? I'm not particularly happy with that approach, as I'd rather we do it correctly the first time, but I know people would like to see the improvements in the SRP initiator as well. What do others think? -- 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-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html