-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
PSA always enabled in 4.0 #8147
Comments
See #5156 for earlier discussion on this |
I think we also want to identify test code that becomes redundant. For example, a lot of things are tested both with and without My gut feeling is that this task is larger than M - it's the kind of thing that looks easy on paper but where you tend to run into surprises. So, I'd be inclined to split it into smaller tasks - I think the first three items could be one task each. Also, I agree with leaving some parts (remove dead code) to follow-up tasks but I'd strongly advise we address those tasks ASAP. One of the expected benefits of this work is to make maintenance easier, and we don't fully get that while there's still a lot of dead code around. |
Test code will be removed when we remove dead code, based on searching for |
We have now created issues for each part, so this planning issue is completed. |
MBEDTLS_USE_PSA_CRYPTO
must be removed frommbedtls_config.h
. The library must behave as if it was always enabled. → Always build with MBEDTLS_USE_PSA_CRYPTO and MBEDTLS_PSA_CRYPTO_CLIENT enabled #9630MBEDTLS_PSA_CRYPTO_C
/MBEDTLS_PSA_CRYPTO_CLIENT
must be always enabled. Disabling both would be needed for people who want an alternative PSA impl (this would not be a supported/validated case) → Always build with MBEDTLS_USE_PSA_CRYPTO and MBEDTLS_PSA_CRYPTO_CLIENT enabled #9630MBEDTLS_PSA_CRYPTO_CONFIG
must be removed, and treated as always-enabled. → Study: PSA_CRYPTO_CONFIG always on #8153#if !defined
that can never be built). Create tasks to remove this code. → Remove code that is only used when MBEDTLS_USE_PSA_CRYPTO is disabled #9631 + Study: PSA_CRYPTO_CONFIG always on #8153The text was updated successfully, but these errors were encountered: