Re: ti-sgx-ddk-um_1.14.3699939: check for wayland in DISTRO_FEATURES

[Denys Dmytriyenko <denys@ti.com>]

Thanks, Eric,

As I was suspecting, wayland libs are required to be present for SGX to work 
even in no-Wayland mode. Good thing OE detects those .so dependencies and 
automatically pulls them in for you.

Have you tried removing Wayland pieces from the rootfs after the fact? I'm 
guessing SGX would stop working due to dynamic linker/loader not being able to 
resolve all the dependencies hardcoded in .so...

-- 
Denys


On Thu, Aug 17, 2017 at 12:02:27PM -0400, Ruei, Eric wrote:
> Hi, Denys:
> 
> Yes, we can make PLSDK image with Weston disabled by removing wayland from 
> the DISTRO_FEATURES list.
> Conf/local.conf:
> DISTRO_FEATURES_remove = "wayland"
> 
> Therefore QT will use eglfs as the default QPA.
> 
> However, some of the wayland related libraries and components are still 
> present at the target file system.
> There is no need to update ti-sgx-ddk-um_1.14.3699939 recipe and SGX should 
> work by using libpvrDRMWSEGL.so.
> 
> Best regards,
> 
> Eric
> 
> 
> -----Original Message-----
> From: Dmytriyenko, Denys 
> Sent: Wednesday, August 16, 2017 4:15 PM
> To: Ankur Tyagi
> Cc: meta-ti@yoctoproject.org; Ruei, Eric; R, Karthik
> Subject: Re: [meta-ti] ti-sgx-ddk-um_1.14.3699939: check for wayland in DISTRO_FEATURES
> 
> +Eric and Karthik.
> 
> On Sat, Aug 12, 2017 at 02:10:50PM +1200, Ankur Tyagi wrote:
> > So even if I added "wayland" in DISTRO_FEATURES_remove, it would still 
> > be packaged in resulting image.
> > 
> > I don't want to use x11, wayland and have also configured Qt to use 
> > eglfs qpa. But it seems wayland will be used anyhow. Am I correct ?
> 
> I suspect it will try to link/load the needed wayland libs, but I haven't tried it myself.
> 
> 
> Eric,
> 
> Since you've been playing with eglfs lately, can you please confirm/clarify 
> whether SGX can work w/o Wayland?
> 
> 
> > Old branch (daisy) was not having such dependency on wayland, may I know
> > why it is now ?
> 
> The old 3D Graphics SDK supported 2 modes - X11 and raw FB. We haven't 
> supported X11 for years. The new SGX DDK binaries are mostly for Wayland 
> graphics stack, as far as I know, since that's what we support on our 
> platforms.
> 
> 
> Karthik,
> 
> Anything you want to add or clarify here?
> 
> -- 
> Denys
> 
> 
> > On Sat, Aug 12, 2017 at 8:07 AM, Denys Dmytriyenko <denys@ti.com> wrote:
> > 
> > > No, it's not an optional dependency, unfortunately:
> > >
> > > $ for i in lib*.so.*.*.*; do echo $i; arm-linux-gnueabihf-readelf -a $i |
> > > grep wayland; done
> > > libdbm.so.1.14.3699939
> > >  0x00000001 (NEEDED)                     Shared library:
> > > [libwayland-server.so.0]
> > > libEGL.so.1.14.3699939
> > >  0x00000001 (NEEDED)                     Shared library:
> > > [libwayland-server.so.0]
> > > libGLES_CM.so.1.14.3699939
> > >  0x00000001 (NEEDED)                     Shared library:
> > > [libwayland-server.so.0]
> > > libGLESv2.so.1.14.3699939
> > >  0x00000001 (NEEDED)                     Shared library:
> > > [libwayland-server.so.0]
> > > libglslcompiler.so.1.14.3699939
> > >  0x00000001 (NEEDED)                     Shared library:
> > > [libwayland-server.so.0]
> > > libIMGegl.so.1.14.3699939
> > >  0x00000001 (NEEDED)                     Shared library:
> > > [libwayland-server.so.0]
> > >     23: 0000f715     4 FUNC    GLOBAL DEFAULT   11
> > > wayland_drm_buffer_get_fo
> > >     39: 0000f6f9    26 FUNC    GLOBAL DEFAULT   11 wayland_drm_uninit
> > >     91: 0000f66d    56 FUNC    GLOBAL DEFAULT   11 wayland_drm_buffer_get
> > >    119: 0000f6a5    84 FUNC    GLOBAL DEFAULT   11 wayland_drm_init
> > >    123: 0000f719     4 FUNC    GLOBAL DEFAULT   11
> > > wayland_drm_buffer_get_bu
> > > libpvr2d.so.1.14.3699939
> > >  0x00000001 (NEEDED)                     Shared library:
> > > [libwayland-server.so.0]
> > > libpvrDRMWSEGL.so.1.14.3699939
> > >  0x00000001 (NEEDED)                     Shared library:
> > > [libwayland-server.so.0]
> > > libpvrGBMWSEGL.so.1.14.3699939
> > >  0x00000001 (NEEDED)                     Shared library:
> > > [libwayland-server.so.0]
> > > libPVRScopeServices.so.1.14.3699939
> > >  0x00000001 (NEEDED)                     Shared library:
> > > [libwayland-server.so.0]
> > > libpvr_wlegl.so.1.14.3699939
> > >  0x00000001 (NEEDED)                     Shared library:
> > > [libwayland-server.so.0]
> > >  0x00000001 (NEEDED)                     Shared library:
> > > [libwayland-client.so.0]
> > > libpvrws_WAYLAND.so.1.14.3699939
> > >  0x00000001 (NEEDED)                     Shared library:
> > > [libwayland-server.so.0]
> > >  0x00000001 (NEEDED)                     Shared library:
> > > [libwayland-client.so.0]
> > > libsrv_init.so.1.14.3699939
> > >  0x00000001 (NEEDED)                     Shared library:
> > > [libwayland-server.so.0]
> > > libsrv_um.so.1.14.3699939
> > >  0x00000001 (NEEDED)                     Shared library:
> > > [libwayland-server.so.0]
> > > libusc.so.1.14.3699939
> > >  0x00000001 (NEEDED)                     Shared library:
> > > [libwayland-server.so.0]
> > >
> > > --
> > > Denys
> > >
> > >
> > > On Fri, Aug 11, 2017 at 12:47:24AM +1200, Ankur Tyagi wrote:
> > > > Signed-off-by: Ankur Tyagi <ankur.tyagi85@gmail.com>
> > > > ---
> > > >  recipes-graphics/libgles/ti-sgx-ddk-um_1.14.3699939.bb | 7 ++++---
> > > >  1 file changed, 4 insertions(+), 3 deletions(-)
> > > >
> > > > diff --git a/recipes-graphics/libgles/ti-sgx-ddk-um_1.14.3699939.bb
> > > b/recipes-graphics/libgles/ti-sgx-ddk-um_1.14.3699939.bb
> > > > index 60f63aa..593e405 100644
> > > > --- a/recipes-graphics/libgles/ti-sgx-ddk-um_1.14.3699939.bb
> > > > +++ b/recipes-graphics/libgles/ti-sgx-ddk-um_1.14.3699939.bb
> > > > @@ -34,8 +34,9 @@ inherit update-rc.d
> > > >  PR = "r26"
> > > >  PROVIDES += "virtual/egl virtual/libgles1 virtual/libgles2
> > > omap5-sgx-ddk-um-linux"
> > > >
> > > > -DEPENDS += "libdrm udev libgbm wayland libffi"
> > > > -RDEPENDS_${PN} += "libdrm libudev libgbm wayland libffi libdrm-omap"
> > > > +WAYLAND = "${@bb.utils.contains('DISTRO_FEATURES', 'wayland',
> > > 'wayland', '', d)}"
> > > > +DEPENDS += "libdrm udev libgbm ${WAYLAND} libffi"
> > > > +RDEPENDS_${PN} += "libdrm libudev libgbm ${WAYLAND} libffi libdrm-omap"
> > > >
> > > >  RPROVIDES_${PN} = "libegl libgles1 libgles2 omap5-sgx-ddk-um-linux"
> > > >  RPROVIDES_${PN}-dev = "libegl-dev libgles1-dev libgles2-dev
> > > omap5-sgx-ddk-um-linux-dev"
> > > > @@ -63,7 +64,7 @@ FILES_${PN} +=  "${includedir}/*"
> > > >  FILES_${PN} +=  "${sysconfdir}/*"
> > > >
> > > >  PACKAGES =+ "${PN}-plugins"
> > > > -FILES_${PN}-plugins = "${libdir}/libsrv_init.so ${libdir}/libsrv_um.so
> > > ${libdir}/libglslcompiler.so ${libdir}/libPVRScopeServices.so
> > > ${libdir}/libGLESv2.so ${libdir}/libpvrDRMWSEGL.so
> > > ${libdir}/libpvrGBMWSEGL.so  ${libdir}/libpvrws_WAYLAND.so"
> > > > +FILES_${PN}-plugins = "${libdir}/libsrv_init.so ${libdir}/libsrv_um.so
> > > ${libdir}/libglslcompiler.so ${libdir}/libPVRScopeServices.so
> > > ${libdir}/libGLESv2.so ${libdir}/libpvrDRMWSEGL.so
> > > ${libdir}/libpvrGBMWSEGL.so  ${@bb.utils.contains('DISTRO_FEATURES',
> > > 'wayland', '${libdir}/libpvrws_WAYLAND.so', '', d)}"
> > > >  RDEPENDS_${PN} += "${PN}-plugins"
> > > >
> > > >  ALLOW_EMPTY_${PN}-plugins = "1"
> > > > --
> > > > 2.7.4
> > > >
> > > > --
> > > > _______________________________________________
> > > > meta-ti mailing list
> > > > meta-ti@yoctoproject.org
> > > > https://lists.yoctoproject.org/listinfo/meta-ti
> > >