[PATCH BlueZ v1 18/27] shared/timeout: Run timeouts of 0 once idle

[Luiz Augusto von Dentz <luiz.dentz@gmail.com>]

From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>

With the mainloop backend timeout_add() with 0 never runs, as a timer
of 0 is disarmed, e.g. bt_hci's deferred event filter update. Use an
idle callback for it instead.

With GLib it runs at default priority, before pending events, so run it
at idle priority, as with the mainloop backend, which is what deferring
to the next iteration means.

Assisted-by: Claude:claude-opus-5-5
---
 src/shared/timeout-glib.c     | 10 +++++++++-
 src/shared/timeout-mainloop.c |  9 ++++++++-
 2 files changed, 17 insertions(+), 2 deletions(-)

diff --git a/src/shared/timeout-glib.c b/src/shared/timeout-glib.c
index 3268d480c3a7..549e3fa5822b 100644
--- a/src/shared/timeout-glib.c
+++ b/src/shared/timeout-glib.c
@@ -52,7 +52,15 @@ unsigned int timeout_add(unsigned int timeout, timeout_func_t func,
 	data->destroy = destroy;
 	data->user_data = user_data;
 
-	id = g_timeout_add_full(G_PRIORITY_DEFAULT, timeout, timeout_callback,
+	/* A timeout of 0 runs once nothing else is pending, as with the
+	 * mainloop backend.
+	 */
+	if (timeout)
+		id = g_timeout_add_full(G_PRIORITY_DEFAULT, timeout,
+					timeout_callback, data,
+					timeout_destroy);
+	else
+		id = g_idle_add_full(G_PRIORITY_DEFAULT_IDLE, timeout_callback,
 						data, timeout_destroy);
 	if (!id)
 		g_free(data);
diff --git a/src/shared/timeout-mainloop.c b/src/shared/timeout-mainloop.c
index 9be803cda020..e6f9c5e826b3 100644
--- a/src/shared/timeout-mainloop.c
+++ b/src/shared/timeout-mainloop.c
@@ -54,7 +54,14 @@ unsigned int timeout_add(unsigned int timeout, timeout_func_t func,
 	data->timeout = timeout;
 	data->destroy = destroy;
 
-	data->id = mainloop_add_timeout(timeout, timeout_callback, data,
+	/* A timeout of 0 runs as soon as the mainloop gets to it, as with the
+	 * other backends, while mainloop_add_timeout() leaves it disarmed.
+	 */
+	if (timeout)
+		data->id = mainloop_add_timeout(timeout, timeout_callback, data,
+							timeout_destroy);
+	else
+		data->id = mainloop_add_idle(timeout_callback, data,
 							timeout_destroy);
 	if (data->id < 0) {
 		free(data);
-- 
2.56.0