]> git.sur5r.net Git - bacula/bacula/blobdiff - bacula/src/stored/askdir.c
- Merge changes made to 1.38.3 into HEAD
[bacula/bacula] / bacula / src / stored / askdir.c
index 0fa4699c764a9d5649b41d960186d247fd7519a9..09e05e98e9d59b07a1fa5bae2386a21ae148f62d 100644 (file)
@@ -11,7 +11,7 @@
 
    This program is free software; you can redistribute it and/or
    modify it under the terms of the GNU General Public License
-   version 2 as ammended with additional clauses defined in the
+   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,
@@ -183,12 +183,7 @@ static bool do_get_volume_info(DCR *dcr)
     bstrncpy(dcr->VolumeName, vol.VolCatName, sizeof(dcr->VolumeName));
     memcpy(&dcr->VolCatInfo, &vol, sizeof(dcr->VolCatInfo));
 
-   /* ***FIXME*** we really should not do this but must for the moment */
-   if (dcr->dev->num_parts < dcr->VolCatInfo.VolCatParts) {
-      dcr->dev->num_parts = dcr->VolCatInfo.VolCatParts;
-   }
-
-    Dmsg2(300, "do_reqest_vol_info got slot=%d Volume=%s\n",
+    Dmsg2(300, "do_reqest_vol_info return true slot=%d Volume=%s\n",
           vol.Slot, vol.VolCatName);
     return true;
 }
@@ -214,7 +209,8 @@ bool dir_get_volume_info(DCR *dcr, enum get_vol_info_rw writing)
     bnet_fsend(dir, Get_Vol_Info, jcr->Job, dcr->VolCatInfo.VolCatName,
        writing==GET_VOL_INFO_FOR_WRITE?1:0);
     Dmsg1(100, ">dird: %s", dir->msg);
-    return do_get_volume_info(dcr);
+    bool OK = do_get_volume_info(dcr);
+    return OK;
 }
 
 /*
@@ -244,12 +240,14 @@ bool dir_find_next_appendable_volume(DCR *dcr)
        unbash_spaces(dcr->media_type);
        unbash_spaces(dcr->pool_name);
        Dmsg1(100, ">dird: %s", dir->msg);
-       if (do_get_volume_info(dcr)) {
-          if (is_volume_in_use(dcr->VolumeName)) {
-             continue;
-          } else {
+       bool OK = do_get_volume_info(dcr);
+       if (OK) {
+          if (dcr->any_volume || !is_volume_in_use(dcr)) {
              found = true;
              break;
+          } else {
+             Dmsg1(100, "Volume %s is in use.\n", dcr->VolumeName);
+             continue;
           }
        } else {
           Dmsg0(200, "No volume info, return false\n");
@@ -258,9 +256,10 @@ bool dir_find_next_appendable_volume(DCR *dcr)
     }
     if (found) {
        Dmsg0(400, "dir_find_next_appendable_volume return true\n");
-       new_volume(dcr->VolumeName, NULL);   /* reserve volume */
+       new_volume(dcr, dcr->VolumeName);   /* reserve volume */
        return true;
     }
+    dcr->VolumeName[0] = 0;
     return false;
 }
 
@@ -283,16 +282,16 @@ bool dir_update_volume_info(DCR *dcr, bool label)
 
    if (vol->VolCatName[0] == 0) {
       Jmsg0(jcr, M_FATAL, 0, _("NULL Volume name. This shouldn't happen!!!\n"));
-      Pmsg0(000, "NULL Volume name. This shouldn't happen!!!\n");
+      Pmsg0(000, _("NULL Volume name. This shouldn't happen!!!\n"));
       return false;
    }
    if (dev->can_read()) {
       Jmsg0(jcr, M_FATAL, 0, _("Attempt to update_volume_info in read mode!!!\n"));
-      Pmsg0(000, "Attempt to update_volume_info in read mode!!!\n");
+      Pmsg0(000, _("Attempt to update_volume_info in read mode!!!\n"));
       return false;
    }
 
-   Dmsg1(300, "Update cat VolFiles=%d\n", dev->file);
+   Dmsg1(100, "Update cat VolFiles=%d\n", dev->file);
    /* Just labeled or relabeled the tape */
    if (label) {
       bstrncpy(vol->VolCatStatus, "Append", sizeof(vol->VolCatStatus));
@@ -313,9 +312,10 @@ bool dir_update_volume_info(DCR *dcr, bool label)
       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);
-      Pmsg2(000, "Didn't get vol info vol=%s: ERR=%s"
+      Pmsg2(000, _("Didn't get vol info vol=%s: ERR=%s")
          vol->VolCatName, jcr->errmsg);
       return false;
    }
@@ -414,6 +414,7 @@ bool dir_ask_sysop_to_create_appendable_volume(DCR *dcr)
    bool first = true;
    DEVICE *dev = dcr->dev;
    JCR *jcr = dcr->jcr;
+   bool OK = false;
 
    Dmsg0(400, "enter dir_ask_sysop_to_create_appendable_volume\n");
    ASSERT(dev->dev_blocked);
@@ -426,9 +427,13 @@ bool dir_ask_sysop_to_create_appendable_volume(DCR *dcr)
          return false;
       }
       /* First pass, we *know* there are no appendable volumes, so no need to call */
