Hi all,
May I know if you have any suggestion or comment on these two patches?
l 10767: Build: Control single test without TEST_S/TEST_NS [TF-M repo] https://review.trustedfirmware.org/c/TF-M/trusted-firmware-m/+/10767
l 10768: Build: Control single test without TEST_S/TEST_NS [Test repo] | https://review.trustedfirmware.org/c/TF-M/tf-m-tests/+/10768
I'd like to merge the changes on 2021.8.20 if there is no more critical comment.
Best Regards Jianliang Shen
From: Jianliang Shen Sent: 06 August 2021 13:01 To: tf-m@lists.trustedfirmware.org; Anton Komlev Anton.Komlev@arm.com Cc: nd@arm.com Subject: Test configurations refinements
Hi all.
I have introduced the topic of test configurations refinements in the tech forum on August 5. With this new feature, you can build single test case to decrease the image size and the time to wait irrelevant test cases running when debugging. You can use all the 22 new flags in test repo:
TEST_NS TEST_S TEST_NS_ATTESTATION TEST_NS_AUDIT TEST_NS_CRYPTO TEST_NS_ITS TEST_NS_PS TEST_NS_PLATFORM TEST_NS_FWU TEST_NS_IPC TEST_NS_CORE TEST_NS_QCBOR TEST_NS_T_COSE TEST_NS_MULTI_CORE TEST_NS_SLIH_IRQ TEST_NS_FLIH_IRQ TEST_S_ATTESTATION TEST_S_AUDIT TEST_S_CRYPTO TEST_S_ITS TEST_S_PS TEST_S_PLATFORM TEST_S_FWU TEST_S_IPC You can easily use the command below to start single test like NS attestation test case:
cmake -S . -B cmake_build -DTFM_PLATFORM=arm/mps2/an521 \ -DTFM_TOOLCHAIN_FILE=toolchain_GNUARM.cmake \ -DCMAKE_BUILD_TYPE=Release \ -DTFM_PSA_API=ON \ -DTEST_NS_ATTESTATION=ON
Meanwhile, you may receive the warning messages when your test inputs are not supported on the platform.
Here are the patches about this change, any suggestions or improvements are welcome in code review.
l 10767: Build: Control single test without TEST_S/TEST_NS [TF-M repo] https://review.trustedfirmware.org/c/TF-M/trusted-firmware-m/+/10767
l 10768: Build: Control single test without TEST_S/TEST_NS [Test repo] | https://review.trustedfirmware.org/c/TF-M/tf-m-tests/+/10768
Best Regards Jianliang Shen