diff options
| author | Stefano Stabellini <stefano.stabellini@eu.citrix.com> | 2010-10-26 12:28:33 -0400 |
|---|---|---|
| committer | Stefano Stabellini <stefano.stabellini@eu.citrix.com> | 2010-10-26 15:33:15 -0400 |
| commit | ea5b8f73933e34d2b47a65284c46d26d49e7edb9 (patch) | |
| tree | 0c7a4b44dc46d2dc66ac6822fde3bf9f12f14bbc | |
| parent | 0e058e527784a9a23f7ed7a73ffafebb53a889da (diff) | |
xen: initialize cpu masks for pv guests in xen_smp_init
Pv guests don't have ACPI and need the cpu masks to be set
correctly as early as possible so we call xen_fill_possible_map from
xen_smp_init.
On the other hand the initial domain supports ACPI so in this case we skip
xen_fill_possible_map and rely on it. However Xen might limit the number
of cpus usable by the domain, so we filter those masks during smp
initialization using the VCPUOP_is_up hypercall.
It is important that the filtering is done before
xen_setup_vcpu_info_placement.
Signed-off-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
| -rw-r--r-- | arch/x86/xen/smp.c | 24 |
1 files changed, 22 insertions, 2 deletions
diff --git a/arch/x86/xen/smp.c b/arch/x86/xen/smp.c index 138676781dd4..834dfeb54e31 100644 --- a/arch/x86/xen/smp.c +++ b/arch/x86/xen/smp.c | |||
| @@ -28,6 +28,7 @@ | |||
| 28 | #include <asm/xen/interface.h> | 28 | #include <asm/xen/interface.h> |
| 29 | #include <asm/xen/hypercall.h> | 29 | #include <asm/xen/hypercall.h> |
| 30 | 30 | ||
| 31 | #include <xen/xen.h> | ||
| 31 | #include <xen/page.h> | 32 | #include <xen/page.h> |
| 32 | #include <xen/events.h> | 33 | #include <xen/events.h> |
| 33 | 34 | ||
| @@ -156,6 +157,25 @@ static void __init xen_fill_possible_map(void) | |||
| 156 | { | 157 | { |
| 157 | int i, rc; | 158 | int i, rc; |
| 158 | 159 | ||
| 160 | if (xen_initial_domain()) | ||
| 161 | return; | ||
| 162 | |||
| 163 | for (i = 0; i < nr_cpu_ids; i++) { | ||
| 164 | rc = HYPERVISOR_vcpu_op(VCPUOP_is_up, i, NULL); | ||
| 165 | if (rc >= 0) { | ||
| 166 | num_processors++; | ||
| 167 | set_cpu_possible(i, true); | ||
| 168 | } | ||
| 169 | } | ||
| 170 | } | ||
| 171 | |||
| 172 | static void __init xen_filter_cpu_maps(void) | ||
| 173 | { | ||
| 174 | int i, rc; | ||
| 175 | |||
| 176 | if (!xen_initial_domain()) | ||
| 177 | return; | ||
| 178 | |||
| 159 | num_processors = 0; | 179 | num_processors = 0; |
| 160 | disabled_cpus = 0; | 180 | disabled_cpus = 0; |
| 161 | for (i = 0; i < nr_cpu_ids; i++) { | 181 | for (i = 0; i < nr_cpu_ids; i++) { |
| @@ -179,6 +199,7 @@ static void __init xen_smp_prepare_boot_cpu(void) | |||
| 179 | old memory can be recycled */ | 199 | old memory can be recycled */ |
| 180 | make_lowmem_page_readwrite(xen_initial_gdt); | 200 | make_lowmem_page_readwrite(xen_initial_gdt); |
| 181 | 201 | ||
| 202 | xen_filter_cpu_maps(); | ||
| 182 | xen_setup_vcpu_info_placement(); | 203 | xen_setup_vcpu_info_placement(); |
| 183 | } | 204 | } |
| 184 | 205 | ||
| @@ -195,8 +216,6 @@ static void __init xen_smp_prepare_cpus(unsigned int max_cpus) | |||
| 195 | if (xen_smp_intr_init(0)) | 216 | if (xen_smp_intr_init(0)) |
| 196 | BUG(); | 217 | BUG(); |
| 197 | 218 | ||
| 198 | xen_fill_possible_map(); | ||
| 199 | |||
| 200 | if (!alloc_cpumask_var(&xen_cpu_initialized_map, GFP_KERNEL)) | 219 | if (!alloc_cpumask_var(&xen_cpu_initialized_map, GFP_KERNEL)) |
| 201 | panic("could not allocate xen_cpu_initialized_map\n"); | 220 | panic("could not allocate xen_cpu_initialized_map\n"); |
| 202 | 221 | ||
| @@ -487,5 +506,6 @@ static const struct smp_ops xen_smp_ops __initdata = { | |||
| 487 | void __init xen_smp_init(void) | 506 | void __init xen_smp_init(void) |
| 488 | { | 507 | { |
| 489 | smp_ops = xen_smp_ops; | 508 | smp_ops = xen_smp_ops; |
| 509 | xen_fill_possible_map(); | ||
| 490 | xen_init_spinlocks(); | 510 | xen_init_spinlocks(); |
| 491 | } | 511 | } |
