Skip to content

Add crypto callback mode for SLH-DSA - #11055

Merged
philljj merged 7 commits into
wolfSSL:masterfrom
padelsbach:slh-dsa-cryptocb
Sep 9, 2026
Merged

Add crypto callback mode for SLH-DSA#11055
philljj merged 7 commits into
wolfSSL:masterfrom
padelsbach:slh-dsa-cryptocb

Conversation

@padelsbach

@padelsbach padelsbach commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Description

Includes CB_ONLY mode for SLH-DSA which saves approx 16kB when enabled and offloaded.

Testing

New tests

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@padelsbach
padelsbach force-pushed the slh-dsa-cryptocb branch 2 times, most recently from cd65e2b to 41681d4 Compare August 12, 2026 00:14
@padelsbach
padelsbach marked this pull request as ready for review August 12, 2026 00:15
@github-actions

Copy link
Copy Markdown

retest this please

@github-actions

github-actions Bot commented Aug 12, 2026

Copy link
Copy Markdown

@wolfSSL-Fenrir-bot wolfSSL-Fenrir-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Fenrir Automated Review — PR #11055

Scan targets checked: wolfcrypt-bugs, wolfcrypt-port-bugs, wolfcrypt-rs-bugs, wolfcrypt-src, wolfssl-bugs, wolfssl-src

Findings: 13
13 finding(s) posted as inline comments (see file-level comments below)

This review was generated automatically by Fenrir. Findings are non-blocking.

Comment thread wolfcrypt/src/wc_slhdsa.c
Comment thread wolfcrypt/test/test.c
Comment thread wolfcrypt/test/test.c
Comment thread wolfcrypt/src/wc_slhdsa.c
Comment thread wolfcrypt/src/wc_slhdsa.c
Comment thread tests/swdev/swdev.c
Comment thread wolfcrypt/src/cryptocb.c
Comment thread tests/swdev/swdev.c
Comment thread wolfcrypt/src/wc_slhdsa.c
Comment thread tests/api/test_slhdsa.c
Comment thread wolfcrypt/src/wc_slhdsa.c
Comment thread wolfcrypt/src/wc_slhdsa.c
Comment thread wolfcrypt/src/wc_slhdsa.c
Comment thread wolfcrypt/src/wc_slhdsa.c
Comment thread wolfcrypt/test/test.c
Comment thread tests/swdev/swdev.c
Comment thread tests/swdev/swdev.c
Comment thread tests/api/test_slhdsa.c
Comment thread wolfcrypt/test/test.c
Comment thread wolfcrypt/src/wc_slhdsa.c
Comment thread wolfcrypt/src/cryptocb.c
Comment thread tests/swdev/swdev.c
@padelsbach
padelsbach force-pushed the slh-dsa-cryptocb branch 2 times, most recently from 56ff337 to ca89c81 Compare August 12, 2026 23:38
Comment thread wolfcrypt/src/wc_slhdsa.c Outdated
@padelsbach
padelsbach force-pushed the slh-dsa-cryptocb branch 2 times, most recently from 52202b8 to 85bb892 Compare August 17, 2026 21:06
@padelsbach

padelsbach commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

jenkins retest this please

@padelsbach padelsbach removed their assignment Aug 21, 2026
@padelsbach

Copy link
Copy Markdown
Contributor Author

Fenrir comments addressed. Back to you @philljj

@philljj

philljj commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Retest this please.

(no logs)

1 similar comment
@philljj

philljj commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Retest this please.

(no logs)

@philljj
philljj self-requested a review August 31, 2026 19:44
Comment thread wolfcrypt/src/cryptocb.c Outdated
Comment thread wolfcrypt/src/wc_slhdsa.c Outdated
Comment thread wolfcrypt/src/wc_slhdsa.c Outdated
@philljj philljj assigned padelsbach and unassigned wolfSSL-Bot Aug 31, 2026
@padelsbach
padelsbach force-pushed the slh-dsa-cryptocb branch 3 times, most recently from e805763 to ee1f3aa Compare September 4, 2026 17:40
@padelsbach padelsbach assigned wolfSSL-Bot and unassigned padelsbach Sep 4, 2026
@philljj

philljj commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Retest this please.

(PRB preflight failed)

@philljj
philljj self-requested a review September 9, 2026 02:50
@philljj

philljj commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Tests good with various multi-test scenarios: all-crypto, wolfssl-all, clang, sanitize, smallstack, linuxkm, etc.

@philljj
philljj merged commit f670b0c into wolfSSL:master Sep 9, 2026
387 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants