Re: [PATCH] 2.5.41, cciss (3 of 3)

[Arjan van de Ven]

--=-M21IVxaMW8U+nC3DHJVN
Content-Type: text/plain
Content-Transfer-Encoding: quoted-printable

On Fri, 2002-10-11 at 16:10, Stephen Cameron wrote:
>=20
> Wait up to 20 seconds for polled commands to complete.  A certain multipo=
rt
> storage box needs this.
>=20
> diff -urN linux-2.5.41-p/drivers/block/cciss.c linux-2.5.41-q/drivers/blo=
ck/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;
> =20
> -        /* Wait (up to 2 seconds) for a command to complete */
> +        /* Wait (up to 20 seconds) for a command to complete */
> =20
> -        for (i =3D 200000; i > 0; i--) {
> +        for (i =3D 2000000; i > 0; i--) {
>                  done =3D hba[ctlr]->access.command_completed(hba[ctlr]);
>                  if (done =3D=3D 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!)

--=-M21IVxaMW8U+nC3DHJVN
Content-Type: application/pgp-signature; name=signature.asc
Content-Description: This is a digitally signed message part

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.7 (GNU/Linux)

iD8DBQA9pt53xULwo51rQBIRAnylAJ4hNzS475fxr0cyUEpSCprZQZcHnQCeIoPm
qXRKqXooA9yhfnMYx9KPHm0=
=TukI
-----END PGP SIGNATURE-----

--=-M21IVxaMW8U+nC3DHJVN--