Re: threshold_init_device/kobject_uevent_env oops

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Ingo Molnar <mingo@...>, <jacob.shin@...>
Cc: Yinghai Lu <yhlu.kernel@...>, Linux Kernel Mailing List <linux-kernel@...>, Linus Torvalds <torvalds@...>
Date: Friday, January 25, 2008 - 7:08 pm

On Fri, Jan 25, 2008 at 11:35:56PM +0100, Ingo Molnar wrote:

In looking at this code some more, I'm a bit confused.  We have an array
of kobjects in per_cpu(threshold_banks, cpu)[bank]->kobj

Now the kobject in the struct threshold_bank structure is a "static"
one, one that should govern the lifecycle of the object, yet there is no
release function for it at all.  I don't see a way for it to ever be
properly torn down.

But it's the bank kobjects that are dynamic.  They look to be created
properly, and the life cycle is correct because they are initialized by
the kobject core now correctly.  But which order are things initialized
by the cpu code?

Ideally the banks are created before the blocks, but by the error
message that we have here, I'm not so sure about this.  Can anyone
confirm this order is always correct?

Can someone send me the sysfs 'tree' output of what these kobjects are
supposed to be looking like?

Also, can someone enable CONFIG_KOBJECT_DEBUG and send me the output of
the startup of this code?  That should help explain what order things
are happening it.

Actually the debug output with this oops would be great, it should show
the offending logic pretty well.

thanks,

greg k-h
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
threshold_init_device/kobject_uevent_env oops, Yinghai Lu, (Fri Jan 25, 5:05 pm)
Re: threshold_init_device/kobject_uevent_env oops, Ingo Molnar, (Fri Jan 25, 6:35 pm)
Re: threshold_init_device/kobject_uevent_env oops, Greg KH, (Fri Jan 25, 7:08 pm)
Re: threshold_init_device/kobject_uevent_env oops, Yinghai Lu, (Fri Jan 25, 7:20 pm)
Re: threshold_init_device/kobject_uevent_env oops, Yinghai Lu, (Fri Jan 25, 7:12 pm)
Re: threshold_init_device/kobject_uevent_env oops, Yinghai Lu, (Sat Jan 26, 2:04 am)
Re: threshold_init_device/kobject_uevent_env oops, Yinghai Lu, (Sat Jan 26, 3:08 am)
Re: threshold_init_device/kobject_uevent_env oops, Yinghai Lu, (Sat Jan 26, 5:26 pm)