svn commit: r199050 - in head/sys: conf dev/ata

[Kostik Belousov]

On Sun, Nov 08, 2009 at 04:40:27PM +0200, Alexander Motin wrote:
> Kostik Belousov wrote:
> > On Sun, Nov 08, 2009 at 02:33:19PM +0000, Alexander Motin wrote:
> >> Author: mav
> >> Date: Sun Nov  8 14:33:19 2009
> >> New Revision: 199050
> >> URL: http://svn.freebsd.org/changeset/base/199050
> >>
> >> Log:
> >>   Introduce define and kernel option ATA_REQUEST_TIMEOUT to control ATA(4)
> >>   command timeout.
> > 
> > Why is this a compile-time option ? Having to recompile kernel for this
> > is too much, IMHO. It should be loader tunable + sysctl, isn't it.
> 
> As I have told once, I think it is not a thing that should be often
> tuned by end-user, especially in real-time. I just turned it into macro,
Absolutely. I agree with that most users never need this tunable.

> as some people complain about using numerous equal numeric constants.
But, why not be somewhat more friendly to a user that already has a
problam, and force him to recompile the kernel only to check whether
it helps or not ?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 196 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/svn-src-all/attachments/20091108/464db1af/attachment.pgp