On Sat, 25 May 2013 11:55:58 +0200, Hannes Reinecke wrote: > > > It should be possible to move the code into scsi_lib and just have > small hooks for the individual transports to use this. We have done something a bit tasteless and unconditionally enabled your new error handling for all scsi drivers. That solved a rather serious problem for us and the subset of scsi drivers we actually exercise seems to be doing fine. Maybe we could add a hook into scsi_times_out() that does the unconditional thing if users opt-in by setting CONFIG_I_MIGHT_BREAK_STUFF or so. We also have a scsi error injector now that should allow testing your patches on any hardware, including kvm. On a trial run the system eventually rebooted (likely through a hung task timeout, haven't checked) with the old error handling and lost a single drive with your new code. Awesome! So my vote is for ignoring the remaining bits and pushing forward. Your code is too useful to wait another half year. And we should brush up the error injector and send it out as well. Testing your code on cheap hardware would be nice. Jörn -- He who knows others is wise. He who knows himself is enlightened. -- Lao Tsu -- 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
