Skip to content

Modify dlopen.c and tfpsacrypto_dlopen.c so that they use PSA API-only dynamic loading #10256

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 7 commits into
base: development
Choose a base branch
from

Conversation

ariwo17
Copy link
Contributor

@ariwo17 ariwo17 commented Jun 26, 2025

Description

  • Replace soon-deprecated mbedtls_md_list() in dlopen.c with psa_hash_compute()
  • Add tfpsacrypto_dlopen.c as a PSA-only shared-library loading test
  • Enable -fPIC for tf-psa-crypto builtins to support shared linking
  • Confirm clean builds and successful dlopen() test execution.

PR checklist

Please remove the segment/s on either side of the | symbol as appropriate, and add any relevant link/s to the end of the line.
If the provided content is part of the present PR remove the # symbol.

  • changelog not required because this change is only to test programs
  • development PR not required because this PR is targeting "development" directly
  • TF-PSA-Crypto PR provided Use PSA API for demo in dlopen.c #10153
  • framework PR not required
  • 3.6 PR not required because this only applies to Mbed TLS 4.0
  • tests not required because this is itself a runtime test rather than an automated test suite.

Notes for the submitter

Please refer to the contributing guidelines, especially the
checklist for PR contributors.

Help make review efficient:

  • Multiple simple commits
    • please structure your PR into a series of small commits, each of which does one thing
  • Avoid force-push
    • please do not force-push to update your PR - just add new commit(s)
  • See our Guidelines for Contributors for more details about the review process.

…nly dynamic loading

- Replaced soon-deprecated mbedtls_md_list() in dlopen.c with psa_hash_compute()
- Added tfpsacrypto_dlopen.c as a PSA-only shared-library loading test
- Enabled -fPIC for tf-psa-crypto builtins to support shared linking
- Confirmed clean builds and successful dlopen() test execution.

Signed-off-by: Ari Weiler-Ofek <ariwei01@e134794.arm.com>
Ari Weiler-Ofek added 5 commits June 26, 2025 17:49
Signed-off-by: Ari Weiler-Ofek <ariwei01@e134794.arm.com>
Signed-off-by: Ari Weiler-Ofek <ariwei01@e134794.arm.com>
Signed-off-by: Ari Weiler-Ofek <ariwei01@e134794.arm.com>
Signed-off-by: Ari Weiler-Ofek <ariwei01@e134794.arm.com>
This reverts commit f7cb8bc.

Signed-off-by: Ari Weiler-Ofek <ariwei01@e134794.arm.com>
Signed-off-by: Ari Weiler-Ofek <ariwei01@e134794.arm.com>
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.

1 participant