Page MenuHomeFreeBSD
Differential D24787

Link efi programs with -pie rather than -shared
ClosedPublic
Actions

Authored by arichardson on May 9 2020, 5:55 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 5, 5:10 PM
Unknown Object (File)
Sat, Oct 26, 2:07 PM
Unknown Object (File)
Thu, Oct 24, 4:39 AM
Unknown Object (File)
Oct 5 2024, 4:26 AM
Unknown Object (File)
Oct 1 2024, 4:39 PM
Unknown Object (File)
Sep 19 2024, 6:36 AM
Unknown Object (File)
Sep 6 2024, 3:04 AM
Unknown Object (File)
Aug 28 2024, 5:34 AM
View All 43 Files
Subscribers
bcran
dab
imp
val_packett.cool

Details

Reviewers
emaste
andrew
jhb
brooks
Commits
rGaa2cddc0d21d: Link efi programs with -pie rather than -shared
rS366644: Link efi programs with -pie rather than -shared
Summary

This was causing build failures in CheriBSD where we were passing -pie
already by default.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

arichardson created this revision.May 9 2020, 5:55 PM
Herald added a subscriber: bcran. · View Herald TranscriptMay 9 2020, 5:55 PM
arichardson requested review of this revision.May 9 2020, 5:55 PM
Harbormaster completed remote builds in B31007: Diff 71596.May 9 2020, 5:55 PM
val_packett.cool added a subscriber: val_packett.cool.May 25 2020, 12:56 PM

Looks like this breaks aarch64, I got a "Synchronous exception" after the loader detects filesystems. Rebuilding after reverting the change fixed it.

andrew added a comment.Jun 23 2020, 7:34 AM
In D24787#550414, @greg_unrelenting.technology wrote:

Looks like this breaks aarch64, I got a "Synchronous exception" after the loader detects filesystems. Rebuilding after reverting the change fixed it.

Does this fix it? https://github.com/CTSRD-CHERI/cheribsd/commit/ef7b5749427620d6ee1c9834ea6a264f816e99ea

val_packett.cool added a comment.Jun 23 2020, 12:21 PM
In D24787#560671, @andrew wrote:
In D24787#550414, @greg_unrelenting.technology wrote:

Looks like this breaks aarch64, I got a "Synchronous exception" after the loader detects filesystems. Rebuilding after reverting the change fixed it.

Does this fix it? https://github.com/CTSRD-CHERI/cheribsd/commit/ef7b5749427620d6ee1c9834ea6a264f816e99ea

Yes.

arichardson added a comment.Sep 4 2020, 8:43 AM

@andrew okay to commit this?

andrew added a comment.Sep 10 2020, 2:14 PM

I've committed the arm64 fix in rS365578.

This revision was not accepted when it landed; it landed in state Needs Review.Oct 12 2020, 11:27 AM
Closed by commit rS366644: Link efi programs with -pie rather than -shared (authored by arichardson). · Explain Why
This revision was automatically updated to reflect the committed changes.
arichardson added a commit: rS366644: Link efi programs with -pie rather than -shared.
Herald added a subscriber: imp. · View Herald TranscriptOct 12 2020, 11:27 AM
kevans added a commit: rGaa2cddc0d21d: Link efi programs with -pie rather than -shared.Oct 8 2021, 4:10 PM
Herald added a subscriber: dab. · View Herald TranscriptOct 8 2021, 4:10 PM

Revision Contents
Changeset List

PathSize
head/
stand/
efi/
boot1/
2 lines
loader/
2 lines

Diff 78129

View Options

head/stand/efi/boot1/Makefile

Loading...
View Options

head/stand/efi/loader/Makefile

Loading...