]> git.sur5r.net Git - bacula/bacula/blobdiff - bacula/src/stored/askdir.c
kes Fix bug #935, and probably also bug #903 where files were not
[bacula/bacula] / bacula / src / stored / askdir.c
index 8811dcbe5a2a84cc74cec5a531fdea08a2dafeac..e533d76f83118a472c23b5033fcf1720395d0b55 100644 (file)
 /*
- *  Subroutines to handle Catalog reqests sent to the Director
- *   Reqests/commands from the Director are handled in dircmd.c
- *
- *   Kern Sibbald, December 2000
- *
- *   Version $Id$
- */
-/*
-   Copyright (C) 2000-2003 Kern Sibbald and John Walker
+   Bacula® - The Network Backup Solution
 
-   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.
+   Copyright (C) 2000-2007 Free Software Foundation Europe e.V.
 
-   This program is distributed in the hope that it will be useful,
-   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   The main author of Bacula is Kern Sibbald, with contributions from
+   many others, a complete list can be found in the file AUTHORS.
+   This program is Free Software; you can redistribute it and/or
+   modify it under the terms of version two of the GNU General Public
+   License as published by the Free Software Foundation and included
+   in the file LICENSE.
+
+   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.
+   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., 51 Franklin Street, Fifth Floor, Boston, MA
+   02110-1301, USA.
 
+   Bacula® is a registered trademark of John Walker.
+   The licensor of Bacula is the Free Software Foundation Europe
+   (FSFE), Fiduciary Program, Sumatrastrasse 25, 8006 Zürich,
+   Switzerland, email:ftf@fsfeurope.org.
+*/
+/*
+ *  Subroutines to handle Catalog reqests sent to the Director
+ *   Reqests/commands from the Director are handled in dircmd.c
+ *
+ *   Kern Sibbald, December 2000
+ *
+ *   Version $Id$
  */
 
 #include "bacula.h"                   /* pull in global headers */
 #include "stored.h"                   /* pull in Storage Deamon headers */
 
 /* Requests sent to the Director */
-static char Find_media[]   = "CatReq Job=%s FindMedia=%d\n";
+static char Find_media[]   = "CatReq Job=%s FindMedia=%d pool_name=%s media_type=%s\n";
 static char Get_Vol_Info[] = "CatReq Job=%s GetVolInfo VolName=%s write=%d\n";
 static char Update_media[] = "CatReq Job=%s UpdateMedia VolName=%s"
    " VolJobs=%u VolFiles=%u VolBlocks=%u VolBytes=%s VolMounts=%u"
    " VolErrors=%u VolWrites=%u MaxVolBytes=%s EndTime=%d VolStatus=%s"
-   " Slot=%d relabel=%d Drive=%d InChanger=%d\n";
-static char Create_job_media[] = "CatReq Job=%s CreateJobMedia" 
-   " FirstIndex=%u LastIndex=%u StartFile=%u EndFile=%u" 
-   " StartBlock=%u EndBlock=%u\n";
+   " Slot=%d relabel=%d InChanger=%d VolReadTime=%s VolWriteTime=%s"
+   " VolFirstWritten=%s VolParts=%u\n";
+static char Create_job_media[] = "CatReq Job=%s CreateJobMedia"
+   " FirstIndex=%u LastIndex=%u StartFile=%u EndFile=%u"
+   " StartBlock=%u EndBlock=%u Copy=%d Strip=%d MediaId=%s\n";
 static char FileAttributes[] = "UpdCat Job=%s FileAttributes ";
-static char Job_status[]     = "3012 Job %s jobstatus %d\n";
-
+static char Job_status[]     = "Status Job=%s JobStatus=%d\n";
 
 /* Responses received from the Director */
-static char OK_media[] = "1000 OK VolName=%127s VolJobs=%u VolFiles=%u"
-   " VolBlocks=%u VolBytes=%" lld " VolMounts=%u VolErrors=%u VolWrites=%u"
-   " MaxVolBytes=%" lld " VolCapacityBytes=%" lld " VolStatus=%20s"
-   " Slot=%d MaxVolJobs=%u MaxVolFiles=%u Drive=%d InChanger=%d";
+static char OK_media[] = "1000 OK VolName=%127s VolJobs=%u VolFiles=%lu"
+   " VolBlocks=%lu VolBytes=%lld VolMounts=%lu VolErrors=%lu VolWrites=%lu"
+   " MaxVolBytes=%lld VolCapacityBytes=%lld VolStatus=%20s"
+   " Slot=%ld MaxVolJobs=%lu MaxVolFiles=%lu InChanger=%ld"
+   " VolReadTime=%lld VolWriteTime=%lld EndFile=%lu EndBlock=%lu"
+   " VolParts=%lu LabelType=%ld MediaId=%lld\n";
+
 
 static char OK_create[] = "1000 OK CreateJobMedia\n";
 
