Page MenuHomeFreeBSD

ktls(4): Add comment about the enabled OpenSSL support for aarch64 and amd64
ClosedPublic

Authored by gbe on Jan 31 2021, 2:01 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 10, 4:31 PM
Unknown Object (File)
Jan 29 2024, 5:08 PM
Unknown Object (File)
Jan 29 2024, 12:22 PM
Unknown Object (File)
Jan 20 2024, 3:11 AM
Unknown Object (File)
Jan 16 2024, 7:47 PM
Unknown Object (File)
Dec 23 2023, 3:37 AM
Unknown Object (File)
Dec 12 2023, 10:11 AM
Unknown Object (File)
Dec 5 2023, 6:04 PM
Subscribers

Details

Summary

ktls(4): Add comment about the enabled OpenSSL support for aarch64 and amd64

Since KTLS is enabled for aarch64 and amd64 and the shipped OpenSSL version
has also support for KTLS, the man page shoud be updated to reflect this.

Test Plan

mandoc output review and 'mandoc -Tlint' checks.

Diff Detail

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

Event Timeline

gbe requested review of this revision.Jan 31 2021, 2:01 PM

Could some one review this? I would like to get it into releng/13.

Given the serf bug, I'm not sure yet if I really want to merge the OpenSSL bits into 13.0 or not, and if they will be available in 13 at all or not. I would perhaps reword the sentence though as something like:

OpenSSL in the base system includes KTLS support when built with
.Dv WITH_OPENSSL_KTLS
which is enabled by default on aarch64 and amd64.

And then add an .Xr src.conf.5 to the SEE ALSO section.

Given what we decided for 13 (not enable by default), I would amend my earlier suggestion to end the sentence after ".Dv WITH_OPENSSL_KTLS ." and not mention defaults at all. src.conf(5) should probably be the canonical place to describe the default setting of that knob on a given release or branch.

@jhb I updated the differential according to your comments.

This revision is now accepted and ready to land.Feb 6 2021, 9:32 PM