]> git.sur5r.net Git - bacula/bacula/blobdiff - bacula/src/stored/acquire.c
This commit was manufactured by cvs2svn to create tag
[bacula/bacula] / bacula / src / stored / acquire.c
index 9e2d4a9fee8c7e674f93284dcd347e3f95a81b34..156c73cf1e0fc5fc6bd80c1915a220d339b964fe 100644 (file)
  *   Version $Id$
  */
 /*
-   Copyright (C) 2002-2005 Kern Sibbald
+   Copyright (C) 2002-2006 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
-   published by the Free Software Foundation; either version 2 of
-   the License, or (at your option) any later version.
+   modify it under the terms of the GNU General Public License
+   version 2 as amended with additional clauses defined in the
+   file LICENSE in the main source directory.
 
    This program is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
-   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
-   General Public License for more details.
-
-   You should have received a copy of the GNU General Public
-   License along with this program; if not, write to the Free
-   Software Foundation, Inc., 59 Temple Place - Suite 330, Boston,
-   MA 02111-1307, USA.
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the 
+   the file LICENSE for additional details.
 
  */
 
 #include "bacula.h"                   /* pull in global headers */
 #include "stored.h"                   /* pull in Storage Deamon headers */
 
-static int can_reserve_drive(DCR *dcr);
-
-/*
- * Create a new Device Control Record and attach
- *   it to the device (if this is a real job).
- */
-DCR *new_dcr(JCR *jcr, DEVICE *dev)
-{
-   if (jcr && jcr->dcr) {
-      return jcr->dcr;
-   }
-   DCR *dcr = (DCR *)malloc(sizeof(DCR));
-   memset(dcr, 0, sizeof(DCR));
-   if (jcr) {
-      jcr->dcr = dcr;
-   }
-   dcr->jcr = jcr;
-   dcr->dev = dev;
-   if (dev) {
-      dcr->device = dev->device;
-   }
-   dcr->block = new_block(dev);
-   dcr->rec = new_record();
-   dcr->spool_fd = -1;
-   dcr->max_spool_size = dev->device->max_spool_size;
-   /* Attach this dcr only if dev is initialized */
-   if (dev->fd != 0 && jcr && jcr->JobType != JT_SYSTEM) {
-      dev->attached_dcrs->append(dcr);  /* attach dcr to device */
-//    jcr->dcrs->append(dcr);         /* put dcr in list for Job */
-   }
-   return dcr;
-}
-
-/*
- * Search the dcrs list for the given dcr. If it is found,
- *  as it should be, then remove it. Also zap the jcr pointer
- *  to the dcr if it is the same one.
- */
-#ifdef needed
-static void remove_dcr_from_dcrs(DCR *dcr)
-{
-   JCR *jcr = dcr->jcr;
-   if (jcr->dcrs) {
-      int i = 0;
-      DCR *ldcr;
-      int num = jcr->dcrs->size();
-      for (i=0; i < num; i++) {
-         ldcr = (DCR *)jcr->dcrs->get(i);
-         if (ldcr == dcr) {
-            jcr->dcrs->remove(i);
-            if (jcr->dcr == dcr) {
-               jcr->dcr = NULL;
-            }
-         }
-      }
-   }
-}
-#endif
-
-/*
- * Free up all aspects of the given dcr -- i.e. dechain it,
- *  release allocated memory, zap pointers, ...
- */
-void free_dcr(DCR *dcr)
-{
-   JCR *jcr = dcr->jcr;
-   DEVICE *dev = dcr->dev;
-
-   /*
-    * If we reserved the device, we must decrement the
-    *  number of writers.
-    */
-   if (dcr->reserved_device) {
-      lock_device(dev);
-      dev->num_writers--;
-      if (dev->num_writers < 0) {
-         Jmsg1(dcr->jcr, M_ERROR, 0, _("Hey! num_writers=%d!!!!\n"), dev->num_writers);
-         dev->num_writers = 0;
-         dcr->reserved_device = false;
-      }
-      unlock_device(dev);
-   }
-
-   /* Detach this dcr only if the dev is initialized */
-   if (dev->fd != 0 && jcr && jcr->JobType != JT_SYSTEM) {
-      dev->attached_dcrs->remove(dcr);  /* detach dcr from device */
-//    remove_dcr_from_dcrs(dcr);      /* remove dcr from jcr list */
-   }
-   if (dcr->block) {
-      free_block(dcr->block);
-   }
-   if (dcr->rec) {
-      free_record(dcr->rec);
-   }
-   if (dcr->jcr) {
-      dcr->jcr->dcr = NULL;
-   }
-   free(dcr);
-}
-
-
-/*
- * We "reserve" the drive by setting the ST_READ bit. No one else
- *  should touch the drive until that is cleared.
- *  This allows the DIR to "reserve" the device before actually
- *  starting the job. If the device is not available, the DIR
- *  can wait (to be implemented 1/05).
- */
-bool reserve_device_for_read(DCR *dcr)
-{
-   DEVICE *dev = dcr->dev;
-   JCR *jcr = dcr->jcr;
-   bool first;
-
-   ASSERT(dcr);
-
-   init_device_wait_timers(dcr);
-
-   dev->block(BST_DOING_ACQUIRE);
-
-   Mmsg(jcr->errmsg, _("Device %s is BLOCKED due to user unmount.\n"),
-        dev->print_name());
-   for (first=true; device_is_unmounted(dev); first=false) {
-      dev->unblock();
-      if (!wait_for_device(dcr, jcr->errmsg, first))  {
-         return false;
-      }
-     dev->block(BST_DOING_ACQUIRE);
-   }
-
-   Mmsg2(jcr->errmsg, _("Device %s is busy. Job %d canceled.\n"),
-         dev->print_name(), jcr->JobId);
-   for (first=true; dev->is_busy(); first=false) {
-      dev->unblock();
-      if (!wait_for_device(dcr, jcr->errmsg, first)) {
-         return false;
-      }
-      dev->block(BST_DOING_ACQUIRE);
-   }
-
-   dev->clear_append();
-   dev->set_read();
-   dev->unblock();
-   return true;
-}
-
 
 /*********************************************************************
  * Acquire device for reading. 
@@ -195,6 +44,7 @@ DCR *acquire_device_for_read(DCR *dcr)
    bool try_autochanger = true;
    int i;
    int vol_label_status;
+   int retry = 0;
    
    dev->block(BST_DOING_ACQUIRE);
 
@@ -221,8 +71,7 @@ DCR *acquire_device_for_read(DCR *dcr)
 
    init_device_wait_timers(dcr);
 
-   tape_previously_mounted = dev->can_read() ||
-                             dev->can_append() ||
+   tape_previously_mounted = dev->can_read() || dev->can_append() ||
                              dev->is_labeled();
    tape_initially_mounted = tape_previously_mounted;
 
@@ -233,9 +82,11 @@ DCR *acquire_device_for_read(DCR *dcr)
       Jmsg1(jcr, M_WARNING, 0, "%s", jcr->errmsg);
    }
    
-   dev->num_parts = dcr->VolCatInfo.VolCatParts;
-   
-   for (i=0; i<5; i++) {
+   for ( ;; ) {
+      /* If not polling limit retries */
+      if (!dev->poll && retry++ > 10) {
+         break;
+      }
       dev->clear_labeled();              /* force reread of label */
       if (job_canceled(jcr)) {
          Mmsg1(dev->errmsg, _("Job %d canceled.\n"), jcr->JobId);
@@ -246,36 +97,24 @@ DCR *acquire_device_for_read(DCR *dcr)
        * reading. If it is a file, it opens it.
        * If it is a tape, it checks the volume name
        */
-      for ( ; !dev->is_open(); ) {
-         Dmsg1(120, "bstored: open vol=%s\n", dcr->VolumeName);
-         if (open_dev(dev, dcr->VolumeName, OPEN_READ_ONLY) < 0) {
-            if (dev->dev_errno == EIO) {   /* no tape loaded */
-              Jmsg3(jcr, M_WARNING, 0, _("Open device %s Volume \"%s\" failed: ERR=%s\n"),
-                    dev->print_name(), dcr->VolumeName, strerror_dev(dev));
-               goto default_path;
-            }
-            
-            /* If we have a dvd that requires mount, 
-             * we need to try to open the label, so the info can be reported
-             * if a wrong volume has been mounted. */
-            if (dev->is_dvd() && (dcr->VolCatInfo.VolCatParts > 0)) {
-               break;
-            }
-            
-            Jmsg3(jcr, M_FATAL, 0, _("Open device %s Volume \"%s\" failed: ERR=%s\n"),
-                dev->print_name(), dcr->VolumeName, strerror_dev(dev));
-            goto get_out;
+      Dmsg1(100, "bstored: open vol=%s\n", dcr->VolumeName);
+      if (dev->open(dcr, OPEN_READ_ONLY) < 0) {
+         if (dev->dev_errno == EIO) {   /* no tape loaded */
+           Jmsg3(jcr, M_WARNING, 0, _("Read open device %s Volume \"%s\" failed (EIO): ERR=%s\n"),
+                 dev->print_name(), dcr->VolumeName, strerror_dev(dev));
+            goto default_path;
          }
-         Dmsg1(129, "open_dev %s OK\n", dev->print_name());
+         
+         Jmsg3(jcr, M_FATAL, 0, _("Read open device %s Volume \"%s\" failed: ERR=%s\n"),
+             dev->print_name(), dcr->VolumeName, strerror_dev(dev));
+         goto get_out;
       }
+      Dmsg1(100, "opened dev %s OK\n", dev->print_name());
       
-      if (dev->is_dvd()) {
-         vol_label_status = read_dev_volume_label_guess(dcr, 0);
-      } else {
-         vol_label_status = read_dev_volume_label(dcr);
-      }
+      /* Read Volume Label */
       
       Dmsg0(200, "calling read-vol-label\n");
+      vol_label_status = read_dev_volume_label(dcr);
       switch (vol_label_status) {
       case VOL_OK:
          vol_ok = true;
@@ -296,6 +135,12 @@ DCR *acquire_device_for_read(DCR *dcr)
             tape_initially_mounted = false;
             goto default_path;
          }
+         /* If polling and got a previous bad name, ignore it */
+         if (dev->poll && strcmp(dev->BadVolName, dev->VolHdr.VolumeName) == 0) {
+            goto default_path;
+         } else {
+             bstrncpy(dev->BadVolName, dev->VolHdr.VolumeName, sizeof(dev->BadVolName));
+         }
          /* Fall through */
       default:
          Jmsg1(jcr, M_WARNING, 0, "%s", jcr->errmsg);
@@ -304,8 +149,8 @@ default_path:
          
          /* If the device requires mount, close it, so the device can be ejected.
           * FIXME: This should perhaps be done for all devices. */
-         if (dev_cap(dev, CAP_REQMOUNT)) {
-            force_close_dev(dev);
+         if (dev->requires_mount()) {
+            force_close_device(dev);
          }
          
          /* Call autochanger only once unless ask_sysop called */
@@ -344,145 +189,18 @@ default_path:
       dcr->VolumeName, dev->print_name());
 
 get_out:
-   dev->unblock();
-   if (!vol_ok) {
-      free_dcr(dcr);
-      dcr = NULL;
-   }
-   return dcr;
-}
-
-/*
- * We reserve the device for appending by incrementing the 
- *  reserved_device. We do virtually all the same work that
- *  is done in acquire_device_for_append(), but we do
- *  not attempt to mount the device. This routine allows
- *  the DIR to reserve multiple devices before *really* 
- *  starting the job. It also permits the SD to refuse 
- *  certain devices (not up, ...).
- *
- * Note, in reserving a device, if the device is for the
- *  same pool and the same pool type, then it is acceptable.
- *  The Media Type has already been checked. If we are
- *  the first tor reserve the device, we put the pool
- *  name and pool type in the device record.
- */
-bool reserve_device_for_append(DCR *dcr)
-{
-   JCR *jcr = dcr->jcr;
-   DEVICE *dev = dcr->dev;
-   bool ok = false;
-   bool first;
-
-   ASSERT(dcr);
-
-   init_device_wait_timers(dcr);
-
-   dev->block(BST_DOING_ACQUIRE);
-
-   Mmsg1(jcr->errmsg, _("Device %s is busy reading.\n"),
-         dev->print_name());
-   for (first=true; dev->can_read(); first=false) {
-      dev->unblock();
-      if (!wait_for_device(dcr, jcr->errmsg, first)) {
-         return false;
-      }
-      dev->block(BST_DOING_ACQUIRE);
-   }
-
-
-   Mmsg(jcr->errmsg, _("Device %s is BLOCKED due to user unmount.\n"),
-        dev->print_name());
-   for (first=true; device_is_unmounted(dev); first=false) {
-      dev->unblock();      
-      if (!wait_for_device(dcr, jcr->errmsg, first))  {
-         return false;
-      }
-     dev->block(BST_DOING_ACQUIRE);
-   }
-
-   Dmsg1(190, "reserve_append device is %s\n", dev->is_tape()?"tape":"disk");
-
-   for ( ;; ) {
-      switch (can_reserve_drive(dcr)) {
-      case 0:
-         Mmsg1(jcr->errmsg, _("Device %s is busy writing on another Volume.\n"), dev->print_name());
-         dev->unblock();      
-         if (!wait_for_device(dcr, jcr->errmsg, first))  {
-            return false;
-         }
-         dev->block(BST_DOING_ACQUIRE);
-         continue;
-      case -1:
-         goto bail_out;               /* error */
-      default:
-         break;                       /* OK, reserve drive */
-      }
-      break;
+   P(dev->mutex);
+   if (dcr->reserved_device) {
+      dev->reserved_device--;
+      Dmsg2(100, "Dec reserve=%d dev=%s\n", dev->reserved_device, dev->print_name());
+      dcr->reserved_device = false;
    }
-
-
-   dev->reserved_device++;
-   dcr->reserved_device = true;
-   ok = true;
-
-bail_out:
+   V(dev->mutex);
    dev->unblock();
-   return ok;
+   Dmsg1(000, "jcr->dcr=%p\n", jcr->dcr);
+   return dcr;
 }
 
-/*
- * Returns: 1 if drive can be reserved
- *          0 if we should wait
- *         -1 on error
- */
-static int can_reserve_drive(DCR *dcr) 
-{
-   DEVICE *dev = dcr->dev;
-   JCR *jcr = dcr->jcr;
-   /*
-    * First handle the case that the drive is not yet in append mode
-    */
-   if (!dev->can_append() && dev->num_writers == 0) {
-      /* Now check if there are any reservations on the drive */
-      if (dev->reserved_device) {           
-         /* Yes, now check if we want the same Pool and pool type */
-         if (strcmp(dev->pool_name, dcr->pool_name) == 0 &&
-             strcmp(dev->pool_type, dcr->pool_type) == 0) {
-            /* OK, compatible device */
-         } else {
-            /* Drive not suitable for us */
-            return 0;                 /* wait */
-         }
-      } else {
-         /* Device is available but not yet reserved, reserve it for us */
-         bstrncpy(dev->pool_name, dcr->pool_name, sizeof(dev->pool_name));
-         bstrncpy(dev->pool_type, dcr->pool_type, sizeof(dev->pool_type));
-      }
-      return 1;                       /* reserve drive */
-   }
-
-   /*
-    * Now check if the device is in append mode 
-    */
-   if (dev->can_append() || dev->num_writers > 0) {
-      Dmsg0(190, "device already in append.\n");
-      /* Yes, now check if we want the same Pool and pool type */
-      if (strcmp(dev->pool_name, dcr->pool_name) == 0 &&
-          strcmp(dev->pool_type, dcr->pool_type) == 0) {
-         /* OK, compatible device */
-      } else {
-         /* Drive not suitable for us */
-         Jmsg(jcr, M_WARNING, 0, _("Device %s is busy writing on another Volume.\n"), dev->print_name());
-         return 0;                    /* wait */
-      }
-   } else {
-      Pmsg0(000, "Logic error!!!! Should not get here.\n");
-      Jmsg0(jcr, M_FATAL, 0, _("Logic error!!!! Should not get here.\n"));
-      return -1;                      /* error, should not get here */
-   }
-   return 1;                          /* reserve drive */
-}
 
 /*
  * Acquire device for writing. We permit multiple writers.
@@ -504,18 +222,15 @@ DCR *acquire_device_for_append(DCR *dcr)
    init_device_wait_timers(dcr);
 
    dev->block(BST_DOING_ACQUIRE);
-   Dmsg1(190, "acquire_append device is %s\n", dev->is_tape()?"tape":"disk");
-
-   if (dcr->reserved_device) {
-      dev->reserved_device--;
-      dcr->reserved_device = false;
-   }
+   Dmsg1(190, "acquire_append device is %s\n", dev->is_tape()?"tape":
+        (dev->is_dvd()?"DVD":"disk"));
 
    /*
     * With the reservation system, this should not happen
     */
    if (dev->can_read()) {
-      Jmsg(jcr, M_FATAL, 0, _("Device %s is busy reading.\n"), dev->print_name());
+      Jmsg1(jcr, M_FATAL, 0, _("Device %s is busy reading.\n"), dev->print_name());
+      Dmsg1(200, "Device %s is busy reading.\n", dev->print_name());
       goto get_out;
    }
 
@@ -529,14 +244,25 @@ DCR *acquire_device_for_append(DCR *dcr)
        *   OK if next volume matches current volume
        *   otherwise mount desired volume obtained from
        *    dir_find_next_appendable_volume
+       *  dev->VolHdr.VolumeName is what is in the drive
+       *  dcr->VolumeName is what we pass into the routines, or
+       *    get back from the subroutines.
        */
-      bstrncpy(dcr->VolumeName, dev->VolHdr.VolName, sizeof(dcr->VolumeName));
+      bstrncpy(dcr->VolumeName, dev->VolHdr.VolumeName, sizeof(dcr->VolumeName));
       if (!dir_get_volume_info(dcr, GET_VOL_INFO_FOR_WRITE) &&
           !(dir_find_next_appendable_volume(dcr) &&
-            strcmp(dev->VolHdr.VolName, dcr->VolumeName) == 0)) { /* wrong tape mounted */
-         Dmsg0(190, "Wrong tape mounted.\n");
-         if (dev->num_writers != 0 || dev->reserved_device) {
-            Jmsg(jcr, M_FATAL, 0, _("Device %s is busy writing on another Volume.\n"), dev->print_name());
+            strcmp(dev->VolHdr.VolumeName, dcr->VolumeName) == 0)) { /* wrong tape mounted */
+         Dmsg2(190, "Wrong tape mounted: %s. wants:%s\n", dev->VolHdr.VolumeName,
+            dcr->VolumeName);
+         /* Release volume reserved by dir_find_next_appendable_volume() */
+         if (dcr->VolumeName[0]) {
+            free_unused_volume(dcr);
+         }
+         if (dev->num_writers != 0) {
+            Jmsg3(jcr, M_FATAL, 0, _("Wanted Volume \"%s\", but device %s is busy writing on \"%s\" .\n"), 
+                 dcr->VolumeName, dev->print_name(), dev->VolHdr.VolumeName);
+            Dmsg3(200, "Wanted Volume \"%s\", but device %s is busy writing on \"%s\" .\n",  
+                 dcr->VolumeName, dev->print_name(), dev->VolHdr.VolumeName);
             goto get_out;
          }
          /* Wrong tape mounted, release it, then fall through to get correct one */
@@ -554,13 +280,13 @@ DCR *acquire_device_for_append(DCR *dcr)
           if (recycle && dev->num_writers != 0) {
              Jmsg(jcr, M_FATAL, 0, _("Cannot recycle volume \"%s\""
                   " on device %s because it is in use by another job.\n"),
-                  dev->VolHdr.VolName, dev->print_name());
+                  dev->VolHdr.VolumeName, dev->print_name());
              goto get_out;
           }
           if (dev->num_writers == 0) {
              memcpy(&dev->VolCatInfo, &dcr->VolCatInfo, sizeof(dev->VolCatInfo));
           }
-       }
+      }
    } else {
       /* Not already in append mode, so mount the device */
       Dmsg0(190, "Not in append mode, try mount.\n");
@@ -576,6 +302,8 @@ DCR *acquire_device_for_append(DCR *dcr)
             /* Reduce "noise" -- don't print if job canceled */
             Jmsg(jcr, M_FATAL, 0, _("Could not ready device %s for append.\n"),
                dev->print_name());
+            Dmsg1(200, "Could not ready device %s for append.\n", 
+               dev->print_name());
          }
          goto get_out;
       }
@@ -585,18 +313,30 @@ DCR *acquire_device_for_append(DCR *dcr)
    if (jcr->NumVolumes == 0) {
       jcr->NumVolumes = 1;
    }
-   goto ok_out;
+   P(dev->mutex);
+   if (dcr->reserved_device) {
+      dev->reserved_device--;
+      Dmsg1(100, "Dec reserve=%d\n", dev->reserved_device);
+      dcr->reserved_device = false;
+   }
+   V(dev->mutex);
+   dev->unblock();
+   return dcr;
 
 /*
- * If we jump here, it is an error return because
- *  rtn_dev will still be NULL
+ * Error return
  */
 get_out:
+   P(dev->mutex);
+   if (dcr->reserved_device) {
+      dev->reserved_device--;
+      Dmsg1(100, "Dec reserve=%d\n", dev->reserved_device);
+      dcr->reserved_device = false;
+   }
+   V(dev->mutex);
    free_dcr(dcr);
-   dcr = NULL;
-ok_out:
    dev->unblock();
-   return dcr;
+   return NULL;
 }
 
 /*
@@ -616,6 +356,7 @@ bool release_device(DCR *dcr)
    /* if device is reserved, job never started, so release the reserve here */
    if (dcr->reserved_device) {
       dev->reserved_device--;
+      Dmsg1(100, "Dec reserve=%d\n", dev->reserved_device);
       dcr->reserved_device = false;
    }
 
@@ -640,10 +381,9 @@ bool release_device(DCR *dcr)
                dcr->VolCatInfo.VolCatName, jcr->Job);
          }
          /* If no more writers, write an EOF */
-         if (!dev->num_writers && dev_can_write(dev)) {
+         if (!dev->num_writers && dev->can_write()) {
             weof_dev(dev, 1);
-            write_ansi_ibm_labels(dcr, ANSI_EOF_LABEL, dev->VolHdr.VolName);
-            Dmsg0(100, "==== write ansi eof label \n");
+            write_ansi_ibm_labels(dcr, ANSI_EOF_LABEL, dev->VolHdr.VolumeName);
          }
          if (!dev->at_weot()) {
             dev->VolCatInfo.VolCatFiles = dev->file;   /* set number of files */
@@ -665,7 +405,7 @@ bool release_device(DCR *dcr)
    /* If no writers, close if file or !CAP_ALWAYS_OPEN */
    if (dev->num_writers == 0 && (!dev->is_tape() || !dev_cap(dev, CAP_ALWAYSOPEN))) {
       offline_or_rewind_dev(dev);
-      close_dev(dev);
+      close_device(dev);
    }
 
    /* Fire off Alert command and include any output */
@@ -675,7 +415,7 @@ bool release_device(DCR *dcr)
       BPIPE *bpipe;
       char line[MAXSTRING];
       alert = get_pool_memory(PM_FNAME);
-      alert = edit_device_codes(dcr, alert, "");
+      alert = edit_device_codes(dcr, alert, dcr->device->alert_command, "");
       bpipe = open_bpipe(alert, 0, "r");
       if (bpipe) {
          while (fgets(line, sizeof(line), bpipe->rfd)) {
@@ -695,8 +435,108 @@ bool release_device(DCR *dcr)
       free_pool_memory(alert);
    }
    unlock_device(dev);
+   if (jcr->read_dcr == dcr) {
+      jcr->read_dcr = NULL;
+   }
+   if (jcr->dcr == dcr) {
+      jcr->dcr = NULL;
+   }
    free_dcr(dcr);
-   jcr->dcr = NULL;
-   pthread_cond_broadcast(&wait_device_release);
    return ok;
 }
+
+/*
+ * Create a new Device Control Record and attach
+ *   it to the device (if this is a real job).
+ */
+DCR *new_dcr(JCR *jcr, DEVICE *dev)
+{
+   DCR *dcr = (DCR *)malloc(sizeof(DCR));
+   memset(dcr, 0, sizeof(DCR));
+   dcr->jcr = jcr;
+   if (dev) {
+      dcr->dev = dev;
+      dcr->device = dev->device;
+      dcr->block = new_block(dev);
+      dcr->rec = new_record();
+      dcr->max_job_spool_size = dev->device->max_job_spool_size;
+      /* Attach this dcr only if dev is initialized */
+      if (dev->fd != 0 && jcr && jcr->JobType != JT_SYSTEM) {
+         dev->attached_dcrs->append(dcr);  /* attach dcr to device */
+//       jcr->dcrs->append(dcr);         /* put dcr in list for Job */
+      }
+   }
+   dcr->spool_fd = -1;
+   return dcr;
+}
+
+/*
+ * Search the dcrs list for the given dcr. If it is found,
+ *  as it should be, then remove it. Also zap the jcr pointer
+ *  to the dcr if it is the same one.
+ */
+#ifdef needed
+static void remove_dcr_from_dcrs(DCR *dcr)
+{
+   JCR *jcr = dcr->jcr;
+   if (jcr->dcrs) {
+      int i = 0;
+      DCR *ldcr;
+      int num = jcr->dcrs->size();
+      for (i=0; i < num; i++) {
+         ldcr = (DCR *)jcr->dcrs->get(i);
+         if (ldcr == dcr) {
+            jcr->dcrs->remove(i);
+            if (jcr->dcr == dcr) {
+               jcr->dcr = NULL;
+            }
+         }
+      }
+   }
+}
+#endif
+
+/*
+ * Free up all aspects of the given dcr -- i.e. dechain it,
+ *  release allocated memory, zap pointers, ...
+ */
+void free_dcr(DCR *dcr)
+{
+   JCR *jcr = dcr->jcr;
+   DEVICE *dev = dcr->dev;
+
+   if (dcr->reserved_device) {
+      lock_device(dev);
+      dev->reserved_device--;
+      Dmsg2(100, "Dec reserve=%d dev=%s\n", dev->reserved_device, dev->print_name());
+      dcr->reserved_device = false;
+      /* If we set read mode in reserving, remove it */
+       if (dev->can_read()) {
+         dev->clear_read();
+      }
+      if (dev->num_writers < 0) {
+         Jmsg1(dcr->jcr, M_ERROR, 0, _("Hey! num_writers=%d!!!!\n"), dev->num_writers);
+         dev->num_writers = 0;
+      }
+      unlock_device(dev);
+   }
+
+   /* Detach this dcr only if the dev is initialized */
+   if (dev->fd != 0 && jcr && jcr->JobType != JT_SYSTEM) {
+      dev->attached_dcrs->remove(dcr);  /* detach dcr from device */
+//    remove_dcr_from_dcrs(dcr);      /* remove dcr from jcr list */
+   }
+   if (dcr->block) {
+      free_block(dcr->block);
+   }
+   if (dcr->rec) {
+      free_record(dcr->rec);
+   }
+   if (dcr->jcr) {
+      dcr->jcr->dcr = NULL;
+   }
+   free_unused_volume(dcr);           /* free unused vols attached to this dcr */
+   free(dcr);
+   pthread_cond_broadcast(&dev->wait_next_vol);
+   pthread_cond_broadcast(&wait_device_release);
+}