linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jerome Forissier <jerome@forissier.org>
To: Sumit Garg <sumit.garg@linaro.org>,
	Jens Wiklander <jens.wiklander@linaro.org>
Cc: linux-kernel@vger.kernel.org, op-tee@lists.trustedfirmware.org,
	Herbert Xu <herbert@gondor.apana.org.au>,
	Devaraj Rangasamy <Devaraj.Rangasamy@amd.com>,
	Rijo Thomas <Rijo-john.Thomas@amd.com>,
	David Howells <dhowells@redhat.com>,
	Tyler Hicks <tyhicks@linux.microsoft.com>,
	Etienne Carriere <etienne.carriere@linaro.org>
Subject: Re: [PATCH v2 07/12] optee: use driver internal tee_contex for some rpc
Date: Fri, 21 Jan 2022 14:28:18 +0100	[thread overview]
Message-ID: <ad7311fe-0655-e235-f755-2eb3425c79c1@forissier.org> (raw)
In-Reply-To: <CAFA6WYN+NU6uj=R5y4A6RomWeKc74orvB7ouia8Sxuwxaz_-2w@mail.gmail.com>



On 1/21/22 13:54, Sumit Garg wrote:
> + Jerome, Etienne
> 
> On Fri, 14 Jan 2022 at 20:38, Jens Wiklander <jens.wiklander@linaro.org> wrote:
>>
>> Uses the new driver internal tee_context when allocating driver private
>> shared memory. This decouples the shared memory object from its original
>> tee_context. This is needed when the life time of such a memory
>> allocation outlives the client tee_context.
>>
>> Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
>> ---
>>  drivers/tee/optee/ffa_abi.c | 16 ++++++++--------
>>  drivers/tee/optee/smc_abi.c |  7 ++++---
>>  2 files changed, 12 insertions(+), 11 deletions(-)
>>
> 
> I guess with this patch we should no longer see issues [1] reported earlier.

Correct. I have tested [1] again in QEMU with this whole series applied
and the issue is gone.

-- 
Jerome

