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)
Thu, May 9, 10:50 PM
Unknown Object (File)
Thu, May 9, 2:25 PM
Unknown Object (File)
Thu, May 9, 2:21 PM
Unknown Object (File)
Sat, May 4, 1:17 PM
Unknown Object (File)
Thu, Apr 25, 1:32 AM
Unknown Object (File)
Fri, Apr 19, 6:28 AM
Unknown Object (File)
Feb 19 2024, 6:34 PM
Unknown Object (File)
Jan 14 2024, 6:40 AM
View All 12 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 55219
Build 52108: 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
1708

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
1694

Don't you need this same change in rack?

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

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/
6 lines

Diff 132211

View Options

sys/netinet/tcp_output.c

Loading...