Documentation: add an overview of MOBJs

Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by Volodymyr Babchuk <vlad.babchuk@gmail.com>
Signed-off-by: Jens Wiklander <jens.wikland

Documentation: add an overview of MOBJs

Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by Volodymyr Babchuk <vlad.babchuk@gmail.com>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

core: add paged secure shared memory

Adds feature to allocate paged secure memory that can be transferred
between TAs as needed.

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

core: add paged secure shared memory

Adds feature to allocate paged secure memory that can be transferred
between TAs as needed.

Acked-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by Volodymyr Babchuk <vlad.babchuk@gmail.com>
Tested-by: Jens Wiklander <jens.wiklander@linaro.org> (QEMU, Hikey)
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

core: use mobj for user mappings

Replaces the old mm and mm_stack elements with mobj_code and mobj_stack
in user context.

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

core: use mobj for user mappings

Replaces the old mm and mm_stack elements with mobj_code and mobj_stack
in user context.

Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by Volodymyr Babchuk <vlad.babchuk@gmail.com>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

core: introduce MOBJ abstraction

Introduces MOBJ for memory objects, no changes in features except
breaking unsafe-nw-param/mem-multipurpose.

Reviewed-by: Etienne Carriere <etienne.carriere@linaro.

core: introduce MOBJ abstraction

Introduces MOBJ for memory objects, no changes in features except
breaking unsafe-nw-param/mem-multipurpose.

Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by Volodymyr Babchuk <vlad.babchuk@gmail.com>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

Fix build error when $(O) starts with ./

When a target starts with './', $@ does not. Therefore we must not use
$@ to access a variable if its name is created from $(out-dir) which
might start with

Fix build error when $(O) starts with ./

When a target starts with './', $@ does not. Therefore we must not use
$@ to access a variable if its name is created from $(out-dir) which
might start with './'. We use a macro to make sure that
$(conf-mk-file-export) is evaluated immediately rather than when the
rule is executed. It is needed because ta/ta.mk may be included twice
with different values for $(conf-mk-file-export) (32-bit and 64-bit
dev kits).

Fixes the following issue:

$ make O=./build
<snip>
GEN build/export-ta_arm32/mk/conf.mk
/bin/bash: -c: line 0: syntax error near unexpected token `)'
/bin/bash: -c: line 0: `() >> build/export-ta_arm32/mk/conf.mk'
ta/ta.mk:120: recipe for target 'build/export-ta_arm32/mk/conf.mk' failed
make: *** [build/export-ta_arm32/mk/conf.mk] Error 1

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

show more ...

cruft: remove unreferenced HEAP_SIZE

There are no consumers of this and it just confuses the issue of how to
set the heap size.

Signed-off-by: Andy Green <andy@warmcat.com>
Acked-by: Jerome Forissi

cruft: remove unreferenced HEAP_SIZE

There are no consumers of this and it just confuses the issue of how to
set the heap size.

Signed-off-by: Andy Green <andy@warmcat.com>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
[Wrap commit description. Rebase on top of master.]
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>

show more ...

libutee: bugfix srcData == NULL cases

Fixes faulty treatment of srcData == NULL in TEE_CipherUpdate(),
TEE_AEUpdate(), TEE_AEEncryptFinal(), TEE_AEDecryptFinal().

Reviewed-by: Jerome Forissier <jer

libutee: bugfix srcData == NULL cases

Fixes faulty treatment of srcData == NULL in TEE_CipherUpdate(),
TEE_AEUpdate(), TEE_AEEncryptFinal(), TEE_AEDecryptFinal().

Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Guanchao Liang <liang.guanchao@linaro.org>
Tested-by: Jens Wiklander <jens.wiklander@linaro.org> (QEMU)
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
[Rebase on top of master]
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>

show more ...

core: arm: imx: add PSCI on

Add PSCI on for plat-imx.

Note:
The CFG_BOOT_SECONDARY_REQUEST includes spin_table which is not
required by PSCI.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
Reviewed-

core: arm: imx: add PSCI on

Add PSCI on for plat-imx.

Note:
The CFG_BOOT_SECONDARY_REQUEST includes spin_table which is not
required by PSCI.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
[s/psci/PSCI/ in commit log. Rebase on top of master.]
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>

