Page MenuHomePhabricator
Create Task
Maniphest T414

Build failed on AN524/musca_b1 platform when -DMBEDTLS_DEBUG=ON
Closed, WontfixPublic
Actions

Description

It looks like code region for crypto is too small for debug version mbedtls.
check below logs:

Error: L6221E: Load region LR_CODE with Load range [0x100c0400,0x1010ae84) overlaps with Load region LR_VENEER with Load range [0x100ff900,0x100ffb40).
Error: L6221E: Execution region TFM_SP_CRYPTO with Execution range [0x100dbca0,0x10104ff8) overlaps with Execution region ER_CODE_CMSE_VENEER with Execution range [0x100ff900,0x100ffb40).
Finished: 0 information, 34 warning and 2 error messages.
app/secure_fw/CMakeFiles/tfm_s_1.dir/build.make:142: recipe for target 'app/secure_fw/tfm_s_1.axf' failed
make[2]: * [app/secure_fw/tfm_s_1.axf] Error 1
CMakeFiles/Makefile2:399: recipe for target 'app/secure_fw/CMakeFiles/tfm_s_1.dir/all' failed
make[1]: * [app/secure_fw/CMakeFiles/tfm_s_1.dir/all] Error 2
Makefile:129: recipe for target 'all' failed
make: *** [all] Error 2

Event Timeline

qixiang created this task.Jul 2 2019, 8:45 AM
KenLSoft assigned this task to kevin-peng-hao.Jul 3 2019, 5:46 AM

Hi Kevin,
Can you help to take a look?
-Ken

adeaarm added a subscriber: adeaarm.Jul 12 2019, 9:50 AM

HI @qixiang , the option to build with the debug version of Mbed Crypto or Mbed TLS is presented in our documentation just for the sake of completeness, but it's not something that we actively guarantee (i.e. we can't guarantee that the debug version of the mbedcrypto/mbedtls library will fit on all our platforms, due to different requirements in size). The option should be left as default and overriden only on those platforms that can afford it, and only on designed debug sessions. In my experience so far with the Crypto service, there is no need to debug the mbedcrypto/mbedtls libraries, as that would be out of scope for a TF-M deployment. Please let me know if you have questions or doubts about this.

qixiang added a comment.Jul 15 2019, 1:46 AM

Thanks for your feedback.

If the AN524 and Musca_b1 have enough memory, we should support all the debug options since the AN524 and Musca platform are the reference platform. Partner maybe use it do anything.

kevin-peng-hao added a comment.Aug 2 2019, 6:27 AM

Hi @qixiang , the problem is right about memory, I don't think we have enough space to support debug options for crypto on both platforms.
If you agree, can we close it as "won't fix". Thanks.

kevin-peng-hao closed this task as Wontfix.Oct 23 2019, 6:29 AM
• susuxof added a subscriber: • susuxof.May 23 2020, 1:17 PM
This comment was removed by dbharbin.
• sonusood2022 added a subscriber: • sonusood2022.May 18 2021, 6:18 PM
This comment was removed by dbharbin.
• sonusood2022 added a comment.May 18 2021, 6:19 PM
This comment was removed by dbharbin.
• sambattherford added a subscriber: • sambattherford.May 20 2021, 1:03 PM
This comment was removed by dbharbin.
• melissachris added a subscriber: • melissachris.May 22 2021, 5:11 AM
This comment was removed by dbharbin.
• AlexaJhon added a subscriber: • AlexaJhon.Jun 3 2021, 12:02 PM
This comment was removed by dbharbin.
• samparker9 added a subscriber: • samparker9.Jul 19 2021, 5:51 AM
This comment was removed by dbharbin.
• roanparker76 added a subscriber: • roanparker76.Aug 13 2021, 4:36 AM
This comment was removed by dbharbin.
• Daviddenk101 added a subscriber: • Daviddenk101.Jan 11 2022, 6:23 AM
This comment was removed by dbharbin.
• simfpayn added a subscriber: • simfpayn.May 3 2022, 7:47 AM
This comment was removed by dbharbin.