core: correct time conversion in delay support

The previous code may overflow in 32bit architectures. This change
fixes the issue by forcing 64bit computation during frequency
to counter conversion.

core: correct time conversion in delay support

The previous code may overflow in 32bit architectures. This change
fixes the issue by forcing 64bit computation during frequency
to counter conversion.

Signed-off-by: Etienne Carriere <etienne.carriere@st.com>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>

show more ...

thread: fix stack space to not be part of tee.bin

Commit 935364080364 ("thread: move stacks to separate sections") makes
stack space to be part of tee.bin which leads to approx. 130K increase
in siz

thread: fix stack space to not be part of tee.bin

Commit 935364080364 ("thread: move stacks to separate sections") makes
stack space to be part of tee.bin which leads to approx. 130K increase
in size of tee.bin for platform with 2 threads.
So this patch fixes the stack space to be in NOLOAD section only.

Fixes: 935364080364 ("thread: move stacks to separate sections")
Signed-off-by: Sumit Garg <sumit.garg@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

core: arm32: fix saving vfp state

Prior to this patch TF-A in AArch32 didn't save the normal world VFP
data registers and thus always restored then as zeroes and causing
problems in normal world.

W

core: arm32: fix saving vfp state

Prior to this patch TF-A in AArch32 didn't save the normal world VFP
data registers and thus always restored then as zeroes and causing
problems in normal world.

With this patch if running with TF-A in AArch32 save the VFP state using
the same logic as in AArch64. Since TF-A saves and restores CPACR_EL1 we
cannot tell if normal world currently is using VFP or not so we have to
assume that it is and always save the VFP data registers if they are
about to be changed.

Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jens Wiklander <jens.wiklander@linaro.org> (Hikey AArch32)
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

pager: enable BestFit allocation when using the pager

When running xtest 6018 we have got panics because of
TEE_ERROR_OUT_OF_MEMORY errors when trying to allocate memory (using
malloc and calloc). T

pager: enable BestFit allocation when using the pager

When running xtest 6018 we have got panics because of
TEE_ERROR_OUT_OF_MEMORY errors when trying to allocate memory (using
malloc and calloc). The reason for this seems to be a fragmented heap
when running with the pager enabled. By enabling the BestFit algorithm
in bget we have seen a much improved use of the heap with a lot less
fragmentation. We have been running xtest on QEMU v8 and HiKey 6220 and
the performance difference seems to be negligible.

Fixes: https://github.com/OP-TEE/optee_os/issues/2580

Signed-off-by: Joakim Bech <joakim.bech@linaro.org>
Tested-by: Joakim Bech <joakim.bech@linaro.org> (HiKey 6220, QEMU v8)
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

Update revision to 3.3

The commit [1] that changed the revision macros from 3.2 to 3.3 was not
pushed to the master branch when I created the 3.3.0-rc1 tag. Pick this
commit now.

[1] Commit b5aecc0

Update revision to 3.3

The commit [1] that changed the revision macros from 3.2 to 3.3 was not
pushed to the master branch when I created the 3.3.0-rc1 tag. Pick this
commit now.

[1] Commit b5aecc05ca32 ("Update revision for release tag 3.3.0-rc1")
(tag 3.3.0-rc1)

Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>

show more ...

scripts/symbolize.py: add help text

Document how the binutils tools (addr2line etc.) are found.

Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wikla

scripts/symbolize.py: add help text

Document how the binutils tools (addr2line etc.) are found.

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>

show more ...

core: user_ta: set freed EXIDX pointer to NULL

Clear reference of user TA exception index table once it
is freed, otherwise it is freed a second time. This
issue occurs when set_exidx() fails for so

core: user_ta: set freed EXIDX pointer to NULL

Clear reference of user TA exception index table once it
is freed, otherwise it is freed a second time. This
issue occurs when set_exidx() fails for some reasons as
an out of memory issue. This change prevents core from
panicking when it occurs.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>

show more ...

