arm64: simplify physmap usage in pmap_bootstrap()
Needs ReviewPublic
Actions

Authored by mhorne on Thu, Jun 27, 6:29 PM.

Details

Reviewers

andrew
markj
manu

Summary

The subr_physmem.c facility provides guarantees about non-empty ranges.
Push this into pmap_bootstrap_dmap() and simplify; the lowest physical
memory range is always in the first item.

While here, use the raw value of physmap_idx.

No functional change intended.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 58380
Build 55268: arc lint + arc unit

Event Timeline

mhorne created this revision.Thu, Jun 27, 6:29 PM

Herald added a reviewer: manu. · View Herald TranscriptThu, Jun 27, 6:29 PM

Herald added subscribers: emaste, imp. · View Herald Transcript

mhorne requested review of this revision.Thu, Jun 27, 6:29 PM

Harbormaster completed remote builds in B58375: Diff 140312.Thu, Jun 27, 6:29 PM

mhorne added inline comments.Thu, Jun 27, 6:33 PM

sys/arm64/arm64/pmap.c
1218	@markj your recent change d03e1ffbea6 coincided with mine, but I believe there is a minor bug here.

markj added inline comments.Thu, Jun 27, 7:27 PM

sys/arm64/arm64/pmap.c
1218	Sigh, I missed this. :( Would you commit this part on its own, with a "Fixes:" tag? Please consider it reviewed by me. If you don't have time, I can take care of it later today.