6 years ago4003: add aes-ctr corner case 3.0.0 3.0.0-rc1 ti2018.00 ti2018.00-rc1 ti2018.00-rc2 ti2018.00-rc3 ti2018.00-rc4 ti2018.00-rc5 ti2018.01 ti2018.01-rc1 ti2018.01-rc2 ti2018.01-rc3 ti2018.01-rc4 ti2018.02-rc1 ti2018.02-rc2 ti2018.02-rc3
4003: add aes-ctr corner case
Adds a AES-CTR corner case to expose a problem in LTC with hardware
accelerated AES encryption.
The fix for the problem is provided in:
https://github.com/OP-TEE/optee_os/pull/2086
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey960)
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Adds a AES-CTR corner case to expose a problem in LTC with hardware
accelerated AES encryption.
The fix for the problem is provided in:
https://github.com/OP-TEE/optee_os/pull/2086
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey960)
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Reduce loop count in regression_1012
Reduce the loop count in xtest 1012 from 1000 to 3 in order to minimize
debug traces. There is no point in using a higher value in this
functional test.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reduce the loop count in xtest 1012 from 1000 to 3 in order to minimize
debug traces. There is no point in using a higher value in this
functional test.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
regression 1008: test with corrupt BSTA
Updates case 1008 to corrupt bootstrap TAs instead.
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Victor Chong <victor.chong@linaro.org> (hikey aosp)
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Updates case 1008 to corrupt bootstrap TAs instead.
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Victor Chong <victor.chong@linaro.org> (hikey aosp)
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
regression 1008: remove load_fake_ta() test
Removes the load_fake_ta() test, it's not applicable with bootstrap TAs.
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Removes the load_fake_ta() test, it's not applicable with bootstrap TAs.
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
regression: remove 9xxx test series
Removes the 9xxx test series as there's new internal tests in OP-TEE
testing this instead. Also when TAs are stored in secure storage all the
file these tests depends on will changed in an even more unpredictable
way.
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Removes the 9xxx test series as there's new internal tests in OP-TEE
testing this instead. Also when TAs are stored in secure storage all the
file these tests depends on will changed in an even more unpredictable
way.
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest: add --install-tas argument
Adds --install-tas argument which will install all bootstrap TAs
(/lib/optee_armtz/*.bsta) in the OP-TEE TA database.
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Adds --install-tas argument which will install all bootstrap TAs
(/lib/optee_armtz/*.bsta) in the OP-TEE TA database.
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
regression: add case 1003 for mutex
Adds regression case 1003 for read/write mutex testing.
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey960)
Tested-by: Jens Wiklander <jens.wiklander@linaro.org> (QEMU)
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Adds regression case 1003 for read/write mutex testing.
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey960)
Tested-by: Jens Wiklander <jens.wiklander@linaro.org> (QEMU)
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
regression: 4005: add NIST aes-gcm vectors
Adds NIST AES-GCM test vectors to regression case 4005 with
CFG_GCM_NIST_VECTORS=y. Only the first test in each group is
used if CFG_GCM_NIST_VECTORS_LIMITED=y.
With CFG_GCM_NIST_VECTORS=y the size of the xtest binary grows with more
than 10 MiB, unless CFG_GCM_NIST_VECTORS_LIMITED=y is given, then only
by ~1 MiB.
Without CFG_GCM_NIST_VECTORS=y, CFG_GCM_NIST_VECTORS_LIMITED has no
effect.
Android not supported.
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey960)
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Adds NIST AES-GCM test vectors to regression case 4005 with
CFG_GCM_NIST_VECTORS=y. Only the first test in each group is
used if CFG_GCM_NIST_VECTORS_LIMITED=y.
With CFG_GCM_NIST_VECTORS=y the size of the xtest binary grows with more
than 10 MiB, unless CFG_GCM_NIST_VECTORS_LIMITED=y is given, then only
by ~1 MiB.
Without CFG_GCM_NIST_VECTORS=y, CFG_GCM_NIST_VECTORS_LIMITED has no
effect.
Android not supported.
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey960)
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
regression: 4005: improve test coverage
Improves test coverage:
* in vect3 by feeding one full block and one almost full block
* in vect4 by feeding the first two blocks as complete blocks
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Improves test coverage:
* in vect3 by feeding one full block and one almost full block
* in vect4 by feeding the first two blocks as complete blocks
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest/Makefile: remove extra '@'
Due to an extra '@', 'make V=1' won't print the link command for xtest.
Fix that.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Due to an extra '@', 'make V=1' won't print the link command for xtest.
Fix that.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Android.mk: change to use OPTEE_BIN instead of BUILD_OPTEE_OS
To workaround the problem reported as following
when build OP-TEE with AOSP master:
external/optee_test/Android.mk: error: xtest: LOCAL_ADDITIONAL_DEPENDENCIES must only contain paths (not module names)
change to use $(OPTEE_BIN) defined in optee_os/mk/aosp_optee.mk
which is path like:
out/target/product/hikey/optee/arm-plat-hikey/core/tee.bin
as dependency for xtest instead of using the target of BUILD_OPTEE_OS
Reviewed-by: Victor Chong <victor.chong@linaro.org>
Tested by: Victor Chong <victor.chong@linaro.org> (hikey aosp)
Tested-by: Yongqin Liu <yongqin.liu@linaro.org> (hikey aosp master)
Signed-off-by: Yongqin Liu <yongqin.liu@linaro.org>
To workaround the problem reported as following
when build OP-TEE with AOSP master:
external/optee_test/Android.mk: error: xtest: LOCAL_ADDITIONAL_DEPENDENCIES must only contain paths (not module names)
change to use $(OPTEE_BIN) defined in optee_os/mk/aosp_optee.mk
which is path like:
out/target/product/hikey/optee/arm-plat-hikey/core/tee.bin
as dependency for xtest instead of using the target of BUILD_OPTEE_OS
Reviewed-by: Victor Chong <victor.chong@linaro.org>
Tested by: Victor Chong <victor.chong@linaro.org> (hikey aosp)
Tested-by: Yongqin Liu <yongqin.liu@linaro.org> (hikey aosp master)
Signed-off-by: Yongqin Liu <yongqin.liu@linaro.org>
xtest storage: add case 6019 for storage independence
The TA's storage should be independent of each other. What ever
TA #2 do should not affect TA #1's stroage.
Signed-off-by: Jianhui Li <airbak.li@hisilicon.com>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
The TA's storage should be independent of each other. What ever
TA #2 do should not affect TA #1's stroage.
Signed-off-by: Jianhui Li <airbak.li@hisilicon.com>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
xtest storage: add a test case of creation of already existing object
Regression xtest 6012 already tests creation and recreation of a
persistent object. This change adds a test in 6012 to re-create the
existing object without setting the overwrite attribute. Creation
should fail but the object should not be altered and later open
should succeed.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Regression xtest 6012 already tests creation and recreation of a
persistent object. This change adds a test in 6012 to re-create the
existing object without setting the overwrite attribute. Creation
should fail but the object should not be altered and later open
should succeed.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
xtest: deprecated bass_return_code
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest: regression 1xxx: at least warn if test pseudo TA not found
Following tests depends on pseudo TAs in the TEE core but xtest is
not expected to report an error if the pTAs are not found.
This change adds some verbosity to help one to understand to test
was skipped and restricts allowed error status to item-not-found.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Following tests depends on pseudo TAs in the TEE core but xtest is
not expected to report an error if the pTAs are not found.
This change adds some verbosity to help one to understand to test
was skipped and restricts allowed error status to item-not-found.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest 1014: minor beautification
Keep a 80char/line alignment.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Keep a 80char/line alignment.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Adds AES-GCM performance test in aes-perf
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
6 years agoxtest:sdp: fix a xtest crash if TEE is built without test pTA 2.6.0 ti2017.05 ti2017.05-rc4 ti2017.06 ti2017.06-rc1 ti2017.06-rc2 ti2017.06-rc3 ti2017.06-rc4
xtest:sdp: fix a xtest crash if TEE is built without test pTA
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
xtest:sdp: support old ION API if new API fails
Implement the old ION API using ion0_ or ION0_ as prefix.
Routine allocate_ion_buffer() attempts to allocate using the new
API. If the 1st request (query heap types) fails, try the old API.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey 4.9/4.12)
Tested-by: Etienne Carriere <etienne.carriere@linaro.org> (Qemus 4.9/4.12)
Implement the old ION API using ion0_ or ION0_ as prefix.
Routine allocate_ion_buffer() attempts to allocate using the new
API. If the 1st request (query heap types) fails, try the old API.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey 4.9/4.12)
Tested-by: Etienne Carriere <etienne.carriere@linaro.org> (Qemus 4.9/4.12)
xtest/sdp: sync with ION udpates from linux kernel 4.12
Userland should query ION for the ID of the target heap.
Ioctls ION_IOC_FREE and ION_IOC_SHARE are still defined in the uapi
but no more supported by the kernel driver. Ioctl ION_IOC_ALLOC
returns returns a generic dmabuf file descriptor that references
the target buffer.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey)
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Userland should query ION for the ID of the target heap.
Ioctls ION_IOC_FREE and ION_IOC_SHARE are still defined in the uapi
but no more supported by the kernel driver. Ioctl ION_IOC_ALLOC
returns returns a generic dmabuf file descriptor that references
the target buffer.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey)
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
regression: add case 1002
Adds regression case 1002 to test passing a memory buffer to a Pseudo
TA.
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Adds regression case 1002 to test passing a memory buffer to a Pseudo
TA.
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest: add -DTA_DIR to meet different requirement
TA_DIR default value is "/lib/optee_armtz", it should be
able to be changed for different requirement.
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jianhui Li <airbak.li@hisilicon.com>
TA_DIR default value is "/lib/optee_armtz", it should be
able to be changed for different requirement.
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jianhui Li <airbak.li@hisilicon.com>
aes_perf: add missing #include <trace.h>
Since ta/aes_perf/ta_entry.c uses EMSG() it must include the proper
header file.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Since ta/aes_perf/ta_entry.c uses EMSG() it must include the proper
header file.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
crypt: Support 64-bit TA address space.
The code is passing pointers around by casting them to 32-bit
integers; convert them to/from offsets from ta_head instead.
Signed-off-by: Alyssa Milburn <amilburn@zall.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
The code is passing pointers around by casting them to 32-bit
integers; convert them to/from offsets from ta_head instead.
Signed-off-by: Alyssa Milburn <amilburn@zall.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
remove deprecated ta/prebuilt/faked_armv7_uta_signed_header.bin
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
6 years agoxtest: fix buffer setup in AES benchmark tests 2.5.0 2.5.0-rc1 2.5.0-rc2 ti2017.03 ti2017.03-rc1 ti2017.03-rc2 ti2017.03-rc3 ti2017.03-rc4 ti2017.04 ti2017.04-rc1 ti2017.04-rc2 ti2017.04-rc3 ti2017.04-rc4
xtest: fix buffer setup in AES benchmark tests
Insure input/output buffers are specified to a valid configuration
when aes_perf is called from xtest benchmark test entry.
This change fixes the xtest benchmark tests 2011 and 2012:
xtest -t benchmark 2011
xtest -t benchmark 2012
Fixes: b85a5f96c393 ("xtest: add SDP performance tests in aes-perf")
Reported-by: Igor Opaniuk <igor.opaniuk@linaro.org>
Signed-off-by: Etienne Carriere <etienne.carriere@st.com>
Reviewed-by: Igor Opaniuk <igor.opaniuk@linaro.org>
Tested-by: Igor Opaniuk <igor.opaniuk@linaro.org> (qemu)
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey)
Insure input/output buffers are specified to a valid configuration
when aes_perf is called from xtest benchmark test entry.
This change fixes the xtest benchmark tests 2011 and 2012:
xtest -t benchmark 2011
xtest -t benchmark 2012
Fixes: b85a5f96c393 ("xtest: add SDP performance tests in aes-perf")
Reported-by: Igor Opaniuk <igor.opaniuk@linaro.org>
Signed-off-by: Etienne Carriere <etienne.carriere@st.com>
Reviewed-by: Igor Opaniuk <igor.opaniuk@linaro.org>
Tested-by: Igor Opaniuk <igor.opaniuk@linaro.org> (qemu)
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey)
Add regression_1016: TA to TA buffer manipulations
This test case is derived from gp_8040 which was found to be failing on
HiKey with a preliminary version of the 64-bit pager enabled
(CFG_ARM64_core=y CFG_WITH_PAGER=y CFG_PAGED_USER_TA=y). This new
regression_1016 test fails in the same circumstances, but also when
paging is enabled in a 32-bit build of the TEE core prior to commit
commit ecb06119e67d ("core: pager: invalidate tlb when clearing
entry").
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
This test case is derived from gp_8040 which was found to be failing on
HiKey with a preliminary version of the 64-bit pager enabled
(CFG_ARM64_core=y CFG_WITH_PAGER=y CFG_PAGED_USER_TA=y). This new
regression_1016 test fails in the same circumstances, but also when
paging is enabled in a 32-bit build of the TEE core prior to commit
commit ecb06119e67d ("core: pager: invalidate tlb when clearing
entry").
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
ta/os_test: remove unused code
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Make sure AES CTR does not buffer data
Similar to commit c13fafac91ae ("Make sure AES GCM does not buffer
data"), update regression_4003_NO_XTS to check that TEE_CipherUpdate()
does not not buffer data when algorithm is TEE_ALG_AES_CTR.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Similar to commit c13fafac91ae ("Make sure AES GCM does not buffer
data"), update regression_4003_NO_XTS to check that TEE_CipherUpdate()
does not not buffer data when algorithm is TEE_ALG_AES_CTR.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Make sure AES GCM does not buffer data
The documentation [1] for TEE_AEUpdate() states that "Unless one or
more calls of this function have supplied sufficient input data, no
output is generated". This can be understood as: "As soon as sufficient
input data have been supplied, some output is generated".
AES GCM effectively works as a stream cipher, so that any amount of
data passed to the algorithm should produce as much output data
immediately. In other words, calling TEE_AEUpdate() with N bytes of
input should generate N bytes of output for any N >= 1.
This commit updates regression_4005 to check the above assertion.
[1] GlobalPlatform TEE Internal Core API Specification v1.1
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
The documentation [1] for TEE_AEUpdate() states that "Unless one or
more calls of this function have supplied sufficient input data, no
output is generated". This can be understood as: "As soon as sufficient
input data have been supplied, some output is generated".
AES GCM effectively works as a stream cipher, so that any amount of
data passed to the algorithm should produce as much output data
immediately. In other words, calling TEE_AEUpdate() with N bytes of
input should generate N bytes of output for any N >= 1.
This commit updates regression_4005 to check the above assertion.
[1] GlobalPlatform TEE Internal Core API Specification v1.1
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
regression 6018: reduce the number of blocks written
Reduces the number of blocks written when testing REE filesystem to
avoid occasional out of memory errors.
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Reduces the number of blocks written when testing REE filesystem to
avoid occasional out of memory errors.
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest: fix 32bit/64bit build directive when GP suite is enabled
OP-TEE core configuration directive for AArch32 is CFG_ARM32_core not
CFG_ARM32.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
OP-TEE core configuration directive for AArch32 is CFG_ARM32_core not
CFG_ARM32.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
regression_5000.c: use size_t instead of uint32_t
Shared memory sizes should be size_t.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Shared memory sizes should be size_t.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
regression_5006: use 32KB buffer
Since OP-TEE client commit 9faeb1b6b023 ("tee_client_api: Correct max
shared memory size define"), TEEC_CONFIG_SHAREDMEM_MAX_SIZE is defined
as ULONG_MAX to signify that there is no fixed limit on the size of
shared memory buffers. This breaks xtest 5006, which assumes the
constant is some "reasonable" size, so that allocation and registration
of such a buffer should succeed.
Let's drop the broken tests. Instead, test allocation and registration
of 32KB, since that was tested in the past and it is good to know our
implementation still supports that (arbitrary) size.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey, QEMU)
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Since OP-TEE client commit 9faeb1b6b023 ("tee_client_api: Correct max
shared memory size define"), TEEC_CONFIG_SHAREDMEM_MAX_SIZE is defined
as ULONG_MAX to signify that there is no fixed limit on the size of
shared memory buffers. This breaks xtest 5006, which assumes the
constant is some "reasonable" size, so that allocation and registration
of such a buffer should succeed.
Let's drop the broken tests. Instead, test allocation and registration
of 32KB, since that was tested in the past and it is good to know our
implementation still supports that (arbitrary) size.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey, QEMU)
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
--sdp-basic: fix misleading messages
Clearly print "Test passed" or "Test failed" for each test, and do not
print "failed" when it is expected.
Fixes: https://github.com/OP-TEE/optee_test/issues/207
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Clearly print "Test passed" or "Test failed" for each test, and do not
print "failed" when it is expected.
Fixes: https://github.com/OP-TEE/optee_test/issues/207
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
regression 6018: reduce the number of blocks written
Reduces the number of blocks written when testing RPMB filesystem to
avoid occasional out of memory errors.
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Reduces the number of blocks written when testing RPMB filesystem to
avoid occasional out of memory errors.
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
SDP basic test TA: get SDP configuration from TA devkit
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
xtest/perfs: fix option for random initialization of input buffers
Before this change, AES and SHA performance tests always filled
input buffers with random values, wherever option "-r" was set or
not.
This change allows the run SHA performance test with zero-initialized
input buffers or random-initialized input buffers.
This change allows the run AES performance test with zero-initialized
input buffers, random-initialized input buffers or with uninitialized
input buffers (which is quite handy when testing with input buffers
into SDP secure memory).
A new option is added in "xtest --aes-perf": argument "--not-inited"
specifies that input buffer does not need to be initialized.
Since the initialization of crypto test buffers can use several
configuration, this change clarifies then, renaming the macro
CRYPTO_NOT_RANDOM into CRYPTO_USE_ZEROS.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Before this change, AES and SHA performance tests always filled
input buffers with random values, wherever option "-r" was set or
not.
This change allows the run SHA performance test with zero-initialized
input buffers or random-initialized input buffers.
This change allows the run AES performance test with zero-initialized
input buffers, random-initialized input buffers or with uninitialized
input buffers (which is quite handy when testing with input buffers
into SDP secure memory).
A new option is added in "xtest --aes-perf": argument "--not-inited"
specifies that input buffer does not need to be initialized.
Since the initialization of crypto test buffers can use several
configuration, this change clarifies then, renaming the macro
CRYPTO_NOT_RANDOM into CRYPTO_USE_ZEROS.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
xtest: add SDP performance tests in aes-perf
xtest "AES performance" support ("xtest --aes-perf") is used to
run AES ciphering performance tests over non-secure SHM buffers.
Argument "--sdp" enables AES performance tests over secure data path
(SDP) memory buffers.
When argument "--sdp" is used, arguments "-I<X>" and "-O<X>" allow to
define target memref parameters used for ciphering references:
- Decryption input and/or output buffer can be allocated into
the non secure shared memory or into a secure memory pool.
- When using secure buffers, test can either register secure test
buffer once, or secure buffer can be registered to TEE before
each TEE invoke, and release once consumed.
SDP memory buffers are allocated through ION. The default memory pool
used for SDP buffers allocation is the ION "unmapped" heap. Argument
"--ion-heap" allows to select a target ION heap ID, in decimal format.
The AES performance test TA is modified to support handling SDP memory
references through a dedicated "PROCESS_SDP" invocation command.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Tested-by: Etienne Carriere <etienne.carriere@linaro.org> (qemu, b2260)
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
xtest "AES performance" support ("xtest --aes-perf") is used to
run AES ciphering performance tests over non-secure SHM buffers.
Argument "--sdp" enables AES performance tests over secure data path
(SDP) memory buffers.
When argument "--sdp" is used, arguments "-I<X>" and "-O<X>" allow to
define target memref parameters used for ciphering references:
- Decryption input and/or output buffer can be allocated into
the non secure shared memory or into a secure memory pool.
- When using secure buffers, test can either register secure test
buffer once, or secure buffer can be registered to TEE before
each TEE invoke, and release once consumed.
SDP memory buffers are allocated through ION. The default memory pool
used for SDP buffers allocation is the ION "unmapped" heap. Argument
"--ion-heap" allows to select a target ION heap ID, in decimal format.
The AES performance test TA is modified to support handling SDP memory
references through a dedicated "PROCESS_SDP" invocation command.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Tested-by: Etienne Carriere <etienne.carriere@linaro.org> (qemu, b2260)
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
xtest: fix SDP basic to nicely export allocate_ion_buffer()
Fix verbosity issue in allocate_ion_buffer(). The function relies
on verbose() for its traces but does not provide a 'verbosity'
variable to hold the verbosity level. This change fixes this,
making 'verbosity' a local variable rather than a global one.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Fix verbosity issue in allocate_ion_buffer(). The function relies
on verbose() for its traces but does not provide a 'verbosity'
variable to hold the verbosity level. This change fixes this,
making 'verbosity' a local variable rather than a global one.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
aes-perf: AES buffer size must be a multiple of 16 bytes
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
aes-perf, sha-perf: add long options
Add long options format for some AES/SHA performance test tool
arguments:
common: -h/--help, -w/--warmup, -r/--random
aes-perf: -i/--in-place
sha-perf: -u/--unalign
Some minor cleanup.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Add long options format for some AES/SHA performance test tool
arguments:
common: -h/--help, -w/--warmup, -r/--random
aes-perf: -i/--in-place
sha-perf: -u/--unalign
Some minor cleanup.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
aes-perf, sha-perf: trace TEE error origin when applicable
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Remove SQL-FS
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
6 years agoregression: add case 9524 ti2017.01 ti2017.01-rc1 ti2017.01-rc2 ti2017.01-rc3 ti2017.01-rc4 ti2017.02 ti2017.02-rc1 ti2017.02-rc2 ti2017.02-rc3 ti2017.02-rc4 ti2017.02-rc5 ti2017.02-rc6
regression: add case 9524
Adds case 9524 testing that if a file is detected as removed from REE FS
it's reported as TEE_ERROR_CORRUPT_OBJECT since the storage obviously has
been tampered with.
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Adds case 9524 testing that if a file is detected as removed from REE FS
it's reported as TEE_ERROR_CORRUPT_OBJECT since the storage obviously has
been tampered with.
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
regression: 6015: remove special case for REE FS
The special case is s not applicable for REE FS any longer.
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
The special case is s not applicable for REE FS any longer.
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Adjust the 9xxx cases for FS updates
Updates for optee_os commit
"core: REE FS: use the new dirfile interface"
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Updates for optee_os commit
"core: REE FS: use the new dirfile interface"
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Android.mk: Rename C source files to its test suite
Signed-off-by: Victor Chong <victor.chong@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Signed-off-by: Victor Chong <victor.chong@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
adbg_run.c: Define TAILQ_CONCAT() for aosp
TAILQ_CONCAT() is currently unavailable in bionic so define it if
undefined.
Signed-off-by: Victor Chong <victor.chong@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
TAILQ_CONCAT() is currently unavailable in bionic so define it if
undefined.
Signed-off-by: Victor Chong <victor.chong@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Adjust the 9xxx cases for FS updates
Updates for optee_os commit
"core: FS: let FOP->create() initialize the new file"
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Updates for optee_os commit
"core: FS: let FOP->create() initialize the new file"
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Renumber regression suite tests 10xxxx and 20xxxx
Renumber regression suite tests 10xxx to 8xxx and 20xxx to 9xxx
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Renumber regression suite tests 10xxx to 8xxx and 20xxx to 9xxx
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Move GlobalPlatform tests to their own test suite
Introduce a new test suite, selected by "-t gp", and move all the
GlobalPlatform tests there. In order to maintain some backward
compatibility, xtest will run both the "regression" and "gp" suites by
default. To implement this behavior, accept a "+" as a separator for
test suite names, and set the default to "regression+gp". Concatenate
test suites before running the tests to obtain a single, consolidated
report at the end of execution.
Rename C source files according to the test suite they belong to:
implement "regression" tests in regression_*.c, the "gp" tests in
gp_*.c (generated), and the "benchmark" tests in benchmark_*.c.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Tested-by: Etienne Carriere <etienne.carriere@linaro.org> (qemu_virt)
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Introduce a new test suite, selected by "-t gp", and move all the
GlobalPlatform tests there. In order to maintain some backward
compatibility, xtest will run both the "regression" and "gp" suites by
default. To implement this behavior, accept a "+" as a separator for
test suite names, and set the default to "regression+gp". Concatenate
test suites before running the tests to obtain a single, consolidated
report at the end of execution.
Rename C source files according to the test suite they belong to:
implement "regression" tests in regression_*.c, the "gp" tests in
gp_*.c (generated), and the "benchmark" tests in benchmark_*.c.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Tested-by: Etienne Carriere <etienne.carriere@linaro.org> (qemu_virt)
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Add case 1015 testing hash tree
Adds case 1015 to test corner cases in hash tree using a pseudo TA.
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Adds case 1015 to test corner cases in hash tree using a pseudo TA.
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Adjust the 20xxx cases for FS hash-tree updates
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest: fix bug
Fixes: 69f914daa4dc ("xtest: fail straight if failing to open tee context")
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Fixes: 69f914daa4dc ("xtest: fail straight if failing to open tee context")
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest: fail straight if failing to open tee context
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
GP: do not expect to find non-standard macros in tee_internal_api.h
With [1], SLogTrace(), SLogError(), SLogWarning() and S_VAR_NOT_USED()
are not defined in tee_internal_api.h anymore. Define them locally.
[1] https://github.com/OP-TEE/optee_os/pull/1449
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
With [1], SLogTrace(), SLogError(), SLogWarning() and S_VAR_NOT_USED()
are not defined in tee_internal_api.h anymore. Define them locally.
[1] https://github.com/OP-TEE/optee_os/pull/1449
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest: fix (another) type issue in sdp_basic.c
Signed-off-by: Etienne Carriere <etienne.carriere@st.com>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Signed-off-by: Etienne Carriere <etienne.carriere@st.com>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest: fix type issue in sdp_basic.c traces
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
xtest: add a SDP test in the non-regression suite
xtest regression test #1014 is dedicated to secure data path.
It is enable only upon CFG_SECURE_DATA_PATH=y.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
xtest regression test #1014 is dedicated to secure data path.
It is enable only upon CFG_SECURE_DATA_PATH=y.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
xtest: minor cleanups in aes/sha performance tests
Fix usage info.
Fix timing traces (prefer plain ASCII traces 'us' than UTF-8 'µs').
Remove undefined VERSION info.
Remove few trailing spaces and empty lines.
Add few spaces for source code layout consistency.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Fix usage info.
Fix timing traces (prefer plain ASCII traces 'us' than UTF-8 'µs').
Remove undefined VERSION info.
Remove few trailing spaces and empty lines.
Add few spaces for source code layout consistency.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
ta/Makefile: include host_include/conf.mk
The compile-time options of OP-TEE OS (CFG_*) are currently not
available from the main TA makefile. Fix this, since we already
have a need (the sdp_basic TA should automatically be selected
when OP-TEE supports the Secure Data Path).
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
The compile-time options of OP-TEE OS (CFG_*) are currently not
available from the main TA makefile. Fix this, since we already
have a need (the sdp_basic TA should automatically be selected
when OP-TEE supports the Secure Data Path).
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
xtest_20000: Fix uninitialized variable warning
When compiled with -Wuninitialized, GCC identifies a false positive
uninitialized variable 'real_offset'. Although we know the default
path will not be taken, GCC does not, for good measure lets just pass
back an error to fix the warning.
Signed-off-by: Andrew F. Davis <afd@ti.com>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
[jf: rebase on top of master]
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
When compiled with -Wuninitialized, GCC identifies a false positive
uninitialized variable 'real_offset'. Although we know the default
path will not be taken, GCC does not, for good measure lets just pass
back an error to fix the warning.
Signed-off-by: Andrew F. Davis <afd@ti.com>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
[jf: rebase on top of master]
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
xtest: 'invoke tests' pTA API from exported include files
Pseudo TA "invoke_tests" now exports its API in the devkit.
Rely on it rather than locally duplicating the API.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Pseudo TA "invoke_tests" now exports its API in the devkit.
Rely on it rather than locally duplicating the API.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
xtest/sdp: test NS client invocation to pTA with SDP memref params
SDP pseudo TAs hold services reserved to the client. Check that
invocation test pTA with this.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
SDP pseudo TAs hold services reserved to the client. Check that
invocation test pTA with this.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
xtest: SDP basic tests TA to pseudo TA with SDP memref
This change relies on the OP-TEE core pseudo TA used for pseudo TA
invocation test. Its API is available from pta_invoke_tests.h.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
This change relies on the OP-TEE core pseudo TA used for pseudo TA
invocation test. Its API is available from pta_invoke_tests.h.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
xtest: SDP basic tests TA to TA with SDP memref
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
xtest: add SDP basic tests
Test setup: non-secure application invokes a TA ("SDP basic" TA) to
inject data into a secure buffer, transform data from/to a secure
buffer and dump data from a secure buffer. Secure buffers are allocated
through ION. Default ION heap tested is the "unmapped" heap: custom+1.
The "SDP basic TA" checks parameter memory attributes and operates
cache synchronization if CFG_CACHE_API is enable.
The non secure application also attempt to map the secure buffer, to
verify its content. This operation may fails. Data read can also be
corrupted as non-secure may fill its cache with data while secure data
are expected fully in physical memory.
Run basic secure buffer access from Trusted Application:
xtest --sdp-basic
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Zoltan Kuscsik <zoltan.kuscsik@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Test setup: non-secure application invokes a TA ("SDP basic" TA) to
inject data into a secure buffer, transform data from/to a secure
buffer and dump data from a secure buffer. Secure buffers are allocated
through ION. Default ION heap tested is the "unmapped" heap: custom+1.
The "SDP basic TA" checks parameter memory attributes and operates
cache synchronization if CFG_CACHE_API is enable.
The non secure application also attempt to map the secure buffer, to
verify its content. This operation may fails. Data read can also be
corrupted as non-secure may fill its cache with data while secure data
are expected fully in physical memory.
Run basic secure buffer access from Trusted Application:
xtest --sdp-basic
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Zoltan Kuscsik <zoltan.kuscsik@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Add test case for allocate + reset persistent object enumerator
xtest 6010 is updated to test TEE_AllocatePersistentObjectEnumerator()
immediately followed by TEE_ResetPersistentObjectEnumerator().
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest 6010 is updated to test TEE_AllocatePersistentObjectEnumerator()
immediately followed by TEE_ResetPersistentObjectEnumerator().
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
case 6018: reduce the number of blocks written
Reduces the number of blocks written to avoid out of memory errors on some
platforms.
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Reduces the number of blocks written to avoid out of memory errors on some
platforms.
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest: os_test TA does not need TA_FLAG_UNSAFE_NW_PARAMS
TA_FLAG_UNSAFE_NW_PARAMS has been renamed TA_FLAG_SECURE_DATA_PATH
in optee_os.
Remove SDP support from os_test TA has it is not used to test SDP.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
TA_FLAG_UNSAFE_NW_PARAMS has been renamed TA_FLAG_SECURE_DATA_PATH
in optee_os.
Remove SDP support from os_test TA has it is not used to test SDP.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
xtest: prevent clean failure when toggling GP tests enable/disable
If one cleans the tests with the GP test suite enabled before the
GP test suite patches were applied, the ta/GP_xxx/ subdirectories
are not yet created and 'clean' fails.
This patch prevents from attempting to clean test TA directories
that do not (yet) exist.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey)
Tested-by: Etienne Carriere <etienne.carriere@linaro.org> (b2260, qemu)
If one cleans the tests with the GP test suite enabled before the
GP test suite patches were applied, the ta/GP_xxx/ subdirectories
are not yet created and 'clean' fails.
This patch prevents from attempting to clean test TA directories
that do not (yet) exist.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey)
Tested-by: Etienne Carriere <etienne.carriere@linaro.org> (b2260, qemu)
make: fix "make clean" against directories generated at build
This change removes the directory created from xtest build and
xtest test TAs build.
Prevent trailing "/." in path string, i.e "$O"=="some/relative/path/.".
These fail are rejected by the rmdir command.
Allow O=. or O=./ by preventing execution of rmdir.
From TA root Makefile and as $(O) can be a relative path or absolute
path, change to makefile directory to 'rmdir' target from make command
execution path.
Let ta/Makefile handle the "ta/" sub-directory info in build/clean.
Let host/xtest/Makefile handle the "xtest/" sub-directory info in
build/clean.
Build output subdirectory should be nicely empty and ready to be
remove if exists. Root output directory is removed only if empty
as it might contain other stuff than our build artifacts.
As 'clean' rules can be ran from partially built environment, only
remove existing directories.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey)
Tested-by: Etienne Carriere <etienne.carriere@linaro.org> (b2260, qemu)
This change removes the directory created from xtest build and
xtest test TAs build.
Prevent trailing "/." in path string, i.e "$O"=="some/relative/path/.".
These fail are rejected by the rmdir command.
Allow O=. or O=./ by preventing execution of rmdir.
From TA root Makefile and as $(O) can be a relative path or absolute
path, change to makefile directory to 'rmdir' target from make command
execution path.
Let ta/Makefile handle the "ta/" sub-directory info in build/clean.
Let host/xtest/Makefile handle the "xtest/" sub-directory info in
build/clean.
Build output subdirectory should be nicely empty and ready to be
remove if exists. Root output directory is removed only if empty
as it might contain other stuff than our build artifacts.
As 'clean' rules can be ran from partially built environment, only
remove existing directories.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey)
Tested-by: Etienne Carriere <etienne.carriere@linaro.org> (b2260, qemu)
aes-perf, sha-perf: display more statistics
Show more statistics to help assess the test repeatability and compare
test runs:
- Print the coefficient of variation (cv = stddev/mean)
- When the -v option is given: also show the 2-sigma interval
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Show more statistics to help assess the test repeatability and compare
test runs:
- Print the coefficient of variation (cv = stddev/mean)
- When the -v option is given: also show the 2-sigma interval
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
xtest_1000.c: define TA_DIR in C file when __ANDROID__ is set
TA_DIR is defined in xtest_1000.c when not building for Android
whereas it is set in Android.mk when doing an Android build.
Move the Android definition to the C file for consistency.
Signed-off-by: Victor Chong <victor.chong@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
[jf: reword commit text]
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
TA_DIR is defined in xtest_1000.c when not building for Android
whereas it is set in Android.mk when doing an Android build.
Move the Android definition to the C file for consistency.
Signed-off-by: Victor Chong <victor.chong@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
[jf: reword commit text]
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Android.mk: Add missing include for sockets TA
Signed-off-by: Victor Chong <victor.chong@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Signed-off-by: Victor Chong <victor.chong@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
xtest: fix dependency to initstate_r() and friends
initstate_r() and friends are glibc extensions which are not available
in all libc. This patch removes this dependency and implements its own
portable implementation.
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
initstate_r() and friends are glibc extensions which are not available
in all libc. This patch removes this dependency and implements its own
portable implementation.
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Add test case 6018
Adds test case 6018 creating a very large secure storage file.
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Adds test case 6018 creating a very large secure storage file.
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Adjust the 20xxx cases for FS htree
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
docs: remove modprobe instruction
Removes the now unneeded modprobe command from instruction.
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Tested-by: Etienne Carriere <etienne.carriere@linaro.org> (b2260 pager=y/n)
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Removes the now unneeded modprobe command from instruction.
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Tested-by: Etienne Carriere <etienne.carriere@linaro.org> (b2260 pager=y/n)
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
docs: Add ifconfig lo
Adds instructions to ifconfig lo prior to running xtest regression suite.
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Adds instructions to ifconfig lo prior to running xtest regression suite.
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Adds test cases 200x for TCP/UDP iSocket interface
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Add convenient pthread wrappers
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest 10001: run lenghty PBKDF2 test only if level >= 15
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Fix issue with pbkdf2_4_dkm not defined
Signed-off-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Signed-off-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
tb_modulus(): fix test name
Signed-off-by: Andy Green <andy@warmcat.com>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
[Change commit message]
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Andy Green <andy@warmcat.com>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
[Change commit message]
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
xtest: 1011: extend to test TA-to-TA and TA private memory references
Extends test case 1011 to test TA-to-TA session both via normal shared
memory reference parameters and TA private memory reference parameters.
Reviewed-by: Guanchao Liang <liang.guanchao@linaro.org>
Tested-by: Guanchao Liang <liang.guanchao@linaro.org> (FVP)
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Tested-by: Jens Wiklander <jens.wiklander@linaro.org> (QEMU)
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Extends test case 1011 to test TA-to-TA session both via normal shared
memory reference parameters and TA private memory reference parameters.
Reviewed-by: Guanchao Liang <liang.guanchao@linaro.org>
Tested-by: Guanchao Liang <liang.guanchao@linaro.org> (FVP)
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Tested-by: Jens Wiklander <jens.wiklander@linaro.org> (QEMU)
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Add AES CTS test case
Add test case for issue https://github.com/OP-TEE/optee_os/issues/1203.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Add test case for issue https://github.com/OP-TEE/optee_os/issues/1203.
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest: distributed registration of cases
Uses distributed registration of test cases, replacing the central
registration in xtest_main.c and xtest_test.h
Updated against GP TestSuite 1.1.0.4.
Remove generation of ADBG_CASE_DECLARE() and ADBG_SUITE_ENTRY()
code during 'patch' make rule. Remove related resources that are no
more useful. adbg_entry_declare.h and adbg_case_declare.h are no more
generated.
Change rule 'patch-filter' to comment definition of filtered tests
from their ADBG_CASE_DEFINE() implementation.
Tested-by: Etienne Carriere <etienne.carriere@linaro.org> (GP suite: QEMU, stm/b2260)
Tested-by: Jens Wiklander <jens.wiklander@linaro.org> (QEMU)
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Uses distributed registration of test cases, replacing the central
registration in xtest_main.c and xtest_test.h
Updated against GP TestSuite 1.1.0.4.
Remove generation of ADBG_CASE_DECLARE() and ADBG_SUITE_ENTRY()
code during 'patch' make rule. Remove related resources that are no
more useful. adbg_entry_declare.h and adbg_case_declare.h are no more
generated.
Change rule 'patch-filter' to comment definition of filtered tests
from their ADBG_CASE_DEFINE() implementation.
Tested-by: Etienne Carriere <etienne.carriere@linaro.org> (GP suite: QEMU, stm/b2260)
Tested-by: Jens Wiklander <jens.wiklander@linaro.org> (QEMU)
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest: adbg: remove unused WhySkipped_p
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest: remove unused regression cases
Removes unused regression cases 7011, 7012, 7014, and 7015.
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Removes unused regression cases 7011, 7012, 7014, and 7015.
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Fix Android build errors
Signed-off-by: Show Liu <show.liu@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Fixes: 47d9a0b2f438 ("xtest: remove unused ADBG_SuiteData_t")
[Rebase on top of master, add Fixes: tag]
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Signed-off-by: Show Liu <show.liu@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Fixes: 47d9a0b2f438 ("xtest: remove unused ADBG_SuiteData_t")
[Rebase on top of master, add Fixes: tag]
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
xtest: remove deprecated USER_SPACE
Legacy USER_SPACE enables thread support in xtest. Withdraw as always
set.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Legacy USER_SPACE enables thread support in xtest. Withdraw as always
set.
Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Update Makefile rule for filtering out GP tests
Forgot to adjust the sed rule in parent commit. As a result, unwanted GP tests
would run.
Fixes: b6bea047b1d4 ("Update GP tests after simplification of test declarations")
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Forgot to adjust the sed rule in parent commit. As a result, unwanted GP tests
would run.
Fixes: b6bea047b1d4 ("Update GP tests after simplification of test declarations")
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
xtest 6017: Test info.dataSize in Persistent objects
Create a persistent object and write some data into it,
get the info.dataSize and then close it, and reopen it
to get the new info.dataSize of this reopen's object,
these two dataSizes should be equal.
Signed-off-by: Guanchao Liang <liang.guanchao@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Create a persistent object and write some data into it,
get the info.dataSize and then close it, and reopen it
to get the new info.dataSize of this reopen's object,
these two dataSizes should be equal.
Signed-off-by: Guanchao Liang <liang.guanchao@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Update GP tests after simplification of test declarations
Fixes: 25a57fe24e5b ("xtest: clean test case declarations")
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Fixes: 25a57fe24e5b ("xtest: clean test case declarations")
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Fix UUID strings in file names of GP TAs
Fixes: 6203b8769bef ("Format UUID strings correctly")
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Fixes: 6203b8769bef ("Format UUID strings correctly")
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest: remove adbg_util.c
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest: clean adbg_int.h
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
xtest: remove security_utils_mem.[ch]
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>