-/* Forward referenced functions */
-static int wait_for_sysop(JCR *jcr, DEVICE *dev, int wait_sec);
+static pthread_mutex_t vol_info_mutex = PTHREAD_MUTEX_INITIALIZER;
+
+#ifdef needed
+
+static char Device_update[] = "DevUpd Job=%s device=%s "
+   "append=%d read=%d num_writers=%d "
+   "open=%d labeled=%d offline=%d "
+   "reserved=%d max_writers=%d "
+   "autoselect=%d autochanger=%d "
+   "changer_name=%s media_type=%s volume_name=%s\n";
+
+
+/* Send update information about a device to Director */
+bool dir_update_device(JCR *jcr, DEVICE *dev)
+{
+   BSOCK *dir = jcr->dir_bsock;
+   POOL_MEM dev_name, VolumeName, MediaType, ChangerName;
+   DEVRES *device = dev->device;
+   bool ok;
+   
+   pm_strcpy(dev_name, device->hdr.name);
+   bash_spaces(dev_name);
+   if (dev->is_labeled()) {
+      pm_strcpy(VolumeName, dev->VolHdr.VolumeName);
+   } else {
+      pm_strcpy(VolumeName, "*");
+   }
+   bash_spaces(VolumeName);
+   pm_strcpy(MediaType, device->media_type);
+   bash_spaces(MediaType);
+   if (device->changer_res) {
+      pm_strcpy(ChangerName, device->changer_res->hdr.name);
+      bash_spaces(ChangerName);
+   } else {
+      pm_strcpy(ChangerName, "*");
+   }
+   ok =bnet_fsend(dir, Device_update, 
+      jcr->Job,
+      dev_name.c_str(),
+      dev->can_append()!=0,
+      dev->can_read()!=0, dev->num_writers, 
+      dev->is_open()!=0, dev->is_labeled()!=0,
+      dev->is_offline()!=0, dev->reserved_device, 
+      dev->is_tape()?100000:1,
+      dev->autoselect, 0, 
+      ChangerName.c_str(), MediaType.c_str(), VolumeName.c_str());
+   Dmsg1(100, ">dird: %s\n", dir->msg);
+   return ok;
+}
+
+bool dir_update_changer(JCR *jcr, AUTOCHANGER *changer)
+{
+   BSOCK *dir = jcr->dir_bsock;
+   POOL_MEM dev_name, MediaType;
+   DEVRES *device;
+   bool ok;
+
+   pm_strcpy(dev_name, changer->hdr.name);
+   bash_spaces(dev_name);
+   device = (DEVRES *)changer->device->first();
+   pm_strcpy(MediaType, device->media_type);
+   bash_spaces(MediaType);
+   /* This is mostly to indicate that we are here */
+   ok = bnet_fsend(dir, Device_update,
+      jcr->Job,
+      dev_name.c_str(),         /* Changer name */
+      0, 0, 0,                  /* append, read, num_writers */
+      0, 0, 0,                  /* is_open, is_labeled, offline */
+      0, 0,                     /* reserved, max_writers */
+      0,                        /* Autoselect */
+      changer->device->size(),  /* Number of devices */
+      "0",                      /* PoolId */
+      "*",                      /* ChangerName */
+      MediaType.c_str(),        /* MediaType */
+      "*");                     /* VolName */
+   Dmsg1(100, ">dird: %s\n", dir->msg);
+   return ok;
+}
+#endif
+
 
 /*
  * Send current JobStatus to Director
  */
-int dir_send_job_status(JCR *jcr)
+bool dir_send_job_status(JCR *jcr)
 {
    return bnet_fsend(jcr->dir_bsock, Job_status, jcr->Job, jcr->JobStatus);
 }
@@ -67,182 +156,282 @@ int dir_send_job_status(JCR *jcr)
  *   dir_get_volume_info()
  * and
  *   dir_find_next_appendable_volume()
- * 
- *  Returns: 1 on success and vol info in jcr->VolCatInfo
- *          0 on failure
+ *
+ *  NOTE!!! All calls to this routine must be protected by
+ *          locking vol_info_mutex before calling it so that
+ *          we don't have one thread modifying the parameters
+ *          and another reading them.
+ *
+ *  Returns: true  on success and vol info in dcr->VolCatInfo
+ *           false on failure
  */
