Skip to content
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

Do not Review: [nrf noup]: Threading: Adding usage of PSA_CRYPTO_THREAD_SAFE #16

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

frkv
Copy link
Contributor

@frkv frkv commented Oct 24, 2024

-This commit allows for thread-safety to be enabled for legacy and/or
HW accelerator mutex support regardless of the thread-safety of
the PSA core in itself. It does this by changing the usage of
MBEDTLS_THREADING_C define in PSA core scope (which is shared) into
PSA_CRYPTO_THREAD_SAFE which is intended to only apply for the
core itself (3 mutexes: for global data, rng and slot management.
-This commit is dependent on propagating of PSA_CRYPTO_THREAD_SAFE
in the generated configuration file (nrf-psa-crypto-config.h)

Note: This commit is being tested out to see if we can achieve a ABI compliance for pre-compiled OpenThread libraries that are currently built to be shared between different build-types.

This is not ready for review. It is used to run tests for the aforementioned ABI compliance issues

-This commit allows for thread-safety to be enabled for legacy and/or
 HW accelerator mutex support regardless of the thread-safety of
 the PSA core in itself. It does this by changing the usage of
 MBEDTLS_THREADING_C define in PSA core scope (which is shared) into
 PSA_CRYPTO_THREAD_SAFE which is intended to only apply for the
 core itself (3 mutexes: for global data, rng and slot management.
-This commit is dependent on propagating of PSA_CRYPTO_THREAD_SAFE
 in the generated configuration file (nrf-psa-crypto-config.h)

Note: This commit is being tested out to see if we can achieve a
ABI compliance for pre-compiled OpenThread libraries that are currently
built to be shared between different build-types.

Signed-off-by: Frank Audun Kvamtrø <frank.kvamtro@nordicsemi.no>
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