Skip to content

Comments

Minor follow-up commits to GSO/GRO feature#2007

Merged
bmah888 merged 3 commits intomasterfrom
bmah-gsro-follow-up
Feb 20, 2026
Merged

Minor follow-up commits to GSO/GRO feature#2007
bmah888 merged 3 commits intomasterfrom
bmah-gsro-follow-up

Conversation

@bmah888
Copy link
Contributor

@bmah888 bmah888 commented Feb 19, 2026

  • Version of iperf3 (or development branch, such as master or
    3.1-STABLE) to which this pull request applies: master

  • Issues fixed (if any):

  • Brief description of code changes (suitable for use as a commit message):

Minor follow-up commits after PR #1925.

@bmah888 bmah888 requested a review from swlars February 19, 2026 23:51
@bmah888 bmah888 marked this pull request as draft February 19, 2026 23:52
@bmah888 bmah888 marked this pull request as ready for review February 20, 2026 00:40
@bmah888
Copy link
Contributor Author

bmah888 commented Feb 20, 2026

@gegles @davidBar-On I can't request you two as reviewers because GitHub won't let me (because you don't have write access to this repo?), but would appreciate any comments you have on these changes.

Copy link
Contributor

@gegles gegles left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @bmah888!

@davidBar-On
Copy link
Contributor

Added one comment (change debug to debug_level. No other comments.

@bmah888 bmah888 force-pushed the bmah-gsro-follow-up branch from 8520f87 to 5a3b09b Compare February 20, 2026 16:08
Restore comments and debug logging in iperf_udp_recv().

Make some debug output in iperf_udp_send() conditional on
DEBUG_LEVEL_DEBUG.

Suggested by:  @davidBar-On
@bmah888 bmah888 force-pushed the bmah-gsro-follow-up branch from 5a3b09b to 593b79d Compare February 20, 2026 18:43
@bmah888 bmah888 merged commit c9771fd into master Feb 20, 2026
7 checks passed
@bmah888 bmah888 deleted the bmah-gsro-follow-up branch February 20, 2026 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants