x86: smpboot - wakeup_secondary should be done via __cpuinit section
A caller (do_boot_cpu) already has __cpuinit attribute. Since HOTPLUG_CPU depends on SMP && HOTPLUG it doesn't lead to panic at moment. [ Impact: cleanup ] Signed-off-by: Cyrill Gorcunov <gorcunov@openvz.org> LKML-Reference: <20090418194528.GD25510@lenovo> Signed-off-by: Ingo Molnar <mingo@elte.hu>
This commit is contained in:
committed by
Ingo Molnar
parent
9a2755c356
commit
cece3155d8
@@ -504,7 +504,7 @@ void __inquire_remote_apic(int apicid)
|
|||||||
* INIT, INIT, STARTUP sequence will reset the chip hard for us, and this
|
* INIT, INIT, STARTUP sequence will reset the chip hard for us, and this
|
||||||
* won't ... remember to clear down the APIC, etc later.
|
* won't ... remember to clear down the APIC, etc later.
|
||||||
*/
|
*/
|
||||||
int __devinit
|
int __cpuinit
|
||||||
wakeup_secondary_cpu_via_nmi(int logical_apicid, unsigned long start_eip)
|
wakeup_secondary_cpu_via_nmi(int logical_apicid, unsigned long start_eip)
|
||||||
{
|
{
|
||||||
unsigned long send_status, accept_status = 0;
|
unsigned long send_status, accept_status = 0;
|
||||||
@@ -538,7 +538,7 @@ wakeup_secondary_cpu_via_nmi(int logical_apicid, unsigned long start_eip)
|
|||||||
return (send_status | accept_status);
|
return (send_status | accept_status);
|
||||||
}
|
}
|
||||||
|
|
||||||
static int __devinit
|
static int __cpuinit
|
||||||
wakeup_secondary_cpu_via_init(int phys_apicid, unsigned long start_eip)
|
wakeup_secondary_cpu_via_init(int phys_apicid, unsigned long start_eip)
|
||||||
{
|
{
|
||||||
unsigned long send_status, accept_status = 0;
|
unsigned long send_status, accept_status = 0;
|
||||||
|
Reference in New Issue
Block a user