Mercurial > hg > nginx
comparison src/event/quic/ngx_event_quic_transport.h @ 9143:48691bab4474
QUIC: fixed probe-congestion deadlock.
When probe timeout expired while congestion window was exhausted, probe PINGs
could not be sent. As a result, lost packets could not be declared lost and
congestion window could not be freed for new packets. This deadlock
continued until connection idle timeout expiration.
Now PINGs are sent separately from the frame queue without congestion control,
as specified by RFC 9002, Section 7:
An endpoint MUST NOT send a packet if it would cause bytes_in_flight
(see Appendix B.2) to be larger than the congestion window, unless the
packet is sent on a PTO timer expiration (see Section 6.2) or when entering
recovery (see Section 7.3.2).
| author | Roman Arutyunyan <arut@nginx.com> |
|---|---|
| date | Mon, 14 Aug 2023 08:28:30 +0400 |
| parents | def8e398d7c5 |
| children | 618132842e7c |
comparison
equal
deleted
inserted
replaced
| 9142:7e8ee4b7cbf4 | 9143:48691bab4474 |
|---|---|
| 269 ngx_msec_t first; | 269 ngx_msec_t first; |
| 270 ngx_msec_t last; | 270 ngx_msec_t last; |
| 271 ssize_t len; | 271 ssize_t len; |
| 272 unsigned need_ack:1; | 272 unsigned need_ack:1; |
| 273 unsigned pkt_need_ack:1; | 273 unsigned pkt_need_ack:1; |
| 274 unsigned flush:1; | |
| 275 | 274 |
| 276 ngx_chain_t *data; | 275 ngx_chain_t *data; |
| 277 union { | 276 union { |
| 278 ngx_quic_ack_frame_t ack; | 277 ngx_quic_ack_frame_t ack; |
| 279 ngx_quic_crypto_frame_t crypto; | 278 ngx_quic_crypto_frame_t crypto; |