search

OP-TEE / optee_os

Trusted side of the TEE
Other
1.59k stars 1.07k forks source link

ci: add build with Mbed TLS and crypto extensions #7002

Closed jforissier closed 2 months ago

jforissier commented 2 months ago

Build for the Juno platform with Mbed TLS as the core crypto library. This would have caught the issue fixed in commit 021fee0affe5 ("core: mbedtls: Fix build").

jenswi-linaro commented 2 months ago

Why Juno? Wouldn't it work with QEMU?

jforissier commented 2 months ago

For some reason the build doesn't fail on QEMUv8, I need to check why.

jforissier commented 2 months ago

For some reason the build doesn't fail on QEMUv8, I need to check why.

Ah, the reason is CFG_CRYPTO_WITH_CE. PLATFORM=vexpress-qemu_armv8a will fail in the same way when CFG_CRYPTO_WITH_CE=y. @jenswi-linaro would you prefer I add a QEMU build instead of Juno?

jenswi-linaro commented 2 months ago

For some reason the build doesn't fail on QEMUv8, I need to check why.

Ah, the reason is CFG_CRYPTO_WITH_CE. PLATFORM=vexpress-qemu_armv8a will fail in the same way when CFG_CRYPTO_WITH_CE=y. @jenswi-linaro would you prefer I add a QEMU build instead of Juno?

Yes, please. :-)

jforissier commented 2 months ago

Update