]> git.sur5r.net Git - bacula/bacula/blobdiff - bacula/src/stored/btape.c
- Add wait.c (oops forgot previously).
[bacula/bacula] / bacula / src / stored / btape.c
index 62f19c87de5124ec5275593054aea7b55d681aba..b1e257489b8a8f25a15cc2a0faaead389ff83cb9 100644 (file)
@@ -14,7 +14,7 @@
  *
  */
 /*
-   Copyright (C) 2000-2004 Kern Sibbald and John Walker
+   Copyright (C) 2000-2005 Kern Sibbald
 
    This program is free software; you can redistribute it and/or
    modify it under the terms of the GNU General Public License as
@@ -45,17 +45,22 @@ int quit = 0;
 char buf[100000];
 int bsize = TAPE_BSIZE;
 char VolName[MAX_NAME_LENGTH];
+STORES *me = NULL;                   /* our Global resource */
+bool forge_on = false;               /* proceed inspite of I/O errors */
+pthread_mutex_t device_release_mutex = PTHREAD_MUTEX_INITIALIZER;
+pthread_cond_t wait_device_release = PTHREAD_COND_INITIALIZER;
 
 /*
- * If you change the format of the state file, 
+ * If you change the format of the state file,
  *  increment this value
- */ 
+ */
 static uint32_t btape_state_level = 2;
 
 DEVICE *dev = NULL;
+DCR *dcr;
 DEVRES *device = NULL;
 
-           
+
 /* Forward referenced subroutines */
 static void do_tape_cmds();
 static void helpcmd();
@@ -70,18 +75,14 @@ static void qfillcmd();
 static void statcmd();
 static void unfillcmd();
 static int flush_block(DEV_BLOCK *block, int dump);
-#ifdef xxx_needed
-static int record_cb(JCR *jcr, DEVICE *dev, DEV_BLOCK *block, DEV_RECORD *rec);
-#endif
-static int quickie_cb(JCR *jcr, DEVICE *dev, DEV_BLOCK *block, DEV_RECORD *rec);
+static bool quickie_cb(DCR *dcr, DEV_RECORD *rec);
 static bool compare_blocks(DEV_BLOCK *last_block, DEV_BLOCK *block);
-static int my_mount_next_read_volume(JCR *jcr, DEVICE *dev, DEV_BLOCK *block);
+static bool my_mount_next_read_volume(DCR *dcr);
 static void scan_blocks();
-static void set_volume_name(char *VolName, int volnum);
+static void set_volume_name(const char *VolName, int volnum);
 static void rawfill_cmd();
 static void bfill_cmd();
 static bool open_the_device();
-static char *edit_device_codes(JCR *jcr, char *omsg, char *imsg, char *cmd);
 static void autochangercmd();
 static void do_unfill();
 
@@ -125,9 +126,9 @@ static uint32_t last_block_num1 = 0;
 static uint32_t last_block_num2 = 0;
 static uint32_t last_block_num = 0;
 static uint32_t BlockNumber = 0;
-static bool simple = true; 
+static bool simple = true;
 
-static char *VolumeName = NULL;
+static const char *VolumeName = NULL;
 static int vol_num = 0;
 
 static JCR *jcr = NULL;
@@ -135,7 +136,7 @@ static JCR *jcr = NULL;
 
 static void usage();
 static void terminate_btape(int sig);
-int get_cmd(char *prompt);
+int get_cmd(const char *prompt);
 
 
 /*********************************************************************
@@ -145,16 +146,39 @@ int get_cmd(char *prompt);
  */
 int main(int margc, char *margv[])
 {
-   int ch;
+   int ch, i;
+   uint32_t x32, y32;
+   uint64_t x64, y64;
+   char buf[1000];
 
    /* Sanity checks */
-   if (TAPE_BSIZE % DEV_BSIZE != 0 || TAPE_BSIZE / DEV_BSIZE == 0) {
+   if (TAPE_BSIZE % B_DEV_BSIZE != 0 || TAPE_BSIZE / B_DEV_BSIZE == 0) {
       Emsg2(M_ABORT, 0, "Tape block size (%d) not multiple of system size (%d)\n",
-        TAPE_BSIZE, DEV_BSIZE);
+        TAPE_BSIZE, B_DEV_BSIZE);
    }
    if (TAPE_BSIZE != (1 << (ffs(TAPE_BSIZE)-1))) {
       Emsg1(M_ABORT, 0, "Tape block size (%d) is not a power of 2\n", TAPE_BSIZE);
    }
+   if (sizeof(off_t) < 8) {
+      Pmsg1(-1, "\n\n!!!! Warning large disk addressing disabled. off_t=%d should be 8 or more !!!!!\n\n\n",
+        sizeof(off_t));
+   }
+   x32 = 123456789;
+   bsnprintf(buf, sizeof(buf), "%u", x32);
+   i = bsscanf(buf, "%u", &y32);
+   if (i != 1 || x32 != y32) {
+      Pmsg3(-1, "32 bit printf/scanf problem. i=%d x32=%u y32=%u\n", i, x32, y32);
+      exit(1);
+   }
+   x64 = 123456789;
+   x64 = x64 << 32;
+   x64 += 123456789;
+   bsnprintf(buf, sizeof(buf), "%" llu, x64);
+   i = bsscanf(buf, "%llu", &y64);
+   if (i != 1 || x64 != y64) {
+      Pmsg3(-1, "64 bit printf/scanf problem. i=%d x64=%" llu " y64=%" llu "\n", i, x64, y64);
+      exit(1);
+   }
 
    printf("Tape block granularity is %d bytes.\n", TAPE_BSIZE);
 
@@ -162,7 +186,7 @@ int main(int margc, char *margv[])
    my_name_is(margc, margv, "btape");
    init_msg(NULL, NULL);
 
-   while ((ch = getopt(margc, margv, "b:c:d:sv?")) != -1) {
+   while ((ch = getopt(margc, margv, "b:c:d:psv?")) != -1) {
       switch (ch) {
       case 'b':                    /* bootstrap file */
         bsr = parse_bsr(NULL, optarg);
@@ -179,12 +203,16 @@ int main(int margc, char *margv[])
       case 'd':                    /* set debug level */
         debug_level = atoi(optarg);
         if (debug_level <= 0) {
-           debug_level = 1; 
+           debug_level = 1;
         }
         break;
 
+      case 'p':
+        forge_on = true;
+        break;
+
       case 's':
-        signals = FALSE;
+        signals = false;
         break;
 
       case 'v':
@@ -196,14 +224,14 @@ int main(int margc, char *margv[])
         helpcmd();
         exit(0);
 
-      }  
+      }
    }
    margc -= optind;
    margv += optind;
 
    cmd = get_pool_memory(PM_FNAME);
    args = get_pool_memory(PM_FNAME);
-   
+
    if (signals) {
       init_signals(terminate_btape);
    }
@@ -228,18 +256,22 @@ int main(int margc, char *margv[])
       exit(1);
    }
 
-   jcr = setup_jcr("btape", margv[0], bsr, NULL);
-   dev = setup_to_access_device(jcr, 0);     /* acquire for write */
+   jcr = setup_jcr("btape", margv[0], bsr, NULL, 0); /* write */
+   if (!jcr) {
+      exit(1);
+   }
+   dev = jcr->dcr->dev;
    if (!dev) {
       exit(1);
    }
+   dcr = jcr->dcr;
    if (!open_the_device()) {
       goto terminate;
    }
 
    Dmsg0(200, "Do tape commands\n");
    do_tape_cmds();
-  
+
 terminate:
    terminate_btape(0);
    return 0;
@@ -248,7 +280,7 @@ terminate:
 static void terminate_btape(int stat)
 {
 
-   sm_check(__FILE__, __LINE__, False);
+   sm_check(__FILE__, __LINE__, false);
    if (configfile) {
       free(configfile);
    }
@@ -262,47 +294,49 @@ static void terminate_btape(int stat)
       cmd = NULL;
    }
 
