Differential D42918

tcp: shift PRR sending cadence left to prevent long runs of ACKs not clocking out a segment
ClosedPublic
Actions

Authored by rscheff on Dec 5 2023, 10:47 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Nov 8, 5:09 PM

	Unknown Object (File)
	Oct 12 2024, 8:21 PM

	Unknown Object (File)
	Sep 27 2024, 9:35 PM

	Unknown Object (File)
	Sep 20 2024, 8:59 AM

	Unknown Object (File)
	Sep 19 2024, 5:31 AM

	Unknown Object (File)
	Sep 19 2024, 1:57 AM

	Unknown Object (File)
	Sep 18 2024, 7:11 PM

	Unknown Object (File)
	Sep 7 2024, 4:29 AM

View All 24 Files

Subscribers

Details

Reviewers

cc
thj
rrs
jtl
glebius
zlei
tuexen

Group Reviewers

Commits

rG35707083c6fb: tcp: shift PRR sending cadence slightly left
rG9276ad23b872: tcp: shift PRR sending cadence slightly left

Summary

PRR currently can start by waiting a short run of ACKs
without sending any segments - partially due to limited
transmit.

This change pulls the transmission opportunities slightly
leftward, preventing long runs of ACKs (initially up to 4)
before a segment is allowed for transmit. Depending on
the expected cwnd reduction, and if ACK compression is
happening, at most every other ACK should now elicit a
new segment.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

rscheff created this revision.Dec 5 2023, 10:47 PM

Herald added a reviewer: transport. · View Herald TranscriptDec 5 2023, 10:47 PM

Herald added subscribers: melifaro, imp. · View Herald Transcript

rscheff requested review of this revision.Dec 5 2023, 10:47 PM

Harbormaster completed remote builds in B54808: Diff 131055.Dec 5 2023, 10:48 PM

tuexen accepted this revision.Dec 7 2023, 12:29 PM

This revision is now accepted and ready to land.Dec 7 2023, 12:29 PM

Closed by commit rG9276ad23b872: tcp: shift PRR sending cadence slightly left (authored by rscheff). · Explain WhyDec 7 2023, 2:40 PM

This revision was automatically updated to reflect the committed changes.

rscheff added a commit: rG9276ad23b872: tcp: shift PRR sending cadence slightly left.

rscheff added a commit: rG35707083c6fb: tcp: shift PRR sending cadence slightly left.Dec 14 2023, 11:06 AM

Revision Contents
Changeset List

Path

Size

sys/

netinet/

2 lines

Diff 131133

sys/netinet/tcp_input.c

Loading...