Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni via buildroot <buildroot@buildroot.org>
To: Romain Naour <romain.naour@smile.fr>
Cc: buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH] support/testing: test_edk2: bump ATF version to 2.9 for binutils 2.39+ support
Date: Wed, 12 Jul 2023 12:04:48 +0200	[thread overview]
Message-ID: <20230712120449.43a7938d@windsurf> (raw)
In-Reply-To: <20230705143023.233143-1-romain.naour@smile.fr>

On Wed,  5 Jul 2023 16:30:23 +0200
Romain Naour <romain.naour@smile.fr> wrote:

> Following the toolchain-external-arm-aarch64 bump to version 12.2-rel1
> by commit [1], binutils 2.39 is now provided by the toolchain.
> 
> This new binutils version break the ATF build due to new linker warnings as
> reported by the EDK2 test:
> 
>   ld.bfd: warning: bl2.elf has a LOAD segment with RWX permissions
> 
> From [2]
> "Users of GNU ld (BPF) from binutils 2.39+ will observe multiple instaces
>  of a new warning when linking the bl*.elf in the form:
> 
>    ld.bfd: warning: stm32mp1_helper.o: missing .note.GNU-stack section implies executable stack
>    ld.bfd: NOTE: This behaviour is deprecated and will be removed in a future version of the linker
>    ld.bfd: warning: bl2.elf has a LOAD segment with RWX permissions
>    ld.bfd: warning: bl32.elf has a LOAD segment with RWX permissions
> 
>  These new warnings are enbaled by default to secure elf binaries:
>   - https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=ba951afb99912da01a6e8434126b8fac7aa75107
>   - https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=0d38576a34ec64a1b4500c9277a8e9d0f07e6774
> "
> 
> Bump the ATF custom version to 2.9 for binutils 2.39+ support.
> 
> [1] 90404e84c1c9d65b4f38cc3ebc30ef6a4da314cb
> [2] https://github.com/ARM-software/arm-trusted-firmware/commit/1f49db5f25cdd4e43825c9bcc0575070b80f628c
> 
> Fixes:
> https://gitlab.com/buildroot.org/buildroot/-/jobs/4581325484
> 
> Signed-off-by: Romain Naour <romain.naour@smile.fr>
> ---
>  support/testing/tests/boot/test_edk2.py | 2 ++
>  1 file changed, 2 insertions(+)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

      reply	other threads:[~2023-07-12 10:05 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-07-05 14:30 [Buildroot] [PATCH] support/testing: test_edk2: bump ATF version to 2.9 for binutils 2.39+ support Romain Naour
2023-07-12 10:04 ` Thomas Petazzoni via buildroot [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230712120449.43a7938d@windsurf \
    --to=buildroot@buildroot.org \
    --cc=romain.naour@smile.fr \
    --cc=thomas.petazzoni@bootlin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox