Skip to content

Conversation

@mpg
Copy link
Contributor

@mpg mpg commented Nov 18, 2025

Description

In builds where MBEDTLS_RSA_NO_CRT is disabled, take advantage of the CRT when generating the initial value for base blinding.

See #10476

PR checklist

  • changelog not required because: performance improvement
  • development PR not required because: on the crypto side
  • TF-PSA-Crypto PR provided #TODO
  • framework PR not required
  • 3.6 PR provided HERE
  • tests not required because: covered by existing tests

mpg added 3 commits November 18, 2025 11:38
Will gain a new implementation using the CRT, so we want to hide the
upcoming complexity in a dedicated function.

Signed-off-by: Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com>
Was only used in one place so far, but will be used in
rsa_gen_rand_with_inverse()'s upcoming CRT-based implementation.

Signed-off-by: Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com>
Signed-off-by: Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com>
@mpg mpg self-assigned this Nov 18, 2025
@mpg mpg 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 labels Nov 20, 2025
@mpg mpg added the size-s Estimated task size: small (~2d) label Nov 20, 2025
@mpg mpg marked this pull request as ready for review November 20, 2025 08:45
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 size-s Estimated task size: small (~2d)

Projects

Development

Successfully merging this pull request may close these issues.

1 participant