]> git.sur5r.net Git - bacula/bacula/commitdiff
- Pull in latest reservation system changes from 1.38
authorKern Sibbald <kern@sibbald.com>
Tue, 20 Dec 2005 13:40:06 +0000 (13:40 +0000)
committerKern Sibbald <kern@sibbald.com>
Tue, 20 Dec 2005 13:40:06 +0000 (13:40 +0000)
- Make .messages command always print messages regardless
  of the automessages flag.

git-svn-id: https://bacula.svn.sourceforge.net/svnroot/bacula/trunk@2693 91ce42f0-d328-0410-95d8-f526ca767f89

bacula/ReleaseNotes
bacula/kes-1.38
bacula/kes-1.39
bacula/src/dird/ua_dotcmds.c
bacula/src/dird/ua_output.c
bacula/src/stored/acquire.c
bacula/src/stored/bscan.c
bacula/src/stored/reserve.c
bacula/src/stored/status.c
bacula/src/stored/wait.c

index 8ed4df38272d4e1d8c06e896056e1a8eda63f939..a1616181dd8ed50af5a7b4beb12b7463d29416a0 100644 (file)
@@ -7,6 +7,19 @@
 Changes to 1.38.3:
 - This is mainly a bug release fix. In addition, the multiple drive
   reservation algorithm has been rewritten.
