Re: [PATCH 3/8] x86: Remove .dynbss from the vDSO linker script

From: Petr Tesarik
Date: Fri Jun 12 2009 - 09:36:11 EST


Petr Tesarik pÃÅe v PÃ 12. 06. 2009 v 15:25 +0200:
> As far as I can see, the .dynbss section does not make any sense
> and cannot even appear in a shared library.
>
> Remove it.

Argh, this commit was wrong. But it changes the context for some of the
following patches. :/ I'll just send the whole patch series again.

Sorry.

Petr Tesarik

> Signed-off-by: Petr Tesarik <ptesarik@xxxxxxx>
> ---
> arch/x86/vdso/vdso-layout.lds.S | 11 ++++-------
> 1 files changed, 4 insertions(+), 7 deletions(-)
>
> diff --git a/arch/x86/vdso/vdso-layout.lds.S b/arch/x86/vdso/vdso-layout.lds.S
> index af3fa61..0386c92 100644
> --- a/arch/x86/vdso/vdso-layout.lds.S
> +++ b/arch/x86/vdso/vdso-layout.lds.S
> @@ -46,15 +46,12 @@ SECTIONS
> } :text :dynamic
>
> .rodata : {
> - *(.rodata*)
> + *(.rodata* .gnu.linkonce.r.*)
> } :text
> .data : {
> - *(.data*)
> + *(.data* .gnu.linkonce.d.*)
> *(.got.plt) *(.got)
> - *(.gnu.linkonce.d.*)
> - *(.bss*)
> - *(.dynbss*)
> - *(.gnu.linkonce.b.*)
> + *(.bss* .gnu.linkonce.b.*)
> }
>
> .altinstructions : {
> @@ -71,7 +68,7 @@ SECTIONS
> . = ALIGN(0x100);
>
> .text : {
> - *(.text*)
> + *(.text* .gnu.linkonce.t.*)
> } :text =0x90909090
> }
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/