powerpc: Link Book-E kernels at the same address as AIM kernels
ClosedPublic
Actions

Authored by jhibbits on Aug 17 2019, 3:44 AM.

Details

Reviewers

luporl
bdragon
nwhitehorn

Commits

rS351245: powerpc: Link Book-E kernels at the same address as AIM kernels

Summary

Reduce the diff between AIM and Book-E even more. This also cleans up
vmparam.h significantly.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhibbits created this revision.Aug 17 2019, 3:44 AM

Harbormaster completed remote builds in B25932: Diff 60928.Aug 17 2019, 3:44 AM

on MPC85XXSPE build:

/usr/src/sys/powerpc/booke/pmap.c:222: warning: integer overflow in expression [-Woverflow]
/usr/src/sys/powerpc/booke/pmap.c:222: warning: overflow in implicit constant conversion [-Woverflow]
/usr/src/sys/powerpc/booke/pmap.c:2148: warning: integer overflow in expression [-Woverflow]

sys/powerpc/booke/pmap.c
222 ↗	(On Diff #60928)	(vm_offset_t)VM_MAXUSER_ADDRESS
2148 ↗	(On Diff #60928)	(vm_offset_t)VM_MAXUSER_ADDRESS

The new KERNBASE allows my RB800 to boot without having to modify it, so that's a definite improvement there.
Will check for regression on G4 / G5 / POWER9 (and maybe test the X5000, if I can finally get uboot loader set up on it so I don't have to futz with mkimg to build test kernels) tomorrow.

Fix 32-bit build.

Harbormaster completed remote builds in B25940: Diff 60957.Aug 18 2019, 2:26 AM

Looks good to me.

This revision is now accepted and ready to land.Aug 19 2019, 1:31 PM

Closed by commit rS351245: powerpc: Link Book-E kernels at the same address as AIM kernels (authored by jhibbits). · Explain WhyAug 20 2019, 1:26 AM

This revision was automatically updated to reflect the committed changes.

jhibbits added a commit: rS351245: powerpc: Link Book-E kernels at the same address as AIM kernels.

Herald added a subscriber: imp. · View Herald TranscriptAug 20 2019, 1:26 AM