Skip to content

Commit

Permalink
Etw manifest and operation cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
guhetier committed Feb 5, 2025
1 parent f2dd1f0 commit 45d32c0
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 1 deletion.
8 changes: 8 additions & 0 deletions src/core/operation.c
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,14 @@ QuicOperationFree(
} else if (ApiCtx->Type == QUIC_API_TYPE_STRM_RECV_SET_ENABLED) {
QuicStreamRelease(ApiCtx->STRM_RECV_SET_ENABLED.Stream, QUIC_STREAM_REF_OPERATION);
} else if (ApiCtx->Type == QUIC_API_TYPE_STRM_PROVIDE_RECV_BUFFERS) {
while (!CxPlatListIsEmpty(&ApiCtx->STRM_PROVIDE_RECV_BUFFERS.Chunks)) {
CXPLAT_FREE(

Check warning on line 124 in src/core/operation.c

View check run for this annotation

Codecov / codecov/patch

src/core/operation.c#L124

Added line #L124 was not covered by tests
CXPLAT_CONTAINING_RECORD(
CxPlatListRemoveHead(&ApiCtx->STRM_PROVIDE_RECV_BUFFERS.Chunks),
QUIC_RECV_CHUNK,
Link),
QUIC_POOL_RECVBUF);
}

Check warning on line 130 in src/core/operation.c

View check run for this annotation

Codecov / codecov/patch

src/core/operation.c#L130

Added line #L130 was not covered by tests
QuicStreamRelease(ApiCtx->STRM_PROVIDE_RECV_BUFFERS.Stream, QUIC_STREAM_REF_OPERATION);
}
CxPlatPoolFree(&Worker->ApiContextPool, ApiCtx);
Expand Down
2 changes: 1 addition & 1 deletion src/core/stream.h
Original file line number Diff line number Diff line change
Expand Up @@ -1033,4 +1033,4 @@ QUIC_STATUS
QuicStreamProvideRecvBuffers(
_In_ QUIC_STREAM* Stream,
_Inout_ CXPLAT_LIST_ENTRY* /* QUIC_RECV_CHUNK */ Chunks
);
);
8 changes: 8 additions & 0 deletions src/manifest/MsQuicEtw.man
Original file line number Diff line number Diff line change
Expand Up @@ -500,6 +500,10 @@
message="$(string.Enum.QUIC_TRACE_API_TYPE.COMPLETE_CERTIFICATE_VALIDATION)"
value="27"
/>
<map
message="$(string.Enum.QUIC_TRACE_API_TYPE.STREAM_PROVIDE_RECEIVE_BUFFERS)"
value="28"
/>
</valueMap>
<valueMap name="map_QUIC_SEND_FLUSH_REASON">
<map
Expand Down Expand Up @@ -4871,6 +4875,10 @@
id="Enum.QUIC_TRACE_API_TYPE.COMPLETE_CERTIFICATE_VALIDATION"
value="COMPLETE_CERTIFICATE_VALIDATION"
/>
<string
id="Enum.QUIC_TRACE_API_TYPE.STREAM_PROVIDE_RECEIVE_BUFFERS"
value="STREAM_PROVIDE_RECEIVE_BUFFERS"
/>
<string
id="Enum.QUIC_SEND_FLUSH_REASON.CONNECTION_FLAGS"
value="CONNECTION_FLAGS"
Expand Down

0 comments on commit 45d32c0

Please sign in to comment.