commit e79e5040a0e7efd622ecdd572bee40c90e59c3bd Author: Miroslav Lichvar Date: Fri Apr 13 17:11:58 2018 +0200 timemaster: restart terminated processes. If a ptp4l or phc2sys process is terminated (e.g. due to a crash) and timemaster was running for at least one second (i.e. it's not an error in ptp4l/phc2sys configuration), start the process again. Restart all processes corresponding to the same time source at the same time to ensure phc2sys is always connected to the currently running ptp4l. Add a new option to disable the restarting. Signed-off-by: Miroslav Lichvar diff --git a/timemaster.8 b/timemaster.8 index e0e22eb..7288972 100644 --- a/timemaster.8 +++ b/timemaster.8 @@ -87,6 +87,16 @@ Specify the first number in a sequence of SHM segments that will be used by can be useful to avoid conflicts with time sources that are not started by \fBtimemaster\fR, e.g. \fBgpsd\fR using segments number 0 and 1. +.TP +.B restart_processes +Enable or disable restarting of processes started by \fBtimemaster\fR. If the +option is set to a non-zero value, all processes except \fBchronyd\fR and +\fBntpd\fR will be automatically restarted when terminated and \fBtimemaster\fR +is running for at least one second (i.e. the process did not terminate due to a +configuration error). If a process was terminated and is not started again, +\fBtimemaster\fR will kill the other processes and exit with a non-zero status. +The default value is 1 (enabled). + .SS [ntp_server address] The \fBntp_server\fR section specifies an NTP server that should be used as a @@ -318,6 +328,7 @@ ptp4l_option delay_mechanism P2P ntp_program chronyd rundir /var/run/timemaster first_shm_segment 1 +restart_processes 0 [chronyd] path /usr/sbin/chronyd diff --git a/timemaster.c b/timemaster.c index fc3ba31..4ba921e 100644 --- a/timemaster.c +++ b/timemaster.c @@ -44,6 +44,7 @@ #define DEFAULT_RUNDIR "/var/run/timemaster" #define DEFAULT_FIRST_SHM_SEGMENT 0 +#define DEFAULT_RESTART_PROCESSES 1 #define DEFAULT_NTP_PROGRAM CHRONYD #define DEFAULT_NTP_MINPOLL 6 @@ -108,6 +109,7 @@ struct timemaster_config { enum ntp_program ntp_program; char *rundir; int first_shm_segment; + int restart_processes; struct program_config chronyd; struct program_config ntpd; struct program_config phc2sys; @@ -122,6 +124,9 @@ struct config_file { struct script { struct config_file **configs; char ***commands; + int **command_groups; + int restart_groups; + int no_restart_group; }; static void free_parray(void **a) @@ -385,6 +390,8 @@ static int parse_timemaster_settings(char **settings, replace_string(value, &config->rundir); } else if (!strcasecmp(name, "first_shm_segment")) { r = parse_int(value, &config->first_shm_segment); + } else if (!strcasecmp(name, "restart_processes")) { + r = parse_int(value, &config->restart_processes); } else { pr_err("unknown timemaster setting %s", name); return 1; @@ -508,6 +515,7 @@ static struct timemaster_config *config_parse(char *path) config->ntp_program = DEFAULT_NTP_PROGRAM; config->rundir = xstrdup(DEFAULT_RUNDIR); config->first_shm_segment = DEFAULT_FIRST_SHM_SEGMENT; + config->restart_processes = DEFAULT_RESTART_PROCESSES; init_program_config(&config->chronyd, "chronyd", NULL, DEFAULT_CHRONYD_SETTINGS, NULL); @@ -632,6 +640,18 @@ static char *get_refid(char *prefix, unsigned int number) return NULL; }; +static void add_command(char **command, int command_group, + struct script *script) +{ + int *group; + + parray_append((void ***)&script->commands, command); + + group = xmalloc(sizeof(int)); + *group = command_group; + parray_append((void ***)&script->command_groups, group); +} + static void add_shm_source(int shm_segment, int poll, int dpoll, double delay, char *ntp_options, char *prefix, struct timemaster_config *config, char **ntp_config) @@ -671,8 +691,8 @@ static int add_ntp_source(struct ntp_server *source, char **ntp_config) static int add_ptp_source(struct ptp_domain *source, struct timemaster_config *config, int *shm_segment, - int ***allocated_phcs, char **ntp_config, - struct script *script) + int *command_group, int ***allocated_phcs, + char **ntp_config, struct script *script) { struct config_file *config_file; char **command, *uds_path, **interfaces, *message_tag; @@ -798,19 +818,19 @@ static int add_ptp_source(struct ptp_domain *source, /* HW time stamping */ command = get_ptp4l_command(&config->ptp4l, config_file, interfaces, 1); - parray_append((void ***)&script->commands, command); + add_command(command, *command_group, script); command = get_phc2sys_command(&config->phc2sys, source->domain, source->phc2sys_poll, *shm_segment, uds_path, message_tag); - parray_append((void ***)&script->commands, command); + add_command(command, (*command_group)++, script); } else { /* SW time stamping */ command = get_ptp4l_command(&config->ptp4l, config_file, interfaces, 0); - parray_append((void ***)&script->commands, command); + add_command(command, (*command_group)++, script); string_appendf(&config_file->content, "clock_servo ntpshm\n" @@ -862,7 +882,8 @@ static char **get_ntpd_command(struct program_config *config, } static struct config_file *add_ntp_program(struct timemaster_config *config, - struct script *script) + struct script *script, + int command_group) { struct config_file *ntp_config = xmalloc(sizeof(*ntp_config)); char **command = NULL; @@ -886,7 +907,7 @@ static struct config_file *add_ntp_program(struct timemaster_config *config, } parray_append((void ***)&script->configs, ntp_config); - parray_append((void ***)&script->commands, command); + add_command(command, command_group, script); return ntp_config; } @@ -894,6 +915,7 @@ static struct config_file *add_ntp_program(struct timemaster_config *config, static void script_destroy(struct script *script) { char ***commands, **command; + int **groups; struct config_file *config, **configs; for (configs = script->configs; *configs; configs++) { @@ -911,6 +933,10 @@ static void script_destroy(struct script *script) } free(script->commands); + for (groups = script->command_groups; *groups; groups++) + free(*groups); + free(script->command_groups); + free(script); } @@ -920,12 +946,15 @@ static struct script *script_create(struct timemaster_config *config) struct source *source, **sources; struct config_file *ntp_config = NULL; int **allocated_phcs = (int **)parray_new(); - int ret = 0, shm_segment; + int ret = 0, shm_segment, command_group = 0; script->configs = (struct config_file **)parray_new(); script->commands = (char ***)parray_new(); + script->command_groups = (int **)parray_new(); + script->no_restart_group = command_group; + script->restart_groups = config->restart_processes; - ntp_config = add_ntp_program(config, script); + ntp_config = add_ntp_program(config, script, command_group++); shm_segment = config->first_shm_segment; for (sources = config->sources; (source = *sources); sources++) { @@ -936,7 +965,7 @@ static struct script *script_create(struct timemaster_config *config) break; case PTP_DOMAIN: if (add_ptp_source(&source->ptp, config, &shm_segment, - &allocated_phcs, + &command_group, &allocated_phcs, &ntp_config->content, script)) ret = 1; break; @@ -1063,10 +1092,11 @@ static int remove_config_files(struct config_file **configs) static int script_run(struct script *script) { + struct timespec ts_start, ts_now; sigset_t mask, old_mask; siginfo_t info; pid_t pid, *pids; - int i, num_commands, status, ret = 0; + int i, group, num_commands, status, quit = 0, ret = 0; for (num_commands = 0; script->commands[num_commands]; num_commands++) ; @@ -1101,7 +1131,9 @@ static int script_run(struct script *script) } } - /* wait for one of the blocked signals */ + clock_gettime(CLOCK_MONOTONIC, &ts_start); + + /* process the blocked signals */ while (1) { if (sigwaitinfo(&mask, &info) < 0) { if (errno == EINTR) @@ -1110,36 +1142,111 @@ static int script_run(struct script *script) break; } - /* - * assume only the first process (i.e. chronyd or ntpd) is - * essential and continue if other processes terminate - */ - if (info.si_signo == SIGCHLD && info.si_pid != pids[0]) { - pr_info("process %d terminated (ignored)", info.si_pid); + clock_gettime(CLOCK_MONOTONIC, &ts_now); + + if (info.si_signo != SIGCHLD) { + if (quit) + continue; + + quit = 1; + pr_debug("exiting on signal %d", info.si_signo); + + /* terminate remaining processes */ + for (i = 0; i < num_commands; i++) { + if (pids[i] > 0) { + pr_debug("killing process %d", pids[i]); + kill(pids[i], SIGTERM); + } + } + continue; } - pr_info("received signal %d", info.si_signo); - break; - } + /* wait for all terminated processes */ + while (1) { + pid = waitpid(-1, &status, WNOHANG); + if (pid <= 0) + break; - /* kill all started processes */ - for (i = 0; i < num_commands; i++) { - if (pids[i] > 0) { - pr_debug("killing process %d", pids[i]); - kill(pids[i], SIGTERM); + if (!WIFEXITED(status)) { + pr_info("process %d terminated abnormally", + pid); + } else { + pr_info("process %d terminated with status %d", + pid, WEXITSTATUS(status)); + } + + for (i = 0; i < num_commands; i++) { + if (pids[i] == pid) + pids[i] = 0; + } } - } - while ((pid = wait(&status)) >= 0) { - if (!WIFEXITED(status)) { - pr_info("process %d terminated abnormally", pid); - ret = 1; - } else { - if (WEXITSTATUS(status)) + /* wait for all processes to terminate when exiting */ + if (quit) { + for (i = 0; i < num_commands; i++) { + if (pids[i]) + break; + } + if (i == num_commands) + break; + + pr_debug("waiting for other processes to terminate"); + continue; + } + + /* + * terminate (and then restart if allowed) all processes in + * groups that have a terminated process + */ + for (group = 0; group < num_commands; group++) { + int terminated = 0, running = 0; + + for (i = 0; i < num_commands; i++) { + if (*(script->command_groups[i]) != group) + continue; + if (pids[i]) + running++; + else + terminated++; + } + + if (!terminated) + continue; + + /* + * exit with a non-zero status if the group should not + * be restarted (i.e. chronyd/ntpd), timemaster is + * running only for a short time (and it is likely a + * configuration error), or restarting is disabled + * completely + */ + if (group == script->no_restart_group || + ts_now.tv_sec - ts_start.tv_sec <= 1 || + !script->restart_groups) { + kill(getpid(), SIGTERM); ret = 1; - pr_info("process %d terminated with status %d", pid, - WEXITSTATUS(status)); + break; + } + + for (i = 0; i < num_commands; i++) { + if (*(script->command_groups[i]) != group) + continue; + + /* terminate all processes in the group first */ + if (running && pids[i]) { + pr_debug("killing process %d", pids[i]); + kill(pids[i], SIGTERM); + } else if (!running && !pids[i]) { + pids[i] = start_program(script->commands[i], + &old_mask); + if (!pids[i]) + kill(getpid(), SIGTERM); + + /* limit restarting rate */ + sleep(1); + } + } } } @@ -1154,6 +1261,7 @@ static int script_run(struct script *script) static void script_print(struct script *script) { char ***commands, **command; + int **groups; struct config_file *config, **configs; for (configs = script->configs; *configs; configs++) { @@ -1162,7 +1270,9 @@ static void script_print(struct script *script) } fprintf(stderr, "commands:\n\n"); - for (commands = script->commands; *commands; commands++) { + for (commands = script->commands, groups = script->command_groups; + *commands; commands++, groups++) { + fprintf(stderr, "[%d] ", **groups); for (command = *commands; *command; command++) fprintf(stderr, "%s ", *command); fprintf(stderr, "\n");