core: default guard reg_shm mobjs

Add a special guard for registered shared memory MOBJs to make sure that
it's not possible to release MOBJs which aren't created by an explicit
registration from no

core: default guard reg_shm mobjs

Add a special guard for registered shared memory MOBJs to make sure that
it's not possible to release MOBJs which aren't created by an explicit
registration from normal world.

Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

core: mobj_reg_shm_free() using mobj_reg_shm_put()

Let reg_shm MOBJs be completely reference counted meaning that
mobj_free() on a reg_shm MOBJ only decrease the reference counter and
only free it i

core: mobj_reg_shm_free() using mobj_reg_shm_put()

Let reg_shm MOBJs be completely reference counted meaning that
mobj_free() on a reg_shm MOBJ only decrease the reference counter and
only free it if it reaches 0.

Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

core: remove mobj_reg_shm_{,un}map()

Removes mobj_reg_shm_map() and mobj_reg_shm_unmap(), they are replaced
by mobj_reg_shm_inc_map() and mobj_reg_shm_dec_map().

Reviewed-by: Etienne Carriere <etie

core: remove mobj_reg_shm_{,un}map()

Removes mobj_reg_shm_map() and mobj_reg_shm_unmap(), they are replaced
by mobj_reg_shm_inc_map() and mobj_reg_shm_dec_map().

Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

core: pta: use mobj_reg_shm_{inc,dec}_map()

Use mobj_reg_shm_inc_map() and mobj_reg_shm_dec_map() instead of
mobj_reg_shm_map() and mobj_reg_shm_unmap().

Acked-by: Etienne Carriere <etienne.carrier

core: pta: use mobj_reg_shm_{inc,dec}_map()

Use mobj_reg_shm_inc_map() and mobj_reg_shm_dec_map() instead of
mobj_reg_shm_map() and mobj_reg_shm_unmap().

Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

core: add mobj_reg_shm_{inc,dec}_map()

mobj_reg_shm_inc_map() used when a reg_shm mobj needs to be mapped and
mobj_reg_shm_dec_map() is called when the mapping isn't needed any
longer.

Reviewed-by:

core: add mobj_reg_shm_{inc,dec}_map()

mobj_reg_shm_inc_map() used when a reg_shm mobj needs to be mapped and
mobj_reg_shm_dec_map() is called when the mapping isn't needed any
longer.

Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

core: remove reg_shm_{put,free}_by_cookie()

Removes the two functions mobj_reg_shm_put_by_cookie() and
mobj_reg_shm_free_by_cookie(). mobj_reg_shm_put() and mobj_free() should
be used instead.

Revi

core: remove reg_shm_{put,free}_by_cookie()

Removes the two functions mobj_reg_shm_put_by_cookie() and
mobj_reg_shm_free_by_cookie(). mobj_reg_shm_put() and mobj_free() should
be used instead.

Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

core: entry_std: free/put by mobj in call cleanup

Use the mobj pointer instead of a shm_ref when cleaning up parameters
in cleanup_shm_refs() at the end of a call.

Reviewed-by: Etienne Carriere <e

core: entry_std: free/put by mobj in call cleanup

Use the mobj pointer instead of a shm_ref when cleaning up parameters
in cleanup_shm_refs() at the end of a call.

Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

core: add mobj_reg_shm_put()

Adds mobj_reg_shm_put() for reference counting without cookie.

Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander

core: add mobj_reg_shm_put()

Adds mobj_reg_shm_put() for reference counting without cookie.

Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

core: correct memory layout trace

Buffers that end at end of the available address range which
may happen on 32bit machine fail have an end address that of
computed as 0. This change uses a 64bit ad

core: correct memory layout trace

Buffers that end at end of the available address range which
may happen on 32bit machine fail have an end address that of
computed as 0. This change uses a 64bit address computation
to prevent the displayed end address being 0.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

core: correct overflows in range overlap functions

