Re: [PATCH 4/5] parisc: Remove *(.text.*) pattern from the linkerscript

[James Bottomley]

On Mon, 2010-06-14 at 13:38 +0100, Matt Fleming wrote:
> From: Matt Fleming <matthew.fleming@xxxxxxxxxx>
> 
> Drop the pattern from the linker script because the pattern is now
> provided by TEXT_TEXT.
> 
> Signed-off-by: Matt Fleming <matthew.fleming@xxxxxxxxxx>
> ---
>  arch/parisc/kernel/vmlinux.lds.S |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/parisc/kernel/vmlinux.lds.S b/arch/parisc/kernel/vmlinux.lds.S
> index d64a6bb..6784e39 100644
> --- a/arch/parisc/kernel/vmlinux.lds.S
> +++ b/arch/parisc/kernel/vmlinux.lds.S
> @@ -61,7 +61,6 @@ SECTIONS
>  		*(.text.sys_exit)
>  		*(.text.do_sigaltstack)
>  		*(.text.do_fork)
> -		*(.text.*)
>  		*(.fixup)
>  		*(.lock.text)		/* out-of-line lock text */
>  		*(.gnu.warning)


This would destroy all of the named parisc text ordering we do above the
removed line because now you'd have swept up all the function sections
before we get to them, won't it?

The ordering is an execution speed up on 32 bit systems because our
relative jump is so short.

James


--
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/