xtest fails on IMX8MPLUS

[z23cc]

On the imx8MPLUS-BB development board, install optee, run xtest on it, and the following error occurs below.

Later on, I modified r = ADBG_EXPECT_TEEC_RESULT(c, TEEC_ERROR_SECURITY, res) into r = ADBG_EXPECT_TEEC_RESULT(c, TEEC_ERROR_GENERIC, res), and then found the following problem again.

Very confused, don't know how to solve it


Best Regards!

[krzysztof-buczak-red]

Old thread and you probably don't care about the answer anymore, but maybe other folks would have the same issue and come upon this post.

In my case switching to another software revision seems to solve the issue. When I was on optee-os and x-test 6.6.52-2.2.0 ("4.4.0"), the test failed the same way as you presented. On optee-os and x-test 6.12.49-2.2.0 ("4.8.0") this is no longer reproducing.

Remember to use NXP github repos for optee and x-test if you are building from scratch:

- https://github.com/nxp-imx/imx-optee-test

- https://github.com/nxp-imx/imx-optee-os

[krzysztof-buczak-red]

Actually, this is the commit that fixed my issue (present on 6.12, not present on 6.6):

commit c6c7967f74d4c6267750b3ff42067c004f8cad33
Author: Jens Wiklander <jens.wiklander@linaro.org>
Date:   Fri Dec 13 10:01:33 2024 +0100

    core: pta: secstore: decrease TA buffer
    
    install_ta() uses a buffer allocated from the heap while hashing a TA
    while installing it. The buffer size is 8kB which is a bit large to
    reliably allocate from the heap, so decrease it to 1kB.
    
    Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
    Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
    Reviewed-by: Etienne Carriere <etienne.carriere@foss.st.com>

diff --git a/core/pta/secstor_ta_mgmt.c b/core/pta/secstor_ta_mgmt.c
index 162de43be..b8dc9283c 100644
--- a/core/pta/secstor_ta_mgmt.c
+++ b/core/pta/secstor_ta_mgmt.c
@@ -44,7 +44,7 @@ static TEE_Result install_ta(struct shdr *shdr, const uint8_t *nw,
        struct tee_tadb_ta_write *ta;
        void *hash_ctx = NULL;
        size_t offs;
-       const size_t buf_size = 2 * 4096;
+       const size_t buf_size = 1024;
        void *buf;
        struct tee_tadb_property property;
        struct shdr_bootstrap_ta bs_ta;

[jimmychan]

Which version of BSP are you using?