50.samba run the smbcontrol in the background. no need to block waiting for it.
[sahlberg/ctdb.git] / server / ctdb_logging.c
index 22baea46586c3e24a02c315ca09ea4fec7b6b786..27b990ef6768b09967b4e8139f0bbf19f9d1f5d8 100644 (file)
@@ -18,8 +18,8 @@
 */
 
 #include "includes.h"
-#include "lib/events/events.h"
-#include "../include/ctdb.h"
+#include "lib/tevent/tevent.h"
+#include "../include/ctdb_client.h"
 #include "../include/ctdb_private.h"
 #include "system/syslog.h"
 #include "system/time.h"
@@ -85,6 +85,7 @@ int start_syslog_daemon(struct ctdb_context *ctdb)
 {
        struct sockaddr_in syslog_sin;
        struct ctdb_syslog_state *state;
+       struct tevent_fd *fde;
 
        state = talloc(ctdb, struct ctdb_syslog_state);
        CTDB_NO_MEMORY(ctdb, state);
@@ -95,7 +96,7 @@ int start_syslog_daemon(struct ctdb_context *ctdb)
                return -1;
        }
        
-       ctdb->syslogd_pid = fork();
+       ctdb->syslogd_pid = ctdb_fork(ctdb);
        if (ctdb->syslogd_pid == (pid_t)-1) {
                printf("Failed to create syslog child process\n");
                close(state->fd[0]);
@@ -115,15 +116,18 @@ int start_syslog_daemon(struct ctdb_context *ctdb)
                return 0;
        }
 
+       debug_extra = talloc_asprintf(NULL, "syslogd:");
        talloc_free(ctdb->ev);
        ctdb->ev = event_context_init(NULL);
+       tevent_loop_allow_nesting(ctdb->ev);
 
        syslog(LOG_ERR, "Starting SYSLOG daemon with pid:%d", (int)getpid());
 
        close(state->fd[0]);
        set_close_on_exec(state->fd[1]);
