stand/elf: Only support swapping headers on powerpc.
ClosedPublic
Actions

Authored by imp on Sep 14 2022, 7:50 PM.

Details

Reviewers

manu
kevans
tsoome

Commits

rG61c9a00f8e5c: stand/elf: Only support swapping headers on powerpc.
rGc0ecae78abbe: stand/elf: Only support swapping headers on powerpc.

Summary

Powerpc is currently the only architecture that we support more than one
endian. It's the only one that benefits from this swapping, so restrict
the code to there. This saves about 1k in the i386 BIOS loader.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

imp created this revision.Sep 14 2022, 7:50 PM

Herald added a reviewer: manu. · View Herald TranscriptSep 14 2022, 7:50 PM

Herald added a subscriber: dab. · View Herald Transcript

imp requested review of this revision.Sep 14 2022, 7:50 PM

Harbormaster completed remote builds in B47393: Diff 110569.Sep 14 2022, 7:50 PM

imp added reviewers: kevans, tsoome.Sep 14 2022, 7:53 PM

imp added a parent revision: D36576: stand/geli: Allocate saved_keys instead of making it static.

This revision was not accepted when it landed; it landed in state Needs Review.Sep 16 2022, 3:54 PM

Closed by commit rGc0ecae78abbe: stand/elf: Only support swapping headers on powerpc. (authored by imp). · Explain Why

This revision was automatically updated to reflect the committed changes.

imp added a commit: rGc0ecae78abbe: stand/elf: Only support swapping headers on powerpc..

imp added a commit: rG61c9a00f8e5c: stand/elf: Only support swapping headers on powerpc..Jan 24 2023, 10:18 PM

Revision Contents
Changeset List

Path

Size

stand/

common/

load_elf.c

16 lines

Diff 110651

View Options

stand/elf: Only support swapping headers on powerpc.ClosedPublicActions