diff options
| author | Mike Travis <travis@sgi.com> | 2008-12-16 20:33:57 -0500 |
|---|---|---|
| committer | Mike Travis <travis@sgi.com> | 2008-12-16 20:40:57 -0500 |
| commit | b78936e14ee47b6b2d628501a0eab5270db80132 (patch) | |
| tree | db43c9d6472aa922941e551627d8125984cb389f | |
| parent | 22f65d31b25a320a5246592160bcb102d2791c45 (diff) | |
xen: convert to cpumask_var_t and new cpumask primitives.
Simple change, and eventual space saving when NR_CPUS >> nr_cpu_ids.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Mike Travis <travis@sgi.com>
Cc: Jeremy Fitzhardinge <jeremy@xensource.com>
| -rw-r--r-- | arch/x86/xen/smp.c | 9 | ||||
| -rw-r--r-- | arch/x86/xen/suspend.c | 3 | ||||
| -rw-r--r-- | arch/x86/xen/xen-ops.h | 2 |
3 files changed, 9 insertions, 5 deletions
diff --git a/arch/x86/xen/smp.c b/arch/x86/xen/smp.c index 2cce362c9874..b3a95868839b 100644 --- a/arch/x86/xen/smp.c +++ b/arch/x86/xen/smp.c | |||
| @@ -33,7 +33,7 @@ | |||
| 33 | #include "xen-ops.h" | 33 | #include "xen-ops.h" |
| 34 | #include "mmu.h" | 34 | #include "mmu.h" |
| 35 | 35 | ||
| 36 | cpumask_t xen_cpu_initialized_map; | 36 | cpumask_var_t xen_cpu_initialized_map; |
| 37 | 37 | ||
| 38 | static DEFINE_PER_CPU(int, resched_irq); | 38 | static DEFINE_PER_CPU(int, resched_irq); |
| 39 | static DEFINE_PER_CPU(int, callfunc_irq); | 39 | static DEFINE_PER_CPU(int, callfunc_irq); |
| @@ -192,7 +192,10 @@ static void __init xen_smp_prepare_cpus(unsigned int max_cpus) | |||
| 192 | if (xen_smp_intr_init(0)) | 192 | if (xen_smp_intr_init(0)) |
| 193 | BUG(); | 193 | BUG(); |
| 194 | 194 | ||
| 195 | xen_cpu_initialized_map = cpumask_of_cpu(0); | 195 | if (!alloc_cpumask_var(&xen_cpu_initialized_map, GFP_KERNEL)) |
| 196 | panic("could not allocate xen_cpu_initialized_map\n"); | ||
| 197 | |||
| 198 | cpumask_copy(xen_cpu_initialized_map, cpumask_of(0)); | ||
| 196 | 199 | ||
| 197 | /* Restrict the possible_map according to max_cpus. */ | 200 | /* Restrict the possible_map according to max_cpus. */ |
| 198 | while ((num_possible_cpus() > 1) && (num_possible_cpus() > max_cpus)) { | 201 | while ((num_possible_cpus() > 1) && (num_possible_cpus() > max_cpus)) { |
| @@ -221,7 +224,7 @@ cpu_initialize_context(unsigned int cpu, struct task_struct *idle) | |||
| 221 | struct vcpu_guest_context *ctxt; | 224 | struct vcpu_guest_context *ctxt; |
| 222 | struct desc_struct *gdt; | 225 | struct desc_struct *gdt; |
| 223 | 226 | ||
| 224 | if (cpu_test_and_set(cpu, xen_cpu_initialized_map)) | 227 | if (cpumask_test_and_set_cpu(cpu, xen_cpu_initialized_map)) |
| 225 | return 0; | 228 | return 0; |
| 226 | 229 | ||
| 227 | ctxt = kzalloc(sizeof(*ctxt), GFP_KERNEL); | 230 | ctxt = kzalloc(sizeof(*ctxt), GFP_KERNEL); |
diff --git a/arch/x86/xen/suspend.c b/arch/x86/xen/suspend.c index 2a234db5949b..212ffe012b76 100644 --- a/arch/x86/xen/suspend.c +++ b/arch/x86/xen/suspend.c | |||
| @@ -35,7 +35,8 @@ void xen_post_suspend(int suspend_cancelled) | |||
| 35 | pfn_to_mfn(xen_start_info->console.domU.mfn); | 35 | pfn_to_mfn(xen_start_info->console.domU.mfn); |
| 36 | } else { | 36 | } else { |
| 37 | #ifdef CONFIG_SMP | 37 | #ifdef CONFIG_SMP |
| 38 | xen_cpu_initialized_map = cpu_online_map; | 38 | BUG_ON(xen_cpu_initialized_map == NULL); |
| 39 | cpumask_copy(xen_cpu_initialized_map, cpu_online_mask); | ||
| 39 | #endif | 40 | #endif |
| 40 | xen_vcpu_restore(); | 41 | xen_vcpu_restore(); |
| 41 | } | 42 | } |
diff --git a/arch/x86/xen/xen-ops.h b/arch/x86/xen/xen-ops.h index 9e1afae8461f..c1f8faf0a2c5 100644 --- a/arch/x86/xen/xen-ops.h +++ b/arch/x86/xen/xen-ops.h | |||
| @@ -58,7 +58,7 @@ void __init xen_init_spinlocks(void); | |||
| 58 | __cpuinit void xen_init_lock_cpu(int cpu); | 58 | __cpuinit void xen_init_lock_cpu(int cpu); |
| 59 | void xen_uninit_lock_cpu(int cpu); | 59 | void xen_uninit_lock_cpu(int cpu); |
| 60 | 60 | ||
| 61 | extern cpumask_t xen_cpu_initialized_map; | 61 | extern cpumask_var_t xen_cpu_initialized_map; |
| 62 | #else | 62 | #else |
| 63 | static inline void xen_smp_init(void) {} | 63 | static inline void xen_smp_init(void) {} |
| 64 | #endif | 64 | #endif |
