Don't try to adjust a TLS TOE socket that has been closed.
ClosedPublic
Actions

Authored by jhb on Dec 11 2020, 9:42 PM.

Tags

None

Referenced Files

	F167889845: D27583.diff
	Tue, Aug 25, 6:56 AM

	Unknown Object (File)
	Sun, Aug 23, 6:36 AM

	Unknown Object (File)
	Sat, Aug 22, 6:29 PM

	Unknown Object (File)
	Sat, Aug 22, 3:38 PM

	Unknown Object (File)
	Sat, Aug 22, 2:05 PM

	Unknown Object (File)
	Fri, Aug 21, 2:58 PM

	Unknown Object (File)
	Fri, Aug 21, 12:38 PM

	Unknown Object (File)
	Fri, Aug 21, 6:29 AM

View All Files

Subscribers

imp

Details

Reviewers

Commits

rG672784750001: Don't try to adjust a TLS TOE socket that has been closed.

Summary

The handshake timer can race with another thread sending a FIN or RST
to close a TOE TLS socket. Just bail from the timer without
rescheduling if the connection is closed when the timer fires.

Test Plan

tested by Chelsio QA

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 35363
Build 32289: arc lint + arc unit

Event Timeline

jhb created this revision.Dec 11 2020, 9:42 PM

Herald added subscribers: ae, imp. · View Herald TranscriptDec 11 2020, 9:42 PM

jhb requested review of this revision.Dec 11 2020, 9:42 PM

Harbormaster completed remote builds in B35363: Diff 80612.Dec 11 2020, 9:43 PM

np accepted this revision.Dec 20 2020, 7:14 AM

This revision is now accepted and ready to land.Dec 20 2020, 7:14 AM

Closed by commit rG672784750001: Don't try to adjust a TLS TOE socket that has been closed. (authored by jhb). · Explain WhyDec 30 2020, 6:02 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rG672784750001: Don't try to adjust a TLS TOE socket that has been closed..

Revision Contents
Changeset List

Path

Size

sys/

dev/

cxgbe/

tom/

t4_tls.c

4 lines

Diff 80612

View Options

sys/dev/cxgbe/tom/t4_tls.c

Don't try to adjust a TLS TOE socket that has been closed.ClosedPublicActions