diff options
author | Kenneth R Westerback <krw@cvs.openbsd.org> | 2005-12-03 16:53:17 +0000 |
---|---|---|
committer | Kenneth R Westerback <krw@cvs.openbsd.org> | 2005-12-03 16:53:17 +0000 |
commit | 59413aaf9936ee396905effecb9206e3e5d9e01f (patch) | |
tree | d820a69874e8e867417c8c87fff410362b921e48 /sys/dev/ic/isp_openbsd.c | |
parent | 734d8d431c424e5467db4143f21b9249f5900059 (diff) |
The first thing done when XXX_scsi_cmd() returns TRY_AGAIN_LATER is
to set xs->error to XS_BUSY. So it is pointless and misleading to
set xs->error to XS_TIMEOUT, XS_DRIVER_STUFFUP or XS_BUSY just before
returning TRY_AGAIN_LATER.
No functional change.
Diffstat (limited to 'sys/dev/ic/isp_openbsd.c')
-rw-r--r-- | sys/dev/ic/isp_openbsd.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/sys/dev/ic/isp_openbsd.c b/sys/dev/ic/isp_openbsd.c index fe4625ee47f..ece2a2aa439 100644 --- a/sys/dev/ic/isp_openbsd.c +++ b/sys/dev/ic/isp_openbsd.c @@ -1,4 +1,4 @@ -/* $OpenBSD: isp_openbsd.c,v 1.27 2004/08/02 19:55:45 art Exp $ */ +/* $OpenBSD: isp_openbsd.c,v 1.28 2005/12/03 16:53:16 krw Exp $ */ /* * Platform (OpenBSD) dependent common attachment code for Qlogic adapters. * @@ -318,7 +318,6 @@ ispcmd(XS_T *xs) */ if (isp->isp_osinfo.blocked) { if (xs->flags & SCSI_POLL) { - xs->error = XS_DRIVER_STUFFUP; ISP_UNLOCK(isp); return (TRY_AGAIN_LATER); } @@ -393,9 +392,6 @@ isp_polled_cmd(struct ispsoftc *isp, XS_T *xs) break; case CMD_RQLATER: case CMD_EAGAIN: - if (XS_NOERR(xs)) { - xs->error = XS_DRIVER_STUFFUP; - } result = TRY_AGAIN_LATER; break; case CMD_COMPLETE: |