Differential D50856

pf.4/pfsync.4: Separate sysctl/tunables >> SYNOPSIS
ClosedPublic
Actions

Authored by ziaee on Sat, Jun 14, 6:01 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Jun 28, 4:32 PM

	Unknown Object (File)
	Fri, Jun 27, 5:37 PM

	Unknown Object (File)
	Fri, Jun 27, 3:30 PM

	Unknown Object (File)
	Thu, Jun 26, 10:43 AM

	Unknown Object (File)
	Thu, Jun 26, 12:08 AM

	Unknown Object (File)
	Wed, Jun 25, 8:36 PM

	Unknown Object (File)
	Tue, Jun 24, 6:14 PM

	Unknown Object (File)
	Tue, Jun 24, 5:30 PM

View All 29 Files

Subscribers

Details

Reviewers

kp
ivy
carlavilla
mhorne

Group Reviewers

Commits

rGdca2ab32e831: pf.4/pfsync.4: Separate sysctl/tunables >> SYNOPSIS

Summary

MFC after: 3 days

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 64873
Build 61756: arc lint + arc unit

Event Timeline

ziaee created this revision.Sat, Jun 14, 6:01 PM

Herald added a subscriber: imp. · View Herald TranscriptSat, Jun 14, 6:01 PM

ziaee requested review of this revision.Sat, Jun 14, 6:01 PM

Harbormaster completed remote builds in B64869: Diff 157035.Sat, Jun 14, 6:01 PM

kp added inline comments.Sat, Jun 14, 6:35 PM

share/man/man4/pf.4
46	filter_local is not a tunable, it can be changed at runtime. As is request_maxcount. rdr_srcport_rewrite_tries isn't user configurable at all.
share/man/man4/pfsync.4
39	pfsync_buckets is a tunable, so sysctl.conf is the wrong place for it.

ziaee retitled this revision from pf.4/pfsync.4: Add sysctls to SYNOPSIS to pf.4/pfsync.4: Separate sysctl/tunables >> SYNOPSIS.Sat, Jun 14, 6:57 PM

thanks! i separated them out into the sections we usually use. maybe adds a lot of bloat?

Harbormaster completed remote builds in B64873: Diff 157040.Sat, Jun 14, 6:58 PM

update per discussion with kp@

Harbormaster completed remote builds in B64874: Diff 157041.Sat, Jun 14, 7:07 PM

mention sysctl.conf, might as well be precise

Harbormaster completed remote builds in B64875: Diff 157042.Sat, Jun 14, 7:08 PM

kp added inline comments.Sat, Jun 14, 7:34 PM

share/man/man4/pf.4
92	tunables?
93	This is the wrong way around. filter_local and request_maxcount are sysctl runtime tunable, the others (states_hashsize, source_nodes_hashsize, ...) are tunables and can only be set through the loader.
107	net.pf.rdr_srcport_rewrite_tries is listed in pf.conf.5 as well. We should probably delete that one as part of this patch (or in a separate patch to delete both mentions) too. (For reference, this was introduced in 339a1977c32414f3d23733504955245ca6f3802d, but without the actual sysctl.)

Spun off the removal into a parent commit.

Harbormaster completed remote builds in B64877: Diff 157048.Sat, Jun 14, 9:19 PM

ziaee added a parent revision: D50859: pf.4/pf.conf.5: Remove unused sysctl.Sat, Jun 14, 9:19 PM

kp accepted this revision.Mon, Jun 16, 11:44 AM

This revision is now accepted and ready to land.Mon, Jun 16, 11:44 AM

Closed by commit rGdca2ab32e831: pf.4/pfsync.4: Separate sysctl/tunables >> SYNOPSIS (authored by ziaee). · Explain WhyWed, Jun 25, 11:22 PM

This revision was automatically updated to reflect the committed changes.

ziaee added a commit: rGdca2ab32e831: pf.4/pfsync.4: Separate sysctl/tunables >> SYNOPSIS.

Revision Contents
Changeset List

Path

Size

share/

man/

man4/

41 lines

30 lines

Diff 157040

share/man/man4/pf.4

Loading...

share/man/man4/pfsync.4

Loading...