reduce duplicate complaints
[unix-history] / usr / src / sys / vax / mba / hp.c
index 83c57d3..07da806 100644 (file)
@@ -1,4 +1,10 @@
-/*     hp.c    4.65    83/02/10        */
+/*
+ * Copyright (c) 1982 Regents of the University of California.
+ * All rights reserved.  The Berkeley software License Agreement
+ * specifies the terms and conditions for redistribution.
+ *
+ *     @(#)hp.c        6.16 (Berkeley) %G%
+ */
 
 #ifdef HPDEBUG
 int    hpdebug;
 
 #ifdef HPDEBUG
 int    hpdebug;
@@ -13,56 +19,63 @@ int hpbdebug;
  * HP disk driver for RP0x+RMxx+ML11
  *
  * TODO:
  * HP disk driver for RP0x+RMxx+ML11
  *
  * TODO:
- *     check RM80 skip sector handling when ECC's occur later
- *     check offset recovery handling
  *     see if DCLR and/or RELEASE set attention status
  *     see if DCLR and/or RELEASE set attention status
- *     print bits of mr && mr2 symbolically
  */
 #include "../machine/pte.h"
 
  */
 #include "../machine/pte.h"
 
-#include "../h/param.h"
-#include "../h/systm.h"
-#include "../h/dk.h"
-#include "../h/buf.h"
-#include "../h/conf.h"
-#include "../h/dir.h"
-#include "../h/user.h"
-#include "../h/map.h"
+#include "param.h"
+#include "systm.h"
+#include "dk.h"
+#include "buf.h"
+#include "conf.h"
+#include "dir.h"
+#include "user.h"
+#include "map.h"
 #include "../vax/mtpr.h"
 #include "../vax/mtpr.h"
-#include "../h/vm.h"
-#include "../h/cmap.h"
-#include "../h/dkbad.h"
-#include "../h/ioctl.h"
-#include "../h/uio.h"
+#include "vm.h"
+#include "cmap.h"
+#include "dkbad.h"
+#include "ioctl.h"
+#include "uio.h"
+#include "syslog.h"
 
 #include "../vax/dkio.h"
 
 #include "../vax/dkio.h"