Buffers that end at end of the available address range which
may happen on 32bit machine fail have an end address that of
computed as 0. This chang

core: correct overflows in range overlap functions

Buffers that end at end of the available address range which
may happen on 32bit machine fail have an end address that of
computed as 0. This change uses the computation already used
in _core_is_buffer_inside() to ensure functions
_core_is_buffer_outside() and _core_is_buffer_intersect()
return a reliable result.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

ci: travis: print statistics (ccache, df)

Print ccache and disk space info before and after the build to help
monitor and troubleshoot issues.

Signed-off-by: Jerome Forissier <jerome.forissier@lina

ci: travis: print statistics (ccache, df)

Print ccache and disk space info before and after the build to help
monitor and troubleshoot issues.

Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

ci: travis: download 32-bit toolchain only

The 64-bit toolchain is not needed to build and run the regression tests
for QEMU. Download only the 32-bit toolchain to save disk space and time.

Signed-

ci: travis: download 32-bit toolchain only

The 64-bit toolchain is not needed to build and run the regression tests
for QEMU. Download only the 32-bit toolchain to save disk space and time.

Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

ci: travis: delete unused optee_os directory

The optee_os cloned using "repo" is not used, because we link to the
directory checked out by the CI process instead. Delete it to save about
160 MiB of

ci: travis: delete unused optee_os directory

The optee_os cloned using "repo" is not used, because we link to the
directory checked out by the CI process instead. Delete it to save about
160 MiB of disk space on the build server.

Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

ci: travis: delete expect5.45 directory after build

Save some more space on the build server.

Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@

ci: travis: delete expect5.45 directory after build

Save some more space on the build server.

Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

ci: travis: delete toolchain binaries after download

Since build.git has been upgraded to GCC 8.2 [1], we are running out of
disk space when running the Travis CI script. This is because the GCC 8.2

ci: travis: delete toolchain binaries after download

Since build.git has been upgraded to GCC 8.2 [1], we are running out of
disk space when running the Travis CI script. This is because the GCC 8.2
binaries are much larger than 6.2:

218M gcc-arm-8.2-2018.08-x86_64-aarch64-linux-gnu.tar.xz
213M gcc-arm-8.2-2018.08-x86_64-arm-linux-gnueabihf.tar.xz
94M gcc-linaro-6.2.1-2016.11-x86_64_aarch64-linux-gnu.tar.xz
88M gcc-linaro-6.2.1-2016.11-x86_64_arm-linux-gnueabihf.tar.xz

This commit makes some room by deleting the packages once they have been
extracted.

Link: [1] https://github.com/OP-TEE/build/commit/53119b77e9f2
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

symbolize: use CROSS_COMPILE directive if available

Get CROSS_COMPILE prefix directive if defined in the environment.
If CROSS_COMPILE is not defined, default to aarch64-linux-gnu- or
arm-linux-gnue

symbolize: use CROSS_COMPILE directive if available

Get CROSS_COMPILE prefix directive if defined in the environment.
If CROSS_COMPILE is not defined, default to aarch64-linux-gnu- or
arm-linux-gnueabihf- as done prior this change according to the
architecture read from the ELF file.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>

show more ...

abort.c: always save VFP state with thread context

Saving VFP state requires a thread context, if none is available print
abort info and panic().

Reviewed-by: Volodymyr Babchuk <volodymyr_babchuk@e

abort.c: always save VFP state with thread context

Saving VFP state requires a thread context, if none is available print
abort info and panic().

Reviewed-by: Volodymyr Babchuk <volodymyr_babchuk@epam.com>
Fixes: cfa34ec63699 ("abort.c: manipulate with VFP state only if thread is active")
Tested-by: Jens Wiklander <jens.wiklander@linaro.org> (Hikey AArch64 pager)
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

core: trace out size of the pager physical page pool

Add a nice info trace about the size of the pager physical
page pool.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by:

core: trace out size of the pager physical page pool

Add a nice info trace about the size of the pager physical
page pool.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...