ucode: fix ubus defer when running from within eloop (integrated with uloop)
authorFelix Fietkau <nbd@nbd.name>
Wed, 1 May 2024 17:00:10 +0000 (19:00 +0200)
committerFelix Fietkau <nbd@nbd.name>
Wed, 1 May 2024 17:00:21 +0000 (19:00 +0200)
Signed-off-by: Felix Fietkau <nbd@nbd.name>
package/utils/ucode/patches/100-ubus-fix-uc_ubus_have_uloop-for-eloop-uloop-combinat.patch [new file with mode: 0644]

diff --git a/package/utils/ucode/patches/100-ubus-fix-uc_ubus_have_uloop-for-eloop-uloop-combinat.patch b/package/utils/ucode/patches/100-ubus-fix-uc_ubus_have_uloop-for-eloop-uloop-combinat.patch
new file mode 100644 (file)
index 0000000..a1659be
--- /dev/null
@@ -0,0 +1,26 @@
+From: Felix Fietkau <nbd@nbd.name>
+Date: Wed, 1 May 2024 18:40:19 +0200
+Subject: [PATCH] ubus: fix uc_ubus_have_uloop for eloop+uloop combination
+
+When uloop has been integrated with eloop (in hostapd/wpa_supplicant),
+uloop_cancelling will return false, since uloop_run is not being called.
+This leads to ubus.defer() calling uloop_run in a context where it can
+prevent the other event loop from running.
+
+Fix this by detecting event loop integration via uloop_fd_set_cb being set
+
+Signed-off-by: Felix Fietkau <nbd@nbd.name>
+---
+
+--- a/lib/ubus.c
++++ b/lib/ubus.c
+@@ -665,6 +665,9 @@ uc_ubus_have_uloop(void)
+       bool prev = uloop_cancelled;
+       bool active;
++      if (uloop_fd_set_cb)
++              return true;
++
+       uloop_cancelled = true;
+       active = uloop_cancelling();
+       uloop_cancelled = prev;