diff options
| author | Satyam Sharma <satyam@infradead.org> | 2007-10-17 12:04:35 -0400 |
|---|---|---|
| committer | Thomas Gleixner <tglx@inhelltoy.tec.linutronix.de> | 2007-10-17 14:15:49 -0400 |
| commit | 7b38493501b76e56f481925ed74ee5ca0f4c6183 (patch) | |
| tree | 6ac83113f1c775d59bebe2c53da06076c3fd1ecf /arch/x86/kernel/vsyscall-sysenter_32.S | |
| parent | f6855f7fb2e22c5675d292dd445ee3c1fc09ac18 (diff) | |
x86: intel_cacheinfo misc section annotation fixes
cache_shared_cpu_map_setup() and cache_remove_shared_cpu_map()
are functions called from another function that is __cpuinit. But the
!CONFIG_SMP empty-body stubs of these functions are unconditionally
marked __init, which is actively wrong, and will lead to oops. But we
never saw this oops, because they always managed to get inlined in their
callsites, by virtue of being empty-body stubs! They should still be
__cpuinit, of course.
assocs[], levels[] and types[] are only referenced from function that is
__cpuinit. So these are candidates for being marked __cpuinitdata.
[akpm@linux-foundation.org: build fix]
Signed-off-by: Satyam Sharma <satyam@infradead.org>
Cc: Andi Kleen <ak@suse.de>
Cc: Venkatesh Pallipadi <venkatesh.pallipadi@intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'arch/x86/kernel/vsyscall-sysenter_32.S')
0 files changed, 0 insertions, 0 deletions