On 2011-05-25 21:13, Linus Torvalds wrote: > On Wed, May 25, 2011 at 12:02 PM, Jens Axboe <jaxboe@xxxxxxxxxxxx> wrote: >> >> This is before you get that far, it's actually oopsing on inserting the >> request on sdev->sdev_queue that is now NULL. The prep state checking >> happens when sr/sd pulls the request off the queue for processing. > > Ok, please just add a "sdev_alive(sdev)" helper that checks for sdev > && sdev->sdev_queue. Apparently the issue isn't "this device is > SDEV_DEL, so don't send commands". Apparently the issue is literally > "don't oops". Yep, will do. > Or maybe the prep state checking could be moved earlier. It seems > stupid to have a "can I do this command" function that is run too late > to actually catch the problems.. I don't think we can move it earlier, we essentially have to do the check at both ends. The "normal" IO path would see that the queue is dead and error the IO, but this is an internal setup that sets up the request and then adds it to the stored queue pointer. So it needs to check this state one way or the other. I think the above fix with the sdev_alive() will be good enough. -- Jens Axboe -- 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
