aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSatyam Sharma <satyam@infradead.org>2007-10-02 16:39:45 -0400
committerTony Luck <tony.luck@intel.com>2007-10-12 17:13:38 -0400
commitdb6a5cef7f474a5bad476a31f4e4c69a69fab8b1 (patch)
tree9f27269dd89c0e88a5bf9396fd124cc22de27983
parent782e3b3b3804c38d5130c7f21d7ec7bf6709023f (diff)
[IA64] tree-wide: Misc __cpu{initdata, init, exit} annotations
* palinfo.c: palinfo_cpu_notifier is a CPU hotplug notifier_block, and can be marked __cpuinitdata, and the callback function palinfo_cpu_callback() itself can be marked __cpuinit. create_palinfo_proc_entries() is only called from __cpuinit callback or general __init code, therefore a candidate for __cpuinit itself. remove_palinfo_proc_entries() is only called from __cpuinit callback or general __exit code, therefore a candidate for __cpuexit. * salinfo.c: The CPU hotplug notifier_block can be __cpuinitdata. The callback salinfo_cpu_callback() is incorrectly marked __devinit -- it must be __cpuinit instead. * topology.c: cache_sysfs_init() is only called at device_initcall() time so marking it as __cpuinit is wrong and wasteful. It should be unconditionally __init. Also cleanup reference to hotplug notifier callback function from this function and replace with cache_add_dev(), which could also enable us to use other tricks to replace __cpuinit{data} annotations, as recently discussed on this list. cache_shared_cpu_map_setup() is only ever called from __cpuinit-marked functions hence both its definitions (SMP or !SMP) are candidates for __cpuinit itself. Also all_cpu_cache_info can be __cpuinitdata because only referenced from __cpuinit code. Signed-off-by: Satyam Sharma <satyam@infradead.org> Cc: "Luck, Tony" <tony.luck@intel.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Tony Luck <tony.luck@intel.com>
-rw-r--r--arch/ia64/kernel/palinfo.c6
-rw-r--r--arch/ia64/kernel/salinfo.c4
-rw-r--r--arch/ia64/kernel/topology.c12
3 files changed, 11 insertions, 11 deletions
diff --git a/arch/ia64/kernel/palinfo.c b/arch/ia64/kernel/palinfo.c
index 85829e27785c..6ef6ffb943a0 100644
--- a/arch/ia64/kernel/palinfo.c
+++ b/arch/ia64/kernel/palinfo.c
@@ -907,7 +907,7 @@ palinfo_read_entry(char *page, char **start, off_t off, int count, int *eof, voi
907 return len; 907 return len;
908} 908}
909 909
910static void 910static void __cpuinit
911create_palinfo_proc_entries(unsigned int cpu) 911create_palinfo_proc_entries(unsigned int cpu)
912{ 912{
913# define CPUSTR "cpu%d" 913# define CPUSTR "cpu%d"
@@ -968,7 +968,7 @@ remove_palinfo_proc_entries(unsigned int hcpu)
968 } 968 }
969} 969}
970 970
971static int palinfo_cpu_callback(struct notifier_block *nfb, 971static int __cpuinit palinfo_cpu_callback(struct notifier_block *nfb,
972 unsigned long action, void *hcpu) 972 unsigned long action, void *hcpu)
973{ 973{
974 unsigned int hotcpu = (unsigned long)hcpu; 974 unsigned int hotcpu = (unsigned long)hcpu;
@@ -986,7 +986,7 @@ static int palinfo_cpu_callback(struct notifier_block *nfb,
986 return NOTIFY_OK; 986 return NOTIFY_OK;
987} 987}
988 988
989static struct notifier_block palinfo_cpu_notifier = 989static struct notifier_block palinfo_cpu_notifier __cpuinitdata =
990{ 990{
991 .notifier_call = palinfo_cpu_callback, 991 .notifier_call = palinfo_cpu_callback,
992 .priority = 0, 992 .priority = 0,
diff --git a/arch/ia64/kernel/salinfo.c b/arch/ia64/kernel/salinfo.c
index 25cd75f50ab1..779c3cca206c 100644
--- a/arch/ia64/kernel/salinfo.c
+++ b/arch/ia64/kernel/salinfo.c
@@ -574,7 +574,7 @@ static const struct file_operations salinfo_data_fops = {
574 .write = salinfo_log_write, 574 .write = salinfo_log_write,
575}; 575};
576 576
577static int __devinit 577static int __cpuinit
578salinfo_cpu_callback(struct notifier_block *nb, unsigned long action, void *hcpu) 578salinfo_cpu_callback(struct notifier_block *nb, unsigned long action, void *hcpu)
579{ 579{
580 unsigned int i, cpu = (unsigned long)hcpu; 580 unsigned int i, cpu = (unsigned long)hcpu;
@@ -615,7 +615,7 @@ salinfo_cpu_callback(struct notifier_block *nb, unsigned long action, void *hcpu
615 return NOTIFY_OK; 615 return NOTIFY_OK;
616} 616}
617 617
618static struct notifier_block salinfo_cpu_notifier = 618static struct notifier_block salinfo_cpu_notifier __cpuinitdata =
619{ 619{
620 .notifier_call = salinfo_cpu_callback, 620 .notifier_call = salinfo_cpu_callback,
621 .priority = 0, 621 .priority = 0,
diff --git a/arch/ia64/kernel/topology.c b/arch/ia64/kernel/topology.c
index 94ae3c87d828..14261fee5f4d 100644
--- a/arch/ia64/kernel/topology.c
+++ b/arch/ia64/kernel/topology.c
@@ -118,11 +118,11 @@ struct cpu_cache_info {
118 struct kobject kobj; 118 struct kobject kobj;
119}; 119};
120 120
121static struct cpu_cache_info all_cpu_cache_info[NR_CPUS]; 121static struct cpu_cache_info all_cpu_cache_info[NR_CPUS] __cpuinitdata;
122#define LEAF_KOBJECT_PTR(x,y) (&all_cpu_cache_info[x].cache_leaves[y]) 122#define LEAF_KOBJECT_PTR(x,y) (&all_cpu_cache_info[x].cache_leaves[y])
123 123
124#ifdef CONFIG_SMP 124#ifdef CONFIG_SMP
125static void cache_shared_cpu_map_setup( unsigned int cpu, 125static void __cpuinit cache_shared_cpu_map_setup( unsigned int cpu,
126 struct cache_info * this_leaf) 126 struct cache_info * this_leaf)
127{ 127{
128 pal_cache_shared_info_t csi; 128 pal_cache_shared_info_t csi;
@@ -157,7 +157,7 @@ static void cache_shared_cpu_map_setup( unsigned int cpu,
157 &csi) == PAL_STATUS_SUCCESS); 157 &csi) == PAL_STATUS_SUCCESS);
158} 158}
159#else 159#else
160static void cache_shared_cpu_map_setup(unsigned int cpu, 160static void __cpuinit cache_shared_cpu_map_setup(unsigned int cpu,
161 struct cache_info * this_leaf) 161 struct cache_info * this_leaf)
162{ 162{
163 cpu_set(cpu, this_leaf->shared_cpu_map); 163 cpu_set(cpu, this_leaf->shared_cpu_map);
@@ -428,13 +428,13 @@ static struct notifier_block __cpuinitdata cache_cpu_notifier =
428 .notifier_call = cache_cpu_callback 428 .notifier_call = cache_cpu_callback
429}; 429};
430 430
431static int __cpuinit cache_sysfs_init(void) 431static int __init cache_sysfs_init(void)
432{ 432{
433 int i; 433 int i;
434 434
435 for_each_online_cpu(i) { 435 for_each_online_cpu(i) {
436 cache_cpu_callback(&cache_cpu_notifier, CPU_ONLINE, 436 struct sys_device *sys_dev = get_cpu_sysdev((unsigned int)i);
437 (void *)(long)i); 437 cache_add_dev(sys_dev);
438 } 438 }
439 439
440 register_hotcpu_notifier(&cache_cpu_notifier); 440 register_hotcpu_notifier(&cache_cpu_notifier);