Differential D28370

Disable PIE for MIPS u-boot-loader.
ClosedPublic
Actions

Authored by dgr_semihalf.com on Jan 27 2021, 11:36 AM.

Tags

Referenced Files

	Unknown Object (File)
	Tue, Apr 9, 7:13 AM

	Unknown Object (File)
	Fri, Mar 29, 4:10 AM

	Unknown Object (File)
	Thu, Mar 28, 10:03 AM

	Unknown Object (File)
	Mar 15 2024, 1:44 AM

	Unknown Object (File)
	Feb 23 2024, 7:46 AM

	Unknown Object (File)
	Jan 14 2024, 2:52 AM

	Unknown Object (File)
	Dec 26 2023, 8:57 AM

	Unknown Object (File)
	Dec 24 2023, 4:10 PM

View All 41 Files

Subscribers

Details

Reviewers

mw
kib
imp
emaste
jhibbits
jhb
andrew
manu

Commits

rGafd4a4940790: Disable PIE for MIPS ubldr
rGea2392f2f2b3: Disable PIE for MIPS ubldr
rGcd992885bcdf: Disable PIE for MIPS ubldr

Summary

When performing buildworld for MIPS with PIE enabled, the build fails
with "position-independent code requires '-mabicalls'" message.
-mno-abicalls and -fno-pic flags are explicitly set in MIPS ubldr
makefile, so to work around this problem, set MK_PIE=no for MIPS
ubldr. This change was only build tested.

Problem was found when testing D28328.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dgr_semihalf.com requested review of this revision.Jan 27 2021, 11:36 AM

dgr_semihalf.com created this revision.

dgr_semihalf.com added child revisions: D28328: Enable PIE by default on 64-bit architectures, D27666: Enable ASLR by default for 64-bit executables..

emaste added inline comments.Jan 27 2021, 2:59 PM

stand/mips/uboot/Makefile
47	I think a comment here would be useful (a short version of the review description above indicating that PIE is incompatible with`-mno-abicalls`)

Added a comment indicating that PIE is incompatible with -mno-abicalls. Thanks, @emaste.

mw mentioned this in D28328: Enable PIE by default on 64-bit architectures.Feb 3 2021, 9:56 AM

emaste accepted this revision.Feb 24 2021, 6:50 PM

This revision is now accepted and ready to land.Feb 24 2021, 6:50 PM

Closed by commit rGcd992885bcdf: Disable PIE for MIPS ubldr (authored by mw). · Explain WhyFeb 24 2021, 11:27 PM

This revision was automatically updated to reflect the committed changes.

mw added a commit: rGcd992885bcdf: Disable PIE for MIPS ubldr.

kevans added a commit: rGea2392f2f2b3: Disable PIE for MIPS ubldr.Oct 8 2021, 4:10 PM

Herald added a subscriber: dab. · View Herald TranscriptOct 8 2021, 4:10 PM

mw added a commit: rGafd4a4940790: Disable PIE for MIPS ubldr.Jan 23 2022, 10:26 AM

Herald added a reviewer: manu. · View Herald TranscriptJan 23 2022, 10:26 AM

Revision Contents
Changeset List

Path

Size

stand/

mips/

uboot/

4 lines

Diff 84633

stand/mips/uboot/Makefile

Loading...