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.

Details

Reviewers

Commits

R10:672784750001: 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

R10 FreeBSD src repository

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

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 R10:672784750001: 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: R10:672784750001: 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 81375

View Options

sys/dev/cxgbe/tom/t4_tls.c

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