Differential D14937

Fix LRO window comparison
ClosedPublic
Actions

Authored by jtl on Apr 2 2018, 6:46 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Jul 1, 2:04 PM

	Unknown Object (File)
	Wed, Jul 1, 2:02 PM

	Unknown Object (File)
	Tue, Jun 30, 3:31 PM

	Unknown Object (File)
	Sat, Jun 27, 6:26 AM

	Unknown Object (File)
	Fri, Jun 26, 1:53 PM

	Unknown Object (File)
	Wed, Jun 24, 6:53 PM

	Unknown Object (File)
	Jun 2 2026, 4:52 PM

	Unknown Object (File)
	May 20 2026, 2:55 PM

Subscribers

Details

Reviewers

jason_eggnet.com
gallatin
rrs
rstone
kbowling

Group Reviewers

Commits

rS331926: r330675 introduced an extra window check in the LRO code to ensure it

Summary

rS330675 (D14540) introduced an extra window check in the LRO code to ensure it captured and reported the highest window advertisement with the same SEQ/ACK. However, the window comparison uses a two's complement comparison rather than a comparison of the absolute values. Because windows use absolute values, rather than two's complement arithmetic (i.e. they don't wrap), compare the absolute values.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jtl created this revision.Apr 2 2018, 6:46 PM

Herald added a reviewer: transport. · View Herald TranscriptApr 2 2018, 6:46 PM

Herald added a subscriber: imp. · View Herald Transcript

Harbormaster completed remote builds in B15944: Diff 40998.Apr 2 2018, 6:46 PM

kbowling accepted this revision.Apr 3 2018, 6:15 AM

This revision is now accepted and ready to land.Apr 3 2018, 6:15 AM

gallatin accepted this revision.Apr 3 2018, 12:44 PM

Closed by commit rS331926: r330675 introduced an extra window check in the LRO code to ensure it (authored by jtl). · Explain WhyApr 3 2018, 1:54 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

netinet/

8 lines

Diff 41018

head/sys/netinet/tcp_seq.h

Loading...