Skip to content

Add MLDSA test programs#772

Open
bjwtaylor wants to merge 2 commits into
Mbed-TLS:developmentfrom
bjwtaylor:remove-mldsa-sign-from-build
Open

Add MLDSA test programs#772
bjwtaylor wants to merge 2 commits into
Mbed-TLS:developmentfrom
bjwtaylor:remove-mldsa-sign-from-build

Conversation

@bjwtaylor
Copy link
Copy Markdown
Contributor

@bjwtaylor bjwtaylor commented Apr 21, 2026

Description

  1. Add the mldsa test programs mldsa_export_public.c mldsa_sign.c, mldsa_verify.c and create a demo script.
  2. Modify the build so the programs only include the symbols that are needed by them.
  3. Add an all.sh component to verify the mldsa_sign is the only program that includes the sign symbols.

resolves: #644

PR checklist

  • changelog provided | not required because: TBC
  • framework PR not required
  • TF-PSA-Crypto development PR provided #HERE
  • TF-PSA-Crypto 1.1 PR not required because: No Backports
  • mbedtls development PR not required because: No changes
  • mbedtls 4.1 PR not required because: No changes
  • mbedtls 3.6 PR not required because: No changes
  • tests provided

@bjwtaylor bjwtaylor changed the title Remove mldsa sign from build Add MLDSA test programs Apr 21, 2026
@bjwtaylor bjwtaylor force-pushed the remove-mldsa-sign-from-build branch from 32e280d to 74b0850 Compare April 23, 2026 07:49
Ben Taylor added 2 commits April 29, 2026 10:53
Signed-off-by: Ben Taylor <ben.taylor@linaro.org>
Enable dead-code elimination for ML-DSA program builds

Signed-off-by: Ben Taylor <ben.taylor@linaro.org>
@bjwtaylor bjwtaylor force-pushed the remove-mldsa-sign-from-build branch from 786427a to 5d4df75 Compare April 29, 2026 10:02
@bjwtaylor bjwtaylor added needs-review Every commit must be reviewed by at least two team members needs-reviewer This PR needs someone to pick it up for review priority-high High priority - will be reviewed soon size-s Estimated task size: small (~2d) labels Apr 29, 2026
@bjwtaylor bjwtaylor marked this pull request as ready for review April 29, 2026 14:29
@bjwtaylor bjwtaylor requested a review from valeriosetti June 4, 2026 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review Every commit must be reviewed by at least two team members needs-reviewer This PR needs someone to pick it up for review priority-high High priority - will be reviewed soon size-s Estimated task size: small (~2d)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ML-DSA build with export-public and verify but not sign

1 participant