On Sat, 2017-03-18 at 07:44 -0500, James Bottomley wrote: > On Fri, 2017-03-17 at 16:40 +0000, Bart Van Assche wrote: > > Does your comment mean that you think there is a scenario in which > > scsi_target_block() or scsi_target_unblock() can be called while the > > text area of a SCSI LLD is being released? I have reviewed all the > > callers of these functions but I have not found such a scenario. > > scsi_target_block() and scsi_target_unblock() are either called from > > a SCSI transport layer implementation (FC, iSCSI, SRP) or from a SCSI > > LLD kernel module (snic_disc). All these kernel modules only call > > scsi_target_*block() for resources (rport or SCSI target > > respectively) that are removed before the code area of > > these modules is released. This is why I think that calling > > scsi_target_*block() without increasing the SCSI LLD module reference > > count is safe. > > The transport code is above the HBA module code and in that code > unblock could be racing with module removal. The original premise was > that once the dev/target/host goes into DEL, nothing can call into > queuecommand or get a reference to the device, so nothing halts removal > after that, but you changed that with your code, which is why it's now > unsafe. Hello James, Thank you for having provided more background information about the design goals of the SCSI code. However, regarding scsi_target_*block(), I think it is safe even for transport modules to call these functions without obtaining an additional reference on the SCSI LLD kernel module. The transport implementation won't attempt to block or unblock a SCSI target anymore once unloading of the LLD text has started. SCSI LLDs release the attached transport before unloading of the SCSI LLD kernel module text starts and SCSI transport modules guarantee that scsi_target_*block() won't be called anymore after the transport module has been released. If you do not agree with the above please provide a call sequence for an existing SCSI LLD or transport module that illustrates the race you described. Thanks, Bart.
