Hi Thomas,
Looks like an MPU setting issue.
But have no idea why single step running
could pass.
I do noticed something is missing in the
IAR linker script for L3 if you check the one for armclang:
https://git.trustedfirmware.org/TF-M/trusted-firmware-m.git/tree/platform/ext/common/armclang/tfm_isolation_l3.sct.template#n63
The IAR one is missing the *_ ATTR_*
part.
Those allow TFM_LINK_SET_XX_IN_PARTITION_SECTION
to tell the linker to put symbols in particular sections.
I think that’s the root cause.
I'm working on
the IAR part of this and I'm having a problem with the
following test in isolation level 3:
---
Running Test Suite TFM IRQ Test (TFM_IRQ_TEST_1xxx)...
> Executing 'TFM_NS_IRQ_TEST_SLIH_1001'
Description: 'SLIH HANDLING Case 1'
---
If I run the test at full speed, it will run into a MemManage
fault while if I single step through the entire test it will
pass.
Is there a race condition somewhere in there?
If I disable the SLIH tests it succeeds.
The exception happens when dereferencing "dev" in this
function:
bool
timer_cmsdk_is_initialized(const struct timer_cmsdk_dev_t*
dev)
{
return dev->data->is_initialized;
}
dev is 0x30002440 at the time.
I've been trying to figure out what part of the memory
protection system it is that is causing the exception.
DACCVIOL is set and MMFAR shows the faulting address.
Ideas?
The build line:
cmake -GNinja -S .. -B . -DTFM_PLATFORM=arm/musca_b1/sse_200
"-DTFM_TOOLCHAIN_FILE=..\toolchain_IARARM.cmake" -DTEST_NS=ON
-DTEST_S=ON -DCMAKE_BUILD_TYPE=Debug -DBL2=ON
-DTFM_ISOLATION_LEVEL=3 -DTEST_NS_SLIH_IRQ=ON
-DTEST_NS_QCBOR=OFF
Notice that this build runs into a linker bug in the IAR
toolchain and I have a non-released fixed version I'm using,
in case anyone is attempting to replicate the issue.
Cheers,
Thomas
Den 2021-12-17 kl. 11:30, skrev
TrustedFirmware Code Review (Code Review):
Attention is currently required from: Kevin Peng, Ken Liu,
David Hu, Chris Brand, Anton Komlev, Thomas Törnblom.
"Build Successful
http://ci.trustedfirmware.org/job/tf-m-static/3226/display/redirect : SUCCESS"
Patch set 16:Verified
+1
View Change
To view, visit
change 12620. To unsubscribe, or for help writing mail
filters, visit
settings.
Gerrit-Project: TF-M/trusted-firmware-m
Gerrit-Change-Id:
I0084aef576d90af45513c7eaae58ba6c6dbbddab
Gerrit-Change-Number: 12620
Gerrit-Reviewer: TrustedFirmware Code
Review
Gerrit-Comment-Date: Fri, 17 Dec 2021
10:30:20 +0000
Gerrit-MessageType: comment