Re: [PATCH 7/8] Fix refcounting for attribute_container

[James Bottomley <James.Bottomley@HansenPartnership.com>]

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