[Xen-devel] changeset 8775

["Jan Beulich" <JBeulich@xxxxxxxxxx>]

While this changeset appears to finally address the cpu_possible_map problems 
(according to other reports on this list)
I wonder if this isn't incomplete. With prefill_possible_map() now being called 
from setup_arch() (why can't it be
called at the same point as native calls it when CONFIG_HOTPLUG_CPU?), the 
immediately following call to
smp_prepare_boot_cpu() (from start_kernel) will destroy this information again. 
With the effort of not setting up
per-CPU information for impossible CPUs (some of this exists in our internal 
trees, not sure how much has been posted to
mainline) this would be prone to break again soon.
Also, some of other code seems then superfluous or even ill:
- drivers/xen/core/smpboot.c: smp_prepare_cpus() should not initialize 
cpu_possible_map again, but should instead have
its main loop controlled by this bit vector (and VCPUOP_is_up doesn't need to 
be called then anymore, too)
- arch/{i386,x86_64}/kernel/mpparse-xen.c: the setting of num_processors was 
and is completely out of sync with the
real set of VCPUs used; I even wonder what purpose the whole file still serves

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel