From: kernel test robot <lkp@intel.com>
To: Etienne Carriere <etienne.carriere@foss.st.com>,
linux-kernel@vger.kernel.org
Cc: oe-kbuild-all@lists.linux.dev,
Jens Wiklander <jens.wiklander@linaro.org>,
Sumit Garg <sumit.garg@linaro.org>,
linux-arm-kernel@lists.infradead.org,
op-tee@lists.trustedfirmware.org,
Jerome Forissier <jerome.forissier@linaro.org>,
Etienne Carriere <etienne.carriere@foss.st.com>
Subject: Re: [PATCH v11 1/4] tee: optee: system call property
Date: Thu, 19 Oct 2023 20:49:57 +0800 [thread overview]
Message-ID: <202310192021.fvb6JDOY-lkp@intel.com> (raw)
In-Reply-To: <20231016090428.1369071-2-etienne.carriere@foss.st.com>
Hi Etienne,
kernel test robot noticed the following build warnings:
[auto build test WARNING on soc/for-next]
[also build test WARNING on linus/master v6.6-rc6 next-20231019]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Etienne-Carriere/tee-optee-system-call-property/20231017-135906
base: https://git.kernel.org/pub/scm/linux/kernel/git/soc/soc.git for-next
patch link: https://lore.kernel.org/r/20231016090428.1369071-2-etienne.carriere%40foss.st.com
patch subject: [PATCH v11 1/4] tee: optee: system call property
config: arm64-randconfig-001-20231019 (https://download.01.org/0day-ci/archive/20231019/202310192021.fvb6JDOY-lkp@intel.com/config)
compiler: aarch64-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231019/202310192021.fvb6JDOY-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202310192021.fvb6JDOY-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> drivers/tee/optee/ffa_abi.c:618: warning: Function parameter or member 'system_thread' not described in 'optee_ffa_do_call_with_arg'
--
drivers/tee/optee/smc_abi.c:818: warning: Function parameter or member 'rpc_arg' not described in 'optee_handle_rpc'
>> drivers/tee/optee/smc_abi.c:890: warning: Function parameter or member 'system_thread' not described in 'optee_smc_do_call_with_arg'
vim +618 drivers/tee/optee/ffa_abi.c
4615e5a34b95e0 Jens Wiklander 2021-07-21 602
4615e5a34b95e0 Jens Wiklander 2021-07-21 603 /**
4615e5a34b95e0 Jens Wiklander 2021-07-21 604 * optee_ffa_do_call_with_arg() - Do a FF-A call to enter OP-TEE in secure world
4615e5a34b95e0 Jens Wiklander 2021-07-21 605 * @ctx: calling context
4615e5a34b95e0 Jens Wiklander 2021-07-21 606 * @shm: shared memory holding the message to pass to secure world
5b4018b959149e Jens Wiklander 2022-01-25 607 * @offs: offset of the message in @shm
4615e5a34b95e0 Jens Wiklander 2021-07-21 608 *
4615e5a34b95e0 Jens Wiklander 2021-07-21 609 * Does a FF-A call to OP-TEE in secure world and handles eventual resulting
4615e5a34b95e0 Jens Wiklander 2021-07-21 610 * Remote Procedure Calls (RPC) from OP-TEE.
4615e5a34b95e0 Jens Wiklander 2021-07-21 611 *
4615e5a34b95e0 Jens Wiklander 2021-07-21 612 * Returns return code from FF-A, 0 is OK
4615e5a34b95e0 Jens Wiklander 2021-07-21 613 */
4615e5a34b95e0 Jens Wiklander 2021-07-21 614
4615e5a34b95e0 Jens Wiklander 2021-07-21 615 static int optee_ffa_do_call_with_arg(struct tee_context *ctx,
9f34a9d2a70d79 Etienne Carriere 2023-10-16 616 struct tee_shm *shm, u_int offs,
9f34a9d2a70d79 Etienne Carriere 2023-10-16 617 bool system_thread)
4615e5a34b95e0 Jens Wiklander 2021-07-21 @618 {
4615e5a34b95e0 Jens Wiklander 2021-07-21 619 struct ffa_send_direct_data data = {
4615e5a34b95e0 Jens Wiklander 2021-07-21 620 .data0 = OPTEE_FFA_YIELDING_CALL_WITH_ARG,
4615e5a34b95e0 Jens Wiklander 2021-07-21 621 .data1 = (u32)shm->sec_world_id,
4615e5a34b95e0 Jens Wiklander 2021-07-21 622 .data2 = (u32)(shm->sec_world_id >> 32),
5b4018b959149e Jens Wiklander 2022-01-25 623 .data3 = offs,
4615e5a34b95e0 Jens Wiklander 2021-07-21 624 };
4064c461148ab1 Jens Wiklander 2021-12-28 625 struct optee_msg_arg *arg;
4064c461148ab1 Jens Wiklander 2021-12-28 626 unsigned int rpc_arg_offs;
4064c461148ab1 Jens Wiklander 2021-12-28 627 struct optee_msg_arg *rpc_arg;
4064c461148ab1 Jens Wiklander 2021-12-28 628
a639b2b18a240d Jens Wiklander 2022-01-12 629 /*
a639b2b18a240d Jens Wiklander 2022-01-12 630 * The shared memory object has to start on a page when passed as
a639b2b18a240d Jens Wiklander 2022-01-12 631 * an argument struct. This is also what the shm pool allocator
a639b2b18a240d Jens Wiklander 2022-01-12 632 * returns, but check this before calling secure world to catch
a639b2b18a240d Jens Wiklander 2022-01-12 633 * eventual errors early in case something changes.
a639b2b18a240d Jens Wiklander 2022-01-12 634 */
a639b2b18a240d Jens Wiklander 2022-01-12 635 if (shm->offset)
a639b2b18a240d Jens Wiklander 2022-01-12 636 return -EINVAL;
a639b2b18a240d Jens Wiklander 2022-01-12 637
5b4018b959149e Jens Wiklander 2022-01-25 638 arg = tee_shm_get_va(shm, offs);
4064c461148ab1 Jens Wiklander 2021-12-28 639 if (IS_ERR(arg))
4064c461148ab1 Jens Wiklander 2021-12-28 640 return PTR_ERR(arg);
4064c461148ab1 Jens Wiklander 2021-12-28 641
4064c461148ab1 Jens Wiklander 2021-12-28 642 rpc_arg_offs = OPTEE_MSG_GET_ARG_SIZE(arg->num_params);
5b4018b959149e Jens Wiklander 2022-01-25 643 rpc_arg = tee_shm_get_va(shm, offs + rpc_arg_offs);
4064c461148ab1 Jens Wiklander 2021-12-28 644 if (IS_ERR(rpc_arg))
4064c461148ab1 Jens Wiklander 2021-12-28 645 return PTR_ERR(rpc_arg);
4615e5a34b95e0 Jens Wiklander 2021-07-21 646
9f34a9d2a70d79 Etienne Carriere 2023-10-16 647 return optee_ffa_yielding_call(ctx, &data, rpc_arg, system_thread);
4615e5a34b95e0 Jens Wiklander 2021-07-21 648 }
4615e5a34b95e0 Jens Wiklander 2021-07-21 649
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2023-10-19 12:50 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-10-16 9:04 [PATCH v11 0/4] tee: introduce TEE system sssion Etienne Carriere
2023-10-16 9:04 ` [PATCH v11 1/4] tee: optee: system call property Etienne Carriere
2023-10-19 12:49 ` kernel test robot [this message]
2023-10-16 9:04 ` [PATCH v11 2/4] tee: system session Etienne Carriere
2023-10-16 9:04 ` [PATCH v11 3/4] tee: optee: support tracking system threads Etienne Carriere
2023-10-17 11:37 ` Sumit Garg
2023-10-19 14:22 ` Jens Wiklander
2023-10-16 9:04 ` [PATCH v11 4/4] firmware: arm_scmi: optee: use optee system invocation Etienne Carriere
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=202310192021.fvb6JDOY-lkp@intel.com \
--to=lkp@intel.com \
--cc=etienne.carriere@foss.st.com \
--cc=jens.wiklander@linaro.org \
--cc=jerome.forissier@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=op-tee@lists.trustedfirmware.org \
--cc=sumit.garg@linaro.org \
/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).