[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] changeset 8775

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


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.