lint
[unix-history] / usr / src / sys / vax / mba / hp.c
index 8646b85..34fc1c4 100644 (file)
@@ -1,9 +1,22 @@
-/*     hp.c    4.17    81/02/28        */
+/*     hp.c    4.42    81/11/18        */
+
+#ifdef HPDEBUG
+int    hpdebug;
+#endif
+#ifdef HPBDEBUG
+int    hpbdebug;
+#endif
 
 #include "hp.h"
 #if NHP > 0
 /*
 
 #include "hp.h"
 #if NHP > 0
 /*
- * HP disk driver for RP0x+RM0x
+ * HP disk driver for RP0x+RMxx
+ *
+ * TODO:
+ *     check RM80 skip sector handling when ECC's occur later
+ *     check offset recovery handling
+ *     see if DCLR and/or RELEASE set attention status
+ *     print bits of mr && mr2 symbolically
  */
 
 #include "../h/param.h"
  */
 
 #include "../h/param.h"
 #include "../h/user.h"
 #include "../h/map.h"
 #include "../h/pte.h"
 #include "../h/user.h"
 #include "../h/map.h"
 #include "../h/pte.h"
-#include "../h/mba.h"
+#include "../h/mbareg.h"
+#include "../h/mbavar.h"
 #include "../h/mtpr.h"
 #include "../h/vm.h"
 #include "../h/cmap.h"
 #include "../h/mtpr.h"
 #include "../h/vm.h"
 #include "../h/cmap.h"
