Page MenuHomeFreeBSD

ktls: Support for TLS 1.3 receive offload.
ClosedPublic

Authored by jhb on Nov 16 2021, 12:37 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 22, 2:47 AM
Unknown Object (File)
Thu, Apr 18, 9:06 AM
Unknown Object (File)
Thu, Apr 18, 6:41 AM
Unknown Object (File)
Wed, Apr 17, 10:45 PM
Unknown Object (File)
Mar 12 2024, 8:43 PM
Unknown Object (File)
Mar 7 2024, 2:12 AM
Unknown Object (File)
Mar 7 2024, 2:12 AM
Unknown Object (File)
Mar 7 2024, 2:08 AM

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 42803
Build 39691: arc lint + arc unit

Event Timeline

jhb requested review of this revision.Nov 16 2021, 12:37 AM

I've tested this with the tests here as well as with an OpenSSL patched with the patches from https://github.com/openssl/openssl/pull/16798.

sys/opencrypto/ktls_ocf.c
668

For NIC TLS RX support we may end up making this bit of code a helper routine that can be shared with the NIC TLS RX path.

sys/kern/uipc_ktls.c
1991

Could the record_type be extracted outside this function? We will need this for the hardware decrypted traffic.

sys/opencrypto/ktls_ocf.c
668

Sounds like a good idea, to factor this bit out. Then you don't really need two separate decryption functions.

sys/opencrypto/ktls_ocf.c
668

You would still need separate decryption functions as some of the other details are different such as the AAD. I think splitting out this routine is probably something sensible to do in a future commit in a series adding 1.3 NIC TLS RX, but I might move it back to sys/kern/uipc_ktls.c. I had started with doing it in uipc_ktls.c but found it simpler to do it here instead.

sys/opencrypto/ktls_ocf.c
668

Should we have another callback function into OCF, which handle already decrypted traffic, to get the trailer length and header type fields correct?

  • Move routine to parse TLS 1.3 trailer to uipc_ktls.c.
jhb marked an inline comment as done.Dec 3 2021, 7:46 PM
jhb added inline comments.
sys/kern/uipc_ktls.c
1991

I think this version should work for you for NIC TLS as you can fall through to the code below with the decrypted record.

jhb marked an inline comment as done.Dec 9 2021, 12:33 AM

Looks good. I'll rebase my patches.

This revision is now accepted and ready to land.Dec 13 2021, 2:22 PM
This revision was automatically updated to reflect the committed changes.