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 > > > > > > > >_______________________________________________ > >[email protected] mailing list > >http://lists.freebsd.org/mailman/listinfo/freebsd-hackers > >To unsubscribe, send any mail to "[email protected]" > -- a13x _______________________________________________ [email protected] mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-hackers To unsubscribe, send any mail to "[email protected]"
