Page MenuHomeFreeBSD

amd64: do not pass -z rodynamic to ld.bfd when building vdso
ClosedPublic

Authored by kib on Oct 18 2024, 2:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 11, 2:46 AM
Unknown Object (File)
Wed, Mar 5, 6:27 AM
Unknown Object (File)
Feb 7 2025, 4:54 AM
Unknown Object (File)
Feb 5 2025, 5:57 PM
Unknown Object (File)
Jan 29 2025, 6:53 AM
Unknown Object (File)
Jan 26 2025, 6:06 PM
Unknown Object (File)
Jan 25 2025, 8:39 PM
Unknown Object (File)
Jan 24 2025, 5:47 PM
Subscribers

Details

Summary

Apparently newer versions of binutils complain instead of silently ignoring the unknown -z option.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib requested review of this revision.Oct 18 2024, 2:21 PM

Not sure how to make kernel old way (config && make) with cross-toolchain.

Also, what is the idiomatic way to distinguish ld.bfd from ld.lld?

For Makefiles we have bsd.linker.mk that sets LINKER_TYPE, with a similar sort of approach to what you've done here. It uses sed to extract the first line of output from ld -v and then various bits of make magic to examine parts of that.

In ports we check ${/usr/bin/ld:L:tA} == /usr/bin/ld.lld to determine if lld is the default linker (from when it was only so on some archs).

For Makefiles we have bsd.linker.mk that sets LINKER_TYPE, with a similar sort of approach to what you've done here. It uses sed to extract the first line of output from ld -v and then various bits of make magic to examine parts of that.

So I am leaving the check as is. IMO it is not worth additional complications to add sed there.

In ports we check ${/usr/bin/ld:L:tA} == /usr/bin/ld.lld to determine if lld is the default linker (from when it was only so on some archs).

One minor comment update

sys/tools/amd64_vdso.sh
43
This revision is now accepted and ready to land.Oct 18 2024, 5:16 PM
This revision was automatically updated to reflect the committed changes.
kib marked an inline comment as done.