]> git.sur5r.net Git - bacula/bacula/blobdiff - bacula/src/stored/mount.c
- Lock autochanger script when running.
[bacula/bacula] / bacula / src / stored / mount.c
index f03f9e68925e9f777c7e3be8ab284d5e71799f35..27d1ecebe38b93618addf85e6e4a2321f50c131c 100644 (file)
@@ -4,11 +4,11 @@
  *    writing.
  *
  *   Kern Sibbald, August MMII
- *                           
+ *
  *   Version $Id$
  */
 /*
-   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
 #include "bacula.h"                   /* pull in global headers */
 #include "stored.h"                   /* pull in Storage Deamon headers */
 
-static bool rewrite_volume_label(DCR *dcr, DEV_BLOCK *block, bool recycle);
-
 
 /*
- * If release is set, we rewind the current volume, 
- * which we no longer want, and ask the user (console) 
+ * If release is set, we rewind the current volume,
+ * which we no longer want, and ask the user (console)
  * to mount the next volume.
  *
  *  Continue trying until we get it, and then ensure
@@ -45,19 +43,20 @@ static bool rewrite_volume_label(DCR *dcr, DEV_BLOCK *block, bool recycle);
  *  impossible to get the requested Volume.
  *
  */
-bool mount_next_write_volume(DCR *dcr, DEV_BLOCK *block, bool release)
+bool mount_next_write_volume(DCR *dcr, bool release)
 {
    int retry = 0;
    bool ask = false, recycle, autochanger;
    int vol_label_status;
    DEVICE *dev = dcr->dev;
    JCR *jcr = dcr->jcr;
+   DEV_BLOCK *block = dcr->block;
 
    Dmsg0(100, "Enter mount_next_volume()\n");
 
    init_dev_wait_timers(dev);
 
-   /* 
+   /*
     * Attempt to mount the next volume. If something non-fatal goes
     *  wrong, we come back here to re-try (new op messages, re-read
     *  Volume, ...)
@@ -66,9 +65,9 @@ mount_next_vol:
    /* Ignore retry if this is poll request */
    if (!dev->poll && retry++ > 4) {
       /* Last ditch effort before giving up, force operator to respond */
-      jcr->VolCatInfo.Slot = 0;
+      dcr->VolCatInfo.Slot = 0;
       if (!dir_ask_sysop_to_mount_volume(dcr)) {
-         Jmsg(jcr, M_FATAL, 0, _("Too many errors trying to mount device %s.\n"), 
+         Jmsg(jcr, M_FATAL, 0, _("Too many errors trying to mount device %s.\n"),
              dev_name(dev));
         return false;
       }
@@ -80,13 +79,13 @@ mount_next_vol:
    recycle = false;
    if (release) {
       Dmsg0(100, "mount_next_volume release=1\n");
-      release_volume(jcr, dev);
+      release_volume(dcr);
       ask = true;                    /* ask operator to mount tape */
    }
 
-   /* 
-    * Get Director's idea of what tape we should have mounted. 
-    *   in jcr->VolCatInfo
+   /*
+    * Get Director's idea of what tape we should have mounted.
+    *   in dcr->VolCatInfo
     */
    Dmsg0(200, "Before dir_find_next_appendable_volume.\n");
    while (!dir_find_next_appendable_volume(dcr)) {
@@ -99,14 +98,16 @@ mount_next_vol:
    if (job_canceled(jcr)) {
       return false;
    }
-   Dmsg2(100, "After find_next_append. Vol=%s Slot=%d\n",
-        jcr->VolCatInfo.VolCatName, jcr->VolCatInfo.Slot);
+   Dmsg3(100, "After find_next_append. Vol=%s Slot=%d Parts=%d\n",
+        dcr->VolCatInfo.VolCatName, dcr->VolCatInfo.Slot, dcr->VolCatInfo.VolCatParts);
+   
+   dev->num_parts = dcr->VolCatInfo.VolCatParts;
 
-   /* 
+   /*
     * Get next volume and ready it for append
     * This code ensures that the device is ready for
     * writing. We start from the assumption that there
-    * may not be a tape mounted. 
+    * may not be a tape mounted.
     *
     * If the device is a file, we create the output
     * file. If it is a tape, we check the volume name
@@ -115,19 +116,17 @@ mount_next_vol:
     * It assumes that the device is not already in use!
     *
     */
-   dev->state &= ~(ST_APPEND|ST_READ|ST_EOT|ST_WEOT|ST_EOF);
-
    if (autoload_device(dcr, 1, NULL) > 0) {
       autochanger = true;
       ask = false;
    } else {
       autochanger = false;
-      jcr->VolCatInfo.Slot = 0;
+      dcr->VolCatInfo.Slot = 0;
    }
    Dmsg1(100, "autoload_dev returns %d\n", autochanger);
    /*
     * If we autochanged to correct Volume or (we have not just
-    *  released the Volume AND we can automount) we go ahead 
+    *  released the Volume AND we can automount) we go ahead
     *  and read the label. If there is no tape in the drive,
     *  we will err, recurse and ask the operator the next time.
     */
@@ -148,50 +147,65 @@ mount_next_vol:
    if (job_canceled(jcr)) {
       return false;
    }
-   Dmsg1(100, "want vol=%s\n", jcr->VolumeName);
+   Dmsg1(100, "want vol=%s\n", dcr->VolumeName);
 
    if (dev->poll && dev_cap(dev, CAP_CLOSEONPOLL)) {
       force_close_dev(dev);
    }
 
    /* Ensure the device is open */
-   if (!open_device(jcr, dev)) {
-      if (dev->poll) {
-        goto mount_next_vol;
-      } else {
-        return false;
+   /* If we have a dvd that requires mount, we first want to guess
+    * which Volume is loaded, so we continue (if the wrong device is
+    * loaded, open_device just below would fail. 
+    */
+   if (!dev->is_dvd()) {
+      if (!open_device(dcr)) {
+        if (dev->poll) {
+           goto mount_next_vol;
+        } else {
+           return false;
+        }
       }
+   } else {
+      /*
+       * Just copy the VolCatName in the device resource 
+       *   (usually done by open_dev).
+       * It is necessary so we can open the real files later.  
+       */
+      bstrncpy(dev->VolCatInfo.VolCatName, dcr->VolCatInfo.VolCatName, sizeof(dev->VolCatInfo.VolCatName));
    }
 
    /*
     * Now make sure we have the right tape mounted
     */
 read_volume:
-   /* 
+   /*
     * If we are writing to a stream device, ASSUME the volume label
     *  is correct.
     */
    if (dev_cap(dev, CAP_STREAM)) {
       vol_label_status = VOL_OK;
-      create_volume_label(dev, jcr->VolumeName, "Default");
+      create_volume_label(dev, dcr->VolumeName, "Default");
       dev->VolHdr.LabelType = PRE_LABEL;
+   } else if (dev->is_dvd()) {
+      vol_label_status = read_dev_volume_label_guess(dcr, 1);
    } else {
-      vol_label_status = read_dev_volume_label(jcr->dcr, block);
+      vol_label_status = read_dev_volume_label(dcr);
    }
    if (job_canceled(jcr)) {
       return false;
    }
 
-   Dmsg2(100, "dirVol=%s dirStat=%s\n", jcr->VolumeName,
-      jcr->VolCatInfo.VolCatStatus);
+   Dmsg2(100, "dirVol=%s dirStat=%s\n", dcr->VolumeName,
+      dcr->VolCatInfo.VolCatStatus);
    /*
     * At this point, dev->VolCatInfo has what is in the drive, if anything,
-    *         and   jcr->VolCatInfo has what the Director wants.
+    *         and   dcr->VolCatInfo has what the Director wants.
     */
    switch (vol_label_status) {
    case VOL_OK:
-      Dmsg1(100, "Vol OK name=%s\n", jcr->VolumeName);
-      memcpy(&dev->VolCatInfo, &jcr->VolCatInfo, sizeof(jcr->VolCatInfo));
+      Dmsg1(100, "Vol OK name=%s\n", dcr->VolumeName);
+      memcpy(&dev->VolCatInfo, &dcr->VolCatInfo, sizeof(dev->VolCatInfo));
       recycle = strcmp(dev->VolCatInfo.VolCatStatus, "Recycle") == 0;
       break;                   /* got a Volume */
    case VOL_NAME_ERROR:
@@ -200,28 +214,28 @@ read_volume:
       /* If not removable, Volume is broken */
       if (!dev_cap(dev, CAP_REM)) {
          Jmsg(jcr, M_WARNING, 0, _("Volume \"%s\" not on device %s.\n"),
-           jcr->VolumeName, dev_name(dev));
-        mark_volume_in_error(jcr, dev);
+           dcr->VolumeName, dev_name(dev));
+        mark_volume_in_error(dcr);
         goto mount_next_vol;
       }
-        
-      Dmsg1(100, "Vol NAME Error Name=%s\n", jcr->VolumeName);
+
+      Dmsg1(100, "Vol NAME Error Name=%s\n", dcr->VolumeName);
       /* If polling and got a previous bad name, ignore it */
       if (dev->poll && strcmp(dev->BadVolName, dev->VolHdr.VolName) == 0) {
         ask = true;
-         Dmsg1(200, "Vol Name error supress due to poll. Name=%s\n", 
-           jcr->VolumeName);
+         Dmsg1(200, "Vol Name error supress due to poll. Name=%s\n",
+           dcr->VolumeName);
         goto mount_next_vol;
       }
-      /* 
+      /*
        * OK, we got a different volume mounted. First save the
        *  requested Volume info (jcr) structure, then query if
        *  this volume is really OK. If not, put back the desired
        *  volume name and continue.
        */
-      memcpy(&VolCatInfo, &jcr->VolCatInfo, sizeof(VolCatInfo));
+      memcpy(&VolCatInfo, &dcr->VolCatInfo, sizeof(VolCatInfo));
       /* Check if this is a valid Volume in the pool */
-      pm_strcpy(&jcr->VolumeName, dev->VolHdr.VolName);                        
+      bstrncpy(dcr->VolumeName, dev->VolHdr.VolName, sizeof(dcr->VolumeName));
       if (!dir_get_volume_info(dcr, GET_VOL_INFO_FOR_WRITE)) {
         bstrncpy(dev->BadVolName, dev->VolHdr.VolName, sizeof(dev->BadVolName));
          Jmsg(jcr, M_WARNING, 0, _("Director wanted Volume \"%s\".\n"
@@ -230,12 +244,12 @@ read_volume:
             VolCatInfo.VolCatName, dev->VolHdr.VolName,
             jcr->dir_bsock->msg);
         /* Restore desired volume name, note device info out of sync */
-        memcpy(&jcr->VolCatInfo, &VolCatInfo, sizeof(jcr->VolCatInfo));
+        memcpy(&dcr->VolCatInfo, &VolCatInfo, sizeof(dcr->VolCatInfo));
         ask = true;
         goto mount_next_vol;
       }
-      Dmsg1(100, "want new name=%s\n", jcr->VolumeName);
-      memcpy(&dev->VolCatInfo, &jcr->VolCatInfo, sizeof(dev->VolCatInfo));
+      Dmsg1(100, "want new name=%s\n", dcr->VolumeName);
+      memcpy(&dev->VolCatInfo, &dcr->VolCatInfo, sizeof(dev->VolCatInfo));
       recycle = strcmp(dev->VolCatInfo.VolCatStatus, "Recycle") == 0;
       break;               /* got a Volume */
    /*
@@ -243,40 +257,45 @@ read_volume:
     */
    case VOL_NO_LABEL:
    case VOL_IO_ERROR:
-      /* 
+      /*
        * If permitted, we label the device, make sure we can do
-       *   it by checking that the VolCatBytes is zero => not labeled, 
+       *   it by checking that the VolCatBytes is zero => not labeled,
        *   once the Volume is labeled we don't want to label another
        *   blank tape with the same name.  For disk, we go ahead and
        *   label it anyway, because the OS insures that there is only
        *   one Volume with that name.
-       * As noted above, at this point jcr->VolCatInfo has what
+       * As noted above, at this point dcr->VolCatInfo has what
        *   the Director wants and dev->VolCatInfo has info on the
        *   previous tape (or nothing).
        */
-      if (dev_cap(dev, CAP_LABEL) && (jcr->VolCatInfo.VolCatBytes == 0 ||
-           (!dev_is_tape(dev) && strcmp(jcr->VolCatInfo.VolCatStatus, 
+      if (dev_cap(dev, CAP_LABEL) && (dcr->VolCatInfo.VolCatBytes == 0 ||
+           (!dev->is_tape() && strcmp(dcr->VolCatInfo.VolCatStatus,
                                    "Recycle") == 0))) {
          Dmsg0(100, "Create volume label\n");
         /* Create a new Volume label and write it to the device */
-        if (!write_new_volume_label_to_dev(jcr->dcr, jcr->VolumeName,
-               jcr->pool_name)) {
+        if (!write_new_volume_label_to_dev(dcr, dcr->VolumeName,
+               dcr->pool_name)) {
             Dmsg0(100, "!write_vol_label\n");
            goto mount_next_vol;
         }
          Dmsg0(100, "dir_update_vol_info. Set Append\n");
          /* Copy Director's info into the device info */
-        memcpy(&dev->VolCatInfo, &jcr->VolCatInfo, sizeof(dev->VolCatInfo));
-        dir_update_volume_info(dcr, true);  /* indicate tape labeled */
+        memcpy(&dev->VolCatInfo, &dcr->VolCatInfo, sizeof(dev->VolCatInfo));
+        if (!dir_update_volume_info(dcr, true)) {  /* indicate tape labeled */
+           return false;
+        }
          Jmsg(jcr, M_INFO, 0, _("Labeled new Volume \"%s\" on device %s.\n"),
-           jcr->VolumeName, dev_name(dev));
+           dcr->VolumeName, dev_name(dev));
         goto read_volume;      /* read label we just wrote */
-      } 
+      }
       /* If not removable, Volume is broken */
       if (!dev_cap(dev, CAP_REM)) {
          Jmsg(jcr, M_WARNING, 0, _("Volume \"%s\" not on device %s.\n"),
-           jcr->VolumeName, dev_name(dev));
-        mark_volume_in_error(jcr, dev);
+           dcr->VolumeName, dev_name(dev));
+        mark_volume_in_error(dcr);
+        if (autochanger) {
+           mark_volume_not_inchanger(dcr);
+        }
         goto mount_next_vol;
       }
       /* NOTE! Fall-through wanted. */
@@ -284,15 +303,19 @@ read_volume:
    default:
       /* Send error message */
       if (!dev->poll) {
-         Jmsg(jcr, M_WARNING, 0, "%s", jcr->errmsg);                         
+         Jmsg(jcr, M_WARNING, 0, "%s", jcr->errmsg);
       } else {
          Dmsg1(200, "Msg suppressed by poll: %s\n", jcr->errmsg);
       }
       ask = true;
+      /* Needed, so the medium can be changed */
+      if (dev_cap(dev, CAP_REQMOUNT)) {
+        close_dev(dev);  
+      }
       goto mount_next_vol;
    }
 
-   /* 
+   /*
     * See if we have a fresh tape or a tape with data.
     *
     * Note, if the LabelType is PRE_LABEL, it was labeled
@@ -306,7 +329,7 @@ read_volume:
     *  If the tape is marked as Recycle, we rewrite the label.
     */
    if (dev->VolHdr.LabelType == PRE_LABEL || recycle) {
-      if (!rewrite_volume_label(jcr->dcr, block, recycle)) {
+      if (!rewrite_volume_label(dcr, recycle)) {
         goto mount_next_vol;
       }
    } else {
@@ -317,11 +340,11 @@ read_volume:
        */
       Dmsg0(200, "Device previously written, moving to end of data\n");
       Jmsg(jcr, M_INFO, 0, _("Volume \"%s\" previously written, moving to end of data.\n"),
-        jcr->VolumeName);
+        dcr->VolumeName);
       if (!eod_dev(dev)) {
          Jmsg(jcr, M_ERROR, 0, _("Unable to position to end of data on device \"%s\". ERR=%s\n"),
            dev_name(dev), strerror_dev(dev));
-        mark_volume_in_error(jcr, dev);
+        mark_volume_in_error(dcr);
         goto mount_next_vol;
       }
       /* *****FIXME**** we should do some checking for files too */
@@ -331,127 +354,62 @@ read_volume:
          * that the database says we should be.
          */
         if (dev->VolCatInfo.VolCatFiles == dev_file(dev)) {
-            Jmsg(jcr, M_INFO, 0, _("Ready to append to end of Volume \"%s\" at file=%d.\n"), 
-                jcr->VolumeName, dev_file(dev));
+            Jmsg(jcr, M_INFO, 0, _("Ready to append to end of Volume \"%s\" at file=%d.\n"),
+                dcr->VolumeName, dev_file(dev));
         } else {
-            Jmsg(jcr, M_ERROR, 0, _("I canot write on Volume \"%s\" because:\n\
-The number of files mismatch! Volume=%u Catalog=%u\n"), 
-                jcr->VolumeName, dev_file(dev), dev->VolCatInfo.VolCatFiles);
-           mark_volume_in_error(jcr, dev);
+            Jmsg(jcr, M_ERROR, 0, _("I canot write on Volume \"%s\" because:\n"
+"The number of files mismatch! Volume=%u Catalog=%u\n"),
+                dcr->VolumeName, dev_file(dev), dev->VolCatInfo.VolCatFiles);
+           mark_volume_in_error(dcr);
            goto mount_next_vol;
         }
       }
       dev->VolCatInfo.VolCatMounts++;     /* Update mounts */
       Dmsg1(100, "update volinfo mounts=%d\n", dev->VolCatInfo.VolCatMounts);
-      dir_update_volume_info(dcr, false);
+      if (!dir_update_volume_info(dcr, false)) {
+        return false;
+      }
       /* Return an empty block */
       empty_block(block);            /* we used it for reading so set for write */
    }
    dev->state |= ST_APPEND;
-   Dmsg0(100, "Normal return from read_dev_for_append\n");
+   Dmsg0(100, "set APPEND, normal return from read_dev_for_append\n");
    return true;
 }
 
-/*
- * Write a volume label. 
- *  Returns: true if OK
- *          false if unable to write it
- */
-static bool rewrite_volume_label(DCR *dcr, DEV_BLOCK *block, bool recycle)
-{
-   DEVICE *dev = dcr->dev;
-   JCR *jcr = dcr->jcr;
-
-   Dmsg1(190, "ready_for_append found freshly labeled volume. dev=%x\n", dev);
-   dev->VolHdr.LabelType = VOL_LABEL; /* set Volume label */
-   if (!write_volume_label_to_block(dcr, block)) {
-      return false;
-   }
-   /*
-    * If we are not dealing with a streaming device,
-    *  write the block now to ensure we have write permission.
-    *  It is better to find out now rather than later.
-    */
-   if (!dev_cap(dev, CAP_STREAM)) {
-      if (!rewind_dev(dev)) {
-         Jmsg2(jcr, M_WARNING, 0, _("Rewind error on device \"%s\". ERR=%s\n"), 
-              dev_name(dev), strerror_dev(dev));
-      }
-      if (recycle) {
-        if (!truncate_dev(dev)) {
-            Jmsg2(jcr, M_WARNING, 0, _("Truncate error on device \"%s\". ERR=%s\n"), 
-                 dev_name(dev), strerror_dev(dev));
-        }
-      }
-      /* Attempt write to check write permission */
-      if (!write_block_to_dev(dcr, block)) {
-         Jmsg2(jcr, M_ERROR, 0, _("Unable to write device \"%s\". ERR=%s\n"),
-           dev_name(dev), strerror_dev(dev));
-        return false;
-      }
-   }
-   /* Set or reset Volume statistics */
-   dev->VolCatInfo.VolCatJobs = 0;
-   dev->VolCatInfo.VolCatFiles = 0;
-   dev->VolCatInfo.VolCatBytes = 1;
-   dev->VolCatInfo.VolCatErrors = 0;
-   dev->VolCatInfo.VolCatBlocks = 0;
-   dev->VolCatInfo.VolCatRBytes = 0;
-   if (recycle) {
-      dev->VolCatInfo.VolCatMounts++;  
-      dev->VolCatInfo.VolCatRecycles++;
-   } else {
-      dev->VolCatInfo.VolCatMounts = 1;
-      dev->VolCatInfo.VolCatRecycles = 0;
-      dev->VolCatInfo.VolCatWrites = 1;
-      dev->VolCatInfo.VolCatReads = 1;
-   }
-   Dmsg0(100, "dir_update_vol_info. Set Append\n");
-   bstrncpy(dev->VolCatInfo.VolCatStatus, "Append", sizeof(dev->VolCatInfo.VolCatStatus));
-   dir_update_volume_info(dcr, true);  /* indicate doing relabel */
-   if (recycle) {
-      Jmsg(jcr, M_INFO, 0, _("Recycled volume \"%s\" on device \"%s\", all previous data lost.\n"),
-        jcr->VolumeName, dev_name(dev));
-   } else {
-      Jmsg(jcr, M_INFO, 0, _("Wrote label to prelabeled Volume \"%s\" on device \"%s\"\n"),
-        jcr->VolumeName, dev_name(dev));
-   }
-   /*
-    * End writing real Volume label (from pre-labeled tape), or recycling
-    *  the volume.
-    */
-    return true;
-}
 
 
 /*
- * Mark volume in error in catalog 
+ * Mark volume in error in catalog
  */
-void mark_volume_in_error(JCR *jcr, DEVICE *dev)
+void mark_volume_in_error(DCR *dcr)
 {
-   Jmsg(jcr, M_INFO, 0, _("Marking Volume \"%s\" in Error in Catalog.\n"),
-       jcr->VolumeName);
+   DEVICE *dev = dcr->dev;
+   Jmsg(dcr->jcr, M_INFO, 0, _("Marking Volume \"%s\" in Error in Catalog.\n"),
+       dcr->VolumeName);
    bstrncpy(dev->VolCatInfo.VolCatStatus, "Error", sizeof(dev->VolCatInfo.VolCatStatus));
    Dmsg0(100, "dir_update_vol_info. Set Error.\n");
-   dir_update_volume_info(jcr->dcr, false);
+   dir_update_volume_info(dcr, false);
 }
 
-/* 
+/*
  * If we are reading, we come here at the end of the tape
  *  and see if there are more volumes to be mounted.
  */
-bool mount_next_read_volume(JCR *jcr, DEVICE *dev, DEV_BLOCK *block)
+bool mount_next_read_volume(DCR *dcr)
 {
+   DEVICE *dev = dcr->dev;
+   JCR *jcr = dcr->jcr;
    Dmsg2(90, "NumVolumes=%d CurVolume=%d\n", jcr->NumVolumes, jcr->CurVolume);
    /*
     * End Of Tape -- mount next Volume (if another specified)
     */
    if (jcr->NumVolumes > 1 && jcr->CurVolume < jcr->NumVolumes) {
       close_dev(dev);
-      dev->state &= ~ST_READ; 
-      if (!acquire_device_for_read(jcr)) {
+      dev->state &= ~ST_READ;
+      if (!acquire_device_for_read(jcr, dev)) {
          Jmsg2(jcr, M_FATAL, 0, "Cannot open Dev=%s, Vol=%s\n", dev_name(dev),
-              jcr->VolumeName);
+              dcr->VolumeName);
         return false;
       }
       return true;                   /* next volume mounted */
@@ -464,31 +422,35 @@ bool mount_next_read_volume(JCR *jcr, DEVICE *dev, DEV_BLOCK *block)
  * Either because we are going to hang a new volume, or because
  *  of explicit user request, we release the current volume.
  */
-void release_volume(JCR *jcr, DEVICE *dev)
+void release_volume(DCR *dcr)
 {
-   if (jcr->dcr->WroteVol) {
+   JCR *jcr = dcr->jcr;
+   DEVICE *dev = dcr->dev;
+   if (dcr->WroteVol) {
       Jmsg0(jcr, M_ERROR, 0, "Hey!!!!! WroteVol non-zero !!!!!\n");
+      Dmsg0(190, "Hey!!!!! WroteVol non-zero !!!!!\n");
    }
-   /* 
-    * First erase all memory of the current volume   
+   /*
+    * First erase all memory of the current volume
     */
    dev->block_num = dev->file = 0;
    dev->EndBlock = dev->EndFile = 0;
    memset(&dev->VolCatInfo, 0, sizeof(dev->VolCatInfo));
-   memset(&jcr->VolCatInfo, 0, sizeof(jcr->VolCatInfo));
+   memset(&dcr->VolCatInfo, 0, sizeof(dcr->VolCatInfo));
    memset(&dev->VolHdr, 0, sizeof(dev->VolHdr));
    /* Force re-read of label */
    dev->state &= ~(ST_LABEL|ST_READ|ST_APPEND);
-   jcr->VolumeName[0] = 0;
+   dev->label_type = B_BACULA_LABEL;
+   dcr->VolumeName[0] = 0;
 
-   if ((dev->state & ST_OPENED) && 
-       (!dev_is_tape(dev) || !dev_cap(dev, CAP_ALWAYSOPEN))) {
+   if (dev->is_open() && (!dev->is_tape() || !dev_cap(dev, CAP_ALWAYSOPEN))) {
       offline_or_rewind_dev(dev);
       close_dev(dev);
    }
 
    /* If we have not closed the device, then at least rewind the tape */
-   if (dev->state & ST_OPENED) {
+   if (dev->is_open()) {
       offline_or_rewind_dev(dev);
    }
+   Dmsg0(190, "===== release_volume ---");
 }