+#include "../h/dkbad.h"
 
 #include "../h/hpreg.h"
 
 
 #include "../h/hpreg.h"
 
 struct size {
        daddr_t nblocks;
        int     cyloff;
 struct size {
        daddr_t nblocks;
        int     cyloff;
-} hp_sizes[8] = {
+} hp6_sizes[8] = {
        15884,  0,              /* A=cyl 0 thru 37 */
        33440,  38,             /* B=cyl 38 thru 117 */
        340670, 0,              /* C=cyl 0 thru 814 */
        0,      0,
        0,      0,
        0,      0,
        15884,  0,              /* A=cyl 0 thru 37 */
        33440,  38,             /* B=cyl 38 thru 117 */
        340670, 0,              /* C=cyl 0 thru 814 */
        0,      0,
        0,      0,
        0,      0,
-       291346, 118,            /* G=cyl 118 thru 814 */
+#ifndef NOBADSECT
+       291280, 118,            /* G=cyl 118 thru 814 */
+#else
+       291346, 118,
+#endif
        0,      0,
        0,      0,
-}, rm_sizes[8] = {
+}, rm3_sizes[8] = {
        15884,  0,              /* A=cyl 0 thru 99 */
        33440,  100,            /* B=cyl 100 thru 309 */
        131680, 0,              /* C=cyl 0 thru 822 */
        15884,  0,              /* A=cyl 0 thru 99 */
        33440,  100,            /* B=cyl 100 thru 309 */
        131680, 0,              /* C=cyl 0 thru 822 */
-       2720,   291,
        0,      0,
        0,      0,
        0,      0,
        0,      0,
-       82080,  310,            /* G=cyl 310 thru 822 */
+       0,      0,
+#ifndef NOBADSECT
+       81984,  310,            /* G=cyl 310 thru 822 */
+#else
+       82080,  310,
+#endif
        0,      0,
 }, rm5_sizes[8] = {
        15884,  0,              /* A=cyl 0 thru 26 */
        33440,  27,             /* B=cyl 27 thru 81 */
        0,      0,
 }, rm5_sizes[8] = {
        15884,  0,              /* A=cyl 0 thru 26 */
        33440,  27,             /* B=cyl 27 thru 81 */
-       500992, 0,              /* C=cyl 0 thru 823 */
+       500384, 0,              /* C=cyl 0 thru 822 */
        15884,  562,            /* D=cyl 562 thru 588 */
        55936,  589,            /* E=cyl 589 thru 680 */
        15884,  562,            /* D=cyl 562 thru 588 */
        55936,  589,            /* E=cyl 589 thru 680 */
-       86944,  681,            /* F=cyl 681 thru 823 */
-       159296, 562,            /* G=cyl 562 thru 823 */
+#ifndef NOBADSECT
+       86240,  681,            /* F=cyl 681 thru 822 */
+       158592, 562,            /* G=cyl 562 thru 822 */
+#else
+       86636,  681,
+       158688, 562,
+#endif
        291346, 82,             /* H=cyl 82 thru 561 */
 }, rm80_sizes[8] = {
        15884,  0,              /* A=cyl 0 thru 36 */
        291346, 82,             /* H=cyl 82 thru 561 */
 }, rm80_sizes[8] = {
        15884,  0,              /* A=cyl 0 thru 36 */
@@ -61,7 +89,16 @@ struct       size {
        0,      0,
        0,      0,
        82080,  115,            /* G=cyl 115 thru 304 */
        0,      0,
        0,      0,
        82080,  115,            /* G=cyl 115 thru 304 */
-       110236, 305,            /* H=cyl 305 thru 558 */
+       110143, 305,            /* H=cyl 305 thru 558 */
+}, hp7_sizes[8] = {
+       15844,  0,              /* A=cyl 0 thru 9 */
+       64000,  10,             /* B=cyl 10 thru 49 */
+       1008000,0,              /* C=cyl 0 thru 629 */
+       15884,  330,            /* D=cyl 330 thru 339 */
+       256000, 340,            /* E=cyl 340 thru 499 */
+       207850, 500,            /* F=cyl 500 thru 629 */
+       479850, 330,            /* G=cyl 330 thru 629 */
+       448000, 50,             /* H=cyl 50 thru 329 */
 };
 /* END OF STUFF WHICH SHOULD BE READ IN PER DISK */
 
 };
 /* END OF STUFF WHICH SHOULD BE READ IN PER DISK */
 
@@ -72,11 +109,12 @@ int        hpSDIST = _hpSDIST;
 int    hpRDIST = _hpRDIST;
 
 short  hptypes[] =
 int    hpRDIST = _hpRDIST;
 
 short  hptypes[] =
-       { MBDT_RM03, MBDT_RM05, MBDT_RP06, MBDT_RM80, 0 };
-struct mba_info *hpinfo[NHP];
-int    hpdkinit(),hpustart(),hpstart(),hpdtint();
+       { MBDT_RM03, MBDT_RM05, MBDT_RP06, MBDT_RM80, MBDT_RP05, MBDT_RP07, 0 };
+struct mba_device *hpinfo[NHP];
+int    hpattach(),hpustart(),hpstart(),hpdtint();
 struct mba_driver hpdriver =
 struct mba_driver hpdriver =
-       { hpdkinit, hpustart, hpstart, hpdtint, 0, hptypes, hpinfo };
+       { hpattach, 0, hpustart, hpstart, hpdtint, 0,
+         hptypes, "hp", 0, hpinfo };
 
 struct hpst {
        short   nsect;
 
 struct hpst {
        short   nsect;
@@ -85,18 +123,28 @@ struct hpst {
        short   ncyl;
        struct  size *sizes;
 } hpst[] = {
        short   ncyl;
        struct  size *sizes;
 } hpst[] = {
-       32,     5,      32*5,   823,    rm_sizes,       /* RM03 */
+       32,     5,      32*5,   823,    rm3_sizes,      /* RM03 */
        32,     19,     32*19,  823,    rm5_sizes,      /* RM05 */
        32,     19,     32*19,  823,    rm5_sizes,      /* RM05 */
-       22,     19,     22*19,  815,    hp_sizes,       /* RP06 */
-       31,     14,     31*14,  559,    rm80_sizes      /* RM80 */
+       22,     19,     22*19,  815,    hp6_sizes,      /* RP06 */
+       31,     14,     31*14,  559,    rm80_sizes,     /* RM80 */
+       22,     19,     22*19,  411,    hp6_sizes,      /* RP05 */
+       50,     32,     50*32,  630,    hp7_sizes,      /* RP07 */
 };
 
 u_char hp_offset[16] = {
 };
 
 u_char hp_offset[16] = {
-    HP_P400, HP_M400, HP_P400, HP_M400, HP_P800, HP_M800, HP_P800, HP_M800,
-    HP_P1200, HP_M1200, HP_P1200, HP_M1200, 0, 0, 0, 0,
+    HPOF_P400, HPOF_M400, HPOF_P400, HPOF_M400,
+    HPOF_P800, HPOF_M800, HPOF_P800, HPOF_M800,
+    HPOF_P1200, HPOF_M1200, HPOF_P1200, HPOF_M1200,
+    0, 0, 0, 0,
 };
 
 struct buf     rhpbuf[NHP];
 };
 
 struct buf     rhpbuf[NHP];
+#ifndef NOBADSECT
+struct buf     bhpbuf[NHP];
+struct dkbad   hpbad[NHP];
+#endif
+char   hpinit[NHP];
+char   hprecal[NHP];
 
 #define        b_cylin b_resid
  
 
 #define        b_cylin b_resid
  
@@ -106,8 +154,9 @@ daddr_t dkblock();
  
 int    hpseek;
 
  
 int    hpseek;
 
-hpdkinit(mi)
-       struct mba_info *mi;
+/*ARGSUSED*/
+hpattach(mi, slave)
+       struct mba_device *mi;
 {
        register struct hpst *st = &hpst[mi->mi_type];
 
 {
        register struct hpst *st = &hpst[mi->mi_type];
 
@@ -118,7 +167,7 @@ hpdkinit(mi)
 hpstrategy(bp)
        register struct buf *bp;
 {
 hpstrategy(bp)
        register struct buf *bp;
 {
-       register struct mba_info *mi;
+       register struct mba_device *mi;
        register struct hpst *st;
        register int unit;
        long sz, bn;
        register struct hpst *st;
        register int unit;
        long sz, bn;
@@ -151,26 +200,45 @@ bad:
 }
 
 hpustart(mi)
 }
 
 hpustart(mi)
-       register struct mba_info *mi;
+       register struct mba_device *mi;
 {
        register struct hpdevice *hpaddr = (struct hpdevice *)mi->mi_drv;
        register struct buf *bp = mi->mi_tab.b_actf;
 {
        register struct hpdevice *hpaddr = (struct hpdevice *)mi->mi_drv;
        register struct buf *bp = mi->mi_tab.b_actf;
-       register struct hpst *st;
+       register struct hpst *st = &hpst[mi->mi_type];
        daddr_t bn;
        daddr_t bn;
-       int sn, dist, flags;
+       int sn, dist;
 
 
+       hpaddr->hpcs1 = 0;
        if ((hpaddr->hpcs1&HP_DVA) == 0)
                return (MBU_BUSY);
        if ((hpaddr->hpcs1&HP_DVA) == 0)
                return (MBU_BUSY);
-       if ((hpaddr->hpds & HP_VV) == 0) {
+       if ((hpaddr->hpds & HPDS_VV) == 0 || hpinit[mi->mi_unit] == 0) {
+#ifndef NOBADSECT
+               struct buf *bbp = &bhpbuf[mi->mi_unit];
+#endif
+
+               hpinit[mi->mi_unit] = 1;
                hpaddr->hpcs1 = HP_DCLR|HP_GO;
                hpaddr->hpcs1 = HP_DCLR|HP_GO;
+               if (mi->mi_mba->mba_drv[0].mbd_as & (1<<mi->mi_drive))
+                       printf("DCLR attn\n");
                hpaddr->hpcs1 = HP_PRESET|HP_GO;
                hpaddr->hpcs1 = HP_PRESET|HP_GO;
-               hpaddr->hpof = HP_FMT22;
+               hpaddr->hpof = HPOF_FMT22;
+               mbclrattn(mi);
+#ifndef NOBADSECT
+               bbp->b_flags = B_READ|B_BUSY;
+               bbp->b_dev = bp->b_dev;
+               bbp->b_bcount = 512;
+               bbp->b_un.b_addr = (caddr_t)&hpbad[mi->mi_unit];
+               bbp->b_blkno = st->ncyl*st->nspc - st->nsect;
+               bbp->b_cylin = st->ncyl - 1;
+               mi->mi_tab.b_actf = bbp;
+               bbp->av_forw = bp;
+               bp = bbp;
+#endif
        }
        if (mi->mi_tab.b_active || mi->mi_hd->mh_ndrive == 1)
                return (MBU_DODATA);
        }
        if (mi->mi_tab.b_active || mi->mi_hd->mh_ndrive == 1)
                return (MBU_DODATA);
-       if ((hpaddr->hpds & (HP_DPR|HP_MOL)) != (HP_DPR|HP_MOL))
+       if ((hpaddr->hpds & HPDS_DREADY) != HPDS_DREADY)
                return (MBU_DODATA);
                return (MBU_DODATA);
-       st = &hpst[mi->mi_type];
        bn = dkblock(bp);
        sn = bn%st->nspc;
        sn = (sn+st->nsect-hpSDIST)%st->nsect;
        bn = dkblock(bp);
        sn = bn%st->nspc;
        sn = (sn+st->nsect-hpSDIST)%st->nsect;
@@ -194,7 +262,7 @@ hpustart(mi)
 }
 
 hpstart(mi)
 }
 
 hpstart(mi)
-       register struct mba_info *mi;
+       register struct mba_device *mi;
 {
        register struct hpdevice *hpaddr = (struct hpdevice *)mi->mi_drv;
        register struct buf *bp = mi->mi_tab.b_actf;
 {
        register struct hpdevice *hpaddr = (struct hpdevice *)mi->mi_drv;
        register struct buf *bp = mi->mi_tab.b_actf;
@@ -206,59 +274,132 @@ hpstart(mi)
        sn = bn%st->nspc;
        tn = sn/st->nsect;
        sn %= st->nsect;
        sn = bn%st->nspc;
        tn = sn/st->nsect;
        sn %= st->nsect;
-       if (mi->mi_tab.b_errcnt >= 16 && (bp->b_flags&B_READ) != 0) {
-               hpaddr->hpof = hp_offset[mi->mi_tab.b_errcnt & 017] | HP_FMT22;
-               hpaddr->hpcs1 = HP_OFFSET|HP_GO;
-               while (hpaddr->hpds & HP_PIP)
-                       ;
-               mbclrattn(mi);
-       }
        hpaddr->hpdc = bp->b_cylin;
        hpaddr->hpda = (tn << 8) + sn;
        hpaddr->hpdc = bp->b_cylin;
        hpaddr->hpda = (tn << 8) + sn;
+       return(0);
 }
 
 }
 
-hpdtint(mi, mbasr)
-       register struct mba_info *mi;
-       int mbasr;
+hpdtint(mi, mbsr)
+       register struct mba_device *mi;
+       int mbsr;
 {
        register struct hpdevice *hpaddr = (struct hpdevice *)mi->mi_drv;
        register struct buf *bp = mi->mi_tab.b_actf;
 {
        register struct hpdevice *hpaddr = (struct hpdevice *)mi->mi_drv;
        register struct buf *bp = mi->mi_tab.b_actf;
+       int retry = 0;
 
 
-       while ((hpaddr->hpds & HP_DRY) == 0)    /* shouldn't happen */
-               printf("hp dry not set\n");
-       if (hpaddr->hpds&HP_ERR || mbasr&MBAEBITS)
-               if (++mi->mi_tab.b_errcnt < 28 && (hpaddr->hper1&HP_WLE)==0) {
-                       if ((hpaddr->hper1&0xffff) != HP_DCK) {
-                               hpaddr->hpcs1 = HP_DCLR|HP_GO;
-                               if ((mi->mi_tab.b_errcnt&07) == 4) {
-                                       hpaddr->hpcs1 = HP_RECAL|HP_GO;
-                                       while (hpaddr->hpds & HP_PIP)
-                                               ;
-                                       mbclrattn(mi);
-                               }
-                               return (MBD_RETRY);
-                       } else if (hpecc(mi))
-                               return (MBD_RESTARTED);
-               } else {
-                       if (hpaddr->hper1&HP_WLE)       
-                               printf("hp%d is write locked\n", dkunit(bp));
-                       else {
-                               harderr(bp);
-                               printf("hp%d mbasr %b er1 %b er2 %b\n",
-                                   dkunit(bp), mbasr, mbasr_bits,
-                                   hpaddr->hper1, HPER1_BITS,
-                                   hpaddr->hper2, HPER2_BITS);
-                       }
-                       hpaddr->hpcs1 = HP_DCLR|HP_GO;
+#ifndef NOBADSECT
+       if (bp->b_flags&B_BAD) {
+               if (hpecc(mi, CONT))
+                       return(MBD_RESTARTED);
+       }
+#endif
+       if (hpaddr->hpds&HPDS_ERR || mbsr&MBSR_EBITS) {
+#ifdef HPDEBUG
+               if (hpdebug) {
+                       int dc = hpaddr->hpdc, da = hpaddr->hpda;
+
+                       printf("hperr: bp %x cyl %d blk %d as %o ",
+                               bp, bp->b_cylin, bp->b_blkno,
+                               hpaddr->hpas&0xff);
+                       printf("dc %x da %x\n",dc&0xffff, da&0xffff);
+                       printf("errcnt %d ", mi->mi_tab.b_errcnt);
+                       printf("mbsr=%b ", mbsr, mbsr_bits);
+                       printf("er1=%b er2=%b\n",
+                           hpaddr->hper1, HPER1_BITS,
+                           hpaddr->hper2, HPER2_BITS);
+                       DELAY(1000000);
+               }
+#endif
+               if (hpaddr->hper1&HPER1_WLE) {
+                       printf("hp%d: write locked\n", dkunit(bp));
                        bp->b_flags |= B_ERROR;
                        bp->b_flags |= B_ERROR;
+               } else if (++mi->mi_tab.b_errcnt > 27 ||
+                   mbsr & MBSR_HARD ||
+                   hpaddr->hper1 & HPER1_HARD ||
+                   hpaddr->hper2 & HPER2_HARD) {
+hard:
+                       harderr(bp, "hp");
+                       if (mbsr & (MBSR_EBITS &~ (MBSR_DTABT|MBSR_MBEXC)))
+                               printf("mbsr=%b ", mbsr, mbsr_bits);
+                       printf("er1=%b er2=%b",
+                           hpaddr->hper1, HPER1_BITS,
+                           hpaddr->hper2, HPER2_BITS);
+                       if (hpaddr->hpmr)
+                               printf(" mr=%o", hpaddr->hpmr&0xffff);
+                       if (hpaddr->hpmr2)
+                               printf(" mr2=%o", hpaddr->hpmr2&0xffff);
+                       printf("\n");
+                       bp->b_flags |= B_ERROR;
+                       hprecal[mi->mi_unit] = 0;
+               } else if (hpaddr->hper2 & HPER2_BSE) {
+#ifndef NOBADSECT
+                       if (hpecc(mi, BSE))
+                               return(MBD_RESTARTED);
+                       else
+#endif
+                               goto hard;
+               } else if (hptypes[mi->mi_type] == MBDT_RM80 &&
+                   hpaddr->hper2&HPER2_SSE) {
+                       (void) hpecc(mi, SSE);
+                       return (MBD_RESTARTED);
+               } else if ((hpaddr->hper1&(HPER1_DCK|HPER1_ECH))==HPER1_DCK) {
+                       if (hpecc(mi, ECC))
+                               return (MBD_RESTARTED);
+                       /* else done */
+               } else
+                       retry = 1;
+               hpaddr->hpcs1 = HP_DCLR|HP_GO;
+               if ((mi->mi_tab.b_errcnt&07) == 4) {
+                       hpaddr->hpcs1 = HP_RECAL|HP_GO;
+                       hprecal[mi->mi_unit] = 1;
+                       return(MBD_RESTARTED);
                }
                }
+               if (retry)
+                       return (MBD_RETRY);
+       }
+#ifdef HPDEBUG
+       else
+               if (hpdebug && hprecal[mi->mi_unit]) {
+                       printf("recal %d ", hprecal[mi->mi_unit]);
+                       printf("errcnt %d\n", mi->mi_tab.b_errcnt);
+                       printf("mbsr=%b ", mbsr, mbsr_bits);
+                       printf("er1=%b er2=%b\n",
+                           hpaddr->hper1, HPER1_BITS,
+                           hpaddr->hper2, HPER2_BITS);
+               }
+#endif
+       switch (hprecal[mi->mi_unit]) {
+
+       case 1:
+               hpaddr->hpdc = bp->b_cylin;
+               hpaddr->hpcs1 = HP_SEEK|HP_GO;
+               hprecal[mi->mi_unit]++;
+               return (MBD_RESTARTED);
+       case 2:
+               if (mi->mi_tab.b_errcnt < 16 ||
+                   (bp->b_flags & B_READ) == 0)
+                       goto donerecal;
+               hpaddr->hpof = hp_offset[mi->mi_tab.b_errcnt & 017]|HPOF_FMT22;
+               hpaddr->hpcs1 = HP_OFFSET|HP_GO;
+               hprecal[mi->mi_unit]++;
+               return (MBD_RESTARTED);
+       donerecal:
+       case 3:
+               hprecal[mi->mi_unit] = 0;
+               return (MBD_RETRY);
+       }
        bp->b_resid = -(mi->mi_mba->mba_bcr) & 0xffff;
        bp->b_resid = -(mi->mi_mba->mba_bcr) & 0xffff;
-       if (mi->mi_tab.b_errcnt) {
+       if (mi->mi_tab.b_errcnt >= 16) {
+               /*
+                * This is fast and occurs rarely; we don't
+                * bother with interrupts.
+                */
                hpaddr->hpcs1 = HP_RTC|HP_GO;
                hpaddr->hpcs1 = HP_RTC|HP_GO;
-               while (hpaddr->hpds & HP_PIP)
+               while (hpaddr->hpds & HPDS_PIP)
                        ;
                mbclrattn(mi);
        }
                        ;
                mbclrattn(mi);
        }
+       hpaddr->hpof = HPOF_FMT22;
        hpaddr->hpcs1 = HP_RELEASE|HP_GO;
        return (MBD_DONE);
 }
        hpaddr->hpcs1 = HP_RELEASE|HP_GO;
        return (MBD_DONE);
 }
@@ -285,64 +426,114 @@ hpwrite(dev)
                physio(hpstrategy, &rhpbuf[unit], dev, B_WRITE, minphys);
 }
 
                physio(hpstrategy, &rhpbuf[unit], dev, B_WRITE, minphys);
 }
 
-hpecc(mi)
-       register struct mba_info *mi;
+hpecc(mi, flag)
+       register struct mba_device *mi;
+       int flag;
 {
        register struct mba_regs *mbp = mi->mi_mba;
        register struct hpdevice *rp = (struct hpdevice *)mi->mi_drv;
        register struct buf *bp = mi->mi_tab.b_actf;
 {
        register struct mba_regs *mbp = mi->mi_mba;
        register struct hpdevice *rp = (struct hpdevice *)mi->mi_drv;
        register struct buf *bp = mi->mi_tab.b_actf;
-       register struct hpst *st;
-       register int i;
-       caddr_t addr;
-       int reg, bit, byte, npf, mask, o;
+       register struct hpst *st = &hpst[mi->mi_type];
+       int npf, o;
        int bn, cn, tn, sn;
        int bn, cn, tn, sn;
-       struct pte mpte;
        int bcr;
 
        bcr = mbp->mba_bcr & 0xffff;
        if (bcr)
                bcr |= 0xffff0000;              /* sxt */
        int bcr;
 
        bcr = mbp->mba_bcr & 0xffff;
        if (bcr)
                bcr |= 0xffff0000;              /* sxt */
-       npf = btop(bcr + bp->b_bcount) - 1;
-       reg = npf;
+#ifndef NOBADSECT
+       if (flag == CONT)
+               npf = bp->b_error;
+       else
+#endif
+               npf = btop(bcr + bp->b_bcount);
        o = (int)bp->b_un.b_addr & PGOFSET;
        o = (int)bp->b_un.b_addr & PGOFSET;
-       printf("%D ", bp->b_blkno + npf);
-       prdev("ECC", bp->b_dev);
-       mask = rp->hpec2&0xffff;
-
-       i = (rp->hpec1&0xffff) - 1;             /* -1 makes 0 origin */
-       bit = i&07;
-       i = (i&~07)>>3;
-       byte = i + o;
-       while (i < 512 && (int)ptob(npf)+i < bp->b_bcount && bit > -11) {
-               mpte = mbp->mba_map[reg+btop(byte)];
-               addr = ptob(mpte.pg_pfnum) + (byte & PGOFSET);
-               putmemc(addr, getmemc(addr)^(mask<<bit));
-               byte++;
-               i++;
-               bit -= 8;
-       }
-       mi->mi_hd->mh_active++;         /* Either complete or continuing */
-       if (bcr == 0)
-               return (0);
-#ifdef notdef
-       rp->hper1 = 0;
-       rp->hpcs1 = HP_RCOM|HP_GO;
-#else
-       rp->hpcs1 = HP_DCLR|HP_GO;
        bn = dkblock(bp);
        bn = dkblock(bp);
-       st = &hpst[mi->mi_type];
        cn = bp->b_cylin;
        cn = bp->b_cylin;
-       sn = bn%(st->nspc) + npf + 1;
+       sn = bn%(st->nspc) + npf;
        tn = sn/st->nsect;
        sn %= st->nsect;
        cn += tn/st->ntrak;
        tn %= st->ntrak;
        tn = sn/st->nsect;
        sn %= st->nsect;
        cn += tn/st->ntrak;
        tn %= st->ntrak;
+       switch (flag) {
+       case ECC:
+               {
+               register int i;
+               caddr_t addr;
+               struct pte mpte;
+               int bit, byte, mask;
+
+               npf--;          /* because block in error is previous block */
+               printf("hp%d%c: soft ecc sn%d\n", dkunit(bp),
+                   'a'+(minor(bp->b_dev)&07), bp->b_blkno + npf);
+               mask = rp->hpec2&0xffff;
+               i = (rp->hpec1&0xffff) - 1;             /* -1 makes 0 origin */
+               bit = i&07;
+               i = (i&~07)>>3;
+               byte = i + o;
+               while (i < 512 && (int)ptob(npf)+i < bp->b_bcount && bit > -11) {
+                       mpte = mbp->mba_map[npf+btop(byte)];
+                       addr = ptob(mpte.pg_pfnum) + (byte & PGOFSET);
+                       putmemc(addr, getmemc(addr)^(mask<<bit));
+                       byte++;
+                       i++;
+                       bit -= 8;
+               }
+               if (bcr == 0)
+                       return (0);
+               npf++;
+               break;
+               }
+
+       case SSE:
+               rp->hpof |= HPOF_SSEI;
+               mbp->mba_bcr = -(bp->b_bcount - (int)ptob(npf));
+               break;
+
+#ifndef NOBADSECT
+       case BSE:
+#ifdef HPBDEBUG
+               if (hpbdebug)
+               printf("hpecc, BSE: bn %d cn %d tn %d sn %d\n", bn, cn, tn, sn);
+#endif
+               if ((bn = isbad(&hpbad[mi->mi_unit], cn, tn, sn)) < 0)
+                       return(0);
+               bp->b_flags |= B_BAD;
+               bp->b_error = npf + 1;
+               bn = st->ncyl*st->nspc - st->nsect - 1 - bn;
+               cn = bn/st->nspc;
+               sn = bn%st->nspc;
+               tn = sn/st->nsect;
+               sn %= st->nsect;
+               mbp->mba_bcr = -512;
+#ifdef HPBDEBUG
+               if (hpbdebug)
+               printf("revector to cn %d tn %d sn %d\n", cn, tn, sn);
+#endif
+               break;
+
+       case CONT:
+#ifdef HPBDEBUG
+               if (hpbdebug)
+               printf("hpecc, CONT: bn %d cn %d tn %d sn %d\n", bn,cn,tn,sn);
+#endif
+               npf = bp->b_error;
+               bp->b_flags &= ~B_BAD;
+               mbp->mba_bcr = -(bp->b_bcount - (int)ptob(npf));
+               if ((mbp->mba_bcr & 0xffff) == 0)
+                       return(0);
+               break;
+#endif
+       }
+       rp->hpcs1 = HP_DCLR|HP_GO;
+       if (rp->hpof&HPOF_SSEI)
+               sn++;
        rp->hpdc = cn;
        rp->hpda = (tn<<8) + sn;
        mbp->mba_sr = -1;
        rp->hpdc = cn;
        rp->hpda = (tn<<8) + sn;
        mbp->mba_sr = -1;
-       mbp->mba_var = (int)ptob(reg+1) + o;
-       rp->hpcs1 = HP_RCOM|HP_GO;
-#endif
+       mbp->mba_var = (int)ptob(npf) + o;
+       rp->hpcs1 = bp->b_flags&B_READ ? HP_RCOM|HP_GO : HP_WCOM|HP_GO;
+       mi->mi_tab.b_errcnt = 0;        /* error has been corrected */
        return (1);
 }
 
        return (1);
 }
 
@@ -351,7 +542,7 @@ hpecc(mi)
 hpdump(dev)
        dev_t dev;
 {
 hpdump(dev)
        dev_t dev;
 {
-       register struct mba_info *mi;
+       register struct mba_device *mi;
        register struct mba_regs *mba;
        struct hpdevice *hpaddr;
        char *start;
        register struct mba_regs *mba;
        struct hpdevice *hpaddr;
        char *start;
@@ -361,29 +552,23 @@ hpdump(dev)
        num = maxfree;
        start = 0;
        unit = minor(dev) >> 3;
        num = maxfree;
        start = 0;
        unit = minor(dev) >> 3;
-       if (unit >= NHP) {
-               printf("bad unit\n");
-               return (-1);
-       }
+       if (unit >= NHP)
+               return (ENXIO);
 #define        phys(a,b)       ((b)((int)(a)&0x7fffffff))
 #define        phys(a,b)       ((b)((int)(a)&0x7fffffff))
-       mi = phys(hpinfo[unit],struct mba_info *);
-       if (mi == 0 || mi->mi_alive == 0) {
-               printf("dna\n");
-               return (-1);
-       }
+       mi = phys(hpinfo[unit],struct mba_device *);
+       if (mi == 0 || mi->mi_alive == 0)
+               return (ENXIO);
        mba = phys(mi->mi_hd, struct mba_hd *)->mh_physmba;
        mba = phys(mi->mi_hd, struct mba_hd *)->mh_physmba;
-       mba->mba_cr = MBAINIT;
+       mba->mba_cr = MBCR_INIT;
        hpaddr = (struct hpdevice *)&mba->mba_drv[mi->mi_drive];
        hpaddr = (struct hpdevice *)&mba->mba_drv[mi->mi_drive];
-       if ((hpaddr->hpds & HP_VV) == 0) {
+       if ((hpaddr->hpds & HPDS_VV) == 0) {
                hpaddr->hpcs1 = HP_DCLR|HP_GO;
                hpaddr->hpcs1 = HP_PRESET|HP_GO;
                hpaddr->hpcs1 = HP_DCLR|HP_GO;
                hpaddr->hpcs1 = HP_PRESET|HP_GO;
-               hpaddr->hpof = HP_FMT22;
+               hpaddr->hpof = HPOF_FMT22;
        }
        st = &hpst[mi->mi_type];
        }
        st = &hpst[mi->mi_type];
-       if (dumplo < 0 || dumplo + num >= st->sizes[minor(dev)&07].nblocks) {
-               printf("oor\n");
-               return (-1);
-       }
+       if (dumplo < 0 || dumplo + num >= st->sizes[minor(dev)&07].nblocks)
+               return (EINVAL);
        while (num > 0) {
                register struct pte *hpte = mba->mba_map;
                register int i;
        while (num > 0) {
                register struct pte *hpte = mba->mba_map;
                register int i;
@@ -404,13 +589,10 @@ hpdump(dev)
                mba->mba_bcr = -(blk*NBPG);
                mba->mba_var = 0;
                hpaddr->hpcs1 = HP_WCOM | HP_GO;
                mba->mba_bcr = -(blk*NBPG);
                mba->mba_var = 0;
                hpaddr->hpcs1 = HP_WCOM | HP_GO;
-               while ((hpaddr->hpds & HP_DRY) == 0)
+               while ((hpaddr->hpds & HPDS_DRY) == 0)
                        ;
                        ;
-               if (hpaddr->hpds&HP_ERR) {
-                       printf("dskerr: (%d,%d,%d) ds=%x er=%x\n",
-                           cn, tn, sn, hpaddr->hpds, hpaddr->hper1);
-                       return (-1);
-               }
+               if (hpaddr->hpds&HPDS_ERR)
+                       return (EIO);
                start += blk*NBPG;
                num -= blk;
        }
                start += blk*NBPG;
                num -= blk;
        }