Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 27 Oct 2009 23:28:49 +0200
From:      Alexander Motin <mav@FreeBSD.org>
To:        "M. Warner Losh" <imp@bsdimp.com>
Cc:        svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org
Subject:   Re: svn commit: r198487 - head/sys/dev/ata
Message-ID:  <4AE76611.4030104@FreeBSD.org>
In-Reply-To: <20091027.150719.-1219478921.imp@bsdimp.com>
References:  <200910261123.n9QBNfJY087028@svn.freebsd.org> <20091027.150719.-1219478921.imp@bsdimp.com>

next in thread | previous in thread | raw e-mail | index | archive | help
M. Warner Losh wrote:
> In message: <200910261123.n9QBNfJY087028@svn.freebsd.org>
>             Alexander Motin <mav@FreeBSD.org> writes:
> : Author: mav
> : Date: Mon Oct 26 11:23:41 2009
> : New Revision: 198487
> : URL: http://svn.freebsd.org/changeset/base/198487
> : 
> : Log:
> :   Round timeout up when converting CAM milliseconds to ATA seconds.
> 
> This implements ceil(timeout) rather than round(timeout).  Is that
> intended?

Yes. It is better to turn 400ms to 1s rather then 0. Usually timeouts
aren't so short to make it important, but I have seen such case once, so
it is possible to predict another.

-- 
Alexander Motin



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