[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
--
Lennart Poettering - Red Hat, Inc.
More information about the systemd-devel
mailing list