BUG/MEDIUM: applets: Don't use task_in_rq().
When deciding if we want to wake the task of an applet up, don't give up
if task_in_rq returns 1, as there's a race condition and another thread
may run it. Instead, always attempt to task_wakeup(), at worst the task
is already in the run queue, and nothing will happen.
diff --git a/src/stream_interface.c b/src/stream_interface.c
index 501897a..1e50c1f 100644
--- a/src/stream_interface.c
+++ b/src/stream_interface.c
@@ -1468,9 +1468,8 @@
* appctx but in the case the task is not in runqueue we may have to
* wakeup the appctx immediately.
*/
- if (!task_in_rq(si_task(si)) &&
- ((si_rx_endp_ready(si) && !si_rx_blocked(si)) ||
- (si_tx_endp_ready(si) && !si_tx_blocked(si))))
+ if ((si_rx_endp_ready(si) && !si_rx_blocked(si)) ||
+ (si_tx_endp_ready(si) && !si_tx_blocked(si)))
appctx_wakeup(si_appctx(si));
}