-#include "../vaxmba/mbareg.h"
-#include "../vaxmba/mbavar.h"
-#include "../vaxmba/hpreg.h"
+#include "mbareg.h"
+#include "mbavar.h"
+#include "hpreg.h"
 
 /* THIS SHOULD BE READ OFF THE PACK, PER DRIVE */
 struct size {
        daddr_t nblocks;
        int     cyloff;
 
 /* THIS SHOULD BE READ OFF THE PACK, PER DRIVE */
 struct size {
        daddr_t nblocks;
        int     cyloff;
-} hp6_sizes[8] = {
+} rp06_sizes[8] = {
        15884,  0,              /* A=cyl 0 thru 37 */
        33440,  38,             /* B=cyl 38 thru 117 */
        340670, 0,              /* C=cyl 0 thru 814 */
        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,  118,            /* D=cyl 118 thru 155 */
+       55936,  156,            /* E=cyl 156 thru 289 */
+       219384, 290,            /* F=cyl 290 thru 814 */
        291280, 118,            /* G=cyl 118 thru 814 */
        0,      0,
        291280, 118,            /* G=cyl 118 thru 814 */
        0,      0,
-}, rm3_sizes[8] = {
+}, rp05_sizes[8] = {
+       15884,  0,              /* A=cyl 0 thru 37 */
+       33440,  38,             /* B=cyl 38 thru 117 */
+       171798, 0,              /* C=cyl 0 thru 410 */
+       15884,  118,            /* D=cyl 118 thru 155 */
+       55936,  156,            /* E=cyl 156 thru 289 */
+       50512,  290,            /* F=cyl 290 thru 410 */
+       122408, 118,            /* G=cyl 118 thru 410 */
+       0,      0,
+}, rm03_sizes[8] = {
        15884,  0,              /* A=cyl 0 thru 99 */
        15884,  0,              /* A=cyl 0 thru 99 */
-       33440,  100,            /* B=cyl 100 thru 309 */
+       33440,  100,            /* B=cyl 100 thru 308 */
        131680, 0,              /* C=cyl 0 thru 822 */
        131680, 0,              /* C=cyl 0 thru 822 */
+       15884,  309,            /* D=cyl 309 thru 408 */
+       55936,  409,            /* E=cyl 409 thru 758 */
+       10144,  759,            /* F=cyl 759 thru 822 */
+       82144,  309,            /* G=cyl 309 thru 822 */
        0,      0,
        0,      0,
-       0,      0,
-       0,      0,
-       81984,  310,            /* G=cyl 310 thru 822 */
-       0,      0,
-}, rm5_sizes[8] = {
+}, rm05_sizes[8] = {
        15884,  0,              /* A=cyl 0 thru 26 */
        33440,  27,             /* B=cyl 27 thru 81 */
        500384, 0,              /* C=cyl 0 thru 822 */
        15884,  0,              /* A=cyl 0 thru 26 */
        33440,  27,             /* B=cyl 27 thru 81 */
        500384, 0,              /* C=cyl 0 thru 822 */
@@ -75,74 +88,68 @@ struct      size {
        15884,  0,              /* A=cyl 0 thru 36 */
        33440,  37,             /* B=cyl 37 thru 114 */
        242606, 0,              /* C=cyl 0 thru 558 */
        15884,  0,              /* A=cyl 0 thru 36 */
        33440,  37,             /* B=cyl 37 thru 114 */
        242606, 0,              /* C=cyl 0 thru 558 */
+       15884,  115,            /* D=cyl 115 thru 151 */
+       55936,  152,            /* E=cyl 152 thru 280 */
+       120559, 281,            /* F=cyl 281 thru 558 */
+       192603, 115,            /* G=cyl 115 thru 558 */
        0,      0,
        0,      0,
-       0,      0,
-       0,      0,
-       82080,  115,            /* G=cyl 115 thru 304 */
-       110143, 305,            /* H=cyl 305 thru 558 */
-}, hp7_sizes[8] = {
+}, rp07_sizes[8] = {
        15884,  0,              /* A=cyl 0 thru 9 */
        15884,  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 */
-}, si9775_sizes[8] = {
-       16640,    0,            /* A=cyl 0 thru 12 */
-       34560,   13,            /* B=cyl 13 thru 39 */
-       1079040,  0,            /* C=cyl 0 thru 842 */
-       0,        0,
-       0,        0,
-       0,        0,
-       513280,  40,            /* G=cyl 40 thru 440 */
-       513280, 441,            /* H=cyl 441 thru 841 */
-}, si9730_sizes[8] = {
+       66880,  10,             /* B=cyl 10 thru 51 */
+       1008000, 0,             /* C=cyl 0 thru 629 */
+       15884,  235,            /* D=cyl 235 thru 244 */
+       307200, 245,            /* E=cyl 245 thru 436 */
+       308650, 437,            /* F=cyl 437 thru 629 */
+       631850, 235,            /* G=cyl 235 thru 629 */
+       291346, 52,             /* H=cyl 52 thru 234 */
+}, cdc9775_sizes[8] = {
+       15884,  0,              /* A=cyl 0 thru 12 */
+       66880,  13,             /* B=cyl 13 thru 65 */
+       1077760, 0,             /* C=cyl 0 thru 841 */
+       15884,  294,            /* D=cyl 294 thru 306 */
+       307200, 307,            /* E=cyl 307 thru 546 */
+       377440, 547,            /* F=cyl 547 thru 841 */
+       701280, 294,            /* G=cyl 294 thru 841 */
+       291346, 66,             /* H=cyl 66 thru 293 */
+}, cdc9730_sizes[8] = {
        15884,  0,              /* A=cyl 0 thru 49 */
        33440,  50,             /* B=cyl 50 thru 154 */
        263360, 0,              /* C=cyl 0 thru 822 */
        15884,  0,              /* A=cyl 0 thru 49 */
        33440,  50,             /* B=cyl 50 thru 154 */
        263360, 0,              /* C=cyl 0 thru 822 */
+       15884,  155,            /* D=cyl 155 thru 204 */
+       55936,  205,            /* E=cyl 205 thru 379 */
+       141664, 380,            /* F=cyl 380 thru 822 */
+       213664, 155,            /* G=cyl 155 thru 822 */
        0,      0,
        0,      0,
-       0,      0,
-       0,      0,
-       0,      0,
-       213664, 155,            /* H=cyl 155 thru 822 */
-}, hpam_sizes[8] = {
+}, capricorn_sizes[8] = {
        15884,  0,              /* A=cyl 0 thru 31 */
        33440,  32,             /* B=cyl 32 thru 97 */
        524288, 0,              /* C=cyl 0 thru 1023 */
        15884,  0,              /* A=cyl 0 thru 31 */
        33440,  32,             /* B=cyl 32 thru 97 */
        524288, 0,              /* C=cyl 0 thru 1023 */
-       27786,  668,
-       27786,  723,
-       125440, 778,
-       181760, 668,            /* G=cyl 668 thru 1022 */
+       15884,  668,            /* D=cyl 668 thru 699 */
+       55936,  700,            /* E=cyl 700 thru 809 */
+       109472, 810,            /* F=cyl 810 thru 1023 */
+       182176, 668,            /* G=cyl 668 thru 1023 */
        291346, 98,             /* H=cyl 98 thru 667 */
        291346, 98,             /* H=cyl 98 thru 667 */
-}, hpfj_sizes[8] = {
-       15884,  0,              /* A=cyl 0 thru 18 */
-       33440,  19,             /* B=cyl 19 thru 58 */
-       724120, 0,              /* C=cyl 0 thru 841 */
-       0, 0,
-       0, 0,
-       0, 0,
-       381711, 398,            /* G=cyl 398 thru 841 */
-       291346, 59,             /* H=cyl 59 thru 397 */
-}, hpfj48_sizes[8] = {
+}, eagle_sizes[8] = {
        15884,  0,              /* A=cyl 0 thru 16 */
        15884,  0,              /* A=cyl 0 thru 16 */
-       33440,  17,             /* B=cyl 17 thru 52 */
-       808176, 0,              /* C=cyl 0 thru 841 */
-       0, 0,
-       0, 0,
-       0, 0,
-       465456, 357,            /* G=cyl 357 thru 841 */
-       291346, 53,             /* H=cyl 52 thru 356 */
+       66880,  17,             /* B=cyl 17 thru 86 */
+       808320, 0,              /* C=cyl 0 thru 841 */
+       15884,  391,            /* D=cyl 391 thru 407 */
+       307200, 408,            /* E=cyl 408 thru 727 */
+       109296, 728,            /* F=cyl 728 thru 841 */
+       432816, 391,            /* G=cyl 391 thru 841 */
+       291346, 87,             /* H=cyl 87 thru 390 */
+}, ampex_sizes[8] = {
+       15884,  0,              /* A=cyl 0 thru 26 */
+       33440,  27,             /* B=cyl 27 thru 81 */
+       495520, 0,              /* C=cyl 0 thru 814 */
+       15884,  562,            /* D=cyl 562 thru 588 */
+       55936,  589,            /* E=cyl 589 thru 680 */
+       81312,  681,            /* F=cyl 681 thru 814 */
+       153664, 562,            /* G=cyl 562 thru 814 */
+       291346, 82,             /* H=cyl 82 thru 561 */
 };
 /* END OF STUFF WHICH SHOULD BE READ IN PER DISK */
 
 };
 /* END OF STUFF WHICH SHOULD BE READ IN PER DISK */
 
-#define        _hpSDIST        2
-#define        _hpRDIST        3
-
-int    hpSDIST = _hpSDIST;
-int    hpRDIST = _hpRDIST;
-
 /*
  * Table for converting Massbus drive types into
  * indices into the partition tables.  Slots are
 /*
  * Table for converting Massbus drive types into
  * indices into the partition tables.  Slots are
@@ -158,25 +165,27 @@ short     hptypes[] = {
        MBDT_RP06,
 #define        HPDT_RM80       3
        MBDT_RM80,
        MBDT_RP06,
 #define        HPDT_RM80       3
        MBDT_RM80,
-#define        HPDT_RP05       4
+#define        HPDT_RP04       4
+       MBDT_RP04,
+#define        HPDT_RP05       5
        MBDT_RP05,
        MBDT_RP05,
-#define        HPDT_RP07       5
+#define        HPDT_RP07       6
        MBDT_RP07,
        MBDT_RP07,
-#define        HPDT_ML11A      6
+#define        HPDT_ML11A      7
        MBDT_ML11A,
        MBDT_ML11A,
-#define        HPDT_ML11B      7
+#define        HPDT_ML11B      8
        MBDT_ML11B,
        MBDT_ML11B,
-#define        HPDT_9775       8
+#define        HPDT_9775       9
        -1,
        -1,
-#define        HPDT_9730       9
+#define        HPDT_9730       10
        -1,
        -1,
-#define        HPDT_CAPRICORN  10
+#define        HPDT_CAPRICORN  11
        -1,
        -1,
-#define HPDT_EAGLE     11
+#define HPDT_EAGLE     12
        -1,
        -1,
-#define        HPDT_EAGLE48    12
+#define        HPDT_9300       13
        -1,
        -1,
-#define HPDT_RM02      13
+#define HPDT_RM02      14
        MBDT_RM02,              /* beware, actually capricorn or eagle */
        0
 };
        MBDT_RM02,              /* beware, actually capricorn or eagle */
        0
 };
@@ -186,26 +195,44 @@ struct    mba_driver hpdriver =
        { hpattach, 0, hpustart, hpstart, hpdtint, 0,
          hptypes, "hp", 0, hpinfo };
 
        { hpattach, 0, hpustart, hpstart, hpdtint, 0,
          hptypes, "hp", 0, hpinfo };
 
