On Mon, 2005-12-12 at 11:55 -0800, Linus Torvalds wrote: > Indeed, that looks pretty subtle. > > James: Brian's patch looks obviously correct to me (scsi_alloc_sdev() will > have called scsi_sysfs_device_initialize() which will set up the release > function to free the queue). Yes it does ... I'll put it in the rc-fixes tree. > This code has been like that forever, though, which makes me wonder. Can > anybody see what has changed to make the bug trigger? Or is there > something I'm missing? The trigger, based on the failure path has to be a slave_alloc failure of an underlying driver (which isn't that common). This may not be visible in the dmesg traces if anyone has one, because reporting the condition is up to the driver. James - : 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
