loader/efi: Only include interpreter's linker script
ClosedPublic
Actions

Authored by imp on Feb 24 2024, 6:41 PM.

Details

Reviewers

kevans
tsoome
manu

Commits

rG7cc0ac566600: loader/efi: Only include interpreter's linker script
rGf5f08e41aa57: loader/efi: Only include interpreter's linker script

Summary

For safety, only include the interpreter's linker script. Note that the
simple loader doesn't have one, but it's not an error to copy a ELF
section that does not exist. No functional change, however.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 56217
Build 53105: arc lint + arc unit

Event Timeline

imp created this revision.Feb 24 2024, 6:41 PM

Herald added a reviewer: manu. · View Herald TranscriptFeb 24 2024, 6:41 PM

Herald added subscribers: dab, bcran. · View Herald Transcript

imp requested review of this revision.Feb 24 2024, 6:41 PM

Harbormaster completed remote builds in B56217: Diff 134942.Feb 24 2024, 6:41 PM

imp added a parent revision: D44063: loader/ficl: Rename the ficl compile set to X4th_compile_set.Feb 24 2024, 6:41 PM

tsoome accepted this revision.Feb 24 2024, 6:50 PM

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

kevans accepted this revision.Feb 24 2024, 11:37 PM

Closed by commit rGf5f08e41aa57: loader/efi: Only include interpreter's linker script (authored by imp). · Explain WhyFeb 25 2024, 1:00 AM

This revision was automatically updated to reflect the committed changes.

imp added a commit: rGf5f08e41aa57: loader/efi: Only include interpreter's linker script.

imp added a commit: rG7cc0ac566600: loader/efi: Only include interpreter's linker script.Apr 16 2024, 8:14 PM

Revision Contents
Changeset List

Path

Size

stand/

efi/

loader/

Makefile

2 lines

Diff 134942

View Options

loader/efi: Only include interpreter's linker scriptClosedPublicActions