search

linaro-swg / linux

Linux kernel source tree
Other
41 stars 79 forks source link

Secure Data Path test setup (SWG-208/209) #35

Closed etienne-lms closed 7 years ago

etienne-lms commented 7 years ago

Adding a ION heap type dedicated to "secure data path buffer pools": buffers unmaped in linux world.

Add ION "umaped heap" creation support in ION dummy driver: possibly get "unmaped" heaps from DT.

This is currently a patch in the ION staging directory. ION implementation does not allow registering heap types from external (non ION) drivers.

This patchset was tested on k4.6 and k4.9 (ION integration changes)

SDP test setup on vexpress-qemu: see also https://github.com/OP-TEE/build/pull/112 and https://github.com/linaro-swg/bios_qemu_tz_arm/pull/10 and https://github.com/OP-TEE/optee_test/pull/148.

etienne-lms commented 7 years ago

As many changes for renaming "unmaped"=>"unmapped", i will squash the commit and propose a new version...

etienne-lms commented 7 years ago

rebased to latest 'optee' branch.

jbech-linaro commented 7 years ago

Just some minor cleanup needed (spellings etc), when that is done: Reviewed-by: Joakim Bech <joakim.bech@linaro.org>

etienne-lms commented 7 years ago

comments addressed, tag applied.

ghost commented 7 years ago

Hi there, may I know that :

  1. Have this pull been merged into main branch ?
  2. Is there any demo dts or config file defined SDP memory start and size ?
jbech-linaro commented 7 years ago

Have this pull been merged into main branch ?

No, as long as the PR is open it hasn't been merged, having that said, the intention is to merge it to get it included in the OP-TEE 2.4.0 (April 7th).

Is there any demo dts or config file defined SDP memory start and size ?

I guess this should be it, @etienne-lms can confirm: https://github.com/OP-TEE/build/pull/112/files

etienne-lms commented 7 years ago

@ken1979, @jbech-linaro I confirm. Current PR provides SDP memory pool allocation on REE side. build PR#112 enables this SDP support, and sets a test configuration for playing SDP of the QEMU config (for qemu_virt and qemu_armv8). This because is hard codes in the kernel config the SDP memory pool location. In optee_os, a default configuration makes the same area reserved to the SDP memory pool: df05d4e.

jforissier commented 7 years ago

@etienne-lms can you please rebase? (the OP-TEE driver v15 and v16 deltas have been merged since).

etienne-lms commented 7 years ago

oups...

etienne-lms commented 7 years ago

OP-TEE/build#112 also rebased.