-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Study: PSA_CRYPTO_CONFIG always on #8153
Comments
Note: there is also significant work to update lots of test dependencies |
Remove tests which test the same thing with/without PSA_CRYPTO_CONFIG |
Note: TF-PSA-Crypto has tasks to do some of this already |
We should also take care that the testing done in all.sh components where PSA_CRYPTO_CONFIG is disabled stay eventually the same with PSA_CRYPTO_CONFIG always enabled. In a component where PSA_CRYPTO_CONFIG is disabled and a crypto mechanism is disabled but gets enabled through the PSA_CRYPTO_CONFIG mechanism, the testing is not the same anymore when PSA_CRYPTO_CONFIG is enabled: a crypto mechanism that was intended to be disabled in the test component is enabled eventually. I have been through all.sh components and I have found that the following components are impacted:
|
The MBEDTLS_PSA_CRYPTO_CONFIG has been removed now. |
Remove all options from
mbedtls_config.h
which have equivalent functionality via the crypto config (PSA_WANT_xxx
) mechanism.Look at corresponding clean-up in the library to see if references to the legacy option can be replaced with a suitable PSA macro, or if it needs to be kept as a non-user-visible macro (i.e., set via
config_psa.h
to control what gets built).The list of options is probably simply everything that gets set automatically via
config_adjust_legacy_from_psa.h
andconfig_adjust_psa_superset_legacy.h
, i.e.grep '#define MBEDTLS_' config_adjust_legacy_from_psa.h|sort|uniq|perl -pe 's/#define ([^ ]*).*/\1/'
Also remove
config_adjust_psa_from_legacy.h
andconfig_adjust_psa_superset_legacy.h
(and verify that this does not result in any changes in configuration as a result).config_adjust_ssl.h
should stay but a few references to legacy options probably need to be replaced with the PSA equivalent (e.g. replaceMBEDTLS_ECDH_C
withPSA_WANT_ALG_ECDH
).Remove obsolete parts of
psa/crypto_adjust_config_key_pair_types.h
In summary:
mbedtls_config.h
NOT IN SCOPE OF THE REPO SPLIT WORK. IT WILL BE DONE AS PART OF THE 4.0 WORK AND LIKELY AFTER THE REPO SPLIT.config_adjust
headersMBEDTLS_PSA_CRYPTO_CONFIG
from mbedtls_config.MBEDTLS_PSA_CRYPTO_CONFIG
from the library and testsSubset of #8147
The text was updated successfully, but these errors were encountered: