diff options
-rw-r--r-- | src/ctl.c | 16 | ||||
-rw-r--r-- | src/progman.c | 9 | ||||
-rw-r--r-- | src/sysvinit.c | 90 |
3 files changed, 83 insertions, 32 deletions
@@ -1875,23 +1875,25 @@ fun_stop (struct json_value *result, struct prog *prog) static int fun_start (struct json_value *result, struct prog *prog) { - if (prog->v.p.status == status_disabled) + switch (prog->v.p.status) { + case status_disabled: prog->v.p.comp->flags &= ~CF_DISABLED; prog->v.p.status = status_enabled; json_object_set_string (result, "status", "OK"); - } - else if (prog->v.p.status == status_sleeping) - { + break; + + case status_sleeping: + case status_finished: prog->v.p.status = status_enabled; prog->v.p.failcount = 0; prog->v.p.timestamp = 0; json_object_set_string (result, "status", "OK"); - } - else - { + break; + + default: json_object_set_string (result, "status", "ER"); json_object_set_string (result, "error_message", "already running"); return 1; } diff --git a/src/progman.c b/src/progman.c index 4ca3824..020dccd 100644 --- a/src/progman.c +++ b/src/progman.c @@ -302,11 +302,11 @@ progman_waiting_p () for (prog = proghead; prog; prog = prog->next) { if (IS_COMPONENT (prog) && prog->v.p.wait && prog->pid > 0) { - debug(1, ("%s: waiting for %s (%lu)", - __FUNCTION__, prog_tag (prog), - (unsigned long) prog->pid)); + debug (3, ("%s: waiting for %s (%lu)", + __FUNCTION__, prog_tag (prog), + (unsigned long) prog->pid)); return 1; } } return 0; @@ -2304,9 +2304,10 @@ progman_cleanup (int expect_term) } } else { - if (prog->v.p.comp->mode >= pies_mark_sysvinit) + if (prog->v.p.comp->mode >= pies_mark_sysvinit + && prog->v.p.comp->mode != pies_comp_ondemand) { sysvinit_acct (SYSV_ACCT_PROC_STOP, "", prog_tag (prog), pid, ""); prog->v.p.status = status_finished; diff --git a/src/sysvinit.c b/src/sysvinit.c index be7aef8..337b922 100644 --- a/src/sysvinit.c +++ b/src/sysvinit.c @@ -237,13 +237,16 @@ enablecomp (struct prog *prog, void *data) case pies_comp_sysinit: case pies_comp_boot: case pies_comp_bootwait: return 0; + + case pies_comp_ondemand: + return prog->v.p.status == status_enabled; + case pies_comp_powerfail: case pies_comp_powerwait: case pies_comp_powerokwait: case pies_comp_ctrlaltdel: - case pies_comp_ondemand: case pies_comp_powerfailnow: case pies_comp_kbrequest: return s && (s->mask & PIES_COMP_MASK (comp->mode)); default: @@ -299,9 +302,30 @@ sysvinit_runlevel_setup (int mask, int *wait) progman_foreach (runlevel_setup_prog, &s); if (wait) *wait = s.wait; } - + +static int +demand_prog (struct prog *prog, void *data) +{ + int *rl = data; + struct component *comp = prog->v.p.comp; + if (comp->mode == pies_comp_ondemand + && comp->runlevels + && strchr (comp->runlevels, *rl)) + { + prog->v.p.status = status_enabled; + debug (1, ("%s: %s", prog_tag (prog), "enabled")); + } + return 0; +} + +static void +sysvinit_demand (int rl) +{ + progman_foreach (demand_prog, &rl); +} + static const char valid_runlevel_arg[] = "0123456789SsQqAaBbCcUu"; int is_valid_runlevel (int c) @@ -457,10 +481,23 @@ sysvinit_fifo_handler (int fd, void *data) default: if (buf.req.runlevel != runlevel) { - dfl_level = buf.req.runlevel; - inittrans (); + switch (buf.req.runlevel) + { + case 'A': + case 'B': + case 'C': + sysvinit_demand (buf.req.runlevel); + break; + + default: + if (runlevel_index (buf.req.runlevel) != -1) + { + dfl_level = buf.req.runlevel; + inittrans (); + } + } } } break; @@ -818,21 +855,21 @@ struct action_parser int (*parser) (struct component *comp, const char *file, unsigned line); }; static struct action_parser action_tab[] = { - { "wait", pies_comp_wait }, - { "once", pies_comp_once }, - { "boot", pies_comp_boot }, - { "bootwait", pies_comp_bootwait }, - { "powerfail", pies_comp_powerfail }, - { "powerwait", pies_comp_powerwait }, - { "powerokwait", pies_comp_powerokwait }, - { "ctrlaltdel", pies_comp_ctrlaltdel }, - { "ondemand", pies_comp_ondemand }, - { "sysinit", pies_comp_sysinit }, + { "wait", pies_comp_wait }, + { "once", pies_comp_once }, + { "boot", pies_comp_boot }, + { "bootwait", pies_comp_bootwait }, + { "powerfail", pies_comp_powerfail }, + { "powerwait", pies_comp_powerwait }, + { "powerokwait", pies_comp_powerokwait }, + { "ctrlaltdel", pies_comp_ctrlaltdel }, + { "ondemand", pies_comp_ondemand }, + { "sysinit", pies_comp_sysinit }, { "powerfailnow", pies_comp_powerfailnow }, - { "kbrequest", pies_comp_kbrequest }, - { "respawn", pies_comp_respawn }, + { "kbrequest", pies_comp_kbrequest }, + { "respawn", pies_comp_respawn }, { NULL } }; static struct action_parser * @@ -845,8 +882,17 @@ find_action_parser (const char *action) return ap; return NULL; } +static char * +strupr (char *s) +{ + char *p; + for (p = s; *p; p++) + *p = toupper (*p); + return s; +} + int inittab_parse (const char *file) { FILE *fp; @@ -930,10 +976,9 @@ inittab_parse (const char *file) process = p; if (!id || !runlevels || !action || !process) { - logmsg (LOG_ERR, "%s:%u: %s", - file, line_no, _("not enough fields")); + logmsg (LOG_ERR, "%s:%u: %s", file, line_no, _("not enough fields")); err = 1; continue; } @@ -949,20 +994,23 @@ inittab_parse (const char *file) initdefault = toupper (runlevels[0]); continue; } + if (strcmp (action, "off") == 0) + /* Ignore the entry */ + continue; + ap = find_action_parser (action); if (!ap) { - logmsg (LOG_ERR, "%s:%u: %s", - file, line_no, _("unknown action")); + logmsg (LOG_ERR, "%s:%u: %s", file, line_no, _("unknown action")); err = 1; continue; } comp = component_create (id); comp->mode = ap->mode; - comp->runlevels = grecs_strdup (runlevels); + comp->runlevels = grecs_strdup (strupr (runlevels)); if (wordsplit (process, &ws, WRDSF_DEFFLAGS)) { component_free (comp); |