+- Implement load balancing code.
+- Rewrite reservation algorithm again. Rename variables to be
+  more logical, add HEARTBEAT with Director, allow cancel of
+  jobs stuck in reservation, add last resourt any_drive.
+- Fix seg fault if user labels a drive directly bug #513
+- Remove quotes around Version as it breaks things.
+- Merge in Aleksandar Milivojevic's mods to the spec file.
+- Apply sparse code fix for raw drives and fifos. Bug 506
+- Thorsten fixed Unicode cd problem with wx-console bug 505.
+- Correct reservation system to do a last ditch try
+  for any mounted volume, then anyone anywhere.
+- Updates to ssh-tunnel from Joshua Kugler.
+- Added a report.pl program from Jonas Bjorklund.            
 - Simplify the O_NONBLOCK open() code for tape drives,
   and always open nonblocking.
 - Do not wait for open() if EIO returned (shouldn't happen).
index 183dc4faabf3e6bfd5729e8642443cf222187c69..ced0fff2fefa9a7f765476f3dbe2f05fc24630a9 100644 (file)
@@ -3,9 +3,17 @@
 
 General:
 
+Beta release 20Dec05:
 Changes to 1.38.3:
+19Dec05
+- Implement load balancing code.
+- Pull a few files from 1.39 where we can maintain compatibility.
+- Rewrite reservation algorithm again. Rename variables to be
+  more logical, add HEARTBEAT with Director, allow cancel of
+  jobs stuck in reservation, add last resourt any_drive.
 17Dec05
-- Fix seg fault if user labels a drive directory bug #513
+- Remove quotes from Version table name -- it breaks things.
+- Fix seg fault if user labels a drive directly bug #513
 - Remove quotes around Version as it breaks things.
 16Dec05
 - Merge in Aleksandar Milivojevic's mods to the spec file.
index 9a613d77b17af6d76a1a38039462191cd6b6907f..0093df18f3dfd38d27d9826db4a5efb82a38dace 100644 (file)
@@ -4,6 +4,10 @@
 General:
 
 Changes to 1.39.3:
+20Dec05
+- Pull in latest reservation system changes from 1.38
+- Make .messages command always print messages regardless
+  of the automessages flag.
 17Dec05
 - Fix seg fault if user labels a drive directory bug #513
 - Remove quotes around Version as it breaks things.
index 16ad9210da2e4d78cd4cc01d28b8429a59b3f6a7..e693d215b779061ff41790a323db6d19a54162a4 100644 (file)
@@ -34,9 +34,10 @@ extern int r_last;
 extern struct s_res resources[];
 extern char my_name[];
 extern const char *client_backups;
+extern int console_msg_pending;
 
 /* Imported functions */
-extern int qmessagescmd(UAContext *ua, const char *cmd);
+extern int do_messages(UAContext *ua, const char *cmd);
 extern int quit_cmd(UAContext *ua, const char *cmd);
 extern int qhelp_cmd(UAContext *ua, const char *cmd);
 extern int qstatus_cmd(UAContext *ua, const char *cmd);
@@ -53,6 +54,7 @@ static int defaultscmd(UAContext *ua, const char *cmd);
 static int typescmd(UAContext *ua, const char *cmd);
 static int backupscmd(UAContext *ua, const char *cmd);
 static int levelscmd(UAContext *ua, const char *cmd);
+static int getmsgscmd(UAContext *ua, const char *cmd);
 
 struct cmdstruct { const char *key; int (*func)(UAContext *ua, const char *cmd); const char *help; };
 static struct cmdstruct commands[] = {
@@ -68,7 +70,7 @@ static struct cmdstruct commands[] = {
  { N_(".status"),     qstatus_cmd,  NULL},
  { N_(".storage"),    storagecmd,   NULL},
  { N_(".defaults"),   defaultscmd,  NULL},
- { N_(".messages"),   qmessagescmd, NULL},
+ { N_(".messages"),   getmsgscmd,   NULL},
  { N_(".help"),       qhelp_cmd,    NULL},
  { N_(".quit"),       quit_cmd,     NULL},
  { N_(".exit"),       quit_cmd,     NULL}
@@ -110,6 +112,14 @@ int do_a_dot_command(UAContext *ua, const char *cmd)
    return stat;
 }
 
+static int getmsgscmd(UAContext *ua, const char *cmd)
+{
+   if (console_msg_pending) {
+      do_messages(ua, cmd);
+   }
+   return 1;
+}
+
 /*
  * Create segmentation fault
  */
index 809b8009cfc405528087be741a4979a0724bb752..33a8343e5b68872e56c50c731f094898abe05ab7 100644 (file)
@@ -635,7 +635,7 @@ void do_messages(UAContext *ua, const char *cmd)
 {
    char msg[2000];
    int mlen;
-   int do_truncate = FALSE;
+   bool do_truncate = false;
 
    Pw(con_lock);
    pthread_cleanup_push(con_lock_release, (void *)NULL);
@@ -646,7 +646,7 @@ void do_messages(UAContext *ua, const char *cmd)
       strcpy(ua->UA_sock->msg, msg);
       ua->UA_sock->msglen = mlen;
       bnet_send(ua->UA_sock);
-      do_truncate = TRUE;
+      do_truncate = true;
    }
    if (do_truncate) {
       ftruncate(fileno(con_fd), 0L);
index b9d0338b4611c1de34aac5cbcf9a58449402ff76..16685138e8b9e2773dcb08e03777a341175f8400 100644 (file)
@@ -111,6 +111,7 @@ void free_dcr(DCR *dcr)
    }
    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);
 }
 
