MINOR: stream-int: remove dependency on CO_FL_WAIT_ROOM for rcv_buf()
The only case where this made sense was with mux_h1 but Since we
introduced CS_FL_MAY_SPLICE, we don't need to rely on this anymore,
thus we don't need to clear it either when we do not splice.
There is a last check on this flag used to determine if the rx channel
is full and that cannot go away unless it's changed to use the CS
instead but for now this wouldn't add any benefit so better not do
it yet.
diff --git a/src/stream_interface.c b/src/stream_interface.c
index a2ea7d7..ba82cae 100644
--- a/src/stream_interface.c
+++ b/src/stream_interface.c
@@ -1316,10 +1316,6 @@
/* splice not possible (anymore), let's go on on standard copy */
}
- else {
- /* be sure not to block regular receive path below */
- conn->flags &= ~CO_FL_WAIT_ROOM;
- }
abort_splice:
if (ic->pipe && unlikely(!ic->pipe->data)) {
@@ -1344,7 +1340,7 @@
* recv().
*/
while ((cs->flags & CS_FL_RCV_MORE) ||
- (!(conn->flags & (CO_FL_ERROR | CO_FL_WAIT_ROOM | CO_FL_HANDSHAKE)) &&
+ (!(conn->flags & (CO_FL_ERROR | CO_FL_HANDSHAKE)) &&
(!(cs->flags & (CS_FL_ERROR|CS_FL_EOS))) && !(ic->flags & CF_SHUTR))) {
/* <max> may be null. This is the mux responsibility to set
* CS_FL_RCV_MORE on the CS if more space is needed.