static int opt_no_sessiond;
static char *opt_sessiond_path;
static pid_t sessiond_pid;
+static volatile int recv_child_signal;
enum {
OPT_SESSION_PATH,
case SIGCHLD:
DBG("SIGCHLD caugth");
waitpid(sessiond_pid, &status, 0);
+ recv_child_signal = 1;
/* Indicate that the session daemon died */
sessiond_pid = 0;
ERR("Session daemon died (exit status %d)", WEXITSTATUS(status));
break;
case SIGUSR1:
/* Notify is done */
+ recv_child_signal = 1;
DBG("SIGUSR1 caugth");
break;
default:
pid_t pid;
MSG("Spawning a session daemon");
+ recv_child_signal = 0;
pid = fork();
if (pid == 0) {
/*
} else {
perror("execlp");
}
- kill(getppid(), SIGTERM); /* unpause parent */
+ kill(getppid(), SIGTERM); /* wake parent */
exit(EXIT_FAILURE);
} else if (pid > 0) {
sessiond_pid = pid;
- /* Wait for lttng-sessiond to start */
- pause();
+ /*
+ * Wait for lttng-sessiond to start. We need to use a
+ * flag to check if the signal has been sent to us,
+ * because the child can be scheduled before the parent,
+ * and thus send the signal before this check. In the
+ * signal handler, we set the recv_child_signal flag, so
+ * anytime we check it after the fork is fine. Note that
+ * sleep() is interrupted before the 1 second delay as
+ * soon as the signal is received, so it will not cause
+ * visible delay for the user.
+ */
+ while (!recv_child_signal) {
+ sleep(1);
+ }
if (!sessiond_pid) {
exit(EXIT_FAILURE);
}
pathname = opt_sessiond_path;
} else {
/* Try LTTNG_SESSIOND_PATH env variable */
- pathname = getenv(LTTNG_SESSIOND_PATH_ENV);
+ pathname = getenv(DEFAULT_SESSIOND_PATH_ENV);
}
/* Let's rock and roll */
int i;
for (i = 0; i < argc; i++) {
- if ((strncmp(argv[i], "-h", 2) == 0) ||
- strncmp(argv[i], "--h", 3) == 0 ||
- strncmp(argv[i], "--list-options", 14)) {
+ if ((strncmp(argv[i], "-h", sizeof("-h")) == 0) ||
+ strncmp(argv[i], "--h", sizeof("--h")) == 0 ||
+ strncmp(argv[i], "--list-options", sizeof("--list-options")) == 0) {
return 1;
}
}
progname = argv[0] ? argv[0] : "lttng";
/* For Mathieu Desnoyers aka Dr Tracing */
- if (strncmp(progname, "drtrace", 7) == 0) {
- MSG("%c[%d;%dmWelcome back Dr Tracing!%c[%dm\n\n", 27,1,33,27,0);
+ if (strncmp(progname, "drtrace", 7) == 0 ||
+ strncmp("compudj", getenv("USER"), 7) == 0) {
+ MSG("%c[%d;%dmWelcome back Dr Tracing!%c[%dm\n", 27,1,33,27,0);
}
ret = set_signal_handler();