]> git.baikalelectronics.ru Git - kernel.git/commit
Kobject: drop child->parent ref at unregistration
authorAlan Stern <stern@rowland.harvard.edu>
Mon, 19 Nov 2007 15:53:40 +0000 (10:53 -0500)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 25 Jan 2008 04:40:36 +0000 (20:40 -0800)
commit81b9c39279aead455dc279e56af840085bc51ba8
tree059011e094839a633597c2e88afc40f779894438
parent85395aff306d0f90352c6f9f47f01352dc4dd9da
Kobject: drop child->parent ref at unregistration

This patch (as1015) reverts changes that were made to the driver core
about four years ago.  The intent back then was to avoid certain kinds
of invalid memory accesses by leaving kernel objects allocated as long
as any of their children were still allocated.  The original and
correct approach was to wait only as long as any children were still
_registered_; that's what this patch reinstates.

This fixes a problem in the SCSI core made visible by the class_device
to regular device conversion: A reference loop (scsi_device holds
reference to request_queue, which is the child of a gendisk, which is
the child of the scsi_device) prevents the data structures from being
released, even though they are deregistered okay.

It's possible that this change will cause a few bugs to surface,
things that have been hidden for several years.  They can be fixed
easily enough by having the child device take an explicit reference to
the parent whenever needed.

Signed-off-by: Alan Stern <stern@rowland.harvard.edu>
Cc: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
lib/kobject.c