On Tuesday, October 19, 2010 3:14:46 pm Alexander Best wrote:
> On Tue Oct 19 10, John Baldwin wrote:
> > On Tuesday, October 19, 2010 10:31:10 am Alexander Best wrote:
> > > On Tue Oct 19 10, Matthew Jacob wrote:
> > > > It would be an effective behavioral change for those of us who remove
> > > > that line.
> > > > Personally, I think 5 seconds is too long- even 2 seconds is more than
> > > > adequate even for moderately old 'other' hardware like scanners.
> > > >
> > > > For -current, why don't you simply remove all of the config lines and
> > > > leave the default at 2000ms?
> > >
> > > hmmm...i can only test the delay value on amd64. i was under the
> > > impression
> > > that archs like arm and mips need the longer delay.
> > >
> > > also at some locations in the code SCSI_DELAY is being set to 15000. i
> > > believe
> > > this is the case when certain drivers (cam, ahb, aha) get loaded as a
> > > kernel
> > > module, but i'm not sure. it looks like this:
> > >
> > > .if !defined(KERNBUILDDIR)
> > > opt_scsi.h:
> > > echo "#define SCSI_DELAY 15000" > ${.TARGET}
> > > .endif
> >
> > I believe this is all old history. SCSI_DELAY used to be set to 15000 in
> > GENERIC many years ago and was lowered to 5000. Most likely these Makefiles
> > were simply not updated at the time.
>
> oh i see. maybe this revised patch would be better suited then.
I think so, but you should post this to scsi@ for the best review.
--
John Baldwin
_______________________________________________
freebsd-hackers@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "freebsd-hackers-unsubscr...@freebsd.org"