-   if (dev) {
-      term_dev(dev);
+   if (bsr) {
+      free_bsr(bsr);
    }
 
-   if (debug_level > 10)
-      print_memory_pool_stats(); 
-
    free_jcr(jcr);
    jcr = NULL;
 
-   if (bsr) {
-      free_bsr(bsr);
+   if (dev) {
+      term_dev(dev);
    }
 
+   if (debug_level > 10)
+      print_memory_pool_stats();
+
    if (this_block) {
       free_block(this_block);
    }
 
+   stop_watchdog();
    term_msg();
    close_memory_pool();              /* free memory in pool */
 
-   sm_dump(False);
+   sm_dump(false);
    exit(stat);
 }
 
 static bool open_the_device()
 {
    DEV_BLOCK *block;
-   
+
    block = new_block(dev);
    lock_device(dev);
-   if (!(dev->state & ST_OPENED)) {
-      Dmsg1(200, "Opening device %s\n", jcr->VolumeName);
-      if (open_dev(dev, jcr->VolumeName, READ_WRITE) < 0) {
+   if (!dev->is_open()) {
+      Dmsg1(200, "Opening device %s\n", dcr->VolumeName);
+      if (open_dev(dev, dcr->VolumeName, OPEN_READ_WRITE) < 0) {
          Emsg1(M_FATAL, 0, _("dev open failed: %s\n"), dev->errmsg);
         unlock_device(dev);
         free_block(block);
         return false;
       }
    }
-   Dmsg1(000, "open_dev %s OK\n", dev_name(dev));
+   Pmsg1(000, "open_dev %s OK\n", dev->print_name());
+   dev->set_append();                /* put volume in append mode */
    unlock_device(dev);
    free_block(block);
    return true;
@@ -315,70 +349,67 @@ void quitcmd()
 }
 
 /*
- * Write a label to the tape   
+ * Write a label to the tape
  */
 static void labelcmd()
 {
    if (VolumeName) {
-      pm_strcpy(&cmd, VolumeName);
+      pm_strcpy(cmd, VolumeName);
    } else {
       if (!get_cmd("Enter Volume Name: ")) {
         return;
       }
    }
-        
-   if (!(dev->state & ST_OPENED)) {
+
+   if (!dev->is_open()) {
       if (!first_open_device(dev)) {
          Pmsg1(0, "Device open failed. ERR=%s\n", strerror_dev(dev));
       }
    }
    rewind_dev(dev);
-   write_volume_label_to_dev(jcr, jcr->device, cmd, "Default");
+   write_new_volume_label_to_dev(dcr, cmd, "Default");
    Pmsg1(-1, "Wrote Volume label for volume \"%s\".\n", cmd);
 }
 
 /*
- * Read the tape label  
+ * Read the tape label
  */
 static void readlabelcmd()
 {
    int save_debug_level = debug_level;
    int stat;
-   DEV_BLOCK *block;
 
-   block = new_block(dev);
-   stat = read_dev_volume_label(jcr, dev, block);
+   stat = read_dev_volume_label(dcr);
    switch (stat) {
-      case VOL_NO_LABEL:
-         Pmsg0(0, "Volume has no label.\n");
-        break;
-      case VOL_OK:
-         Pmsg0(0, "Volume label read correctly.\n");
-        break;
-      case VOL_IO_ERROR:
-         Pmsg1(0, "I/O error on device: ERR=%s", strerror_dev(dev));
-        break;
-      case VOL_NAME_ERROR:
-         Pmsg0(0, "Volume name error\n");
-        break;
-      case VOL_CREATE_ERROR:
-         Pmsg1(0, "Error creating label. ERR=%s", strerror_dev(dev));
-        break;
-      case VOL_VERSION_ERROR:
-         Pmsg0(0, "Volume version error.\n");
-        break;
-      case VOL_LABEL_ERROR:
-         Pmsg0(0, "Bad Volume label type.\n");
-        break;
-      default:
-         Pmsg0(0, "Unknown error.\n");
-        break;
+   case VOL_NO_LABEL:
+      Pmsg0(0, "Volume has no label.\n");
+      break;
+   case VOL_OK:
+      Pmsg0(0, "Volume label read correctly.\n");
+      break;
+   case VOL_IO_ERROR:
+      Pmsg1(0, "I/O error on device: ERR=%s", strerror_dev(dev));
+      break;
+   case VOL_NAME_ERROR:
+      Pmsg0(0, "Volume name error\n");
+      break;
+   case VOL_CREATE_ERROR:
+      Pmsg1(0, "Error creating label. ERR=%s", strerror_dev(dev));
+      break;
+   case VOL_VERSION_ERROR:
+      Pmsg0(0, "Volume version error.\n");
+      break;
+   case VOL_LABEL_ERROR:
+      Pmsg0(0, "Bad Volume label type.\n");
+      break;
+   default:
+      Pmsg0(0, "Unknown error.\n");
+      break;
    }
 
    debug_level = 20;
-   dump_volume_label(dev); 
+   dump_volume_label(dev);
    debug_level = save_debug_level;
-   free_block(block);
 }
 
 
@@ -392,11 +423,11 @@ static void loadcmd()
    if (!load_dev(dev)) {
       Pmsg1(0, "Bad status from load. ERR=%s\n", strerror_dev(dev));
    } else
-      Pmsg1(0, "Loaded %s\n", dev_name(dev));
+      Pmsg1(0, "Loaded %s\n", dev->print_name());
 }
 
 /*
- * Rewind the tape.   
+ * Rewind the tape.
  */
 static void rewindcmd()
 {
@@ -404,12 +435,12 @@ static void rewindcmd()
       Pmsg1(0, "Bad status from rewind. ERR=%s\n", strerror_dev(dev));
       clrerror_dev(dev, -1);
    } else {
-      Pmsg1(0, "Rewound %s\n", dev_name(dev));
+      Pmsg1(0, "Rewound %s\n", dev->print_name());
    }
 }
 
 /*
- * Clear any tape error   
+ * Clear any tape error
  */
 static void clearcmd()
 {
@@ -417,7 +448,7 @@ static void clearcmd()
 }
 
 /*
- * Write and end of file on the tape   
+ * Write and end of file on the tape
  */
 static void weofcmd()
 {
@@ -434,12 +465,12 @@ static void weofcmd()
       Pmsg2(0, "Bad status from weof %d. ERR=%s\n", stat, strerror_dev(dev));
       return;
    } else {
-      Pmsg3(0, "Wrote %d EOF%s to %s\n", num, num==1?"":"s", dev_name(dev));
+      Pmsg3(0, "Wrote %d EOF%s to %s\n", num, num==1?"":"s", dev->print_name());
    }
 }
 
 
-/* Go to the end of the medium -- raw command  
+/* Go to the end of the medium -- raw command
  * The idea was orginally that the end of the Bacula
  * medium would be flagged differently. This is not
  * currently the case. So, this is identical to the
@@ -465,7 +496,7 @@ static void eodcmd()
 }
 
 /*
- * Backspace file   
+ * Backspace file
  */
 static void bsfcmd()
 {
@@ -485,7 +516,7 @@ static void bsfcmd()
 }
 
 /*
- * Backspace record   
+ * Backspace record
  */
 static void bsrcmd()
 {
@@ -504,7 +535,7 @@ static void bsrcmd()
 }
 
 /*
- * List device capabilities as defined in the 
+ * List device capabilities as defined in the
  *  stored.conf file.
  */
 static void capcmd()
@@ -516,6 +547,7 @@ static void capcmd()
    printf("%sFSR ", dev->capabilities & CAP_FSR ? "" : "!");
    printf("%sFSF ", dev->capabilities & CAP_FSF ? "" : "!");
    printf("%sFASTFSF ", dev->capabilities & CAP_FASTFSF ? "" : "!");
+   printf("%sBSFATEOM ", dev->capabilities & CAP_BSFATEOM ? "" : "!");
    printf("%sEOM ", dev->capabilities & CAP_EOM ? "" : "!");
    printf("%sREM ", dev->capabilities & CAP_REM ? "" : "!");
    printf("%sRACCESS ", dev->capabilities & CAP_RACCESS ? "" : "!");
@@ -523,18 +555,19 @@ static void capcmd()
    printf("%sLABEL ", dev->capabilities & CAP_LABEL ? "" : "!");
    printf("%sANONVOLS ", dev->capabilities & CAP_ANONVOLS ? "" : "!");
    printf("%sALWAYSOPEN ", dev->capabilities & CAP_ALWAYSOPEN ? "" : "!");
+   printf("%sMTIOCGET ", dev->capabilities & CAP_MTIOCGET ? "" : "!");
    printf("\n");
 
    printf(_("Device status:\n"));
-   printf("%sOPENED ", dev->state & ST_OPENED ? "" : "!");
-   printf("%sTAPE ", dev->state & ST_TAPE ? "" : "!");
-   printf("%sLABEL ", dev->state & ST_LABEL ? "" : "!");
+   printf("%sOPENED ", dev->is_open() ? "" : "!");
+   printf("%sTAPE ", dev->is_tape() ? "" : "!");
+   printf("%sLABEL ", dev->is_labeled() ? "" : "!");
    printf("%sMALLOC ", dev->state & ST_MALLOC ? "" : "!");
-   printf("%sAPPEND ", dev->state & ST_APPEND ? "" : "!");
-   printf("%sREAD ", dev->state & ST_READ ? "" : "!");
-   printf("%sEOT ", dev->state & ST_EOT ? "" : "!");
+   printf("%sAPPEND ", dev->can_append() ? "" : "!");
+   printf("%sREAD ", dev->can_read() ? "" : "!");
+   printf("%sEOT ", dev->at_eot() ? "" : "!");
    printf("%sWEOT ", dev->state & ST_WEOT ? "" : "!");
-   printf("%sEOF ", dev->state & ST_EOF ? "" : "!");
+   printf("%sEOF ", dev->at_eof() ? "" : "!");
    printf("%sNEXTVOL ", dev->state & ST_NEXTVOL ? "" : "!");
    printf("%sSHORT ", dev->state & ST_SHORT ? "" : "!");
    printf("\n");
@@ -550,7 +583,7 @@ static void capcmd()
 }
 
 /*
- * Test writting larger and larger records.  
+ * Test writting larger and larger records.
  * This is a torture test for records.
  */
 static void rectestcmd()
@@ -571,7 +604,7 @@ static void rectestcmd()
       return;
    }
 
-   sm_check(__FILE__, __LINE__, False);
+   sm_check(__FILE__, __LINE__, false);
    block = new_block(dev);
    rec = new_record();
 
@@ -579,7 +612,7 @@ static void rectestcmd()
       rec->data = check_pool_memory_size(rec->data, i);
       memset(rec->data, i & 0xFF, i);
       rec->data_len = i;
-      sm_check(__FILE__, __LINE__, False);
+      sm_check(__FILE__, __LINE__, false);
       if (write_record_to_block(block, rec)) {
         empty_block(block);
         blkno++;
@@ -587,11 +620,11 @@ static void rectestcmd()
       } else {
         break;
       }
-      sm_check(__FILE__, __LINE__, False);
+      sm_check(__FILE__, __LINE__, false);
    }
    free_record(rec);
    free_block(block);
-   sm_check(__FILE__, __LINE__, False);
+   sm_check(__FILE__, __LINE__, false);
 }
 
 /*
@@ -603,7 +636,7 @@ static void rectestcmd()
  */
 static int re_read_block_test()
 {
-   DEV_BLOCK *block;
+   DEV_BLOCK *block = dcr->block;
    DEV_RECORD *rec;
    int stat = 0;
    int len;
@@ -615,44 +648,44 @@ static int re_read_block_test()
 
    Pmsg0(-1, _("\n=== Write, backup, and re-read test ===\n\n"
       "I'm going to write three records and an EOF\n"
-      "then backup over the EOF and re-read the last record.\n"     
+      "then backup over the EOF and re-read the last record.\n"
       "Bacula does this after writing the last block on the\n"
       "tape to verify that the block was written correctly.\n\n"
-      "This is not an *essential* feature ...\n\n")); 
+      "This is not an *essential* feature ...\n\n"));
    rewindcmd();
-   block = new_block(dev);
+   empty_block(block);
    rec = new_record();
    rec->data = check_pool_memory_size(rec->data, block->buf_len);
    len = rec->data_len = block->buf_len-100;
    memset(rec->data, 1, rec->data_len);
    if (!write_record_to_block(block, rec)) {
-      Pmsg0(0, _("Error writing record to block.\n")); 
+      Pmsg0(0, _("Error writing record to block.\n"));
       goto bail_out;
    }
-   if (!write_block_to_dev(jcr, dev, block)) {
-      Pmsg0(0, _("Error writing block to device.\n")); 
+   if (!write_block_to_dev(dcr)) {
+      Pmsg0(0, _("Error writing block to device.\n"));
       goto bail_out;
    } else {
       Pmsg1(0, _("Wrote first record of %d bytes.\n"), rec->data_len);
    }
    memset(rec->data, 2, rec->data_len);
    if (!write_record_to_block(block, rec)) {
-      Pmsg0(0, _("Error writing record to block.\n")); 
+      Pmsg0(0, _("Error writing record to block.\n"));
       goto bail_out;
    }
-   if (!write_block_to_dev(jcr, dev, block)) {
-      Pmsg0(0, _("Error writing block to device.\n")); 
+   if (!write_block_to_dev(dcr)) {
+      Pmsg0(0, _("Error writing block to device.\n"));
       goto bail_out;
    } else {
       Pmsg1(0, _("Wrote second record of %d bytes.\n"), rec->data_len);
    }
    memset(rec->data, 3, rec->data_len);
    if (!write_record_to_block(block, rec)) {
-      Pmsg0(0, _("Error writing record to block.\n")); 
+      Pmsg0(0, _("Error writing record to block.\n"));
       goto bail_out;
    }
-   if (!write_block_to_dev(jcr, dev, block)) {
-      Pmsg0(0, _("Error writing block to device.\n")); 
+   if (!write_block_to_dev(dcr)) {
+      Pmsg0(0, _("Error writing block to device.\n"));
       goto bail_out;
    } else {
       Pmsg1(0, _("Wrote third record of %d bytes.\n"), rec->data_len);
@@ -677,13 +710,15 @@ static int re_read_block_test()
       goto bail_out;
    }
    Pmsg0(0, "Backspace record OK.\n");
-   if (!read_block_from_dev(jcr, dev, block, NO_BLOCK_NUMBER_CHECK)) {
-      Pmsg1(0, _("Read block failed! ERR=%s\n"), strerror(dev->dev_errno));
+   if (!read_block_from_dev(dcr, NO_BLOCK_NUMBER_CHECK)) {
+      berrno be;
+      Pmsg1(0, _("Read block failed! ERR=%s\n"), be.strerror(dev->dev_errno));
       goto bail_out;
    }
    memset(rec->data, 0, rec->data_len);
    if (!read_record_from_block(block, rec)) {
-      Pmsg1(0, _("Read block failed! ERR=%s\n"), strerror(dev->dev_errno));
+      berrno be;
+      Pmsg1(0, _("Read block failed! ERR=%s\n"), be.strerror(dev->dev_errno));
       goto bail_out;
    }
    for (int i=0; i<len; i++) {
@@ -698,7 +733,6 @@ static int re_read_block_test()
    stat = 1;
 
 bail_out:
-   free_block(block);
    free_record(rec);
    if (stat == 0) {
       Pmsg0(0, _("This is not terribly serious since Bacula only uses\n"
@@ -707,7 +741,7 @@ bail_out:
                  "if you add:\n\n"
                   "Backward Space Record = No\n\n"
                   "to your Storage daemon's Device resource definition.\n"));
-   }   
+   }
    return stat;
 }
 
@@ -727,10 +761,10 @@ static int write_read_test()
 
    Pmsg0(-1, _("\n=== Write, rewind, and re-read test ===\n\n"
       "I'm going to write 1000 records and an EOF\n"
-      "then write 1000 records and an EOF, then rewind,\n"     
+      "then write 1000 records and an EOF, then rewind,\n"
       "and re-read the data to verify that it is correct.\n\n"
-      "This is an *essential* feature ...\n\n")); 
-   block = new_block(dev);
+      "This is an *essential* feature ...\n\n"));
+   block = dcr->block;
    rec = new_record();
    if (!rewind_dev(dev)) {
       Pmsg1(0, "Bad status from rewind. ERR=%s\n", strerror_dev(dev));
@@ -745,11 +779,11 @@ static int write_read_test()
         *p++ = i;
       }
       if (!write_record_to_block(block, rec)) {
-         Pmsg0(0, _("Error writing record to block.\n")); 
+         Pmsg0(0, _("Error writing record to block.\n"));
         goto bail_out;
       }
-      if (!write_block_to_dev(jcr, dev, block)) {
-         Pmsg0(0, _("Error writing block to device.\n")); 
+      if (!write_block_to_dev(dcr)) {
+         Pmsg0(0, _("Error writing block to device.\n"));
         goto bail_out;
       }
    }
@@ -761,11 +795,11 @@ static int write_read_test()
         *p++ = i;
       }
       if (!write_record_to_block(block, rec)) {
-         Pmsg0(0, _("Error writing record to block.\n")); 
+         Pmsg0(0, _("Error writing record to block.\n"));
         goto bail_out;
       }
