Skip to content

Fix backward compatibility issue of client incorrectly print 0 UDP packets loss#1988

Merged
bmah888 merged 1 commit intoesnet:masterfrom
davidBar-On:issue-1984-UDP-0-loss-despite-non-zero-loss
Feb 23, 2026
Merged

Fix backward compatibility issue of client incorrectly print 0 UDP packets loss#1988
bmah888 merged 1 commit intoesnet:masterfrom
davidBar-On:issue-1984-UDP-0-loss-despite-non-zero-loss

Conversation

@davidBar-On
Copy link
Copy Markdown
Contributor

@davidBar-On davidBar-On commented Dec 20, 2025

Fix backward compatibility issue of client showing UDP 0 packets loss, although packets were lost. The issue is with older iperf3 servers that did not send omitted packets info.

The main issue was that for some reason, the omitted error count variable was not auto initialized to zero. However, when I added a printf() of its value, it was initialized to 0. Maybe a compiler issue, as the later initialization of the variable is done under if statement with no else.

For robustness, also ignored the omitted values if omitting time is not specified. That allows showing the real packets loss when server does not support sending the omitting info.

@bmah888
Copy link
Copy Markdown
Contributor

bmah888 commented Jan 9, 2026

Thanks for the PR! We'll review and comment.

@bmah888 bmah888 self-assigned this Feb 23, 2026
Copy link
Copy Markdown
Contributor

@bmah888 bmah888 left a comment

Choose a reason for hiding this comment

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

I see this problem when doing UDP tests from a relatively new iperf3 to an older iperf3 (such as iperf3-darwin, which seems to ble based on 8.9.1). Need to set a sending rate fast enough trigger some packet loss. The server counts missing packets, but the client reports that the server had no loss.

This PR fixes the issue.

@bmah888 bmah888 merged commit 4dbe8bc into esnet:master Feb 23, 2026
7 checks passed
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.

iperf3 client summary reports 0% UDP loss despite non-zero loss in server output

2 participants