SCSI_DELAY cleanup

Alexander Best arundel at freebsd.org
Tue Oct 19 10:34:32 UTC 2010


On Mon Oct 18 10, Matthew Jacob wrote:
>  What problem are you solving by this change?

code cleanup.

the scsi delay value currently defaults to 2000ms. however that doesn't make
sense, since on almost all platforms it gets set to 5000ms in the default
config. what's the purpose of having a default value, if it is much more often
overwritten than actually used?

that's why this patch changes the default scsi delay value to 5000ms. now all
of the lines that were setting the scsi delay value to 5000ms can be removed.
also default values should be chosen very conservatively. users can always
lower the delay value via their kernel config or sysctl.

cheers.
alex

> 
> >any thoughts on this patch?
> >
> >i noticed the "default" SCSI_DELAY value of 2000ms was only used in very 
> >few
> >places so i thought it would make more sense making 5000ms the default and
> >adding a few special cases where SCSI_DELAY can in fact be lowered down to
> >2000ms.
> >
> >cheers.
> >alex
> >
> >
> >
> >_______________________________________________
> >freebsd-hackers at freebsd.org mailing list
> >http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
> >To unsubscribe, send any mail to "freebsd-hackers-unsubscribe at freebsd.org"
> 

-- 
a13x


More information about the freebsd-hackers mailing list