stand: use -z norelro when linking EFI components
Needs ReviewPublic
Actions

Authored by emaste on Jan 16 2018, 9:58 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Aug 13, 1:11 PM

	Unknown Object (File)
	Wed, Aug 12, 9:58 AM

	Unknown Object (File)
	Tue, Aug 11, 11:25 PM

	Unknown Object (File)
	Tue, Aug 11, 11:19 PM

	Unknown Object (File)
	Sun, Aug 9, 11:05 PM

	Unknown Object (File)
	Fri, Aug 7, 8:17 PM

	Unknown Object (File)
	Fri, Aug 7, 12:11 AM

	Unknown Object (File)
	Thu, Aug 6, 6:34 AM

View All 90 Files

Subscribers

None

Details

Reviewers

imp
tsoome
andrew
kevans

Summary

Linking armv7 EFI components with lld as the linker failed with:
ld: error: section: .dynamic is not contiguous with other relro sections

ld.bfd defaults to having relro off. explicitly set -z norelro to turn it off for ld.lld as well, since relro is meaningless in our EFI context anyway.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

emaste created this revision.Jan 16 2018, 9:58 PM

imp added inline comments.Jan 16 2018, 10:03 PM

stand/efi/boot1/Makefile
63	any reason these lines can't be hoisted to stand/efi/Makefile.inc?

kevans mentioned this in D13942: stand: Move sections around to fix stand/ build with ld.lld on armv7.Jan 16 2018, 10:32 PM

If the root cause is addressed by D13942 we could abandon this change, even though I think it doesn't hurt to have this in.

Revision Contents
Changeset List

Path

Size

stand/

efi/

boot1/

Makefile

2 lines

loader/

Makefile

2 lines

Diff 38065

View Options

stand/efi/boot1/Makefile