Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 15 Nov 2010 14:03:36 +1100 (EST)
From:      Bruce Evans <brde@optusnet.com.au>
To:        Bruce Cran <brucec@freebsd.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r215281 - head/sys/kern
Message-ID:  <20101115135506.J1683@besplex.bde.org>
In-Reply-To: <201011140609.oAE69oqd072190@svn.freebsd.org>
References:  <201011140609.oAE69oqd072190@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, 14 Nov 2010, Bruce Cran wrote:

> Log:
>  Add some descriptions to sys/kern sysctls.

> Modified: head/sys/kern/kern_tc.c
> ==============================================================================
> --- head/sys/kern/kern_tc.c	Sun Nov 14 05:05:41 2010	(r215280)
> +++ head/sys/kern/kern_tc.c	Sun Nov 14 06:09:50 2010	(r215281)
> @@ -103,7 +103,7 @@ SYSCTL_NODE(_kern_timecounter, OID_AUTO,
>
> static int timestepwarnings;
> SYSCTL_INT(_kern_timecounter, OID_AUTO, stepwarnings, CTLFLAG_RW,
> -    &timestepwarnings, 0, "");
> +    &timestepwarnings, 0, "Log time steps");

Another problem (other than style bugs) with these and some old descriptions
is inconsistent descriptions of boolean variables.  I like always starting
with "Enable".

> @@ -768,7 +768,7 @@ pps_event(struct pps_state *pps, int eve
>  */
>
> static int tc_tick;
> -SYSCTL_INT(_kern_timecounter, OID_AUTO, tick, CTLFLAG_RD, &tc_tick, 0, "");
> +SYSCTL_INT(_kern_timecounter, OID_AUTO, tick, CTLFLAG_RD, &tc_tick, 0, "Approximate number of hardclock ticks in a millisecond");

It can't possibly be that if a hardclock tick is much larger than a
millisecond, and this doesn't describe its purpose.  It is actually a
rate limiter for timecounter updates, to avoid the natural rate of once
per hardclock tick if that is considered too frequent.  Thus it has nothing
to do with milliseconds except once per millisecond is considered frequent
enough.  It is the number of hardclock ticks between timercounter updates.

Bruce



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20101115135506.J1683>