[systemd-devel] [PATCH 0/3] more watchdog patches

Michael Olbrich m.olbrich at pengutronix.de
Fri Feb 3 12:14:22 PST 2012


On Thu, Feb 02, 2012 at 07:42:46PM +0100, Lennart Poettering wrote:
> b) hook up the watchdog with the existing failure logic and introduce
> WatchdogSec for that.

This is the first patch. It works, but I'm not sure about the
implementation. I tried to use the existing Watch. However, as far as I can
tell there are collisions e.g. with exec post handling.

> c) pass watchdog frequency to executed processes via env var

The second patch. This is rather trivial.

> e) extend start logic to do configurable rate limiting of starts,
> plus optionally reboots when rate limiter is triggered. New options for
> this:
> 
>     StartLimitInterval=5min
>     StartLimitBurst=5000
>     StartLimitAction=none|reboot|reboot-force|reboot-immediate

The third patch. This is work in progress. Just the variables so far. I
tried to follow other examples. Not sure if I did everything correctly.
And I could not find any place to initialize the rate limiter with the
configured values. They are available when querying via dbus but not in any
initialization function that I could find. Where would I do that?

And can you elaborate reboot, reboot-force and reboot-immediate exactly
mean?

Michael

Michael Olbrich (3):
  introduce WatchdogSec and hook up the watchdog with the existing
    failure logic
  set WATCHDOG_USEC environmen variable
  WIP: service: add StartLimitInterval/StartLimitBurst/StartLimitAction

 man/systemd.service.xml          |   17 +++++++++++++
 src/dbus-service.c               |    9 +++++++
 src/load-fragment-gperf.gperf.m4 |    4 +++
 src/load-fragment.c              |    1 +
 src/load-fragment.h              |    1 +
 src/service.c                    |   50 +++++++++++++++++++++++++++++++++++++-
 src/service.h                    |   19 ++++++++++++++
 7 files changed, 100 insertions(+), 1 deletions(-)

-- 
1.7.8.3



More information about the systemd-devel mailing list