On 12/2/20 11:27 PM, Ming Lei wrote: > BTW, scsi_eh_lock_door() returns void, and it can't be sync because > there may not be any driver tag available. Even though it is available, > the host state isn't running yet, so the command can't be queued to LLD > yet. > > Maybe the above lines should be put after host state is updated to > RUNNING. > > Also changing to NOWAIT can't avoid the issue completely, what if 'none' > is used? Hi Ming, I am considering to drop this patch since the latest version of the SPI DV patch no longer introduces a new blk_mq_freeze_queue() call in the SPI DV code. In other words, any potential issues with scsi_eh_lock_door() are existing issues and are not made worse by my patch series. Thanks, Bart.
