nginx-quic: src/event/ngx_event

comparison src/event/ngx_event_quic.c @ 8051:38f1be6c603f quic

QUIC: sending probe packets on PTO timer expiration. The PTO handler is split into separate PTO and loss detection handlers that operate interchangeably depending on which timer should be set. The present ngx_quic_lost_handler is now only used for packet loss detection. It replaces ngx_quic_pto_handler if there are packets preceeding largest_ack. Once there is no more such packets, ngx_quic_pto_handler is installed again. Probes carry unacknowledged data previously sent in the oldest packet number, one per each packet number space. That is, it could be up to two probes. PTO backoff is now increased before scheduling next probes.

author	Sergey Kandaurov <pluknet@nginx.com>
date	Wed, 19 Aug 2020 13:24:23 +0300
parents	6e1c88f82280
children	52c88f41d24c

comparison

equal deleted inserted replaced

-:6e1c88f82280
+:38f1be6c603f
 ngx_quic_send_ctx_t *ctx, ngx_queue_t *frames);
 static void ngx_quic_set_packet_number(ngx_quic_header_t *pkt,
 ngx_quic_send_ctx_t *ctx);
 static void ngx_quic_pto_handler(ngx_event_t *ev);
-static ngx_int_t ngx_quic_detect_lost(ngx_connection_t *c, ngx_uint_t ack);
+static void ngx_quic_lost_handler(ngx_event_t *ev);
+static ngx_int_t ngx_quic_detect_lost(ngx_connection_t *c);
+static ngx_int_t ngx_quic_resend_frames(ngx_connection_t *c,
+ngx_quic_send_ctx_t *ctx, ngx_quic_frame_t *start);
 static void ngx_quic_push_handler(ngx_event_t *ev);
 static void ngx_quic_rbtree_insert_stream(ngx_rbtree_node_t *temp,
 ngx_rbtree_node_t *node, ngx_rbtree_node_t *sentinel);
 static ngx_quic_stream_t *ngx_quic_find_stream(ngx_rbtree_t *rbtree,
 {
 return NGX_ERROR;
 }
 }
-return ngx_quic_detect_lost(c, 1);
+return ngx_quic_detect_lost(c);
 }
 static ngx_int_t
 ngx_quic_handle_ack_frame_range(ngx_connection_t *c, ngx_quic_send_ctx_t *ctx,
 static void
 ngx_quic_pto_handler(ngx_event_t *ev)
 {
-ngx_connection_t  *c;
+ngx_uint_t              i;
+ngx_queue_t            *q;
+ngx_connection_t       *c;
+ngx_quic_frame_t       *start;
+ngx_quic_send_ctx_t    *ctx;
+ngx_quic_connection_t  *qc;
 ngx_log_debug0(NGX_LOG_DEBUG_EVENT, ev->log, 0, "quic pto timer");
 c = ev->data;
+qc = c->quic;
-if (ngx_quic_detect_lost(c, 0) != NGX_OK) {
-ngx_quic_close_connection(c, NGX_ERROR);
+qc->pto_count++;
-return;
-}
+for (i = 0; i < NGX_QUIC_SEND_CTX_LAST; i++) {
-c->quic->pto_count++;
+ctx = &qc->send_ctx[i];
+if (ngx_queue_empty(&ctx->sent)) {
+continue;
+}
+q = ngx_queue_head(&ctx->sent);
+start = ngx_queue_data(q, ngx_quic_frame_t, queue);
+if (start->pnum <= ctx->largest_ack
+&& ctx->largest_ack != (uint64_t) -1)
+{
+continue;
+}
+ngx_log_debug3(NGX_LOG_DEBUG_EVENT, c->log, 0,
+"quic pto pnum:%ui pto_count:%ui level:%d",
+start->pnum, c->quic->pto_count, start->level);
+if (ngx_quic_resend_frames(c, ctx, start) != NGX_OK) {
+ngx_quic_close_connection(c, NGX_ERROR);
+return;
+}
+}
 }
 static void
 ngx_quic_push_handler(ngx_event_t *ev)
 return;
 }
 }
+static
+void ngx_quic_lost_handler(ngx_event_t *ev)
+{
+ngx_connection_t  *c;
+ngx_log_debug0(NGX_LOG_DEBUG_EVENT, ev->log, 0, "quic lost timer");
+c = ev->data;
+if (ngx_quic_detect_lost(c) != NGX_OK) {
+ngx_quic_close_connection(c, NGX_ERROR);
+}
+}
 static ngx_int_t
