On Fri, Apr 5, 2013 at 7:49 PM, Ted Unangst <t...@tedunangst.com> wrote:
> On Fri, Apr 05, 2013 at 15:35, Matthew Dempsky wrote:
>>> - { "rthreads", CTLTYPE_INT }, \
>>> + { "rthreads", 0 }, \
>>
>> Maybe change "rthreads" to "gap" like the other removed sysctl entries?
>
> So my thinking here is that "gap" leaks into sysctl. Compare.
>
> xtu:~> sysctl kern.gap
> sysctl: kern.gap: value is not available
> xtu:~> sysctl kern.gapx
> sysctl: second level name gapx in kern.gapx is invalid
>
> I kind of like the first error message more for deprecated sysctls, as in I'd
> like it to say sysctl: kern.rthreads: value is not available.
Hmm, I disagree. I think "kern.rthreads" should be treated the same
as "kern.sldfjslkdfhj" after your change. The bug here is that
"kern.gap" isn't treated the same as "kern.dslkfjsldkfmn". It's a
minor bug because we don't document the kern.gap name in the sysctl(8)
manpage, so no program or script should be coded to rely on it.
Which reminds me: please update sysctl(3) and sysctl(8) too.
Philip Guenther
