]> git.sur5r.net Git - bacula/bacula/commitdiff
kes Add read volume list code to SD -- not yet used.
authorKern Sibbald <kern@sibbald.com>
Tue, 21 Oct 2008 21:06:13 +0000 (21:06 +0000)
committerKern Sibbald <kern@sibbald.com>
Tue, 21 Oct 2008 21:06:13 +0000 (21:06 +0000)
kes  Add James' binutils patch

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

bacula/src/stored/acquire.c
bacula/src/stored/btape.c
bacula/src/stored/butil.c
bacula/src/stored/dev.h
bacula/src/stored/parse_bsr.c
bacula/src/stored/protos.h
bacula/src/stored/stored.c
bacula/src/stored/vol_mgr.c
bacula/src/version.h
bacula/technotes-2.5

index e40c7f3f6f42d8bbe55d61cd9cc46ca8d8fb4033..f4203cd3956f9f6ca5dd2ff5296d176f76bbb14d 100644 (file)
@@ -436,6 +436,7 @@ bool release_device(DCR *dcr)
          dev->is_labeled(), vol->VolCatName);
       if (dev->is_labeled() && vol->VolCatName[0] != 0) {
          dir_update_volume_info(dcr, false, false); /* send Volume info to Director */
+         remove_read_volume(jcr, dcr->VolumeName);
          volume_unused(dcr);
       }
    } else if (dev->num_writers > 0) {
index 4aa6a07da91c21cb074e29cc29474fbed74ed844..1908cbda0515d4f45378c3d87ebe87a620641c19 100644 (file)
@@ -344,7 +344,7 @@ static void terminate_btape(int stat)
    free_jcr(jcr);
    jcr = NULL;
 
-   free_volume_list();
+   free_volume_lists();
 
    if (dev) {
       dev->term();
index 034a4c100e96328b3f35252ec74ff4bc7986308a..adf41d88b329741080966a273dfaaa63e1fc26eb 100644 (file)
@@ -107,7 +107,7 @@ JCR *setup_jcr(const char *name, char *dev_name, BSR *bsr,
    pm_strcpy(jcr->fileset_md5, "Dummy.fileset.md5");
 
    init_autochangers();
-   create_volume_list();
+   create_volume_lists();
 
    dcr = setup_to_access_device(jcr, dev_name, VolumeName, mode);
    if (!dcr) {
index 4fb41efe53ecb0277e2f187cf6aa03cb82cdf748..57adacce32f116b08f5e0f001721a800c79c736d 100644 (file)
@@ -564,6 +564,7 @@ class VOLRES {
    bool m_swapping;                   /* set when swapping to another drive */
    bool m_in_use;                     /* set when volume reserved or in use */
    int32_t m_slot;                    /* slot of swapping volume */
+   uint32_t m_JobId;                  /* JobId for read volumes */
 public:
    dlink link;
    char *vol_name;                    /* Volume name */
@@ -578,6 +579,8 @@ public:
    void set_slot(int32_t slot) { m_slot = slot; };
    void clear_slot() { m_slot = -1; };
    int32_t get_slot() const { return m_slot; };
+   uint32_t get_jobid() const { return m_JobId; };
+   void set_jobid(uint32_t JobId) { m_JobId = JobId; };
 };
 
 
index fd3148a844733e1d4db89c52cef79e3621759c05..595b1fea731548f5cc9d61403bb0fc7a10fb3d12 100644 (file)
@@ -877,6 +877,9 @@ static bool add_restore_volume(JCR *jcr, VOL_LIST *vol)
 {
    VOL_LIST *next = jcr->VolList;
 
+   /* Add volume to volume manager's read list */
+   add_read_volume(jcr, vol->VolumeName);
+
    if (!next) {                       /* list empty ? */
       jcr->VolList = vol;             /* yes, add volume */
    } else {
@@ -904,13 +907,14 @@ static bool add_restore_volume(JCR *jcr, VOL_LIST *vol)
 
 void free_restore_volume_list(JCR *jcr)
 {
-   VOL_LIST *next = jcr->VolList;
+   VOL_LIST *vol = jcr->VolList;
    VOL_LIST *tmp;
 
-   for ( ; next; ) {
-      tmp = next->next;
-      free(next);
-      next = tmp;
+   for ( ; vol; ) {
+      tmp = vol->next;
+      remove_read_volume(jcr, vol->VolumeName);
+      free(vol);
+      vol = tmp;
    }
    jcr->VolList = NULL;
 }
index 4ed9ef281f6afe6329460a00072912e91fd01aa6..8902bf81ddb260c331dfe8ebb12d378aa9314873 100644 (file)
@@ -261,13 +261,6 @@ extern int reservations_lock_count;
 #define unlock_reservations() _unlock_reservations()
 #define lock_volumes() _lock_volumes()
 #define unlock_volumes() _unlock_volumes()
-bool    volume_unused(DCR *dcr);
-void    create_volume_list();
-void    free_volume_list();
-void    list_volumes(void sendit(const char *msg, int len, void *sarg), void *arg);
-bool    is_volume_in_use(DCR *dcr);
-void debug_list_volumes(const char *imsg);
-extern int vol_list_lock_count;
 
 #endif
 
@@ -280,6 +273,15 @@ bool    free_volume(DEVICE *dev);
 bool    is_vol_list_empty();
 dlist  *dup_vol_list(JCR *jcr);
 void    free_temp_vol_list(dlist *temp_vol_list);
+bool    volume_unused(DCR *dcr);
+void    create_volume_lists();
+void    free_volume_lists();
+void    list_volumes(void sendit(const char *msg, int len, void *sarg), void *arg);
+bool    is_volume_in_use(DCR *dcr);
+void    debug_list_volumes(const char *imsg);
+extern  int vol_list_lock_count;
+void    add_read_volume(JCR *jcr, const char *VolumeName);
+void    remove_read_volume(JCR *jcr, const char *VolumeName);
 
 
 /* From spool.c */
index 3368cbdd6bd6884ac0b656baea1619eaf4e8d5f0..700bd31c88b904060a4413f7e6d44f37dc0a5a76 100644 (file)
@@ -284,7 +284,7 @@ int main (int argc, char *argv[])
     /*
      * Start the device allocation thread
      */
-   create_volume_list();              /* do before device_init */
+   create_volume_lists();             /* do before device_init */
    if (pthread_create(&thid, NULL, device_initialization, NULL) != 0) {
       berrno be;
       Emsg1(M_ABORT, 0, _("Unable to create thread. ERR=%s\n"), be.bstrerror());
@@ -625,7 +625,7 @@ void terminate_stored(int sig)
 
    Dmsg1(200, "In terminate_stored() sig=%d\n", sig);
 
-   free_volume_list();
+   free_volume_lists();
 
    foreach_res(device, R_DEVICE) {
       Dmsg1(10, "Term device %s\n", device->device_name);
index 17dba76950738f8e63a194500c93ae14b401eab5..f48dddb58fc5af492e70568812d210114397dc83 100644 (file)
@@ -43,16 +43,39 @@ const int dbglvl =  50;
 
 static dlist *vol_list = NULL;
 static brwlock_t vol_list_lock;
+static dlist *read_vol_list = NULL;
+static pthread_mutex_t read_vol_lock = PTHREAD_MUTEX_INITIALIZER;
 
 /* Forward referenced functions */
 static void free_vol_item(VOLRES *vol);
+static VOLRES *new_vol_item(DCR *dcr, const char *VolumeName);
 
-
+/*
+ * For append volumes the key is the VolumeName.
+ */
 static int my_compare(void *item1, void *item2)
 {
    return strcmp(((VOLRES *)item1)->vol_name, ((VOLRES *)item2)->vol_name);
 }
 
+/*
+ * For read volumes the key is JobId, VolumeName.
+ */
+static int read_compare(void *item1, void *item2)
+{
+   VOLRES *vol1 = (VOLRES *)item1;
+   VOLRES *vol2 = (VOLRES *)item2;
+
+   if (vol1->get_jobid() == vol2->get_jobid()) {
+      return strcmp(vol1->vol_name, vol2->vol_name);
+   }
+   if (vol1->get_jobid() < vol2->get_jobid()) {
+      return -1;
+   }
+   return 1;
+}
+
+
 bool is_vol_list_empty() 
 {
    return vol_list->empty();
@@ -60,6 +83,9 @@ bool is_vol_list_empty()
 
 int vol_list_lock_count = 0;
 
+/*
+ *  Initialized the main volume list. Note, we are using a recursive lock.
+ */
 void init_vol_list_lock()
 {
    int errstat;
@@ -102,60 +128,63 @@ void _unlock_volumes()
    }
 }
 
-dlist *dup_vol_list(JCR *jcr)
+void lock_read_volumes()
 {
-   dlist *temp_vol_list;
-   VOLRES *vol = NULL;
+   P(read_vol_lock);
+}
 
-   lock_volumes();
-   Dmsg0(dbglvl, "lock volumes\n");                           
+void unlock_read_volumes()
+{
+   V(read_vol_lock);
+}
 
-   /*  
-    * Create a temporary copy of the volume list.  We do this,
-    *   to avoid having the volume list locked during the
-    *   call to reserve_device(), which would cause a deadlock.
-    * Note, we may want to add an update counter on the vol_list
-    *   so that if it is modified while we are traversing the copy
-    *   we can take note and act accordingly (probably redo the 
-    *   search at least a few times).
-    */
-   Dmsg0(dbglvl, "duplicate vol list\n");
-   temp_vol_list = New(dlist(vol, &vol->link));
-   foreach_dlist(vol, vol_list) {
-      VOLRES *nvol;
-      VOLRES *tvol = (VOLRES *)malloc(sizeof(VOLRES));
-      memset(tvol, 0, sizeof(VOLRES));
-      tvol->vol_name = bstrdup(vol->vol_name);
-      tvol->dev = vol->dev;
-      nvol = (VOLRES *)temp_vol_list->binary_insert(tvol, my_compare);
-      if (tvol != nvol) {
-         tvol->dev = NULL;                   /* don't zap dev entry */
-         free_vol_item(tvol);
-         Pmsg0(000, "Logic error. Duplicating vol list hit duplicate.\n");
-         Jmsg(jcr, M_WARNING, 0, "Logic error. Duplicating vol list hit duplicate.\n");
-      }
+/*
+ * Add a volume to the read list.
+ * Note, we use VOLRES because it simplifies the code
+ *   even though, the only part of VOLRES that we need is
+ *   the volume name.  The same volume may be in the list
+ *   multiple times, but each one is distinguished by the 
+ *   JobId.  We use JobId, VolumeName as the key.
+ * We can get called multiple times for the same volume because
+ *   when parsing the bsr, the volume name appears multiple times.
+ */
+void add_read_volume(JCR *jcr, const char *VolumeName)
+{
+   VOLRES *nvol, *vol;
+
+   lock_read_volumes();
+   nvol = new_vol_item(NULL, VolumeName);
+   nvol->set_jobid(jcr->JobId);
+   vol = (VOLRES *)read_vol_list->binary_insert(nvol, read_compare);
+   if (vol != nvol) {
+      free_vol_item(nvol);
+      Dmsg2(1, "read_vol=%s JobId=%d already in list.\n", VolumeName, jcr->JobId);
+   } else {
+      Dmsg2(1, "add read_vol=%s JobId=%d\n", VolumeName, jcr->JobId);
    }
-   Dmsg0(dbglvl, "unlock volumes\n");
-   unlock_volumes();
-   return temp_vol_list;
+   unlock_read_volumes();
 }
 
-void free_temp_vol_list(dlist *temp_vol_list)
+/*
+ * Remove a given volume name from the read list.
+ */
+void remove_read_volume(JCR *jcr, const char *VolumeName)
 {
-   dlist *save_vol_list;
-   
-   lock_volumes();
-   save_vol_list = vol_list;
-   vol_list = temp_vol_list;
-   free_volume_list();                  /* release temp_vol_list */
-   vol_list = save_vol_list;
-   Dmsg0(dbglvl, "deleted temp vol list\n");
-   Dmsg0(dbglvl, "unlock volumes\n");
-   unlock_volumes();
-   debug_list_volumes("after free temp table");
+   VOLRES vol, *fvol;
+   lock_read_volumes();
+   vol.vol_name = bstrdup(VolumeName);
+   vol.set_jobid(jcr->JobId);
+   fvol = (VOLRES *)read_vol_list->binary_search(&vol, read_compare);
+   free(vol.vol_name);
+   Dmsg3(1, "remove_read_vol=%s JobId=%d found=%d\n", VolumeName, jcr->JobId, fvol!=NULL);
+   debug_list_volumes("remove_read_volume");
+   if (fvol) {
+      read_vol_list->remove(fvol);
+      free_vol_item(fvol);
+   }
+   unlock_read_volumes();
 }
 
-
 /*
  * List Volumes -- this should be moved to status.c
  */
@@ -211,6 +240,15 @@ void list_volumes(void sendit(const char *msg, int len, void *sarg), void *arg)
       }
    }
    unlock_volumes();
+
+   lock_read_volumes();
+   foreach_dlist(vol, read_vol_list) {
+      len = Mmsg(msg, "%s read volume JobId=%d\n", vol->vol_name, 
+            vol->get_jobid());
+      sendit(msg.c_str(), len, arg);
+   }
+   unlock_read_volumes();
+
 }
 
 /*
@@ -223,9 +261,11 @@ static VOLRES *new_vol_item(DCR *dcr, const char *VolumeName)
    vol = (VOLRES *)malloc(sizeof(VOLRES));
    memset(vol, 0, sizeof(VOLRES));
    vol->vol_name = bstrdup(VolumeName);
-   vol->dev = dcr->dev;
-   Dmsg3(dbglvl, "new Vol=%s at %p dev=%s\n",
-         VolumeName, vol->vol_name, vol->dev->print_name());
+   if (dcr) {
+      vol->dev = dcr->dev;
+      Dmsg3(dbglvl, "new Vol=%s at %p dev=%s\n",
+            VolumeName, vol->vol_name, vol->dev->print_name());
+   }
    return vol;
 }
 
@@ -545,34 +585,62 @@ bool free_volume(DEVICE *dev)
 
       
 /* Create the Volume list */
-void create_volume_list()
+void create_volume_lists()
 {
    VOLRES *vol = NULL;
    if (vol_list == NULL) {
       vol_list = New(dlist(vol, &vol->link));
    }
+   if (read_vol_list == NULL) {
+      read_vol_list = New(dlist(vol, &vol->link));
+   }
 }
 
-/* Release all Volumes from the list */
-void free_volume_list()
+/*
+ * Free normal append volumes list
+ */
+static void free_volume_list()
 {
    VOLRES *vol;
-   if (!vol_list) {
-      return;
+   if (vol_list) {
+      lock_volumes();
+      foreach_dlist(vol, vol_list) {
+         if (vol->dev) {
+            Dmsg2(dbglvl, "free vol_list Volume=%s dev=%s\n", vol->vol_name, vol->dev->print_name());
+         } else {
+            Dmsg1(dbglvl, "free vol_list Volume=%s No dev\n", vol->vol_name);
+         }
+         free(vol->vol_name);
+         vol->vol_name = NULL;
+      }
+      delete vol_list;
+      vol_list = NULL;
+      unlock_volumes();
    }
-   lock_volumes();
-   foreach_dlist(vol, vol_list) {
-      if (vol->dev) {
-         Dmsg2(dbglvl, "free vol_list Volume=%s dev=%s\n", vol->vol_name, vol->dev->print_name());
-      } else {
-         Dmsg1(dbglvl, "free vol_list Volume=%s No dev\n", vol->vol_name);
+}
+
+/* Release all Volumes from the list */
+void free_volume_lists()
+{
+   VOLRES *vol;
+
+   free_volume_list();           /* normal append list */
+
+   if (read_vol_list) {
+      lock_read_volumes();
+      foreach_dlist(vol, read_vol_list) {
+         if (vol->dev) {
+            Dmsg2(dbglvl, "free read_vol_list Volume=%s dev=%s\n", vol->vol_name, vol->dev->print_name());
+         } else {
+            Dmsg1(dbglvl, "free read_vol_list Volume=%s No dev\n", vol->vol_name);
+         }
+         free(vol->vol_name);
+         vol->vol_name = NULL;
       }
-      free(vol->vol_name);
-      vol->vol_name = NULL;
+      delete read_vol_list;
+      read_vol_list = NULL;
+      unlock_read_volumes();
    }
-   delete vol_list;
-   vol_list = NULL;
-   unlock_volumes();
 }
 
 bool DCR::can_i_use_volume()
@@ -610,3 +678,59 @@ get_out:
    return rtn;
 
 }
+
+/*  
+ * Create a temporary copy of the volume list.  We do this,
+ *   to avoid having the volume list locked during the
+ *   call to reserve_device(), which would cause a deadlock.
+ * Note, we may want to add an update counter on the vol_list
+ *   so that if it is modified while we are traversing the copy
+ *   we can take note and act accordingly (probably redo the 
+ *   search at least a few times).
+ */
+dlist *dup_vol_list(JCR *jcr)
+{
+   dlist *temp_vol_list;
+   VOLRES *vol = NULL;
+
+   lock_volumes();
+   Dmsg0(dbglvl, "lock volumes\n");                           
+
+   Dmsg0(dbglvl, "duplicate vol list\n");
+   temp_vol_list = New(dlist(vol, &vol->link));
+   foreach_dlist(vol, vol_list) {
+      VOLRES *nvol;
+      VOLRES *tvol = (VOLRES *)malloc(sizeof(VOLRES));
+      memset(tvol, 0, sizeof(VOLRES));
+      tvol->vol_name = bstrdup(vol->vol_name);
+      tvol->dev = vol->dev;
+      nvol = (VOLRES *)temp_vol_list->binary_insert(tvol, my_compare);
+      if (tvol != nvol) {
+         tvol->dev = NULL;                   /* don't zap dev entry */
+         free_vol_item(tvol);
+         Pmsg0(000, "Logic error. Duplicating vol list hit duplicate.\n");
+         Jmsg(jcr, M_WARNING, 0, "Logic error. Duplicating vol list hit duplicate.\n");
+      }
+   }
+   Dmsg0(dbglvl, "unlock volumes\n");
+   unlock_volumes();
+   return temp_vol_list;
+}
+
+/*
+ * Free the specified temp list.
+ */
+void free_temp_vol_list(dlist *temp_vol_list)
+{
+   dlist *save_vol_list;
+   
+   lock_volumes();
+   save_vol_list = vol_list;
+   vol_list = temp_vol_list;
+   free_volume_list();                  /* release temp_vol_list */
+   vol_list = save_vol_list;
+   Dmsg0(dbglvl, "deleted temp vol list\n");
+   Dmsg0(dbglvl, "unlock volumes\n");
+   unlock_volumes();
+   debug_list_volumes("after free temp table");
+}
index 10997e774deee5662cf8258f75d1659e70a3b359..1c80647d483595c9b314f46da3c30b20e631bf20 100644 (file)
@@ -3,9 +3,9 @@
  */
 
 #undef  VERSION
-#define VERSION "2.5.16-beta"
-#define BDATE   "20 October 2008"
-#define LSMDATE "20Oct08"
+#define VERSION "2.5.17"
+#define BDATE   "21 October 2008"
+#define LSMDATE "21Oct08"
 
 #define PROG_COPYRIGHT "Copyright (C) %d-2008 Free Software Foundation Europe e.V.\n"
 #define BYEAR "2008"       /* year for copyright messages in progs */
index 157c241f171ff34a213cf2d5468aaa5a95589588..301f8babdec48d02f1dba6073fdc0c8ada2ca0b0 100644 (file)
@@ -51,6 +51,8 @@ libtool on the configure command line with:
 
 General:
 21Oct08
+kes  Add read volume list code to SD -- not yet used.
+kes  Add James' binutils patch
 kes  Split volume management code out of src/stored/reserve.c into
      a new file vol_mgr.c 
 kes  Modify configure to do an automatic make clean.  This ensures