kobj: fix threshold_init_device/kobject_uevent_env oops
authorGreg KH <gregkh@suse.de>
Wed, 30 Jan 2008 12:29:58 +0000 (13:29 +0100)
committerIngo Molnar <mingo@elte.hu>
Wed, 30 Jan 2008 12:29:58 +0000 (13:29 +0100)
commit213eca7f4888e9817e8076cdab6b9f7295c181f6
tree94b88ee10d6748f20d944eebb2447390ae5b62a4
parent85004cc367abc000aa36c0d0e270ab609a68b0cb
kobj: fix threshold_init_device/kobject_uevent_env oops

the logic in this function is just crazy.  It's recursive, but we
can circumvent the creation for the kobject and whole creation of the
threshold_block if some conditions are met.  That's why we see the
allocate_threshold_blocks so many times in the callstack, yet only a few
kobjects created.

Then we blow up in kobject_uevent_env() on the first debug printk.
Which means that we are just passing in garbage.

Man, this is one time that comments in code would have been very nice to
have, and why forward goto's into major code blocks are just evil...

Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
arch/x86/kernel/cpu/mcheck/mce_amd_64.c