[meta-ti] ti-sgx-ddk-um_1.14.3699939: check for wayland in DISTRO_FEATURES
Denys Dmytriyenko
denys at ti.com
Fri Aug 18 16:25:15 PDT 2017
On Sat, Aug 19, 2017 at 11:18:08AM +1200, Ankur Tyagi wrote:
> Thanks for the info Eric.
>
> So when wayland is removed from DISTRO_FEATURES, QT uses eglfs QPA and SGX
> DDK UM uses
As long as you have the required wayland libs present.
> libpvrDRMWSEGL.so: EGLFS (Raw), then why hard coded dependency on wayland ?
Ask Imagination.
> Why to package libraries in target rootfs if they are not needed ?
Because they are linked against and hence are needed to be present.
--
Denys
> On Fri, Aug 18, 2017 at 4:33 AM, Ruei, Eric <e-ruei1 at ti.com> wrote:
>
> > Hi, Denys:
> >
> > Yes and yes, we do need the wayland libraries to be present at the target
> > file system because they are required by the SGX DDM UM binaries.
> > The SGX DDK UM supports the following three (EGL) window systems and
> > auto-detect which one should be used.
> >
> > libpvrDRMWSEGL.so: EGLFS (Raw)
> > libpvrGBMWSEGL.so: Wayland-Server, DRM owner such as kmscube, or QT QPA
> > EGLFS_KMS.
> > libpvrws_WAYLAND.so: Wayland Client
> >
> > Best regards,
> >
> > Eric
> >
> > -----Original Message-----
> > From: Dmytriyenko, Denys
> > Sent: Thursday, August 17, 2017 12:14 PM
> > To: Ruei, Eric
> > Cc: Ankur Tyagi; meta-ti at yoctoproject.org; R, Karthik
> > Subject: Re: [meta-ti] ti-sgx-ddk-um_1.14.3699939: check for wayland in
> > DISTRO_FEATURES
> >
> > 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 at 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 at 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 at 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 at yoctoproject.org
> > > > > > https://lists.yoctoproject.org/listinfo/meta-ti
> > > > >
> >
More information about the meta-ti
mailing list