All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Randy Li" <ayaka@soulik.info>
To: Martin Jansa <Martin.Jansa@gmail.com>
Cc: Andre McCurdy <armccurdy@gmail.com>,
	OE Core mailing list <openembedded-core@lists.openembedded.org>,
	Alexander Kanavin <alex.kanavin@gmail.com>,
	openembedded-devel@lists.openembedded.org
Subject: Re: [OE-core] [PATCH] systemd-boot: use ld.bfd as efi-ld when gold is being used by default with ld-is-gold
Date: Wed, 4 Aug 2021 23:54:53 +0800	[thread overview]
Message-ID: <9D74C891-7B8D-42E1-8A01-5F8258C38D4F@soulik.info> (raw)
In-Reply-To: <CA+chaQd3Mf14rf1Ai-00yNTQjF_83Qzqyc8D47y-UvqmvY4H4Q@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 3745 bytes --]

I found I sent this email to wrong list, let me CC to it.

> On Aug 4, 2021, at 3:28 AM, Martin Jansa <Martin.Jansa@gmail.com> wrote:
> 
> Probably as lld isn't supported as well:
> https://github.com/systemd/systemd/commit/e39288193fcdf3a36dbc49b78e6c9bf86a764e31
> 
You can’t use systemd in Android of course, I decide to ignore those packages won’t be used for Android vendor partition.
>> On Tue, Aug 3, 2021 at 9:19 PM Andre McCurdy <armccurdy@gmail.com> wrote:
>> On Tue, Aug 3, 2021 at 12:04 PM Martin Jansa <Martin.Jansa@gmail.com> wrote:
>> >
>> > * since the upgrade to 249.1 in:
>> >   https://git.openembedded.org/openembedded-core/commit/?id=323ec445dfe22860cd450c303db5ed8fcb4e791c
>> >   the builds with ld-is-gold are failing with:
>> >
>> >   [17/21] Generating linuxx64.elf.stub with a custom command
>> >   FAILED: src/boot/efi/linuxx64.elf.stub
>> >   /OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/systemd-boot/249.1-r0/recipe-sysroot-native/usr/bin/x86_64-oe-linux/x86_64-oe-linux-ld -o src/boot/efi/linuxx64.elf.stub -T /OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/systemd-boot/249.1-r0/recipe-sysroot/usr/lib/elf_x86_64_efi.lds -shared -Bsymbolic -nostdlib -znocombreloc --build-id=sha1 -L /OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/systemd-boot/249.1-r0/recipe-sysroot/usr/lib /OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/systemd-boot/249.1-r0/recipe-sysroot/usr/lib/crt0-efi-x86_64.o src/boot/efi/string-util-fundamental.c.o src/boot/efi/disk.c.o src/boot/efi/graphics.c.o src/boot/efi/measure.c.o src/boot/efi/pe.c.o src/boot/efi/secure-boot.c.o src/boot/efi/util.c.o src/boot/efi/linux.c.o src/boot/efi/splash.c.o src/boot/efi/stub.c.o -lefi -lgnuefi /OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/systemd-boot/249.1-r0/recipe-sysroot/usr/lib/../lib/x86_64-oe-linux/11.2.0/libgcc.a
>> >   /OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/systemd-boot/249.1-r0/recipe-sysroot-native/usr/bin/x86_64-oe-linux/x86_64-oe-linux-ld: internal error in segment_precedes, at ../../gold/layout.cc:3721
>> >   [18/21] Generating boot.c.o with a custom command
>> >   ninja: build stopped: subcommand failed.
>> >   WARNING: exit code 1 from a shell command.
>> >
>> >   as reported in:
>> >   https://lists.openembedded.org/g/openembedded-core/message/154246
>> >
>> >   work around this by using ld.bfd when ld-is-gold is in DISTRO_FEATURES
>> >
>> > Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
>> > ---
>> >  meta/recipes-core/systemd/systemd-boot_249.1.bb | 4 +++-
>> >  1 file changed, 3 insertions(+), 1 deletion(-)
>> >
>> > diff --git a/meta/recipes-core/systemd/systemd-boot_249.1.bb b/meta/recipes-core/systemd/systemd-boot_249.1.bb
>> > index d767b5bf8d..18f4fe55ce 100644
>> > --- a/meta/recipes-core/systemd/systemd-boot_249.1.bb
>> > +++ b/meta/recipes-core/systemd/systemd-boot_249.1.bb
>> > @@ -19,11 +19,13 @@ objcopy = ${@meson_array('OBJCOPY', d)}
>> >  EOF
>> >  }
>> >
>> > +EFI_LD = "${@d.getVar('LD').split()[0]}"
>> > +
>> >  EXTRA_OEMESON += "-Defi=true \
>> >                    -Dgnu-efi=true \
>> >                    -Defi-includedir=${STAGING_INCDIR}/efi \
>> >                    -Defi-libdir=${STAGING_LIBDIR} \
>> > -                  -Defi-ld=${@ d.getVar('LD').split()[0]} \
>> > +                  -Defi-ld=${@bb.utils.contains('DISTRO_FEATURES', 'ld-is-gold', 'ld.bfd', '${EFI_LD}', d)} \
>> 
>> Could this just be set to ld.bfd in all cases?
https://github.com/android/ndk/issues/1288
Android has dropped the support for gold linker and it has been marked as deprecated for a long time.
I just heard that Gstreamer doesn’t support lld linker but I prefer to solve it in Gstreamer.
> 
> 
> 

[-- Attachment #2: Type: text/html, Size: 6010 bytes --]

  reply	other threads:[~2021-08-04 15:54 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1696468844291952.7588@lists.openembedded.org>
2021-08-03 19:04 ` [PATCH] systemd-boot: use ld.bfd as efi-ld when gold is being used by default with ld-is-gold Martin Jansa
2021-08-03 19:19   ` [OE-core] " Andre McCurdy
2021-08-03 19:28     ` Martin Jansa
2021-08-04 15:54       ` Randy Li [this message]
2021-08-04 16:17         ` [oe] " Khem Raj

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=9D74C891-7B8D-42E1-8A01-5F8258C38D4F@soulik.info \
    --to=ayaka@soulik.info \
    --cc=Martin.Jansa@gmail.com \
    --cc=alex.kanavin@gmail.com \
    --cc=armccurdy@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=openembedded-devel@lists.openembedded.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.