On Mon, Sep 08, 2014 at 04:31:01PM -0400, Douglas Gilbert wrote: > stop_all_queued() is doing hrtimer_cancel(), del_timer_sync() > or tasklet_kill() on all the scsi_cmnd objects that are > "in play". Unless another mechanism calls the .eh_abort_handler > entry point reliably on each "in play" command then the module > cannot be removed. That is because some timer expiry callbacks > are pending. scsi_remove_host disabled all queueing of new commands, so all these timers and tasklets will eventually expire or run and allow the removal to complete. Of course this could be sped up by cancelling them, but you don't need the sync version > >Something like the (untested) patch below would do the trick. > >We'd still need Dougs patch for the EH case, though. > > The only other call to stop_all_queued() is from the > .eh_host_reset_handler entry point. True, but you also have stop_queued_cmnd for a abort case which also needs that treatment. -- 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
