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
F167764134: D12764.id34246.diff
Mon, Aug 24, 9:10 AM
Unknown Object (File)
Sun, Aug 23, 12:31 AM
Unknown Object (File)
Sat, Aug 22, 10:59 PM
Unknown Object (File)
Fri, Aug 21, 7:37 AM
Unknown Object (File)
Wed, Aug 19, 5:43 AM
Unknown Object (File)
Mon, Aug 17, 3:22 AM
Unknown Object (File)
Mon, Aug 17, 3:22 AM
Unknown Object (File)
Sat, Aug 15, 4:34 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

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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
head/
sys/
amd64/
amd64/
15 lines
arm/
arm/
7 lines
arm64/
arm64/
6 lines
i386/
i386/
11 lines

Diff 34255

View Options

head/sys/amd64/amd64/pmap.c

Loading...
View Options

head/sys/arm/arm/pmap-v6.c

Loading...
View Options

head/sys/arm64/arm64/pmap.c

Loading...
View Options

head/sys/i386/i386/pmap.c

Loading...