diff options
| author | Greg Kroah-Hartman <gregkh@suse.de> | 2008-03-05 21:52:27 -0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2008-03-05 21:52:27 -0800 |
| commit | f8f36cdab19251174a20c40767ddbf3cb5007839 (patch) | |
| tree | 956b3a505937db3dbd534bcc384a7a0d01fa44c9 /pending | |
| parent | 1bdd35aafab740b2cc36135bc31703a8afe652f8 (diff) | |
| download | patches-f8f36cdab19251174a20c40767ddbf3cb5007839.tar.gz | |
kobject debugobjects patch
Diffstat (limited to 'pending')
| -rw-r--r-- | pending/kobjects-hook-them-up-to-the-debugobjects-infrastructure.patch | 89 |
1 files changed, 89 insertions, 0 deletions
diff --git a/pending/kobjects-hook-them-up-to-the-debugobjects-infrastructure.patch b/pending/kobjects-hook-them-up-to-the-debugobjects-infrastructure.patch new file mode 100644 index 00000000000000..b8667578c4c0d1 --- /dev/null +++ b/pending/kobjects-hook-them-up-to-the-debugobjects-infrastructure.patch @@ -0,0 +1,89 @@ +From: Greg Kroah-Hartman <gregkh@suse.de> +Subject: kobjects: hook them up to the debugobjects infrastructure + +I think this is right, but it's hard to tell... + +Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de> + +--- + lib/kobject.c | 53 +++++++++++++++++++++++++++++++++++++++++++++++++++++ + 1 file changed, 53 insertions(+) + +--- a/lib/kobject.c ++++ b/lib/kobject.c +@@ -18,8 +18,59 @@ + #include <linux/stat.h> + #include <linux/slab.h> + #include <linux/kallsyms.h> ++#include <linux/debugobjects.h> + #include <asm-generic/sections.h> + ++#ifdef CONFIG_DEBUG_OBJECTS ++static int kobject_fixup_init(void *addr, enum debug_obj_state state) ++{ ++ struct kobject *kobj = addr; ++ ++ switch (state) { ++ case ODEBUG_STATE_ACTIVE: ++ printk("kobject: '%s' (%p): is being initialized after " ++ "it was already active\n", kobject_name(kobj), kobj); ++ return 1; ++ default: ++ return 0; ++ } ++} ++ ++static int kobject_fixup_destroy(void *addr, enum debug_obj_state state) ++{ ++ struct kobject *kobj = addr; ++ ++ switch (state) { ++ case ODEBUG_STATE_ACTIVE: ++ printk("kobject: '%s' (%p): is being destroyed yet it is " ++ "still active\n", kobject_name(kobj), kobj); ++ return 1; ++ default: ++ return 0; ++ } ++} ++ ++static struct debug_obj_descr kobject_debug_descr = { ++ .name = "kobjects", ++ .fixup_init = kobject_fixup_init, ++ .fixup_destroy = kobject_fixup_destroy, ++}; ++ ++static void debug_kobject_init(struct kobject *kobj) ++{ ++ debug_object_init(kobj, &kobject_debug_descr); ++} ++ ++static void debug_kobject_destroy(struct kobject *kobj) ++{ ++ debug_object_destroy(kobj, &kobject_debug_descr); ++} ++ ++#else ++static inline void debug_kobject_init(struct kobject *kobj) { } ++static inline void debug_kobject_destroy(struct kobject *kobj) { } ++#endif ++ + #ifdef CONFIG_X86_32 + static int ptr_in_range(void *ptr, void *start, void *end) + { +@@ -202,6 +253,7 @@ static void kobject_init_internal(struct + { + if (!kobj) + return; ++ debug_kobject_init(kobj); + kref_init(&kobj->kref); + INIT_LIST_HEAD(&kobj->entry); + kobj->state_in_sysfs = 0; +@@ -599,6 +651,7 @@ static void kobject_cleanup(struct kobje + pr_debug("kobject: '%s' (%p): %s\n", + kobject_name(kobj), kobj, __FUNCTION__); + ++ debug_kobject_destroy(kobj); + if (t && !t->release) + pr_debug("kobject: '%s' (%p): does not have a release() " + "function, it is broken and must be fixed.\n", |