Xen devel - Oct 2013 - [PATCH] kobject: show debug info on delayed kobject release

Useful for locating buggy drivers on kernel oops.

It may add dozens of new lines to boot dmesg. DEBUG_KOBJECT_RELEASE is
hopefully only enabled in debug kernels (like maybe the Fedora rawhide
one, or at developers), so being a bit more verbose is likely ok.

CC: Russell King - ARM Linux <linux@arm.linux.org.uk>
CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
---
 lib/kobject.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/kobject.c b/lib/kobject.c
index 669bf19..084f7b1 100644
--- a/lib/kobject.c
+++ b/lib/kobject.c
@@ -592,7 +592,7 @@ static void kobject_release(struct kref *kref)
 {
 	struct kobject *kobj = container_of(kref, struct kobject, kref);
 #ifdef CONFIG_DEBUG_KOBJECT_RELEASE
-	pr_debug("kobject: ''%s'' (%p): %s, parent %p
(delayed)\n",
+	pr_info("kobject: ''%s'' (%p): %s, parent %p
(delayed)\n",
 		 kobject_name(kobj), kobj, __func__, kobj->parent);
 	INIT_DELAYED_WORK(&kobj->release, kobject_delayed_cleanup);
 	schedule_delayed_work(&kobj->release, HZ);
-- 
1.7.10.4

On Wed, Oct 09, 2013 at 09:26:21AM +0800, Fengguang Wu
wrote:
> Useful for locating buggy drivers on kernel oops.
> 
> It may add dozens of new lines to boot dmesg. DEBUG_KOBJECT_RELEASE is
> hopefully only enabled in debug kernels (like maybe the Fedora rawhide
> one, or at developers), so being a bit more verbose is likely ok.
> 
> CC: Russell King - ARM Linux <linux@arm.linux.org.uk>
Acked-by: Russell King <rmk+kernel@arm.linux.org.uk>

Thanks.