On Thu, 2010-09-16 at 19:26 -0400, Chetan Loke wrote: > On Thu, Sep 16, 2010 at 6:35 PM, Nicholas A. Bellinger > <nab@xxxxxxxxxxxxxxx> wrote: > > > and LLDs have adopted over the years. The changes involved here are really > > quite straight forward, but please note that none of this code has been tested > > with actual hardware yet, and is intended for generating comments for the relivent > > SCSI LLD driver maintainers and other interested folks. > > > > NO. Why not try testing it w/ an abort storm first else we will > seriously end up screwing the filesystems.... The only real issue here would be an immediate struct Scsi_Host->host_lock dead lock for those LLDs that still use legacy unlock() > do_work() -> lock() in their SHT->queuecommand() caller. This is *not* going to happen under heavy load with a bunch of outstanding WRITEs, but immediate the first couple of times that scsi_dispatch_cmd() gets called, so there is really little to no fear of filesystem corruption with these patches. Anyways, please feel free to test them yourself with an TMR ABORT storm if you so desire.. Best, --nab -- 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