Messages in this thread | | | Date | Sun, 30 Apr 2006 23:13:03 -0700 | From | Andrew Morton <> | Subject | Re: [PATCH 004 of 11] md: Increase the delay before marking metadata clean, and make it configurable. |
| |
Neil Brown <neilb@suse.de> wrote: > > On Sunday April 30, akpm@osdl.org wrote: > > NeilBrown <neilb@suse.de> wrote: > > > > > > > > > When a md array has been idle (no writes) for 20msecs it is marked as > > > 'clean'. This delay turns out to be too short for some real > > > workloads. So increase it to 200msec (the time to update the metadata > > > should be a tiny fraction of that) and make it sysfs-configurable. > > > > > > > > > ... > > > > > > + safe_mode_delay > > > + When an md array has seen no write requests for a certain period > > > + of time, it will be marked as 'clean'. When another write > > > + request arrive, the array is marked as 'dirty' before the write > > > + commenses. This is known as 'safe_mode'. > > > + The 'certain period' is controlled by this file which stores the > > > + period as a number of seconds. The default is 200msec (0.200). > > > + Writing a value of 0 disables safemode. > > > + > > > > Why not make the units milliseconds? Rename this to safe_mode_delay_msecs > > to remove any doubt. > > Because umpteen years ago when I was adding thread-usage statistics to > /proc/net/rpc/nfsd I used milliseconds and Linus asked me to make it > seconds - a much more "obvious" unit. See Email below. > It seems very sensible to me.
That's output. It's easier to do the conversion with output. And I guess one could argue that lots of people read /proc files, but few write to them.
Generally I don't think we should be teaching the kernel to accept pretend-floating-point numbers like this, especially when a) "delay in milliseconds" is such a simple concept and b) it's so easy to go from float to milliseconds in userspace.
Do you really expect that humans (really dumb ones ;)) will be echoing numbers into this file? Or will it mainly be a thing for mdadm to fiddle with? - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |