Bug 235085 - [PATCH] Option to make rc.d/sysctl more verbose (again)
Summary: [PATCH] Option to make rc.d/sysctl more verbose (again)
Status: New
Alias: None
Product: Base System
Classification: Unclassified
Component: conf (show other bugs)
Version: CURRENT
Hardware: Any Any
: --- Affects Some People
Assignee: freebsd-bugs (Nobody)
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2019-01-20 17:35 UTC by Oliver Fromme
Modified: 2022-10-17 12:36 UTC (History)
0 users

See Also:
olli: mfc-stable12?
olli: mfc-stable11?


Attachments
Patch to add "sysctl_verbose" rc.conf variable. (1.75 KB, patch)
2019-01-20 17:35 UTC, Oliver Fromme
no flags Details | Diff
Fixed patch (there was a duplicate entry for the manual page). (1.75 KB, patch)
2019-01-20 18:47 UTC, Oliver Fromme
no flags Details | Diff
Fixed version of the patch. Really. (1.44 KB, patch)
2019-01-20 18:50 UTC, Oliver Fromme
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Oliver Fromme freebsd_committer freebsd_triage 2019-01-20 17:35:31 UTC
Created attachment 201290 [details]
Patch to add "sysctl_verbose" rc.conf variable.

Several years ago, the output from sysctl(8) commands inside rc.d/sysctl was visible during boot and logged to console.log (if enabled). That was useful because you could see the default values of the sysctl variables that were changed during boot. For example, I have "vfs.ufs.dirhash_maxmem=33554432" (and other things) in my /etc/sysctl.conf because of recommendations from mailing lists several years ago. I can see the change in my console.log:

Oct  3 13:17:38 hexagon kernel: vfs.ufs.dirhash_maxmem: 26963968 -> 33554432

However, that output was disabled by r179965:

https://svnweb.freebsd.org/base/head/libexec/rc/rc.d/sysctl?r1=174438&r2=179965

Now it is impossible (or very difficult at least) to find out what the default value was. For example, when I update from FreeBSD 11 to 12, maybe the default value of vfs.ufs.dirhash_maxmem changed, so maybe I have to adapt my /etc/sysctl.conf, but I can't easily find out if that's really the case.

The patch attached to this PR introduces a new rc.conf variable "sysctl_verbose". The default is "NO", so the current behavior is not changed. If set to "YES", the output caused by /etc/sysctl.conf is not suppressed, so any changes made to sysctl variables during boot are visible, as has been the case until r179965.

The patch was made for HEAD, but it applies as well to stable/12 and stable/11 (the rc.d/sysctl script is the same on all branches). So MFC should be no problem.
Comment 1 Oliver Fromme freebsd_committer freebsd_triage 2019-01-20 18:47:51 UTC
Created attachment 201292 [details]
Fixed patch (there was a duplicate entry for the manual page).

I'm sorry, the inital attachment contained a duplicated entry for the rc.conf(5) manual page. Fixed in this version of the patch (duplicate entry removed).
Comment 2 Oliver Fromme freebsd_committer freebsd_triage 2019-01-20 18:50:16 UTC
Created attachment 201293 [details]
Fixed version of the patch. Really.
Comment 3 Graham Perrin freebsd_committer freebsd_triage 2022-10-17 12:36:03 UTC
Keyword: 

    patch
or  patch-ready

– in lieu of summary line prefix: 

    [patch]

* bulk change for the keyword
* summary lines may be edited manually (not in bulk). 

Keyword descriptions and search interface: 

    <https://bugs.freebsd.org/bugzilla/describekeywords.cgi>