Looks like this was caused by the push operation, and it was caused by accessing an address out of ARM_LIB_STACK.
Needs Triage (15)
Needs Triage (15)
- Wed, Jan 22, 3:31 PM
Assigned: davidhuziji- Wed, Jan 22, 9:32 AM
Assigned: KenLSoft- Wed, Jan 22, 7:57 AM
Assigned: kevin-peng-hao- Fri, Jan 3, 3:44 AM
Assigned: KenLSoft- Dec 26 2019, 8:19 AM
Assigned: kevin-peng-hao- Tue, Jan 7, 11:10 AM
Today
Today
Yesterday
Yesterday
After tfm_core_svc_handler() returns, the execution jumps to tfm_nspm_thread_entry().
Strange is, that during this jump PSP is changed from 0x30000FE0 to 0x30001048.
So the PSP, during main().
PSP = 0x300001000 PSPLIM = 0x30000800
jf549 moved T656: Crypto: Add support for persistent keys from Restricted Project Column to Restricted Project Column on the Restricted Project board.
https://review.trustedfirmware.org/c/trusted-firmware-m/+/3252
https://review.trustedfirmware.org/c/trusted-firmware-m/+/3253
Fri, Jan 24, 10:44 AM · Restricted Project
Hi Ken,
Thank you by taking care about it.
Guess, the issue may be caused by the way we are using TFM. We have to compile TFM Secure in one project (instead of separate libraries). Guess, it may cause that some parts of data/code may be placed to unwanted memory sections.
I will try to analyze the generated map file.
Thu, Jan 23
Thu, Jan 23
Wed, Jan 22
Wed, Jan 22
In preparation, the Mbed Crypto config file is updated to the latest version which contains the proper flags needed to enable persistent keys: https://review.trustedfirmware.org/c/trusted-firmware-m/+/3176/
Wed, Jan 22, 5:19 PM · Restricted Project
Wed, Jan 22, 5:18 PM · Restricted Project
jf549 moved T655: SST: Fix initialisation of object table entry from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Wed, Jan 22, 5:11 PM · Restricted Project
Wed, Jan 22, 5:10 PM · Restricted Project
kevin-peng-hao renamed T644: Build system: clean up and minor refines from Build system: clean up deprecated build configurations to Build system: clean up and minor refines.
Hi Ken,
yes, we are using UART in core to print some info like protection status and boot progress.
For the hw initialization, SystemInit called from startup code is not sufficient enough as it cannot pass status and data to the rest of the platform code written in C. This was the main reason for adding post init function.
Wed, Jan 22, 1:15 AM · Restricted Project
Tue, Jan 21
Tue, Jan 21
davidhuziji moved T574: Move document from tf.org wiki into TF-M code repo from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Tue, Jan 21, 2:22 AM · Restricted Project
Sun, Jan 19
Sun, Jan 19
Hi Andrei,
Sun, Jan 19, 5:43 AM · Restricted Project
Fri, Jan 17
Fri, Jan 17
Changes for this issue had been merged.
Thu, Jan 16
Thu, Jan 16
Thu, Jan 16, 10:07 PM · Restricted Project
Thu, Jan 16, 4:39 PM · Restricted Project
Thu, Jan 16, 3:48 PM · Restricted Project
kevin-peng-hao triaged T650: Update document to clarify the differences between configurations under configs folder as Normal priority.
Wed, Jan 15
Wed, Jan 15
Hi @RobertRostohar ,
All issues are closed. The crypto header issue should follow up in mbedcrypto changes, not in TF-M as we have aligned.
So I'm going to close this root issue.
Please let me know if you have any question.
Wed, Jan 15, 9:42 AM · Restricted Project
Wed, Jan 15, 9:38 AM · Restricted Project
davidwang closed T533: Crypto header clash with mbed-crypto, a subtask of T532: TF-M / CMSIS-Pack Alignment, as Resolved.
Wed, Jan 15, 9:38 AM · Restricted Project
Had the workshop and aligned the solution for this issue.
The changes are made and verified locally for CMSIS Pack.
Next step is to upstream the solution to mbedCrypto.
As it's not TF-M issue anymore, so close this issue.
Wed, Jan 15, 9:38 AM · Restricted Project
Tue, Jan 14
Tue, Jan 14
odeprez added a comment to T554: Build can fail on systems with stack protection enabled by default.
Nerver mind :) Glad if it works now. Cheers, Olivier.