From f508b3e091d0ae665a8ac0a56efae0a72f9bab64 Mon Sep 17 00:00:00 2001 From: Richard Patel Date: Tue, 26 Nov 2024 16:03:37 +0000 Subject: [PATCH] tpu: fix overreporting of 'abandoned' metric --- src/app/fdctl/run/tiles/fd_quic_tile.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/app/fdctl/run/tiles/fd_quic_tile.c b/src/app/fdctl/run/tiles/fd_quic_tile.c index 9f2f079b1e..833a38d4c7 100644 --- a/src/app/fdctl/run/tiles/fd_quic_tile.c +++ b/src/app/fdctl/run/tiles/fd_quic_tile.c @@ -399,13 +399,12 @@ quic_stream_rx( fd_quic_conn_t * conn, slot = fd_tpu_reasm_prepare( reasm, conn_uid, stream_id, tspub ); /* infallible */ ctx->metrics.reasm_started++; ctx->metrics.reasm_active++; + conn->srx->rx_streams_active++; } else if( slot->k.state != FD_TPU_REASM_STATE_BUSY ) { ctx->metrics.frag_dup_cnt++; return FD_QUIC_SUCCESS; } - conn->srx->rx_streams_active++; - int reasm_res = fd_tpu_reasm_frag( reasm, slot, data, data_sz, offset ); if( FD_UNLIKELY( reasm_res != FD_TPU_REASM_SUCCESS ) ) { int is_gap = reasm_res==FD_TPU_REASM_ERR_SKIP;