diff options
author | Mike Travis <travis@sgi.com> | 2008-07-08 17:35:21 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-07-13 13:11:58 -0400 |
commit | 11369f356b66d363a615fde2c5526962f7683674 (patch) | |
tree | 9723013073973c413e7cde295e99221ff40afca3 /arch | |
parent | ce8b06b985ae48f9425de6e4641e77cb3613ef00 (diff) |
x86: change _node_to_cpumask_ptr to return const ptr
* Strengthen the return type for the _node_to_cpumask_ptr to be
a const pointer. This adds compiler checking to insure that
node_to_cpumask_map[] is not changed inadvertently.
Signed-off-by: Mike Travis <travis@sgi.com>
Cc: "akpm@linux-foundation.org" <akpm@linux-foundation.org>
Cc: Yinghai Lu <yhlu.kernel@gmail.com>
Acked-by: Vegard Nossum <vegard.nossum@gmail.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/x86/kernel/setup_percpu.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/arch/x86/kernel/setup_percpu.c b/arch/x86/kernel/setup_percpu.c index 5fc310f746fc..cac68430d31f 100644 --- a/arch/x86/kernel/setup_percpu.c +++ b/arch/x86/kernel/setup_percpu.c | |||
@@ -343,23 +343,23 @@ static const cpumask_t cpu_mask_none; | |||
343 | /* | 343 | /* |
344 | * Returns a pointer to the bitmask of CPUs on Node 'node'. | 344 | * Returns a pointer to the bitmask of CPUs on Node 'node'. |
345 | */ | 345 | */ |
346 | cpumask_t *_node_to_cpumask_ptr(int node) | 346 | const cpumask_t *_node_to_cpumask_ptr(int node) |
347 | { | 347 | { |
348 | if (node_to_cpumask_map == NULL) { | 348 | if (node_to_cpumask_map == NULL) { |
349 | printk(KERN_WARNING | 349 | printk(KERN_WARNING |
350 | "_node_to_cpumask_ptr(%d): no node_to_cpumask_map!\n", | 350 | "_node_to_cpumask_ptr(%d): no node_to_cpumask_map!\n", |
351 | node); | 351 | node); |
352 | dump_stack(); | 352 | dump_stack(); |
353 | return &cpu_online_map; | 353 | return (const cpumask_t *)&cpu_online_map; |
354 | } | 354 | } |
355 | if (node >= nr_node_ids) { | 355 | if (node >= nr_node_ids) { |
356 | printk(KERN_WARNING | 356 | printk(KERN_WARNING |
357 | "_node_to_cpumask_ptr(%d): node > nr_node_ids(%d)\n", | 357 | "_node_to_cpumask_ptr(%d): node > nr_node_ids(%d)\n", |
358 | node, nr_node_ids); | 358 | node, nr_node_ids); |
359 | dump_stack(); | 359 | dump_stack(); |
360 | return (cpumask_t *)&cpu_mask_none; | 360 | return &cpu_mask_none; |
361 | } | 361 | } |
362 | return (cpumask_t *)&node_to_cpumask_map[node]; | 362 | return &node_to_cpumask_map[node]; |
363 | } | 363 | } |
364 | EXPORT_SYMBOL(_node_to_cpumask_ptr); | 364 | EXPORT_SYMBOL(_node_to_cpumask_ptr); |
365 | 365 | ||