svn commit: r280308 - head/sys/fs/devfs

[Konstantin Belousov]

On Sat, Mar 21, 2015 at 11:00:09PM +1100, Kubilay Kocak wrote:
> On 21/03/2015 12:14 PM, Xin LI wrote:
> > Author: delphij
> > Date: Sat Mar 21 01:14:11 2015
> > New Revision: 280308
> > URL: https://svnweb.freebsd.org/changeset/base/280308
> > 
> > Log:
> >   Disable timestamping on devfs read/write operations by default.
> >   
> >   Currently we update timestamps unconditionally when doing read or
> >   write operations.  This may slow things down on hardware where
> >   reading timestamps is expensive (e.g. HPET, because of the default
> >   vfs.timestamp_precision setting is nanosecond now) with limited
> >   benefit.
> >   
> >   A new sysctl variable, vfs.devfs.dotimes is added, which can be
> >   set to non-zero value when the old behavior is desirable.
> 
> Perhaps *devfs_timestamp_update* is a better name. thoughts?
> 
> If something shorter is desired: devfs_ts_update might be ok, though I
> prefer the former.
> 
> Let's not forget these are user visible system control knobs, not just
> code variables, though I do understand the desire to shorten them, and
> that we do have descriptions for them ... mostly :)

should it be
vfs.devfs.devfs_ts_devfs_ts_devfs_update_devfs
so that user is _absolutely_ sure that it is devfs related, if 'devfs'
used only one time in the MIB prevents him from realizing this ?