Page MenuHomeFreeBSD

Add many sysctl descriptions in kern
ClosedPublic

Authored by antranigv_freebsd.am on Feb 19 2020, 5:39 PM.
Tags
None
Referenced Files
F83090659: D23759.id69075.diff
Mon, May 6, 3:48 AM
Unknown Object (File)
Fri, May 3, 12:34 PM
Unknown Object (File)
Fri, May 3, 12:20 AM
Unknown Object (File)
Mon, Apr 29, 3:26 AM
Unknown Object (File)
Sat, Apr 27, 5:10 PM
Unknown Object (File)
Wed, Apr 24, 8:42 PM
Unknown Object (File)
Feb 6 2024, 4:06 PM
Unknown Object (File)
Feb 1 2024, 2:47 PM

Details

Reviewers
kaktus
Group Reviewers
docs
Commits
rS358548: sys/: Document few more sysctls.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 29496
Build 27366: arc lint + arc unit

Event Timeline

sys/kern/kern_exec.c
135

Use style(9) similar to the entry before, the documentation string should be in a separate line, using proper indentation of 4 spaces.

sys/kern/kern_shutdown.c
170

This should be four spaces and doc sting in a new line.

sys/kern/kern_synch.c
98

Here too.

sys/kern/sched_4bsd.c
438

And too.

sys/kern/sched_ule.c
3130

And too.

sys/kern/subr_kobj.c
71

And too :-)

jhb added inline comments.
sys/kern/kern_exec.c
127

Perhaps "Location of process' ps_strings structure"

132

Perhaps "Top of process stack" (usrstack is the top, not the bottom of the stack, and even then we store various things on the stack such as argv[] and environment arrays before the actual stack used by main())

135

Perhaps "Stack memory permissions". This has the same value as the "prot" argument to mmap (bitmask of PROT_* constants), and the description of "prot" in mmap(2) is "permissions"

sys/kern/kern_synch.c
98

Might be nice to say what it is used for? I think it might be the scaling factor used for load average values, but haven't looked to see how userland (e.g. top or ps) uses this.

sys/kern/kern_umtx.c
243

s/Maximal/Maximum/? maybe s/one/each/ as well

antranigv_freebsd.am marked 4 inline comments as done.

addressing jhb's comments

This revision is now accepted and ready to land.Mar 2 2020, 3:28 PM
This revision was automatically updated to reflect the committed changes.