Skip to content

Commit 0081207

Browse files
Vge0rgedanieldegrasse
authored andcommitted
samples: tfm: tfm_psa_test: Add temporary isolation workaround
Add a workaround which sets the isolation level to 2 in CMake. The TF-M tests require the large TF-M profile because it supports the full list of crypto algorithms needed, not because of the solation level. For the TF-M tests the isolation level is irrelevant so we set it to 2 here so that we don't exclude the platforms which don't support the isolation level 3. I communicated with a TF-M maintainer and he informed me that this workaround will be included in the TF-M 2.2 release branch later so this is a short lived workaround that can be reverted when the relevant commit from the TF-M branch is cherry-picked. Signed-off-by: Georgios Vasilakis <georgios.vasilakis@nordicsemi.no>
1 parent 9166d6a commit 0081207

File tree

1 file changed

+11
-0
lines changed

1 file changed

+11
-0
lines changed

samples/tfm_integration/tfm_psa_test/CMakeLists.txt

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,17 @@ set_property(TARGET zephyr_property_target
6363
-DTEST_PSA_API=${TEST_PSA_API}
6464
)
6565

66+
# Workaround: The TF-M tests require the large TF-M profile because it supports
67+
# the full list of crypto algorithms needed, not because of the isolation level.
68+
# For the TF-M tests the isolation level is irrelevant so we set it to 2 here so
69+
# that we don't exclude the platforms which don't support the isolation level 3.
70+
# This is a short lived workaround because the TF-M cmake logic will do this workaround
71+
# automatically in the future.
72+
set_property(TARGET zephyr_property_target
73+
APPEND PROPERTY TFM_CMAKE_OPTIONS
74+
-DTFM_ISOLATION_LEVEL=2
75+
)
76+
6677
include(ExternalProject)
6778

6879
ExternalProject_Add(tfm_psa_arch_test_app

0 commit comments

Comments
 (0)