Page MenuHomeFreeBSD
Differential D12764

Fix the VM_NRESERVLEVEL == 0 build
ClosedPublic
Actions

Authored by markj on Oct 23 2017, 12:00 AM.
Tags
None
Referenced Files
F133557130: D12764.id34255.diff
Sun, Oct 26, 3:43 PM
F133553671: D12764.id34255.diff
Sun, Oct 26, 3:04 PM
Unknown Object (File)
Sun, Oct 26, 9:00 AM
Unknown Object (File)
Sun, Oct 26, 4:21 AM
Unknown Object (File)
Sat, Oct 25, 7:46 PM
Unknown Object (File)
Sat, Oct 25, 7:46 PM
Unknown Object (File)
Sat, Oct 25, 1:14 PM
Unknown Object (File)
Sat, Oct 18, 5:15 PM
View All 100 Files
Subscribers
andrew
emaste

Details

Reviewers
alc
kib
Commits
rS324920: Fix the VM_NRESERVLEVEL == 0 build.
Summary

Add VM_NRESERVLEVEL guards in the pmaps that implement transparent
superpage promotion using reservations.

Test Plan

Verified that a VM_NRESERVELEVEL == 0 amd64 kernel boots.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 12183
Build 12482: arc lint + arc unit

Event Timeline

markj created this revision.Oct 23 2017, 12:00 AM
Herald added subscribers: emaste, andrew. · View Herald TranscriptOct 23 2017, 12:00 AM
Harbormaster completed remote builds in B12182: Diff 34244.Oct 23 2017, 12:00 AM
markj edited the test plan for this revision. (Show Details)Oct 23 2017, 12:00 AM
markj added reviewers: alc, kib.
alc added inline comments.Oct 23 2017, 1:01 AM
sys/amd64/conf/BHYVE
5 ↗(On Diff #34244)

I presume that this is an unintended leftover from your testing.

markj updated this revision to Diff 34246.Oct 23 2017, 2:20 AM

Remove unintentonal private kernel config update

markj marked an inline comment as done.Oct 23 2017, 2:21 AM
markj added inline comments.
sys/amd64/conf/BHYVE
5 ↗(On Diff #34244)

Fixed, thanks.

alc accepted this revision.Oct 23 2017, 4:50 AM
This revision is now accepted and ready to land.Oct 23 2017, 4:50 AM
kib accepted this revision.Oct 23 2017, 7:48 AM
Closed by commit rS324920: Fix the VM_NRESERVLEVEL == 0 build. (authored by markj). · Explain WhyOct 23 2017, 3:34 PM
This revision was automatically updated to reflect the committed changes.
markj marked an inline comment as done.

Revision Contents
Changeset List

PathSize
sys/
amd64/
amd64/
15 lines
arm/
arm/
7 lines
arm64/
arm64/
6 lines
i386/
i386/
11 lines

Diff 34246

View Options

sys/amd64/amd64/pmap.c

Loading...
View Options

sys/arm/arm/pmap-v6.c

Loading...
View Options

sys/arm64/arm64/pmap.c

Loading...
View Options

sys/i386/i386/pmap.c

Loading...