From ddbeb20633b0cfaf03e98e5e6b0b4885e9241231 Mon Sep 17 00:00:00 2001
From: Joris van Rantwijk <joris@jorisvr.nl>
Date: Sun, 29 Sep 2024 22:15:58 +0200
Subject: [PATCH] No multithreading for now

---
 sw/src/userspace/remotectl.cpp | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/sw/src/userspace/remotectl.cpp b/sw/src/userspace/remotectl.cpp
index bd763b1..b0b0c7f 100644
--- a/sw/src/userspace/remotectl.cpp
+++ b/sw/src/userspace/remotectl.cpp
@@ -1832,11 +1832,14 @@ int run_remote_control_server(
     acq_server.start_server();
     timetagger_server.start_server();
 
+    // We could start a few background threads, each calling io.run().
+    // It may improve performance a little bit.
+    // However, if there are synchronization bugs, it would cause weird issues
+    // that are difficult to track down. So let's keep just one thread for now.
+
     log(LOG_INFO, "Running, press Ctrl-C to stop");
     io.run();
 
-    // TODO -- multi-threading
-
     return command_handler.exit_status();
 }