Page MenuHomeFreeBSD
Differential D28213

Reuse the amd64 loader relocation code on arm64
ClosedPublic
Actions

Authored by andrew on Jan 17 2021, 6:22 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Feb 10, 5:53 PM
Unknown Object (File)
Sun, Feb 8, 8:09 AM
Unknown Object (File)
Sat, Feb 7, 9:33 PM
Unknown Object (File)
Sat, Feb 7, 2:52 PM
Unknown Object (File)
Fri, Feb 6, 9:32 AM
Unknown Object (File)
Tue, Feb 3, 6:53 AM
Unknown Object (File)
Sat, Jan 31, 3:26 AM
Unknown Object (File)
Fri, Jan 30, 1:14 PM
View All Files
Subscribers
imp

Details

Reviewers
kib
Group Reviewers
arm64
Commits
rGf6f0b849fb26: Reuse the amd64 loader relocation code on arm64
Summary

There is no need to keep multiple copies of the relocation code. The
amd64 code works on arm64 with a few small changes to relocation types.

Diff Detail

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

Event Timeline

andrew created this revision.Jan 17 2021, 6:22 PM
Herald added a subscriber: imp. · View Herald TranscriptJan 17 2021, 6:22 PM
andrew requested review of this revision.Jan 17 2021, 6:22 PM
Harbormaster completed remote builds in B36300: Diff 82432.Jan 17 2021, 6:22 PM
kib added a comment.Jan 18 2021, 5:45 PM

I must admit it is a weird source before the change. IMO we should split this code into per-arch file/function, not merge arches.

But I do not deal with this code, so this is only an opinion.

This revision was not accepted when it landed; it landed in state Needs Review.Jan 20 2021, 6:33 PM
Closed by commit rGf6f0b849fb26: Reuse the amd64 loader relocation code on arm64 (authored by andrew). · Explain Why
This revision was automatically updated to reflect the committed changes.
andrew added a commit: rGf6f0b849fb26: Reuse the amd64 loader relocation code on arm64.

Revision Contents
Changeset List

PathSize
stand/
common/
35 lines

Diff 82646

View Options

stand/common/reloc_elf.c

Loading...