show more ...

core: arm: add PSCI support

Introduce basic PSCI support.
When CFG_PSCI_ARM32 is defined, PSCI functions will be enabled.
Add __weak property for the PSCI functions, developers could
have their own

core: arm: add PSCI support

Introduce basic PSCI support.
When CFG_PSCI_ARM32 is defined, PSCI functions will be enabled.
Add __weak property for the PSCI functions, developers could
have their own implementation.

PSCI is handled in monitor mode, and when finished, directly returns
to nsec world.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
[s/psci/PSCI/ in commit log. Rebase on top of master.]
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>

show more ...

Update CHANGELOG.md for 2.3.0

Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Tested-by: Peng Fan <peng.fan@nxp.com> (imx

Update CHANGELOG.md for 2.3.0

Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Tested-by: Peng Fan <peng.fan@nxp.com> (imx-mx6ulevk)
Tested-by: Sören Brinkmann <soren.brinkmann@xilinx.com> (zynqmp)
Tested-by: Joakim Bech <joakim.bech@linaro.org> (MTK8173)
Tested-by: Jens Wiklander <jens.wiklander@linaro.org> (Juno)
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey)
Tested-by: Igor Opaniuk <igor.opaniuk@linaro.org> (RPi3)
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (D02)
Tested-by: Jens Wiklander <jens.wiklander@linaro.org> (FVP)
Tested-by: Volodymyr Babchuk <vlad.babchuk@gmail.com> (RCAR H3)
Tested-by: Sumit Garg <sumit.garg@nxp.com> (LS1021A)
Tested-by: Joakim Bech <joakim.bech@linaro.org> (dra7)
Tested-by: Etienne Carriere <etienne.carriere@linaro.org> (b2120 with GP)
Tested-by: Etienne Carriere <etienne.carriere@linaro.org> (b2260 with GP)
Tested-by: Etienne Carriere <etienne.carriere@linaro.org> (QEMU with GP)
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (vexpress-qemu_armv8a)

show more ...

mk: step up OP-TEE revision to 2.3.0

Signed-off-by: Joakim Bech <joakim.bech@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>

mk/cleandirs.mk: don't fail if $(O) is not set

When mk/cleandirs.mk is included by the makefile of a TA, it can happen
that $(O) is empty. Handle this case properly so that "make clean" will
run wit

mk/cleandirs.mk: don't fail if $(O) is not set

When mk/cleandirs.mk is included by the makefile of a TA, it can happen
that $(O) is empty. Handle this case properly so that "make clean" will
run without error.

Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Igor Opaniuk <igor.opaniuk@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

Make sure "make clean" removes everything "make" has created under $(O)

"make clean" would leave behind some files and many directories. Fix
this by correctly tracking the files and directories crea

Make sure "make clean" removes everything "make" has created under $(O)

"make clean" would leave behind some files and many directories. Fix
this by correctly tracking the files and directories created under $(O)
during the build process:
- Fix incorrect file names in $(cleanfiles) and add a few missing
ones.
- Introduce a makefile macro: $(cleandirs-for-rmdir), defined in a new
file: mk/cleandirs.mk. It returns the list of directories that should
be removed, given a list of files.
The clean target removes the files, then all the directories in depth-
first order. $(O) is also removed, if found to be empty.

Note that a more straightforward approach was discussed in [1]: use
"rm -rf $(O)/some_dir" and get rid of the whole file and directory
tracking via $(cleanfiles) and $(cleandirs). Although it was agreed it
would be safe, doing so would necessarily break the backward
compatibility for build scripts relying on "make O=<some path>", due to
the additional level ($(O)/some_dir).

Finally, mk/cleandirs.mk is exported to the TA dev kit and the clean
rule for the TAs is updated.

[1] https://github.com/OP-TEE/optee_os/pull/1270

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>
Acked-by: Andy Green <andy@warmcat.com>

show more ...

core: plat-vexpress: qemu_armv8a: correct DRAM config

The previous fix for DRAM configuration wasn't correct for QEMU virt
ARMv8A. This patch fixes that.

Acked-by: Jerome Forissier <jerome.forissie

