On Wed, Feb 06, 2008 at 09:13:33AM -0600, Russ Anderson wrote: > On Tue, Feb 05, 2008 at 04:00:09PM -0800, Luck, Tony wrote: > > > Given that ia64_mca_cpu_init() does not get called in > > > the hot-plug path > > > > Surely it must be ... > > That's what I thought, until testing showed otherwise.
My bad. Additional testing shows ia64_mca_cpu_init() does get called. I suspect my problem was when CONFIG_HOTPLUG_CPU was turned on, not all the files were rebuilt. > The two code paths are setup_arch->cpu_init->ia64_mca_cpu_init > for the boot processor and > start_secondary->cpu_init->ia64_mca_cpu_init for the other cpus > at boot time. In the hot-plug path, __cpu_up -> start_seconday . > > you even have code in it to cope > > with this case. The: > > > > if (__per_cpu_mca[cpu]) { > > > > test can only be true if we are back in this code for a > > second run through if the cpu was taken offline and added > > again. Just like it says in the comment above this test. > > Written on the assumption that the hot-plug code would > eventually call it. It does. The second time through the old memory is re-used. Sorry about the noise... -- Russ Anderson, OS RAS/Partitioning Project Lead SGI - Silicon Graphics Inc [EMAIL PROTECTED] - To unsubscribe from this list: send the line "unsubscribe linux-ia64" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html