mirrors/haproxy
1
0
Fork 0
mirror of https://git.haproxy.org/git/haproxy.git/ synced 2025-08-10 00:57:02 +02:00
Code Issues Projects Releases Wiki Activity

MINOR: mux-h2: simplify the wake up code in h2_rcv_buf()

Browse Source 
The code used to decide when to restart reading is far from being trivial
and will cause trouble after the forthcoming changes: it checks if the
current stream is the same that is being demuxed, and only if so, wakes
the demux to restart reading. Once streams will start to use multiple
buffers, this condition will make no sense anymore. Actually the real
reason is split into two steps:
  - detect if the demux is currently blocked on the current stream, and
    if so remove SFULL
  - detect if any demux blocking flags were removed during the operations,
    and if so, wake demuxing.

For now this doesn't change anything.
This commit is contained in:
Willy Tarreau 2024-10-11 14:49:45 +02:00
parent a0ed92f3dd
commit b74bedf157
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/mux_h2.c
View File

@ -7193,6 +7193,7 @@ static size_t h2_rcv_buf(struct stconn *sc, struct buffer *buf, size_t count, in
struct htx *h2s_htx = NULL;
struct htx *buf_htx = NULL;
size_t ret = 0;
uint prev_h2c_flags = h2c->flags;
TRACE_ENTER(H2_EV_STRM_RECV, h2c->conn, h2s);
@ -7255,11 +7256,13 @@ static size_t h2_rcv_buf(struct stconn *sc, struct buffer *buf, size_t count, in
h2s_propagate_term_flags(h2c, h2s);
}
if (ret && h2c->dsi == h2s->id) {
/* demux is blocking on this stream's buffer */
/* the demux might have been blocking on this stream's buffer */
if (ret && h2c->dsi == h2s->id)
h2c->flags &= ~H2_CF_DEM_SFULL;
/* wake up processing if we've unblocked something */
if ((prev_h2c_flags & ~h2c->flags) & H2_CF_DEM_SFULL)
h2c_restart_reading(h2c, 1);
}
TRACE_LEAVE(H2_EV_STRM_RECV, h2c->conn, h2s);
return ret;