Skip to content

QOL: speed up the receive window / zwp test - #337

Open
jthomas43 wants to merge 2 commits into
holepunchto:mainfrom
jthomas43:receive-window-2-test-speedup
Open

QOL: speed up the receive window / zwp test#337
jthomas43 wants to merge 2 commits into
holepunchto:mainfrom
jthomas43:receive-window-2-test-speedup

Conversation

@jthomas43

@jthomas43 jthomas43 commented Oct 30, 2026

Copy link
Copy Markdown
Contributor
  • sends less data overall, only needs ~3 zero window probes
  • test ends on reading EOF instead of waiting for the timer to drain the applications buffer

Also fixes up a few comments.

@jthomas43
speed up receive window / zwp test: 1. send less data 2. end when the…
9bc08f0 
… end is received instead of when the data is drained. test takes ~7s now, down from ~19s.
@jthomas43
jthomas43 requested review from a team and marcus-pousette-hp October 30, 2026 16:07
marcus-pousette-hp
marcus-pousette-hp requested changes Nov 2, 2026
View reviewed changes

@marcus-pousette-hp marcus-pousette-hp left a comment •
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit otherwise good

Co-reviewed with AI

Comment thread test/stream-write-read-receive-window-2.c Resolved
@jthomas43 @marcus-pousette-hp
Update test/stream-write-read-receive-window-2.c
176f743 
Co-authored-by: Marcus Pousette <marcus.pousette@tether.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@marcus-pousette-hp marcus-pousette-hp marcus-pousette-hp approved these changes

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jthomas43 @marcus-pousette-hp