-      if (!write_block_to_dev(jcr, dev, block)) {
-         Pmsg0(0, _("Error writing block to device.\n")); 
+      if (!write_block_to_dev(dcr)) {
+         Pmsg0(0, _("Error writing block to device.\n"));
         goto bail_out;
       }
    }
@@ -782,17 +816,21 @@ static int write_read_test()
    }
    for (i=1; i<=2000; i++) {
 read_again:
-      if (!read_block_from_dev(jcr, dev, block, NO_BLOCK_NUMBER_CHECK)) {
+      if (!read_block_from_dev(dcr, NO_BLOCK_NUMBER_CHECK)) {
+        berrno be;
         if (dev_state(dev, ST_EOF)) {
             Pmsg0(-1, _("Got EOF on tape.\n"));
-           goto read_again;
+           if (i == 1001) {
+              goto read_again;
+           }
         }
-         Pmsg1(0, _("Read block failed! ERR=%s\n"), strerror(dev->dev_errno));
+         Pmsg2(0, _("Read block %d failed! ERR=%s\n"), i, be.strerror(dev->dev_errno));
         goto bail_out;
       }
       memset(rec->data, 0, rec->data_len);
       if (!read_record_from_block(block, rec)) {
-         Pmsg1(0, _("Read record failed! ERR=%s\n"), strerror(dev->dev_errno));
+        berrno be;
+         Pmsg2(0, _("Read record failed. Block %d! ERR=%s\n"), i, be.strerror(dev->dev_errno));
         goto bail_out;
       }
       p = (int *)rec->data;
@@ -812,7 +850,6 @@ read_again:
    stat = 1;
 
 bail_out:
-   free_block(block);
    free_record(rec);
    return stat;
 }
@@ -824,7 +861,7 @@ bail_out:
  */
 static int position_test()
 {
-   DEV_BLOCK *block;
+   DEV_BLOCK *block = dcr->block;
    DEV_RECORD *rec;
    int stat = 0;
    int len, i, j;
@@ -832,13 +869,14 @@ static int position_test()
    int recno = 0;
    int file = 0, blk = 0;
    int *p;
+   bool got_eof = false;
 
    Pmsg0(-1, _("\n=== Write, rewind, and position test ===\n\n"
       "I'm going to write 1000 records and an EOF\n"
-      "then write 1000 records and an EOF, then rewind,\n"     
+      "then write 1000 records and an EOF, then rewind,\n"
       "and position to a few blocks and verify that it is correct.\n\n"
-      "This is an *essential* feature ...\n\n")); 
-   block = new_block(dev);
+      "This is an *essential* feature ...\n\n"));
+   empty_block(block);
    rec = new_record();
    if (!rewind_dev(dev)) {
       Pmsg1(0, "Bad status from rewind. ERR=%s\n", strerror_dev(dev));
@@ -853,11 +891,11 @@ static int position_test()
         *p++ = i;
       }
       if (!write_record_to_block(block, rec)) {
-         Pmsg0(0, _("Error writing record to block.\n")); 
+         Pmsg0(0, _("Error writing record to block.\n"));
         goto bail_out;
       }
-      if (!write_block_to_dev(jcr, dev, block)) {
-         Pmsg0(0, _("Error writing block to device.\n")); 
+      if (!write_block_to_dev(dcr)) {
+         Pmsg0(0, _("Error writing block to device.\n"));
         goto bail_out;
       }
    }
@@ -869,11 +907,11 @@ static int position_test()
         *p++ = i;
       }
       if (!write_record_to_block(block, rec)) {
-         Pmsg0(0, _("Error writing record to block.\n")); 
+         Pmsg0(0, _("Error writing record to block.\n"));
         goto bail_out;
       }
-      if (!write_block_to_dev(jcr, dev, block)) {
-         Pmsg0(0, _("Error writing block to device.\n")); 
+      if (!write_block_to_dev(dcr)) {
+         Pmsg0(0, _("Error writing block to device.\n"));
         goto bail_out;
       }
    }
@@ -932,17 +970,34 @@ static int position_test()
         goto bail_out;
       }
 read_again:
-      if (!read_block_from_dev(jcr, dev, block, NO_BLOCK_NUMBER_CHECK)) {
+      if (!read_block_from_dev(dcr, NO_BLOCK_NUMBER_CHECK)) {
+        berrno be;
         if (dev_state(dev, ST_EOF)) {
             Pmsg0(-1, _("Got EOF on tape.\n"));
-           goto read_again;
+           if (!got_eof) {
+              got_eof = true;
+              goto read_again;
+           }
         }
-         Pmsg1(0, _("Read block failed! ERR=%s\n"), strerror(dev->dev_errno));
+         Pmsg4(0, _("Read block %d failed! file=%d blk=%d. ERR=%s\n\n"),
+           recno, file, blk, be.strerror(dev->dev_errno));
+         Pmsg0(0, _("This may be because the tape drive block size is not\n"
+                    " set to variable blocking as normally used by Bacula.\n"
+                    " Please see the Tape Testing chapter in the manual and \n"
+                    " look for using mt with defblksize and setoptions\n"
+                    "If your tape drive block size is correct, then perhaps\n"
+                    " your SCSI driver is *really* stupid and does not\n"
+                    " correctly report the file:block after a FSF. In this\n"
+                    " case try setting:\n"
+                    "    Fast Forward Space File = no\n"
+                    " in your Device resource.\n"));
+
         goto bail_out;
       }
       memset(rec->data, 0, rec->data_len);
       if (!read_record_from_block(block, rec)) {
-         Pmsg1(0, _("Read record failed! ERR=%s\n"), strerror(dev->dev_errno));
+        berrno be;
+         Pmsg1(0, _("Read record failed! ERR=%s\n"), be.strerror(dev->dev_errno));
         goto bail_out;
       }
       p = (int *)rec->data;
@@ -953,13 +1008,12 @@ read_again:
            goto bail_out;
         }
       }
-      Pmsg0(-1, _("Block re-read correctly.\n"));
+      Pmsg1(-1, _("Block %d re-read correctly.\n"), recno);
    }
    Pmsg0(-1, _("=== Test Succeeded. End Write, rewind, and re-read test ===\n\n"));
    stat = 1;
 
 bail_out:
-   free_block(block);
    free_record(rec);
    return stat;
 }
@@ -994,10 +1048,14 @@ static int append_test()
    if (dev_cap(dev, CAP_TWOEOF)) {
       weofcmd();
    }
+   force_close_dev(dev);             /* release device */
+   if (!open_the_device()) {
+      return -1;
+   }
    rewindcmd();
    Pmsg0(0, _("Now moving to end of medium.\n"));
    eodcmd();