> 
> FWIW,
> 
> Reviewed-by: Sumit Garg <sumit.garg@linaro.org>
> 
> [1] https://github.com/OP-TEE/optee_os/issues/1918
> 
> -Sumit
> 
>> diff --git a/drivers/tee/optee/ffa_abi.c b/drivers/tee/optee/ffa_abi.c
>> index 88a028d4fb7b..5ec484b42432 100644
>> --- a/drivers/tee/optee/ffa_abi.c
>> +++ b/drivers/tee/optee/ffa_abi.c
>> @@ -424,6 +424,7 @@ static struct tee_shm_pool *optee_ffa_shm_pool_alloc_pages(void)
>>   */
>>
>>  static void handle_ffa_rpc_func_cmd_shm_alloc(struct tee_context *ctx,
>> +                                             struct optee *optee,
>>                                               struct optee_msg_arg *arg)
>>  {
>>         struct tee_shm *shm;
>> @@ -439,7 +440,7 @@ static void handle_ffa_rpc_func_cmd_shm_alloc(struct tee_context *ctx,
>>                 shm = optee_rpc_cmd_alloc_suppl(ctx, arg->params[0].u.value.b);
>>                 break;
>>         case OPTEE_RPC_SHM_TYPE_KERNEL:
>> -               shm = tee_shm_alloc_priv_kernel_buf(ctx,
>> +               shm = tee_shm_alloc_priv_kernel_buf(optee->ctx,
>>                                                     arg->params[0].u.value.b);
>>                 break;
>>         default:
>> @@ -493,14 +494,13 @@ static void handle_ffa_rpc_func_cmd_shm_free(struct tee_context *ctx,
>>  }
>>
>>  static void handle_ffa_rpc_func_cmd(struct tee_context *ctx,
>> +                                   struct optee *optee,
>>                                     struct optee_msg_arg *arg)
>>  {
>> -       struct optee *optee = tee_get_drvdata(ctx->teedev);
>> -
>>         arg->ret_origin = TEEC_ORIGIN_COMMS;
>>         switch (arg->cmd) {
>>         case OPTEE_RPC_CMD_SHM_ALLOC:
>> -               handle_ffa_rpc_func_cmd_shm_alloc(ctx, arg);
>> +               handle_ffa_rpc_func_cmd_shm_alloc(ctx, optee, arg);
>>                 break;
>>         case OPTEE_RPC_CMD_SHM_FREE:
>>                 handle_ffa_rpc_func_cmd_shm_free(ctx, optee, arg);
>> @@ -510,12 +510,12 @@ static void handle_ffa_rpc_func_cmd(struct tee_context *ctx,
>>         }
>>  }
>>
>> -static void optee_handle_ffa_rpc(struct tee_context *ctx, u32 cmd,
>> -                                struct optee_msg_arg *arg)
>> +static void optee_handle_ffa_rpc(struct tee_context *ctx, struct optee *optee,
>> +                                u32 cmd, struct optee_msg_arg *arg)
>>  {
>>         switch (cmd) {
>>         case OPTEE_FFA_YIELDING_CALL_RETURN_RPC_CMD:
>> -               handle_ffa_rpc_func_cmd(ctx, arg);
>> +               handle_ffa_rpc_func_cmd(ctx, optee, arg);
>>                 break;
>>         case OPTEE_FFA_YIELDING_CALL_RETURN_INTERRUPT:
>>                 /* Interrupt delivered by now */
>> @@ -582,7 +582,7 @@ static int optee_ffa_yielding_call(struct tee_context *ctx,
>>                  * above.
>>                  */
>>                 cond_resched();
>> -               optee_handle_ffa_rpc(ctx, data->data1, rpc_arg);
>> +               optee_handle_ffa_rpc(ctx, optee, data->data1, rpc_arg);
>>                 cmd = OPTEE_FFA_YIELDING_CALL_RESUME;
>>                 data->data0 = cmd;
>>                 data->data1 = 0;
>> diff --git a/drivers/tee/optee/smc_abi.c b/drivers/tee/optee/smc_abi.c
>> index 1dbb13b08381..f2ef76451443 100644
>> --- a/drivers/tee/optee/smc_abi.c
>> +++ b/drivers/tee/optee/smc_abi.c
>> @@ -621,6 +621,7 @@ static void handle_rpc_func_cmd_shm_free(struct tee_context *ctx,
>>  }
>>
>>  static void handle_rpc_func_cmd_shm_alloc(struct tee_context *ctx,
>> +                                         struct optee *optee,
>>                                           struct optee_msg_arg *arg,
>>                                           struct optee_call_ctx *call_ctx)
>>  {
>> @@ -650,7 +651,7 @@ static void handle_rpc_func_cmd_shm_alloc(struct tee_context *ctx,
>>                 shm = optee_rpc_cmd_alloc_suppl(ctx, sz);
>>                 break;
>>         case OPTEE_RPC_SHM_TYPE_KERNEL:
>> -               shm = tee_shm_alloc_priv_kernel_buf(ctx, sz);
>> +               shm = tee_shm_alloc_priv_kernel_buf(optee->ctx, sz);
>>                 break;
>>         default:
>>                 arg->ret = TEEC_ERROR_BAD_PARAMETERS;
>> @@ -746,7 +747,7 @@ static void handle_rpc_func_cmd(struct tee_context *ctx, struct optee *optee,
>>         switch (arg->cmd) {
>>         case OPTEE_RPC_CMD_SHM_ALLOC:
>>                 free_pages_list(call_ctx);
>> -               handle_rpc_func_cmd_shm_alloc(ctx, arg, call_ctx);
>> +               handle_rpc_func_cmd_shm_alloc(ctx, optee, arg, call_ctx);
>>                 break;
>>         case OPTEE_RPC_CMD_SHM_FREE:
>>                 handle_rpc_func_cmd_shm_free(ctx, arg);
>> @@ -775,7 +776,7 @@ static void optee_handle_rpc(struct tee_context *ctx,
>>
>>         switch (OPTEE_SMC_RETURN_GET_RPC_FUNC(param->a0)) {
>>         case OPTEE_SMC_RPC_FUNC_ALLOC:
>> -               shm = tee_shm_alloc_priv_kernel_buf(ctx, param->a1);
>> +               shm = tee_shm_alloc_priv_kernel_buf(optee->ctx, param->a1);
>>                 if (!IS_ERR(shm) && !tee_shm_get_pa(shm, 0, &pa)) {
>>                         reg_pair_from_64(&param->a1, &param->a2, pa);
>>                         reg_pair_from_64(&param->a4, &param->a5,
>> --
>> 2.31.1
>>

  reply	other threads:[~2022-01-21 13:28 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-14 15:08 [PATCH v2 00/12] tee: shared memory updates Jens Wiklander
2022-01-14 15:08 ` [PATCH v2 01/12] hwrng: optee-rng: use tee_shm_alloc_kernel_buf() Jens Wiklander
2022-01-18 12:12   ` Sumit Garg
2022-01-14 15:08 ` [PATCH v2 02/12] tee: remove unused tee_shm_pool_alloc_res_mem() Jens Wiklander
2022-01-18 12:57   ` Sumit Garg
2022-01-14 15:08 ` [PATCH v2 03/12] tee: add tee_shm_alloc_user_buf() Jens Wiklander
2022-01-18 13:11   ` Sumit Garg
2022-01-19  7:36     ` Jens Wiklander
2022-01-14 15:08 ` [PATCH v2 04/12] tee: simplify shm pool handling Jens Wiklander
2022-01-20 10:06   ` Sumit Garg
2022-01-20 12:56     ` Jens Wiklander
2022-01-21 13:06       ` Sumit Garg
2022-01-21 13:47         ` Jens Wiklander
2022-01-14 15:08 ` [PATCH v2 05/12] tee: replace tee_shm_alloc() Jens Wiklander
2022-01-20 10:41   ` Sumit Garg
2022-01-20 13:02     ` Jens Wiklander
2022-01-14 15:08 ` [PATCH v2 06/12] optee: add driver private tee_context Jens Wiklander
2022-01-21 12:48   ` Sumit Garg
2022-01-24 13:58     ` Jens Wiklander
2022-01-14 15:08 ` [PATCH v2 07/12] optee: use driver internal tee_contex for some rpc Jens Wiklander
2022-01-21 12:54   ` Sumit Garg
2022-01-21 13:28     ` Jerome Forissier [this message]
2022-01-21 13:37       ` Sumit Garg
2022-01-21 14:02         ` Jens Wiklander
2022-01-14 15:08 ` [PATCH v2 08/12] optee: add optee_pool_op_free_helper() Jens Wiklander
2022-01-20 10:57   ` Sumit Garg
2022-01-14 15:08 ` [PATCH v2 09/12] tee: add tee_shm_register_{user,kernel}_buf() Jens Wiklander
2022-01-20 11:00   ` Sumit Garg
2022-01-20 13:06     ` Jens Wiklander
2022-01-14 15:08 ` [PATCH v2 10/12] KEYS: trusted: tee: use tee_shm_register_kernel_buf() Jens Wiklander
2022-01-20 11:03   ` Sumit Garg
2022-01-14 15:08 ` [PATCH v2 11/12] tee: replace tee_shm_register() Jens Wiklander
2022-01-20 11:51   ` Sumit Garg
2022-01-20 14:12     ` Jens Wiklander
2022-01-21 12:57       ` Sumit Garg
2022-01-14 15:08 ` [PATCH v2 12/12] tee: refactor TEE_SHM_* flags Jens Wiklander
2022-01-20 13:03   ` Sumit Garg
2022-01-21  7:23     ` Jens Wiklander

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ad7311fe-0655-e235-f755-2eb3425c79c1@forissier.org \
    --to=jerome@forissier.org \
    --cc=Devaraj.Rangasamy@amd.com \
    --cc=Rijo-john.Thomas@amd.com \
    --cc=dhowells@redhat.com \
    --cc=etienne.carriere@linaro.org \
    --cc=herbert@gondor.apana.org.au \
    --cc=jens.wiklander@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=op-tee@lists.trustedfirmware.org \
    --cc=sumit.garg@linaro.org \
    --cc=tyhicks@linux.microsoft.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).