[v2,0/6] arch/arm: optee: fix TODOs and change status to "Tech Preview"

Message

Volodymyr Babchuk Sept. 18, 2019, 6:50 p.m. UTC

Hello,

This is the second version for maturing the OP-TEE mediator.

Changes also can be pulled from [2].

Changes from v1:
 - Added patch that updates SUPPORT.md
 - Instead of removing "experimental" status I changed it to "Tech Preview"
 - Other changes are described in the corresponding patches

Cover letter for v1:

This patch series fixes various unfinished items in the OP-TEE mediator.
Mostly this is about limiting resources that guest can consume. This
includes both memory and time - how many buffers guest can share with
OP-TEE (this uses Xen memory) and when mediator should preempt itself,
to make sure that guest does not stress scheduling.

Apart from this, there were one case, when mediator's actions might lead
to memory leak in a good-behaving guest. To fix this issue I had to
extend mediator logic, so now it can issue RPC requests to guest in the
same way, as OP-TEE does this. This is useful feature, because it
allows to preempt mediator during long operations. So, in the future
it will be possible to remove shared buffer size limitation, because
mediator can preempt self during buffer translation.

This patch series can be pulled from [1].

[1] https://github.com/lorc/xen/tree/optee3_v1
[2] https://github.com/lorc/xen/tree/optee3_v2


Volodymyr Babchuk (6):
  xen/arm: optee: impose limit on shared buffer size
  xen/arm: optee: check for preemption while freeing shared buffers
  xen/arm: optee: limit number of shared buffers
  xen/arm: optee: handle shared buffer translation error
  SUPPORT.md: Describe OP-TEE mediator
  xen/arm: optee: update description in Kconfig

 SUPPORT.md               |   4 +
 xen/arch/arm/tee/Kconfig |  12 +-
 xen/arch/arm/tee/optee.c | 259 ++++++++++++++++++++++++++++++---------
 3 files changed, 213 insertions(+), 62 deletions(-)

Comments

Julien Grall Sept. 23, 2019, 10:24 a.m. UTC | #1

On 18/09/2019 19:50, Volodymyr Babchuk wrote:
> Hello,

Hi,

> This is the second version for maturing the OP-TEE mediator.
> 
> Changes also can be pulled from [2].
> 
> Changes from v1:
>   - Added patch that updates SUPPORT.md
>   - Instead of removing "experimental" status I changed it to "Tech Preview"
>   - Other changes are described in the corresponding patches
> 
> Cover letter for v1:
> 
> This patch series fixes various unfinished items in the OP-TEE mediator.
> Mostly this is about limiting resources that guest can consume. This
> includes both memory and time - how many buffers guest can share with
> OP-TEE (this uses Xen memory) and when mediator should preempt itself,
> to make sure that guest does not stress scheduling.
> 
> Apart from this, there were one case, when mediator's actions might lead
> to memory leak in a good-behaving guest. To fix this issue I had to
> extend mediator logic, so now it can issue RPC requests to guest in the
> same way, as OP-TEE does this. This is useful feature, because it
> allows to preempt mediator during long operations. So, in the future
> it will be possible to remove shared buffer size limitation, because
> mediator can preempt self during buffer translation.
> 
> This patch series can be pulled from [1].
> 
> [1] https://github.com/lorc/xen/tree/optee3_v1
> [2] https://github.com/lorc/xen/tree/optee3_v2
> 
> 
> Volodymyr Babchuk (6):
>    xen/arm: optee: impose limit on shared buffer size
>    xen/arm: optee: check for preemption while freeing shared buffers
>    xen/arm: optee: limit number of shared buffers

I have committed patch 1-3.

Cheers,