diff mbox series

arm/optee-client: set parameter CFG_TEE_PLUGIN_LOAD_PATH as ${libdir}

Message ID 20240304035527.1281859-1-Meng.Li@windriver.com
State New
Headers show
Series arm/optee-client: set parameter CFG_TEE_PLUGIN_LOAD_PATH as ${libdir} | expand

Commit Message

Meng Li March 4, 2024, 3:55 a.m. UTC
In optee-client package, the CFG_TEE_PLUGIN_LOAD_PATH is set as
/usr/lib/tee-supplicant/plugins/. But when multilib is enabled in yocto,
the ${libdir} is /usr/lib64, the optee-examples and optee-test package
will install the plugins into usr/lib64. This will cause that optee feature
doesn't work when it can't find out the necessary plugin. So, set parameter
CFG_TEE_PLUGIN_LOAD_PATH as ${libdir} to support for multilib and
non-multilib feature.

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

Comments

Sverdlin, Alexander March 4, 2024, 10:21 a.m. UTC | #1
Hello Meng!

On Mon, 2024-03-04 at 11:55 +0800, Meng Li via lists.yoctoproject.org wrote:
> In optee-client package, the CFG_TEE_PLUGIN_LOAD_PATH is set as
> /usr/lib/tee-supplicant/plugins/. But when multilib is enabled in yocto,
> the ${libdir} is /usr/lib64, the optee-examples and optee-test package
> will install the plugins into usr/lib64. This will cause that optee feature
> doesn't work when it can't find out the necessary plugin. So, set parameter
> CFG_TEE_PLUGIN_LOAD_PATH as ${libdir} to support for multilib and
> non-multilib feature.
> 
> Signed-off-by: Meng Li <Meng.Li@windriver.com>

LGTM,
Reviewed-by: Alexander Sverdlin <alexander.sverdlin@siemens.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 3b9943cc..ef03045c 100644
> --- a/meta-arm/recipes-security/optee/optee-client.inc
> +++ b/meta-arm/recipes-security/optee/optee-client.inc
> @@ -20,6 +20,7 @@ S = "${WORKDIR}/git"
>  EXTRA_OECMAKE = " \
>      -DBUILD_SHARED_LIBS=ON \
>      -DCFG_TEE_FS_PARENT_PATH='${localstatedir}/lib/tee' \
> +    -DCFG_TEE_PLUGIN_LOAD_PATH='${libdir}/tee-supplicant/plugins/' \
>  "
>  EXTRA_OECMAKE:append:toolchain-clang = " -DCFG_WERROR=0"

-- 
Alexander Sverdlin
Siemens AG
www.siemens.com
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 3b9943cc..ef03045c 100644
--- a/meta-arm/recipes-security/optee/optee-client.inc
+++ b/meta-arm/recipes-security/optee/optee-client.inc
@@ -20,6 +20,7 @@  S = "${WORKDIR}/git"
 EXTRA_OECMAKE = " \
     -DBUILD_SHARED_LIBS=ON \
     -DCFG_TEE_FS_PARENT_PATH='${localstatedir}/lib/tee' \
+    -DCFG_TEE_PLUGIN_LOAD_PATH='${libdir}/tee-supplicant/plugins/' \
 "
 EXTRA_OECMAKE:append:toolchain-clang = " -DCFG_WERROR=0"