linux-kernel - Re: 2.6.26-rc1 on x86: ld: warning: dot moved backwards before `.text'

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] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 12 May 2008 08:15:28 +0400
From:	"Cyrill Gorcunov" <gorcunov@...il.com>
To:	"Rafael J. Wysocki" <rjw@...k.pl>
Cc:	"H. Peter Anvin" <hpa@...or.com>,
	"Sam Ravnborg" <sam@...nborg.org>,
	"Mikael Pettersson" <mikpe@...uu.se>,
	"Pavel Machek" <pavel@....cz>, tglx@...utronix.de,
	mingo@...hat.com, linux-kernel@...r.kernel.org
Subject: Re: 2.6.26-rc1 on x86: ld: warning: dot moved backwards before `.text'

On Mon, May 12, 2008 at 1:22 AM, Rafael J. Wysocki <rjw@...k.pl> wrote:
> On Sunday, 11 of May 2008, Cyrill Gorcunov wrote:
>> [H. Peter Anvin - Sun, May 11, 2008 at 11:25:54AM -0700]
>> > Cyrill Gorcunov wrote:
>> >> Hi Peter,
>> >> actually I only doubt about exactly position of this headers - I mean
>> >> meguess it better should be _before_ "_end" not after. Since the others
>> >> script uses (acpi/wakeup_rm.S):
>> >>    wakeup_code_start:
>> >>    .incbin "arch/x86/kernel/acpi/realmode/wakeup.bin"
>> >>    wakeup_code_end:
>> >> and kernel uses these wakeup_code_* marks to save/restore memory I think
>> >> it should be like
>> >>    . = HEADER_OFFSET;
>> >>    .header : {
>> >>            ...
>> >>    }
>> >>    . = ALIGN(16);
>> >>    _end = .;
>> >>    /DISCARD/ ...
>> >
>> > Yes, I believe that's correct.
>> >
>> >     -hpa
>> >
>>
>> Thanks for review, Peter, I'll cook a patch and wait for
>> Pavel's opinion ;-)
>
> I've tested the appended one.  If that's what you mean, we'll ask Ingo to take
> it.
>
> Thanks,
> Rafael
>
>
> ---
>  arch/x86/kernel/acpi/realmode/wakeup.lds.S |   10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
>
> Index: linux-2.6/arch/x86/kernel/acpi/realmode/wakeup.lds.S
> ===================================================================
> --- linux-2.6.orig/arch/x86/kernel/acpi/realmode/wakeup.lds.S
> +++ linux-2.6/arch/x86/kernel/acpi/realmode/wakeup.lds.S
> @@ -12,11 +12,6 @@ ENTRY(_start)
>
>  SECTIONS
>  {
> -       . = HEADER_OFFSET;
> -       .header : {
> -                *(.header)
> -       }
> -
>        . = 0;
>        .text : {
>                 *(.text*)
> @@ -50,6 +45,11 @@ SECTIONS
>                __bss_end = .;
>        }
>
> +       . = HEADER_OFFSET;
> +       .header : {
> +                *(.header)
> +       }
> +
>        . = ALIGN(16);
>        _end = .;
>
>

Thanks Rafael,

it's exactly what I meant - http://lkml.org/lkml/2008/5/11/120 ;)
--
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/