+/*
+ * These variable are all measured in sectors.  
+ * Sdist is how much to "lead" in the search for a desired sector
+ * (i.e. if want N, search for N-sdist.)
+ * Maxdist and mindist define the region right before our desired sector within
+ * which we don't bother searching.  We don't search when we are already less
+ * then maxdist and more than mindist sectors "before" our desired sector.
+ * Maxdist should be >= sdist.
+ * 
+ * Beware, sdist, mindist and maxdist are not well tuned
+ * for many of the drives listed in this table.
+ * Try patching things with something i/o intensive
+ * running and watch iostat.
+ */
 struct hpst {
 struct hpst {
-       short   nsect;
-       short   ntrak;
-       short   nspc;
-       short   ncyl;
-       struct  size *sizes;
+       short   nsect;          /* # sectors/track */
+       short   ntrak;          /* # tracks/cylinder */
+       short   nspc;           /* # sector/cylinders */
+       short   ncyl;           /* # cylinders */
+       struct  size *sizes;    /* partition tables */
+       short   sdist;          /* seek distance metric */
+       short   maxdist;        /* boundaries of non-searched area */
+       short   mindist;        /* preceding the target sector */
 } hpst[] = {
 } hpst[] = {
-       32,     5,      32*5,   823,    rm3_sizes,      /* RM03 */
-       32,     19,     32*19,  823,    rm5_sizes,      /* RM05 */
-       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 */
-       1,      1,      1,      1,      0,              /* ML11A */
-       1,      1,      1,      1,      0,              /* ML11B */
-       32,     40,     32*40,  843,    si9775_sizes,   /* 9775 */
-       32,     10,     32*10,  823,    si9730_sizes,   /* 9730 */
-       32,     16,     32*16,  1024,   hpam_sizes,     /* AMPEX capricorn */
-       43,     20,     43*20,  842,    hpfj_sizes,     /* Fujitsu EAGLE */
-       48,     20,     48*20,  842,    hpfj48_sizes,   /* 48 sector EAGLE */
+    { 32, 5,   32*5,   823,    rm03_sizes,     7, 4, 1 },      /* RM03 */
+    { 32, 19,  32*19,  823,    rm05_sizes,     7, 4, 1 },      /* RM05 */
+    { 22,19,   22*19,  815,    rp06_sizes,     7, 4, 1 },      /* RP06 */
+    { 31, 14,  31*14,  559,    rm80_sizes,     7, 4, 1 },      /* RM80 */
+    { 22, 19,  22*19,  411,    rp05_sizes,     7, 4, 1 },      /* RP04 */
+    { 22, 19,  22*19,  411,    rp05_sizes,     7, 4, 1 },      /* RP05 */
+    { 50, 32,  50*32,  630,    rp07_sizes,    15, 8, 3 },      /* RP07 */
+    { 1, 1,    1,      1,      0,              0, 0, 0 },      /* ML11A */
+    { 1, 1,    1,      1,      0,              0, 0, 0 },      /* ML11B */
+    { 32, 40,  32*40,  843,    cdc9775_sizes,  7, 4, 1 },      /* 9775 */
+    { 32, 10,  32*10,  823,    cdc9730_sizes,  7, 4, 1 },      /* 9730 */
+    { 32, 16,  32*16,  1024,   capricorn_sizes,10,4, 3 },      /* Capricorn */
+    { 48, 20,  48*20,  842,    eagle_sizes,   15, 8, 3 },      /* EAGLE */
+    { 32, 19,  32*19,  815,    ampex_sizes,    7, 4, 1 },      /* 9300 */
 };
 
 u_char hp_offset[16] = {
 };
 
 u_char hp_offset[16] = {
@@ -225,6 +252,8 @@ struct      hpsoftc {
        u_char  sc_hdr;         /* next i/o includes header */
        u_char  sc_doseeks;     /* perform explicit seeks */
        daddr_t sc_mlsize;      /* ML11 size */
        u_char  sc_hdr;         /* next i/o includes header */
        u_char  sc_doseeks;     /* perform explicit seeks */
        daddr_t sc_mlsize;      /* ML11 size */
+       int     sc_blkdone;     /* amount sucessfully transfered */
+       daddr_t sc_badbn;       /* replacement block number */
 } hpsoftc[NHP];
 
 #define        b_cylin b_resid
 } hpsoftc[NHP];
 
 #define        b_cylin b_resid
@@ -234,9 +263,9 @@ struct      hpsoftc {
 #define        RP06    (hptypes[mi->mi_type] <= MBDT_RP06)
 #define        RM80    (hptypes[mi->mi_type] == MBDT_RM80)
 
 #define        RP06    (hptypes[mi->mi_type] <= MBDT_RP06)
 #define        RM80    (hptypes[mi->mi_type] == MBDT_RM80)
 
-#ifdef INTRLVE
-daddr_t dkblock();
-#endif
+#define hpunit(dev)    (minor(dev) >> 3)
+#define        MASKREG(reg)    ((reg)&0xffff)
+#define HPWAIT(mi, addr) (((addr)->hpds & HPDS_DRY) || hpwait(mi))
 
 /*ARGSUSED*/
 hpattach(mi, slave)
 
 /*ARGSUSED*/
 hpattach(mi, slave)
@@ -265,7 +294,7 @@ hpmaptype(mi)
        register int type = mi->mi_type;
 
        /*
        register int type = mi->mi_type;
 
        /*
-        * Model-byte processing for SI 9400 controllers.
+        * Model-byte processing for SI controllers.
         * NB:  Only deals with RM03 and RM05 emulations.
         */
        if (type == HPDT_RM03 || type == HPDT_RM05) {
         * NB:  Only deals with RM03 and RM05 emulations.
         */
        if (type == HPDT_RM03 || type == HPDT_RM05) {
@@ -276,32 +305,40 @@ hpmaptype(mi)
                switch ((hpsn & SIMB_MB) & ~(SIMB_S6|SIRM03|SIRM05)) {
 
                case SI9775D:
                switch ((hpsn & SIMB_MB) & ~(SIMB_S6|SIRM03|SIRM05)) {
 
                case SI9775D:
-                       printf("hp%d: si 9775 (direct)\n", mi->mi_unit);
+                       printf("hp%d: 9775 (direct)\n", mi->mi_unit);
                        type = HPDT_9775;
                        break;
 
                case SI9730D:
                        type = HPDT_9775;
                        break;
 
                case SI9730D:
-                       printf("hp%d: si 9730 (direct)\n", mi->mi_unit);
+                       printf("hp%d: 9730 (direct)\n", mi->mi_unit);
                        type = HPDT_9730;
                        break;
 
                /*
                        type = HPDT_9730;
                        break;
 
                /*
-                * AMPEX 9300, SI Combination needs a have the
-                * drive cleared before we start.  We do not know
-                * why, but tests show that the recalibrate fixes
-                * the problem.
+                * Beware, since the only SI controller we
+                * have has a 9300 instead of a 9766, we map the
+                * drive type into the 9300.  This means that
+                * on a 9766 you lose the last 8 cylinders (argh).
                 */
                case SI9766:
                 */
                case SI9766:
-                       printf("hp%d: 9776/9300\n", mi->mi_unit);
-                       type = HPDT_RM05;
-                       hpaddr->hpcs1 = HP_RECAL|HP_GO;
-                       DELAY(100000);
+                       printf("hp%d: 9300\n", mi->mi_unit);
+                       type = HPDT_9300;
                        break;
 
                case SI9762:
                        printf("hp%d: 9762\n", mi->mi_unit);
                        type = HPDT_RM03;
                        break;
                        break;
 
                case SI9762:
                        printf("hp%d: 9762\n", mi->mi_unit);
                        type = HPDT_RM03;
                        break;
+
+               case SICAPD:
+                       printf("hp%d: capricorn\n", mi->mi_unit);
+                       type = HPDT_CAPRICORN;
+                       break;
+
+               case SI9751D:
+                       printf("hp%d: eagle\n", mi->mi_unit);
+                       type = HPDT_EAGLE;
+                       break;
                }
                return (type);
        }
                }
                return (type);
        }
@@ -312,31 +349,34 @@ hpmaptype(mi)
        if (type == HPDT_RM02) {
                int ntracks, nsectors;
 
        if (type == HPDT_RM02) {
                int ntracks, nsectors;
 
+               hpaddr->hpof = HPOF_FMT22;
+               mbclrattn(mi);
                hpaddr->hpcs1 = HP_NOP;
                hpaddr->hphr = HPHR_MAXTRAK;
                hpaddr->hpcs1 = HP_NOP;
                hpaddr->hphr = HPHR_MAXTRAK;
-               ntracks = (hpaddr->hphr & 0xffff) + 1;
+               ntracks = MASKREG(hpaddr->hphr) + 1;
                if (ntracks == 16) {
                        printf("hp%d: capricorn\n", mi->mi_unit);
                        type = HPDT_CAPRICORN;
                        goto done;
                }
                if (ntracks == 16) {
                        printf("hp%d: capricorn\n", mi->mi_unit);
                        type = HPDT_CAPRICORN;
                        goto done;
                }
-               if (ntracks != 20) {
-                       printf("hp%d: ntracks %d: unknown device\n", ntracks);
+               if (ntracks == 19) {
+                       printf("hp%d: 9300\n", mi->mi_unit);
+                       type = HPDT_9300;
                        goto done;
                }
                hpaddr->hpcs1 = HP_NOP;
                hpaddr->hphr = HPHR_MAXSECT;
                        goto done;
                }
                hpaddr->hpcs1 = HP_NOP;
                hpaddr->hphr = HPHR_MAXSECT;
-               nsectors = (hpaddr->hphr & 0xffff) + 1;
-               printf("hp%d: ", mi->mi_unit);
-               if (nsectors == 43)
+               nsectors = MASKREG(hpaddr->hphr) + 1;
+               if (ntracks == 20 && nsectors == 48) {
                        type = HPDT_EAGLE;
                        type = HPDT_EAGLE;
-               else {
-                       type = HPDT_EAGLE48;
-                       printf("modified ");
+                       printf("hp%d: eagle\n", mi->mi_unit);
+                       goto done;
                }
                }
-               printf("eagle\n");
+               printf("hp%d: ntracks %d, nsectors %d: unknown device\n",
+                       mi->mi_unit, ntracks, nsectors);
 done:
                hpaddr->hpcs1 = HP_DCLR|HP_GO;
 done:
                hpaddr->hpcs1 = HP_DCLR|HP_GO;
+               mbclrattn(mi);          /* conservative */
                return (type);
        } 
 
                return (type);
        } 
 
@@ -363,7 +403,7 @@ done:
 hpopen(dev)
        dev_t dev;
 {
 hpopen(dev)
        dev_t dev;
 {
-       register int unit = minor(dev) >> 3;
+       register int unit = hpunit(dev);
        register struct mba_device *mi;
 
        if (unit >= NHP || (mi = hpinfo[unit]) == 0 || mi->mi_alive == 0)
        register struct mba_device *mi;
 
        if (unit >= NHP || (mi = hpinfo[unit]) == 0 || mi->mi_alive == 0)
@@ -377,31 +417,47 @@ hpstrategy(bp)
        register struct mba_device *mi;
        register struct hpst *st;
        register int unit;
        register struct mba_device *mi;
        register struct hpst *st;
        register int unit;
-       long sz, bn;
+       long sz;
        int xunit = minor(bp->b_dev) & 07;
        int s;
 
        sz = bp->b_bcount;
        sz = (sz+511) >> 9;
        int xunit = minor(bp->b_dev) & 07;
        int s;
 
        sz = bp->b_bcount;
        sz = (sz+511) >> 9;
-       unit = dkunit(bp);
-       if (unit >= NHP)
+       unit = hpunit(bp->b_dev);
+       if (unit >= NHP) {
+               bp->b_error = ENXIO;
                goto bad;
                goto bad;
+       }
        mi = hpinfo[unit];
        mi = hpinfo[unit];
-       if (mi == 0 || mi->mi_alive == 0)
+       if (mi == 0 || mi->mi_alive == 0) {
+               bp->b_error = ENXIO;
                goto bad;
                goto bad;
+       }
        st = &hpst[mi->mi_type];
        if (ML11) {
                struct hpsoftc *sc = &hpsoftc[unit];
 
                if (bp->b_blkno < 0 ||
        st = &hpst[mi->mi_type];
        if (ML11) {
                struct hpsoftc *sc = &hpsoftc[unit];
 
                if (bp->b_blkno < 0 ||
-                   dkblock(bp)+sz > sc->sc_mlsize)
+                   bp->b_blkno+sz > sc->sc_mlsize) {
+                       if (bp->b_blkno == sc->sc_mlsize) {
+                           bp->b_resid = bp->b_bcount;
+                           goto done;
+                       }
+                       bp->b_error = EINVAL;
                        goto bad;
                        goto bad;
+               }
                bp->b_cylin = 0;
        } else {
                if (bp->b_blkno < 0 ||
                bp->b_cylin = 0;
        } else {
                if (bp->b_blkno < 0 ||
-                   (bn = dkblock(bp))+sz > st->sizes[xunit].nblocks)
+                   bp->b_blkno+sz > st->sizes[xunit].nblocks) {
+                       if (bp->b_blkno == st->sizes[xunit].nblocks) {
+                           bp->b_resid = bp->b_bcount;
+                           goto done;
+                       }
+                       bp->b_error = EINVAL;
                        goto bad;
                        goto bad;
-               bp->b_cylin = bn/st->nspc + st->sizes[xunit].cyloff;
+               }
+               bp->b_cylin = bp->b_blkno/st->nspc + st->sizes[xunit].cyloff;
        }
        s = spl5();
        disksort(&mi->mi_tab, bp);
        }
        s = spl5();
        disksort(&mi->mi_tab, bp);
@@ -412,6 +468,7 @@ hpstrategy(bp)
 
 bad:
        bp->b_flags |= B_ERROR;
 
 bad:
        bp->b_flags |= B_ERROR;
+done:
        iodone(bp);
        return;
 }
        iodone(bp);
        return;
 }
@@ -421,14 +478,28 @@ hpustart(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 = &hpst[mi->mi_type];
+       register struct hpst *st;
        struct hpsoftc *sc = &hpsoftc[mi->mi_unit];
        daddr_t bn;
        struct hpsoftc *sc = &hpsoftc[mi->mi_unit];
        daddr_t bn;
-       int sn, dist;
+       int sn, tn, dist;
 
 
+       st = &hpst[mi->mi_type];
        hpaddr->hpcs1 = 0;
        if ((hpaddr->hpcs1&HP_DVA) == 0)
                return (MBU_BUSY);
        hpaddr->hpcs1 = 0;
        if ((hpaddr->hpcs1&HP_DVA) == 0)
                return (MBU_BUSY);
+
+       switch (sc->sc_recal) {
+
+       case 1:
+               HPWAIT(mi, hpaddr);
+               hpaddr->hpdc = bp->b_cylin;
+               hpaddr->hpcs1 = HP_SEEK|HP_GO;
+               sc->sc_recal++;
+               return (MBU_STARTED);
+       case 2:
+               break;
+       }
+       sc->sc_recal = 0;
        if ((hpaddr->hpds & HPDS_VV) == 0 || !sc->sc_hpinit) {
                struct buf *bbp = &bhpbuf[mi->mi_unit];
 
        if ((hpaddr->hpds & HPDS_VV) == 0 || !sc->sc_hpinit) {
                struct buf *bbp = &bhpbuf[mi->mi_unit];
 
@@ -452,29 +523,39 @@ hpustart(mi)
                        bp = bbp;
                }
        }
                        bp = bbp;
                }
        }
-       if (mi->mi_tab.b_active || mi->mi_hd->mh_ndrive == 1)
+       if (mi->mi_tab.b_active || mi->mi_hd->mh_ndrive == 1) {
+               if (mi->mi_tab.b_errcnt >= 16 && (bp->b_flags & B_READ)) {
+                       hpaddr->hpof =
+                           hp_offset[mi->mi_tab.b_errcnt & 017]|HPOF_FMT22;
+                       hpaddr->hpcs1 = HP_OFFSET|HP_GO;
+                       HPWAIT(mi, hpaddr);
+                       mbclrattn(mi);
+               }
                return (MBU_DODATA);
                return (MBU_DODATA);
+       }
        if (ML11)
                return (MBU_DODATA);
        if ((hpaddr->hpds & HPDS_DREADY) != HPDS_DREADY)
                return (MBU_DODATA);
        if (ML11)
                return (MBU_DODATA);
        if ((hpaddr->hpds & HPDS_DREADY) != HPDS_DREADY)
                return (MBU_DODATA);
-       bn = dkblock(bp);
-       sn = bn%st->nspc;
-       sn = (sn+st->nsect-hpSDIST)%st->nsect;
-       if (bp->b_cylin == (hpaddr->hpdc & 0xffff)) {
-               if (sc->sc_seek)
+       bn = bp->b_blkno;
+       sn = bn % st->nspc;
+       tn = sn / st->nsect;
+       sn = sn % st->nsect;
+       if (bp->b_cylin == MASKREG(hpaddr->hpdc)) {
+               if (sc->sc_doseeks)
                        return (MBU_DODATA);
                        return (MBU_DODATA);
-               dist = ((hpaddr->hpla & 0xffff)>>6) - st->nsect + 1;
+               dist = sn - (MASKREG(hpaddr->hpla) >> 6) - 1;
                if (dist < 0)
                        dist += st->nsect;
                if (dist < 0)
                        dist += st->nsect;
-               if (dist > st->nsect - hpRDIST)
+               if (dist > st->maxdist || dist < st->mindist)
                        return (MBU_DODATA);
        } else
                hpaddr->hpdc = bp->b_cylin;
                        return (MBU_DODATA);
        } else
                hpaddr->hpdc = bp->b_cylin;
-       if (sc->sc_seek)
+       if (sc->sc_doseeks)
                hpaddr->hpcs1 = HP_SEEK|HP_GO;
        else {
                hpaddr->hpcs1 = HP_SEEK|HP_GO;
        else {
-               hpaddr->hpda = sn;
+               sn = (sn + st->nsect - st->sdist) % st->nsect;
+               hpaddr->hpda = (tn << 8) + sn;
                hpaddr->hpcs1 = HP_SEARCH|HP_GO;
        }
        return (MBU_STARTED);
                hpaddr->hpcs1 = HP_SEARCH|HP_GO;
        }
        return (MBU_STARTED);
@@ -488,17 +569,27 @@ hpstart(mi)
        register struct hpst *st = &hpst[mi->mi_type];
        struct hpsoftc *sc = &hpsoftc[mi->mi_unit];
        daddr_t bn;
        register struct hpst *st = &hpst[mi->mi_type];
        struct hpsoftc *sc = &hpsoftc[mi->mi_unit];
        daddr_t bn;
-       int sn, tn;
+       int sn, tn, cn;
 
 
-       bn = dkblock(bp);
        if (ML11)
        if (ML11)
-               hpaddr->hpda = bn;
+               hpaddr->hpda = bp->b_blkno + sc->sc_blkdone;
        else {
        else {
-               sn = bn%st->nspc;
-               tn = sn/st->nsect;
+               if (bp->b_flags & B_BAD) {
+                       bn = sc->sc_badbn;
+                       cn = bn / st->nspc;
+               } else {
+                       bn = bp->b_blkno;
+                       cn = bp->b_cylin;
+               }
+               sn = bn % st->nspc;
+               if ((bp->b_flags & B_BAD) == 0)
+                       sn += sc->sc_blkdone;
+               tn = sn / st->nsect;
                sn %= st->nsect;
                sn %= st->nsect;
-               hpaddr->hpdc = bp->b_cylin;
+               cn += tn / st->ntrak;
+               tn %= st->ntrak;
                hpaddr->hpda = (tn << 8) + sn;
                hpaddr->hpda = (tn << 8) + sn;
+               hpaddr->hpdc = cn;
        }
        if (sc->sc_hdr) {
                if (bp->b_flags & B_READ)
        }
        if (sc->sc_hdr) {
                if (bp->b_flags & B_READ)
@@ -515,141 +606,138 @@ hpdtint(mi, 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;
-       register struct hpst *st = &hpst[mi->mi_type];
+       register struct hpst *st;
        register int er1, er2;
        struct hpsoftc *sc = &hpsoftc[mi->mi_unit];
        register int er1, er2;
        struct hpsoftc *sc = &hpsoftc[mi->mi_unit];
-       int retry = 0;
+       int retry = 0, i;
 
 
-       if (bp->b_flags&B_BAD) {
-               if (hpecc(mi, CONT))
-                       return(MBD_RESTARTED);
-       }
+       st = &hpst[mi->mi_type];
        if (hpaddr->hpds&HPDS_ERR || mbsr&MBSR_EBITS) {
        if (hpaddr->hpds&HPDS_ERR || mbsr&MBSR_EBITS) {
+               er1 = hpaddr->hper1;
+               er2 = hpaddr->hper2;
+               if (HPWAIT(mi, hpaddr) == 0)
+                       goto hard;
 #ifdef HPDEBUG
                if (hpdebug) {
                        int dc = hpaddr->hpdc, da = hpaddr->hpda;
 
 #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);
+                       log(LOG_DEBUG,
+                   "hperr: bp %x cyl %d blk %d blkdone %d as %o dc %x da %x\n",
+                               bp, bp->b_cylin, bp->b_blkno, sc->sc_blkdone,
+                               hpaddr->hpas&0xff, MASKREG(dc), MASKREG(da));
+                       log(LOG_DEBUG, "errcnt %d mbsr=%b er1=%b er2=%b\n",
+                               mi->mi_tab.b_errcnt, mbsr, mbsr_bits,
+                               MASKREG(er1), HPER1_BITS,
+                               MASKREG(er2), HPER2_BITS);
                }
 #endif
                }
 #endif
-               er1 = hpaddr->hper1;
-               er2 = hpaddr->hper2;
                if (er1 & HPER1_HCRC) {
                        er1 &= ~(HPER1_HCE|HPER1_FER);
                        er2 &= ~HPER2_BSE;
                }
                if (er1 & HPER1_HCRC) {
                        er1 &= ~(HPER1_HCE|HPER1_FER);
                        er2 &= ~HPER2_BSE;
                }
-               if (er1&HPER1_WLE) {
-                       printf("hp%d: write locked\n", dkunit(bp));
+               if (er1 & HPER1_WLE) {
+                       log(LOG_WARNING, "hp%d: write locked\n",
+                           hpunit(bp->b_dev));
                        bp->b_flags |= B_ERROR;
                        bp->b_flags |= B_ERROR;
-               } else if ((er1&0xffff) == HPER1_FER && RP06 && !sc->sc_hdr) {
+               } else if (sc->sc_hdr) {
+                       goto hard;
+               } else if (RM80 && er2&HPER2_SSE) {
+                       (void) hpecc(mi, SSE);
+                       return (MBD_RESTARTED);
+               } else if ((er2 & HPER2_BSE) && !ML11) {
+                       if (hpecc(mi, BSE))
+                               return (MBD_RESTARTED);
+                       goto hard;
+               } else if (MASKREG(er1) == HPER1_FER && RP06) {
                        if (hpecc(mi, BSE))
                        if (hpecc(mi, BSE))
-                               return(MBD_RESTARTED);
-                       else
-                               goto hard;
+                               return (MBD_RESTARTED);
+                       goto hard;
+               } else if ((er1 & (HPER1_DCK | HPER1_ECH)) == HPER1_DCK &&
+                   mi->mi_tab.b_errcnt >= 3) {
+                       if (hpecc(mi, ECC))
+                               return (MBD_RESTARTED);
+                       /*
+                        * ECC corrected.  Only log retries below
+                        * if we got errors other than soft ECC
+                        * (as indicated by additional retries).
+                        */
+                       if (mi->mi_tab.b_errcnt == 3)
+                               mi->mi_tab.b_errcnt = 0;
+               } else if ((er1 & HPER1_HCRC) && !ML11 && hpecc(mi, BSE)) {
+                       /*
+                        * HCRC means the header is screwed up and the sector
+                        * might well exist in the bad sector table, 
+                        * better check....
+                        */
+                       return (MBD_RESTARTED);
                } else if (++mi->mi_tab.b_errcnt > 27 ||
                } else if (++mi->mi_tab.b_errcnt > 27 ||
+                   (ML11 && mi->mi_tab.b_errcnt > 15) ||
                    mbsr & MBSR_HARD ||
                    er1 & HPER1_HARD ||
                    mbsr & MBSR_HARD ||
                    er1 & HPER1_HARD ||
-                   sc->sc_hdr ||
                    (!ML11 && (er2 & HPER2_HARD))) {
 hard:
                    (!ML11 && (er2 & HPER2_HARD))) {
 hard:
-                       if (ML11)
-                               bp->b_blkno = hpaddr->hpda&0xffff;
-                       else
-                               bp->b_blkno = (hpaddr->hpdc*st->nspc)&0xffff +
-                                         ((hpaddr->hpda>>8)&0xffff)*st->nsect +
-                                               (hpaddr->hpda&0x1f);
+                       if (bp->b_flags & B_BAD)
+                               bp->b_blkno = sc->sc_badbn;
+                       else {
+                               bp->b_blkno = bp->b_blkno + btop(bp->b_bcount -
+                                   MASKREG(-mi->mi_mba->mba_bcr));
+                               if (er1 & (HPER1_DCK | HPER1_ECH))
+                                       bp->b_blkno--;
+                       }
                        harderr(bp, "hp");
                        if (mbsr & (MBSR_EBITS &~ (MBSR_DTABT|MBSR_MBEXC)))
                                printf("mbsr=%b ", mbsr, mbsr_bits);
                        printf("er1=%b er2=%b",
                        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);
+                           MASKREG(hpaddr->hper1), HPER1_BITS,
+                           MASKREG(hpaddr->hper2), HPER2_BITS);
+                       if (sc->sc_hdr)
+                               printf(" (hdr i/o)");
                        printf("\n");
                        bp->b_flags |= B_ERROR;
                        printf("\n");
                        bp->b_flags |= B_ERROR;
-                       retry = 0;
-                       sc->sc_recal = 0;
-               } else if ((er2 & HPER2_BSE) && !ML11) {
-                       if (hpecc(mi, BSE))
-                               return(MBD_RESTARTED);
-                       else
-                               goto hard;
-               } else if (RM80 && er2&HPER2_SSE) {
-                       (void) hpecc(mi, SSE);
-                       return (MBD_RESTARTED);
-               } else if ((er1&(HPER1_DCK|HPER1_ECH))==HPER1_DCK) {
-                       if (hpecc(mi, ECC))
-                               return (MBD_RESTARTED);
-                       /* else done */
+                       bp->b_flags &= ~B_BAD;
                } else
                        retry = 1;
                hpaddr->hpcs1 = HP_DCLR|HP_GO;
                } else
                        retry = 1;
                hpaddr->hpcs1 = HP_DCLR|HP_GO;
-               if (ML11) {
-                       if (mi->mi_tab.b_errcnt >= 16)
-                               goto hard;
-               } else if ((mi->mi_tab.b_errcnt&07) == 4) {
+               if ((mi->mi_tab.b_errcnt & 07) == 4) {
                        hpaddr->hpcs1 = HP_RECAL|HP_GO;
                        sc->sc_recal = 1;
                        hpaddr->hpcs1 = HP_RECAL|HP_GO;
                        sc->sc_recal = 1;
-                       return(MBD_RESTARTED);
+                       return (MBD_REPOSITION);
                }
                }
-               if (retry)
-                       return (MBD_RETRY);
        }
 #ifdef HPDEBUG
        else
                if (hpdebug && sc->sc_recal) {
        }
 #ifdef HPDEBUG
        else
                if (hpdebug && sc->sc_recal) {
-                       printf("recal %d ", sc->sc_recal);
-                       printf("errcnt %d\n", mi->mi_tab.b_errcnt);
-                       printf("mbsr=%b ", mbsr, mbsr_bits);
-                       printf("er1=%b er2=%b\n",
+                       log(LOG_DEBUG,
+                           "recal %d errcnt %d mbsr=%b er1=%b er2=%b\n",
+                           sc->sc_recal, mi->mi_tab.b_errcnt, mbsr, mbsr_bits,
                            hpaddr->hper1, HPER1_BITS,
                            hpaddr->hper2, HPER2_BITS);
                }
 #endif
                            hpaddr->hper1, HPER1_BITS,
                            hpaddr->hper2, HPER2_BITS);
                }
 #endif
-       switch (sc->sc_recal) {
-
-       case 1:
-               hpaddr->hpdc = bp->b_cylin;
-               hpaddr->hpcs1 = HP_SEEK|HP_GO;
-               sc->sc_recal++;
-               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;
-               sc->sc_recal++;
-               return (MBD_RESTARTED);
-       donerecal:
-       case 3:
-               sc->sc_recal = 0;
+       HPWAIT(mi, hpaddr);
+       if (retry)
                return (MBD_RETRY);
                return (MBD_RETRY);
-       }
-       sc->sc_hdr = 0;
-       bp->b_resid = -(mi->mi_mba->mba_bcr) & 0xffff;
        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;
        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;
-               while (hpaddr->hpds & HPDS_PIP)
-                       ;
+               HPWAIT(mi, hpaddr);
                mbclrattn(mi);
        }
                mbclrattn(mi);
        }
