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)
Wed, Jan 15, 2:22 AM
Unknown Object (File)
Dec 10 2024, 11:19 PM
Unknown Object (File)
Dec 4 2024, 6:08 PM
Unknown Object (File)
Nov 26 2024, 1:32 PM
Unknown Object (File)
Nov 9 2024, 10:10 PM
Unknown Object (File)
Nov 9 2024, 8:48 PM
Unknown Object (File)
Oct 21 2024, 1:52 AM
Unknown Object (File)
Oct 21 2024, 1:52 AM
View All 88 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...