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
F157594074: D19461.id54708.diff
Sat, May 23, 5:03 AM
F157534411: D19461.diff
Fri, May 22, 12:58 PM
F157534404: D19461.diff
Fri, May 22, 12:58 PM
Unknown Object (File)
Tue, May 19, 12:44 PM
Unknown Object (File)
Tue, May 19, 4:46 AM
Unknown Object (File)
Tue, May 19, 3:54 AM
Unknown Object (File)
Sat, May 16, 9:29 AM
Unknown Object (File)
Thu, May 14, 1:12 PM
View All Files
Subscribers
imp

Details

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

Diff Detail

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

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 ↗(On Diff #54708)

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 ↗(On Diff #54708)

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
head/
sys/
kern/
2 lines
sys/
1 line
usr.bin/
procstat/
9 lines
8 lines

Diff 54742

View Options

head/sys/kern/kern_proc.c

Loading...
View Options

head/sys/sys/user.h

Loading...
View Options

head/usr.bin/procstat/procstat.1

Loading...
View Options

head/usr.bin/procstat/procstat_vm.c

Loading...