On Thu, 09.02.12 13:05, Michael Olbrich ([email protected]) 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.
_______________________________________________
systemd-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Reply via email to