]> git.sur5r.net Git - bacula/bacula/blobdiff - bacula/src/lib/jcr.c
- Fix mode change open in btape.c
[bacula/bacula] / bacula / src / lib / jcr.c
index d0ca12e3332f3ae492ffb473dd08d1b2008ee379..fbb1356ac840a640c49eb8ab0e4ef6087e7e11fe 100755 (executable)
@@ -17,7 +17,7 @@
  *  in the list while traversing it rather than a single lock
  *  at the beginning of a traversal and one at the end.  This
  *  incurs slightly more overhead, but effectively eliminates 
- *  the possibilty of race conditions.  In addition, with the
+ *  the possibilty of race conditions. In addition, with the
  *  exception of the global locking of the list during the
  *  re-reading of the config file, no recursion is needed.
  *
@@ -27,7 +27,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,
@@ -61,8 +61,7 @@ int num_jobs_run;
 dlist *last_jobs = NULL;
 const int max_last_jobs = 10;
  
-static dlist *jcrs = NULL;            /* JCR chain */
-//static brwlock_t lock;                /* lock for last jobs and JCR chain */
+static dlist *jcrs = NULL;           /* JCR chain */
 static pthread_mutex_t jcr_lock = PTHREAD_MUTEX_INITIALIZER;
 
 static pthread_mutex_t job_start_mutex = PTHREAD_MUTEX_INITIALIZER;
@@ -79,15 +78,10 @@ void unlock_jobs()
 
 void init_last_jobs_list()
 {
-// int errstat;
-   JCR *jcr;
+   JCR *jcr = NULL;
    struct s_last_job *job_entry = NULL;
    if (!last_jobs) {
       last_jobs = New(dlist(job_entry, &job_entry->link));
-//    if ((errstat=rwl_init(&lock)) != 0) {
-//       Emsg1(M_ABORT, 0, _("Unable to initialize jcr_chain lock. ERR=%s\n"),
-//             strerror(errstat));
-//    }
    }
    if (!jcrs) {
       jcrs = New(dlist(jcr, &jcr->link));
@@ -98,14 +92,16 @@ void term_last_jobs_list()
 {
    if (last_jobs) {
       while (!last_jobs->empty()) {
-         void *je = last_jobs->first();
-         last_jobs->remove(je);
-         free(je);
+        void *je = last_jobs->first();
+        last_jobs->remove(je);
+        free(je);
       }
       delete last_jobs;
       last_jobs = NULL;
-//    rwl_destroy(&lock);
+   }
+   if (jcrs) {
       delete jcrs;
+      jcrs = NULL;
    }
 }
 
@@ -128,20 +124,20 @@ void read_last_jobs_list(int fd, uint64_t addr)
    for ( ; num; num--) {
       if (read(fd, &job, sizeof(job)) != sizeof(job)) {
          Dmsg1(000, "Read job entry. ERR=%s\n", strerror(errno));
-         return;
+        return;
       }
       if (job.JobId > 0) {
-         je = (struct s_last_job *)malloc(sizeof(struct s_last_job));
-         memcpy((char *)je, (char *)&job, sizeof(job));
-         if (!last_jobs) {
-            init_last_jobs_list();
-         }
-         last_jobs->append(je);
-         if (last_jobs->size() > max_last_jobs) {
-            je = (struct s_last_job *)last_jobs->first();
-            last_jobs->remove(je);
-            free(je);
-         }
+        je = (struct s_last_job *)malloc(sizeof(struct s_last_job));
+        memcpy((char *)je, (char *)&job, sizeof(job));
+        if (!last_jobs) {
+           init_last_jobs_list();
+        }
+        last_jobs->append(je);
+        if (last_jobs->size() > max_last_jobs) {
+           je = (struct s_last_job *)last_jobs->first();
+           last_jobs->remove(je);
+           free(je);
+        }
       }
    }
 }
@@ -160,13 +156,13 @@ uint64_t write_last_jobs_list(int fd, uint64_t addr)
       num = last_jobs->size();
       if (write(fd, &num, sizeof(num)) != sizeof(num)) {
          Dmsg1(000, "Error writing num_items: ERR=%s\n", strerror(errno));
-         return 0;
+        return 0;
       }
       foreach_dlist(je, last_jobs) {
-         if (write(fd, je, sizeof(struct s_last_job)) != sizeof(struct s_last_job)) {
+        if (write(fd, je, sizeof(struct s_last_job)) != sizeof(struct s_last_job)) {
             Dmsg1(000, "Error writing job: ERR=%s\n", strerror(errno));
-            return 0;
-         }
+           return 0;
+        }
       }
    }
    /* Return current address */
