tcp: remove superfluous checks for ECN session
Needs ReviewPublic
Actions

Authored by rscheff on Jan 3 2024, 10:22 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Jul 23, 12:54 PM

	Unknown Object (File)
	Tue, Jul 16, 11:32 AM

	Unknown Object (File)
	Wed, Jul 3, 12:29 PM

	Unknown Object (File)
	Jun 23 2024, 9:12 PM

	Unknown Object (File)
	May 15 2024, 2:23 PM

	Unknown Object (File)
	May 9 2024, 10:57 PM

	Unknown Object (File)
	May 9 2024, 8:19 PM

	Unknown Object (File)
	May 9 2024, 2:23 PM

Subscribers

Details

Reviewers

cc
tuexen
rrs

Group Reviewers

transport

Summary

When a congestion control reaction is triggered, and the session
is in RFC3168 ECN mode, eventually a CWR should be sent. However,
sending of the TCP Flag is also gated on the same condition - so
it is not needed to validate before setting the internal tracking
flag.

No external functional change.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 55226
Build 52115: arc lint + arc unit

Event Timeline

rscheff created this revision.Jan 3 2024, 10:22 PM

Herald added a reviewer: transport. · View Herald TranscriptJan 3 2024, 10:22 PM

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

rscheff requested review of this revision.Jan 3 2024, 10:22 PM

Harbormaster completed remote builds in B55226: Diff 132235.Jan 3 2024, 10:22 PM

rrs added inline comments.Mar 31 2024, 12:05 PM

sys/netinet/tcp_stacks/rack.c
5926	The only thing I find a bit strange about this change is that this now means if ECN is not negotiated, and there is any type of loss the tflags2 is going to carry with it forever TF2_ECN_SND_CWR... which should be noted in a comment somewhere for some future person debugging the stacks...