home *** CD-ROM | disk | FTP | other *** search
/ Amiga GigaPD 3 / Amiga_GigaPD_v3_3of3.iso / netbsd / docs / mailinglist-archive / 1993-09 / text0024.txt < prev    next >
Encoding:
Text File  |  1993-06-25  |  11.3 KB  |  362 lines
  1.     It's been tape drive weekend here at hack central.
  2.  
  3.     I've had two problems using my A3070 drive with NetBSD -- the
  4. first is that I couldn't work with multiple files on a single tape.
  5. Every command after doing a 'mt -f /dev/nrst0 fsf 1' resulted in an
  6. error.  That kinda sucks.  The problem appears to be that the SCSI tape
  7. driver (.../amiga/dev/st.c) will send a mode-select command to the
  8. driver every time you open the device and the Caliper CP150 (what's
  9. inside the A3070) doesn't like that very much.  It only seems to take
  10. mode-select when the tape is fully rewound.  (The mode-select chooses
  11. between 120M and 150M operation, and probably does some other stuff,
  12. too).  Here are the patches I used (613 source base).  The second
  13. problem is described after this:
  14.  
  15. ------------------------------------------------------------------------
  16. *** arch/amiga/dev/st.c-orig    Thu Aug 12 14:49:01 1993
  17. --- arch/amiga/dev/st.c    Sun Sep  5 21:48:40 1993
  18. ***************
  19. *** 111,117 ****
  20.   extern int sterror (int unit, struct st_softc *sc, int stat);
  21.   extern int stxsense (int ctlr, int slave, int unit, struct st_softc *sc);
  22.   extern int prtkey (int unit, struct st_softc *sc);
  23. ! extern int dumpxsense (struct st_xsense *sensebuf);
  24.   extern int prtmodsel (struct mode_select_data *msd, int modlen);
  25.   extern int prtmodstat (struct mode_sense *mode);
  26.   
  27. --- 111,117 ----
  28.   extern int sterror (int unit, struct st_softc *sc, int stat);
  29.   extern int stxsense (int ctlr, int slave, int unit, struct st_softc *sc);
  30.   extern int prtkey (int unit, struct st_softc *sc);
  31. ! extern int dumpxsense (struct st_xsense *sensebuf, struct st_softc *sc);
  32.   extern int prtmodsel (struct mode_select_data *msd, int modlen);
  33.   extern int prtmodstat (struct mode_sense *mode);
  34.   
  35. ***************
  36. *** 156,163 ****
  37.    */
  38.   static struct st_xsense {
  39.       struct    scsi_xsense sc_xsense;    /* data from sense */
  40. !     struct    exb_xsense exb_xsense;    /* additional info from exabyte */
  41.   } st_xsense[NST];
  42.   
  43.   static struct scsi_fmt_cdb stcmd[NST];
  44.   
  45. --- 156,168 ----
  46.    */
  47.   static struct st_xsense {
  48.       struct    scsi_xsense sc_xsense;    /* data from sense */
  49. !     union {
  50. !         struct exb_xsense uexb_xsense; /* additional info from exabyte */
  51. !         struct cpr_xsense ucpr_xsense; /* additional info from caliper */
  52. !     } u;
  53.   } st_xsense[NST];
  54. + #define exb_xsense u.uexb_xsense
  55. + #define cpr_xsense u.ucpr_xsense
  56.   
  57.   static struct scsi_fmt_cdb stcmd[NST];
  58.   
  59. ***************
  60. *** 316,325 ****
  61.               if (idstr[i] != ' ')
  62.                   break;
  63.           idstr[i+1] = 0;
  64. !         printf("st%d: %s >%s< rev %s\n", ad->amiga_unit, idstr, &idstr[8],
  65.                  &idstr[24]);
  66.       } else if (inqlen == 5)
  67. !         /* great it's a stupid device, doesn't know it's know name */
  68.           idstr[0] = idstr[8] = '\0';
  69.       else
  70.           idstr[8] = '\0';
  71. --- 321,330 ----
  72.               if (idstr[i] != ' ')
  73.                   break;
  74.           idstr[i+1] = 0;
  75. !         printf("st%d: %s %s rev %s\n", ad->amiga_unit, idstr, &idstr[8],
  76.                  &idstr[24]);
  77.       } else if (inqlen == 5)
  78. !         /* great it's a stupid device, doesn't know it's own name */
  79.           idstr[0] = idstr[8] = '\0';
  80.       else
  81.           idstr[8] = '\0';
  82. ***************
  83. *** 362,367 ****
  84. --- 367,378 ----
  85.           sc->sc_datalen[CMD_INQUIRY] = 36;
  86.           sc->sc_datalen[CMD_MODE_SELECT] = 12;
  87.           sc->sc_datalen[CMD_MODE_SENSE] = 12;
  88. +     } else if (bcmp("CP150", &idstr[8], 5) == 0) {
  89. +         sc->sc_tapeid = MT_ISCALIPER;
  90. +         sc->sc_datalen[CMD_REQUEST_SENSE] = 14;
  91. +         sc->sc_datalen[CMD_INQUIRY] = 36;
  92. +         sc->sc_datalen[CMD_MODE_SELECT] = 12;
  93. +         sc->sc_datalen[CMD_MODE_SENSE] = 12;
  94.       } else {
  95.           if (idstr[8] == '\0')
  96.               printf("st%d: No ID, assuming Archive\n", ad->amiga_unit);
  97. ***************
  98. *** 414,419 ****
  99. --- 425,431 ----
  100.       struct mode_select_data msd;
  101.       struct mode_sense mode;
  102.       int modlen;
  103. +     int skip_modsel = 0;
  104.       static struct scsi_fmt_cdb modsel = {
  105.           6,
  106.           CMD_MODE_SELECT, 0, 0, 0, sizeof(msd), 0
  107. ***************
  108. *** 471,476 ****
  109. --- 483,497 ----
  110.           sc->sc_blklen = 512;
  111.           break;
  112.   #endif
  113. +     case MT_ISCALIPER:
  114. +         sc->sc_blklen = 512;
  115. +         stxsense(ctlr, slave, unit, sc);
  116. +         /* Calipers can't handle a mode-select if the tape
  117. +          * isn't rewound.
  118. +          */
  119. +         if (xsense->cpr_xsense.b11)
  120. +             skip_modsel = 1;
  121. +         break;
  122.       case MT_ISWANGTEK:
  123.           sc->sc_blklen = 512;
  124.           break;
  125. ***************
  126. *** 485,490 ****
  127. --- 506,514 ----
  128.               sc->sc_blklen = 512;
  129.       }
  130.   
  131. +     if (skip_modsel)
  132. +         goto mode_selected;
  133. + 
  134.       /* setup for mode select */
  135.       msd.rsvd1 = 0;
  136.       msd.rsvd2 = 0;
  137. ***************
  138. *** 513,518 ****
  139. --- 537,543 ----
  140.               msd.density = 0x4;
  141.           }
  142.           break;
  143. +     case MT_ISCALIPER:
  144.       case MT_ISWANGTEK:
  145.           if (minor (dev) & STDEV_HIDENSITY)
  146.               msd.density = 0x10;
  147. ***************
  148. *** 586,591 ****
  149. --- 611,617 ----
  150.           goto retryselect;
  151.       }
  152.   
  153. + mode_selected:
  154.       /* drive ready ? */
  155.       stat = scsi_test_unit_rdy(ctlr, slave, unit);
  156.   
  157. ***************
  158. *** 622,627 ****
  159. --- 648,654 ----
  160.           case MT_ISHPDAT:
  161.           case MT_ISVIPER1:
  162.           case MT_ISPYTHON:
  163. +         case MT_ISCALIPER:
  164.           case MT_ISWANGTEK:
  165.               if (xsense->sc_xsense.key == XSK_UNTATTEN)
  166.                   stat = scsi_test_unit_rdy(ctlr, slave, unit);
  167. ***************
  168. *** 654,660 ****
  169.           stxsense(ctlr, slave, unit, sc);
  170.   #ifdef DEBUG
  171.           if (st_debug & ST_OPEN)
  172. !             dumpxsense(xsense);
  173.   #endif
  174.       }
  175.       if (stat)
  176. --- 681,687 ----
  177.           stxsense(ctlr, slave, unit, sc);
  178.   #ifdef DEBUG
  179.           if (st_debug & ST_OPEN)
  180. !             dumpxsense(xsense, sc);
  181.   #endif
  182.       }
  183.       if (stat)
  184. ***************
  185. *** 1387,1394 ****
  186.   
  187.   #ifdef DEBUG
  188.   
  189. ! dumpxsense(sensebuf)
  190.       struct st_xsense *sensebuf;
  191.   {
  192.           struct st_xsense *xp = sensebuf;
  193.   
  194. --- 1414,1422 ----
  195.   
  196.   #ifdef DEBUG
  197.   
  198. ! dumpxsense(sensebuf, sc)
  199.       struct st_xsense *sensebuf;
  200. +     struct st_softc *sc;
  201.   {
  202.           struct st_xsense *xp = sensebuf;
  203.   
  204. ***************
  205. *** 1404,1432 ****
  206.               (xp->sc_xsense.info3<<8)|(xp->sc_xsense.info4)) );
  207.       printf("ASenseL 0x%x\n", xp->sc_xsense.len);
  208.   
  209. !     if (xp->sc_xsense.len != 0x12) /* MT_ISEXB Exabyte only ?? */
  210. !         return;            /* What about others */
  211.   
  212. !     printf("ASenseC 0x%x\n", xp->exb_xsense.addsens);
  213. !     printf("AsenseQ 0x%x\n", xp->exb_xsense.addsensq);
  214. !     printf("R/W Errors 0x%lx\n", 
  215. !            (u_long)((xp->exb_xsense.rwerrcnt2<<16)|
  216. !             (xp->exb_xsense.rwerrcnt1<<8)|
  217. !             (xp->exb_xsense.rwerrcnt1)) );
  218. !     printf("PF   0x%x BPE  0x%x FPE 0x%x ME   0x%x ECO 0x%x TME 0x%x TNP 0x%x BOT 0x%x\n",
  219. !            xp->exb_xsense.pf, xp->exb_xsense.bpe, xp->exb_xsense.fpe, 
  220. !            xp->exb_xsense.me, xp->exb_xsense.eco, xp->exb_xsense.tme, 
  221. !            xp->exb_xsense.tnp, xp->exb_xsense.bot);
  222. !     printf("XFR  0x%x TMD  0x%x WP  0x%x FMKE 0x%x URE 0x%x WE1 0x%x SSE 0x%x FE  0x%x\n",
  223. !            xp->exb_xsense.xfr, xp->exb_xsense.tmd, xp->exb_xsense.wp, 
  224. !            xp->exb_xsense.fmke, xp->exb_xsense.ure, xp->exb_xsense.we1, 
  225. !            xp->exb_xsense.sse, xp->exb_xsense.fe);
  226. !     printf("WSEB 0x%x WSEO 0x%x\n",
  227. !            xp->exb_xsense.wseb, xp->exb_xsense.wseo);
  228. !     printf("Remaining Tape 0x%lx\n", 
  229. !            (u_long)((xp->exb_xsense.tplft2<<16)|
  230. !             (xp->exb_xsense.tplft1<<8)|
  231. !             (xp->exb_xsense.tplft0)) );
  232.   }
  233.   
  234.   prtmodsel(msd, modlen)
  235. --- 1432,1474 ----
  236.               (xp->sc_xsense.info3<<8)|(xp->sc_xsense.info4)) );
  237.       printf("ASenseL 0x%x\n", xp->sc_xsense.len);
  238.   
  239. !     switch (sc->sc_tapeid) {
  240. !     case MT_ISEXABYTE:
  241. !         printf("ASenseC 0x%x\n", xp->exb_xsense.addsens);
  242. !         printf("AsenseQ 0x%x\n", xp->exb_xsense.addsensq);
  243. !         printf("R/W Errors 0x%lx\n", 
  244. !                (u_long)((xp->exb_xsense.rwerrcnt2<<16)|
  245. !                 (xp->exb_xsense.rwerrcnt1<<8)|
  246. !                 (xp->exb_xsense.rwerrcnt1)) );
  247. !         printf("PF   0x%x BPE  0x%x FPE 0x%x ME   0x%x ECO 0x%x TME 0x%x TNP 0x%x BOT 0x%x\n",
  248. !                xp->exb_xsense.pf, xp->exb_xsense.bpe,
  249. !                xp->exb_xsense.fpe, xp->exb_xsense.me,
  250. !                xp->exb_xsense.eco, xp->exb_xsense.tme,
  251. !                xp->exb_xsense.tnp, xp->exb_xsense.bot);
  252. !         printf("XFR  0x%x TMD  0x%x WP  0x%x FMKE 0x%x URE 0x%x WE1 0x%x SSE 0x%x FE  0x%x\n",
  253. !                xp->exb_xsense.xfr, xp->exb_xsense.tmd,
  254. !                xp->exb_xsense.wp, xp->exb_xsense.fmke,
  255. !                xp->exb_xsense.ure, xp->exb_xsense.we1,
  256. !                xp->exb_xsense.sse, xp->exb_xsense.fe);
  257. !         printf("WSEB 0x%x WSEO 0x%x\n",
  258. !                xp->exb_xsense.wseb, xp->exb_xsense.wseo);
  259. !         printf("Remaining Tape 0x%lx\n", 
  260. !                (u_long)((xp->exb_xsense.tplft2<<16)|
  261. !                 (xp->exb_xsense.tplft1<<8)|
  262. !                 (xp->exb_xsense.tplft0)) );
  263. !         break;
  264. ! 
  265. !     case MT_ISCALIPER:
  266. !         printf("b8 0x%x (wp 0x%x) b9 0x%x b10 0x%x b11 0x%x b12 0x%x b13 0x%x\n",
  267. !                xp->cpr_xsense.b8, xp->cpr_xsense.b8 & CPR_WP,
  268. !                xp->cpr_xsense.b9, xp->cpr_xsense.b10,
  269. !                xp->cpr_xsense.b11, xp->cpr_xsense.b12,
  270. !                xp->cpr_xsense.b13);
  271. !         break;
  272.   
  273. !     default:
  274. !         break;
  275. !     }
  276.   }
  277.   
  278.   prtmodsel(msd, modlen)
  279. *** arch/amiga/dev/stvar.h-orig    Sat May 15 06:45:00 1993
  280. --- arch/amiga/dev/stvar.h    Sun Sep  5 00:05:41 1993
  281. ***************
  282. *** 101,106 ****
  283. --- 101,116 ----
  284.       char    venderunique[16];
  285.   };
  286.   
  287. + struct cpr_xsense {
  288. +     u_char    b8;
  289. + #define CPR_WP    0x10
  290. +     u_char    b9;
  291. +     u_char  b10;
  292. +     u_char  b11;
  293. +     u_char  b12;
  294. +     u_char  b13;
  295. + };
  296. + 
  297.   struct    st_mode {
  298.       u_char    sdl;    
  299.       u_char  medtype;
  300. *** sys/mtio.h-orig    Sat Jun 19 09:45:39 1993
  301. --- sys/mtio.h    Sat Sep  4 16:42:02 1993
  302. ***************
  303. *** 96,101 ****
  304. --- 96,102 ----
  305.   #define MT_ISPYTHON    0x0f        /* Archive Python (DAT) */
  306.   #define MT_ISHPDAT    0x10        /* HP 35450A DAT drive */
  307.   #define MT_ISWANGTEK    0x11        /* WANGTEK 5150ES */
  308. + #define MT_ISCALIPER    0x12        /* Caliper CP150 */
  309.   
  310.   /* mag tape io control commands */
  311.   #define    MTIOCTOP    _IOW('m', 1, struct mtop)    /* do a mag tape op */
  312.  
  313. ------------------------------------------------------------------------
  314.  
  315.     The other problem I had was that a tape error would crash the
  316. kernel -- woo hoo!  My specific test case was putting a fresh tape in
  317. the drive and doing a 'mt -f /dev/rst0 fsf 1'.  The problem goes
  318. something like this:  When an I/O operation is done and hasn't failed
  319. early because something like a mode-select failed, the tape driver
  320. calls stfinish(), which then calls biodone() on the I/O buffer which
  321. was being used for the I/O operation.  biodone() wakes up anyone who
  322. was waiting on the buffer; someone (probably the tape driver but I
  323. haven't checked) is stuck in biowait() sleeping on that buffer.
  324. biowait() wakes up, looks at the buffer, sees that its error flag is
  325. set, and proceeds to remove it from its hash queue and put it somewhere
  326. else.  Unfortunately, that buffer was never on a hash queue -- it was
  327. an element in an array of buffers defined inside st.c.  Wa ha ha ha!
  328. What a good joke!  The b_back and b_forw pointers are 0!  Here's what I
  329. did to fix that:
  330.  
  331. ------------------------------------------------------------------------
  332. *** kern/vfs_bio.c-orig    Sun Aug  8 05:47:13 1993
  333. --- kern/vfs_bio.c    Sun Sep  5 21:48:07 1993
  334. ***************
  335. *** 653,660 ****
  336.    *    and i believe it's wrong to let bp->b_error override B_ERROR
  337.    *    but it certainly appears to work OK this way...
  338.    */    
  339. !             bremhash(bp);
  340. !             binshash(bp, bfreelist + BQ_AGE);
  341.           }
  342.           if (!bp->b_error)
  343.               bp->b_error = EIO;
  344. --- 653,663 ----
  345.    *    and i believe it's wrong to let bp->b_error override B_ERROR
  346.    *    but it certainly appears to work OK this way...
  347.    */    
  348. !             /* Some bufs aren't in the buffer cache, you fool. */
  349. !             if (bp->b_back && bp->b_forw) {
  350. !                 bremhash(bp);
  351. !                 binshash(bp, bfreelist + BQ_AGE);
  352. !             }
  353.           }
  354.           if (!bp->b_error)
  355.               bp->b_error = EIO;
  356. ------------------------------------------------------------------------
  357.  
  358. --
  359. ------------------------------------------------------------------------
  360. Andy Heffernan                                            ahh@netcom.com
  361.  
  362.