-      if (!first && dir_find_next_appendable_volume(dcr)) { /* get suggested volume */
-         unmounted = (dev->dev_blocked == BST_UNMOUNTED) ||
-                     (dev->dev_blocked == BST_UNMOUNTED_WAITING_FOR_SYSOP);
+      if (!first) {
+         P(dev->mutex);
+         OK = dir_find_next_appendable_volume(dcr);   /* get suggested volume */
+         V(dev->mutex);
+      }
+      if (!first && OK) {
+         unmounted = is_device_unmounted(dev);
          /*
           * If we have a valid volume name and we are not
           *   removable media, return now, or if we have a
@@ -472,7 +477,7 @@ bool dir_ask_sysop_to_create_appendable_volume(DCR *dcr)
 
       stat = wait_for_sysop(dcr);
       if (dev->poll) {
-         Dmsg1(400, "Poll timeout in create append vol on device %s\n", dev->print_name());
+         Dmsg1(100, "Poll timeout in create append vol on device %s\n", dev->print_name());
          continue;
       }
 
@@ -481,7 +486,7 @@ bool dir_ask_sysop_to_create_appendable_volume(DCR *dcr)
             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());
+            Dmsg1(100, "Gave up waiting on device %s\n", dev->print_name());
             return false;             /* exceeded maximum waits */
          }
          continue;
@@ -498,11 +503,13 @@ bool dir_ask_sysop_to_create_appendable_volume(DCR *dcr)
          Jmsg(jcr, M_WARNING, 0, _("pthread error in mount_next_volume stat=%d ERR=%s\n"), stat,
             be.strerror(stat));
       }
-      Dmsg1(400, "Someone woke me for device %s\n", dev->print_name());
+      Dmsg1(100, "Someone woke me for device %s\n", dev->print_name());
 
       /* If no VolumeName, and cannot get one, try again */
+      P(dev->mutex);
       if (dcr->VolumeName[0] == 0 && !job_canceled(jcr) &&
           !dir_find_next_appendable_volume(dcr)) {
+         V(dev->mutex);
          Jmsg(jcr, M_MOUNT, 0, _(
 "Someone woke me up, but I cannot find any appendable\n"
 "volumes for Job=%s.\n"), jcr->Job);
@@ -510,9 +517,10 @@ bool dir_ask_sysop_to_create_appendable_volume(DCR *dcr)
          init_device_wait_timers(dcr);
          continue;
       }
-      unmounted = (dev->dev_blocked == BST_UNMOUNTED) ||
-                  (dev->dev_blocked == BST_UNMOUNTED_WAITING_FOR_SYSOP);
+      V(dev->mutex);
+      unmounted = is_device_unmounted(dev);
       if (unmounted) {
+         Dmsg0(400, "Device is unmounted. Must wait.\n");
          continue;                    /* continue to wait */
       }
 
@@ -523,7 +531,7 @@ bool dir_ask_sysop_to_create_appendable_volume(DCR *dcr)
    }
    set_jcr_job_status(jcr, JS_Running);
    dir_send_job_status(jcr);
-   Dmsg0(400, "leave dir_ask_sysop_to_mount_create_appendable_volume\n");
+   Dmsg0(100, "leave dir_ask_sysop_to_mount_create_appendable_volume\n");
    return true;
 }
 
@@ -542,7 +550,6 @@ bool dir_ask_sysop_to_create_appendable_volume(DCR *dcr)
 bool dir_ask_sysop_to_mount_volume(DCR *dcr)
 {
    int stat = 0;
-   const char *msg;
    DEVICE *dev = dcr->dev;
    JCR *jcr = dcr->jcr;
 
@@ -559,12 +566,14 @@ bool dir_ask_sysop_to_mount_volume(DCR *dcr)
          return false;
       }
 
+      if (dev->is_dvd()) {   
+         unmount_dev(dev, 0);
+      }
       if (!dev->poll) {
-         msg = _("Please mount");
-         Jmsg(jcr, M_MOUNT, 0, _("%s Volume \"%s\" on Storage Device %s for Job %s\n"),
-              msg, dcr->VolumeName, dev->print_name(), jcr->Job);
+         Jmsg(jcr, M_MOUNT, 0, _("Please mount Volume \"%s\" on Storage Device %s for Job %s\n"),
+              dcr->VolumeName, dev->print_name(), jcr->Job);
          Dmsg3(400, "Mount \"%s\" on device \"%s\" for Job %s\n",
-               dcr->VolumeName, dcr->dev_name, jcr->Job);
+               dcr->VolumeName, dev->print_name(), jcr->Job);
       }
 
       jcr->JobStatus = JS_WaitMount;
@@ -573,7 +582,7 @@ bool dir_ask_sysop_to_mount_volume(DCR *dcr)
       stat = wait_for_sysop(dcr);    ;     /* wait on device */
       if (dev->poll) {
          Dmsg1(400, "Poll timeout in mount vol on device %s\n", dev->print_name());
-         Dmsg1(400, "Blocked=%s\n", edit_blocked_reason(dev));
+         Dmsg1(400, "Blocked=%s\n", dev->print_blocked());
          return true;
       }