Page MenuHomeFreeBSD
Differential D34086

ktls: Write-lock the INP when changing a transmit TLS session.
ClosedPublic
Actions

Authored by jhb on Jan 28 2022, 9:30 PM.
Tags
None
Referenced Files
F132112243: D34086.id102686.diff
Mon, Oct 13, 8:01 PM
F132112242: D34086.id102185.diff
Mon, Oct 13, 8:01 PM
F132112240: D34086.id102061.diff
Mon, Oct 13, 8:01 PM
F132112239: D34086.id.diff
Mon, Oct 13, 8:01 PM
Unknown Object (File)
Mon, Oct 13, 6:32 AM
Unknown Object (File)
Thu, Oct 9, 11:28 PM
Unknown Object (File)
Wed, Oct 1, 9:06 PM
Unknown Object (File)
Sep 12 2025, 5:55 PM
View All 92 Files
Subscribers
• hselasky
imp
rrs

Details

Reviewers
gallatin
• hselasky
Commits
rG48e6b7ad3557: ktls: Write-lock the INP when changing a transmit TLS session.
rGcd0525f615fe: ktls: Write-lock the INP when changing a transmit TLS session.
Summary

The TCP rate pacing code relies on being able to read this pointer
safely while holding an INP lock. The initial TLS session pointer is
set while holding the write lock already.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb created this revision.Jan 28 2022, 9:30 PM
Herald added a subscriber: imp. · View Herald TranscriptJan 28 2022, 9:30 PM
jhb requested review of this revision.Jan 28 2022, 9:30 PM
Harbormaster completed remote builds in B44151: Diff 102061.Jan 28 2022, 9:30 PM
jhb added a parent revision: D34085: tcp_ratelimit: Handle some edge cases with TLS + RL send tags..Jan 28 2022, 9:30 PM
gallatin accepted this revision.Jan 28 2022, 9:49 PM
This revision is now accepted and ready to land.Jan 28 2022, 9:49 PM
• hselasky added a comment.Jan 29 2022, 9:50 AM

Is it allowed to lock the INP after SOCKBUF? I thought you needed to lock INP first, then SOCKBUF?

jhb updated this revision to Diff 102185.Feb 1 2022, 12:42 AM
  • Lock INP before socket buffer.
This revision now requires review to proceed.Feb 1 2022, 12:42 AM
Harbormaster completed remote builds in B44205: Diff 102185.Feb 1 2022, 12:43 AM
• hselasky accepted this revision.Feb 1 2022, 9:57 AM
This revision is now accepted and ready to land.Feb 1 2022, 9:57 AM
gallatin accepted this revision.Feb 1 2022, 3:58 PM
Closed by commit rGcd0525f615fe: ktls: Write-lock the INP when changing a transmit TLS session. (authored by jhb). · Explain WhyFeb 11 2022, 11:17 PM
This revision was automatically updated to reflect the committed changes.
jhb added a commit: rGcd0525f615fe: ktls: Write-lock the INP when changing a transmit TLS session..
jhb added a commit: rG48e6b7ad3557: ktls: Write-lock the INP when changing a transmit TLS session..May 11 2022, 12:30 AM

Revision Contents
Changeset List

PathSize
sys/
kern/
2 lines

Diff 102686

View Options

sys/kern/uipc_ktls.c

Loading...