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
F158918152: D12764.diff
Sun, Jun 7, 5:46 PM
Unknown Object (File)
Sat, May 30, 10:02 PM
Unknown Object (File)
Sat, May 30, 8:57 PM
Unknown Object (File)
Sat, May 30, 7:16 PM
Unknown Object (File)
Fri, May 29, 3:08 AM
Unknown Object (File)
Mon, May 18, 12:42 AM
Unknown Object (File)
Sat, May 9, 6:01 PM
Unknown Object (File)
Sat, May 9, 2:19 PM
View All 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...