On Tue, 2008-03-18 at 14:32 +0100, Hannes Reinecke wrote: > attribute_container_add_device() takes an explicit reference on the > parent device, making it impossible to remove the parent by doing > a simple put_device(). So we'd rather _not_ take a reference here as > attribute_container will be handled explicitly by calls to > attribute_container_remove_device()/_destroy_device() anyway. I really don't think this is right. The attribute container does hold a reference to the device, so we should make it explicit. You could argue that in the new parent indicates class device model there's *already* a reference, so this one is superfluous, but it does help to make the relationship explicit. Regardless, because we have an implicit reference, your comment for the patch can't work (whether we explicitly take the ref or not, you can't expect the device to be unrefereced by the attribute container). James -- 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
