linux-kernel - Re: [PATCH 0/1] x86/boot: lld fix

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]

Message-ID: <20200520233357.GA4160490@rani.riverdale.lan>
Date:   Wed, 20 May 2020 19:33:57 -0400
From:   Arvind Sankar <nivedita@...m.mit.edu>
To:     Arvind Sankar <nivedita@...m.mit.edu>
Cc:     clang-built-linux@...glegroups.com, x86@...nel.org,
        Fangrui Song <maskray@...gle.com>,
        Nick Desaulniers <ndesaulniers@...gle.com>,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/1] x86/boot: lld fix

On Wed, May 20, 2020 at 06:56:53PM -0400, Arvind Sankar wrote:
> arch/x86/boot/setup.elf currently has an orphan section .text.startup,
> and lld git as of ebf14d9b6d8b is breaking on 64-bit due to what seems
> to be a change in behavior on orphan section placement (details in patch
> commit message).
> 
> I'm not sure if this was an intentional change in lld, but it seems like
> a good idea to explicitly include .text.startup anyway.
> 
> Arvind Sankar (1):
>   x86/boot: Add .text.startup to setup.ld
> 
>  arch/x86/boot/setup.ld | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> -- 
> 2.26.2
> 

Actually I'm not sure this is a recent change. I updated to the git tip
because on my system (gentoo with gcc-10), lld-10.0.0 segfaults when
trying to link the compressed kernel, and updating to lld git fixed
that. So I previously probably didn't get to the stage where it links
setup.elf.