diff mbox series

[meta-arm,1/1] optee-client: disable build on qemux86-64

Message ID 20250221013348.2622293-1-haixiao.yan.cn@eng.windriver.com
State New
Headers show
Series [meta-arm,1/1] optee-client: disable build on qemux86-64 | expand

Commit Message

haixiao.yan.cn@eng.windriver.com Feb. 21, 2025, 1:33 a.m. UTC
From: Haixiao Yan <haixiao.yan.cn@windriver.com>

optee-client will be built on qemux86-64, there is no OP-TEE supplicant
device on this platform.

Signed-off-by: Haixiao Yan <haixiao.yan.cn@windriver.com>
---
 meta-arm/recipes-security/optee/optee-client.inc | 1 +
 1 file changed, 1 insertion(+)

Comments

Jon Mason Feb. 21, 2025, 3:06 p.m. UTC | #1
On Thu, Feb 20, 2025 at 8:34 PM Yan, Haixiao (CN) via
lists.yoctoproject.org
<Haixiao.Yan.CN=windriver.com@lists.yoctoproject.org> wrote:
>
> From: Haixiao Yan <haixiao.yan.cn@windriver.com>
>
> optee-client will be built on qemux86-64, there is no OP-TEE supplicant
> device on this platform.

I don't think I understand what you mean.  More verbosity in the
commit message might help in the future.
I was able to add optee-client to qemux86-64 and it compiled without issue.

Either way, this patch breaks corstone1000 and fvp-base (with
fvp-base-ts).  You can see it in
https://gitlab.com/jonmason00/meta-arm/-/jobs/9203534679
https://gitlab.com/jonmason00/meta-arm/-/jobs/9203534694

Thanks,
Jon

> Signed-off-by: Haixiao Yan <haixiao.yan.cn@windriver.com>
> ---
>  meta-arm/recipes-security/optee/optee-client.inc | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/meta-arm/recipes-security/optee/optee-client.inc b/meta-arm/recipes-security/optee/optee-client.inc
> index f387c80574b0..aafb1efcf726 100644
> --- a/meta-arm/recipes-security/optee/optee-client.inc
> +++ b/meta-arm/recipes-security/optee/optee-client.inc
> @@ -6,6 +6,7 @@ LICENSE = "BSD-2-Clause"
>  LIC_FILES_CHKSUM = "file://LICENSE;md5=69663ab153298557a59c67a60a743e5b"
>
>  inherit systemd update-rc.d cmake useradd
> +require optee.inc
>
>  SRC_URI = " \
>      git://github.com/OP-TEE/optee_client.git;branch=master;protocol=https \
> --
> 2.34.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#6376): https://lists.yoctoproject.org/g/meta-arm/message/6376
> Mute This Topic: https://lists.yoctoproject.org/mt/111301214/3616920
> Group Owner: meta-arm+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-arm/unsub [jdmason@kudzu.us]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Yan, Haixiao (CN) Feb. 25, 2025, 12:41 a.m. UTC | #2
My local branch is mickledore. tee-supplicant.service start failed on 
qemux86-64.

https://git.yoctoproject.org/meta-arm/commit/meta-arm/recipes-security/optee/optee-client.inc?id=1bff41eee67d6dafc7c58b0c88923a3c6f7a2a7c 


this commit can resolve my issue.

My question is could we add the COMPATIBLE_MACHINE for optee-client?  So 
for qemux86-64, if there is no OP-TEE supplicant device, optee-client build

can skip.


root@qemux86-64:~#  systemctl status tee-supplicant.service

x tee-supplicant.service - TEE Supplicant

      Loaded: loaded (/lib/systemd/system/tee-supplicant.service; 
enabled; preset: enabled)

      Active: failed (Result: exit-code) since Sun 2025-02-23 22:41:08 
UTC; 32s ago

    Duration: 191ms

     Process: 468 ExecStart=/usr/sbin/tee-supplicant $OPTARGS 
(code=exited, status=1/FAILURE)

    Main PID: 468 (code=exited, status=1/FAILURE)

Feb 23 22:41:08 qemux86-64 systemd[1]: Started TEE Supplicant.

Feb 23 22:41:08 qemux86-64 tee-supplicant[468]: ERR [468] TSUP:main:884: 
failed to find an OP-TEE suppli>

