On Thu, 2012-12-13 at 04:24 +0000, Greg KH wrote: > On Wed, Dec 12, 2012 at 09:02:45PM -0700, Toshi Kani wrote: > > On Wed, 2012-12-12 at 15:54 -0800, Greg KH wrote: > > > On Wed, Dec 12, 2012 at 04:17:14PM -0700, Toshi Kani wrote: > > > > Added hotplug.c, which is the hotplug framework code. > > > > > > Again, better naming please. > > > > Yes, I will change it to be more specific, something like > > "sys_hotplug.c". > > Ugh, what's wrong with just a simple "system_bus.c" or something like > that, and then put all of the needed system bus logic in there and tie > the cpus and other sysdev code into that? The issue is that the framework does not provide the system bus structure. This is because the system bus structure is not used for CPU and memory initialization at boot (as I explained in my other email). The framework manages the calling sequence of hotplug operations, which is similar to the boot sequence managed by start_kernel(), kernel_init(), do_initcalls(), etc. In such sense, this file might not be a good fit for drivers/base, but I could not find a better place for it. Thanks, -Toshi -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>