> From: Arjan van de Ven wrote:
> On Fri, 2002-10-11 at 16:10, Stephen Cameron wrote:
> >
> > Wait up to 20 seconds for polled commands to complete. A
> certain multiport
> > storage box needs this.
> >
> > diff -urN linux-2.5.41-p/drivers/block/cciss.c
> linux-2.5.41-q/drivers/block/cciss.c
> > --- linux-2.5.41-p/drivers/block/cciss.c Wed Oct 9 15:22:14 2002
> > +++ linux-2.5.41-q/drivers/block/cciss.c Wed Oct 9 15:54:35 2002
> > @@ -1318,9 +1318,9 @@
> > unsigned long done;
> > int i;
> >
> > - /* Wait (up to 2 seconds) for a command to complete */
> > + /* Wait (up to 20 seconds) for a command to complete */
> >
> > - for (i = 200000; i > 0; i--) {
> > + for (i = 2000000; i > 0; i--) {
> > done =
> hba[ctlr]->access.command_completed(hba[ctlr]);
> > if (done == FIFO_EMPTY) {
> > udelay(10); /* a short fixed delay */
>
> ugh 20 seconds udelay....
>
> why can't you sleep here ?
> (and yes 20 seconds WILL trigger watchdogs!)
Ok. Nevermind that patch then. We'll have to rethink it.
-- steve
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Tue Oct 15 2002 - 22:00:41 EST