core: plat-vexpress: qemu_armv8a: correct DRAM config

The previous fix for DRAM configuration wasn't correct for QEMU virt
ARMv8A. This patch fixes that.

Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jens Wiklander <jens.wiklander@linaro.org> (QEMU ARMv8-A)
Fixes: 0f8333b888f1 ("plat-vexpress/qemu: correct DRAM layout")
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>

show more ...

docs: Remove images not up-to-date

Signed-off-by: Joakim Bech <joakim.bech@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Etienne Carriere <etienne.carriere@linaro.or

docs: Remove images not up-to-date

Signed-off-by: Joakim Bech <joakim.bech@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>

show more ...

docs: Cleaning up optee_design.md

- Removed information that isn't up-to-date (better to remove it than
having the wrong information).
- Fixed various links etc.
- Created some tables to make thin

docs: Cleaning up optee_design.md

- Removed information that isn't up-to-date (better to remove it than
having the wrong information).
- Fixed various links etc.
- Created some tables to make things more readable.
- Formatted according to 80 character width.
- Added links in the table of contents.

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

show more ...

docs: Remove rpi3.md

Signed-off-by: Joakim Bech <joakim.bech@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>

docs: Remove debug.md

Signed-off-by: Joakim Bech <joakim.bech@linaro.org>
Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Etienne Carriere <etienne.carriere@linaro.org>

docs: Cleanup README.md

Since we have decided to move non-architectural docmentation out from
optee_os.git we've removed everything that now has been included in
https://github.com/OP-TEE/build

Sig

docs: Cleanup README.md

Since we have decided to move non-architectural docmentation out from
optee_os.git we've removed everything that now has been included in
https://github.com/OP-TEE/build

Signed-off-by: Joakim Bech <joakim.bech@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>

show more ...

docs: Add ftp-upload to prerequisites

Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>

core: sql_fs_write(): update meta.length only if length has changed

If the data size is not increased as a result of the write operation,
meta.length must not be changed.

Fixes: https://github.com/

core: sql_fs_write(): update meta.length only if length has changed

If the data size is not increased as a result of the write operation,
meta.length must not be changed.

Fixes: https://github.com/OP-TEE/optee_os/issues/1268
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey)
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Igor Opaniuk <igor.opaniuk@linaro.org>

show more ...

core: fix cpu_spin_trylock() against debug lock counting

To avoid confusion in cpu_spin_trylock() it now returns a boolean
value: true on success, false on failure.

Signed-off-by: Etienne Carriere

core: fix cpu_spin_trylock() against debug lock counting

To avoid confusion in cpu_spin_trylock() it now returns a boolean
value: true on success, false on failure.

Signed-off-by: Etienne Carriere <etienne.carriere@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
[Rebase on top of master]
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>

show more ...

core: fix AArch64 __cpu_spin_trylock() return value

Before this change __cpu_spin_trylock() return value was always
the lock address, not the trylock status.

Signed-off-by: Jens Wiklander <jens.wik

core: fix AArch64 __cpu_spin_trylock() return value

Before this change __cpu_spin_trylock() return value was always
the lock address, not the trylock status.

Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Etienne Carriere <etienne.carriere@linaro.org>
[Rebase on top of master]
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>

show more ...

Cleanup documentation due to new TSK

Signed-off-by: Kai Renken <code@koffeinsucht.de>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
[Rebase on top of master]
Signed-off-by: Jerome Forissier <jer

Cleanup documentation due to new TSK

Signed-off-by: Kai Renken <code@koffeinsucht.de>
Reviewed-by: Joakim Bech <joakim.bech@linaro.org>
[Rebase on top of master]
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>

show more ...

Update readme and travis for mx6ulevk

Update readme and travis for mx6ulevk

Signed-off-by: Peng Fan <peng.fan@nxp.com>
Reviewed-by: Igor Opaniuk <igor.opaniuk@linaro.org>
Reviewed-by: Jens Wiklande

Update readme and travis for mx6ulevk

Update readme and travis for mx6ulevk

Signed-off-by: Peng Fan <peng.fan@nxp.com>
Reviewed-by: Igor Opaniuk <igor.opaniuk@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
[Rebase on top of master]
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>

show more ...