Skip to content

Conversation

@philljj
Copy link
Contributor

@philljj philljj commented Nov 5, 2025

Description

Initial wolfcrypt-only, no-asm, FreeBSD kernel module support.

Build with --enable-freebsdkm.

Options:

  • bsd kernel tree root: --with-kernel-source=PATH (default /usr/src/sys).
  • bsd export syms: --with-bsd-export-syms=LIST. LIST = YES, NO, or list of symbols. (default NO). Exporting symbols is not strictly necessary if dependent modules use MODULE_DEPEND().

Testing

Tested with:

./configure --enable-freebsdkm --enable-cryptonly --enable-crypttests --enable-all-crypto
make
sudo kldload bsdkm/libwolfssl.ko

Also tested with bsd_example.ko that calls wolfcrypt API. See kernel/bsdkm/README.md in wolfssl-examples PR for full details:

@philljj philljj self-assigned this Nov 5, 2025
@devin-ai-integration

This comment was marked as resolved.

Copy link
Contributor

@douzzer douzzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1st round -- next round, will want help actually building locally. Actually naah, I have an IRL FreeBSD server online that I'll try it on. It's currently 14.2-RELEASE-p1.

@philljj
Copy link
Contributor Author

philljj commented Nov 17, 2025

Retest this please.

(FIPS windows test misc issue).

@philljj philljj removed their assignment Nov 17, 2025
Copy link
Contributor

@douzzer douzzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is nearly ready. Left a few nits around warnings/typography, and one substantive to-do around setting up the target object list.

I ran the PR through '.*insmod.*' '.*linuxkm.*' rebased on then-current master and It Was Good:

final tally for 1f5701111e-on-5c421a152d with build env f2ab35ee71: all 104 selected checks succeeded.

@philljj philljj requested a review from douzzer November 18, 2025 15:08
Copy link
Contributor

@douzzer douzzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ran through full multi-test:

final tally for 32db8b89e0 with build env 15c3562ffd: all 418 selected checks succeeded.

@philljj philljj removed their assignment Nov 19, 2025
@JacobBarthelmeh JacobBarthelmeh merged commit 96dde5b into wolfSSL:master Nov 19, 2025
292 of 293 checks passed
@philljj philljj deleted the bsdkm branch November 21, 2025 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

For This Release Release version 5.8.4

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants