Page MenuHomeFreeBSD

[PowerPC] Remove obsolete MK_LOADER_FORCE_LE

Authored by bdragon on Mon, Sep 14, 10:34 PM.



In D12421, the ability to compile stand/ in little-endian was added, with the intention to extend loader.kboot to run in Petitboot.

However, no further work was done, as the kernel then gained self-execution capabilities as Petitboot was taught to load FreeBSD kernels directly.

The FreeBSD installer on powerpc64 (on POWER8 and POWER9) uses /boot/etc/kboot.conf instead of loader.

As this option does nothing but cause stand/ to be miscompiled and actively causes confusion, remove it.

(I have a functioning petitboot loader in my local tree, however, it turned out to be quite inconvient to use due to the current petitboot plugin design so I put it on hold.)

Diff Detail

rS FreeBSD src repository
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

bdragon created this revision.Mon, Sep 14, 10:34 PM
bdragon requested review of this revision.Mon, Sep 14, 10:34 PM
bdragon updated this revision to Diff 77033.Mon, Sep 14, 10:42 PM
bdragon edited the summary of this revision. (Show Details)

Properly rerun makeman.

Not a powerpc expert, but seems reasonable to me - no objection here.

jhibbits accepted this revision.Tue, Sep 15, 2:00 AM
This revision is now accepted and ready to land.Tue, Sep 15, 2:00 AM
imp accepted this revision.Tue, Sep 15, 2:07 AM
This revision was automatically updated to reflect the committed changes.