+       if (mi->mi_tab.b_errcnt && (bp->b_flags & B_ERROR) == 0)
+               log(LOG_INFO, "hp%d%c: %d retries %sing sn%d\n",
+                   hpunit(bp->b_dev), 'a'+(minor(bp->b_dev)&07),
+                   mi->mi_tab.b_errcnt,
+                   (bp->b_flags & B_READ) ? "read" : "writ",
+                   (bp->b_flags & B_BAD) ?
+                   sc->sc_badbn : bp->b_blkno + sc->sc_blkdone);
+       if ((bp->b_flags & B_BAD) && hpecc(mi, CONT))
+               return (MBD_RESTARTED);
+       sc->sc_hdr = 0;
+       sc->sc_blkdone = 0;
+       bp->b_resid = MASKREG(-mi->mi_mba->mba_bcr);
        if (!ML11) {
                hpaddr->hpof = HPOF_FMT22;
                hpaddr->hpcs1 = HP_RELEASE|HP_GO;
        if (!ML11) {
                hpaddr->hpof = HPOF_FMT22;
                hpaddr->hpcs1 = HP_RELEASE|HP_GO;
@@ -657,11 +745,28 @@ hard:
        return (MBD_DONE);
 }
 
        return (MBD_DONE);
 }
 
+/*
+ * Wait (for a bit) for a drive to come ready;
+ * returns nonzero on success.
+ */
+hpwait(mi)
+       register struct mba_device *mi;
+{
+       register struct hpdevice *hpaddr = (struct hpdevice *)mi->mi_drv;
+       register i = 100000;
+
+       while ((hpaddr->hpds & HPDS_DRY) == 0 && --i)
+               DELAY(10);
+       if (i == 0)
+               printf("hp%d: intr, not ready\n", mi->mi_unit);
+       return (i);
+}
+
 hpread(dev, uio)
        dev_t dev;
        struct uio *uio;
 {
 hpread(dev, uio)
        dev_t dev;
        struct uio *uio;
 {
-       register int unit = minor(dev) >> 3;
+       register int unit = hpunit(dev);
 
        if (unit >= NHP)
                return (ENXIO);
 
        if (unit >= NHP)
                return (ENXIO);
@@ -672,7 +777,7 @@ hpwrite(dev, uio)
        dev_t dev;
        struct uio *uio;
 {
        dev_t dev;
        struct uio *uio;
 {
-       register int unit = minor(dev) >> 3;
+       register int unit = hpunit(dev);
 
        if (unit >= NHP)
                return (ENXIO);
 
        if (unit >= NHP)
                return (ENXIO);
@@ -690,7 +795,7 @@ hpioctl(dev, cmd, data, flag)
        switch (cmd) {
 
        case DKIOCHDR:  /* do header read/write */
        switch (cmd) {
 
        case DKIOCHDR:  /* do header read/write */
-               hpsoftc[minor(dev) >> 3].sc_hdr = 1;
+               hpsoftc[hpunit(dev)].sc_hdr = 1;
                return (0);
 
        default:
                return (0);
 
        default:
@@ -706,25 +811,25 @@ hpecc(mi, flag)
        register struct hpdevice *rp = (struct hpdevice *)mi->mi_drv;
        register struct buf *bp = mi->mi_tab.b_actf;
        register struct hpst *st = &hpst[mi->mi_type];
        register struct hpdevice *rp = (struct hpdevice *)mi->mi_drv;
        register struct buf *bp = mi->mi_tab.b_actf;
        register struct hpst *st = &hpst[mi->mi_type];
+       struct hpsoftc *sc = &hpsoftc[mi->mi_unit];
        int npf, o;
        int bn, cn, tn, sn;
        int bcr;
 
        int npf, o;
        int bn, cn, tn, sn;
        int bcr;
 
-       bcr = mbp->mba_bcr & 0xffff;
-       if (bcr)
-               bcr |= 0xffff0000;              /* sxt */
-       if (flag == CONT)
+       bcr = MASKREG(-mbp->mba_bcr);
+       if (bp->b_flags & B_BAD)
                npf = bp->b_error;
        else
                npf = bp->b_error;
        else
-               npf = btop(bcr + bp->b_bcount);
+               npf = btop(bp->b_bcount - bcr);
        o = (int)bp->b_un.b_addr & PGOFSET;
        o = (int)bp->b_un.b_addr & PGOFSET;
-       bn = dkblock(bp);
+       bn = bp->b_blkno;
        cn = bp->b_cylin;
        sn = bn%(st->nspc) + npf;
        tn = sn/st->nsect;
        sn %= st->nsect;
        cn += tn/st->ntrak;
        tn %= st->ntrak;
        cn = bp->b_cylin;
        sn = bn%(st->nspc) + npf;
        tn = sn/st->nsect;
        sn %= st->nsect;
        cn += tn/st->ntrak;
        tn %= st->ntrak;
+       bn += npf;
        switch (flag) {
        case ECC: {
                register int i;
        switch (flag) {
        case ECC: {
                register int i;
@@ -733,10 +838,13 @@ hpecc(mi, flag)
                int bit, byte, mask;
 
                npf--;          /* because block in error is previous block */
                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 */
+               bn--;
+               if (bp->b_flags & B_BAD)
+                       bn = sc->sc_badbn;
+               log(LOG_WARNING, "hp%d%c: soft ecc sn%d\n", hpunit(bp->b_dev),
+                   'a'+(minor(bp->b_dev)&07), bn);
+               mask = MASKREG(rp->hpec2);
+               i = MASKREG(rp->hpec1) - 1;             /* -1 makes 0 origin */
                bit = i&07;
                i = (i&~07)>>3;
                byte = i + o;
                bit = i&07;
                i = (i&~07)>>3;
                byte = i + o;
@@ -756,44 +864,54 @@ hpecc(mi, flag)
 
        case SSE:
                rp->hpof |= HPOF_SSEI;
 
        case SSE:
                rp->hpof |= HPOF_SSEI;
+               if (bp->b_flags & B_BAD) {
+                       bn = sc->sc_badbn;
+                       goto fixregs;
+               }
                mbp->mba_bcr = -(bp->b_bcount - (int)ptob(npf));
                break;
 
        case BSE:
                mbp->mba_bcr = -(bp->b_bcount - (int)ptob(npf));
                break;
 
        case BSE:
+               if (rp->hpof & HPOF_SSEI)
+                       sn++;
 #ifdef HPBDEBUG
                if (hpbdebug)
 #ifdef HPBDEBUG
                if (hpbdebug)
-               printf("hpecc, BSE: bn %d cn %d tn %d sn %d\n", bn, cn, tn, sn);
+               log(LOG_DEBUG, "hpecc, BSE: bn %d cn %d tn %d sn %d\n", bn, cn, tn, sn);
 #endif
 #endif
+               if (bp->b_flags & B_BAD)
+                       return (0);
                if ((bn = isbad(&hpbad[mi->mi_unit], cn, tn, sn)) < 0)
                if ((bn = isbad(&hpbad[mi->mi_unit], cn, tn, sn)) < 0)
-                       return(0);
+                       return (0);
                bp->b_flags |= B_BAD;
                bp->b_error = npf + 1;
                bp->b_flags |= B_BAD;
                bp->b_error = npf + 1;
+               rp->hpof &= ~HPOF_SSEI;
                bn = st->ncyl*st->nspc - st->nsect - 1 - bn;
                bn = st->ncyl*st->nspc - st->nsect - 1 - bn;
+               sc->sc_badbn = bn;
+       fixregs:
                cn = bn/st->nspc;
                sn = bn%st->nspc;
                tn = sn/st->nsect;
                sn %= st->nsect;
                cn = bn/st->nspc;
                sn = bn%st->nspc;
                tn = sn/st->nsect;
                sn %= st->nsect;
-               mbp->mba_bcr = -512;
+               mbp->mba_bcr = -(min(512, bp->b_bcount - (int)ptob(npf)));
 #ifdef HPBDEBUG
                if (hpbdebug)
 #ifdef HPBDEBUG
                if (hpbdebug)
-               printf("revector to cn %d tn %d sn %d\n", cn, tn, sn);
+               log(LOG_DEBUG, "revector to cn %d tn %d sn %d\n", cn, tn, sn);
 #endif
                break;
 
        case CONT:
 #ifdef HPBDEBUG
                if (hpbdebug)
 #endif
                break;
 
        case CONT:
 #ifdef HPBDEBUG
                if (hpbdebug)
-               printf("hpecc, CONT: bn %d cn %d tn %d sn %d\n", bn,cn,tn,sn);
+               log(LOG_DEBUG, "hpecc, CONT: bn %d cn %d tn %d sn %d\n", bn,cn,tn,sn);
 #endif
 #endif
-               npf = bp->b_error;
                bp->b_flags &= ~B_BAD;
                bp->b_flags &= ~B_BAD;
+               if ((int)ptob(npf) >= bp->b_bcount)
+                       return (0);
                mbp->mba_bcr = -(bp->b_bcount - (int)ptob(npf));
                mbp->mba_bcr = -(bp->b_bcount - (int)ptob(npf));
-               if ((mbp->mba_bcr & 0xffff) == 0)
-                       return(0);
                break;
        }
        rp->hpcs1 = HP_DCLR|HP_GO;
                break;
        }
        rp->hpcs1 = HP_DCLR|HP_GO;
-       if (rp->hpof&HPOF_SSEI)
+       if (rp->hpof & HPOF_SSEI)
                sn++;
        rp->hpdc = cn;
        rp->hpda = (tn<<8) + sn;
                sn++;
        rp->hpdc = cn;
        rp->hpda = (tn<<8) + sn;
@@ -801,6 +919,7 @@ hpecc(mi, flag)
        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 */
        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 */
+       sc->sc_blkdone = npf;
        return (1);
 }
 
        return (1);
 }
 
@@ -818,7 +937,7 @@ hpdump(dev)
 
        num = maxfree;
        start = 0;
 
        num = maxfree;
        start = 0;
-       unit = minor(dev) >> 3;
+       unit = hpunit(dev);
        if (unit >= NHP)
                return (ENXIO);
 #define        phys(a,b)       ((b)((int)(a)&0x7fffffff))
        if (unit >= NHP)
                return (ENXIO);
 #define        phys(a,b)       ((b)((int)(a)&0x7fffffff))
@@ -834,8 +953,10 @@ hpdump(dev)
                hpaddr->hpof = HPOF_FMT22;
        }
        st = &hpst[mi->mi_type];
                hpaddr->hpof = HPOF_FMT22;
        }
        st = &hpst[mi->mi_type];
-       if (dumplo < 0 || dumplo + num >= st->sizes[minor(dev)&07].nblocks)
+       if (dumplo < 0)
                return (EINVAL);
                return (EINVAL);
+       if (dumplo + num >= st->sizes[minor(dev)&07].nblocks)
+               num = st->sizes[minor(dev)&07].nblocks - dumplo;
        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;
@@ -857,7 +978,7 @@ hpdump(dev)
                mba->mba_var = 0;
                hpaddr->hpcs1 = HP_WCOM | HP_GO;
                while ((hpaddr->hpds & HPDS_DRY) == 0)
                mba->mba_var = 0;
                hpaddr->hpcs1 = HP_WCOM | HP_GO;
                while ((hpaddr->hpds & HPDS_DRY) == 0)
-                       ;
+                       DELAY(10);
                if (hpaddr->hpds&HPDS_ERR)
                        return (EIO);
                start += blk*NBPG;
                if (hpaddr->hpds&HPDS_ERR)
                        return (EIO);
                start += blk*NBPG;
@@ -865,4 +986,17 @@ hpdump(dev)
        }
        return (0);
 }
        }
        return (0);
 }
+
+hpsize(dev)
+       dev_t dev;
+{
+       int unit = hpunit(dev);
+       struct mba_device *mi;
+       struct hpst *st;
+
+       if (unit >= NHP || (mi = hpinfo[unit]) == 0 || mi->mi_alive == 0)
+               return (-1);
+       st = &hpst[mi->mi_type];
+       return ((int)st->sizes[minor(dev) & 07].nblocks);
+}
 #endif
 #endif