diff options
author | Naoto Yamaguchi <naoto.yamaguchi@aisin.co.jp> | 2024-12-07 00:44:11 +0900 |
---|---|---|
committer | Naoto Yamaguchi <naoto.yamaguchi@aisin.co.jp> | 2024-12-09 01:15:49 +0900 |
commit | 2357f7c66221db30f49838c8eb294f59e78ccc3b (patch) | |
tree | 3ff1665ef59ea24cc1986affb5f9f3ec400d6912 | |
parent | 542964ec7d3c1d26c864c173e88b712eb597838b (diff) |
Repair drm lease patch for agl-compositor
After the c8f26648ef9ed09ca842fd81202802b73a07f789, The drm lease
support patch for agl-compositor can't apply. It causes build
error in container guest environment for AGL demo IVIs.
This patch updates drm lease support patch to fix build error.
Bug-AGL: SPEC-5318
Change-Id: Ifd257d30c8a676e4a17869458336b4e08d2ffb8e
Signed-off-by: Naoto Yamaguchi <naoto.yamaguchi@aisin.co.jp>
3 files changed, 24 insertions, 56 deletions
diff --git a/meta-agl-drm-lease/dynamic-layers/meta-agl-core/recipes-graphics/agl-compositor/agl-compositor/0001-Add-drm-lease-support.patch b/meta-agl-drm-lease/dynamic-layers/meta-agl-core/recipes-graphics/agl-compositor/agl-compositor/0001-Add-drm-lease-support.patch index a5853d36..a535664f 100644 --- a/meta-agl-drm-lease/dynamic-layers/meta-agl-core/recipes-graphics/agl-compositor/agl-compositor/0001-Add-drm-lease-support.patch +++ b/meta-agl-drm-lease/dynamic-layers/meta-agl-core/recipes-graphics/agl-compositor/agl-compositor/0001-Add-drm-lease-support.patch @@ -1,4 +1,4 @@ -From 7963529fc8e3e17a9596ef27cccc5c1b9e065e30 Mon Sep 17 00:00:00 2001 +From a1168c605680a106f52b4c926d473e8d9f6ae453 Mon Sep 17 00:00:00 2001 From: Marius Vlad <marius.vlad@collabora.com> Date: Fri, 3 May 2024 14:29:13 +0300 Subject: [PATCH] Add drm-lease support @@ -14,19 +14,19 @@ Change-Id: I78f8b9425280d4c0606003cd86f4cf30c065b888 --- meson.build | 6 +++++ meson_options.txt | 8 +++++++ - src/compositor.c | 7 ++++++ + src/compositor.c | 8 +++++++ src/drm-lease.c | 52 ++++++++++++++++++++++++++++++++++++++++++++ src/drm-lease.h | 19 ++++++++++++++++ src/ivi-compositor.h | 2 ++ - 6 files changed, 94 insertions(+) + 6 files changed, 95 insertions(+) create mode 100644 src/drm-lease.c create mode 100644 src/drm-lease.h diff --git a/meson.build b/meson.build -index aa811ad..f5a98e8 100644 +index 215b5ac..f25833b 100644 --- a/meson.build +++ b/meson.build -@@ -163,6 +163,12 @@ elif policy_to_install == 'rba' +@@ -159,6 +159,12 @@ elif policy_to_install == 'rba' message('Installing rba policy') endif @@ -63,27 +63,31 @@ index 7c0e103..e0463f4 100644 + description: 'Support for running weston with a leased DRM Master' +) diff --git a/src/compositor.c b/src/compositor.c -index 078157a..156a0ae 100644 +index 6aec416..a6db5e0 100644 --- a/src/compositor.c +++ b/src/compositor.c -@@ -57,6 +57,8 @@ +@@ -57,6 +57,7 @@ #include "config.h" #include "agl-shell-server-protocol.h" +#include "drm-lease.h" -+ - #ifdef HAVE_REMOTING - #include "remote.h" - #endif -@@ -1069,6 +1071,7 @@ load_drm_backend(struct ivi_compositor *ivi, int *argc, char *argv[], - bool force_pixman = false; + + #define WINDOWED_DEFAULT_WIDTH 1024 + #define WINDOWED_DEFAULT_HEIGHT 768 +@@ -905,10 +906,12 @@ load_drm_backend(struct ivi_compositor *ivi, int *argc, char *argv[], bool use_shadow; bool without_input = false; + struct ivi_backend *ivi_backend = NULL; + char *drm_lease_name = NULL; const struct weston_option options[] = { { WESTON_OPTION_STRING, "seat", 0, &config.seat_id }, -@@ -1094,6 +1097,7 @@ load_drm_backend(struct ivi_compositor *ivi, int *argc, char *argv[], + { WESTON_OPTION_STRING, "drm-device", 0, &config.specific_device }, ++ { WESTON_OPTION_STRING, "drm-lease", 0, &drm_lease_name }, + { WESTON_OPTION_BOOLEAN, "current-mode", 0, &use_current_mode }, + { WESTON_OPTION_BOOLEAN, "use-pixman", 0, &force_pixman }, + { WESTON_OPTION_BOOLEAN, "continue-without-input", false, &without_input } +@@ -930,6 +933,7 @@ load_drm_backend(struct ivi_compositor *ivi, int *argc, char *argv[], &config.pageflip_timeout, 0); weston_config_section_get_bool(section, "pixman-shadow", &use_shadow, 1); config.use_pixman_shadow = use_shadow; @@ -91,13 +95,12 @@ index 078157a..156a0ae 100644 if (without_input) ivi->compositor->require_input = !without_input; -@@ -1114,12 +1118,14 @@ load_drm_backend(struct ivi_compositor *ivi, int *argc, char *argv[], +@@ -954,10 +958,13 @@ load_drm_backend(struct ivi_compositor *ivi, int *argc, char *argv[], goto error; } + free(drm_lease_name); - load_remoting_plugin(ivi, ivi->config); - ++ return 0; error: @@ -106,7 +109,7 @@ index 078157a..156a0ae 100644 free(config.seat_id); return -1; } -@@ -2263,6 +2269,7 @@ error_compositor: +@@ -2271,6 +2278,7 @@ error_compositor: free(modules); modules = NULL; @@ -198,7 +201,7 @@ index 0000000..9fdc428 +#endif +#endif /* DRM_LEASE_H */ diff --git a/src/ivi-compositor.h b/src/ivi-compositor.h -index 695cf95..e3df79a 100644 +index 1e8c55a..ea8138a 100644 --- a/src/ivi-compositor.h +++ b/src/ivi-compositor.h @@ -36,6 +36,7 @@ @@ -209,7 +212,7 @@ index 695cf95..e3df79a 100644 #include "agl-shell-server-protocol.h" -@@ -145,6 +146,7 @@ struct ivi_compositor { +@@ -148,6 +149,7 @@ struct ivi_compositor { bool need_ivi_output_relayout; struct wl_list child_process_list; diff --git a/meta-agl-drm-lease/dynamic-layers/meta-agl-core/recipes-graphics/agl-compositor/agl-compositor/0001-compositor-Add-missing-drm-lease-name.patch b/meta-agl-drm-lease/dynamic-layers/meta-agl-core/recipes-graphics/agl-compositor/agl-compositor/0001-compositor-Add-missing-drm-lease-name.patch deleted file mode 100644 index 57ff8486..00000000 --- a/meta-agl-drm-lease/dynamic-layers/meta-agl-core/recipes-graphics/agl-compositor/agl-compositor/0001-compositor-Add-missing-drm-lease-name.patch +++ /dev/null @@ -1,35 +0,0 @@ -From f2e5939476630add3bf2c9254f72a0b57101f848 Mon Sep 17 00:00:00 2001 -From: Marius Vlad <marius.vlad@collabora.com> -Date: Mon, 10 Jun 2024 12:02:33 +0300 -Subject: [PATCH] compositor: Add missing drm-lease-name - -This was an oversight from commit 7963529fc8e3e, 'Add drm-lease -support', which backported the drm-lease support in the AGL compositor -for next release. - -Without a drm-lease-name, the device_id will be an negative integer -resulting in not using the parameter. This should fix leasing with the -AGL compositor. - -Bug-AGL: SPEC-5160 -Signed-off-by: Marius Vlad <marius.vlad@collabora.com> -Change-Id: I4282c115d2038852f6a56a77ce6ab81f2b8f62b4 ---- - src/compositor.c | 1 + - 1 file changed, 1 insertion(+) - -diff --git a/src/compositor.c b/src/compositor.c -index 156a0ae..65c61d8 100644 ---- a/src/compositor.c -+++ b/src/compositor.c -@@ -1076,6 +1076,7 @@ load_drm_backend(struct ivi_compositor *ivi, int *argc, char *argv[], - const struct weston_option options[] = { - { WESTON_OPTION_STRING, "seat", 0, &config.seat_id }, - { WESTON_OPTION_STRING, "drm-device", 0, &config.specific_device }, -+ { WESTON_OPTION_STRING, "drm-lease", 0, &drm_lease_name }, - { WESTON_OPTION_BOOLEAN, "current-mode", 0, &use_current_mode }, - { WESTON_OPTION_BOOLEAN, "use-pixman", 0, &force_pixman }, - { WESTON_OPTION_BOOLEAN, "continue-without-input", false, &without_input } --- -2.43.0 - diff --git a/meta-agl-drm-lease/dynamic-layers/meta-agl-core/recipes-graphics/agl-compositor/agl-compositor_git.bbappend b/meta-agl-drm-lease/dynamic-layers/meta-agl-core/recipes-graphics/agl-compositor/agl-compositor_git.bbappend index 63138299..3c64ce1f 100644 --- a/meta-agl-drm-lease/dynamic-layers/meta-agl-core/recipes-graphics/agl-compositor/agl-compositor_git.bbappend +++ b/meta-agl-drm-lease/dynamic-layers/meta-agl-core/recipes-graphics/agl-compositor/agl-compositor_git.bbappend @@ -1,6 +1,6 @@ FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}:" -SRC_URI += "file://0001-Add-drm-lease-support.patch file://0001-compositor-Add-missing-drm-lease-name.patch" +SRC_URI += "file://0001-Add-drm-lease-support.patch" PACKAGECONFIG[drm-lease] = "-Ddrm-lease=true,-Ddrm-lease=false,drm-lease-manager" PACKAGECONFIG:append = " drm-lease" |