Scott Cheloha wrote:
> > > diff --git sys/arch/sparc64/dev/fd.c sys/arch/sparc64/dev/fd.c
> > > index 8d548062f83..654d8c95524 100644
> > > --- sys/arch/sparc64/dev/fd.c
> > > +++ sys/arch/sparc64/dev/fd.c
> > > @@ -1632,7 +1632,7 @@ loop:
> > >           fdc->sc_state = RECALCOMPLETE;
> > >           if (fdc->sc_flags & FDC_NEEDHEADSETTLE) {
> > >                   /* allow 1/30 second for heads to settle */
> > > -                 timeout_add(&fdc->fdcpseudointr_to, hz / 30);
> > > +                 timeout_add_msec(&fdc->fdcpseudointr_to, 33);
> > >                   return (1);             /* will return later */
> > >           }
> > >  
> > 
> > Wonder if this should be 30 or 40 since 33 is rather odd.
> 
> The intent was to wait a 30th of a second.  In practice it has always
> fired 30ms hence.  It's a magic number, so I'd just call it 30ms.
> miod might have an opinion on whether those extra milliseconds would
> be useful in the event that sparc64 is ever able to timeout with
> millisecond or better precision on OpenBSD.

make it 30 and the let the next sparc64 user with a floppy drive worry about
the fallout. :)

Reply via email to