On Mon, Mar 21 2005, James Bottomley wrote: > On Mon, 2005-03-21 at 14:26 +0100, Jens Axboe wrote: > > scsi_allocate_request() doesn't hold a reference to the device that it > > points to, that is not good. This patch fixes that up. > > Actually, I don't think this is correct. The reference is taken when > the command is attached to a request in the scsi_request_fn function > after first checking that the entity is in a condition to have this > happen.
Where does this happen, specifically? I can verify that this patch solves at least one of the oopses I am seeing with hotplug. IMHO, the reference should be taken when you reference the device (ie ->sr_device is assigned in scsi_allocate_request()). > The problem is that the device could have been torn down (surprise > ejection etc) when some of the routines that call scsi_allocate_request > run. What's the actual problem this is trying to solve? Same problem, SCSI blowing up with a hotplug device. Try with your pendrive, you should have no problem oopsing the kernel. As mentioned earlier in this thread, there are unsolvable reference counting problems between sdev and the request_queue right now. -- Jens Axboe - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
