On Thu, 2017-01-26 at 00:38 -0800, Christoph Hellwig wrote: > On Wed, Jan 25, 2017 at 03:36:12PM -0800, Bart Van Assche wrote: > > The most important changes in this patch series are: > > * ABORT and LUN RESET handling is made synchronous. > > * LUN lookup during ABORT handling is moved from the tcm_qla2xxx > > into the target core. > > * A deadlock in the XCOPY implementation is fixed. > > can you split this up a bit - XCOPY deadlock first, lun lookup > and other cleanups next, rewrite of abort last. > > This is mostly because the abort changes have been rather contentious > in the past and might need longer than usual review. > > Last but not least this is an extremly long and hard to review > series to start with. Hello Christoph, I will reorder the patches in this series if I have to repost this patch series. But please keep in mind that for most patches in this patch series a previous version has been posted before. The four patches that move LUN lookup from the tcm_qla2xxx ABORT handler into the target core had already been posted a few weeks ago. The patches that make ABORT and LUN RESET handling synchronous are a refreshed version of the patch series I started to work on almost two years ago. Several patches with spelling fixes are new but these are trivial. The two patches that fix the XCOPY deadlock are new but follow a traditional pattern: introduce a reference count API and next use that API to move code out of a critical section. Bart.-- 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
