Page MenuHomeFreeBSD
Differential D43213

tcp: retry CWR transmission if TSO doesn't support ECN
Needs ReviewPublic
Actions

Authored by rscheff on Dec 27 2023, 2:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jan 26, 11:34 PM
Unknown Object (File)
Wed, Jan 21, 10:08 AM
Unknown Object (File)
Dec 26 2025, 11:47 PM
Unknown Object (File)
Dec 11 2025, 1:10 PM
Unknown Object (File)
Dec 11 2025, 2:10 AM
Unknown Object (File)
Dec 9 2025, 12:11 PM
Unknown Object (File)
Dec 7 2025, 5:47 AM
Unknown Object (File)
Nov 26 2025, 5:20 PM
View All Files
Subscribers
cc
imp
melifaro

Details

Reviewers
np
gallatin
tuexen
rrs
glebius
Group Reviewers
transport
Summary

If the virtio network driver is not told by the host
that proper ECN support is available with TSO, it returns
ENOTSUP. This is not dealt with other than an effective
packet discard - and subsequently, a CWR-flagged TSO
transmission attempt is made until RTO reduces the
congestion window to a single packet.

Split the CWR-flagged TSO transmission in this case,
and resend immediately with the first segment with CWR
as non-TSO.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 55136
Build 52025: arc lint + arc unit

Event Timeline

rscheff created this revision.Dec 27 2023, 2:21 PM
Herald added a reviewer: transport. · View Herald TranscriptDec 27 2023, 2:21 PM
Herald added subscribers: melifaro, imp. · View Herald Transcript
rscheff requested review of this revision.Dec 27 2023, 2:21 PM
Harbormaster completed remote builds in B55136: Diff 131906.Dec 27 2023, 2:22 PM
cc added a subscriber: cc.Jan 3 2024, 2:09 PM
cc added inline comments.
sys/netinet/tcp_output.c
1704

Put this comment below and I think it looks good to me.

rscheff updated this revision to Diff 132211.Jan 3 2024, 5:55 PM
  • make ENOTSUP dedicated case
Harbormaster completed remote builds in B55219: Diff 132211.Jan 3 2024, 5:55 PM
rrs added inline comments.Mar 31 2024, 12:06 PM
sys/netinet/tcp_output.c
1696

Don't you need this same change in rack?

tuexen added inline comments.Apr 1 2024, 12:36 PM
sys/netinet/tcp_output.c
1696

I think we first should agree on how what drivers report in specific error cases... Then fix all stacks...

Revision Contents
Changeset List

PathSize
sys/
netinet/
7 lines

Diff 131906

View Options

sys/netinet/tcp_output.c

Loading...