Re: [EXTERNAL] Re: Weston Desktop Shell

["Ruei, Eric" <e-ruei1@ti.com>]

[-- Attachment #1.1: Type: text/plain, Size: 2634 bytes --]

Hi, Adam:

The problem shows as the following line.

[19:24:55.055] initializing drm backend
[19:24:55.091] using /dev/dri/card1

The SGX DDK creates another dri-like device as card1, but it should not be used as DRM device.

Please apply the following patch to the Weston build.
0001-udev-seat-restrict-udev-enumeration-to-card0.patch.

You may need other patches under meta-arago.meta-arago-distro/recipes-graphics/wayland/weston including
0002-Weston-Allow-visual_id-to-be-0.patch

Best regards,

Eric



From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-bounces@yoctoproject.org] On Behalf Of Adam Lee
Sent: Monday, February 12, 2018 11:04 AM
To: meta-ti@yoctoproject.org
Subject: [EXTERNAL] Re: [meta-ti] Weston Desktop Shell

I spent a bit of time getting the logs:

root@am57xx-evm:~# weston --backend=drm-backend.so --tty=4
Date: 2018-01-08 UTC
[19:24:55.053] weston 2.0.0
               http://wayland.freedesktop.org
               Bug reports to: https://bugs.freedesktop.org/enter_bug.cgi?product=Wayland&component=weston&version=2.0.0
               Build: c26dbf3-dirty weston-launch: Provide a default version that doesn't require PAM (2018-02-09 11:58:14 -0500)
[19:24:55.053] Command line: weston --backend=drm-backend.so --tty=4
[19:24:55.053] OS: Linux, 4.9.69-g89d085d1a4, #10 SMP PREEMPT Thu Feb 8 12:11:21 EST 2018, armv7l
[19:24:55.053] Using config file '/etc//weston.ini'
[19:24:55.054] Output repaint window is 7 ms maximum.
[19:24:55.054] Loading module '/usr/lib/libweston-2/drm-backend.so'
[19:24:55.055] initializing drm backend
[19:24:55.091] using /dev/dri/card1
[19:24:55.091] Loading module '/usr/lib/libweston-2/gl-renderer.so'
failed to load module: /usr/lib/gbm/gbm_dri.so: cannot open shared object file: No such file or directory
failed to load module: /usr/lib/gbm/gbm_gallium_drm.so: cannot open shared object file: No such file or directory
loaded module : gbm_pvr.so
found valid GBM backend : gbm_pvr.so
PVR:(Error): [ 1157-> 1157] <              gbm_pvr_create_device():592|ERROR> Failed to create DBM device: No such device [0, ]
[19:24:55.095] failed to initialize egl
[19:24:55.120] fatal: failed to create compositor backend

Any insight as to why this is happening, it would be much appreciated.

Adam

On Fri, Feb 9, 2018 at 3:26 PM Adam Lee <adam.yh.lee@gmail.com<mailto:adam.yh.lee@gmail.com>> wrote:
Has anyone successfully brought up Weston desktop shell on the current master on Beagleboard X15 or AM57xx-EVM? I am having EGL initialization error. I don't know if it is something I have mis-configured.

BR,

Adam

[-- Attachment #1.2: Type: text/html, Size: 9641 bytes --]

[-- Attachment #2: 0001-udev-seat-restrict-udev-enumeration-to-card0.patch --]
[-- Type: application/octet-stream, Size: 1343 bytes --]

From e8e7a9f7dfa164a75fdbdca87622a2e13334478a Mon Sep 17 00:00:00 2001
From: Anand Balagopalakrishnan <anandb@ti.com>
Date: Sat, 23 Jan 2016 22:48:07 +0530
Subject: [PATCH 1/1] udev-seat: restrict udev enumeration to card0

In case of separate GPU and Display devices as found in embedded systems, we
could have modeset node and render node controlled by different drivers.
There is a distinct possibility that udev enumeration returns the DRM device
corresponding to render node as the primary DRM device.

Obviously, modeset operations cannot be done on the GPU DRM device.

Restrict the udev enumeration to card0 and ensure that DRM device corresponding
to display is returned as the primary DRM device.

Upstream-Status: Pending

Signed-off-by: Anand Balagopalakrishnan <anandb@ti.com>
---
 src/compositor-drm.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/compositor-drm.c b/src/compositor-drm.c
index 6777bf8..59c2cc5 100644
--- a/src/compositor-drm.c
+++ b/src/compositor-drm.c
@@ -2827,7 +2827,7 @@ find_primary_gpu(struct drm_backend *b, const char *seat)

	e = udev_enumerate_new(b->udev);
	udev_enumerate_add_match_subsystem(e, "drm");
-	udev_enumerate_add_match_sysname(e, "card[0-9]*");
+	udev_enumerate_add_match_sysname(e, "card0");

	udev_enumerate_scan_devices(e);
	drm_device = NULL;
--
1.7.9.5

[-- Attachment #3: 0002-Weston-Allow-visual_id-to-be-0.patch --]
[-- Type: application/octet-stream, Size: 934 bytes --]

From 5b5de6a814b43ca023a92b85b407b3d061dbc64f Mon Sep 17 00:00:00 2001
From: Eric Ruei <e-ruei1@ti.com>
Date: Thu, 9 Mar 2017 14:32:24 -0500
Subject: [PATCH 2/4] Weston: Allow visual_id to be 0

The inquiry of visual id from egl API eglGetConfigAttrib(EGL_NATIVE_VISUAL_ID)
is an optional feature. The visual id will be set to 0 if this feature is
not supported. Therefore, the return condition @function match_config_to_visual()
should be (id == visual_id || id == 0) instead of (id == visual_id)

Signed-off-by: Eric Ruei <e-ruei1@ti.com>
---
 src/gl-renderer.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/gl-renderer.c b/src/gl-renderer.c
index 23c0cd7..4c1f170 100644
--- a/src/gl-renderer.c
+++ b/src/gl-renderer.c
@@ -2462,7 +2462,7 @@ match_config_to_visual(EGLDisplay egl_display,
 				&id))
 			continue;
 
-		if (id == visual_id)
+		if (id == visual_id || id == 0)
 			return i;
 	}
 
-- 
1.9.1