core/manager: do not exclude watchdog logic from busy-loop protection

As reported in https://github.com/systemd/systemd/issues/35405, if the watchdog
ping failed, we effectively started a busy loop here. The previous commits
should fix this, but in general, the protection here is intended as a safety
net in case the logic is broken somewhere else. We shouldn't exclude the
watchdog stuff from this.

diff --git a/src/core/manager.c b/src/core/manager.c
index 45f3464dd443e1275ab79148743d952c0f7edd20..afccb48d7edda30fcc3f132f8faad5acde8631ac 100644 (file)
--- a/src/core/manager.c
+++ b/src/core/manager.c
@@ -3352,14 +3352,14 @@ int manager_loop(Manager *m) {
 
         while (m->objective == MANAGER_OK) {
 
-                (void) watchdog_ping();
-
                 if (!ratelimit_below(&rl)) {
                         /* Yay, something is going seriously wrong, pause a little */
                         log_warning("Looping too fast. Throttling execution a little.");
                         sleep(1);
                 }
 
+                (void) watchdog_ping();
+
                 if (manager_dispatch_load_queue(m) > 0)
                         continue;