MEDIUM: stconn: Be able to unblock zero-copy data forwarding from done_fastfwd

This part is only experienced by applet. When an applet try to forward data
via an iobuf, it may decide to block for any reason even if there is free
space in the buffer. For instance, the stats applet don't procude data if
the buffer is almost full.

However, in this case, it could be good to let the consumer decide a new
attempt is possible because more space was made. So, if IOBUF_FL_FF_BLOCKED
flag is removed by the consumer when done_fastfwd() callback function is
called, the SE_FL_WANT_ROOM flag is removed on the producer sedesc. It is
only done for applets. And thanks to this change, the applet can be woken up
for a new attempt.

This patch is required for a fix on the QUIC multiplexer.

diff --git a/src/applet.c b/src/applet.c
index c528963c3721babfe04087eab460134be5c7b8af..2ad93161132998d0790d9afad3f80093b84e4fee 100644 (file)
--- a/src/applet.c
+++ b/src/applet.c
@@ -716,8 +716,9 @@ int appctx_fastfwd(struct stconn *sc, unsigned int count, unsigned int flags)
        /* else */
        /*      applet_have_more_data(appctx); */
 
-       if (se_done_ff(sdo) != 0) {
-               /* Something was forwarding, don't reclaim more room */
+       if (se_done_ff(sdo) != 0 || !(sdo->iobuf.flags & IOBUF_FL_FF_BLOCKED)) {
+               /* Something was forwarding or the consumer states it is not
+                * blocked anyore, don't reclaim more room */
                se_fl_clr(appctx->sedesc, SE_FL_WANT_ROOM);
                TRACE_STATE("more room available", APPLET_EV_RECV|APPLET_EV_BLK, appctx);
        }