Page MenuHomeFreeBSD
Differential D19461

Show wiring state of map entries in procstat -v.
ClosedPublic
Actions

Authored by markj on Mar 4 2019, 8:57 PM.
Tags
None
Referenced Files
F138008573: D19461.id54742.diff
Fri, Nov 28, 2:38 AM
Unknown Object (File)
Tue, Nov 25, 4:08 AM
Unknown Object (File)
Sun, Nov 23, 3:43 PM
Unknown Object (File)
Fri, Nov 21, 1:41 AM
Unknown Object (File)
Fri, Nov 21, 1:35 AM
Unknown Object (File)
Fri, Nov 21, 1:32 AM
Unknown Object (File)
Fri, Nov 21, 1:26 AM
Unknown Object (File)
Fri, Nov 21, 1:23 AM
View All 83 Files
Subscribers
imp

Details

Reviewers
alc
kib
Group Reviewers
manpages
Commits
rS344823: Show wiring state of map entries in procstat -v.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 22890
Build 21976: arc lint + arc unit

Event Timeline

markj created this revision.Mar 4 2019, 8:57 PM
Harbormaster completed remote builds in B22885: Diff 54708.Mar 4 2019, 8:57 PM
markj added reviewers: alc, kib.Mar 4 2019, 9:00 PM
kib accepted this revision.Mar 4 2019, 9:23 PM
kib added inline comments.
usr.bin/procstat/procstat.1
666

Wouldn't it be better to say that the range was mlock(2)ed ? Wired is the kernel term.

This revision is now accepted and ready to land.Mar 4 2019, 9:23 PM
markj updated this revision to Diff 54718.Mar 4 2019, 10:20 PM
  • Change man page wording.
  • Call the flag KVME_FLAG_USER_WIRED; we do not set it if the mapping is wired by vslock().
This revision now requires review to proceed.Mar 4 2019, 10:20 PM
Harbormaster completed remote builds in B22890: Diff 54718.Mar 4 2019, 10:20 PM
markj marked an inline comment as done.Mar 4 2019, 10:21 PM
markj added inline comments.
usr.bin/procstat/procstat.1
666

Sure, though we use the term in e.g., top and vmstat -s output.

This revision was not accepted when it landed; it landed in state Needs Review.Mar 5 2019, 7:45 PM
Closed by commit rS344823: Show wiring state of map entries in procstat -v. (authored by markj). · Explain Why
This revision was automatically updated to reflect the committed changes.
markj marked an inline comment as done.
Herald added a reviewer: manpages. · View Herald TranscriptMar 5 2019, 7:45 PM
Herald added a subscriber: imp. · View Herald Transcript

Revision Contents
Changeset List

PathSize
sys/
kern/
2 lines
sys/
1 line
usr.bin/
procstat/
9 lines
8 lines

Diff 54718

View Options

sys/kern/kern_proc.c

Loading...
View Options

sys/sys/user.h

Loading...
View Options

usr.bin/procstat/procstat.1

Loading...
View Options

usr.bin/procstat/procstat_vm.c

Loading...