-   Pmsg2(-1, _("We should be in file 3. I am at file %d. This is %s\n"), 
+   Pmsg2(-1, _("We should be in file 3. I am at file %d. This is %s\n"),
       dev->file, dev->file == 3 ? "correct!" : "NOT correct!!!!");
 
    if (dev->file != 3) {
@@ -1005,7 +1063,7 @@ static int append_test()
    }
 
    Pmsg0(-1, _("\nNow the important part, I am going to attempt to append to the tape.\n\n"));
-   wrcmd(); 
+   wrcmd();
    weofcmd();
    if (dev_cap(dev, CAP_TWOEOF)) {
       weofcmd();
@@ -1015,7 +1073,7 @@ static int append_test()
    Pmsg0(-1, "Doing Bacula scan of blocks:\n");
    scan_blocks();
    Pmsg0(-1, _("End scanning the tape.\n"));
-   Pmsg2(-1, _("We should be in file 4. I am at file %d. This is %s\n"), 
+   Pmsg2(-1, _("We should be in file 4. I am at file %d. This is %s\n"),
       dev->file, dev->file == 4 ? "correct!" : "NOT correct!!!!");
 
    if (dev->file != 4) {
@@ -1032,19 +1090,21 @@ static int autochanger_test()
 {
    POOLMEM *results, *changer;
    int slot, status, loaded;
-   int timeout = 120;
+   int timeout = dcr->device->max_changer_wait;
    int sleep_time = 0;
 
+   Dmsg1(100, "Max changer wait = %d sec\n", timeout);
    if (!dev_cap(dev, CAP_AUTOCHANGER)) {
       return 1;
    }
-   if (!(jcr->device && jcr->device->changer_name && jcr->device->changer_command)) {
+   if (!(dcr->device && dcr->device->changer_name && dcr->device->changer_command)) {
       Pmsg0(-1, "\nAutochanger enabled, but no name or no command device specified.\n");
       return 1;
    }
 
-   Pmsg0(-1, "\nTo test the autochanger you must have a blank tape in Slot 1.\n"
-             "I'm going to write on it.\n");
+   Pmsg0(-1, "\nAh, I see you have an autochanger configured.\n"
+             "To test the autochanger you must have a blank tape\n"
+             " that I can write on in Slot 1.\n");
    if (!get_cmd("\nDo you wish to continue with the Autochanger test? (y/n): ")) {
       return 0;
    }
@@ -1059,72 +1119,90 @@ static int autochanger_test()
 
 try_again:
    slot = 1;
-   jcr->VolCatInfo.Slot = slot;
+   dcr->VolCatInfo.Slot = slot;
    /* Find out what is loaded, zero means device is unloaded */
    Pmsg0(-1, _("3301 Issuing autochanger \"loaded\" command.\n"));
-   changer = edit_device_codes(jcr, changer, jcr->device->changer_command, 
-                "loaded");
+   changer = edit_device_codes(dcr, changer, "loaded");
    status = run_program(changer, timeout, results);
-   Dmsg3(100, "run_prog: %s stat=%d result=%s\n", changer, status, results);
+   Dmsg3(100, "run_prog: %s stat=%d result=\"%s\"\n", changer, status, results);
    if (status == 0) {
       loaded = atoi(results);
    } else {
-      Pmsg1(-1, _("3991 Bad autochanger \"load slot\" status=%d.\n"), status);
-      loaded = -1;             /* force unload */
+      berrno be;
+      Pmsg1(-1, _("3991 Bad autochanger command: %s\n"), changer);
+      Pmsg2(-1, _("3991 result=\"%s\": ERR=%s\n"), results, be.strerror(status));
       goto bail_out;
    }
    if (loaded) {
       Pmsg1(-1, "Slot %d loaded. I am going to unload it.\n", loaded);
    } else {
-      Pmsg0(-1, "Nothing loaded into the drive. OK.\n");
+      Pmsg0(-1, "Nothing loaded in the drive. OK.\n");
    }
    Dmsg1(100, "Results from loaded query=%s\n", results);
    if (loaded) {
+      dcr->VolCatInfo.Slot = loaded;
       offline_or_rewind_dev(dev);
       /* We are going to load a new tape, so close the device */
       force_close_dev(dev);
-      Pmsg0(-1, _("3302 Issuing autochanger \"unload\" command.\n"));
-      changer = edit_device_codes(jcr, changer, 
-                     jcr->device->changer_command, "unload");
-      status = run_program(changer, timeout, NULL);
+      Pmsg2(-1, _("3302 Issuing autochanger \"unload %d %d\" command.\n"),
+        loaded, dev->drive_index);
+      changer = edit_device_codes(dcr, changer, "unload");
+      status = run_program(changer, timeout, results);
       Pmsg2(-1, "unload status=%s %d\n", status==0?"OK":"Bad", status);
+      if (status != 0) {
+        berrno be;
+         Pmsg1(-1, _("3992 Bad autochanger command: %s\n"), changer);
+         Pmsg2(-1, _("3992 result=\"%s\": ERR=%s\n"), results, be.strerror(status));
+      }
    }
 
    /*
     * Load the Slot 1
     */
-   Pmsg1(-1, _("3303 Issuing autochanger \"load slot %d\" command.\n"), slot);
-   changer = edit_device_codes(jcr, changer, jcr->device->changer_command, "load");
-   Dmsg1(200, "Changer=%s\n", changer);
-   status = run_program(changer, timeout, NULL);
+
+   slot = 1;
+   dcr->VolCatInfo.Slot = slot;
+   Pmsg2(-1, _("3303 Issuing autochanger \"load slot %d %d\" command.\n"),
+      slot, dev->drive_index);
+   changer = edit_device_codes(dcr, changer, "load");
+   Dmsg1(100, "Changer=%s\n", changer);
+   force_close_dev(dev);
+   status = run_program(changer, timeout, results);
    if (status == 0) {
-      Pmsg1(-1,  _("3304 Autochanger \"load slot %d\" status is OK.\n"), slot);
+      Pmsg2(-1,  _("3303 Autochanger \"load slot %d %d\" status is OK.\n"),
+        slot, dev->drive_index);
    } else {
-      Pmsg1(-1,  _("3992 Bad autochanger \"load slot\" status=%d.\n"), status);
+      berrno be;
+      Pmsg1(-1, _("3993 Bad autochanger command: %s\n"), changer);
+      Pmsg2(-1, _("3993 result=\"%s\": ERR=%s\n"), results, be.strerror(status));
       goto bail_out;
    }
 
    if (!open_the_device()) {
       goto bail_out;
    }
+   /*
+    * Start with sleep_time 0 then increment by 30 seconds if we get
+    * a failure.
+    */
    bmicrosleep(sleep_time, 0);
-   if (!rewind_dev(dev)) {
+   if (!rewind_dev(dev) || weof_dev(dev,1) < 0) {
       Pmsg1(0, "Bad status from rewind. ERR=%s\n", strerror_dev(dev));
       clrerror_dev(dev, -1);
       Pmsg0(-1, "\nThe test failed, probably because you need to put\n"
-                "a longer sleep time in the mtx-script in the load) case.\n" 
+                "a longer sleep time in the mtx-script in the load) case.\n"
                 "Adding a 30 second sleep and trying again ...\n");
       sleep_time += 30;
       goto try_again;
    } else {
-      Pmsg1(0, "Rewound %s\n", dev_name(dev));
+      Pmsg1(0, "Rewound %s\n", dev->print_name());
    }
-      
+
    if ((status = weof_dev(dev, 1)) < 0) {
       Pmsg2(0, "Bad status from weof %d. ERR=%s\n", status, strerror_dev(dev));
       goto bail_out;
    } else {
-      Pmsg1(0, "Wrote EOF to %s\n", dev_name(dev));
+      Pmsg1(0, "Wrote EOF to %s\n", dev->print_name());
    }
 
    if (sleep_time) {
@@ -1161,7 +1239,7 @@ static void autochangercmd()
 static int fsf_test()
 {
    bool set_off = false;
-   
+
    Pmsg0(-1, _("\n\n=== Forward space files test ===\n\n"
                "This test is essential to Bacula.\n\n"
                "I'm going to write five files then test forward spacing\n\n"));
@@ -1192,7 +1270,7 @@ test_again:
       Pmsg1(0, "Bad status from fsr. ERR=%s\n", strerror_dev(dev));
       goto bail_out;
    }
-   Pmsg2(-1, _("We should be in file 1. I am at file %d. This is %s\n"), 
+   Pmsg2(-1, _("We should be in file 1. I am at file %d. This is %s\n"),
       dev->file, dev->file == 1 ? "correct!" : "NOT correct!!!!");
 
    if (dev->file != 1) {
@@ -1204,7 +1282,7 @@ test_again:
       Pmsg1(0, "Bad status from fsr. ERR=%s\n", strerror_dev(dev));
       goto bail_out;
    }
-   Pmsg2(-1, _("We should be in file 3. I am at file %d. This is %s\n"), 
+   Pmsg2(-1, _("We should be in file 3. I am at file %d. This is %s\n"),
       dev->file, dev->file == 3 ? "correct!" : "NOT correct!!!!");
 
    if (dev->file != 3) {
@@ -1217,7 +1295,7 @@ test_again:
       Pmsg1(0, "Bad status from fsr. ERR=%s\n", strerror_dev(dev));
       goto bail_out;
    }
-   Pmsg2(-1, _("We should be in file 4. I am at file %d. This is %s\n"), 
+   Pmsg2(-1, _("We should be in file 4. I am at file %d. This is %s\n"),
       dev->file, dev->file == 4 ? "correct!" : "NOT correct!!!!");
 
    if (dev->file != 4) {
@@ -1234,12 +1312,12 @@ test_again:
    if (!fsf_dev(dev, 1)) {
       Pmsg1(0, "Bad status from fsr. ERR=%s\n", strerror_dev(dev));
    }
-   Pmsg2(-1, _("We should be in file 5. I am at file %d. This is %s\n"), 
+   Pmsg2(-1, _("We should be in file 5. I am at file %d. This is %s\n"),
       dev->file, dev->file == 5 ? "correct!" : "NOT correct!!!!");
    if (dev->file != 5) {
       goto bail_out;
    }
-
+   Pmsg0(-1, _("\n=== End Forward space files test ===\n\n"));
    return 1;
 
 bail_out:
@@ -1251,7 +1329,10 @@ bail_out:
       set_off = true;
       goto test_again;
    }
-   Pmsg0(-1, "You must correct this error or Bacula will not work.\n");
+   Pmsg0(-1, "You must correct this error or Bacula will not work.\n"
+            "Some systems, e.g. OpenBSD, require you to set\n"
+            "   Use MTIOCGET= no\n"
+            "in your device resource. Use with caution.\n");
    return -2;
 }
 
@@ -1259,7 +1340,7 @@ bail_out:
 
 
 
-/* 
+/*
  * This is a general test of Bacula's functions
  *   needed to read and write the tape.
  */
@@ -1328,7 +1409,11 @@ failed:
             "nnn must match your tape driver's block size, which\n"
             "can be determined by reading your tape manufacturers\n"
             "information, and the information on your kernel dirver.\n"
-            "Fixed block sizes, however, are not normally an ideal solution.\n");
+            "Fixed block sizes, however, are not normally an ideal solution.\n"
+            "\n"
+            "Some systems, e.g. OpenBSD, require you to set\n"
+            "   Use MTIOCGET= no\n"
+            "in your device resource. Use with caution.\n");
        return;
    }
 
@@ -1342,7 +1427,7 @@ all_done:
         "End of File mark.\n"
         "3 blocks of 64448 bytes in file 3\n"
         "End of File mark.\n"
-        "1 block of 64448 bytes in file 4\n" 
+        "1 block of 64448 bytes in file 4\n"
         "End of File mark.\n"
         "Total files=4, blocks=7, bytes = 451,136\n"
         "=== End sample correct output ===\n\n"));
@@ -1359,7 +1444,7 @@ all_done:
    fsf_test();                       /* do fast forward space file test */
 
    autochanger_test();               /* do autochanger test */
-   
+
 }
 
 /* Forward space a file */
@@ -1402,27 +1487,28 @@ static void fsrcmd()
  */
 static void wrcmd()
 {
-   DEV_BLOCK *block;
-   DEV_RECORD *rec;
+   DEV_BLOCK *block = dcr->block;
+   DEV_RECORD *rec = dcr->rec;
    int i;
 
-   sm_check(__FILE__, __LINE__, False);
-   block = new_block(dev);
-   rec = new_record();
-   dump_block(block, "test");
+   sm_check(__FILE__, __LINE__, false);
+   empty_block(block);
+   if (verbose > 1) {
+      dump_block(block, "test");
+   }
 
    i = block->buf_len - 100;
    ASSERT (i > 0);
    rec->data = check_pool_memory_size(rec->data, i);
    memset(rec->data, i & 0xFF, i);
    rec->data_len = i;
-   sm_check(__FILE__, __LINE__, False);
+   sm_check(__FILE__, __LINE__, false);
    if (!write_record_to_block(block, rec)) {
-      Pmsg0(0, _("Error writing record to block.\n")); 
+      Pmsg0(0, _("Error writing record to block.\n"));
       goto bail_out;
    }
-   if (!write_block_to_dev(jcr, dev, block)) {
-      Pmsg0(0, _("Error writing block to device.\n")); 
+   if (!write_block_to_dev(dcr)) {
+      Pmsg0(0, _("Error writing block to device.\n"));
       goto bail_out;
    } else {
       Pmsg1(0, _("Wrote one record of %d bytes.\n"), i);
@@ -1430,13 +1516,11 @@ static void wrcmd()
    Pmsg0(0, _("Wrote block to device.\n"));
 
 bail_out:
-   sm_check(__FILE__, __LINE__, False);
-   free_record(rec);
-   free_block(block);
-   sm_check(__FILE__, __LINE__, False);
+   sm_check(__FILE__, __LINE__, false);
+   sm_check(__FILE__, __LINE__, false);
 }
 
-/* 
+/*
  * Read a record from the tape
  */
 static void rrcmd()
@@ -1457,8 +1541,9 @@ static void rrcmd()
    if (stat > 0 && stat <= len) {
       errno = 0;
    }
+   berrno be;
    Pmsg3(0, _("Read of %d bytes gives stat=%d. ERR=%s\n"),
-      len, stat, strerror(errno));
+      len, stat, be.strerror());
    free(buf);
 }
 
@@ -1481,19 +1566,20 @@ static void scancmd()
    bytes = 0;
    if (dev->state & ST_EOT) {
       Pmsg0(0, "End of tape\n");
-      return; 
+      return;
    }
    update_pos_dev(dev);
    tot_files = dev->file;
    Pmsg1(0, _("Starting scan at file %u\n"), dev->file);
    for (;;) {
       if ((stat = read(dev->fd, buf, sizeof(buf))) < 0) {
+        berrno be;
         clrerror_dev(dev, -1);
-         Mmsg2(&dev->errmsg, "read error on %s. ERR=%s.\n",
-           dev->dev_name, strerror(dev->dev_errno));
+         Mmsg2(dev->errmsg, "read error on %s. ERR=%s.\n",
+           dev->dev_name, be.strerror());
          Pmsg2(0, "Bad status from read %d. ERR=%s\n", stat, strerror_dev(dev));
         if (blocks > 0)
-            printf("%d block%s of %d bytes in file %d\n",        
+            printf("%d block%s of %d bytes in file %d\n",
                     blocks, blocks>1?"s":"", block_size, dev->file);
         return;
       }
@@ -1502,7 +1588,7 @@ static void scancmd()
       if (stat != block_size) {
         update_pos_dev(dev);
         if (blocks > 0) {
-            printf("%d block%s of %d bytes in file %d\n", 
+            printf("%d block%s of %d bytes in file %d\n",
                  blocks, blocks>1?"s":"", block_size, dev->file);
            blocks = 0;
         }
@@ -1531,7 +1617,7 @@ static void scancmd()
    }
    update_pos_dev(dev);
    tot_files = dev->file - tot_files;
-   printf("Total files=%d, blocks=%d, bytes = %s\n", tot_files, tot_blocks, 
+   printf("Total files=%d, blocks=%d, bytes = %s\n", tot_files, tot_blocks,
       edit_uint64_with_commas(bytes, ec1));
 }
 
@@ -1547,21 +1633,21 @@ static void scan_blocks()
    int blocks, tot_blocks, tot_files;
    uint32_t block_size;
    uint64_t bytes;
-   DEV_BLOCK *block;
+   DEV_BLOCK *block = dcr->block;
    char ec1[50];
 
-   block = new_block(dev);
    blocks = block_size = tot_blocks = 0;
    bytes = 0;
 
+   empty_block(block);
    update_pos_dev(dev);
    tot_files = dev->file;
    for (;;) {
-      if (!read_block_from_device(jcr, dev, block, NO_BLOCK_NUMBER_CHECK)) {
+      if (!read_block_from_device(dcr, NO_BLOCK_NUMBER_CHECK)) {
          Dmsg1(100, "!read_block(): ERR=%s\n", strerror_dev(dev));
         if (dev->state & ST_EOT) {
            if (blocks > 0) {
-               printf("%d block%s of %d bytes in file %d\n", 
+               printf("%d block%s of %d bytes in file %d\n",
                     blocks, blocks>1?"s":"", block_size, dev->file);
               blocks = 0;
            }
@@ -1569,7 +1655,7 @@ static void scan_blocks()
         }
         if (dev->state & ST_EOF) {
            if (blocks > 0) {
-               printf("%d block%s of %d bytes in file %d\n",        
+               printf("%d block%s of %d bytes in file %d\n",
                        blocks, blocks>1?"s":"", block_size, dev->file);
               blocks = 0;
            }
@@ -1578,7 +1664,7 @@ static void scan_blocks()
         }
         if (dev->state & ST_SHORT) {
            if (blocks > 0) {
-               printf("%d block%s of %d bytes in file %d\n",        
+               printf("%d block%s of %d bytes in file %d\n",
                        blocks, blocks>1?"s":"", block_size, dev->file);
               blocks = 0;
            }
@@ -1590,7 +1676,7 @@ static void scan_blocks()
       }
       if (block->block_len != block_size) {
         if (blocks > 0) {
-            printf("%d block%s of %d bytes in file %d\n",        
+            printf("%d block%s of %d bytes in file %d\n",
                     blocks, blocks>1?"s":"", block_size, dev->file);
            blocks = 0;
         }
@@ -1617,9 +1703,8 @@ static void scan_blocks()
 
    }
 bail_out:
-   free_block(block);
    tot_files = dev->file - tot_files;
-   printf("Total files=%d, blocks=%d, bytes = %s\n", tot_files, tot_blocks, 
+   printf("Total files=%d, blocks=%d, bytes = %s\n", tot_files, tot_blocks,
       edit_uint64_with_commas(bytes, ec1));
 }
 
@@ -1636,14 +1721,14 @@ static void statcmd()
 }
 
 
-/* 
+/*
  * First we label the tape, then we fill
  *  it with data get a new tape and write a few blocks.
- */                           
+ */
 static void fillcmd()
 {
    DEV_RECORD rec;
-   DEV_BLOCK  *block;
+   DEV_BLOCK  *block = dcr->block;
    char ec1[50];
    int fd;
    uint32_t i;
@@ -1656,22 +1741,22 @@ static void fillcmd()
    last_block_num = 0;
    BlockNumber = 0;
 
-   Pmsg0(-1, "\n\
-This command simulates Bacula writing to a tape.\n\
-It requires either one or two blank tapes, which it\n\
-will label and write.\n\n\
-If you have an autochanger configured, it will use\n\
-the tapes that are in slots 1 and 2, otherwise, you will\n\
-be prompted to insert the tapes when necessary.\n\n\
-It will print a status approximately\n\
-every 322 MB, and write an EOF every 3.2 GB.  If you have\n\
-selected the simple test option, after writing the first tape\n\
-it will rewind it and re-read the last block written.\n\n\
-If you have selected the multiple tape test, when the first tape\n\
-fills, it will ask for a second, and after writing a few more \n\
-blocks, it will stop.  Then it will begin re-reading the\n\
-two tapes.\n\n\
-This may take a long time -- hours! ...\n\n");
+   Pmsg0(-1, "\n"
+"This command simulates Bacula writing to a tape.\n"
+"It requires either one or two blank tapes, which it\n"
+"will label and write.\n\n"
+"If you have an autochanger configured, it will use\n"
+"the tapes that are in slots 1 and 2, otherwise, you will\n"
+"be prompted to insert the tapes when necessary.\n\n"
+"It will print a status approximately\n"
+"every 322 MB, and write an EOF every 3.2 GB.  If you have\n"
+"selected the simple test option, after writing the first tape\n"
+"it will rewind it and re-read the last block written.\n\n"
+"If you have selected the multiple tape test, when the first tape\n"
+"fills, it will ask for a second, and after writing a few more \n"
+"blocks, it will stop.  Then it will begin re-reading the\n"
+"two tapes.\n\n"
+"This may take a long time -- hours! ...\n\n");
 
    get_cmd("Do you want to run the simplified test (s) with one tape\n"
            "or the complete multiple tape (m) test: (s/m) ");
@@ -1679,27 +1764,37 @@ This may take a long time -- hours! ...\n\n");
       Pmsg0(-1, "Simple test (single tape) selected.\n");
       simple = true;
    } else if (cmd[0] == 'm') {
-      Pmsg0(-1, "Multiple tape test selected.\n"); 
+      Pmsg0(-1, "Multiple tape test selected.\n");
       simple = false;
    } else {
       Pmsg0(000, "Command aborted.\n");
       return;
    }
 
-   Dmsg1(20, "Begin append device=%s\n", dev_name(dev));
+   Dmsg1(20, "Begin append device=%s\n", dev->print_name());
    Dmsg1(20, "MaxVolSize=%s\n", edit_uint64(dev->max_volume_size, ec1));
 
    /* Use fixed block size to simplify read back */
    min_block_size = dev->min_block_size;
    dev->min_block_size = dev->max_block_size;
+   set_volume_name("TestVolume1", 1);
+
+   if (!dev->rewind()) {
+      Pmsg0(000, "Rewind failed.\n");
+   }
+   if (!dev->weof()) {
+      Pmsg0(000, "Write EOF failed.\n");
+   }
+   labelcmd();
+   dev->set_append();                /* force volume to be relabeled */
 
-   /* 
+   /*
     * Acquire output device for writing.  Note, after acquiring a
     *  device, we MUST release it, which is done at the end of this
     *  subroutine.
     */
    Dmsg0(100, "just before acquire_device\n");
-   if (!acquire_device_for_append(jcr)) {
+   if (!acquire_device_for_append(dcr)) {
       set_jcr_job_status(jcr, JS_ErrorTerminated);
       return;
    }
@@ -1709,13 +1804,13 @@ This may take a long time -- hours! ...\n\n");
    /*
     * Write Begin Session Record
     */
-   if (!write_session_label(jcr, block, SOS_LABEL)) {
+   if (!write_session_label(dcr, SOS_LABEL)) {
       set_jcr_job_status(jcr, JS_ErrorTerminated);
       Jmsg1(jcr, M_FATAL, 0, _("Write session label failed. ERR=%s\n"),
         strerror_dev(dev));
       ok = false;
    }
-   Pmsg0(-1, "Wrote Start Of Session label.\n");
+   Pmsg0(-1, "Wrote Start of Session label.\n");
 
    memset(&rec, 0, sizeof(rec));
    rec.data = get_memory(100000);     /* max record size */
@@ -1723,7 +1818,7 @@ This may take a long time -- hours! ...\n\n");
 #define REC_SIZE 32768
    rec.data_len = REC_SIZE;
 
-   /* 
+   /*
     * Put some random data in the record
     */
    fd = open("/dev/urandom", O_RDONLY);
@@ -1738,8 +1833,8 @@ This may take a long time -- hours! ...\n\n");
       }
    }
 
-   /* 
-    * Generate data as if from File daemon, write to device   
+   /*
+    * Generate data as if from File daemon, write to device
     */
    jcr->dcr->VolFirstIndex = 0;
    time(&jcr->run_time);             /* start counting time for rates */
@@ -1762,9 +1857,9 @@ This may take a long time -- hours! ...\n\n");
       }
 
       Dmsg4(250, "before write_rec FI=%d SessId=%d Strm=%s len=%d\n",
-        rec.FileIndex, rec.VolSessionId, stream_to_ascii(rec.Stream, rec.FileIndex), 
+        rec.FileIndex, rec.VolSessionId, stream_to_ascii(rec.Stream, rec.FileIndex),
         rec.data_len);
-       
+
       while (!write_record_to_block(block, &rec)) {
         /*
          * When we get here we have just filled a block
@@ -1786,7 +1881,7 @@ This may take a long time -- hours! ...\n\n");
               now = 1;          /* prevent divide error */
            }
            kbs = (double)dev->VolCatInfo.VolCatBytes / (1000.0 * (double)now);
-            Pmsg4(-1, "Wrote blk_block=%u, dev_blk_num=%u VolBytes=%s rate=%.1f KB/s\n", 
+            Pmsg4(-1, "Wrote blk_block=%u, dev_blk_num=%u VolBytes=%s rate=%.1f KB/s\n",
               block->BlockNumber, dev->block_num,
               edit_uint64_with_commas(dev->VolCatInfo.VolCatBytes, ec1), (float)kbs);
         }
@@ -1800,7 +1895,7 @@ This may take a long time -- hours! ...\n\n");
 
         /* Get out after writing 10 blocks to the second tape */
         if (++BlockNumber > 10 && stop != 0) {      /* get out */
-           break;    
+           break;
         }
       }
       if (!ok) {
@@ -1809,13 +1904,13 @@ This may take a long time -- hours! ...\n\n");
       }
       jcr->JobBytes += rec.data_len;   /* increment bytes this job */
       Dmsg4(190, "write_record FI=%s SessId=%d Strm=%s len=%d\n",
-        FI_to_ascii(rec.FileIndex), rec.VolSessionId, 
+        FI_to_ascii(rec.FileIndex), rec.VolSessionId,
         stream_to_ascii(rec.Stream, rec.FileIndex), rec.data_len);
 
       /* Get out after writing 10 blocks to the second tape */
       if (BlockNumber > 10 && stop != 0) {     /* get out */
          Pmsg0(-1, "Done writing ...\n");
-        break;    
+        break;
       }
    }
    if (vol_num > 1) {
@@ -1826,16 +1921,16 @@ This may take a long time -- hours! ...\n\n");
       } else if (!ok) {
         set_jcr_job_status(jcr, JS_ErrorTerminated);
       }
-      if (!write_session_label(jcr, block, EOS_LABEL)) {
+      if (!write_session_label(dcr, EOS_LABEL)) {
          Pmsg1(000, _("Error writting end session label. ERR=%s\n"), strerror_dev(dev));
-        ok = FALSE;
+        ok = false;
       }
       /* Write out final block of this session */
-      if (!write_block_to_device(jcr, dev, block)) {
-         Pmsg0(-1, _("Set ok=FALSE after write_block_to_device.\n"));
-        ok = FALSE;
+      if (!write_block_to_device(dcr)) {
+         Pmsg0(-1, _("Set ok=false after write_block_to_device.\n"));
+        ok = false;
       }
-      Pmsg0(-1, _("Wrote End Of Session label.\n"));
+      Pmsg0(-1, _("Wrote End of Session label.\n"));
 
       /* Save last block info for second tape */
       last_block_num2 = last_block_num;
@@ -1862,20 +1957,15 @@ This may take a long time -- hours! ...\n\n");
       Pmsg2(-1, "Wrote state file last_block_num1=%d last_block_num2=%d\n",
         last_block_num1, last_block_num2);
    } else {
+      berrno be;
       Pmsg2(-1, _("Could not create state file: %s ERR=%s\n"), buf,
-                strerror(errno));
-   }
-
-   /* Release the device */
-   if (!release_device(jcr)) {
-      Pmsg0(-1, _("Error in release_device\n"));
-      ok = FALSE;
+                be.strerror());
    }
 
+   Pmsg4(-1, _("\n\nDone filling tape%s at %d:%d. Now beginning re-read of %stape ...\n"),
+      simple?"":"s", jcr->dcr->dev->file, jcr->dcr->dev->block_num, simple?"":"first ");
 
-   Pmsg2(-1, _("\n\nDone filling tape%s. Now beginning re-read of %stape ...\n"),
-      simple?"":"s", simple?"":"first ");
-
+   jcr->dcr->block = block;
    do_unfill();
 
    dev->min_block_size = min_block_size;
@@ -1898,7 +1988,7 @@ static void unfillcmd()
    sprintf(buf, "%s/btape.state", working_directory);
    fd = open(buf, O_RDONLY);
    if (fd >= 0) {
-      uint32_t state_level;             
+      uint32_t state_level;
       read(fd, &state_level, sizeof(btape_state_level));
       read(fd, &simple, sizeof(simple));
       read(fd, &last_block_num1, sizeof(last_block_num1));
@@ -1915,8 +2005,9 @@ static void unfillcmd()
          return;
        }
    } else {
+      berrno be;
       Pmsg2(-1, "\nCould not find the state file: %s ERR=%s\n"
-             "You must redo the fill command.\n", buf, strerror(errno));
+             "You must redo the fill command.\n", buf, be.strerror());
       return;
    }
    do_unfill();
@@ -1925,13 +2016,12 @@ static void unfillcmd()
 
 static void do_unfill()
 {
-   DEV_BLOCK *block;
+   DEV_BLOCK *block = dcr->block;
    bool autochanger;
 
    dumped = 0;
    VolBytes = 0;
    LastBlock = 0;
-   block = new_block(dev);
 
    Dmsg0(20, "Enter do_unfill\n");
    dev->capabilities |= CAP_ANONVOLS; /* allow reading any volume */
@@ -1955,39 +2045,44 @@ static void do_unfill()
       if (dev_cap(dev, CAP_OFFLINEUNMOUNT)) {
         offline_dev(dev);
       }
-      autochanger = autoload_device(jcr, dev, 1, NULL);
+      autochanger = autoload_device(dcr, 1, NULL);
       if (!autochanger) {
         force_close_dev(dev);
-         get_cmd(_("Mount first tape. Press enter when ready: ")); 
+         get_cmd(_("Mount first tape. Press enter when ready: "));
       }
-   
       free_vol_list(jcr);
+      jcr->dcr = new_dcr(jcr, dev);
       set_volume_name("TestVolume1", 1);
       jcr->bsr = NULL;
       create_vol_list(jcr);
       close_dev(dev);
-      dev->state &= ~ST_READ;
-      if (!acquire_device_for_read(jcr)) {
+      dev->state &= ~(ST_READ|ST_APPEND);
+      dev->num_writers = 0;
+      if (!acquire_device_for_read(dcr)) {
          Pmsg1(-1, "%s", dev->errmsg);
         goto bail_out;
       }
    }
    /*
     * We now have the first tape mounted.
-    * Note, re-reading last block may have caused us to 
+    * Note, re-reading last block may have caused us to
     *  loose track of where we are (block number unknown).
     */
-   rewind_dev(dev);                  /* get to a known place on tape */
-   /* Read the first 1000 records */
-   Pmsg0(-1, _("Reading the first 1000 records.\n"));
-   read_records(jcr, dev, quickie_cb, my_mount_next_read_volume);
+   if (!rewind_dev(dev)) {               /* get to a known place on tape */
+      goto bail_out;
+   }
+   /* Read the first 10000 records */
+   Pmsg0(-1, _("Reading the first 10000 records.\n"));
+   quickie_count = 0;
+   read_records(dcr, quickie_cb, my_mount_next_read_volume);
    Pmsg4(-1, _("Reposition from %u:%u to %u:%u\n"), dev->file, dev->block_num,
         last_file, last_block_num);
    if (!reposition_dev(dev, last_file, last_block_num)) {
       Pmsg1(-1, "Reposition error. ERR=%s\n", strerror_dev(dev));
+      goto bail_out;
    }
    Pmsg1(-1, _("Reading block %u.\n"), last_block_num);
-   if (!read_block_from_device(jcr, dev, block, NO_BLOCK_NUMBER_CHECK)) {
+   if (!read_block_from_device(dcr, NO_BLOCK_NUMBER_CHECK)) {
       Pmsg1(-1, _("Error reading block: ERR=%s\n"), strerror_dev(dev));
       goto bail_out;
    }
@@ -2017,14 +2112,14 @@ static void do_unfill()
    set_volume_name("TestVolume2", 2);
    jcr->bsr = NULL;
    create_vol_list(jcr);
-   autochanger = autoload_device(jcr, dev, 1, NULL);
+   autochanger = autoload_device(dcr, 1, NULL);
    if (!autochanger) {
       force_close_dev(dev);
-      get_cmd(_("Mount second tape. Press enter when ready: ")); 
+      get_cmd(_("Mount second tape. Press enter when ready: "));
    }
 
-   dev->state &= ~ST_READ;
-   if (!acquire_device_for_read(jcr)) {
+   dev->clear_read();
+   if (!acquire_device_for_read(dcr)) {
       Pmsg1(-1, "%s", dev->errmsg);
       goto bail_out;
    }
@@ -2038,7 +2133,7 @@ static void do_unfill()
       goto bail_out;
    }
    Pmsg1(-1, _("Reading block %d.\n"), dev->block_num);
-   if (!read_block_from_device(jcr, dev, block, NO_BLOCK_NUMBER_CHECK)) {
+   if (!read_block_from_device(dcr, NO_BLOCK_NUMBER_CHECK)) {
       Pmsg1(-1, _("Error reading block: ERR=%s\n"), strerror_dev(dev));
       goto bail_out;
    }
@@ -2054,7 +2149,7 @@ static void do_unfill()
       goto bail_out;
    }
    Pmsg1(-1, _("Reading block %d.\n"), dev->block_num);
-   if (!read_block_from_device(jcr, dev, block, NO_BLOCK_NUMBER_CHECK)) {
+   if (!read_block_from_device(dcr, NO_BLOCK_NUMBER_CHECK)) {
       Pmsg1(-1, _("Error reading block: ERR=%s\n"), strerror_dev(dev));
       goto bail_out;
    }
@@ -2063,26 +2158,34 @@ static void do_unfill()
    }
 
 bail_out:
-   free_block(block);
    free_block(last_block1);
    free_block(last_block2);
    free_block(first_block);
 }
 
-/* Read 1000 records then stop */
-static int quickie_cb(JCR *jcr, DEVICE *dev, DEV_BLOCK *block, DEV_RECORD *rec)
+/* Read 10000 records then stop */
+static bool quickie_cb(DCR *dcr, DEV_RECORD *rec)
 {
+   DEVICE *dev = dcr->dev;
+   if (dev->file != 0) {
+      Pmsg3(-1, "ERROR! device at %d:%d count=%d\n", dev->file, dev->block_num,
+        quickie_count);
+      return false;
+   }
    quickie_count++;
-   return quickie_count <= 1000;
+   if (quickie_count == 10000) {
+      Pmsg2(-1, "1000 records read now at %d:%d\n", dev->file, dev->block_num);
+   }
+   return quickie_count < 10000;
 }
 
-static bool compare_blocks(DEV_BLOCK *last_block, DEV_BLOCK *block) 
+static bool compare_blocks(DEV_BLOCK *last_block, DEV_BLOCK *block)
 {
    char *p, *q;
    uint32_t CheckSum, block_len;
    ser_declare;
 
-   p = last_block->buf;      
+   p = last_block->buf;
    q = block->buf;
    unser_begin(q, BLKHDR2_LENGTH);
    unser_uint32(CheckSum);
@@ -2123,10 +2226,10 @@ static bool compare_blocks(DEV_BLOCK *last_block, DEV_BLOCK *block)
 static int flush_block(DEV_BLOCK *block, int dump)
 {
    char ec1[50];
-   lock_device(dev);
    DEV_BLOCK *tblock;
    uint32_t this_file, this_block_num;
 
+   lock_device(dev);
    if (!this_block) {
       this_block = new_block(dev);
    }
@@ -2136,13 +2239,13 @@ static int flush_block(DEV_BLOCK *block, int dump)
    /* Copy block */
    this_file = dev->file;
    this_block_num = dev->block_num;
-   if (!write_block_to_dev(jcr, dev, block)) {
-      Pmsg3(000, "Last block at: %u:%u this_dev_block_num=%d\n", 
+   if (!write_block_to_dev(dcr)) {
+      Pmsg3(000, "Last block at: %u:%u this_dev_block_num=%d\n",
                  last_file, last_block_num, this_block_num);
       if (vol_num == 1) {
-        /* 
+        /*
          * This is 1st tape, so save first tape info separate
-         *  from second tape info 
+         *  from second tape info
          */
         last_block_num1 = last_block_num;
         last_file1 = last_file;
@@ -2151,7 +2254,7 @@ static int flush_block(DEV_BLOCK *block, int dump)
         first_block = dup_block(block); /* first block second tape */
       }
       if (verbose) {
-         Pmsg3(000, "Block not written: FileIndex=%u blk_block=%u Size=%u\n", 
+         Pmsg3(000, "Block not written: FileIndex=%u blk_block=%u Size=%u\n",
            (unsigned)file_index, block->BlockNumber, block->block_len);
          dump_block(last_block, "Last block written");
          Pmsg0(-1, "\n");
@@ -2170,16 +2273,17 @@ static int flush_block(DEV_BLOCK *block, int dump)
       }
       kbs = (double)dev->VolCatInfo.VolCatBytes / (1000 * now);
       vol_size = dev->VolCatInfo.VolCatBytes;
-      Pmsg2(000, "End of tape. VolumeCapacity=%s. Write rate = %.1f KB/s\n", 
+      Pmsg4(000, "End of tape %d:%d. VolumeCapacity=%s. Write rate = %.1f KB/s\n",
+        dev->file, dev->block_num,
         edit_uint64_with_commas(dev->VolCatInfo.VolCatBytes, ec1), kbs);
 
       if (simple) {
         stop = -1;                   /* stop, but do simplified test */
       } else {
         /* Full test in progress */
-        if (!fixup_device_block_write_error(jcr, dev, block)) {
+        if (!fixup_device_block_write_error(jcr->dcr)) {
             Pmsg1(000, _("Cannot fixup device error. %s\n"), strerror_dev(dev));
-           ok = FALSE;
+           ok = false;
            unlock_device(dev);
            return 0;
         }
@@ -2193,12 +2297,13 @@ static int flush_block(DEV_BLOCK *block, int dump)
    memcpy(this_block->buf, block->buf, this_block->buf_len);
 
    /*
-    * Toggle between two allocated blocks for efficiency.
+    * Note, we always read/write to block, but we toggle
+    *  copying it to one or another of two allocated blocks.
     * Switch blocks so that the block just successfully written is
-    *  always in last_block. 
+    *  always in last_block.
     */
    tblock = last_block;
-   last_block = this_block; 
+   last_block = this_block;
    this_block = tblock;
    last_file = this_file;
    last_block_num = this_block_num;
@@ -2208,14 +2313,14 @@ static int flush_block(DEV_BLOCK *block, int dump)
 }
 
 
-/* 
+/*
  * First we label the tape, then we fill
  *  it with data get a new tape and write a few blocks.
- */                           
+ */
 static void qfillcmd()
 {
-   DEV_BLOCK *block;
-   DEV_RECORD *rec;
+   DEV_BLOCK *block = dcr->block;
+   DEV_RECORD *rec = dcr->rec;
    int i, count;
 
    Pmsg0(0, "Test writing blocks of 64512 bytes to tape.\n");
@@ -2227,9 +2332,7 @@ static void qfillcmd()
       count = 1000;
    }
 
-   sm_check(__FILE__, __LINE__, False);
-   block = new_block(dev);
-   rec = new_record();
+   sm_check(__FILE__, __LINE__, false);
 
    i = block->buf_len - 100;
    ASSERT (i > 0);
@@ -2244,11 +2347,11 @@ static void qfillcmd()
         fflush(stdout);
       }
       if (!write_record_to_block(block, rec)) {
-         Pmsg0(0, _("Error writing record to block.\n")); 
+         Pmsg0(0, _("Error writing record to block.\n"));
         goto bail_out;
       }
-      if (!write_block_to_dev(jcr, dev, block)) {
-         Pmsg0(0, _("Error writing block to device.\n")); 
+      if (!write_block_to_dev(dcr)) {
+         Pmsg0(0, _("Error writing block to device.\n"));
         goto bail_out;
       }
    }
@@ -2261,11 +2364,7 @@ static void qfillcmd()
    scan_blocks();
 
 bail_out:
-   sm_check(__FILE__, __LINE__, False);
-   free_record(rec);
-   free_block(block);
-   sm_check(__FILE__, __LINE__, False);
-
+   sm_check(__FILE__, __LINE__, false);
 }
 
 /*
@@ -2273,7 +2372,7 @@ bail_out:
  */
 static void rawfill_cmd()
 {
-   DEV_BLOCK *block;
+   DEV_BLOCK *block = dcr->block;
    int stat;
    int fd;
    uint32_t block_num = 0;
@@ -2281,7 +2380,6 @@ static void rawfill_cmd()
    int my_errno;
    uint32_t i;
 
-   block = new_block(dev);
    fd = open("/dev/urandom", O_RDONLY);
    if (fd) {
       read(fd, block->buf, block->buf_len);
@@ -2313,26 +2411,25 @@ static void rawfill_cmd()
    }
    my_errno = errno;
    printf("\n");
+   berrno be;
    printf("Write failed at block %u. stat=%d ERR=%s\n", block_num, stat,
-      strerror(my_errno));
+      be.strerror(my_errno));
    weofcmd();
-   free_block(block);
 }
 
 
 /*
- * Fill a tape using raw write() command
+ * Fill a tape using Bacula block writes
  */
 static void bfill_cmd()
 {
-   DEV_BLOCK *block;
+   DEV_BLOCK *block = dcr->block;
    uint32_t block_num = 0;
    uint32_t *p;
    int my_errno;
-   int fd;   
+   int fd;
    uint32_t i;
 
-   block = new_block(dev);
    fd = open("/dev/urandom", O_RDONLY);
    if (fd) {
       read(fd, block->buf, block->buf_len);
@@ -2350,7 +2447,7 @@ static void bfill_cmd()
       *p = block_num;
       block->binbuf = block->buf_len;
       block->bufp = block->buf + block->binbuf;
-      if (!write_block_to_dev(jcr, dev, block)) {
+      if (!write_block_to_dev(dcr)) {
         break;
       }
       if ((block_num++ % 100) == 0) {
@@ -2364,13 +2461,12 @@ static void bfill_cmd()
    }
    my_errno = errno;
    printf("\n");
-   printf("Write failed at block %u.\n", block_num);     
+   printf("Write failed at block %u.\n", block_num);
    weofcmd();
-   free_block(block);
 }
 
 
-struct cmdstruct { char *key; void (*func)(); char *help; }; 
+struct cmdstruct { const char *key; void (*func)(); const char *help; };
 static struct cmdstruct commands[] = {
  {"autochanger", autochangercmd, "test autochanger"},
  {"bsf",        bsfcmd,       "backspace file"},
@@ -2387,17 +2483,17 @@ static struct cmdstruct commands[] = {
  {"help",       helpcmd,      "print this command"},
  {"label",      labelcmd,     "write a Bacula label to the tape"},
  {"load",       loadcmd,      "load a tape"},
- {"quit",       quitcmd,      "quit btape"},   
+ {"quit",       quitcmd,      "quit btape"},
  {"rawfill",    rawfill_cmd,  "use write() to fill tape"},
  {"readlabel",  readlabelcmd, "read and print the Bacula tape label"},
  {"rectest",    rectestcmd,   "test record handling functions"},
  {"rewind",     rewindcmd,    "rewind the tape"},
- {"scan",       scancmd,      "read() tape block by block to EOT and report"}, 
+ {"scan",       scancmd,      "read() tape block by block to EOT and report"},
  {"scanblocks", scan_blocks,  "Bacula read block by block to EOT and report"},
  {"status",     statcmd,      "print tape status"},
  {"test",       testcmd,      "General test Bacula tape functions"},
  {"weof",       weofcmd,      "write an EOF on the tape"},
- {"wr",         wrcmd,        "write a single Bacula block"}, 
+ {"wr",         wrcmd,        "write a single Bacula block"},
  {"rr",         rrcmd,        "read a single record"},
  {"qfill",      qfillcmd,     "quick fill command"},
             };
@@ -2410,7 +2506,7 @@ do_tape_cmds()
    bool found;
 
    while (get_cmd("*")) {
-      sm_check(__FILE__, __LINE__, False);
+      sm_check(__FILE__, __LINE__, false);
       found = false;
       parse_args(cmd, &args, &argc, argk, argv, MAX_CMD_ARGS);
       for (i=0; i<comsize; i++)       /* search for command */
@@ -2440,25 +2536,27 @@ static void helpcmd()
 static void usage()
 {
    fprintf(stderr, _(
+"Copyright (C) 2000-2005 Kern Sibbald.\n"
 "\nVersion: " VERSION " (" BDATE ")\n\n"
 "Usage: btape <options> <device_name>\n"
 "       -b <file>   specify bootstrap file\n"
 "       -c <file>   set configuration file to file\n"
 "       -d <nn>     set debug level to nn\n"
+"       -p          proceed inspite of I/O errors\n"
 "       -s          turn off signals\n"
 "       -v          be verbose\n"
-"       -?          print this message.\n"  
+"       -?          print this message.\n"
 "\n"));
 
 }
 
-/*     
+/*
  * Get next input command from terminal.  This
  * routine is REALLY primitive, and should be enhanced
  * to have correct backspacing, etc.
  */
-int 
-get_cmd(char *prompt)
+int
+get_cmd(const char *prompt)
 {
    int i = 0;
    int ch;
@@ -2468,7 +2566,7 @@ get_cmd(char *prompt)
     * up a bit.
     */
    cmd[i] = 0;
-   while ((ch = fgetc(stdin)) != EOF) { 
+   while ((ch = fgetc(stdin)) != EOF) {
       if (ch == '\n') {
         strip_trailing_junk(cmd);
         return 1;
@@ -2476,8 +2574,8 @@ get_cmd(char *prompt)
         if (i > 0)
            cmd[--i] = 0;
         continue;
-      } 
-        
+      }
+
       cmd[i++] = ch;
       cmd[i] = 0;
    }
@@ -2486,40 +2584,41 @@ get_cmd(char *prompt)
 }
 
 /* Dummies to replace askdir.c */
-int    dir_update_file_attributes(JCR *jcr, DEV_RECORD *rec) { return 1;}
-int    dir_send_job_status(JCR *jcr) {return 1;}
+bool   dir_update_file_attributes(DCR *dcr, DEV_RECORD *rec) { return 1;}
+bool   dir_send_job_status(JCR *jcr) {return 1;}
 
-int dir_update_volume_info(JCR *jcr, DEVICE *dev, int relabel) 
-{ 
+bool dir_update_volume_info(DCR *dcr, bool relabel)
+{
    return 1;
 }
 
 
-int dir_get_volume_info(JCR *jcr, enum get_vol_info_rw writing)             
+bool dir_get_volume_info(DCR *dcr, enum get_vol_info_rw  writing)
 {
    Dmsg0(20, "Enter dir_get_volume_info\n");
-   bstrncpy(jcr->VolCatInfo.VolCatName, jcr->VolumeName, sizeof(jcr->VolCatInfo.VolCatName));
+   bstrncpy(dcr->VolCatInfo.VolCatName, dcr->VolumeName, sizeof(dcr->VolCatInfo.VolCatName));
    return 1;
 }
 
-int dir_create_jobmedia_record(JCR *jcr)
+bool dir_create_jobmedia_record(DCR *dcr)
 {
-   jcr->dcr->WroteVol = false;
+   dcr->WroteVol = false;
    return 1;
 }
 
 
-int dir_find_next_appendable_volume(JCR *jcr) 
-{ 
+bool dir_find_next_appendable_volume(DCR *dcr)
+{
    Dmsg1(20, "Enter dir_find_next_appendable_volume. stop=%d\n", stop);
-   return jcr->VolumeName[0] != 0;
+   return dcr->VolumeName[0] != 0;
 }
 
-int dir_ask_sysop_to_mount_volume(JCR *jcr, DEVICE *dev)
+bool dir_ask_sysop_to_mount_volume(DCR *dcr)
 {
+   DEVICE *dev = dcr->dev;
    Dmsg0(20, "Enter dir_ask_sysop_to_mount_volume\n");
-   if (jcr->VolumeName[0] == 0) {
-      return dir_ask_sysop_to_create_appendable_volume(jcr, dev);
+   if (dcr->VolumeName[0] == 0) {
+      return dir_ask_sysop_to_create_appendable_volume(dcr);
    }
    /* Close device so user can use autochanger if desired */
    if (dev_cap(dev, CAP_OFFLINEUNMOUNT)) {
@@ -2527,20 +2626,21 @@ int dir_ask_sysop_to_mount_volume(JCR *jcr, DEVICE *dev)
    }
    force_close_dev(dev);
    Pmsg1(-1, "%s", dev->errmsg);           /* print reason */
-   if (jcr->VolumeName[0] == 0 || strcmp(jcr->VolumeName, "TestVolume2") == 0) {
+   if (dcr->VolumeName[0] == 0 || strcmp(dcr->VolumeName, "TestVolume2") == 0) {
       fprintf(stderr, "Mount second Volume on device %s and press return when ready: ",
-        dev_name(dev));
+        dev->print_name());
    } else {
       fprintf(stderr, "Mount Volume \"%s\" on device %s and press return when ready: ",
-        jcr->VolumeName, dev_name(dev));
+        dcr->VolumeName, dev->print_name());
    }
-   getchar();  
-   return 1;
+   getchar();
+   return true;
 }
 
-int dir_ask_sysop_to_create_appendable_volume(JCR *jcr, DEVICE *dev)
+bool dir_ask_sysop_to_create_appendable_volume(DCR *dcr)
 {
    bool autochanger;
+   DEVICE *dev = dcr->dev;
    Dmsg0(20, "Enter dir_ask_sysop_to_create_appendable_volume\n");
    if (stop == 0) {
       set_volume_name("TestVolume1", 1);
@@ -2551,26 +2651,28 @@ int dir_ask_sysop_to_create_appendable_volume(JCR *jcr, DEVICE *dev)
    if (dev_cap(dev, CAP_OFFLINEUNMOUNT)) {
       offline_dev(dev);
    }
-   autochanger = autoload_device(jcr, dev, 1, NULL);
+   autochanger = autoload_device(dcr, 1, NULL);
    if (!autochanger) {
       force_close_dev(dev);
       fprintf(stderr, "Mount blank Volume on device %s and press return when ready: ",
-        dev_name(dev));
-      getchar();   
+        dev->print_name());
+      getchar();
    }
-   open_device(jcr, dev);
+   open_device(dcr);
    labelcmd();
    VolumeName = NULL;
    BlockNumber = 0;
-   return 1;
+   return true;
 }
 
-static int my_mount_next_read_volume(JCR *jcr, DEVICE *dev, DEV_BLOCK *block)
+static bool my_mount_next_read_volume(DCR *dcr)
 {
    char ec1[50];
+   JCR *jcr = dcr->jcr;
+   DEV_BLOCK *block = dcr->block;
 
    Dmsg0(20, "Enter my_mount_next_read_volume\n");
-   Pmsg1(000, "End of Volume \"%s\"\n", jcr->VolumeName);
+   Pmsg1(000, "End of Volume \"%s\"\n", dcr->VolumeName);
 
    if (LastBlock != block->BlockNumber) {
       VolBytes += block->block_len;
@@ -2585,9 +2687,9 @@ static int my_mount_next_read_volume(JCR *jcr, DEVICE *dev, DEV_BLOCK *block)
    Pmsg3(-1, "Read block=%u, VolBytes=%s rate=%.1f KB/s\n", block->BlockNumber,
            edit_uint64_with_commas(VolBytes, ec1), (float)kbs);
 
-   if (strcmp(jcr->VolumeName, "TestVolume2") == 0) {
+   if (strcmp(dcr->VolumeName, "TestVolume2") == 0) {
       end_of_tape = 1;
-      return 0;
+      return false;
    }
 
    free_vol_list(jcr);
@@ -2595,198 +2697,21 @@ static int my_mount_next_read_volume(JCR *jcr, DEVICE *dev, DEV_BLOCK *block)
    jcr->bsr = NULL;
    create_vol_list(jcr);
    close_dev(dev);
-   dev->state &= ~ST_READ; 
-   if (!acquire_device_for_read(jcr)) {
-      Pmsg2(0, "Cannot open Dev=%s, Vol=%s\n", dev_name(dev), jcr->VolumeName);
-      return 0;
+   dev->clear_read();
+   if (!acquire_device_for_read(dcr)) {
+      Pmsg2(0, "Cannot open Dev=%s, Vol=%s\n", dev->print_name(), dcr->VolumeName);
+      return false;
    }
-   return 1;                      /* next volume mounted */
+   return true;                   /* next volume mounted */
 }
 
-static void set_volume_name(char *VolName, int volnum) 
+static void set_volume_name(const char *VolName, int volnum)
 {
    DCR *dcr = jcr->dcr;
    VolumeName = VolName;
    vol_num = volnum;
-   pm_strcpy(&jcr->VolumeName, VolName);
    bstrncpy(dev->VolCatInfo.VolCatName, VolName, sizeof(dev->VolCatInfo.VolCatName));
-   bstrncpy(jcr->VolCatInfo.VolCatName, VolName, sizeof(jcr->VolCatInfo.VolCatName));
    bstrncpy(dcr->VolCatInfo.VolCatName, VolName, sizeof(dcr->VolCatInfo.VolCatName));
    bstrncpy(dcr->VolumeName, VolName, sizeof(dcr->VolumeName));
-   jcr->VolCatInfo.Slot = volnum;
    dcr->VolCatInfo.Slot = volnum;
 }
-
-/*
- * Edit codes into ChangerCommand
- *  %% = %
- *  %a = archive device name
- *  %c = changer device name
- *  %f = Client's name
- *  %j = Job name
- *  %o = command
- *  %s = Slot base 0
- *  %S = Slot base 1
- *  %v = Volume name
- *
- *
- *  omsg = edited output message
- *  imsg = input string containing edit codes (%x)
- *  cmd = command string (load, unload, ...) 
- *
- */
-static char *edit_device_codes(JCR *jcr, char *omsg, char *imsg, char *cmd) 
-{
-   char *p;
-   const char *str;
-   char add[20];
-
-   *omsg = 0;
-   Dmsg1(400, "edit_device_codes: %s\n", imsg);
-   for (p=imsg; *p; p++) {
-      if (*p == '%') {
-        switch (*++p) {
-         case '%':
-            str = "%";
-           break;
-         case 'a':
-           str = dev_name(jcr->device->dev);
-           break;
-         case 'c':
-           str = NPRT(jcr->device->changer_name);
-           break;
-         case 'o':
-           str = NPRT(cmd);
-           break;
-         case 's':
-            sprintf(add, "%d", jcr->VolCatInfo.Slot - 1);
-           str = add;
-           break;
-         case 'S':
-            sprintf(add, "%d", jcr->VolCatInfo.Slot);
-           str = add;
-           break;
-         case 'j':                    /* Job name */
-           str = jcr->Job;
-           break;
-         case 'v':
-           str = NPRT(jcr->VolumeName);
-           break;
-         case 'f':
-           str = NPRT(jcr->client_name);
-           break;
-
-        default:
-            add[0] = '%';
-           add[1] = *p;
-           add[2] = 0;
-           str = add;
-           break;
-        }
-      } else {
-        add[0] = *p;
-        add[1] = 0;
-        str = add;
-      }
-      Dmsg1(400, "add_str %s\n", str);
-      pm_strcat(&omsg, (char *)str);
-      Dmsg1(400, "omsg=%s\n", omsg);
-   }
-   return omsg;
-}
-
-#ifdef xxxx_needed
-/* 
- * We are called here from "unfill" for each record on the tape.
- */
-static int record_cb(JCR *jcr, DEVICE *dev, DEV_BLOCK *block, DEV_RECORD *rec)
-{
-   SESSION_LABEL label;
-
-   if (stop > 1 && !dumped) {        /* on second tape */
-      dumped = 1;
-      if (verbose) {
-         dump_block(block, "First block on second tape");
-      }
-      Pmsg4(-1, "Blk: FileIndex=%d: block=%u size=%d vol=%s\n", 
-          rec->FileIndex, block->BlockNumber, block->block_len, dev->VolHdr.VolName);
-      Pmsg6(-1, "   Rec: VId=%d VT=%d FI=%s Strm=%s len=%d state=%x\n",
-          rec->VolSessionId, rec->VolSessionTime, 
-          FI_to_ascii(rec->FileIndex), stream_to_ascii(rec->Stream, rec->FileIndex),
-          rec->data_len, rec->state);
-   }
-   if (rec->FileIndex < 0) {
-      if (verbose > 1) {
-        dump_label_record(dev, rec, 1);
-      }
-      switch (rec->FileIndex) {
-      case PRE_LABEL:
-         Pmsg0(-1, "Volume is prelabeled. This tape cannot be scanned.\n");
-        return 1;;
-      case VOL_LABEL:
-        unser_volume_label(dev, rec);
-         Pmsg3(-1, "VOL_LABEL: block=%u size=%d vol=%s\n", block->BlockNumber, 
-           block->block_len, dev->VolHdr.VolName);
-        stop++;
-        break;
-      case SOS_LABEL:
-        unser_session_label(&label, rec);
-         Pmsg1(-1, "SOS_LABEL: JobId=%u\n", label.JobId);
-        break;
-      case EOS_LABEL:
-        unser_session_label(&label, rec);
-         Pmsg2(-1, "EOS_LABEL: block=%u JobId=%u\n", block->BlockNumber, 
-           label.JobId);
-        break;
-      case EOM_LABEL:
-         Pmsg0(-1, "EOM_LABEL:\n");
-        break;
-      case EOT_LABEL:             /* end of all tapes */
-        char ec1[50];
-
-        if (LastBlock != block->BlockNumber) {
-           VolBytes += block->block_len;
-        }
-        LastBlock = block->BlockNumber;
-        now = time(NULL);
-        now -= jcr->run_time;
-        if (now <= 0) {
-           now = 1;
-        }
-        kbs = (double)VolBytes / (1000 * now);
-         Pmsg3(000, "Read block=%u, VolBytes=%s rate=%.1f KB/s\n", block->BlockNumber,
-                 edit_uint64_with_commas(VolBytes, ec1), (float)kbs);
-
-         Pmsg0(000, "End of all tapes.\n");
-
-        break;
-      default:
-        break;
-      }
-      return 1;
-   }
-   if (++file_index != rec->FileIndex) {
-      Pmsg3(000, "Incorrect FileIndex in Block %u. Got %d, expected %d.\n", 
-        block->BlockNumber, rec->FileIndex, file_index);
-   }
-   if (LastBlock != block->BlockNumber) {
-      VolBytes += block->block_len;
-   }
-   if ((block->BlockNumber != LastBlock) && (block->BlockNumber % 50000) == 0) {
-      char ec1[50];
-      now = time(NULL);
-      now -= jcr->run_time;
-      if (now <= 0) {
-        now = 1;
-      }
-      kbs = (double)VolBytes / (1000 * now);
-      Pmsg3(000, "Read block=%u, VolBytes=%s rate=%.1f KB/s\n", block->BlockNumber,
-              edit_uint64_with_commas(VolBytes, ec1), (float)kbs);
-   }
-   LastBlock = block->BlockNumber;
-   if (end_of_tape) {
-      Pmsg1(000, "End of all blocks. Block=%u\n", block->BlockNumber);
-   }
-   return 1;
-}
-#endif