fix up watchdog init code
authorJohn Crispin <blogic@openwrt.org>
Wed, 10 Jul 2013 12:10:39 +0000 (14:10 +0200)
committerJohn Crispin <blogic@openwrt.org>
Wed, 10 Jul 2013 22:29:42 +0000 (00:29 +0200)
Signed-off-by: John Crispin <blogic@openwrt.org>
state.c
watchdog.c

diff --git a/state.c b/state.c
index cf0c81d..71890a2 100644 (file)
--- a/state.c
+++ b/state.c
@@ -46,6 +46,9 @@ static void state_enter(void)
                break;
 
        case STATE_INIT:
                break;
 
        case STATE_INIT:
+               // check if the wdt appeared during coldplug
+               if (!watchdog_fd())
+                       watchdog_init();
                LOG("- init -\n");
                log_init();
                procd_connect_ubus();
                LOG("- init -\n");
                log_init();
                procd_connect_ubus();
index 2a516b4..e4b82f5 100644 (file)
@@ -95,20 +95,19 @@ void watchdog_init(void)
 {
        char *env = getenv("WDTFD");
 
 {
        char *env = getenv("WDTFD");
 
+
        wdt_timeout.cb = watchdog_timeout_cb;
        if (env) {
        wdt_timeout.cb = watchdog_timeout_cb;
        if (env) {
-               LOG("- watchdog -\n");
                DEBUG(1, "Watchdog handover: fd=%s\n", env);
                wdt_fd = atoi(env);
                DEBUG(1, "Watchdog handover: fd=%s\n", env);
                wdt_fd = atoi(env);
-               fcntl(wdt_fd, F_SETFD, fcntl(wdt_fd, F_GETFD) | FD_CLOEXEC);
                unsetenv("WDTFD");
        } else {
                wdt_fd = open("/dev/watchdog", O_WRONLY);
                unsetenv("WDTFD");
        } else {
                wdt_fd = open("/dev/watchdog", O_WRONLY);
-               if (getpid() != 1)
-                       fcntl(wdt_fd, F_SETFD, fcntl(wdt_fd, F_GETFD) | FD_CLOEXEC);
        }
        if (wdt_fd < 0)
                return;
        }
        if (wdt_fd < 0)
                return;
+       fcntl(wdt_fd, F_SETFD, fcntl(wdt_fd, F_GETFD) | FD_CLOEXEC);
+       LOG("- watchdog -\n");
        watchdog_timeout(30);
        watchdog_timeout_cb(&wdt_timeout);
 
        watchdog_timeout(30);
        watchdog_timeout_cb(&wdt_timeout);