| From stable-bounces@linux.kernel.org Wed Feb 28 20:14:03 2007 |
| From: Sam Ravnborg <sam@ravnborg.org> |
| Date: Wed, 28 Feb 2007 20:12:31 -0800 |
| Subject: fix section mismatch warning in lockdep |
| To: torvalds@linux-foundation.org |
| Cc: akpm@linux-foundation.org, sam@ravnborg.org, mingo@elte.hu, stable@kernel.org |
| Message-ID: <200703010412.l214CVNl004099@shell0.pdx.osdl.net> |
| |
| |
| From: Sam Ravnborg <sam@ravnborg.org> |
| |
| lockdep_init() is marked __init but used in several places |
| outside __init code. This causes following warnings: |
| $ scripts/mod/modpost kernel/lockdep.o |
| WARNING: kernel/built-in.o - Section mismatch: reference to .init.text:lockdep_init from .text.lockdep_init_map after 'lockdep_init_map' (at offset 0x105) |
| WARNING: kernel/built-in.o - Section mismatch: reference to .init.text:lockdep_init from .text.lockdep_reset_lock after 'lockdep_reset_lock' (at offset 0x35) |
| WARNING: kernel/built-in.o - Section mismatch: reference to .init.text:lockdep_init from .text.__lock_acquire after '__lock_acquire' (at offset 0xb2) |
| |
| The warnings are less obviously due to heavy inlining by gcc - this is not |
| altered. |
| |
| Fix the section mismatch warnings by removing the __init marking, which |
| seems obviously wrong. |
| |
| Signed-off-by: Sam Ravnborg <sam@ravnborg.org> |
| Acked-by: Ingo Molnar <mingo@elte.hu> |
| Signed-off-by: Andrew Morton <akpm@linux-foundation.org> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de> |
| |
| --- |
| kernel/lockdep.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- linux-2.6.20.1.orig/kernel/lockdep.c |
| +++ linux-2.6.20.1/kernel/lockdep.c |
| @@ -2577,7 +2577,7 @@ out_restore: |
| raw_local_irq_restore(flags); |
| } |
| |
| -void __init lockdep_init(void) |
| +void lockdep_init(void) |
| { |
| int i; |
| |