Re: [tip:x86/kbuild] x86, vmlinux.lds: fix relocatable symbols

[Ingo Molnar <mingo@elte.hu>]

* tip-bot for Ingo Molnar <mingo@elte.hu> wrote:

> Commit-ID:  fd0731944333db6e9e91b6954c6ef95f4b71ab04
> Gitweb:     http://git.kernel.org/tip/fd0731944333db6e9e91b6954c6ef95f4b71ab04
> Author:     Ingo Molnar <mingo@elte.hu>
> AuthorDate: Wed, 29 Apr 2009 12:56:58 +0200
> Committer:  Ingo Molnar <mingo@elte.hu>
> CommitDate: Wed, 29 Apr 2009 13:01:18 +0200
> 
> x86, vmlinux.lds: fix relocatable symbols
> 
> __init_begin/_end symbols should be inside sections as well,
> otherwise the relocatable kernel gets confused when freeing
> init sections in the wrong place.
> 
> [ Impact: fix bootup crash ]
> 
> Cc: Sam Ravnborg <sam@ravnborg.org>
> Cc: Tim Abbott <tabbott@MIT.EDU>
> Cc: Linus Torvalds <torvalds@linux-foundation.org>
> LKML-Reference: <20090429105056.GA28720@uranus.ravnborg.org>
> Signed-off-by: Ingo Molnar <mingo@elte.hu>
> 
> 
> ---
>  arch/x86/kernel/vmlinux.lds.S |    7 +++++--
>  1 files changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/x86/kernel/vmlinux.lds.S b/arch/x86/kernel/vmlinux.lds.S
> index 0bdbaa5..4c85b2e 100644
> --- a/arch/x86/kernel/vmlinux.lds.S
> +++ b/arch/x86/kernel/vmlinux.lds.S
> @@ -255,8 +255,8 @@ SECTIONS
>  
>  	/* Init code and data - will be freed after init */
>  	. = ALIGN(PAGE_SIZE);
> -	__init_begin = .; /* paired with __init_end */
>  	.init.text : AT(ADDR(.init.text) - LOAD_OFFSET) {
> +		__init_begin = .; /* paired with __init_end */
>  		_sinittext = .;
>  		INIT_TEXT
>  		_einittext = .;
> @@ -346,8 +346,11 @@ SECTIONS
>  #endif
>  
>  	. = ALIGN(PAGE_SIZE);
> +
>  	/* freed after init ends here */
> -	__init_end = .;
> +	.init.end : AT(ADDR(.init.end) - LOAD_OFFSET) {
> +		__init_end = .;
> +	}
>  
>  #ifdef CONFIG_X86_64
>  	.data_nosave : AT(ADDR(.data_nosave) - LOAD_OFFSET) {

For the record - this fixed the crash i reported. No further 
problems so far, with about 200 test iterations on 7 boxes.

	Ingo