Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 10 Sep 1995 12:19:31 +0200 (MET DST)
From:      J Wunsch <j@uriah.heep.sax.de>
To:        nistler@millcomm.com
Cc:        FreeBSD-gnats-submit@FreeBSD.ORG, bugs@FreeBSD.ORG
Subject:   Re: misc/717: ft.c attach fail on my Mountain tape drive
Message-ID:  <199509101019.MAA01669@uriah.heep.sax.de>
In-Reply-To: <199509011732.MAA00403@chaos.millcomm.com> from "John Nistler" at Sep 1, 95 12:32:46 pm

next in thread | previous in thread | raw e-mail | index | archive | help
As John Nistler wrote:
> 
> 	[2: ft0: Mountain tape]	
> 
> >Description:

> 	When the ft.c code does the attach it uses polling in the
> ftintr_waiti(in the if (ft->attach), in case FTCMD_SEEK) with a
> delay(100) which is too short.  I changed this to 1000.

> >Fix:
> 	
> 	/usr/src/sys/i386/isa> diff ft.c.orig  ft.c
> 	1480c1480,1481
> 	<                       DELAY(100);
> 	---
> 	>                   DELAY(1000);
> 	>                   /* DELAY(100); */

Please, supply context diffs!  It's rather hard to track simple diffs
if the source has been changed meanwhile.

In this case, the entire logic has been slightly modified since 2.0R,
where your patch apparently is for.  Can you please try if the
following patch would also solve your problem?  This is the
appropriate code fragment out of 2.0.5.  If this does solve it, too,
we can close the PR.

Index: ft.c
===================================================================
RCS file: /home/cvs/src/sys/i386/isa/ft.c,v
retrieving revision 1.14
diff -u -r1.14 ft.c
--- 1.14	1994/11/15 14:34:42
+++ ft.c	1995/09/10 10:14:32
@@ -1469,15 +1503,16 @@
 	    case FTCMD_RECAL:
 	    case FTCMD_SEEK:
 		for (retries = 0; retries < 10000; retries++) {
+			DELAY(150);
 			out_fdc(fdcu, NE7CMD_SENSEI);
 			st0 = in_fdc(fdcu);
+			if ((st0 & 0xc0) == 0x80) continue;
 			pcn = in_fdc(fdcu);
 			if (st0 & 0x20) {
 				ft->sts_wait = FTSTS_INTERRUPT;
 				ft->pcn = pcn;
 				goto intrdone;
 			}
-			DELAY(100);
 		}
 		break;
 	}


-- 
cheers, J"org

joerg_wunsch@uriah.heep.sax.de -- http://www.sax.de/~joerg/
Never trust an operating system you don't have sources for. ;-)



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199509101019.MAA01669>