Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update from iperf3 version 3.11 to 3.13 #35

Closed
wants to merge 1 commit into from

Conversation

HughNhan
Copy link
Contributor

To overcome iperf_udp: implement UDP connect retry mechanism #1144

@k-rister
Copy link
Contributor

k-rister commented Mar 8, 2024

What is the status of this PR?

@HughNhan
Copy link
Contributor Author

HughNhan commented Mar 8, 2024

At the time, this update meant to pick up PR esnet/iperf#1402. It has been a while. I think we need to verify the latest and fast forward if it is good.

@k-rister
Copy link
Contributor

k-rister commented Mar 8, 2024

What do we need to do in order to verify the latest?

@HughNhan
Copy link
Contributor Author

HughNhan commented Mar 9, 2024

The problem we had with iperf3 was UDP under congestion. iperf3 uses UDP for both control traffic and data traffic. So when the congestion happens, the conrol traffic got lost and all hell broke loose.
I am not sure if latest has fixed this specific problem or not, but to ensure no regression we should run multibench and compare UDP stream throughputs specifically bench id 21-26. If there are no new failures and/or lower throughput, we can fast forward.

@k-rister
Copy link
Contributor

This has been stale for so long that I'm going to close it. If we want to revisit this then we need to rebase, reopen, and potentially do so with a more up to date version of iperf since we let this PR languish for so long.

@k-rister k-rister closed this Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants