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 10:16:54 +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 8:15 AM, Cyrill Gorcunov <gorcunov@...il.com> wrote:
> 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 ;)
>

Heh, now we have even two patches (exactly the same in contents). So, Ingo,
could you pick up any patch, please? ;-) (i'm in office now without access to
my linux machine,  so I can't send the patch with my and Rafael's sign-offs)
--
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/

Powered by blists - more mailing lists