[PATCH u-boot 16/39] build: use thin archives instead of incremental linking

[Marek Behún]

On Mon, 8 Mar 2021 19:32:10 +0800
Bin Meng <bmeng.cn at gmail.com> wrote:

> On Mon, Mar 8, 2021 at 7:18 PM Marek Behun <marek.behun at nic.cz> wrote:
> >
> > On Mon, 8 Mar 2021 18:44:58 +0800
> > Bin Meng <bmeng.cn at gmail.com> wrote:
> >  
> > > Could you investigate why?  
> >
> > I could, but I don't understand why exactly I should
> > - Linux is also using --whole-archive
> > - it is working
> > - I have other things I would like to work on
> >
> > Maybe you could look into this? :)  
> 
> Yes, I can look into this. I wonder if you already knew this which
> could save some time as this is a normal review process, asking for
> clarifications if something isn't clear.

Bin, CI is failing without the --whole-archive option.

What is interesting is that the binaries build successfully, but
testing them fails :)

You can try this (with and without the --whole-archive options) (note
that this is without LTO)
  make qemu_arm_defconfig
  CROSS_COMPILE=arm-compiler- make -j8
  qemu-system-arm -M virt -nographic \
    -netdev user,id=net0,tftp=$(pwd) \
    -device e1000,netdev=net0 -device virtio-rng-pci \
    -bios u-boot.bin -serial mon:stdio

With --whole-archive option this boots successfully into U-Boot,
without --whole-archive it just hangs.

Marek