-
Notifications
You must be signed in to change notification settings - Fork 8.2k
Closed
Labels
EnhancementChanges/Updates/Additions to existing featuresChanges/Updates/Additions to existing featuresarea: Crypto / RNG
Description
Having submitted #6131, I see that there're more issues, so opening a more generic ticket to collect them and consider ways to address.
- Zephyr's mbedTLS configs live in ext/lib/crypto/mbedtls/configs/ , which is not the right location, because these are Zephyr's configs, not 3rd-party content (many configs start from 3rd-party/upstream samples, but they are usually modifies and are Zephyr's own config). This is an old issue, which was already raised IIRC.
- We don't have enough consistency among the configs, but should strive for that. For example, most systems Zephyr targets would have much larger ROM than RAM, and RAM being a scarce resource (especially with TLS usage) at all. So, MBEDTLS_AES_ROM_TABLES should be enabled by default. Currently, only 3 of 5 configs define it (but one of them twice). To achieve manageable consistency for this, we apparently need to introduce multi-level configs, e.g. introduce
config-zephyr-common.hwhich other "leaf" configs would include.
nashif
Metadata
Metadata
Labels
EnhancementChanges/Updates/Additions to existing featuresChanges/Updates/Additions to existing featuresarea: Crypto / RNG