-static int do_get_volume_info(JCR *jcr)
+static bool do_get_volume_info(DCR *dcr)
 {
+    JCR *jcr = dcr->jcr;
     BSOCK *dir = jcr->dir_bsock;
     VOLUME_CAT_INFO vol;
+    int n;
+    int32_t InChanger;
 
-    jcr->VolumeName[0] = 0;          /* No volume */
-    if (bnet_recv(dir) <= 0) {
+    if (dir->recv() <= 0) {
        Dmsg0(200, "getvolname error bnet_recv\n");
-       Mmsg(&jcr->errmsg, _("Network error on bnet_recv in req_vol_info.\n"));
-       return 0;
+       Mmsg(jcr->errmsg, _("Network error on bnet_recv in req_vol_info.\n"));
+       return false;
     }
     memset(&vol, 0, sizeof(vol));
-    if (sscanf(dir->msg, OK_media, vol.VolCatName, 
-              &vol.VolCatJobs, &vol.VolCatFiles,
-              &vol.VolCatBlocks, &vol.VolCatBytes,
-              &vol.VolCatMounts, &vol.VolCatErrors,
-              &vol.VolCatWrites, &vol.VolCatMaxBytes,
-              &vol.VolCatCapacityBytes, vol.VolCatStatus,
-              &vol.Slot, &vol.VolCatMaxJobs, &vol.VolCatMaxFiles,
-              &vol.Drive, &vol.InChanger) != 16) {
-
-       Dmsg1(200, "Bad response from Dir: %s\n", dir->msg);
-       Mmsg(&jcr->errmsg, _("Error getting Volume info: %s\n"), dir->msg);
-       return 0;
+    Dmsg1(100, "<dird %s\n", dir->msg);
+    n = sscanf(dir->msg, OK_media, vol.VolCatName,
+               &vol.VolCatJobs, &vol.VolCatFiles,
+               &vol.VolCatBlocks, &vol.VolCatBytes,
+               &vol.VolCatMounts, &vol.VolCatErrors,
+               &vol.VolCatWrites, &vol.VolCatMaxBytes,
+               &vol.VolCatCapacityBytes, vol.VolCatStatus,
+               &vol.Slot, &vol.VolCatMaxJobs, &vol.VolCatMaxFiles,
+               &InChanger, &vol.VolReadTime, &vol.VolWriteTime,
+               &vol.EndFile, &vol.EndBlock, &vol.VolCatParts,
+               &vol.LabelType, &vol.VolMediaId);
+    if (n != 22) {
+       Dmsg3(100, "Bad response from Dir fields=%d, len=%d: %s", n, dir->msglen, dir->msg);
+       Mmsg(jcr->errmsg, _("Error getting Volume info: %s"), dir->msg);
+       return false;
     }
+    vol.InChanger = InChanger;        /* bool in structure */
     unbash_spaces(vol.VolCatName);
-    pm_strcpy(&jcr->VolumeName, vol.VolCatName); /* set desired VolumeName */
-    memcpy(&jcr->VolCatInfo, &vol, sizeof(jcr->VolCatInfo));
-    
-    Dmsg2(200, "do_reqest_vol_info got slot=%d Volume=%s\n", 
-         vol.Slot, vol.VolCatName);
-    return 1;
+    bstrncpy(dcr->VolumeName, vol.VolCatName, sizeof(dcr->VolumeName));
+    dcr->VolCatInfo = vol;            /* structure assignment */
+
+    Dmsg2(100, "do_reqest_vol_info return true slot=%d Volume=%s\n",
+          vol.Slot, vol.VolCatName);
+    return true;
 }
 
 
 /*
  * Get Volume info for a specific volume from the Director's Database
  *
- * Returns: 1 on success   (not Director guarantees that Pool and MediaType
- *                         are correct and VolStatus==Append or
- *                         VolStatus==Recycle)
- *         0 on failure
+ * Returns: true  on success   (Director guarantees that Pool and MediaType
+ *                              are correct and VolStatus==Append or
+ *                              VolStatus==Recycle)
+ *          false on failure
  *
- *         Volume information returned in jcr
+ *          Volume information returned in dcr->VolCatInfo
  */
-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)
 {
+    JCR *jcr = dcr->jcr;
     BSOCK *dir = jcr->dir_bsock;
 
-    bstrncpy(jcr->VolCatInfo.VolCatName, jcr->VolumeName, sizeof(jcr->VolCatInfo.VolCatName));
-    Dmsg1(200, "dir_get_volume_info=%s\n", jcr->VolCatInfo.VolCatName);
-    bash_spaces(jcr->VolCatInfo.VolCatName);
-    bnet_fsend(dir, Get_Vol_Info, jcr->Job, jcr->VolCatInfo.VolCatName, 
+    P(vol_info_mutex);
+    bstrncpy(dcr->VolCatInfo.VolCatName, dcr->VolumeName, sizeof(dcr->VolCatInfo.VolCatName));
+    bash_spaces(dcr->VolCatInfo.VolCatName);
+    dir->fsend(Get_Vol_Info, jcr->Job, dcr->VolCatInfo.VolCatName,
        writing==GET_VOL_INFO_FOR_WRITE?1:0);
-    return do_get_volume_info(jcr);
+    Dmsg1(100, ">dird: %s\n", dir->msg);
+    unbash_spaces(dcr->VolCatInfo.VolCatName);
+    bool ok = do_get_volume_info(dcr);
+    V(vol_info_mutex);
+    return ok;
 }
 
 
 
 /*
  * Get info on the next appendable volume in the Director's database
- * Returns: 1 on success
- *         0 on failure
  *
- *         Volume information returned in jcr
+ * Returns: true  on success dcr->VolumeName is volume
+ *                reserve_volume() called on Volume name
+ *          false on failure dcr->VolumeName[0] == 0
+ *                also sets dcr->volume_in_use if at least one 
+ *                in use volume was found.
+ *
+ *          Volume information returned in dcr
  *
  */
-int dir_find_next_appendable_volume(JCR *jcr)
+bool dir_find_next_appendable_volume(DCR *dcr)
 {
+    JCR *jcr = dcr->jcr;
     BSOCK *dir = jcr->dir_bsock;
+    bool found = false;
 
     Dmsg0(200, "dir_find_next_appendable_volume\n");
-    bnet_fsend(dir, Find_media, jcr->Job, 1);
-    return do_get_volume_info(jcr);
+    /*
+     * Try the twenty oldest or most available volumes.  Note,
+     *   the most available could already be mounted on another
+     *   drive, so we continue looking for a not in use Volume.
+     */
+    lock_reservations();
+    P(vol_info_mutex);
+    dcr->volume_in_use = false;
+    for (int vol_index=1;  vol_index < 20; vol_index++) {
+       bash_spaces(dcr->media_type);
+       bash_spaces(dcr->pool_name);
+       dir->fsend(Find_media, jcr->Job, vol_index, dcr->pool_name, dcr->media_type);
+       unbash_spaces(dcr->media_type);
+       unbash_spaces(dcr->pool_name);
+       Dmsg1(100, ">dird: %s", dir->msg);
+       bool ok = do_get_volume_info(dcr);
+       if (ok) {
+          if (!is_volume_in_use(dcr)) {
+             found = true;
+             break;
+          } else {
+             Dmsg1(100, "Volume %s is in use.\n", dcr->VolumeName);
+             dcr->volume_in_use = true;
+             continue;
+          }
+       } else {
+          Dmsg2(100, "No vol. index %d return false. dev=%s\n", vol_index,
+             dcr->dev->print_name());
+          found = false;
+          break;
+       }
+    }
+    if (found) {
+       Dmsg0(400, "dir_find_next_appendable_volume return true\n");
+       if (reserve_volume(dcr, dcr->VolumeName) == 0) {
+          Dmsg2(100, "Could not reserve volume %s on %s\n", dcr->VolumeName,
+              dcr->dev->print_name());
+          goto bail_out;
+       }
+       V(vol_info_mutex);
+       unlock_reservations();
+       return true;
+    }
+
+bail_out:
+    dcr->VolumeName[0] = 0;
+    V(vol_info_mutex);
+    unlock_reservations();
+    return false;
 }
 
-    
+
 /*
  * After writing a Volume, send the updated statistics
- * back to the director.
+ * back to the director. The information comes from the
+ * dev record.
  */
-int dir_update_volume_info(JCR *jcr, DEVICE *dev, int label)
+bool dir_update_volume_info(DCR *dcr, bool label)
 {
+   JCR *jcr = dcr->jcr;
    BSOCK *dir = jcr->dir_bsock;
-   time_t EndTime = time(NULL);
-   char ed1[50], ed2[50];
+   DEVICE *dev = dcr->dev;
+   time_t LastWritten = time(NULL);
    VOLUME_CAT_INFO *vol = &dev->VolCatInfo;
+   char ed1[50], ed2[50], ed3[50], ed4[50], ed5[50];
+   int InChanger;
+   bool ok = false;
+   POOL_MEM VolumeName;
+
+   /* If system job, do not update catalog */
+   if (jcr->JobType == JT_SYSTEM) {
+      return true;
+   }
 
    if (vol->VolCatName[0] == 0) {
-      Jmsg0(jcr, M_ERROR, 0, _("NULL Volume name. This shouldn't happen!!!\n"));
-      return 0;
-   }
-   if (dev_state(dev, ST_READ)) {
-      Jmsg0(jcr, M_ERROR, 0, _("Attempt to update_volume_info in read mode!!!\n"));
-      return 0;
-   }
-   if (!dev_state(dev, ST_LABEL)) {
-      Jmsg0(jcr, M_ERROR, 0, _("Attempt to update_volume_info on non-labeled Volume!!!\n"));
-      return 0;
+      Jmsg0(jcr, M_FATAL, 0, _("NULL Volume name. This shouldn't happen!!!\n"));
+      Pmsg0(000, _("NULL Volume name. This shouldn't happen!!!\n"));
+      return false;
    }
 
-   dev->VolCatInfo.VolCatFiles = dev->file;   /* set number of files */
+   /* Lock during Volume update */
+   P(vol_info_mutex);
    Dmsg1(100, "Update cat VolFiles=%d\n", dev->file);
    /* Just labeled or relabeled the tape */
    if (label) {
       bstrncpy(vol->VolCatStatus, "Append", sizeof(vol->VolCatStatus));
-      vol->VolCatBytes = 1;          /* indicates tape labeled */
    }
-   bash_spaces(vol->VolCatName);
-   bnet_fsend(dir, Update_media, jcr->Job, 
-      vol->VolCatName, vol->VolCatJobs, vol->VolCatFiles,
+   pm_strcpy(VolumeName, vol->VolCatName);
+   bash_spaces(VolumeName);
+   InChanger = vol->InChanger;
+   bnet_fsend(dir, Update_media, jcr->Job,
+      VolumeName.c_str(), vol->VolCatJobs, vol->VolCatFiles,
       vol->VolCatBlocks, edit_uint64(vol->VolCatBytes, ed1),
       vol->VolCatMounts, vol->VolCatErrors,
-      vol->VolCatWrites, edit_uint64(vol->VolCatMaxBytes, ed2), 
-      EndTime, vol->VolCatStatus, vol->Slot, label, vol->Drive, 
-      vol->InChanger);
-   Dmsg1(120, "update_volume_data(): %s", dir->msg);
-   unbash_spaces(vol->VolCatName);
-
-   if (!do_get_volume_info(jcr)) {
-      Jmsg(jcr, M_ERROR, 0, "%s", jcr->errmsg);
-      return 0;
+      vol->VolCatWrites, edit_uint64(vol->VolCatMaxBytes, ed2),
+      LastWritten, vol->VolCatStatus, vol->Slot, label,
+      InChanger,                      /* bool in structure */
+      edit_int64(vol->VolReadTime, ed3),
+      edit_int64(vol->VolWriteTime, ed4),
+      edit_uint64(vol->VolFirstWritten, ed5),
+      vol->VolCatParts);
+    Dmsg1(100, ">dird: %s", dir->msg);
+
+   /* Do not lock device here because it may be locked from label */
+   if (!do_get_volume_info(dcr)) {
+      Jmsg(jcr, M_FATAL, 0, "%s", jcr->errmsg);
+      Dmsg2(100, _("Didn't get vol info vol=%s: ERR=%s"), 
+         vol->VolCatName, jcr->errmsg);
+      goto bail_out;
    }
+   Dmsg1(420, "get_volume_info(): %s", dir->msg);
    /* Update dev Volume info in case something changed (e.g. expired) */
-   memcpy(&dev->VolCatInfo, &jcr->VolCatInfo, sizeof(dev->VolCatInfo));
-   return 1;
+   dev->VolCatInfo = dcr->VolCatInfo;
+   ok = true;
+
+bail_out:
+   V(vol_info_mutex);
+   return ok;
 }
 
 /*
  * After writing a Volume, create the JobMedia record.
  */
-int dir_create_jobmedia_record(JCR *jcr)
+bool dir_create_jobmedia_record(DCR *dcr)
 {
+   JCR *jcr = dcr->jcr;
    BSOCK *dir = jcr->dir_bsock;
+   char ed1[50];
+
+   /* If system job, do not update catalog */
+   if (jcr->JobType == JT_SYSTEM) {
+      return true;
+   }
 
-   if (!jcr->WroteVol) {
-      return 1;                      /* nothing written to tape */
+   if (!dcr->WroteVol) {
+      return true;                    /* nothing written to tape */
    }
 
-   jcr->WroteVol = false;
-   bnet_fsend(dir, Create_job_media, jcr->Job, 
-      jcr->VolFirstIndex, jcr->VolLastIndex,
-      jcr->StartFile, jcr->EndFile,
-      jcr->StartBlock, jcr->EndBlock);
-   Dmsg1(100, "create_jobmedia(): %s", dir->msg);
+   dcr->WroteVol = false;
+   bnet_fsend(dir, Create_job_media, jcr->Job,
+      dcr->VolFirstIndex, dcr->VolLastIndex,
+      dcr->StartFile, dcr->EndFile,
+      dcr->StartBlock, dcr->EndBlock, 
+      dcr->Copy, dcr->Stripe, 
+      edit_uint64(dcr->VolMediaId, ed1));
+    Dmsg1(100, ">dird: %s", dir->msg);
    if (bnet_recv(dir) <= 0) {
       Dmsg0(190, "create_jobmedia error bnet_recv\n");
-      Jmsg(jcr, M_ERROR, 0, _("Error creating JobMedia record: ERR=%s\n"), 
-          bnet_strerror(dir));
-      return 0;
+      Jmsg(jcr, M_FATAL, 0, _("Error creating JobMedia record: ERR=%s\n"),
+           bnet_strerror(dir));
+      return false;
    }
-   Dmsg1(120, "Create_jobmedia: %s", dir->msg);
+   Dmsg1(100, "<dir: %s", dir->msg);
    if (strcmp(dir->msg, OK_create) != 0) {
       Dmsg1(130, "Bad response from Dir: %s\n", dir->msg);
-      Jmsg(jcr, M_ERROR, 0, _("Error creating JobMedia record: %s\n"), dir->msg);
-      return 0;
+      Jmsg(jcr, M_FATAL, 0, _("Error creating JobMedia record: %s\n"), dir->msg);
+      return false;
    }
-   return 1;
+   return true;
 }
 
 
-/* 
+/*
  * Update File Attribute data
  */
-int dir_update_file_attributes(JCR *jcr, DEV_RECORD *rec)
+bool dir_update_file_attributes(DCR *dcr, DEV_RECORD *rec)
 {
+   JCR *jcr = dcr->jcr;
    BSOCK *dir = jcr->dir_bsock;
    ser_declare;
 
+#ifdef NO_ATTRIBUTES_TEST
+   return true;
+#endif
+
    dir->msglen = sprintf(dir->msg, FileAttributes, jcr->Job);
-   dir->msg = check_pool_memory_size(dir->msg, dir->msglen + 
-               sizeof(DEV_RECORD) + rec->data_len);
+   dir->msg = check_pool_memory_size(dir->msg, dir->msglen +
+                sizeof(DEV_RECORD) + rec->data_len);
    ser_begin(dir->msg + dir->msglen, 0);
    ser_uint32(rec->VolSessionId);
    ser_uint32(rec->VolSessionTime);
@@ -251,298 +440,189 @@ int dir_update_file_attributes(JCR *jcr, DEV_RECORD *rec)
    ser_uint32(rec->data_len);
    ser_bytes(rec->data, rec->data_len);
    dir->msglen = ser_length(dir->msg);
+   Dmsg1(1800, ">dird: %s\n", dir->msg);    /* Attributes */
    return bnet_send(dir);
 }
 
 
 /*
- *   Request to mount next Volume, which Volume not specified
+ *   Request the sysop to create an appendable volume
  *
  *   Entered with device blocked.
  *   Leaves with device blocked.
  *
- *   Returns: 1 on success (operator issues a mount command)
- *           0 on failure
- *             Note, must create dev->errmsg on error return.
+ *   Returns: true  on success (operator issues a mount command)
+ *            false on failure
+ *              Note, must create dev->errmsg on error return.
  *
- *    On success, jcr->VolumeName and jcr->VolCatInfo contain
- *     information on suggested volume, but this may not be the
- *     same as what is actually mounted.
+ *    On success, dcr->VolumeName and dcr->VolCatInfo contain
+ *      information on suggested volume, but this may not be the
+ *      same as what is actually mounted.
  *
  *    When we return with success, the correct tape may or may not
- *     actually be mounted. The calling routine must read it and
- *     verify the label.
+ *      actually be mounted. The calling routine must read it and
+ *      verify the label.
  */
-int dir_ask_sysop_to_mount_next_volume(JCR *jcr, DEVICE *dev)
+bool dir_ask_sysop_to_create_appendable_volume(DCR *dcr)
 {
-   int stat = 0, jstat;
-   /* ******FIXME******* put these on config variable */
-   int min_wait = 60 * 60;
-   int max_wait = 24 * 60 * 60;
-   int max_num_wait = 9;             /* 5 waits =~ 1 day, then 1 day at a time */
-
-   int wait_sec;
-   int num_wait = 0;
-
-   Dmsg0(130, "enter dir_ask_sysop_to_mount_next_volume\n");
-   ASSERT(dev->dev_blocked);
-   wait_sec = min_wait;
+   int stat = W_TIMEOUT;
+   DEVICE *dev = dcr->dev;
+   JCR *jcr = dcr->jcr;
+   bool got_vol = false;
+
+   Dmsg0(400, "enter dir_ask_sysop_to_create_appendable_volume\n");
+   ASSERT(dev->blocked());
    for ( ;; ) {
       if (job_canceled(jcr)) {
-         Mmsg(&dev->errmsg, _("Job %s canceled while waiting for mount on Storage Device \"%s\".\n"), 
-             jcr->Job, jcr->dev_name);
+         Mmsg(dev->errmsg,
+              _("Job %s canceled while waiting for mount on Storage Device \"%s\".\n"),
+              jcr->Job, dev->print_name());
          Jmsg(jcr, M_INFO, 0, "%s", dev->errmsg);
-        return 0;
+         return false;
       }
-      if (dir_find_next_appendable_volume(jcr)) {    /* get suggested volume */
-        jstat = JS_WaitMount;
-        /*
-         * If we have a valid volume name and we are not
-         *   removable media, return now, or if we have a
-         *   Slot for an autochanger, otherwise wait
-         *   for the operator to mount the media.
-         */
-        if ((jcr->VolumeName[0] && !dev_cap(dev, CAP_REM) && dev_cap(dev, CAP_LABEL)) ||
-            (jcr->VolumeName[0] && jcr->VolCatInfo.Slot)) {
-            Dmsg0(100, "Return 1 from mount without wait.\n");
-           return 1;
-        }
-        Jmsg(jcr, M_MOUNT, 0, _(
-"Please mount Volume \"%s\" on Storage Device \"%s\" for Job %s\n"
-"Use \"mount\" command to release Job.\n"),
-             jcr->VolumeName, jcr->dev_name, jcr->Job);
-         Dmsg3(190, "Mount %s on %s for Job %s\n",
-               jcr->VolumeName, jcr->dev_name, jcr->Job);
+      dev->dlock();  
+      got_vol = dir_find_next_appendable_volume(dcr);   /* get suggested volume */
+      dev->dunlock();
+      if (got_vol) {
+         return true;
       } else {
-        jstat = JS_WaitMedia;
-        Jmsg(jcr, M_MOUNT, 0, _(
-"Job %s waiting. Cannot find any appendable volumes.\n\
-Please use the \"label\"  command to create a new Volume for:\n\
-    Storage:      %s\n\
-    Media type:   %s\n\
-    Pool:         %s\n"),
-             jcr->Job, 
-             jcr->dev_name, 
-             jcr->media_type,
-             jcr->pool_name);
+         if (stat == W_TIMEOUT || stat == W_MOUNT) {
+            Jmsg(jcr, M_MOUNT, 0, _(
+"Job %s waiting. Cannot find any appendable volumes.\n"
+"Please use the \"label\"  command to create a new Volume for:\n"
+"    Storage:      %s\n"
+"    Pool:         %s\n"
+"    Media type:   %s\n"),
+               jcr->Job,
+               dev->print_name(),
+               dcr->pool_name,
+               dcr->media_type);
+         }
       }
 
-      jcr->JobStatus = jstat;
+      set_jcr_job_status(jcr, JS_WaitMedia);
       dir_send_job_status(jcr);
 
-      stat = wait_for_sysop(jcr, dev, wait_sec);
-
-      if (stat == ETIMEDOUT) {
-        wait_sec *= 2;               /* double wait time */
-        if (wait_sec > max_wait) {   /* but not longer than maxtime */
-           wait_sec = max_wait;
-        }
-        num_wait++;
-        if (num_wait >= max_num_wait) {
-            Mmsg(&dev->errmsg, _("Gave up waiting to mount Storage Device \"%s\" for Job %s\n"), 
-                jcr->dev_name, jcr->Job);
+      stat = wait_for_sysop(dcr);
+      Dmsg1(100, "Back from wait_for_sysop stat=%d\n", stat);
+      if (dev->poll) {
+         Dmsg1(100, "Poll timeout in create append vol on device %s\n", dev->print_name());
+         continue;
+      }
+
+      if (stat == W_TIMEOUT) {
+         if (!double_dev_wait_time(dev)) {
+            Mmsg(dev->errmsg, _("Max time exceeded waiting to mount Storage Device %s for Job %s\n"),
+               dev->print_name(), jcr->Job);
             Jmsg(jcr, M_FATAL, 0, "%s", dev->errmsg);
-            Dmsg1(190, "Gave up waiting on device %s\n", dev_name(dev));
-           return 0;                 /* exceeded maximum waits */
-        }
-        continue;
+            Dmsg1(100, "Gave up waiting on device %s\n", dev->print_name());
+            return false;             /* exceeded maximum waits */
+         }
+         continue;
       }
-      if (stat == EINVAL) {
-         Mmsg2(&dev->errmsg, _("pthread error in mount_next_volume stat=%d ERR=%s\n"),
-              stat, strerror(stat));
+      if (stat == W_ERROR) {
+         berrno be;
+         Mmsg0(dev->errmsg, _("pthread error in mount_next_volume.\n"));
          Jmsg(jcr, M_FATAL, 0, "%s", dev->errmsg);
-        return 0;
+         return false;
       }
-      if (stat != 0) {
-         Jmsg(jcr, M_WARNING, 0, _("pthread error in mount_next_volume stat=%d ERR=%s\n"), stat,
-           strerror(stat));
-      }
-      Dmsg1(190, "Someone woke me for device %s\n", dev_name(dev));
-
-      /* Restart wait counters */
-      wait_sec = min_wait;
-      num_wait = 0;
-      /* If no VolumeName, and cannot get one, try again */
-      if (jcr->VolumeName[0] == 0 && !job_canceled(jcr) &&
-         !dir_find_next_appendable_volume(jcr)) {
-        Jmsg(jcr, M_MOUNT, 0, _(
-"Someone woke me up, but I cannot find any appendable\n\
-volumes for Job=%s.\n"), jcr->Job);
-        continue;
-      }       
-      break;
+      Dmsg1(100, "Someone woke me for device %s\n", dev->print_name());
    }
    set_jcr_job_status(jcr, JS_Running);
    dir_send_job_status(jcr);
-   Dmsg0(130, "leave dir_ask_sysop_to_mount_next_volume\n");
-   return 1;
+   Dmsg0(100, "leave dir_ask_sysop_to_mount_create_appendable_volume\n");
+   return true;
 }
 
 /*
  *   Request to mount specific Volume
  *
- *   Entered with device blocked and jcr->VolumeName is desired
- *     volume.
+ *   Entered with device blocked and dcr->VolumeName is desired
+ *      volume.
  *   Leaves with device blocked.
  *
- *   Returns: 1 on success (operator issues a mount command)
- *           0 on failure
- *             Note, must create dev->errmsg on error return.
+ *   Returns: true  on success (operator issues a mount command)
+ *            false on failure
+ *                  Note, must create dev->errmsg on error return.
  *
  */
-int dir_ask_sysop_to_mount_volume(JCR *jcr, DEVICE *dev)
+bool dir_ask_sysop_to_mount_volume(DCR *dcr)
 {
-   int stat = 0;
-   /* ******FIXME******* put these on config variable */
-   int min_wait = 60 * 60;
-   int max_wait = 24 * 60 * 60;
-   int max_num_wait = 9;             /* 5 waits =~ 1 day, then 1 day at a time */
-   int wait_sec;
-   int num_wait = 0;
-   char *msg;
-
-   Dmsg0(130, "enter dir_ask_sysop_to_mount_next_volume\n");
-   if (!jcr->VolumeName[0]) {
-      Mmsg0(&dev->errmsg, _("Cannot request another volume: no volume name given.\n"));
-      return 0;
+   int stat = W_TIMEOUT;
+   DEVICE *dev = dcr->dev;
+   JCR *jcr = dcr->jcr;
+
+   Dmsg0(400, "enter dir_ask_sysop_to_mount_volume\n");
+   if (!dcr->VolumeName[0]) {
+      Mmsg0(dev->errmsg, _("Cannot request another volume: no volume name given.\n"));
+      return false;
    }
-   ASSERT(dev->dev_blocked);
-   wait_sec = min_wait;
+   ASSERT(dev->blocked());
    for ( ;; ) {
       if (job_canceled(jcr)) {
-         Mmsg(&dev->errmsg, _("Job %s canceled while waiting for mount on Storage Device \"%s\".\n"), 
-             jcr->Job, jcr->dev_name);
-        return 0;
+         Mmsg(dev->errmsg, _("Job %s canceled while waiting for mount on Storage Device %s.\n"),
+              jcr->Job, dev->print_name());
+         return false;
       }
-      msg = _("Please mount");
-      Jmsg(jcr, M_MOUNT, 0, _("%s Volume \"%s\" on Storage Device \"%s\" for Job %s\n"),
-          msg, jcr->VolumeName, jcr->dev_name, jcr->Job);
-      Dmsg3(190, "Mount %s on %s for Job %s\n",
-           jcr->VolumeName, jcr->dev_name, jcr->Job);
 
-      jcr->JobStatus = JS_WaitMount;
-      dir_send_job_status(jcr);
-
-      stat = wait_for_sysop(jcr, dev, wait_sec); /* wait on device */
-
-      if (stat == ETIMEDOUT) {
-        wait_sec *= 2;               /* double wait time */
-        if (wait_sec > max_wait) {   /* but not longer than maxtime */
-           wait_sec = max_wait;
-        }
-        num_wait++;
-        if (num_wait >= max_num_wait) {
-            Mmsg(&dev->errmsg, _("Gave up waiting to mount Storage Device \"%s\" for Job %s\n"), 
-                jcr->dev_name, jcr->Job);
-            Jmsg(jcr, M_FATAL, 0, "%s", dev->errmsg);
-            Dmsg1(190, "Gave up waiting on device %s\n", dev_name(dev));
-           return 0;                 /* exceeded maximum waits */
-        }
-        continue;
-      }
-      if (stat == EINVAL) {
-         Mmsg2(&dev->errmsg, _("pthread error in mount_volume stat=%d ERR=%s\n"),
-              stat, strerror(stat));
-         Jmsg(jcr, M_FATAL, 0, "%s", dev->errmsg);
-        return 0;
+      if (dev->is_dvd()) {   
+         dev->unmount(0);
       }
-      if (stat != 0) {
-         Jmsg(jcr, M_ERROR, 0, _("pthread error in mount_next_volume stat=%d ERR=%s\n"), stat,
-           strerror(stat));
+      
+      /*
+       * If we are not polling, and the wait timeout or the
+       *   user explicitly did a mount, send him the message.
+       *   Otherwise skip it.
+       */
+      if (!dev->poll && (stat == W_TIMEOUT || stat == W_MOUNT)) {
+         Jmsg(jcr, M_MOUNT, 0, _("Please mount Volume \"%s\" or label a new one for:\n"
+              "    Job:          %s\n"
+              "    Storage:      %s\n"
+              "    Pool:         %s\n"
+              "    Media type:   %s\n"),
+              dcr->VolumeName,
+              jcr->Job,
+              dev->print_name(),
+              dcr->pool_name,
+              dcr->media_type);
+         Dmsg3(400, "Mount \"%s\" on device \"%s\" for Job %s\n",
+               dcr->VolumeName, dev->print_name(), jcr->Job);
       }
-      Dmsg1(190, "Someone woke me for device %s\n", dev_name(dev));
 
-      /* Restart wait counters */
-      wait_sec = min_wait;
-      num_wait = 0;
-      break;
-   }
-   set_jcr_job_status(jcr, JS_Running);
-   dir_send_job_status(jcr);
-   Dmsg0(130, "leave dir_ask_sysop_to_mount_next_volume\n");
-   return 1;
-}
-
-/*
- * Wait for SysOp to mount a tape
- */
-static int wait_for_sysop(JCR *jcr, DEVICE *dev, int wait_sec)
-{
-   struct timeval tv;
-   struct timezone tz;
-   struct timespec timeout;
-   int dev_blocked;
-   time_t start = time(NULL);
-   time_t last_heartbeat = 0;
-   int stat = 0;
-   int add_wait;
-   
-   /*
-    * Wait requested time (wait_sec).  However, we also wake up every
-    *   HB_TIME seconds and send a heartbeat to the FD and the Director
-    *   to keep stateful firewalls from closing them down while waiting
-    *   for the operator.
-    */
-   add_wait = wait_sec;
-   if (me->heartbeat_interval && add_wait > me->heartbeat_interval) {
-      add_wait = me->heartbeat_interval;
-   }
-   gettimeofday(&tv, &tz);
-   timeout.tv_nsec = tv.tv_usec * 1000;
-   timeout.tv_sec = tv.tv_sec + add_wait;
-
-   P(dev->mutex);
-   dev_blocked = dev->dev_blocked;
-   dev->dev_blocked = BST_WAITING_FOR_SYSOP; /* indicate waiting for mount */
-
-   for ( ; !job_canceled(jcr); ) {
-      time_t now;
-
-      Dmsg3(100, "I'm going to sleep on device %s. HB=%d wait=%d\n", dev_name(dev),
-        (int)me->heartbeat_interval, wait_sec);
-      stat = pthread_cond_timedwait(&dev->wait_next_vol, &dev->mutex, &timeout);
-      Dmsg1(100, "Wokeup from sleep on device stat=%d\n", stat);
-
-      now = time(NULL);
-
-      /* Note, this always triggers the first time. We want that. */
-      if (me->heartbeat_interval) {
-        if (now - last_heartbeat >= me->heartbeat_interval) {
-           /* send heartbeats */
-           if (jcr->file_bsock) {
-              bnet_sig(jcr->file_bsock, BNET_HEARTBEAT);
-               Dmsg0(100, "Send heartbeat to FD.\n");
-           }
-           if (jcr->dir_bsock) {
-              bnet_sig(jcr->dir_bsock, BNET_HEARTBEAT);
-           }
-           last_heartbeat = now;
-        }
-      }
+      set_jcr_job_status(jcr, JS_WaitMount);
+      dir_send_job_status(jcr);
 
-      if (dev->dev_blocked == BST_MOUNT) {   /* mount request ? */
-        stat = 0;
-        break;
+      stat = wait_for_sysop(dcr);          /* wait on device */
+      Dmsg1(100, "Back from wait_for_sysop stat=%d\n", stat);
+      if (dev->poll) {
+         Dmsg1(400, "Poll timeout in mount vol on device %s\n", dev->print_name());
+         Dmsg1(400, "Blocked=%s\n", dev->print_blocked());
+         goto bail_out;
       }
 
-      if (stat != ETIMEDOUT) {    /* we blocked the device */
-        break;                    /* on error return */
-      }
-      if (now - start >= wait_sec) {  /* on exceeding wait time return */
-         Dmsg0(100, "Exceed wait time.\n");
-        break;
+      if (stat == W_TIMEOUT) {
+         if (!double_dev_wait_time(dev)) {
+            Mmsg(dev->errmsg, _("Max time exceeded waiting to mount Storage Device %s for Job %s\n"),
+               dev->print_name(), jcr->Job);
+            Jmsg(jcr, M_FATAL, 0, "%s", dev->errmsg);
+            Dmsg1(400, "Gave up waiting on device %s\n", dev->print_name());
+            return false;             /* exceeded maximum waits */
+         }
+         continue;
       }
-      add_wait = wait_sec - (now - start);
-      if (me->heartbeat_interval && add_wait > me->heartbeat_interval) {
-        add_wait = me->heartbeat_interval;
+      if (stat == W_ERROR) {
+         berrno be;
+         Mmsg(dev->errmsg, _("pthread error in mount_volume\n"));
+         Jmsg(jcr, M_FATAL, 0, "%s", dev->errmsg);
+         return false;
       }
-      gettimeofday(&tv, &tz);
-      timeout.tv_nsec = tv.tv_usec * 1000;
-      timeout.tv_sec = tv.tv_sec + add_wait; /* additional wait */
-      Dmsg1(100, "Additional wait %d sec.\n", add_wait);
+      Dmsg1(400, "Someone woke me for device %s\n", dev->print_name());
+      break;
    }
 
-   dev->dev_blocked = dev_blocked;    /* restore entry state */
-   V(dev->mutex);
-   return stat;
+bail_out:
+   set_jcr_job_status(jcr, JS_Running);
+   dir_send_job_status(jcr);
+   Dmsg0(400, "leave dir_ask_sysop_to_mount_volume\n");
+   return true;
 }