Scott Soule Cheloha <[email protected]> wrote:

> CVSROOT:      /cvs
> Module name:  src
> Changes by:   [email protected] 2020/01/21 17:17:46
> 
> Modified files:
>       sys/scsi       : scsi_base.c 
> 
> Log message:
> scsi_delay(): sleep without lbolt
> 
> If we want to sleep for a multiple of seconds we can do that without
> involving lbolt.
> 
> This may cause some paths to sleep longer than they have on average,
> as sleeping on lbolt wakes you up within one second, not after one
> second.  If this is a problem we will need to shorten the intervals
> given to scsi_delay().

Well... It means some codepaths (particularily those with very small
sleeps) have been sleeping far less than expected.  Like you look
in a manual and it says "sleep 1/100th of a second", and you code for
that, but lbolt fires early.

So this change makes it more correct: undersleep is a bug.

Reply via email to