From d35ff05910d7bc7dfc28f2b665727f38b08848dd Mon Sep 17 00:00:00 2001
From: Alex Knauth <alexander@knauth.org>
Date: Sun, 17 Nov 2024 10:05:14 -0500
Subject: [PATCH] Add a timeout message (#809)

To distinguish between errors where an auto-splitter update took too long, from other errors, such as panics, running out of stack space, etc.
---
 src/auto_splitting/mod.rs | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/src/auto_splitting/mod.rs b/src/auto_splitting/mod.rs
index a14bdb62..9ffaa7e5 100644
--- a/src/auto_splitting/mod.rs
+++ b/src/auto_splitting/mod.rs
@@ -920,6 +920,7 @@ fn run<T: event::CommandSink + TimerQuery + Send>(
 
 fn watchdog<T: event::CommandSink + TimerQuery + Send>(shared_state: Arc<SharedState<T>>) {
     const TIMEOUT: Duration = Duration::from_secs(5);
+    let mut has_timed_out = false;
 
     let Ok(mut state) = shared_state.watchdog_state.lock() else {
         return;
@@ -944,9 +945,15 @@ fn watchdog<T: event::CommandSink + TimerQuery + Send>(shared_state: Arc<SharedS
                 };
 
                 if result.timed_out() {
+                    if !has_timed_out {
+                        log::error!(target: "Auto Splitter", "timeout, no update in {} seconds", TIMEOUT.as_secs_f32());
+                        has_timed_out = true;
+                    }
                     if let Some(auto_splitter) = &*shared_state.auto_splitter.load() {
                         auto_splitter.interrupt_handle().interrupt();
                     }
+                } else {
+                    has_timed_out = false;
                 }
 
                 new_state