Feb 23 22:41:08 qemux86-64 systemd[1]: tee-supplicant.service: Main 
process exited, code=exited, status=>

Feb 23 22:41:08 qemux86-64 systemd[1]: tee-supplicant.service: Failed 
with result 'exit-code'.

Thanks,
Haixiao


On 2/21/2025 11:06 PM, Jon Mason wrote:
> CAUTION: This email comes from a non Wind River email account!
> Do not click links or open attachments unless you recognize the sender and know the content is safe.
>
> On Thu, Feb 20, 2025 at 8:34 PM Yan, Haixiao (CN) via
> lists.yoctoproject.org
> <Haixiao.Yan.CN=windriver.com@lists.yoctoproject.org> wrote:
>> From: Haixiao Yan <haixiao.yan.cn@windriver.com>
>>
>> optee-client will be built on qemux86-64, there is no OP-TEE supplicant
>> device on this platform.
> I don't think I understand what you mean.  More verbosity in the
> commit message might help in the future.
> I was able to add optee-client to qemux86-64 and it compiled without issue.
>
> Either way, this patch breaks corstone1000 and fvp-base (with
> fvp-base-ts).  You can see it in
> https://gitlab.com/jonmason00/meta-arm/-/jobs/9203534679
> https://gitlab.com/jonmason00/meta-arm/-/jobs/9203534694
>
> Thanks,
> Jon
>
>> Signed-off-by: Haixiao Yan <haixiao.yan.cn@windriver.com>
>> ---
>>   meta-arm/recipes-security/optee/optee-client.inc | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/meta-arm/recipes-security/optee/optee-client.inc b/meta-arm/recipes-security/optee/optee-client.inc
>> index f387c80574b0..aafb1efcf726 100644
>> --- a/meta-arm/recipes-security/optee/optee-client.inc
>> +++ b/meta-arm/recipes-security/optee/optee-client.inc
>> @@ -6,6 +6,7 @@ LICENSE = "BSD-2-Clause"
>>   LIC_FILES_CHKSUM = "file://LICENSE;md5=69663ab153298557a59c67a60a743e5b"
>>
>>   inherit systemd update-rc.d cmake useradd
>> +require optee.inc
>>
>>   SRC_URI = " \
>>       git://github.com/OP-TEE/optee_client.git;branch=master;protocol=https \
>> --
>> 2.34.1
>>
>>
>> -=-=-=-=-=-=-=-=-=-=-=-
>> Links: You receive all messages sent to this group.
>> View/Reply Online (#6376): https://lists.yoctoproject.org/g/meta-arm/message/6376
>> Mute This Topic: https://lists.yoctoproject.org/mt/111301214/3616920
>> Group Owner: meta-arm+owner@lists.yoctoproject.org
>> Unsubscribe: https://lists.yoctoproject.org/g/meta-arm/unsub [jdmason@kudzu.us]
>> -=-=-=-=-=-=-=-=-=-=-=-
>>
Mikko Rapeli Feb. 25, 2025, 7:57 a.m. UTC | #3
Hi,

On Tue, Feb 25, 2025 at 08:41:04AM +0800, Yan, Haixiao (CN) via lists.yoctoproject.org wrote:
> My local branch is mickledore. tee-supplicant.service start failed on
> qemux86-64.
> 
> https://git.yoctoproject.org/meta-arm/commit/meta-arm/recipes-security/optee/optee-client.inc?id=1bff41eee67d6dafc7c58b0c88923a3c6f7a2a7c
> 
> 
> this commit can resolve my issue.
> 
> My question is could we add the COMPATIBLE_MACHINE for optee-client?  So for
> qemux86-64, if there is no OP-TEE supplicant device, optee-client build
> 
> can skip.
> 
> 
> root@qemux86-64:~#  systemctl status tee-supplicant.service
> 
> x tee-supplicant.service - TEE Supplicant
> 
>      Loaded: loaded (/lib/systemd/system/tee-supplicant.service; enabled;
> preset: enabled)
> 
>      Active: failed (Result: exit-code) since Sun 2025-02-23 22:41:08 UTC;
> 32s ago
> 
>    Duration: 191ms
> 
>     Process: 468 ExecStart=/usr/sbin/tee-supplicant $OPTARGS (code=exited,
> status=1/FAILURE)
> 
>    Main PID: 468 (code=exited, status=1/FAILURE)
> 
> Feb 23 22:41:08 qemux86-64 systemd[1]: Started TEE Supplicant.
> 
> Feb 23 22:41:08 qemux86-64 tee-supplicant[468]: ERR [468] TSUP:main:884:
> failed to find an OP-TEE suppli>
> 
> Feb 23 22:41:08 qemux86-64 systemd[1]: tee-supplicant.service: Main process
> exited, code=exited, status=>
> 
> Feb 23 22:41:08 qemux86-64 systemd[1]: tee-supplicant.service: Failed with
> result 'exit-code'.

One way to fix this is to change tee-supplicante.service to be started by udev
when optee kernel drivers get loaded and optee is found from firmware.

Patches were sent but caused regressions and fixing those are on my todo list, sadly
way back there.

https://lists.yoctoproject.org/g/meta-arm/message/6203

sysvinit scripts are a different story.

Cheers,

-Mikko

> Thanks,
> Haixiao
> 
> 
> On 2/21/2025 11:06 PM, Jon Mason wrote:
> > CAUTION: This email comes from a non Wind River email account!
> > Do not click links or open attachments unless you recognize the sender and know the content is safe.
> > 
> > On Thu, Feb 20, 2025 at 8:34 PM Yan, Haixiao (CN) via
> > lists.yoctoproject.org
> > <Haixiao.Yan.CN=windriver.com@lists.yoctoproject.org> wrote:
> > > From: Haixiao Yan <haixiao.yan.cn@windriver.com>
> > > 
> > > optee-client will be built on qemux86-64, there is no OP-TEE supplicant
> > > device on this platform.
> > I don't think I understand what you mean.  More verbosity in the
> > commit message might help in the future.
> > I was able to add optee-client to qemux86-64 and it compiled without issue.
> > 
> > Either way, this patch breaks corstone1000 and fvp-base (with
> > fvp-base-ts).  You can see it in
> > https://gitlab.com/jonmason00/meta-arm/-/jobs/9203534679
> > https://gitlab.com/jonmason00/meta-arm/-/jobs/9203534694
> > 
> > Thanks,
> > Jon
> > 
> > > Signed-off-by: Haixiao Yan <haixiao.yan.cn@windriver.com>
> > > ---
> > >   meta-arm/recipes-security/optee/optee-client.inc | 1 +
> > >   1 file changed, 1 insertion(+)
> > > 
> > > diff --git a/meta-arm/recipes-security/optee/optee-client.inc b/meta-arm/recipes-security/optee/optee-client.inc
> > > index f387c80574b0..aafb1efcf726 100644
> > > --- a/meta-arm/recipes-security/optee/optee-client.inc
> > > +++ b/meta-arm/recipes-security/optee/optee-client.inc
> > > @@ -6,6 +6,7 @@ LICENSE = "BSD-2-Clause"
> > >   LIC_FILES_CHKSUM = "file://LICENSE;md5=69663ab153298557a59c67a60a743e5b"
> > > 
> > >   inherit systemd update-rc.d cmake useradd
> > > +require optee.inc
> > > 
> > >   SRC_URI = " \
> > >       git://github.com/OP-TEE/optee_client.git;branch=master;protocol=https \
> > > --
> > > 2.34.1
> > > 
> > > 
> > > 
> > > 

> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#6383): https://lists.yoctoproject.org/g/meta-arm/message/6383
> Mute This Topic: https://lists.yoctoproject.org/mt/111301214/7159507
> Group Owner: meta-arm+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-arm/unsub [mikko.rapeli@linaro.org]
> -=-=-=-=-=-=-=-=-=-=-=-
>
diff mbox series

Patch

diff --git a/meta-arm/recipes-security/optee/optee-client.inc b/meta-arm/recipes-security/optee/optee-client.inc
index f387c80574b0..aafb1efcf726 100644
--- a/meta-arm/recipes-security/optee/optee-client.inc
+++ b/meta-arm/recipes-security/optee/optee-client.inc
@@ -6,6 +6,7 @@  LICENSE = "BSD-2-Clause"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=69663ab153298557a59c67a60a743e5b"
 
 inherit systemd update-rc.d cmake useradd
+require optee.inc
 
 SRC_URI = " \
     git://github.com/OP-TEE/optee_client.git;branch=master;protocol=https \