Page MenuHomeFreeBSD
Differential D20857

[PPC64] pseries: fix realmaxaddr calculation
ClosedPublic
Actions

Authored by luporl on Jul 4 2019, 7:07 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 26, 3:48 PM
Unknown Object (File)
Thu, Oct 16, 6:19 PM
Unknown Object (File)
Thu, Oct 16, 6:19 PM
Unknown Object (File)
Thu, Oct 16, 6:19 PM
Unknown Object (File)
Thu, Oct 16, 6:18 PM
Unknown Object (File)
Oct 16 2025, 7:58 AM
Unknown Object (File)
Sep 28 2025, 8:13 PM
Unknown Object (File)
Sep 27 2025, 5:14 AM
View All Files
Subscribers
alfredo
imp

Details

Reviewers
jhibbits
bdragon
nwhitehorn
Commits
rS349885: [PPC64] pseries: fix realmaxaddr calculation
Summary

On POWER9/pseries, QEMU passes several regions of memory,
instead of a single region containing all memory, as the
code was expecting.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 25202
Build 23882: arc lint + arc unit

Event Timeline

luporl created this revision.Jul 4 2019, 7:07 PM
Harbormaster completed remote builds in B25202: Diff 59405.Jul 4 2019, 7:07 PM
bdragon added a comment.Jul 7 2019, 5:21 PM

Makes sense to me. That's the same sort of thing that chrp_mem_regions() is doing to determine it for its own purposes.

jhibbits added a comment.Jul 7 2019, 6:08 PM

Looks good overall. Just a couple nits..

sys/powerpc/aim/slb.c
419

This looks to be a different patch altogether. I don't think it belongs in this change.

sys/powerpc/pseries/platform_chrp.c
151

realmaxaddr = MAX(off, realmaxaddr) should do the same thing.

luporl added inline comments.Jul 8 2019, 12:34 PM
sys/powerpc/aim/slb.c
419

Ok, I'll commit this part separately.

sys/powerpc/pseries/platform_chrp.c
151

Right, I'll change this part to use MAX.

luporl updated this revision to Diff 59533.Jul 8 2019, 12:40 PM
luporl edited the summary of this revision. (Show Details)
  • Addressed jhibbits' comments
Harbormaster completed remote builds in B25244: Diff 59533.Jul 8 2019, 12:40 PM
luporl marked 2 inline comments as done.Jul 8 2019, 12:41 PM
jhibbits accepted this revision.Jul 8 2019, 4:24 PM
This revision is now accepted and ready to land.Jul 8 2019, 4:24 PM
Closed by commit rS349885: [PPC64] pseries: fix realmaxaddr calculation (authored by luporl). · Explain WhyJul 10 2019, 1:36 PM
This revision was automatically updated to reflect the committed changes.
luporl added a commit: rS349885: [PPC64] pseries: fix realmaxaddr calculation.
Herald added a subscriber: imp. · View Herald TranscriptJul 10 2019, 1:36 PM

Revision Contents
Changeset List

PathSize
sys/
powerpc/
aim/
1 line
pseries/
10 lines

Diff 59405

View Options

sys/powerpc/aim/slb.c

Loading...
View Options

sys/powerpc/pseries/platform_chrp.c

Loading...