On Thu, 2015-05-21 at 10:30 +0200, Christoph Hellwig wrote: > Looks good, > > Reviewed-by: Christoph Hellwig <hch@xxxxxx> This series has a number of issues that completely breaks all sorts of different things related to TMRs, et al. Not only does it drop special case checks without any reasoning or mention of doing so, it's not thought through at all for anything beyond the hacked up FCoE target usage. Also, Qlogic is working on a patch series to address TMR issues with tcm_qla2xxx post TFO->abort_task conversion, and I'm not going to pull the rug out from under them with this type of change. I would have commented on this already, but am currently busy addressing existing issues within for-next code, and finishing up other v4.2 changes to be posted later today. So that said, I'm not going to consider anything of this nature for v4.2 until everything else stabilizes, and especially not until this type of change is tested on fabrics that people actually care about. --nab -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
