summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNaoto Yamaguchi <naoto.yamaguchi@aisin.co.jp>2024-12-07 00:44:11 +0900
committerNaoto Yamaguchi <naoto.yamaguchi@aisin.co.jp>2024-12-09 01:15:49 +0900
commit2357f7c66221db30f49838c8eb294f59e78ccc3b (patch)
tree3ff1665ef59ea24cc1986affb5f9f3ec400d6912
parent542964ec7d3c1d26c864c173e88b712eb597838b (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>
-rw-r--r--meta-agl-drm-lease/dynamic-layers/meta-agl-core/recipes-graphics/agl-compositor/agl-compositor/0001-Add-drm-lease-support.patch43
-rw-r--r--meta-agl-drm-lease/dynamic-layers/meta-agl-core/recipes-graphics/agl-compositor/agl-compositor/0001-compositor-Add-missing-drm-lease-name.patch35
-rw-r--r--meta-agl-drm-lease/dynamic-layers/meta-agl-core/recipes-graphics/agl-compositor/agl-compositor_git.bbappend2
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"