index 759f0edeb71d5cd3f5a34266d43ce70e0d640249..88a79c8760cd500304123cb541bc0a7859d609f4 100644 (file)
@@ -377,7 +377,7 @@ static bool record_cb(DCR *dcr, DEV_RECORD *rec)
 
    if (rec->data_len > 0) {
       mr.VolBytes += rec->data_len + WRITE_RECHDR_LENGTH; /* Accumulate Volume bytes */
-      if (showProgress) {
+      if (showProgress && currentVolumeSize > 0) {
          int pct = (mr.VolBytes * 100) / currentVolumeSize;
          if (pct != last_pct) {
             fprintf(stdout, _("done: %d%%\n"), pct);
index eba33ae08c7db5696b366dd3fded18c6170043ed..ab1c3ea02c51ed1e9072dd2a0220363f4b3768d7 100644 (file)
@@ -48,17 +48,20 @@ public:
 /* Reserve context */
 class RCTX {
 public:
-   alist *errors;
    JCR *jcr;
    char *device_name;
    DIRSTORE *store;
    DEVRES   *device;
-   bool PreferMountedVols;
-   bool exact_match;
-   bool have_volume;
-   bool do_not_wait;
-   bool available_autochanger;
-   char VolumeName[MAX_NAME_LENGTH];
+   DEVICE *low_use_drive;             /* Low use drive candidate */
+   int num_writers;                   /* for selecting low use drive */
+   bool try_low_use_drive;            /* see if low use drive available */
+   bool any_drive;                    /* Accept any drive if set */
+   bool PreferMountedVols;            /* Prefer volumes already mounted */
+   bool exact_match;                  /* Want exact volume */
+   bool have_volume;                  /* Have DIR suggested vol name */
+   bool suitable_device;              /* at least one device is suitable */
+   bool autochanger_only;             /* look at autochangers only */
+   char VolumeName[MAX_NAME_LENGTH];  /* Vol name suggested by DIR */
 };
 
 static dlist *vol_list = NULL;
@@ -284,6 +287,17 @@ bool is_volume_in_use(DCR *dcr)
 }
 
 
+/*
+ * We get the following type of information:
+ *
+ * use storage=xxx media_type=yyy pool_name=xxx pool_type=yyy append=1 copy=0 strip=0
+ *  use device=zzz
+ *  use device=aaa
+ *  use device=bbb
+ * use storage=xxx media_type=yyy pool_name=xxx pool_type=yyy append=0 copy=0 strip=0
+ *  use device=bbb
+ *
+ */
 static bool use_storage_cmd(JCR *jcr)
 {
    POOL_MEM store_name, dev_name, media_type, pool_name, pool_type;
@@ -293,9 +307,6 @@ static bool use_storage_cmd(JCR *jcr)
    int Copy, Stripe;
    DIRSTORE *store;
    RCTX rctx;
-#ifdef implemented
-   char *error;
-#endif
 
    memset(&rctx, 0, sizeof(RCTX));
    rctx.jcr = jcr;
@@ -362,82 +373,78 @@ static bool use_storage_cmd(JCR *jcr)
     * Wiffle through them and find one that can do the backup.
     */
    if (ok) {
-#ifdef OLD_ALGORITHM
-      /*
-       * First look for an exact match of Volume name as the
-       *  tape may already be mounted.
-       */
-      rctx.do_not_wait = true;
-      rctx.exact_match = true;
-      if ((ok = find_suitable_device_for_job(jcr, rctx))) {
-         goto done;
-      }
-      rctx.exact_match = false;
-
-      /* Now search if an unused autochanger slot is available */
-      rctx.available_autochanger = true;
-      if ((ok = find_suitable_device_for_job(jcr, rctx))) {
-         goto done;
-      }
-      rctx.available_autochanger = false;
-
-
-      /*
-       * Make up to two passes. The first with PreferMountedVols possibly
-       *   set to true.  In that case, we look only for an available 
-       *   drive with something mounted. If that fails, then we
-       *   do a second pass with PerferMountedVols set false.
-       */
-      rctx.PreferMountedVols = jcr->PreferMountedVols;
-      if (!rctx.PreferMountedVols) {
-         rctx.do_not_wait = false;
-      }
-      if ((ok = find_suitable_device_for_job(jcr, rctx))) {
-         goto done;
-      }
-      if (rctx.PreferMountedVols) {
-         rctx.PreferMountedVols = false;
-         rctx.do_not_wait = false;
+      bool first = true;           /* print wait message once */
+      for ( ; !job_canceled(jcr); ) {
+         rctx.suitable_device = false;
+         rctx.have_volume = false;
+         rctx.any_drive = false;
+         if (!jcr->PreferMountedVols) {
+            /* Look for unused drives in autochangers */
+            rctx.num_writers = 20000000;   /* start with impossible number */
+            rctx.low_use_drive = NULL;
+            rctx.PreferMountedVols = false;                
+            rctx.exact_match = false;
+            rctx.autochanger_only = true;
+            Dmsg5(100, "PrefMnt=%d exact=%d suitable=%d chgronly=%d any=%d\n",
+               rctx.PreferMountedVols, rctx.exact_match, rctx.suitable_device,
+               rctx.autochanger_only, rctx.any_drive);
+            if ((ok = find_suitable_device_for_job(jcr, rctx))) {
+               goto done;
+            }
+            /* Look through all drives possibly for low_use drive */
+            if (rctx.low_use_drive) {
+               rctx.try_low_use_drive = true;
+               if ((ok = find_suitable_device_for_job(jcr, rctx))) {
+                  goto done;
+               }
+               rctx.try_low_use_drive = false;
+            }
+            rctx.autochanger_only = false;
+            Dmsg5(100, "PrefMnt=%d exact=%d suitable=%d chgronly=%d any=%d\n",
+               rctx.PreferMountedVols, rctx.exact_match, rctx.suitable_device,
+               rctx.autochanger_only, rctx.any_drive);
+            if ((ok = find_suitable_device_for_job(jcr, rctx))) {
+               goto done;
+            }
+         }
+         /* Look for an exact match all drives */
+         rctx.PreferMountedVols = true;
+         rctx.exact_match = true;
+         rctx.autochanger_only = false;
+         Dmsg5(100, "PrefMnt=%d exact=%d suitable=%d chgronly=%d any=%d\n",
+            rctx.PreferMountedVols, rctx.exact_match, rctx.suitable_device,
+            rctx.autochanger_only, rctx.any_drive);
          if ((ok = find_suitable_device_for_job(jcr, rctx))) {
             goto done;
          }
-      }
-#else /* NEW SEARCH ALGORITHM */ 
-      rctx.do_not_wait = true;
-      if (!jcr->PreferMountedVols) {
-         /* Look for unused drives in autochangers */
-         rctx.PreferMountedVols = false;                
+         /* Look for any mounted drive */
          rctx.exact_match = false;
-         rctx.available_autochanger = true;
+         Dmsg5(100, "PrefMnt=%d exact=%d suitable=%d chgronly=%d any=%d\n",
+            rctx.PreferMountedVols, rctx.exact_match, rctx.suitable_device,
+            rctx.autochanger_only, rctx.any_drive);
          if ((ok = find_suitable_device_for_job(jcr, rctx))) {
             goto done;
          }
-         /* Look through all drives */
-         rctx.available_autochanger = false;
+         /* Try any drive */
+         rctx.any_drive = true;
+         Dmsg5(100, "PrefMnt=%d exact=%d suitable=%d chgronly=%d any=%d\n",
+            rctx.PreferMountedVols, rctx.exact_match, rctx.suitable_device,
+            rctx.autochanger_only, rctx.any_drive);
          if ((ok = find_suitable_device_for_job(jcr, rctx))) {
             goto done;
          }
+         if (!rctx.suitable_device || !wait_for_device(jcr, first)) {
+            break;       /* Get out, failure ... */
+         }   
+         first = false;
+         bnet_sig(dir, BNET_HEARTBEAT);  /* Inform Dir that we are alive */
       }
-      /* Look for an exact match all drives */
-      rctx.PreferMountedVols = true;
-      rctx.exact_match = true;
-      rctx.available_autochanger = false;
-      if ((ok = find_suitable_device_for_job(jcr, rctx))) {
-         goto done;
-      }
-      /* Look for any mounted drive */
-      rctx.exact_match = false;
-      if ((ok = find_suitable_device_for_job(jcr, rctx))) {
-         goto done;
-      }
-      /* Wait for any drive anywhere */
-      rctx.PreferMountedVols = false;
-      rctx.exact_match = false;
-      rctx.do_not_wait = false;
-      if ((ok = find_suitable_device_for_job(jcr, rctx))) {
-         goto done;
-      }
-#endif
+
+      /*
+       * If we get here, there are no suitable devices available, which
+       *  means nothing configured.  If a device is suitable but busy
+       *  with another Volume, we will not come here.
+       */
       if (verbose) {
          unbash_spaces(dir->msg);
          pm_strcpy(jcr->errmsg, dir->msg);
@@ -447,12 +454,7 @@ static bool use_storage_cmd(JCR *jcr)
          "     Device \"%s\" with MediaType \"%s\" requested by DIR not found in SD Device resources.\n"),
            dev_name.c_str(), media_type.c_str());
       bnet_fsend(dir, NO_device, dev_name.c_str());
-#ifdef implemented
-      for (error=(char*)rctx->errors.first(); error;
-           error=(char*)rctx->errors.next()) {
-         Jmsg(jcr, M_INFO, 0, "%s", error);
-      }
-#endif
+
       Dmsg1(100, ">dird: %s", dir->msg);
    } else {
       unbash_spaces(dir->msg);
@@ -471,8 +473,7 @@ done:
    }
    delete jcr->dirstore;
 #ifdef implemented
-   for (error=(char*)rctx->errors.first(); error;
-        error=(char*)rctx->errors.next()) {
+   while (error=(char*)rctx->errors.first()) {
       free(error);
    }
 #endif
@@ -485,56 +486,41 @@ done:
  */
 bool find_suitable_device_for_job(JCR *jcr, RCTX &rctx)
 {
-   bool first = true;
    bool ok;
    DIRSTORE *store;
    char *device_name;
 
-   Dmsg4(100, "PrefMnt=%d exact=%d no_wait=%d availchgr=%d\n",
-      rctx.PreferMountedVols, rctx.exact_match, rctx.do_not_wait,
-      rctx.available_autochanger);
+   /* 
+    * For each storage device that the user specified, we
+    *  search and see if there is a resource for that device.
+    */
+   Dmsg4(100, "PrefMnt=%d exact=%d suitable=%d chgronly=%d\n",
+      rctx.PreferMountedVols, rctx.exact_match, rctx.suitable_device,
+      rctx.autochanger_only);
    init_jcr_device_wait_timers(jcr);
-   for ( ;; ) {
-      int can_wait = false;
-      ok = false;
-      P(search_lock);
-      foreach_alist(store, jcr->dirstore) {
-         rctx.store = store;
-         foreach_alist(device_name, store->device) {
-            int stat;
-            rctx.device_name = device_name;
-            stat = search_res_for_device(rctx); 
-            if (stat == 1) {             /* found available device */
-               ok = true;
-               break;
-            } else if (stat == 0) {      /* device busy */
-               can_wait = true;
-            }
-            /* otherwise error */
-//             rctx->errors.push(bstrdup(jcr->errmsg));
-         }
-         if (ok) {
+   ok = false;
+   P(search_lock);
+   foreach_alist(store, jcr->dirstore) {
+      rctx.store = store;
+      foreach_alist(device_name, store->device) {
+         int stat;
+         rctx.device_name = device_name;
+         stat = search_res_for_device(rctx); 
+         if (stat == 1) {             /* found available device */
+            rctx.suitable_device = true;
+            ok = true;
             break;
+         } else if (stat == 0) {      /* device busy */
+            rctx.suitable_device = true;   /* but it is busy, so continue looking */
          }
+         /* otherwise error */
       }
-      V(search_lock);
-      /*
-       * We did not find a suitable device, so
-       *  if there is some device for which we can wait, then
-       *  wait and try again until the wait time expires
-       */
-      if (rctx.do_not_wait || !can_wait || !wait_for_device(jcr, first)) {
+      if (ok) {
          break;
       }
-      first = false;                  /* first wait complete */
-
-#ifdef implemented
-      for (error=(char*)rctx->errors.first(); error;
-           error=(char*)rctx->errors.next()) {
-         free(error);
-      }
-#endif
    }
+   V(search_lock);
+
    return ok;
 }
 
@@ -550,25 +536,9 @@ static int search_res_for_device(RCTX &rctx)
    int stat;
 
    Dmsg1(100, "Search res for %s\n", rctx.device_name);
-   if (!rctx.available_autochanger) {
-      foreach_res(rctx.device, R_DEVICE) {
-         Dmsg1(100, "Try res=%s\n", rctx.device->hdr.name);
-         /* Find resource, and make sure we were able to open it */
-         if (fnmatch(rctx.device_name, rctx.device->hdr.name, 0) == 0) {
-            stat = reserve_device(rctx);
-            if (stat != 1) {
-               return stat;
-            }
-            Dmsg1(220, "Got: %s", dir->msg);
-            bash_spaces(rctx.device_name);
-            ok = bnet_fsend(dir, OK_device, rctx.device_name);
-            Dmsg1(100, ">dird dev: %s", dir->msg);
-            return ok ? 1 : -1;
-         }
-      }
-   }
+   /* Look through Autochangers first */
    foreach_res(changer, R_AUTOCHANGER) {
-      Dmsg1(100, "Try changer res=%s\n", changer->hdr.name);
+      Dmsg1(150, "Try match changer res=%s\n", changer->hdr.name);
       /* Find resource, and make sure we were able to open it */
       if (fnmatch(rctx.device_name, changer->hdr.name, 0) == 0) {
          /* Try each device in this AutoChanger */
@@ -594,6 +564,25 @@ static int search_res_for_device(RCTX &rctx)
          }
       }
    }
+
+   /* Now if requested look through regular devices */
+   if (!rctx.autochanger_only) {
+      foreach_res(rctx.device, R_DEVICE) {
+         Dmsg1(150, "Try match res=%s\n", rctx.device->hdr.name);
+         /* Find resource, and make sure we were able to open it */
+         if (fnmatch(rctx.device_name, rctx.device->hdr.name, 0) == 0) {
+            stat = reserve_device(rctx);
+            if (stat != 1) {
+               return stat;
+            }
+            Dmsg1(220, "Got: %s", dir->msg);
+            bash_spaces(rctx.device_name);
+            ok = bnet_fsend(dir, OK_device, rctx.device_name);
+            Dmsg1(100, ">dird dev: %s", dir->msg);
+            return ok ? 1 : -1;
+         }
+      }
+   }
    return 0;                    /* nothing found */
 }
 
@@ -615,6 +604,7 @@ static int reserve_device(RCTX &rctx)
       return -1;
    }
 
+   /* Make sure device exists -- i.e. we can stat() it */
    if (!rctx.device->dev) {
       rctx.device->dev = init_dev(rctx.jcr, rctx.device);
    }
@@ -630,6 +620,7 @@ static int reserve_device(RCTX &rctx)
       }
       return -1;  /* no use waiting */
    }  
+
    Dmsg2(100, "Try reserve %s jobid=%d\n", rctx.device->hdr.name,
          rctx.jcr->JobId);
    dcr = new_dcr(rctx.jcr, rctx.device->dev);
@@ -745,12 +736,14 @@ static bool reserve_device_for_append(DCR *dcr, RCTX &rctx)
 
    P(dev->mutex);
 
+   /* If device is being read, we cannot write it */
    if (dev->can_read()) {
       Mmsg1(jcr->errmsg, _("Device %s is busy reading.\n"), dev->print_name());
       Dmsg1(100, "%s", jcr->errmsg);
       goto bail_out;
    }
 
+   /* If device is unmounted, we are out of luck */
    if (is_device_unmounted(dev)) {
       Mmsg(jcr->errmsg, _("Device %s is BLOCKED due to user unmount.\n"), dev->print_name());
       Dmsg1(100, "%s", jcr->errmsg);
@@ -759,6 +752,7 @@ static bool reserve_device_for_append(DCR *dcr, RCTX &rctx)
 
    Dmsg1(100, "reserve_append device is %s\n", dev->is_tape()?"tape":"disk");
 
+   /* Now do detailed tests ... */
    if (can_reserve_drive(dcr, rctx) != 1) {
       Dmsg0(100, "can_reserve_drive!=1\n");
       goto bail_out;
@@ -778,37 +772,63 @@ bail_out:
 /*
  * Returns: 1 if drive can be reserved
  *          0 if we should wait
- *         -1 on error
+ *         -1 on error or impossibility
  */
 static int can_reserve_drive(DCR *dcr, RCTX &rctx) 
 {
    DEVICE *dev = dcr->dev;
    JCR *jcr = dcr->jcr;
 
-   /* If he wants a free drive, but this one is busy, no go */
-   if (!rctx.PreferMountedVols && dev->is_busy()) {
-      Dmsg1(100, "!prefMnt && busy. JobId=%d\n", jcr->JobId);
-      return 0;
-   }
-   Dmsg3(100, "can_reserve_drive: prefmnt=%d busy=%d res=%d\n", rctx.PreferMountedVols, 
-      dev->is_busy(), dev->reserved_device);
+   Dmsg5(100, "PrefMnt=%d exact=%d suitable=%d chgronly=%d any=%d\n",
+         rctx.PreferMountedVols, rctx.exact_match, rctx.suitable_device,
+         rctx.autochanger_only, rctx.any_drive);
 
-   /* Check for prefer mounted volumes */
-   if (rctx.PreferMountedVols && !dev->VolHdr.VolumeName[0] && dev->is_tape()) {
-      Dmsg1(100, "failed: want mounted -- no vol JobId=%d\n", jcr->JobId);
-      return 0;                 /* No volume mounted */
-   }
+   /* setting any_drive overrides PreferMountedVols flag */
+   if (!rctx.any_drive) {
+      /*
+       * When PreferMountedVols is set, we keep track of the 
+       *  drive in use that has the least number of writers, then if
+       *  no unmounted drive is found, we try that drive. This   
+       *  helps spread the load to the least used drives.  
+       */
+      if (rctx.try_low_use_drive && dev == rctx.low_use_drive) {
+         Dmsg3(100, "OK dev=%s == low_drive=%s. JobId=%d\n",
+            dev->print_name(), rctx.low_use_drive->print_name(), jcr->JobId);
+         return 1;
+      }
+      /* If he wants a free drive, but this one is busy, no go */
+      if (!rctx.PreferMountedVols && dev->is_busy()) {
+         /* Save least used drive */
+         if ((dev->num_writers + dev->reserved_device) < rctx.num_writers) {
+            rctx.num_writers = dev->num_writers + dev->reserved_device;
+            rctx.low_use_drive = dev;
+            Dmsg2(100, "set low use drive=%s num_writers=%d\n", dev->print_name(),
+               rctx.num_writers);
+         } else {
+            Dmsg1(100, "not low use num_writers=%d\n", dev->num_writers+ 
+               dev->reserved_device);
+         }
+         Dmsg1(100, "failed: !prefMnt && busy. JobId=%d\n", jcr->JobId);
+         return 0;
+      }
 
-   /* Check for exact Volume name match */
-   if (rctx.exact_match && rctx.have_volume &&
-       strcmp(dev->VolHdr.VolumeName, rctx.VolumeName) != 0) {
-      Dmsg2(100, "failed: Not exact match have=%s want=%s\n",
-            dev->VolHdr.VolumeName, rctx.VolumeName);
-      return 0;
+      /* Check for prefer mounted volumes */
+      if (rctx.PreferMountedVols && !dev->VolHdr.VolumeName[0] && dev->is_tape()) {
+         Dmsg1(100, "failed: want mounted -- no vol JobId=%d\n", jcr->JobId);
+         return 0;                 /* No volume mounted */
+      }
+
+      /* Check for exact Volume name match */
+      if (rctx.exact_match && rctx.have_volume &&
+          strcmp(dev->VolHdr.VolumeName, rctx.VolumeName) != 0) {
+         Dmsg2(100, "failed: Not exact match have=%s want=%s\n",
+               dev->VolHdr.VolumeName, rctx.VolumeName);
+         return 0;
+      }
    }
 
    /* Check for unused autochanger drive */
-   if (rctx.available_autochanger && dev->num_writers == 0 &&
+   if (rctx.autochanger_only && dev->num_writers == 0 &&
        dev->VolHdr.VolumeName[0] == 0) {
       /* Device is available but not yet reserved, reserve it for us */
       Dmsg2(100, "OK Res Unused autochanger %s JobId=%d.\n",
@@ -828,7 +848,7 @@ static int can_reserve_drive(DCR *dcr, RCTX &rctx)
          if (strcmp(dev->pool_name, dcr->pool_name) == 0 &&
              strcmp(dev->pool_type, dcr->pool_type) == 0) {
             /* OK, compatible device */
-            Dmsg2(100, "OK got dev: %s num_writers=0, reserved, pool matches JobId=%d\n",
+            Dmsg2(100, "OK dev: %s num_writers=0, reserved, pool matches JobId=%d\n",
                dev->print_name(), jcr->JobId);
             return 1;
          } else {
@@ -841,13 +861,13 @@ static int can_reserve_drive(DCR *dcr, RCTX &rctx)
          /* Device in append mode, check if changing pool */
          if (strcmp(dev->pool_name, dcr->pool_name) == 0 &&
              strcmp(dev->pool_type, dcr->pool_type) == 0) {
-            Dmsg2(100, "OK got dev: %s num_writers=0, can_append, pool matches. JobId=%d\n",
+            Dmsg2(100, "OK dev: %s num_writers=0, can_append, pool matches. JobId=%d\n",
                dev->print_name(), jcr->JobId);
             /* OK, compatible device */
             return 1;
          } else {
             /* Changing pool, unload old tape if any in drive */
-            Dmsg0(100, "got dev: num_writers=0, not reserved, pool change, unload changer\n");
+            Dmsg0(100, "OK dev: num_writers=0, not reserved, pool change, unload changer\n");
             unload_autochanger(dcr, 0);
          }
       }
@@ -867,7 +887,7 @@ static int can_reserve_drive(DCR *dcr, RCTX &rctx)
       /* 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) {
-         Dmsg2(100, "OK got dev: %s num_writers>=0, can_append, pool matches. JobId=%d\n",
+         Dmsg2(100, "OK dev: %s num_writers>=0, can_append, pool matches. JobId=%d\n",
             dev->print_name(), jcr->JobId);
          /* OK, compatible device */
          return 1;
index 6be4e9a6cd0f6ad162d9e789229ed61bd30f781d..46b58899dfddf694fc00d485e8b5e703294e1dae 100644 (file)
@@ -160,13 +160,12 @@ bool status_cmd(JCR *jcr)
    list_volumes(user);
    bnet_fsend(user, _("====\n\n"));
        
-
 #ifdef xxx
-   if (debug_level > 0) {
+   if (debug_level > 10) {
       bnet_fsend(user, _("====\n\n"));
       dump_resource(R_DEVICE, resources[R_DEVICE-r_first].res_head, sendit, user);
+      bnet_fsend(user, _("====\n\n"));
    }
-   bnet_fsend(user, _("====\n\n"));
 #endif
 
    list_spool_stats(user);
index b08e80f46e562c0e6ea9bf002096e17813b3406c..ba7b203ff4d9dbc0cc86b8529309cda05832483f 100644 (file)
@@ -168,6 +168,7 @@ bool wait_for_device(JCR *jcr, bool first)
    struct timespec timeout;
    int stat = 0;
    bool ok = true;
+   const int wait_time = 10 * 60;       /* wait 10 minutes */
 
    Dmsg0(100, "Enter wait_for_device\n");
    P(device_release_mutex);
@@ -178,7 +179,7 @@ bool wait_for_device(JCR *jcr, bool first)
 
    gettimeofday(&tv, &tz);
    timeout.tv_nsec = tv.tv_usec * 1000;
-   timeout.tv_sec = tv.tv_sec + 120;
+   timeout.tv_sec = tv.tv_sec + wait_time;
 
    Dmsg0(100, "I'm going to wait for a device.\n");