diff options
author | Paul Gortmaker <paul.gortmaker@windriver.com> | 2013-06-17 15:51:20 -0400 |
---|---|---|
committer | Tony Luck <tony.luck@intel.com> | 2013-06-24 18:44:27 -0400 |
commit | ccce9bb83ed20bca52f82ff9d7cf889d23a2ec01 (patch) | |
tree | 92603d8bec61c30766dd0ae9e46fff1b6416131e /arch/ia64/sn | |
parent | 69cbc0464d87bfa38c0a4fe4d8bb751c1ec8cedf (diff) |
[IA64] Delete __cpuinit usage from all ia64 users
The __cpuinit type of throwaway sections might have made sense
some time ago when RAM was more constrained, but now the savings
do not offset the cost and complications. For example, the fix in
commit 5e427ec2d0 ("x86: Fix bit corruption at CPU resume time")
is a good example of the nasty type of bugs that can be created
with improper use of the various __init prefixes.
After a discussion on LKML[1] it was decided that cpuinit should go
the way of devinit and be phased out. Once all the users are gone,
we can then finally remove the macros themselves from linux/init.h.
This removes all the ia64 uses of the __cpuinit macros.
[1] https://lkml.org/lkml/2013/5/20/589
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
Signed-off-by: Tony Luck <tony.luck@intel.com>
Diffstat (limited to 'arch/ia64/sn')
-rw-r--r-- | arch/ia64/sn/kernel/setup.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/arch/ia64/sn/kernel/setup.c b/arch/ia64/sn/kernel/setup.c index f82e7b462b7b..53b01b8e2f19 100644 --- a/arch/ia64/sn/kernel/setup.c +++ b/arch/ia64/sn/kernel/setup.c | |||
@@ -192,7 +192,7 @@ void __init early_sn_setup(void) | |||
192 | } | 192 | } |
193 | 193 | ||
194 | extern int platform_intr_list[]; | 194 | extern int platform_intr_list[]; |
195 | static int __cpuinitdata shub_1_1_found; | 195 | static int shub_1_1_found; |
196 | 196 | ||
197 | /* | 197 | /* |
198 | * sn_check_for_wars | 198 | * sn_check_for_wars |
@@ -200,7 +200,7 @@ static int __cpuinitdata shub_1_1_found; | |||
200 | * Set flag for enabling shub specific wars | 200 | * Set flag for enabling shub specific wars |
201 | */ | 201 | */ |
202 | 202 | ||
203 | static inline int __cpuinit is_shub_1_1(int nasid) | 203 | static inline int is_shub_1_1(int nasid) |
204 | { | 204 | { |
205 | unsigned long id; | 205 | unsigned long id; |
206 | int rev; | 206 | int rev; |
@@ -212,7 +212,7 @@ static inline int __cpuinit is_shub_1_1(int nasid) | |||
212 | return rev <= 2; | 212 | return rev <= 2; |
213 | } | 213 | } |
214 | 214 | ||
215 | static void __cpuinit sn_check_for_wars(void) | 215 | static void sn_check_for_wars(void) |
216 | { | 216 | { |
217 | int cnode; | 217 | int cnode; |
218 | 218 | ||
@@ -558,7 +558,7 @@ static void __init sn_init_pdas(char **cmdline_p) | |||
558 | * Also sets up a few fields in the nodepda. Also known as | 558 | * Also sets up a few fields in the nodepda. Also known as |
559 | * platform_cpu_init() by the ia64 machvec code. | 559 | * platform_cpu_init() by the ia64 machvec code. |
560 | */ | 560 | */ |
561 | void __cpuinit sn_cpu_init(void) | 561 | void sn_cpu_init(void) |
562 | { | 562 | { |
563 | int cpuid; | 563 | int cpuid; |
564 | int cpuphyid; | 564 | int cpuphyid; |