@@ -241,7 +237,7 @@ JCR *new_jcr(int size, JCR_free_HANDLER *daemon_free_jcr)
    /* Setup some dummy values */
    bstrncpy(jcr->Job, "*System*", sizeof(jcr->Job));
    jcr->JobId = 0;
-   jcr->JobType = JT_SYSTEM;          /* internal job until defined */
+   jcr->JobType = JT_SYSTEM;         /* internal job until defined */
    jcr->JobLevel = L_NONE;
    jcr->JobStatus = JS_Created;
 
@@ -272,13 +268,13 @@ JCR *new_jcr(int size, JCR_free_HANDLER *daemon_free_jcr)
 /*
  * Remove a JCR from the chain
  * NOTE! The chain must be locked prior to calling
- *       this routine.
+ *      this routine.
  */
 static void remove_jcr(JCR *jcr)
 {
    Dmsg0(3400, "Enter remove_jcr\n");
    if (!jcr) {
-      Emsg0(M_ABORT, 0, "NULL jcr.\n");
+      Emsg0(M_ABORT, 0, _("NULL jcr.\n"));
    }
    jcrs->remove(jcr);
    Dmsg0(3400, "Leave remove_jcr\n");
@@ -313,17 +309,17 @@ static void free_common_jcr(JCR *jcr)
       last_job.end_time = time(NULL);
       /* Keep list of last jobs, but not Console where JobId==0 */
       if (last_job.JobId > 0) {
-         je = (struct s_last_job *)malloc(sizeof(struct s_last_job));
-         memcpy((char *)je, (char *)&last_job, sizeof(last_job));
-         if (!last_jobs) {
-            init_last_jobs_list();
-         }
-         last_jobs->append(je);
-         if (last_jobs->size() > max_last_jobs) {
-            je = (struct s_last_job *)last_jobs->first();
-            last_jobs->remove(je);
-            free(je);
-         }
+        je = (struct s_last_job *)malloc(sizeof(struct s_last_job));
+        memcpy((char *)je, (char *)&last_job, sizeof(last_job));
+        if (!last_jobs) {
+           init_last_jobs_list();
+        }
+        last_jobs->append(je);
+        if (last_jobs->size() > max_last_jobs) {
+           je = (struct s_last_job *)last_jobs->first();
+           last_jobs->remove(je);
+           free(je);
+        }
       }
       break;
    default:
@@ -332,7 +328,7 @@ static void free_common_jcr(JCR *jcr)
    pthread_mutex_destroy(&jcr->mutex);
 
    delete jcr->msg_queue;
-   close_msg(jcr);                    /* close messages for this job */
+   close_msg(jcr);                   /* close messages for this job */
 
    /* do this after closing messages */
    if (jcr->client_name) {
@@ -395,29 +391,29 @@ void free_jcr(JCR *jcr)
 
    dequeue_messages(jcr);
    lock_jcr_chain();
-   jcr->dec_use_count();              /* decrement use count */
+   jcr->dec_use_count();             /* decrement use count */
    if (jcr->use_count < 0) {
       Emsg2(M_ERROR, 0, _("JCR use_count=%d JobId=%d\n"),
-         jcr->use_count, jcr->JobId);
+        jcr->use_count, jcr->JobId);
    }
    Dmsg3(3400, "Dec free_jcr 0x%x use_count=%d jobid=%d\n", jcr, jcr->use_count, jcr->JobId);
-   if (jcr->use_count > 0) {          /* if in use */
+   if (jcr->use_count > 0) {         /* if in use */
       unlock_jcr_chain();
       Dmsg2(3400, "free_jcr 0x%x use_count=%d\n", jcr, jcr->use_count);
       return;
    }
 
-   remove_jcr(jcr);                   /* remove Jcr from chain */
+   remove_jcr(jcr);                  /* remove Jcr from chain */
    unlock_jcr_chain();
 
-   job_end_pop(jcr);                  /* pop and call hooked routines */
+   job_end_pop(jcr);                 /* pop and call hooked routines */
 
    Dmsg1(3400, "End job=%d\n", jcr->JobId);
    if (jcr->daemon_free_jcr) {
       jcr->daemon_free_jcr(jcr);      /* call daemon free routine */
    }
    free_common_jcr(jcr);
-   close_msg(NULL);                   /* flush any daemon messages */
+   close_msg(NULL);                  /* flush any daemon messages */
    Dmsg0(3400, "Exit free_jcr\n");
 }
 
@@ -425,18 +421,18 @@ void free_jcr(JCR *jcr)
 /*
  * Given a JobId, find the JCR
  *   Returns: jcr on success
- *            NULL on failure
+ *           NULL on failure
  */
 JCR *get_jcr_by_id(uint32_t JobId)
 {
    JCR *jcr;
 
-   lock_jcr_chain();                    /* lock chain */
+   lock_jcr_chain();                   /* lock chain */
    foreach_dlist(jcr, jcrs) {
       if (jcr->JobId == JobId) {
-         jcr->inc_use_count();
+        jcr->inc_use_count();
          Dmsg2(3400, "Inc get_jcr 0x%x use_count=%d\n", jcr, jcr->use_count);
-         break;
+        break;
       }
    }
    unlock_jcr_chain();
@@ -446,7 +442,7 @@ JCR *get_jcr_by_id(uint32_t JobId)
 /*
  * Given a SessionId and SessionTime, find the JCR
  *   Returns: jcr on success
- *            NULL on failure
+ *           NULL on failure
  */
 JCR *get_jcr_by_session(uint32_t SessionId, uint32_t SessionTime)
 {
@@ -455,10 +451,10 @@ JCR *get_jcr_by_session(uint32_t SessionId, uint32_t SessionTime)
    lock_jcr_chain();
    foreach_dlist(jcr, jcrs) {
       if (jcr->VolSessionId == SessionId &&
-          jcr->VolSessionTime == SessionTime) {
-         jcr->inc_use_count();
+         jcr->VolSessionTime == SessionTime) {
+        jcr->inc_use_count();
          Dmsg2(3400, "Inc get_jcr 0x%x use_count=%d\n", jcr, jcr->use_count);
-         break;
+        break;
       }
    }
    unlock_jcr_chain();
@@ -471,7 +467,7 @@ JCR *get_jcr_by_session(uint32_t SessionId, uint32_t SessionTime)
  *  compares on the number of characters in Job
  *  thus allowing partial matches.
  *   Returns: jcr on success
- *            NULL on failure
+ *           NULL on failure
  */
 JCR *get_jcr_by_partial_name(char *Job)
 {
@@ -485,9 +481,9 @@ JCR *get_jcr_by_partial_name(char *Job)
    len = strlen(Job);
    foreach_dlist(jcr, jcrs) {
       if (strncmp(Job, jcr->Job, len) == 0) {
-         jcr->inc_use_count();
+        jcr->inc_use_count();
          Dmsg2(3400, "Inc get_jcr 0x%x use_count=%d\n", jcr, jcr->use_count);
-         break;
+        break;
       }
    }
    unlock_jcr_chain();
@@ -499,7 +495,7 @@ JCR *get_jcr_by_partial_name(char *Job)
  * Given a Job, find the JCR
  *  requires an exact match of names.
  *   Returns: jcr on success
- *            NULL on failure
+ *           NULL on failure
  */
 JCR *get_jcr_by_full_name(char *Job)
 {
@@ -511,9 +507,9 @@ JCR *get_jcr_by_full_name(char *Job)
    lock_jcr_chain();
    foreach_dlist(jcr, jcrs) {
       if (strcmp(jcr->Job, Job) == 0) {
-         jcr->inc_use_count();
+        jcr->inc_use_count();
          Dmsg2(3400, "Inc get_jcr 0x%x use_count=%d\n", jcr, jcr->use_count);
-         break;
+        break;
       }
    }
    unlock_jcr_chain();
@@ -551,15 +547,10 @@ static void b_lock_jcr_chain(const char *fname, int line)
 static void lock_jcr_chain()
 #endif
 {
-// int errstat;
 #ifdef TRACE_JCR_CHAIN
    Dmsg3(3400, "Lock jcr chain %d from %s:%d\n", ++lock_count,
       fname, line);
 #endif
-// if ((errstat=rwl_writelock(&lock)) != 0) {
-//    Emsg1(M_ABORT, 0, "rwl_writelock failure. ERR=%s\n",
-//         strerror(errstat));
-// }
    P(jcr_lock);
 }
 
@@ -572,15 +563,10 @@ static void b_unlock_jcr_chain(const char *fname, int line)
 static void unlock_jcr_chain()
 #endif
 {
-// int errstat;
 #ifdef TRACE_JCR_CHAIN
    Dmsg3(3400, "Unlock jcr chain %d from %s:%d\n", lock_count--,
       fname, line);
 #endif
-// if ((errstat=rwl_writeunlock(&lock)) != 0) {
-//    Emsg1(M_ABORT, 0, "rwl_writeunlock failure. ERR=%s\n",
-//         strerror(errstat));
-// }
    V(jcr_lock);
 }
 
@@ -604,8 +590,8 @@ bool init_jcr_subsystem(void)
    watchdog_t *wd = new_watchdog();
 
    wd->one_shot = false;
-   wd->interval = 30;   /* FIXME: should be configurable somewhere, even
-                         if only with a #define */
+   wd->interval = 30;  /* FIXME: should be configurable somewhere, even
+                        if only with a #define */
    wd->callback = jcr_timeout_check;
 
    register_watchdog(wd);
@@ -626,44 +612,44 @@ static void jcr_timeout_check(watchdog_t *self)
     */
    foreach_jcr(jcr) {
       if (jcr->JobId == 0) {
-         free_jcr(jcr);
-         continue;
+        free_jcr(jcr);
+        continue;
       }
       fd = jcr->store_bsock;
       if (fd) {
-         timer_start = fd->timer_start;
-         if (timer_start && (watchdog_time - timer_start) > fd->timeout) {
-            fd->timer_start = 0;      /* turn off timer */
-            fd->timed_out = TRUE;
-            Jmsg(jcr, M_ERROR, 0, _(
+        timer_start = fd->timer_start;
+        if (timer_start && (watchdog_time - timer_start) > fd->timeout) {
+           fd->timer_start = 0;      /* turn off timer */
+           fd->timed_out = TRUE;
+           Jmsg(jcr, M_ERROR, 0, _(
 "Watchdog sending kill after %d secs to thread stalled reading Storage daemon.\n"),
-                 watchdog_time - timer_start);
-            pthread_kill(jcr->my_thread_id, TIMEOUT_SIGNAL);
-         }
+                watchdog_time - timer_start);
+           pthread_kill(jcr->my_thread_id, TIMEOUT_SIGNAL);
+        }
       }
       fd = jcr->file_bsock;
       if (fd) {
-         timer_start = fd->timer_start;
-         if (timer_start && (watchdog_time - timer_start) > fd->timeout) {
-            fd->timer_start = 0;      /* turn off timer */
-            fd->timed_out = TRUE;
-            Jmsg(jcr, M_ERROR, 0, _(
+        timer_start = fd->timer_start;
+        if (timer_start && (watchdog_time - timer_start) > fd->timeout) {
+           fd->timer_start = 0;      /* turn off timer */
+           fd->timed_out = TRUE;
+           Jmsg(jcr, M_ERROR, 0, _(
 "Watchdog sending kill after %d secs to thread stalled reading File daemon.\n"),
-                 watchdog_time - timer_start);
-            pthread_kill(jcr->my_thread_id, TIMEOUT_SIGNAL);
-         }
+                watchdog_time - timer_start);
+           pthread_kill(jcr->my_thread_id, TIMEOUT_SIGNAL);
+        }
       }
       fd = jcr->dir_bsock;
       if (fd) {
-         timer_start = fd->timer_start;
-         if (timer_start && (watchdog_time - timer_start) > fd->timeout) {
-            fd->timer_start = 0;      /* turn off timer */
-            fd->timed_out = TRUE;
-            Jmsg(jcr, M_ERROR, 0, _(
+        timer_start = fd->timer_start;
+        if (timer_start && (watchdog_time - timer_start) > fd->timeout) {
+           fd->timer_start = 0;      /* turn off timer */
+           fd->timed_out = TRUE;
+           Jmsg(jcr, M_ERROR, 0, _(
 "Watchdog sending kill after %d secs to thread stalled reading Director.\n"),
-                 watchdog_time - timer_start);
-            pthread_kill(jcr->my_thread_id, TIMEOUT_SIGNAL);
-         }
+                watchdog_time - timer_start);
+           pthread_kill(jcr->my_thread_id, TIMEOUT_SIGNAL);
+        }
       }
       free_jcr(jcr);
    }
@@ -676,5 +662,5 @@ static void jcr_timeout_check(watchdog_t *self)
  */
 extern "C" void timeout_handler(int sig)
 {
-   return;                            /* thus interrupting the function */
+   return;                           /* thus interrupting the function */
 }