ARM: kernel: Fix section mismatches caused by commit bba0859a99
authorJon Medhurst (Tixy) <tixy@linaro.org>
Tue, 10 Sep 2013 04:01:44 +0000 (12:01 +0800)
committerAlex Shi <alex.shi@linaro.org>
Wed, 11 Sep 2013 14:52:56 +0000 (22:52 +0800)
commit98ff17449dd2a9f3bb664120a95ada62d1984acb
treeb9caf0e2510a7cd9519954cddbd5f744b5581297
parent19c4b0592637d5948e8eb1f93b5f40c2f556f903
ARM: kernel: Fix section mismatches caused by commit bba0859a99

Commit bba0859a99 (arm: versatile: don't mark pen as __INIT)
introduced the following section mismatch warnings:

WARNING: vmlinux.o(.text+0x18208): Section mismatch in reference from the variable pen to the function .cpuinit.text:secondary_startup()
WARNING: vmlinux.o(.text+0x18210): Section mismatch in reference from the variable pen to the variable .cpuinit.data:pen_release

The first is handled by removing __cpuinitdata from pen_release. This
also fixes and potential bug because the issue commit bba0859a99 was
aimed at fixing meant a CPU not known to the kernel could be spinning
forever in versatile_secondary_startup and polling this pen_release
variable, so it is important its memory isn't discarded and reused after
boot.

The second section mismatch warning is removed by taking __CPUINIT
away from before secondary_startup.

Signed-off-by: Jon Medhurst <tixy@linaro.org>
Signed-off-by: Alex Shi <alex.shi@linaro.org>
arch/arm/kernel/head.S
arch/arm/kernel/smp.c