-ngx_quic_detect_lost(ngx_connection_t *c, ngx_uint_t ack)
+ngx_quic_detect_lost(ngx_connection_t *c)
 {
-uint64_t                pn;
 ngx_uint_t              i;
 ngx_msec_t              now, wait, min_wait, thr;
-ngx_queue_t            *q, range;
+ngx_queue_t            *q;
-ngx_quic_frame_t       *f, *start;
+ngx_quic_frame_t       *start;
 ngx_quic_send_ctx_t    *ctx;
 ngx_quic_connection_t  *qc;
 qc = c->quic;
 now = ngx_current_msec;
 min_wait = 0;
-#if (NGX_SUPPRESS_WARN)
+thr = NGX_QUIC_TIME_THR * ngx_max(qc->latest_rtt, qc->avg_rtt);
-thr = 0;
+thr = ngx_max(thr, NGX_QUIC_TIME_GRANULARITY);
-#endif
-if (ack) {
-thr = NGX_QUIC_TIME_THR * ngx_max(qc->latest_rtt, qc->avg_rtt);
-thr = ngx_max(thr, NGX_QUIC_TIME_GRANULARITY);
-}
 for (i = 0; i < NGX_QUIC_SEND_CTX_LAST; i++) {
 ctx = &qc->send_ctx[i];
-if (ngx_queue_empty(&ctx->sent)) {
+while (!ngx_queue_empty(&ctx->sent)) {
-continue;
-}
+q = ngx_queue_head(&ctx->sent);
-if (!ack) {
-thr = ngx_quic_pto(c, ctx);
-}
-q = ngx_queue_head(&ctx->sent);
-do {
 start = ngx_queue_data(q, ngx_quic_frame_t, queue);
 wait = start->last + thr - now;
+ngx_log_debug4(NGX_LOG_DEBUG_EVENT, c->log, 0,
+"quic detect_lost pnum:%ui thr:%M wait:%i level:%d",
+start->pnum, thr, (ngx_int_t) wait, start->level);
 if ((ngx_msec_int_t) wait >= 0) {
 if (min_wait == 0 || wait < min_wait) {
 min_wait = wait;
 }
-if (!ack) {
-break;
-}
 if ((start->pnum > ctx->largest_ack)
+|| ctx->largest_ack == (uint64_t) -1
 || ((ctx->largest_ack - start->pnum) < NGX_QUIC_PKT_THR))
 {
 break;
 }
 }
-pn = start->pnum;
+if (ngx_quic_resend_frames(c, ctx, start) != NGX_OK) {
-ngx_queue_init(&range);
-/* send frames with same packet number to the wire */
-do {
-f = ngx_queue_data(q, ngx_quic_frame_t, queue);
-if (f->pnum != pn) {
-break;
-}
-q = ngx_queue_next(q);
-ngx_queue_remove(&f->queue);
-ngx_queue_insert_tail(&range, &f->queue);
-} while (q != ngx_queue_sentinel(&ctx->sent));
-ngx_quic_congestion_lost(c, start);
-if (ngx_quic_send_frames(c, ctx, &range) != NGX_OK) {
 return NGX_ERROR;
 }
+}
-} while (q != ngx_queue_sentinel(&ctx->sent));
+}
-}
+/* no more preceeding packets */
+if (min_wait == 0) {
+qc->pto.handler = ngx_quic_pto_handler;
+return NGX_OK;
+}
+qc->pto.handler = ngx_quic_lost_handler;
 if (qc->pto.timer_set) {
 ngx_del_timer(&qc->pto);
 }
-if (min_wait > 0) {
+ngx_add_timer(&qc->pto, min_wait);
-ngx_add_timer(&qc->pto, min_wait);
-}
 return NGX_OK;
+}
+static ngx_int_t
+ngx_quic_resend_frames(ngx_connection_t *c, ngx_quic_send_ctx_t *ctx,
+ngx_quic_frame_t *start)
+{
+ngx_queue_t       *q, range;
+ngx_quic_frame_t  *f;
+ngx_queue_init(&range);
+/* send frames with same packet number to the wire */
+q = ngx_queue_head(&ctx->sent);
+do {
+f = ngx_queue_data(q, ngx_quic_frame_t, queue);
+if (f->pnum != start->pnum) {
+break;
+}
+q = ngx_queue_next(q);
+ngx_queue_remove(&f->queue);
+ngx_queue_insert_tail(&range, &f->queue);
+} while (q != ngx_queue_sentinel(&ctx->sent));
+ngx_quic_congestion_lost(c, start);
+return ngx_quic_send_frames(c, ctx, &range);
 }
 ngx_connection_t *
 ngx_quic_open_stream(ngx_connection_t *c, ngx_uint_t bidi)

Mercurial > hg > nginx-quic

comparison src/event/ngx_event_quic.c @ 8051:38f1be6c603f quic