[systemd-devel] [PATCH v3] service: add StartLimitInterval/StartLimitBurst/StartLimitAction

Lennart Poettering lennart at poettering.net
Thu Feb 9 12:17:38 PST 2012

On Thu, 09.02.12 13:05, Michael Olbrich (m.olbrich at pengutronix.de) wrote:

>  src/dbus-service.c               |    7 +++++
>  src/load-fragment-gperf.gperf.m4 |    3 ++
>  src/load-fragment.c              |    1 +
>  src/load-fragment.h              |    1 +
>  src/service.c                    |   54 ++++++++++++++++++++++++++++++++++---
>  src/service.h                    |   16 ++++++++++-
>  6 files changed, 76 insertions(+), 6 deletions(-)

Thanks! Applied.

I made a minor change (a dbus_error_free() was missing) and added
documentation for the new options.

I also added a bit of code which implies NotifyAccess=main if
WatchdogSec= is used, so that WATCHDOG=1 can actually be delivered.

I guess this leaves two items open for full watchdog support:

a) maybe new Restart=on-watchdog and/or Restart=on-abort-or-timeout
restart options

b) Hookup with /dev/watchdog

Of this a) should be fairly easy I guess, and it's optional I
guess. Would be happy to take patches for this though. b) probably
requires me to make my mind up on how I would like to see this work in
the end. I'll try to think about it.


Lennart Poettering - Red Hat, Inc.

More information about the systemd-devel mailing list