-       event_add_fd(ctdb->ev, state, state->fd[1], EVENT_FD_READ|EVENT_FD_AUTOCLOSE,
+       fde = event_add_fd(ctdb->ev, state, state->fd[1], EVENT_FD_READ,
                     ctdb_syslog_terminate_handler, state);
+       tevent_fd_set_auto_close(fde);
 
        state->syslog_fd = socket(AF_INET, SOCK_DGRAM, IPPROTO_UDP);
        if (state->syslog_fd == -1) {
@@ -147,8 +151,9 @@ int start_syslog_daemon(struct ctdb_context *ctdb)
        }
 
 
-       event_add_fd(ctdb->ev, state, state->syslog_fd, EVENT_FD_READ|EVENT_FD_AUTOCLOSE,
+       fde = event_add_fd(ctdb->ev, state, state->syslog_fd, EVENT_FD_READ,
                     ctdb_syslog_handler, state);
+       tevent_fd_set_auto_close(fde);
 
        event_loop_wait(ctdb->ev);
 
@@ -213,15 +218,16 @@ static void ctdb_syslog_log(const char *format, va_list ap)
                break;          
        }
 
-       len = offsetof(struct syslog_message, message) + strlen(s) + 1;
+       len = offsetof(struct syslog_message, message) + strlen(debug_extra) + strlen(s) + 1;
        msg = malloc(len);
        if (msg == NULL) {
                free(s);
                return;
        }
        msg->level = level;
-       msg->len   = strlen(s);
-       strcpy(msg->message, s);
+       msg->len   = strlen(debug_extra) + strlen(s);
+       strcpy(msg->message, debug_extra);
+       strcat(msg->message, s);
 
        if (syslogd_is_started == 0) {
                syslog(msg->level, "%s", msg->message);
@@ -275,8 +281,9 @@ static void ctdb_logfile_log(const char *format, va_list ap)
 
        strftime(tbuf,sizeof(tbuf)-1,"%Y/%m/%d %H:%M:%S", tm);
 
-       ret = asprintf(&s2, "%s.%06u [%5u]: %s",
-                tbuf, (unsigned)t.tv_usec, (unsigned)getpid(), s);
+       ret = asprintf(&s2, "%s.%06u [%s%5u]: %s",
+                      tbuf, (unsigned)t.tv_usec,
+                      debug_extra, (unsigned)getpid(), s);
        free(s);
        if (ret == -1) {
                const char *errstr = "asprintf failed\n";
@@ -433,9 +440,8 @@ struct ctdb_log_state *ctdb_fork_with_logging(TALLOC_CTX *mem_ctx,
                                              void *logfn_private, pid_t *pid)
 {
        int p[2];
-       int old_stdout, old_stderr;
-       int saved_errno;
        struct ctdb_log_state *log;
+       struct tevent_fd *fde;
 
        log = talloc_zero(mem_ctx, struct ctdb_log_state);
        CTDB_NO_MEMORY_NULL(ctdb, log);
@@ -448,50 +454,36 @@ struct ctdb_log_state *ctdb_fork_with_logging(TALLOC_CTX *mem_ctx,
                goto free_log;
        }
 
-       /* We'll fail if stderr/stdout not already open; it's simpler. */
-       old_stdout = dup(STDOUT_FILENO);
-       old_stderr = dup(STDERR_FILENO);
-       if (dup2(p[1], STDOUT_FILENO) < 0 || dup2(p[1], STDERR_FILENO) < 0) {
-               DEBUG(DEBUG_ERR,(__location__ " Failed to setup output for child\n"));
-               goto close_pipe;
-       }
-       close(p[1]);
-
-       *pid = fork();
+       *pid = ctdb_fork(ctdb);
 
        /* Child? */
        if (*pid == 0) {
-               close(old_stdout);
-               close(old_stderr);
+               close(STDOUT_FILENO);
+               close(STDERR_FILENO);
+               dup2(p[1], STDOUT_FILENO);
+               dup2(p[1], STDERR_FILENO);
                close(p[0]);
+               close(p[1]);
                return log;
        }
-
-       saved_errno = errno;
-       dup2(STDOUT_FILENO, old_stdout);
-       dup2(STDERR_FILENO, old_stderr);
-       close(old_stdout);
-       close(old_stderr);
+       close(p[1]);
 
        /* We failed? */
        if (*pid < 0) {
                DEBUG(DEBUG_ERR, (__location__ " fork failed for child process\n"));
                close(p[0]);
-               errno = saved_errno;
                goto free_log;
        }
 
        log->pfd = p[0];
        set_close_on_exec(log->pfd);
        talloc_set_destructor(log, log_context_destructor);
-       event_add_fd(ctdb->ev, log, log->pfd,
-                    EVENT_FD_READ | EVENT_FD_AUTOCLOSE,
-                    ctdb_log_handler, log);
+       fde = event_add_fd(ctdb->ev, log, log->pfd,
+                          EVENT_FD_READ, ctdb_log_handler, log);
+       tevent_fd_set_auto_close(fde);
+
        return log;
 
-close_pipe:
-       close(p[0]);
-       close(p[1]);
 free_log:
        talloc_free(log);
        return NULL;
@@ -503,9 +495,10 @@ free_log:
 int ctdb_set_child_logging(struct ctdb_context *ctdb)
 {
        int p[2];
-       int ret;
+       int old_stdout, old_stderr;
+       struct tevent_fd *fde;
 
-       if (ctdb->log->fd == 1) {
+       if (ctdb->log->fd == STDOUT_FILENO) {
                /* not needed for stdout logging */
                return 0;
        }
@@ -516,35 +509,84 @@ int ctdb_set_child_logging(struct ctdb_context *ctdb)
                return -1;
        }
 
-       event_add_fd(ctdb->ev, ctdb->log, p[0],
-                    EVENT_FD_READ | EVENT_FD_AUTOCLOSE,
-                    ctdb_log_handler, ctdb->log);
+       /* We'll fail if stderr/stdout not already open; it's simpler. */
+       old_stdout = dup(STDOUT_FILENO);
+       old_stderr = dup(STDERR_FILENO);
+       if (dup2(p[1], STDOUT_FILENO) < 0 || dup2(p[1], STDERR_FILENO) < 0) {
+               int saved_errno = errno;
+               dup2(old_stdout, STDOUT_FILENO);
+               dup2(old_stderr, STDERR_FILENO);
+               close(old_stdout);
+               close(old_stderr);
+               close(p[0]);
+               close(p[1]);
+               errno = saved_errno;
+
+               printf(__location__ " dup2 failed: %s\n",
+                       strerror(errno));
+               return -1;
+       }
+       close(p[1]);
+       close(old_stdout);
+       close(old_stderr);
+
+       /* Is this correct for STDOUT and STDERR ? */
+       set_close_on_exec(STDOUT_FILENO);
+       set_close_on_exec(STDERR_FILENO);
        set_close_on_exec(p[0]);
+
+       fde = event_add_fd(ctdb->ev, ctdb->log, p[0],
+                          EVENT_FD_READ, ctdb_log_handler, ctdb->log);
+       tevent_fd_set_auto_close(fde);
+
        ctdb->log->pfd = p[0];
 
-       DEBUG(DEBUG_NOTICE, (__location__ " Created PIPE FD:%d for logging\n", p[0]));
+       DEBUG(DEBUG_DEBUG, (__location__ " Created PIPE FD:%d for logging\n", p[0]));
 
-       close(1);
-       close(2);
-       if (p[1] != 1) {
-               ret = dup2(p[1], 1);
-               if (ret == -1) {
-                       printf("dup2 failed: %s\n", strerror(errno));
-                       return -1;
-               }
-               close(p[1]);
-       }
-       /* also catch stderr of subcommands to the log */
-       ret = dup2(1, 2);
-       if (ret == -1) {
-               printf("dup2 failed: %s\n", strerror(errno));
-               return -1;
+       return 0;
+}
+
+
+/*
+ * set up a log handler to catch logging from TEVENT
+ */
+static void ctdb_tevent_logging(void *private_data,
+                               enum tevent_debug_level level,
+                               const char *fmt,
+                               va_list ap)
+{
+       enum debug_level lvl = DEBUG_EMERG;
+
+       switch (level) {
+       case TEVENT_DEBUG_FATAL:
+               lvl = DEBUG_EMERG;
+               break;
+       case TEVENT_DEBUG_ERROR:
+               lvl = DEBUG_ERR;
+               break;
+       case TEVENT_DEBUG_WARNING:
+               lvl = DEBUG_WARNING;
+               break;
+       case TEVENT_DEBUG_TRACE:
+               lvl = DEBUG_DEBUG;
+               break;
        }
 
-       return 0;
+       if (lvl <= LogLevel) {
+               this_log_level = lvl;
+               do_debug_v(fmt, ap);
+       }
 }
 
+int ctdb_init_tevent_logging(struct ctdb_context *ctdb)
+{
+       int ret;
 
+       ret = tevent_set_debug(ctdb->ev,
+                       ctdb_tevent_logging,
+                       ctdb);
+       return ret;
+}