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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Wed, 29 Apr 2009 13:34:54 +0200
From:	Ingo Molnar <mingo@...e.hu>
To:	linux-tip-commits@...r.kernel.org
Cc:	linux-kernel@...r.kernel.org, hpa@...or.com, mingo@...hat.com,
	torvalds@...ux-foundation.org, sam@...nborg.org, tabbott@....EDU,
	tglx@...utronix.de
Subject: Re: [tip:x86/kbuild] x86, vmlinux.lds: fix relocatable symbols


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

> Commit-ID:  fd0731944333db6e9e91b6954c6ef95f4b71ab04
> Gitweb:     http://git.kernel.org/tip/fd0731944333db6e9e91b6954c6ef95f4b71ab04
> Author:     Ingo Molnar <mingo@...e.hu>
> AuthorDate: Wed, 29 Apr 2009 12:56:58 +0200
> Committer:  Ingo Molnar <mingo@...e.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@...nborg.org>
> Cc: Tim Abbott <tabbott@....EDU>
> Cc: Linus Torvalds <torvalds@...ux-foundation.org>
> LKML-Reference: <20090429105056.GA28720@...nus.ravnborg.org>
> Signed-off-by: Ingo Molnar <mingo@...e.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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/