Differential D56023

pagedaemon.4: Document page daemon
AcceptedPublic
Actions

Authored by 0mp on Sun, Mar 22, 7:04 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Apr 3, 1:59 AM

	Unknown Object (File)
	Mon, Mar 30, 12:45 PM

	Unknown Object (File)
	Sun, Mar 29, 5:10 AM

	Unknown Object (File)
	Sat, Mar 28, 11:36 PM

	Unknown Object (File)
	Fri, Mar 27, 10:52 PM

Subscribers

Details

Reviewers

Group Reviewers

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 71635
Build 68518: arc lint + arc unit

Event Timeline

0mp created this revision.Sun, Mar 22, 7:04 AM

Herald added subscribers: ziaee, imp. · View Herald TranscriptSun, Mar 22, 7:04 AM

0mp requested review of this revision.Sun, Mar 22, 7:04 AM

Harbormaster completed remote builds in B71635: Diff 174104.Sun, Mar 22, 7:04 AM

0mp added a reviewer: manpages.Sun, Mar 22, 7:04 AM

someone who understands the page daemon still needs to review this, but I like it! Thanks for writing this!

share/man/man4/pagedaemon.4
11	please dont quote this because its unnecessary and could confuse people
33	are these tunables or systcls? We have standard sections for both, but we didn't write them down yet (i have a draft).

This revision is now accepted and ready to land.Tue, Mar 31, 2:55 AM

tembun_bk.ru added a subscriber: tembun_bk.ru.Fri, Apr 3, 8:57 PM

tembun_bk.ru added inline comments.

share/man/man4/pagedaemon.4
7	Shouldn't the section here be 4, instead of 9?
69	This page doesn't exist as of now.

ziaee added a parent revision: D18768: Describe FreeBSD's virtual memory in memory(7).Sun, Apr 5, 12:23 AM

ziaee added inline comments.Sun, Apr 5, 12:27 AM

share/man/man4/pagedaemon.4
7	Yeah, nice catch!
43	Please move this parenthetical prose here and on line 51 to the prose in their item bodies. This will increase clarity by separating semantics from prose. This would also make the rendered page one line shorter at default terminal rendering width and eliminate a linter warning.
69	It is proposed in another revision, I linked it as a parent.

Revision Contents
Changeset List

Path

Size

share/

man/

man4/

75 lines

Diff 174104

share/man/man4/pagedaemon.4

Loading...