diff options
author | Lennart Poettering <lennart@poettering.net> | 2012-07-04 00:23:42 +0200 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2012-07-04 00:23:42 +0200 |
commit | 451b34cc1d74960ee0262eef970a6346ec814f31 (patch) | |
tree | 8781bb56a2846f6d870e002593d3ccecb787bcd5 | |
parent | 6ffd3337867076067955ec4d5d1a0cde521d9b86 (diff) |
service: flush the start counter in "systemctl reset-failed"
-rw-r--r-- | man/systemd.service.xml | 10 | ||||
-rw-r--r-- | src/core/service.c | 2 | ||||
-rw-r--r-- | src/shared/ratelimit.h | 7 |
3 files changed, 18 insertions, 1 deletions
diff --git a/man/systemd.service.xml b/man/systemd.service.xml index 60155e68cb..fcd3a7dd38 100644 --- a/man/systemd.service.xml +++ b/man/systemd.service.xml @@ -768,7 +768,15 @@ however they may still be restarted manually at a later point from which point on the restart logic is again - activated.</para></listitem> + activated. Note that + <command>systemctl + reset-failed</command> will cause the + restart rate counter for a service to + be flushed, which is useful if the + administrator wants to manually start + a service and the start limit + interferes with + that.</para></listitem> </varlistentry> <varlistentry> diff --git a/src/core/service.c b/src/core/service.c index d82803db2b..0c29d4aefa 100644 --- a/src/core/service.c +++ b/src/core/service.c @@ -3655,6 +3655,8 @@ static void service_reset_failed(Unit *u) { s->result = SERVICE_SUCCESS; s->reload_result = SERVICE_SUCCESS; + + RATELIMIT_RESET(s->start_limit); } static int service_kill(Unit *u, KillWho who, KillMode mode, int signo, DBusError *error) { diff --git a/src/shared/ratelimit.h b/src/shared/ratelimit.h index 07e8205c81..ad00267a6e 100644 --- a/src/shared/ratelimit.h +++ b/src/shared/ratelimit.h @@ -48,6 +48,13 @@ typedef struct RateLimit { _r->begin = 0; \ } while (false) +#define RATELIMIT_RESET(v) \ + do { \ + RateLimit *_r = &(v); \ + _r->num = 0; \ + _r->begin = 0; \ + } while (false) + bool ratelimit_test(RateLimit *r); #endif |