From f4c7dcac228f7b4a749f94f38efff6d15b08fc8a Mon Sep 17 00:00:00 2001 From: Marius Vlad Date: Tue, 28 Dec 2021 16:48:17 +0200 Subject: meta-agl-core/recipes-graphics/wayland: Backport two fixes for libweston-8 This backports two small fixes for some potential libweston crashes happening on the DRM backend tear down, or EGL initailization failure paths. Bug-AGL: SPEC-4197 Signed-off-by: Marius Vlad Reported-by: Scott Murray Change-Id: Idc28bb7ef6d8fde8c149bb5eb35a2ba40a25706a Reviewed-on: https://gerrit.automotivelinux.org/gerrit/c/AGL/meta-agl/+/27068 Tested-by: Jenkins Job builder account ci-image-build: Jenkins Job builder account ci-image-boot-test: Jenkins Job builder account Reviewed-by: Scott Murray Reviewed-by: Jan-Simon Moeller --- ...enderer-Avoid-double-free-on-init-failure.patch | 33 ++++++++++++++ ...ackend-drm-Re-order-gbm-destruction-at-DR.patch | 51 ++++++++++++++++++++++ .../wayland/weston_8.0_aglcore.inc | 2 + 3 files changed, 86 insertions(+) create mode 100644 meta-agl-core/recipes-graphics/wayland/weston/0001-gl-renderer-Avoid-double-free-on-init-failure.patch create mode 100644 meta-agl-core/recipes-graphics/wayland/weston/0001-libweston-backend-drm-Re-order-gbm-destruction-at-DR.patch diff --git a/meta-agl-core/recipes-graphics/wayland/weston/0001-gl-renderer-Avoid-double-free-on-init-failure.patch b/meta-agl-core/recipes-graphics/wayland/weston/0001-gl-renderer-Avoid-double-free-on-init-failure.patch new file mode 100644 index 000000000..582a33161 --- /dev/null +++ b/meta-agl-core/recipes-graphics/wayland/weston/0001-gl-renderer-Avoid-double-free-on-init-failure.patch @@ -0,0 +1,33 @@ +From d76947b6668e0fabe0a4551ac6c2c978f93768cd Mon Sep 17 00:00:00 2001 +From: Daniel Stone +Upstream-Status: Backport +Date: Mon, 3 Feb 2020 20:01:21 +0000 +Subject: [PATCH] gl-renderer: Avoid double-free on init failure + +If gl-renderer fails its initialisation, we return to compositor +teardown, which will try to free the renderer if ec->renderer was set. +This is unfortunate when we've already torn it down whilst failing +gl-renderer init, so just clear the renderer member so we don't try to +tear down twice. + +Signed-off-by: Daniel Stone +Reported-by: Emil Velikov +--- + libweston/renderer-gl/gl-renderer.c | 1 + + 1 file changed, 1 insertion(+) + +diff --git a/libweston/renderer-gl/gl-renderer.c b/libweston/renderer-gl/gl-renderer.c +index a40db3607..79285c008 100644 +--- a/libweston/renderer-gl/gl-renderer.c ++++ b/libweston/renderer-gl/gl-renderer.c +@@ -3596,6 +3596,7 @@ fail_terminate: + eglTerminate(gr->egl_display); + fail: + free(gr); ++ ec->renderer = NULL; + return -1; + } + +-- +2.33.0 + diff --git a/meta-agl-core/recipes-graphics/wayland/weston/0001-libweston-backend-drm-Re-order-gbm-destruction-at-DR.patch b/meta-agl-core/recipes-graphics/wayland/weston/0001-libweston-backend-drm-Re-order-gbm-destruction-at-DR.patch new file mode 100644 index 000000000..e8e324c32 --- /dev/null +++ b/meta-agl-core/recipes-graphics/wayland/weston/0001-libweston-backend-drm-Re-order-gbm-destruction-at-DR.patch @@ -0,0 +1,51 @@ +From d171c7b3ba346c4d0bd6494f45ebf0be3c3cc5fb Mon Sep 17 00:00:00 2001 +From: Marius Vlad +Upstream-Status: Backport +Date: Thu, 1 Apr 2021 00:12:00 +0300 +Subject: [PATCH] libweston/backend-drm: Re-order gbm destruction at + DRM-backend tear down + +Tearing down the drm-backend when there are no input devices, would call +for the gbm device destruction before compositor shutdown. The latter +would call into the renderer detroy function and assume that the +EGLDisplay, which was created using the before-mentioned gbm device, is +still available. This patch re-orders the gbm destruction after the +compositor shutdown when no one would make use of it. + +Fixes: #314 + +Signed-off-by: Marius Vlad +Suggested-by: Daniel Stone +--- + libweston/backend-drm/drm.c | 8 ++++---- + 1 file changed, 4 insertions(+), 4 deletions(-) + +diff --git a/libweston/backend-drm/drm.c b/libweston/backend-drm/drm.c +index c780c1c16..abfcabd3b 100644 +--- a/libweston/backend-drm/drm.c ++++ b/libweston/backend-drm/drm.c +@@ -3203,10 +3203,6 @@ err_drm_source: + err_udev_input: + udev_input_destroy(&b->input); + err_sprite: +-#ifdef BUILD_DRM_GBM +- if (b->gbm) +- gbm_device_destroy(b->gbm); +-#endif + destroy_sprites(b); + err_create_crtc_list: + drmModeFreeResources(res); +@@ -3218,6 +3214,10 @@ err_launcher: + weston_launcher_destroy(compositor->launcher); + err_compositor: + weston_compositor_shutdown(compositor); ++#ifdef BUILD_DRM_GBM ++ if (b->gbm) ++ gbm_device_destroy(b->gbm); ++#endif + free(b); + return NULL; + } +-- +2.33.0 + diff --git a/meta-agl-core/recipes-graphics/wayland/weston_8.0_aglcore.inc b/meta-agl-core/recipes-graphics/wayland/weston_8.0_aglcore.inc index 584c81357..71d27005a 100644 --- a/meta-agl-core/recipes-graphics/wayland/weston_8.0_aglcore.inc +++ b/meta-agl-core/recipes-graphics/wayland/weston_8.0_aglcore.inc @@ -4,6 +4,8 @@ SRC_URI:append = "\ file://0002-Allow-weston-to-start-from-a-systemd-user-session.patch \ file://0005-correctly-tear-down-drm-backend.patch \ file://0001-clients-window-Add-set_app_id-function-change-Waylan.patch \ + file://0001-libweston-backend-drm-Re-order-gbm-destruction-at-DR.patch \ + file://0001-gl-renderer-Avoid-double-free-on-init-failure.patch \ " # Workaround for incorrect upstream definition -- cgit 1.2.3-korg