Page MenuHomeFreeBSD
Differential D46271

Fix EFI loader reproducibility with lld
ClosedPublic
Actions

Authored by dim on Sun, Aug 11, 7:54 PM.
Tags
None
Referenced Files
F93562590: D46271.diff
Tue, Sep 10, 1:25 PM
Unknown Object (File)
Sun, Sep 8, 4:10 PM
Unknown Object (File)
Wed, Aug 21, 1:35 PM
Unknown Object (File)
Tue, Aug 13, 4:27 PM
Unknown Object (File)
Tue, Aug 13, 1:19 PM
Unknown Object (File)
Tue, Aug 13, 8:57 AM
Unknown Object (File)
Tue, Aug 13, 8:27 AM
Unknown Object (File)
Mon, Aug 12, 1:39 PM
View All 9 Files
Subscribers
bcran
dab
imp

Details

Reviewers
brooks
cperciva
emaste
manu
Commits
rGe664cd1dc5fb: Fix EFI loader reproducibility with lld
rG8a4fd438ef6c: Fix EFI loader reproducibility with lld
rG85434cac6ef2: Fix EFI loader reproducibility with lld
rGf5ce3f4ef562: Fix EFI loader reproducibility with lld
Summary

When the various loaders under stand/efi are built, the resulting
binaries differ over multiple runs, even if WITH_REPRODUCIBLE_BUILD is
used. This is caused by lld multithreading and the custom linker scripts
for the loaders, and affects the following binaries:

  • loader_4th.efi
  • loader_4th.sym
  • loader_4th.sym.full
  • loader_lua.efi
  • loader_lua.sym
  • loader_lua.sym.full
  • loader_simp.efi
  • loader_simp.sym
  • loader_simp.sym.full

Work around this by disabling lld threading for these binaries.

MFC after: 3 days

Diff Detail

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

Event Timeline

dim created this revision.Sun, Aug 11, 7:54 PM
Herald added a reviewer: manu. · View Herald TranscriptSun, Aug 11, 7:54 PM
Herald added subscribers: dab, bcran, imp. · View Herald Transcript
dim requested review of this revision.Sun, Aug 11, 7:54 PM
Harbormaster completed remote builds in B58987: Diff 141996.Sun, Aug 11, 7:54 PM
emaste accepted this revision.Sun, Aug 11, 8:49 PM

OK as a workaround but we ought to report this upstream

This revision is now accepted and ready to land.Sun, Aug 11, 8:49 PM
imp added a comment.EditedSun, Aug 11, 9:54 PM

Looks good, assuming we don't already hsve a LDFLAGS.lld that we add to LDFLAGS like we do for CFLAGS.

cperciva added a comment.Mon, Aug 12, 12:40 AM
In D46271#1055178, @imp wrote:

Looks good, assuming we don't already hsve a LDFLAGS.lld that we add to LDFLAGS like we do for CFLAGS.

We do have LDFLAGS.lld, and using that works (without needing the .if/.endif).

dim updated this revision to Diff 142010.Mon, Aug 12, 11:54 AM

Use LDFLAGS.lld.

This revision now requires review to proceed.Mon, Aug 12, 11:54 AM
Harbormaster completed remote builds in B58990: Diff 142010.Mon, Aug 12, 11:54 AM
dim updated this revision to Diff 142011.Mon, Aug 12, 11:54 AM

Undo arc stupidity.

Harbormaster completed remote builds in B58991: Diff 142011.Mon, Aug 12, 11:54 AM
emaste accepted this revision.Mon, Aug 12, 11:59 AM
This revision is now accepted and ready to land.Mon, Aug 12, 11:59 AM
Closed by commit rGf5ce3f4ef562: Fix EFI loader reproducibility with lld (authored by dim). · Explain WhyMon, Aug 12, 1:32 PM
This revision was automatically updated to reflect the committed changes.
dim added a commit: rGf5ce3f4ef562: Fix EFI loader reproducibility with lld.
dim added a commit: rG85434cac6ef2: Fix EFI loader reproducibility with lld.Thu, Aug 15, 6:34 AM
dim added a commit: rG8a4fd438ef6c: Fix EFI loader reproducibility with lld.Thu, Aug 15, 6:39 AM
dim added a commit: rGe664cd1dc5fb: Fix EFI loader reproducibility with lld.Thu, Aug 15, 7:43 AM

Revision Contents
Changeset List

PathSize
stand/
efi/
loader/
5 lines

Diff 142013

View Options

stand/efi/loader/Makefile

Loading...