On 08/07/05 10:59, Alan Stern wrote: > Mike: > > What sort of synchronization is there between scsi_remove_host and the > error-handler thread? Offhand I can see two possible problems, depending > on how the LLD is written: > > 1. The error handler asks the LLD to perform a reset just as the LLD > calls scsi_remove_host as part of rmmod. scsi_remove_host > returns, the LLD's remove method returns, and the LLD is unloaded > from memory while its reset routine is executing. > > 2. The error handler asks the LLD to perform a reset just before the > LLD calls scsi_remove_host. The LLD's remove method can't return > until the reset completes (otherwise the reset would be executing > code that is about to be unloaded, as in (1)). But the reset > can't get started until it acquires the host adapter's device > semaphore, which is owned by the rmmod thread. > > (1) causes an oops and (2) causes a deadlock. Note that (1) can be > avoided by making scis_remove_host wait until the error handler thread has > exited, but that will make (2) much more likely to happen. Hmm, this doesn't sound good. I'm thinking, if the LLDD knows that it's being unloaded, then it can return the EH reset with success de facto. > In general, there needs to be some way for the error handler to prevent > the LLD from being unloaded. I don't know what the best answer is. Has > anyone thought about this? Two ways: 1) Let one entity be the active one and the other the passive one, always. That is, let the LLDD send an "event" and then let the Managing layer decide what to do. 2) Tack a kobject, or kset to each struct you have and set the parent pointer accordingly to how the struct objects relate to each other. So in effect, the kobj relationship would represent how those objects actually relate to each other in the kernel. Advantage you get is the automatic kref. ... or something like this. ;-) Luben - : 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
