intel-xe.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling
@ 2025-12-15 15:28 Jani Nikula
  2025-12-15 15:28 ` [PATCH 01/15] drm/i915: move display/intel_plane_initial.c to i915_initial_plane.c Jani Nikula
                   ` (20 more replies)
  0 siblings, 21 replies; 23+ messages in thread
From: Jani Nikula @ 2025-12-15 15:28 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

The i915 and xe initial plane handling deviate at a too high level,
leading to lots of duplication between the two. This series starts to
clean it up by moving it to display parent interface, and deduplicating
piecemeal.

This is intentionally done in small chunks to ease review and catch
regressions (hopefully none).

There's still more to be done, e.g. it's pointless to have both i915 and
xe call intel_framebuffer_init(), but there's some error path stuff to
figure out before doing this. And I ran out of steam a bit, and the
series got pretty long already.

Anyway, by reducing duplication, the series highlights the differences
between i915 and xe.

BR,
Jani.


Jani Nikula (15):
  drm/i915: move display/intel_plane_initial.c to i915_initial_plane.c
  drm/xe/display: rename xe_plane_initial.c to xe_initial_plane.c
  drm/i915: rename intel_plane_initial.h to intel_initial_plane.h
  drm/{i915,xe}: move initial plane calls to parent interface
  drm/{i915,xe}: deduplicate intel_initial_plane_config() between i915
    and xe
  drm/{i915,xe}: deduplicate plane_config_fini() between i915 and xe
  drm/{i915,xe}: start deduplicating intel_find_initial_plane_obj()
    between i915 and xe
  drm/i915: return plane_state from intel_reuse_initial_plane_obj()
  drm/xe: return plane_state from intel_reuse_initial_plane_obj()
  drm/i915: further deduplicate intel_find_initial_plane_obj()
  drm/{i915,xe}: deduplicate intel_alloc_initial_plane_obj() FB modifier
    checks
  drm/{i915,xe}: deduplicate initial plane setup
  drm/{i915,xe}: pass struct drm_plane_state instead of struct drm_crtc
    to ->setup
  drm/{i915,xe}: pass struct drm_device instead of drm_device to
    ->alloc_obj
  drm/i915: drop dependency on struct intel_display from i915 initial
    plane

 drivers/gpu/drm/i915/Makefile                 |   3 +-
 drivers/gpu/drm/i915/display/intel_display.c  |   8 +-
 .../drm/i915/display/intel_display_driver.c   |   2 +-
 .../drm/i915/display/intel_initial_plane.c    | 193 ++++++++
 ..._plane_initial.h => intel_initial_plane.h} |   6 +-
 .../drm/i915/display/intel_plane_initial.c    | 442 ------------------
 drivers/gpu/drm/i915/i915_driver.c            |   2 +
 drivers/gpu/drm/i915/i915_initial_plane.c     | 290 ++++++++++++
 drivers/gpu/drm/i915/i915_initial_plane.h     |   9 +
 drivers/gpu/drm/xe/Makefile                   |   3 +-
 drivers/gpu/drm/xe/display/xe_display.c       |   2 +
 drivers/gpu/drm/xe/display/xe_initial_plane.c | 189 ++++++++
 drivers/gpu/drm/xe/display/xe_initial_plane.h |   9 +
 drivers/gpu/drm/xe/display/xe_plane_initial.c | 321 -------------
 include/drm/intel/display_parent_interface.h  |  17 +
 15 files changed, 723 insertions(+), 773 deletions(-)
 create mode 100644 drivers/gpu/drm/i915/display/intel_initial_plane.c
 rename drivers/gpu/drm/i915/display/{intel_plane_initial.h => intel_initial_plane.h} (60%)
 delete mode 100644 drivers/gpu/drm/i915/display/intel_plane_initial.c
 create mode 100644 drivers/gpu/drm/i915/i915_initial_plane.c
 create mode 100644 drivers/gpu/drm/i915/i915_initial_plane.h
 create mode 100644 drivers/gpu/drm/xe/display/xe_initial_plane.c
 create mode 100644 drivers/gpu/drm/xe/display/xe_initial_plane.h
 delete mode 100644 drivers/gpu/drm/xe/display/xe_plane_initial.c

-- 
2.47.3


^ permalink raw reply	[flat|nested] 23+ messages in thread

* [PATCH 01/15] drm/i915: move display/intel_plane_initial.c to i915_initial_plane.c
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
@ 2025-12-15 15:28 ` Jani Nikula
  2025-12-15 15:28 ` [PATCH 02/15] drm/xe/display: rename xe_plane_initial.c to xe_initial_plane.c Jani Nikula
                   ` (19 subsequent siblings)
  20 siblings, 0 replies; 23+ messages in thread
From: Jani Nikula @ 2025-12-15 15:28 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

intel_plane_initial.c is i915 specific. Move it to i915 core. Start
renaming stuff with the slightly more natural "initial plane" rather
than "plane initial".

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/Makefile                   |  2 +-
 ...tel_plane_initial.c => i915_initial_plane.c} | 17 +++++++++--------
 2 files changed, 10 insertions(+), 9 deletions(-)
 rename drivers/gpu/drm/i915/{display/intel_plane_initial.c => i915_initial_plane.c} (97%)

diff --git a/drivers/gpu/drm/i915/Makefile b/drivers/gpu/drm/i915/Makefile
index f01b5d8a07c7..8d2ca226c86f 100644
--- a/drivers/gpu/drm/i915/Makefile
+++ b/drivers/gpu/drm/i915/Makefile
@@ -77,6 +77,7 @@ i915-$(CONFIG_PERF_EVENTS) += \
 # core display adaptation
 i915-y += \
 	i915_hdcp_gsc.o \
+	i915_initial_plane.o \
 	i915_panic.o
 
 # "Graphics Technology" (aka we talk to the gpu)
@@ -299,7 +300,6 @@ i915-y += \
 	display/intel_pch_display.o \
 	display/intel_pch_refclk.o \
 	display/intel_plane.o \
-	display/intel_plane_initial.o \
 	display/intel_pmdemand.o \
 	display/intel_psr.o \
 	display/intel_quirks.o \
diff --git a/drivers/gpu/drm/i915/display/intel_plane_initial.c b/drivers/gpu/drm/i915/i915_initial_plane.c
similarity index 97%
rename from drivers/gpu/drm/i915/display/intel_plane_initial.c
rename to drivers/gpu/drm/i915/i915_initial_plane.c
index ff1afd3a8f20..87c964485a41 100644
--- a/drivers/gpu/drm/i915/display/intel_plane_initial.c
+++ b/drivers/gpu/drm/i915/i915_initial_plane.c
@@ -5,17 +5,18 @@
 
 #include <drm/drm_print.h>
 
+#include "display/intel_crtc.h"
+#include "display/intel_display.h"
+#include "display/intel_display_core.h"
+#include "display/intel_display_types.h"
+#include "display/intel_fb.h"
+#include "display/intel_frontbuffer.h"
+#include "display/intel_plane.h"
+#include "display/intel_plane_initial.h"
 #include "gem/i915_gem_lmem.h"
 #include "gem/i915_gem_region.h"
+
 #include "i915_drv.h"
-#include "intel_crtc.h"
-#include "intel_display.h"
-#include "intel_display_core.h"
-#include "intel_display_types.h"
-#include "intel_fb.h"
-#include "intel_frontbuffer.h"
-#include "intel_plane.h"
-#include "intel_plane_initial.h"
 
 void intel_plane_initial_vblank_wait(struct intel_crtc *crtc)
 {
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH 02/15] drm/xe/display: rename xe_plane_initial.c to xe_initial_plane.c
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
  2025-12-15 15:28 ` [PATCH 01/15] drm/i915: move display/intel_plane_initial.c to i915_initial_plane.c Jani Nikula
@ 2025-12-15 15:28 ` Jani Nikula
  2025-12-15 15:28 ` [PATCH 03/15] drm/i915: rename intel_plane_initial.h to intel_initial_plane.h Jani Nikula
                   ` (18 subsequent siblings)
  20 siblings, 0 replies; 23+ messages in thread
From: Jani Nikula @ 2025-12-15 15:28 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

Follow i915 with the more naturally flowing naming.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/xe/Makefile                                     | 2 +-
 .../drm/xe/display/{xe_plane_initial.c => xe_initial_plane.c}   | 0
 2 files changed, 1 insertion(+), 1 deletion(-)
 rename drivers/gpu/drm/xe/display/{xe_plane_initial.c => xe_initial_plane.c} (100%)

diff --git a/drivers/gpu/drm/xe/Makefile b/drivers/gpu/drm/xe/Makefile
index 7f08b4cd91d6..84fd9f88f63d 100644
--- a/drivers/gpu/drm/xe/Makefile
+++ b/drivers/gpu/drm/xe/Makefile
@@ -217,8 +217,8 @@ xe-$(CONFIG_DRM_XE_DISPLAY) += \
 	display/xe_dsb_buffer.o \
 	display/xe_fb_pin.o \
 	display/xe_hdcp_gsc.o \
+	display/xe_initial_plane.o \
 	display/xe_panic.o \
-	display/xe_plane_initial.o \
 	display/xe_stolen.o \
 	display/xe_tdf.o
 
diff --git a/drivers/gpu/drm/xe/display/xe_plane_initial.c b/drivers/gpu/drm/xe/display/xe_initial_plane.c
similarity index 100%
rename from drivers/gpu/drm/xe/display/xe_plane_initial.c
rename to drivers/gpu/drm/xe/display/xe_initial_plane.c
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH 03/15] drm/i915: rename intel_plane_initial.h to intel_initial_plane.h
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
  2025-12-15 15:28 ` [PATCH 01/15] drm/i915: move display/intel_plane_initial.c to i915_initial_plane.c Jani Nikula
  2025-12-15 15:28 ` [PATCH 02/15] drm/xe/display: rename xe_plane_initial.c to xe_initial_plane.c Jani Nikula
@ 2025-12-15 15:28 ` Jani Nikula
  2025-12-15 15:28 ` [PATCH 04/15] drm/{i915, xe}: move initial plane calls to parent interface Jani Nikula
                   ` (17 subsequent siblings)
  20 siblings, 0 replies; 23+ messages in thread
From: Jani Nikula @ 2025-12-15 15:28 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

Follow the more naturally flowing naming. Rename both the header and the
vblank wait function.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/display/intel_display.c              | 8 ++++----
 drivers/gpu/drm/i915/display/intel_display_driver.c       | 2 +-
 .../{intel_plane_initial.h => intel_initial_plane.h}      | 6 +++---
 drivers/gpu/drm/i915/i915_initial_plane.c                 | 6 +++---
 drivers/gpu/drm/xe/display/xe_initial_plane.c             | 6 +++---
 5 files changed, 14 insertions(+), 14 deletions(-)
 rename drivers/gpu/drm/i915/display/{intel_plane_initial.h => intel_initial_plane.h} (60%)

diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
index 9c6d3ecdb589..a29a12bcec0b 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -100,6 +100,7 @@
 #include "intel_frontbuffer.h"
 #include "intel_hdmi.h"
 #include "intel_hotplug.h"
+#include "intel_initial_plane.h"
 #include "intel_link_bw.h"
 #include "intel_lt_phy.h"
 #include "intel_lvds.h"
@@ -113,7 +114,6 @@
 #include "intel_pfit.h"
 #include "intel_pipe_crc.h"
 #include "intel_plane.h"
-#include "intel_plane_initial.h"
 #include "intel_pmdemand.h"
 #include "intel_pps.h"
 #include "intel_psr.h"
@@ -639,7 +639,7 @@ void intel_plane_disable_noatomic(struct intel_crtc *crtc,
 	if ((crtc_state->active_planes & ~BIT(PLANE_CURSOR)) == 0 &&
 	    hsw_ips_disable(crtc_state)) {
 		crtc_state->ips_enabled = false;
-		intel_plane_initial_vblank_wait(crtc);
+		intel_initial_plane_vblank_wait(crtc);
 	}
 
 	/*
@@ -653,7 +653,7 @@ void intel_plane_disable_noatomic(struct intel_crtc *crtc,
 	 */
 	if (HAS_GMCH(display) &&
 	    intel_set_memory_cxsr(display, false))
-		intel_plane_initial_vblank_wait(crtc);
+		intel_initial_plane_vblank_wait(crtc);
 
 	/*
 	 * Gen2 reports pipe underruns whenever all planes are disabled.
@@ -663,7 +663,7 @@ void intel_plane_disable_noatomic(struct intel_crtc *crtc,
 		intel_set_cpu_fifo_underrun_reporting(display, crtc->pipe, false);
 
 	intel_plane_disable_arm(NULL, plane, crtc_state);
-	intel_plane_initial_vblank_wait(crtc);
+	intel_initial_plane_vblank_wait(crtc);
 }
 
 unsigned int
diff --git a/drivers/gpu/drm/i915/display/intel_display_driver.c b/drivers/gpu/drm/i915/display/intel_display_driver.c
index e282b533d5b6..b471e230fcb1 100644
--- a/drivers/gpu/drm/i915/display/intel_display_driver.c
+++ b/drivers/gpu/drm/i915/display/intel_display_driver.c
@@ -54,11 +54,11 @@
 #include "intel_hdcp.h"
 #include "intel_hotplug.h"
 #include "intel_hti.h"
+#include "intel_initial_plane.h"
 #include "intel_modeset_lock.h"
 #include "intel_modeset_setup.h"
 #include "intel_opregion.h"
 #include "intel_overlay.h"
-#include "intel_plane_initial.h"
 #include "intel_pmdemand.h"
 #include "intel_pps.h"
 #include "intel_psr.h"
diff --git a/drivers/gpu/drm/i915/display/intel_plane_initial.h b/drivers/gpu/drm/i915/display/intel_initial_plane.h
similarity index 60%
rename from drivers/gpu/drm/i915/display/intel_plane_initial.h
rename to drivers/gpu/drm/i915/display/intel_initial_plane.h
index 5c315acda210..5f9a347be8f0 100644
--- a/drivers/gpu/drm/i915/display/intel_plane_initial.h
+++ b/drivers/gpu/drm/i915/display/intel_initial_plane.h
@@ -3,13 +3,13 @@
  * Copyright © 2021 Intel Corporation
  */
 
-#ifndef __INTEL_PLANE_INITIAL_H__
-#define __INTEL_PLANE_INITIAL_H__
+#ifndef __INTEL_INITIAL_PLANE_H__
+#define __INTEL_INITIAL_PLANE_H__
 
 struct intel_crtc;
 struct intel_display;
 
 void intel_initial_plane_config(struct intel_display *display);
-void intel_plane_initial_vblank_wait(struct intel_crtc *crtc);
+void intel_initial_plane_vblank_wait(struct intel_crtc *crtc);
 
 #endif
diff --git a/drivers/gpu/drm/i915/i915_initial_plane.c b/drivers/gpu/drm/i915/i915_initial_plane.c
index 87c964485a41..b7f115708c32 100644
--- a/drivers/gpu/drm/i915/i915_initial_plane.c
+++ b/drivers/gpu/drm/i915/i915_initial_plane.c
@@ -11,14 +11,14 @@
 #include "display/intel_display_types.h"
 #include "display/intel_fb.h"
 #include "display/intel_frontbuffer.h"
+#include "display/intel_initial_plane.h"
 #include "display/intel_plane.h"
-#include "display/intel_plane_initial.h"
 #include "gem/i915_gem_lmem.h"
 #include "gem/i915_gem_region.h"
 
 #include "i915_drv.h"
 
-void intel_plane_initial_vblank_wait(struct intel_crtc *crtc)
+void intel_initial_plane_vblank_wait(struct intel_crtc *crtc)
 {
 	intel_crtc_wait_for_next_vblank(crtc);
 }
@@ -436,7 +436,7 @@ void intel_initial_plane_config(struct intel_display *display)
 		intel_find_initial_plane_obj(crtc, plane_configs);
 
 		if (display->funcs.display->fixup_initial_plane_config(crtc, plane_config))
-			intel_plane_initial_vblank_wait(crtc);
+			intel_initial_plane_vblank_wait(crtc);
 
 		plane_config_fini(plane_config);
 	}
diff --git a/drivers/gpu/drm/xe/display/xe_initial_plane.c b/drivers/gpu/drm/xe/display/xe_initial_plane.c
index 01c105a93bb9..9d5760e56c4c 100644
--- a/drivers/gpu/drm/xe/display/xe_initial_plane.c
+++ b/drivers/gpu/drm/xe/display/xe_initial_plane.c
@@ -19,15 +19,15 @@
 #include "intel_fb.h"
 #include "intel_fb_pin.h"
 #include "intel_frontbuffer.h"
+#include "intel_initial_plane.h"
 #include "intel_plane.h"
-#include "intel_plane_initial.h"
 #include "xe_bo.h"
 #include "xe_vram_types.h"
 #include "xe_wa.h"
 
 #include <generated/xe_device_wa_oob.h>
 
-void intel_plane_initial_vblank_wait(struct intel_crtc *crtc)
+void intel_initial_plane_vblank_wait(struct intel_crtc *crtc)
 {
 	/* Early xe has no irq */
 	struct xe_device *xe = to_xe_device(crtc->base.dev);
@@ -314,7 +314,7 @@ void intel_initial_plane_config(struct intel_display *display)
 		intel_find_initial_plane_obj(crtc, plane_configs);
 
 		if (display->funcs.display->fixup_initial_plane_config(crtc, plane_config))
-			intel_plane_initial_vblank_wait(crtc);
+			intel_initial_plane_vblank_wait(crtc);
 
 		plane_config_fini(plane_config);
 	}
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH 04/15] drm/{i915, xe}: move initial plane calls to parent interface
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
                   ` (2 preceding siblings ...)
  2025-12-15 15:28 ` [PATCH 03/15] drm/i915: rename intel_plane_initial.h to intel_initial_plane.h Jani Nikula
@ 2025-12-15 15:28 ` Jani Nikula
  2025-12-15 15:28 ` [PATCH 05/15] drm/{i915, xe}: deduplicate intel_initial_plane_config() between i915 and xe Jani Nikula
                   ` (16 subsequent siblings)
  20 siblings, 0 replies; 23+ messages in thread
From: Jani Nikula @ 2025-12-15 15:28 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

Add the initial plane handling functions to the display parent
interface. Add the call wrappers in dedicated intel_initial_plane.c
instead of intel_parent.c, as we'll be refactoring the calls heavily.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/Makefile                 |  1 +
 .../drm/i915/display/intel_initial_plane.c    | 20 +++++++++++++++++++
 drivers/gpu/drm/i915/i915_driver.c            |  2 ++
 drivers/gpu/drm/i915/i915_initial_plane.c     | 16 +++++++++++----
 drivers/gpu/drm/i915/i915_initial_plane.h     |  9 +++++++++
 drivers/gpu/drm/xe/Makefile                   |  1 +
 drivers/gpu/drm/xe/display/xe_display.c       |  2 ++
 drivers/gpu/drm/xe/display/xe_initial_plane.c | 17 ++++++++++++----
 drivers/gpu/drm/xe/display/xe_initial_plane.h |  9 +++++++++
 include/drm/intel/display_parent_interface.h  |  9 +++++++++
 10 files changed, 78 insertions(+), 8 deletions(-)
 create mode 100644 drivers/gpu/drm/i915/display/intel_initial_plane.c
 create mode 100644 drivers/gpu/drm/i915/i915_initial_plane.h
 create mode 100644 drivers/gpu/drm/xe/display/xe_initial_plane.h

diff --git a/drivers/gpu/drm/i915/Makefile b/drivers/gpu/drm/i915/Makefile
index 8d2ca226c86f..6f6b8e0ba647 100644
--- a/drivers/gpu/drm/i915/Makefile
+++ b/drivers/gpu/drm/i915/Makefile
@@ -288,6 +288,7 @@ i915-y += \
 	display/intel_hotplug.o \
 	display/intel_hotplug_irq.o \
 	display/intel_hti.o \
+	display/intel_initial_plane.o \
 	display/intel_link_bw.o \
 	display/intel_load_detect.o \
 	display/intel_lpe_audio.o \
diff --git a/drivers/gpu/drm/i915/display/intel_initial_plane.c b/drivers/gpu/drm/i915/display/intel_initial_plane.c
new file mode 100644
index 000000000000..c68d7555aee5
--- /dev/null
+++ b/drivers/gpu/drm/i915/display/intel_initial_plane.c
@@ -0,0 +1,20 @@
+// SPDX-License-Identifier: MIT
+/* Copyright © 2025 Intel Corporation */
+
+#include <drm/intel/display_parent_interface.h>
+
+#include "intel_display_core.h"
+#include "intel_display_types.h"
+#include "intel_initial_plane.h"
+
+void intel_initial_plane_vblank_wait(struct intel_crtc *crtc)
+{
+	struct intel_display *display = to_intel_display(crtc);
+
+	display->parent->initial_plane->vblank_wait(&crtc->base);
+}
+
+void intel_initial_plane_config(struct intel_display *display)
+{
+	display->parent->initial_plane->config(display->drm);
+}
diff --git a/drivers/gpu/drm/i915/i915_driver.c b/drivers/gpu/drm/i915/i915_driver.c
index 9bc5ef988be7..a6057c7cdd5c 100644
--- a/drivers/gpu/drm/i915/i915_driver.c
+++ b/drivers/gpu/drm/i915/i915_driver.c
@@ -98,6 +98,7 @@
 #include "i915_gmch.h"
 #include "i915_hdcp_gsc.h"
 #include "i915_hwmon.h"
+#include "i915_initial_plane.h"
 #include "i915_ioc32.h"
 #include "i915_ioctl.h"
 #include "i915_irq.h"
@@ -769,6 +770,7 @@ static bool vgpu_active(struct drm_device *drm)
 
 static const struct intel_display_parent_interface parent = {
 	.hdcp = &i915_display_hdcp_interface,
+	.initial_plane = &i915_display_initial_plane_interface,
 	.irq = &i915_display_irq_interface,
 	.panic = &i915_display_panic_interface,
 	.rpm = &i915_display_rpm_interface,
diff --git a/drivers/gpu/drm/i915/i915_initial_plane.c b/drivers/gpu/drm/i915/i915_initial_plane.c
index b7f115708c32..f26563eed9ba 100644
--- a/drivers/gpu/drm/i915/i915_initial_plane.c
+++ b/drivers/gpu/drm/i915/i915_initial_plane.c
@@ -4,6 +4,7 @@
  */
 
 #include <drm/drm_print.h>
+#include <drm/intel/display_parent_interface.h>
 
 #include "display/intel_crtc.h"
 #include "display/intel_display.h"
@@ -17,10 +18,11 @@
 #include "gem/i915_gem_region.h"
 
 #include "i915_drv.h"
+#include "i915_initial_plane.h"
 
-void intel_initial_plane_vblank_wait(struct intel_crtc *crtc)
+static void i915_initial_plane_vblank_wait(struct drm_crtc *crtc)
 {
-	intel_crtc_wait_for_next_vblank(crtc);
+	intel_crtc_wait_for_next_vblank(to_intel_crtc(crtc));
 }
 
 static bool
@@ -406,8 +408,9 @@ static void plane_config_fini(struct intel_initial_plane_config *plane_config)
 		i915_vma_put(plane_config->vma);
 }
 
-void intel_initial_plane_config(struct intel_display *display)
+static void i915_initial_plane_config(struct drm_device *drm)
 {
+	struct intel_display *display = to_intel_display(drm);
 	struct intel_initial_plane_config plane_configs[I915_MAX_PIPES] = {};
 	struct intel_crtc *crtc;
 
@@ -436,8 +439,13 @@ void intel_initial_plane_config(struct intel_display *display)
 		intel_find_initial_plane_obj(crtc, plane_configs);
 
 		if (display->funcs.display->fixup_initial_plane_config(crtc, plane_config))
-			intel_initial_plane_vblank_wait(crtc);
+			i915_initial_plane_vblank_wait(&crtc->base);
 
 		plane_config_fini(plane_config);
 	}
 }
+
+const struct intel_display_initial_plane_interface i915_display_initial_plane_interface = {
+	.vblank_wait = i915_initial_plane_vblank_wait,
+	.config = i915_initial_plane_config,
+};
diff --git a/drivers/gpu/drm/i915/i915_initial_plane.h b/drivers/gpu/drm/i915/i915_initial_plane.h
new file mode 100644
index 000000000000..99ba462659a6
--- /dev/null
+++ b/drivers/gpu/drm/i915/i915_initial_plane.h
@@ -0,0 +1,9 @@
+/* SPDX-License-Identifier: MIT */
+/* Copyright © 2025 Intel Corporation */
+
+#ifndef __I915_INITIAL_PLANE_H__
+#define __I915_INITIAL_PLANE_H__
+
+extern const struct intel_display_initial_plane_interface i915_display_initial_plane_interface;
+
+#endif
diff --git a/drivers/gpu/drm/xe/Makefile b/drivers/gpu/drm/xe/Makefile
index 84fd9f88f63d..89dc48cd73e2 100644
--- a/drivers/gpu/drm/xe/Makefile
+++ b/drivers/gpu/drm/xe/Makefile
@@ -294,6 +294,7 @@ xe-$(CONFIG_DRM_XE_DISPLAY) += \
 	i915-display/intel_hotplug.o \
 	i915-display/intel_hotplug_irq.o \
 	i915-display/intel_hti.o \
+	i915-display/intel_initial_plane.o \
 	i915-display/intel_link_bw.o \
 	i915-display/intel_lspcon.o \
 	i915-display/intel_lt_phy.o \
diff --git a/drivers/gpu/drm/xe/display/xe_display.c b/drivers/gpu/drm/xe/display/xe_display.c
index eda65a05f601..f8a831b5dc7d 100644
--- a/drivers/gpu/drm/xe/display/xe_display.c
+++ b/drivers/gpu/drm/xe/display/xe_display.c
@@ -37,6 +37,7 @@
 #include "skl_watermark.h"
 #include "xe_display_rpm.h"
 #include "xe_hdcp_gsc.h"
+#include "xe_initial_plane.h"
 #include "xe_module.h"
 #include "xe_panic.h"
 #include "xe_stolen.h"
@@ -538,6 +539,7 @@ static const struct intel_display_irq_interface xe_display_irq_interface = {
 
 static const struct intel_display_parent_interface parent = {
 	.hdcp = &xe_display_hdcp_interface,
+	.initial_plane = &xe_display_initial_plane_interface,
 	.irq = &xe_display_irq_interface,
 	.panic = &xe_display_panic_interface,
 	.rpm = &xe_display_rpm_interface,
diff --git a/drivers/gpu/drm/xe/display/xe_initial_plane.c b/drivers/gpu/drm/xe/display/xe_initial_plane.c
index 9d5760e56c4c..dd69f1c65903 100644
--- a/drivers/gpu/drm/xe/display/xe_initial_plane.c
+++ b/drivers/gpu/drm/xe/display/xe_initial_plane.c
@@ -6,6 +6,8 @@
 /* for ioread64 */
 #include <linux/io-64-nonatomic-lo-hi.h>
 
+#include <drm/intel/display_parent_interface.h>
+
 #include "regs/xe_gtt_defs.h"
 #include "xe_ggtt.h"
 #include "xe_mmio.h"
@@ -27,9 +29,10 @@
 
 #include <generated/xe_device_wa_oob.h>
 
-void intel_initial_plane_vblank_wait(struct intel_crtc *crtc)
+/* Early xe has no irq */
+static void xe_initial_plane_vblank_wait(struct drm_crtc *_crtc)
 {
-	/* Early xe has no irq */
+	struct intel_crtc *crtc = to_intel_crtc(_crtc);
 	struct xe_device *xe = to_xe_device(crtc->base.dev);
 	struct xe_reg pipe_frmtmstmp = XE_REG(i915_mmio_reg_offset(PIPE_FRMTMSTMP(crtc->pipe)));
 	u32 timestamp;
@@ -284,8 +287,9 @@ static void plane_config_fini(struct intel_initial_plane_config *plane_config)
 	}
 }
 
-void intel_initial_plane_config(struct intel_display *display)
+static void xe_initial_plane_config(struct drm_device *drm)
 {
+	struct intel_display *display = to_intel_display(drm);
 	struct intel_initial_plane_config plane_configs[I915_MAX_PIPES] = {};
 	struct intel_crtc *crtc;
 
@@ -314,8 +318,13 @@ void intel_initial_plane_config(struct intel_display *display)
 		intel_find_initial_plane_obj(crtc, plane_configs);
 
 		if (display->funcs.display->fixup_initial_plane_config(crtc, plane_config))
-			intel_initial_plane_vblank_wait(crtc);
+			xe_initial_plane_vblank_wait(&crtc->base);
 
 		plane_config_fini(plane_config);
 	}
 }
+
+const struct intel_display_initial_plane_interface xe_display_initial_plane_interface = {
+	.vblank_wait = xe_initial_plane_vblank_wait,
+	.config = xe_initial_plane_config,
+};
diff --git a/drivers/gpu/drm/xe/display/xe_initial_plane.h b/drivers/gpu/drm/xe/display/xe_initial_plane.h
new file mode 100644
index 000000000000..399d15f14441
--- /dev/null
+++ b/drivers/gpu/drm/xe/display/xe_initial_plane.h
@@ -0,0 +1,9 @@
+/* SPDX-License-Identifier: MIT */
+/* Copyright © 2025 Intel Corporation */
+
+#ifndef __XE_INITIAL_PLANE_H__
+#define __XE_INITIAL_PLANE_H__
+
+extern const struct intel_display_initial_plane_interface xe_display_initial_plane_interface;
+
+#endif
diff --git a/include/drm/intel/display_parent_interface.h b/include/drm/intel/display_parent_interface.h
index 55d4df714645..76341b4b03f7 100644
--- a/include/drm/intel/display_parent_interface.h
+++ b/include/drm/intel/display_parent_interface.h
@@ -7,6 +7,7 @@
 #include <linux/types.h>
 
 struct dma_fence;
+struct drm_crtc;
 struct drm_device;
 struct drm_scanout_buffer;
 struct intel_hdcp_gsc_context;
@@ -25,6 +26,11 @@ struct intel_display_hdcp_interface {
 	void (*gsc_context_free)(struct intel_hdcp_gsc_context *gsc_context);
 };
 
+struct intel_display_initial_plane_interface {
+	void (*vblank_wait)(struct drm_crtc *crtc);
+	void (*config)(struct drm_device *drm);
+};
+
 struct intel_display_irq_interface {
 	bool (*enabled)(struct drm_device *drm);
 	void (*synchronize)(struct drm_device *drm);
@@ -90,6 +96,9 @@ struct intel_display_parent_interface {
 	/** @hdcp: HDCP GSC interface */
 	const struct intel_display_hdcp_interface *hdcp;
 
+	/** @initial_plane: Initial plane interface */
+	const struct intel_display_initial_plane_interface *initial_plane;
+
 	/** @irq: IRQ interface */
 	const struct intel_display_irq_interface *irq;
 
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH 05/15] drm/{i915, xe}: deduplicate intel_initial_plane_config() between i915 and xe
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
                   ` (3 preceding siblings ...)
  2025-12-15 15:28 ` [PATCH 04/15] drm/{i915, xe}: move initial plane calls to parent interface Jani Nikula
@ 2025-12-15 15:28 ` Jani Nikula
  2025-12-15 15:28 ` [PATCH 06/15] drm/{i915, xe}: deduplicate plane_config_fini() " Jani Nikula
                   ` (15 subsequent siblings)
  20 siblings, 0 replies; 23+ messages in thread
From: Jani Nikula @ 2025-12-15 15:28 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

Move the parent interface at one step lower level, allowing
deduplication.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 .../drm/i915/display/intel_initial_plane.c    | 48 ++++++++++++++++++-
 drivers/gpu/drm/i915/i915_initial_plane.c     | 47 +++---------------
 drivers/gpu/drm/xe/display/xe_initial_plane.c | 47 +++---------------
 include/drm/intel/display_parent_interface.h  |  4 +-
 4 files changed, 62 insertions(+), 84 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_initial_plane.c b/drivers/gpu/drm/i915/display/intel_initial_plane.c
index c68d7555aee5..561a2ba2a486 100644
--- a/drivers/gpu/drm/i915/display/intel_initial_plane.c
+++ b/drivers/gpu/drm/i915/display/intel_initial_plane.c
@@ -14,7 +14,53 @@ void intel_initial_plane_vblank_wait(struct intel_crtc *crtc)
 	display->parent->initial_plane->vblank_wait(&crtc->base);
 }
 
+static void
+intel_find_initial_plane_obj(struct intel_crtc *crtc,
+			     struct intel_initial_plane_config plane_configs[])
+{
+	struct intel_display *display = to_intel_display(crtc);
+
+	display->parent->initial_plane->find_obj(&crtc->base, plane_configs);
+}
+
+static void plane_config_fini(struct intel_display *display,
+			      struct intel_initial_plane_config *plane_config)
+{
+	display->parent->initial_plane->config_fini(plane_config);
+}
+
 void intel_initial_plane_config(struct intel_display *display)
 {
-	display->parent->initial_plane->config(display->drm);
+	struct intel_initial_plane_config plane_configs[I915_MAX_PIPES] = {};
+	struct intel_crtc *crtc;
+
+	for_each_intel_crtc(display->drm, crtc) {
+		const struct intel_crtc_state *crtc_state =
+			to_intel_crtc_state(crtc->base.state);
+		struct intel_initial_plane_config *plane_config =
+			&plane_configs[crtc->pipe];
+
+		if (!crtc_state->hw.active)
+			continue;
+
+		/*
+		 * Note that reserving the BIOS fb up front prevents us
+		 * from stuffing other stolen allocations like the ring
+		 * on top.  This prevents some ugliness at boot time, and
+		 * can even allow for smooth boot transitions if the BIOS
+		 * fb is large enough for the active pipe configuration.
+		 */
+		display->funcs.display->get_initial_plane_config(crtc, plane_config);
+
+		/*
+		 * If the fb is shared between multiple heads, we'll
+		 * just get the first one.
+		 */
+		intel_find_initial_plane_obj(crtc, plane_configs);
+
+		if (display->funcs.display->fixup_initial_plane_config(crtc, plane_config))
+			intel_initial_plane_vblank_wait(crtc);
+
+		plane_config_fini(display, plane_config);
+	}
 }
diff --git a/drivers/gpu/drm/i915/i915_initial_plane.c b/drivers/gpu/drm/i915/i915_initial_plane.c
index f26563eed9ba..68cf2499855a 100644
--- a/drivers/gpu/drm/i915/i915_initial_plane.c
+++ b/drivers/gpu/drm/i915/i915_initial_plane.c
@@ -312,9 +312,10 @@ intel_alloc_initial_plane_obj(struct intel_crtc *crtc,
 }
 
 static void
-intel_find_initial_plane_obj(struct intel_crtc *crtc,
-			     struct intel_initial_plane_config plane_configs[])
+i915_find_initial_plane_obj(struct drm_crtc *_crtc,
+			    struct intel_initial_plane_config plane_configs[])
 {
+	struct intel_crtc *crtc = to_intel_crtc(_crtc);
 	struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
 	struct intel_initial_plane_config *plane_config =
 		&plane_configs[crtc->pipe];
@@ -392,7 +393,7 @@ intel_find_initial_plane_obj(struct intel_crtc *crtc,
 	atomic_or(plane->frontbuffer_bit, &to_intel_frontbuffer(fb)->bits);
 }
 
-static void plane_config_fini(struct intel_initial_plane_config *plane_config)
+static void i915_plane_config_fini(struct intel_initial_plane_config *plane_config)
 {
 	if (plane_config->fb) {
 		struct drm_framebuffer *fb = &plane_config->fb->base;
@@ -408,44 +409,8 @@ static void plane_config_fini(struct intel_initial_plane_config *plane_config)
 		i915_vma_put(plane_config->vma);
 }
 
-static void i915_initial_plane_config(struct drm_device *drm)
-{
-	struct intel_display *display = to_intel_display(drm);
-	struct intel_initial_plane_config plane_configs[I915_MAX_PIPES] = {};
-	struct intel_crtc *crtc;
-
-	for_each_intel_crtc(display->drm, crtc) {
-		const struct intel_crtc_state *crtc_state =
-			to_intel_crtc_state(crtc->base.state);
-		struct intel_initial_plane_config *plane_config =
-			&plane_configs[crtc->pipe];
-
-		if (!crtc_state->hw.active)
-			continue;
-
-		/*
-		 * Note that reserving the BIOS fb up front prevents us
-		 * from stuffing other stolen allocations like the ring
-		 * on top.  This prevents some ugliness at boot time, and
-		 * can even allow for smooth boot transitions if the BIOS
-		 * fb is large enough for the active pipe configuration.
-		 */
-		display->funcs.display->get_initial_plane_config(crtc, plane_config);
-
-		/*
-		 * If the fb is shared between multiple heads, we'll
-		 * just get the first one.
-		 */
-		intel_find_initial_plane_obj(crtc, plane_configs);
-
-		if (display->funcs.display->fixup_initial_plane_config(crtc, plane_config))
-			i915_initial_plane_vblank_wait(&crtc->base);
-
-		plane_config_fini(plane_config);
-	}
-}
-
 const struct intel_display_initial_plane_interface i915_display_initial_plane_interface = {
 	.vblank_wait = i915_initial_plane_vblank_wait,
-	.config = i915_initial_plane_config,
+	.find_obj = i915_find_initial_plane_obj,
+	.config_fini = i915_plane_config_fini,
 };
diff --git a/drivers/gpu/drm/xe/display/xe_initial_plane.c b/drivers/gpu/drm/xe/display/xe_initial_plane.c
index dd69f1c65903..0007337c60da 100644
--- a/drivers/gpu/drm/xe/display/xe_initial_plane.c
+++ b/drivers/gpu/drm/xe/display/xe_initial_plane.c
@@ -204,9 +204,10 @@ intel_alloc_initial_plane_obj(struct intel_crtc *crtc,
 }
 
 static void
-intel_find_initial_plane_obj(struct intel_crtc *crtc,
-			     struct intel_initial_plane_config plane_configs[])
+xe_find_initial_plane_obj(struct drm_crtc *_crtc,
+			  struct intel_initial_plane_config plane_configs[])
 {
+	struct intel_crtc *crtc = to_intel_crtc(_crtc);
 	struct intel_initial_plane_config *plane_config =
 		&plane_configs[crtc->pipe];
 	struct intel_plane *plane =
@@ -274,7 +275,7 @@ intel_find_initial_plane_obj(struct intel_crtc *crtc,
 	intel_plane_disable_noatomic(crtc, plane);
 }
 
-static void plane_config_fini(struct intel_initial_plane_config *plane_config)
+static void xe_plane_config_fini(struct intel_initial_plane_config *plane_config)
 {
 	if (plane_config->fb) {
 		struct drm_framebuffer *fb = &plane_config->fb->base;
@@ -287,44 +288,8 @@ static void plane_config_fini(struct intel_initial_plane_config *plane_config)
 	}
 }
 
-static void xe_initial_plane_config(struct drm_device *drm)
-{
-	struct intel_display *display = to_intel_display(drm);
-	struct intel_initial_plane_config plane_configs[I915_MAX_PIPES] = {};
-	struct intel_crtc *crtc;
-
-	for_each_intel_crtc(display->drm, crtc) {
-		const struct intel_crtc_state *crtc_state =
-			to_intel_crtc_state(crtc->base.state);
-		struct intel_initial_plane_config *plane_config =
-			&plane_configs[crtc->pipe];
-
-		if (!crtc_state->hw.active)
-			continue;
-
-		/*
-		 * Note that reserving the BIOS fb up front prevents us
-		 * from stuffing other stolen allocations like the ring
-		 * on top.  This prevents some ugliness at boot time, and
-		 * can even allow for smooth boot transitions if the BIOS
-		 * fb is large enough for the active pipe configuration.
-		 */
-		display->funcs.display->get_initial_plane_config(crtc, plane_config);
-
-		/*
-		 * If the fb is shared between multiple heads, we'll
-		 * just get the first one.
-		 */
-		intel_find_initial_plane_obj(crtc, plane_configs);
-
-		if (display->funcs.display->fixup_initial_plane_config(crtc, plane_config))
-			xe_initial_plane_vblank_wait(&crtc->base);
-
-		plane_config_fini(plane_config);
-	}
-}
-
 const struct intel_display_initial_plane_interface xe_display_initial_plane_interface = {
 	.vblank_wait = xe_initial_plane_vblank_wait,
-	.config = xe_initial_plane_config,
+	.find_obj = xe_find_initial_plane_obj,
+	.config_fini = xe_plane_config_fini,
 };
diff --git a/include/drm/intel/display_parent_interface.h b/include/drm/intel/display_parent_interface.h
index 76341b4b03f7..b6559fe22d1c 100644
--- a/include/drm/intel/display_parent_interface.h
+++ b/include/drm/intel/display_parent_interface.h
@@ -11,6 +11,7 @@ struct drm_crtc;
 struct drm_device;
 struct drm_scanout_buffer;
 struct intel_hdcp_gsc_context;
+struct intel_initial_plane_config;
 struct intel_panic;
 struct intel_stolen_node;
 struct ref_tracker;
@@ -28,7 +29,8 @@ struct intel_display_hdcp_interface {
 
 struct intel_display_initial_plane_interface {
 	void (*vblank_wait)(struct drm_crtc *crtc);
-	void (*config)(struct drm_device *drm);
+	void (*find_obj)(struct drm_crtc *crtc, struct intel_initial_plane_config *plane_configs);
+	void (*config_fini)(struct intel_initial_plane_config *plane_configs);
 };
 
 struct intel_display_irq_interface {
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH 06/15] drm/{i915, xe}: deduplicate plane_config_fini() between i915 and xe
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
                   ` (4 preceding siblings ...)
  2025-12-15 15:28 ` [PATCH 05/15] drm/{i915, xe}: deduplicate intel_initial_plane_config() between i915 and xe Jani Nikula
@ 2025-12-15 15:28 ` Jani Nikula
  2025-12-15 15:28 ` [PATCH 07/15] drm/{i915, xe}: start deduplicating intel_find_initial_plane_obj() " Jani Nikula
                   ` (14 subsequent siblings)
  20 siblings, 0 replies; 23+ messages in thread
From: Jani Nikula @ 2025-12-15 15:28 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

Move the common code to display. Retain empty xe_plane_config_fini() for
now, in case it's needed in the future.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/display/intel_initial_plane.c | 10 ++++++++++
 drivers/gpu/drm/i915/i915_initial_plane.c          | 10 ----------
 drivers/gpu/drm/xe/display/xe_initial_plane.c      |  9 ---------
 3 files changed, 10 insertions(+), 19 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_initial_plane.c b/drivers/gpu/drm/i915/display/intel_initial_plane.c
index 561a2ba2a486..62f4fb3be9f0 100644
--- a/drivers/gpu/drm/i915/display/intel_initial_plane.c
+++ b/drivers/gpu/drm/i915/display/intel_initial_plane.c
@@ -26,6 +26,16 @@ intel_find_initial_plane_obj(struct intel_crtc *crtc,
 static void plane_config_fini(struct intel_display *display,
 			      struct intel_initial_plane_config *plane_config)
 {
+	if (plane_config->fb) {
+		struct drm_framebuffer *fb = &plane_config->fb->base;
+
+		/* We may only have the stub and not a full framebuffer */
+		if (drm_framebuffer_read_refcount(fb))
+			drm_framebuffer_put(fb);
+		else
+			kfree(fb);
+	}
+
 	display->parent->initial_plane->config_fini(plane_config);
 }
 
diff --git a/drivers/gpu/drm/i915/i915_initial_plane.c b/drivers/gpu/drm/i915/i915_initial_plane.c
index 68cf2499855a..d8f2a2a11dd0 100644
--- a/drivers/gpu/drm/i915/i915_initial_plane.c
+++ b/drivers/gpu/drm/i915/i915_initial_plane.c
@@ -395,16 +395,6 @@ i915_find_initial_plane_obj(struct drm_crtc *_crtc,
 
 static void i915_plane_config_fini(struct intel_initial_plane_config *plane_config)
 {
-	if (plane_config->fb) {
-		struct drm_framebuffer *fb = &plane_config->fb->base;
-
-		/* We may only have the stub and not a full framebuffer */
-		if (drm_framebuffer_read_refcount(fb))
-			drm_framebuffer_put(fb);
-		else
-			kfree(fb);
-	}
-
 	if (plane_config->vma)
 		i915_vma_put(plane_config->vma);
 }
diff --git a/drivers/gpu/drm/xe/display/xe_initial_plane.c b/drivers/gpu/drm/xe/display/xe_initial_plane.c
index 0007337c60da..95000f8b0662 100644
--- a/drivers/gpu/drm/xe/display/xe_initial_plane.c
+++ b/drivers/gpu/drm/xe/display/xe_initial_plane.c
@@ -277,15 +277,6 @@ xe_find_initial_plane_obj(struct drm_crtc *_crtc,
 
 static void xe_plane_config_fini(struct intel_initial_plane_config *plane_config)
 {
-	if (plane_config->fb) {
-		struct drm_framebuffer *fb = &plane_config->fb->base;
-
-		/* We may only have the stub and not a full framebuffer */
-		if (drm_framebuffer_read_refcount(fb))
-			drm_framebuffer_put(fb);
-		else
-			kfree(fb);
-	}
 }
 
 const struct intel_display_initial_plane_interface xe_display_initial_plane_interface = {
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH 07/15] drm/{i915, xe}: start deduplicating intel_find_initial_plane_obj() between i915 and xe
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
                   ` (5 preceding siblings ...)
  2025-12-15 15:28 ` [PATCH 06/15] drm/{i915, xe}: deduplicate plane_config_fini() " Jani Nikula
@ 2025-12-15 15:28 ` Jani Nikula
  2025-12-15 15:28 ` [PATCH 08/15] drm/i915: return plane_state from intel_reuse_initial_plane_obj() Jani Nikula
                   ` (13 subsequent siblings)
  20 siblings, 0 replies; 23+ messages in thread
From: Jani Nikula @ 2025-12-15 15:28 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

Move some easy common parts to display. Initially, the
intel_find_initial_plane_obj() error path seems silly, but it'll be more
helpful this way for later changes.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 .../drm/i915/display/intel_initial_plane.c    | 29 ++++++++++++++--
 drivers/gpu/drm/i915/i915_initial_plane.c     | 34 +++----------------
 drivers/gpu/drm/xe/display/xe_initial_plane.c | 27 +++------------
 include/drm/intel/display_parent_interface.h  |  2 +-
 4 files changed, 38 insertions(+), 54 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_initial_plane.c b/drivers/gpu/drm/i915/display/intel_initial_plane.c
index 62f4fb3be9f0..9e67da94ac02 100644
--- a/drivers/gpu/drm/i915/display/intel_initial_plane.c
+++ b/drivers/gpu/drm/i915/display/intel_initial_plane.c
@@ -19,8 +19,33 @@ intel_find_initial_plane_obj(struct intel_crtc *crtc,
 			     struct intel_initial_plane_config plane_configs[])
 {
 	struct intel_display *display = to_intel_display(crtc);
-
-	display->parent->initial_plane->find_obj(&crtc->base, plane_configs);
+	struct intel_initial_plane_config *plane_config = &plane_configs[crtc->pipe];
+	struct intel_plane *plane = to_intel_plane(crtc->base.primary);
+	int ret;
+
+	/*
+	 * TODO:
+	 *   Disable planes if get_initial_plane_config() failed.
+	 *   Make sure things work if the surface base is not page aligned.
+	 */
+	if (!plane_config->fb)
+		return;
+
+	ret = display->parent->initial_plane->find_obj(&crtc->base, plane_configs);
+	if (ret)
+		goto nofb;
+
+	return;
+
+nofb:
+	/*
+	 * We've failed to reconstruct the BIOS FB.  Current display state
+	 * indicates that the primary plane is visible, but has a NULL FB,
+	 * which will lead to problems later if we don't fix it up.  The
+	 * simplest solution is to just disable the primary plane now and
+	 * pretend the BIOS never had it enabled.
+	 */
+	intel_plane_disable_noatomic(crtc, plane);
 }
 
 static void plane_config_fini(struct intel_display *display,
diff --git a/drivers/gpu/drm/i915/i915_initial_plane.c b/drivers/gpu/drm/i915/i915_initial_plane.c
index d8f2a2a11dd0..57afe6e29ce3 100644
--- a/drivers/gpu/drm/i915/i915_initial_plane.c
+++ b/drivers/gpu/drm/i915/i915_initial_plane.c
@@ -311,7 +311,7 @@ intel_alloc_initial_plane_obj(struct intel_crtc *crtc,
 	return false;
 }
 
-static void
+static int
 i915_find_initial_plane_obj(struct drm_crtc *_crtc,
 			    struct intel_initial_plane_config plane_configs[])
 {
@@ -326,39 +326,13 @@ i915_find_initial_plane_obj(struct drm_crtc *_crtc,
 	struct drm_framebuffer *fb;
 	struct i915_vma *vma;
 
-	/*
-	 * TODO:
-	 *   Disable planes if get_initial_plane_config() failed.
-	 *   Make sure things work if the surface base is not page aligned.
-	 */
-	if (!plane_config->fb)
-		return;
-
 	if (intel_alloc_initial_plane_obj(crtc, plane_config)) {
 		fb = &plane_config->fb->base;
 		vma = plane_config->vma;
-		goto valid_fb;
+	} else if (!intel_reuse_initial_plane_obj(crtc, plane_configs, &fb, &vma)) {
+		return -EINVAL;
 	}
 
-	/*
-	 * Failed to alloc the obj, check to see if we should share
-	 * an fb with another CRTC instead
-	 */
-	if (intel_reuse_initial_plane_obj(crtc, plane_configs, &fb, &vma))
-		goto valid_fb;
-
-	/*
-	 * We've failed to reconstruct the BIOS FB.  Current display state
-	 * indicates that the primary plane is visible, but has a NULL FB,
-	 * which will lead to problems later if we don't fix it up.  The
-	 * simplest solution is to just disable the primary plane now and
-	 * pretend the BIOS never had it enabled.
-	 */
-	intel_plane_disable_noatomic(crtc, plane);
-
-	return;
-
-valid_fb:
 	plane_state->uapi.rotation = plane_config->rotation;
 	intel_fb_fill_view(to_intel_framebuffer(fb),
 			   plane_state->uapi.rotation, &plane_state->view);
@@ -391,6 +365,8 @@ i915_find_initial_plane_obj(struct drm_crtc *_crtc,
 	intel_plane_copy_uapi_to_hw_state(plane_state, plane_state, crtc);
 
 	atomic_or(plane->frontbuffer_bit, &to_intel_frontbuffer(fb)->bits);
+
+	return 0;
 }
 
 static void i915_plane_config_fini(struct intel_initial_plane_config *plane_config)
diff --git a/drivers/gpu/drm/xe/display/xe_initial_plane.c b/drivers/gpu/drm/xe/display/xe_initial_plane.c
index 95000f8b0662..ddf22631240e 100644
--- a/drivers/gpu/drm/xe/display/xe_initial_plane.c
+++ b/drivers/gpu/drm/xe/display/xe_initial_plane.c
@@ -203,7 +203,7 @@ intel_alloc_initial_plane_obj(struct intel_crtc *crtc,
 	return false;
 }
 
-static void
+static int
 xe_find_initial_plane_obj(struct drm_crtc *_crtc,
 			  struct intel_initial_plane_config plane_configs[])
 {
@@ -217,18 +217,10 @@ xe_find_initial_plane_obj(struct drm_crtc *_crtc,
 	struct drm_framebuffer *fb;
 	struct i915_vma *vma;
 
-	/*
-	 * TODO:
-	 *   Disable planes if get_initial_plane_config() failed.
-	 *   Make sure things work if the surface base is not page aligned.
-	 */
-	if (!plane_config->fb)
-		return;
-
 	if (intel_alloc_initial_plane_obj(crtc, plane_config))
 		fb = &plane_config->fb->base;
 	else if (!intel_reuse_initial_plane_obj(crtc, plane_configs, &fb))
-		goto nofb;
+		return -EINVAL;
 
 	plane_state->uapi.rotation = plane_config->rotation;
 	intel_fb_fill_view(to_intel_framebuffer(fb),
@@ -237,7 +229,7 @@ xe_find_initial_plane_obj(struct drm_crtc *_crtc,
 	vma = intel_fb_pin_to_ggtt(fb, &plane_state->view.gtt,
 				   0, 0, 0, false, &plane_state->flags);
 	if (IS_ERR(vma))
-		goto nofb;
+		return PTR_ERR(vma);
 
 	plane_state->ggtt_vma = vma;
 
@@ -262,17 +254,8 @@ xe_find_initial_plane_obj(struct drm_crtc *_crtc,
 	atomic_or(plane->frontbuffer_bit, &to_intel_frontbuffer(fb)->bits);
 
 	plane_config->vma = vma;
-	return;
-
-nofb:
-	/*
-	 * We've failed to reconstruct the BIOS FB.  Current display state
-	 * indicates that the primary plane is visible, but has a NULL FB,
-	 * which will lead to problems later if we don't fix it up.  The
-	 * simplest solution is to just disable the primary plane now and
-	 * pretend the BIOS never had it enabled.
-	 */
-	intel_plane_disable_noatomic(crtc, plane);
+
+	return 0;
 }
 
 static void xe_plane_config_fini(struct intel_initial_plane_config *plane_config)
diff --git a/include/drm/intel/display_parent_interface.h b/include/drm/intel/display_parent_interface.h
index b6559fe22d1c..1be6320a5fc4 100644
--- a/include/drm/intel/display_parent_interface.h
+++ b/include/drm/intel/display_parent_interface.h
@@ -29,7 +29,7 @@ struct intel_display_hdcp_interface {
 
 struct intel_display_initial_plane_interface {
 	void (*vblank_wait)(struct drm_crtc *crtc);
-	void (*find_obj)(struct drm_crtc *crtc, struct intel_initial_plane_config *plane_configs);
+	int (*find_obj)(struct drm_crtc *crtc, struct intel_initial_plane_config *plane_configs);
 	void (*config_fini)(struct intel_initial_plane_config *plane_configs);
 };
 
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH 08/15] drm/i915: return plane_state from intel_reuse_initial_plane_obj()
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
                   ` (6 preceding siblings ...)
  2025-12-15 15:28 ` [PATCH 07/15] drm/{i915, xe}: start deduplicating intel_find_initial_plane_obj() " Jani Nikula
@ 2025-12-15 15:28 ` Jani Nikula
  2025-12-15 15:28 ` [PATCH 09/15] drm/xe: " Jani Nikula
                   ` (12 subsequent siblings)
  20 siblings, 0 replies; 23+ messages in thread
From: Jani Nikula @ 2025-12-15 15:28 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

Initialize fb and vma in the same level as the other code path.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/i915_initial_plane.c | 26 ++++++++++++-----------
 1 file changed, 14 insertions(+), 12 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_initial_plane.c b/drivers/gpu/drm/i915/i915_initial_plane.c
index 57afe6e29ce3..40b9f981c9ac 100644
--- a/drivers/gpu/drm/i915/i915_initial_plane.c
+++ b/drivers/gpu/drm/i915/i915_initial_plane.c
@@ -25,11 +25,9 @@ static void i915_initial_plane_vblank_wait(struct drm_crtc *crtc)
 	intel_crtc_wait_for_next_vblank(to_intel_crtc(crtc));
 }
 
-static bool
+static const struct intel_plane_state *
 intel_reuse_initial_plane_obj(struct intel_crtc *this,
-			      const struct intel_initial_plane_config plane_configs[],
-			      struct drm_framebuffer **fb,
-			      struct i915_vma **vma)
+			      const struct intel_initial_plane_config plane_configs[])
 {
 	struct intel_display *display = to_intel_display(this);
 	struct intel_crtc *crtc;
@@ -48,14 +46,11 @@ intel_reuse_initial_plane_obj(struct intel_crtc *this,
 		if (!plane_state->ggtt_vma)
 			continue;
 
-		if (plane_configs[this->pipe].base == plane_configs[crtc->pipe].base) {
-			*fb = plane_state->hw.fb;
-			*vma = plane_state->ggtt_vma;
-			return true;
-		}
+		if (plane_configs[this->pipe].base == plane_configs[crtc->pipe].base)
+			return plane_state;
 	}
 
-	return false;
+	return NULL;
 }
 
 static enum intel_memory_type
@@ -329,8 +324,15 @@ i915_find_initial_plane_obj(struct drm_crtc *_crtc,
 	if (intel_alloc_initial_plane_obj(crtc, plane_config)) {
 		fb = &plane_config->fb->base;
 		vma = plane_config->vma;
-	} else if (!intel_reuse_initial_plane_obj(crtc, plane_configs, &fb, &vma)) {
-		return -EINVAL;
+	} else {
+		const struct intel_plane_state *other_plane_state;
+
+		other_plane_state = intel_reuse_initial_plane_obj(crtc, plane_configs);
+		if (!other_plane_state)
+			return -EINVAL;
+
+		fb = other_plane_state->hw.fb;
+		vma = other_plane_state->ggtt_vma;
 	}
 
 	plane_state->uapi.rotation = plane_config->rotation;
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH 09/15] drm/xe: return plane_state from intel_reuse_initial_plane_obj()
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
                   ` (7 preceding siblings ...)
  2025-12-15 15:28 ` [PATCH 08/15] drm/i915: return plane_state from intel_reuse_initial_plane_obj() Jani Nikula
@ 2025-12-15 15:28 ` Jani Nikula
  2025-12-15 15:28 ` [PATCH 10/15] drm/i915: further deduplicate intel_find_initial_plane_obj() Jani Nikula
                   ` (11 subsequent siblings)
  20 siblings, 0 replies; 23+ messages in thread
From: Jani Nikula @ 2025-12-15 15:28 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

Initialize fb in the same level as the other code path.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/xe/display/xe_initial_plane.c | 26 +++++++++++--------
 1 file changed, 15 insertions(+), 11 deletions(-)

diff --git a/drivers/gpu/drm/xe/display/xe_initial_plane.c b/drivers/gpu/drm/xe/display/xe_initial_plane.c
index ddf22631240e..42d2c2f8138e 100644
--- a/drivers/gpu/drm/xe/display/xe_initial_plane.c
+++ b/drivers/gpu/drm/xe/display/xe_initial_plane.c
@@ -45,10 +45,9 @@ static void xe_initial_plane_vblank_wait(struct drm_crtc *_crtc)
 		drm_warn(&xe->drm, "waiting for early vblank failed with %i\n", ret);
 }
 
-static bool
+static const struct intel_plane_state *
 intel_reuse_initial_plane_obj(struct intel_crtc *this,
-			      const struct intel_initial_plane_config plane_configs[],
-			      struct drm_framebuffer **fb)
+			      const struct intel_initial_plane_config plane_configs[])
 {
 	struct xe_device *xe = to_xe_device(this->base.dev);
 	struct intel_crtc *crtc;
@@ -67,13 +66,11 @@ intel_reuse_initial_plane_obj(struct intel_crtc *this,
 		if (!plane_state->ggtt_vma)
 			continue;
 
-		if (plane_configs[this->pipe].base == plane_configs[crtc->pipe].base) {
-			*fb = plane_state->hw.fb;
-			return true;
-		}
+		if (plane_configs[this->pipe].base == plane_configs[crtc->pipe].base)
+			return plane_state;
 	}
 
-	return false;
+	return NULL;
 }
 
 static struct xe_bo *
@@ -217,10 +214,17 @@ xe_find_initial_plane_obj(struct drm_crtc *_crtc,
 	struct drm_framebuffer *fb;
 	struct i915_vma *vma;
 
-	if (intel_alloc_initial_plane_obj(crtc, plane_config))
+	if (intel_alloc_initial_plane_obj(crtc, plane_config)) {
 		fb = &plane_config->fb->base;
-	else if (!intel_reuse_initial_plane_obj(crtc, plane_configs, &fb))
-		return -EINVAL;
+	} else {
+		const struct intel_plane_state *other_plane_state;
+
+		other_plane_state = intel_reuse_initial_plane_obj(crtc, plane_configs);
+		if (!other_plane_state)
+			return -EINVAL;
+
+		fb = other_plane_state->hw.fb;
+	}
 
 	plane_state->uapi.rotation = plane_config->rotation;
 	intel_fb_fill_view(to_intel_framebuffer(fb),
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH 10/15] drm/i915: further deduplicate intel_find_initial_plane_obj()
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
                   ` (8 preceding siblings ...)
  2025-12-15 15:28 ` [PATCH 09/15] drm/xe: " Jani Nikula
@ 2025-12-15 15:28 ` Jani Nikula
  2025-12-15 15:28 ` [PATCH 11/15] drm/{i915, xe}: deduplicate intel_alloc_initial_plane_obj() FB modifier checks Jani Nikula
                   ` (10 subsequent siblings)
  20 siblings, 0 replies; 23+ messages in thread
From: Jani Nikula @ 2025-12-15 15:28 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

Move intel_reuse_initial_plane_obj() into common display code, and split
the ->find_obj hook into ->alloc_obj and ->setup hooks.

Return the struct drm_gem_object from ->alloc_obj in preparation for
moving more things to display.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 .../drm/i915/display/intel_initial_plane.c    | 55 ++++++++++++++-
 drivers/gpu/drm/i915/i915_initial_plane.c     | 70 ++++---------------
 drivers/gpu/drm/xe/display/xe_initial_plane.c | 67 ++++--------------
 include/drm/intel/display_parent_interface.h  |  7 +-
 4 files changed, 88 insertions(+), 111 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_initial_plane.c b/drivers/gpu/drm/i915/display/intel_initial_plane.c
index 9e67da94ac02..cbfcc52f180f 100644
--- a/drivers/gpu/drm/i915/display/intel_initial_plane.c
+++ b/drivers/gpu/drm/i915/display/intel_initial_plane.c
@@ -14,6 +14,43 @@ void intel_initial_plane_vblank_wait(struct intel_crtc *crtc)
 	display->parent->initial_plane->vblank_wait(&crtc->base);
 }
 
+static const struct intel_plane_state *
+intel_reuse_initial_plane_obj(struct intel_crtc *this,
+			      const struct intel_initial_plane_config plane_configs[])
+{
+	struct intel_display *display = to_intel_display(this);
+	struct intel_crtc *crtc;
+
+	for_each_intel_crtc(display->drm, crtc) {
+		struct intel_plane *plane =
+			to_intel_plane(crtc->base.primary);
+		const struct intel_plane_state *plane_state =
+			to_intel_plane_state(plane->base.state);
+		const struct intel_crtc_state *crtc_state =
+			to_intel_crtc_state(crtc->base.state);
+
+		if (!crtc_state->hw.active)
+			continue;
+
+		if (!plane_state->ggtt_vma)
+			continue;
+
+		if (plane_configs[this->pipe].base == plane_configs[crtc->pipe].base)
+			return plane_state;
+	}
+
+	return NULL;
+}
+
+static struct drm_gem_object *
+intel_alloc_initial_plane_obj(struct intel_crtc *crtc,
+			      struct intel_initial_plane_config *plane_config)
+{
+	struct intel_display *display = to_intel_display(crtc);
+
+	return display->parent->initial_plane->alloc_obj(&crtc->base, plane_config);
+}
+
 static void
 intel_find_initial_plane_obj(struct intel_crtc *crtc,
 			     struct intel_initial_plane_config plane_configs[])
@@ -21,6 +58,8 @@ intel_find_initial_plane_obj(struct intel_crtc *crtc,
 	struct intel_display *display = to_intel_display(crtc);
 	struct intel_initial_plane_config *plane_config = &plane_configs[crtc->pipe];
 	struct intel_plane *plane = to_intel_plane(crtc->base.primary);
+	struct drm_framebuffer *fb;
+	struct i915_vma *vma;
 	int ret;
 
 	/*
@@ -31,7 +70,21 @@ intel_find_initial_plane_obj(struct intel_crtc *crtc,
 	if (!plane_config->fb)
 		return;
 
-	ret = display->parent->initial_plane->find_obj(&crtc->base, plane_configs);
+	if (intel_alloc_initial_plane_obj(crtc, plane_config)) {
+		fb = &plane_config->fb->base;
+		vma = plane_config->vma;
+	} else {
+		const struct intel_plane_state *other_plane_state;
+
+		other_plane_state = intel_reuse_initial_plane_obj(crtc, plane_configs);
+		if (!other_plane_state)
+			goto nofb;
+
+		fb = other_plane_state->hw.fb;
+		vma = other_plane_state->ggtt_vma;
+	}
+
+	ret = display->parent->initial_plane->setup(&crtc->base, plane_config, fb, vma);
 	if (ret)
 		goto nofb;
 
diff --git a/drivers/gpu/drm/i915/i915_initial_plane.c b/drivers/gpu/drm/i915/i915_initial_plane.c
index 40b9f981c9ac..0117f9e115d1 100644
--- a/drivers/gpu/drm/i915/i915_initial_plane.c
+++ b/drivers/gpu/drm/i915/i915_initial_plane.c
@@ -25,34 +25,6 @@ static void i915_initial_plane_vblank_wait(struct drm_crtc *crtc)
 	intel_crtc_wait_for_next_vblank(to_intel_crtc(crtc));
 }
 
-static const struct intel_plane_state *
-intel_reuse_initial_plane_obj(struct intel_crtc *this,
-			      const struct intel_initial_plane_config plane_configs[])
-{
-	struct intel_display *display = to_intel_display(this);
-	struct intel_crtc *crtc;
-
-	for_each_intel_crtc(display->drm, crtc) {
-		struct intel_plane *plane =
-			to_intel_plane(crtc->base.primary);
-		const struct intel_plane_state *plane_state =
-			to_intel_plane_state(plane->base.state);
-		const struct intel_crtc_state *crtc_state =
-			to_intel_crtc_state(crtc->base.state);
-
-		if (!crtc_state->hw.active)
-			continue;
-
-		if (!plane_state->ggtt_vma)
-			continue;
-
-		if (plane_configs[this->pipe].base == plane_configs[crtc->pipe].base)
-			return plane_state;
-	}
-
-	return NULL;
-}
-
 static enum intel_memory_type
 initial_plane_memory_type(struct intel_display *display)
 {
@@ -258,10 +230,11 @@ initial_plane_vma(struct intel_display *display,
 	return NULL;
 }
 
-static bool
-intel_alloc_initial_plane_obj(struct intel_crtc *crtc,
-			      struct intel_initial_plane_config *plane_config)
+static struct drm_gem_object *
+i915_alloc_initial_plane_obj(struct drm_crtc *_crtc,
+			     struct intel_initial_plane_config *plane_config)
 {
+	struct intel_crtc *crtc = to_intel_crtc(_crtc);
 	struct intel_display *display = to_intel_display(crtc);
 	struct drm_mode_fb_cmd2 mode_cmd = {};
 	struct drm_framebuffer *fb = &plane_config->fb->base;
@@ -277,12 +250,12 @@ intel_alloc_initial_plane_obj(struct intel_crtc *crtc,
 		drm_dbg(display->drm,
 			"Unsupported modifier for initial FB: 0x%llx\n",
 			fb->modifier);
-		return false;
+		return NULL;
 	}
 
 	vma = initial_plane_vma(display, plane_config);
 	if (!vma)
-		return false;
+		return NULL;
 
 	mode_cmd.pixel_format = fb->format->format;
 	mode_cmd.width = fb->width;
@@ -299,41 +272,25 @@ intel_alloc_initial_plane_obj(struct intel_crtc *crtc,
 	}
 
 	plane_config->vma = vma;
-	return true;
+	return intel_bo_to_drm_bo(vma->obj);
 
 err_vma:
 	i915_vma_put(vma);
-	return false;
+	return NULL;
 }
 
 static int
-i915_find_initial_plane_obj(struct drm_crtc *_crtc,
-			    struct intel_initial_plane_config plane_configs[])
+i915_initial_plane_setup(struct drm_crtc *_crtc,
+			 struct intel_initial_plane_config *plane_config,
+			 struct drm_framebuffer *fb,
+			 struct i915_vma *vma)
 {
 	struct intel_crtc *crtc = to_intel_crtc(_crtc);
 	struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
-	struct intel_initial_plane_config *plane_config =
-		&plane_configs[crtc->pipe];
 	struct intel_plane *plane =
 		to_intel_plane(crtc->base.primary);
 	struct intel_plane_state *plane_state =
 		to_intel_plane_state(plane->base.state);
-	struct drm_framebuffer *fb;
-	struct i915_vma *vma;
-
-	if (intel_alloc_initial_plane_obj(crtc, plane_config)) {
-		fb = &plane_config->fb->base;
-		vma = plane_config->vma;
-	} else {
-		const struct intel_plane_state *other_plane_state;
-
-		other_plane_state = intel_reuse_initial_plane_obj(crtc, plane_configs);
-		if (!other_plane_state)
-			return -EINVAL;
-
-		fb = other_plane_state->hw.fb;
-		vma = other_plane_state->ggtt_vma;
-	}
 
 	plane_state->uapi.rotation = plane_config->rotation;
 	intel_fb_fill_view(to_intel_framebuffer(fb),
@@ -379,6 +336,7 @@ static void i915_plane_config_fini(struct intel_initial_plane_config *plane_conf
 
 const struct intel_display_initial_plane_interface i915_display_initial_plane_interface = {
 	.vblank_wait = i915_initial_plane_vblank_wait,
-	.find_obj = i915_find_initial_plane_obj,
+	.alloc_obj = i915_alloc_initial_plane_obj,
+	.setup = i915_initial_plane_setup,
 	.config_fini = i915_plane_config_fini,
 };
diff --git a/drivers/gpu/drm/xe/display/xe_initial_plane.c b/drivers/gpu/drm/xe/display/xe_initial_plane.c
index 42d2c2f8138e..2ce0f78cd7b1 100644
--- a/drivers/gpu/drm/xe/display/xe_initial_plane.c
+++ b/drivers/gpu/drm/xe/display/xe_initial_plane.c
@@ -45,34 +45,6 @@ static void xe_initial_plane_vblank_wait(struct drm_crtc *_crtc)
 		drm_warn(&xe->drm, "waiting for early vblank failed with %i\n", ret);
 }
 
-static const struct intel_plane_state *
-intel_reuse_initial_plane_obj(struct intel_crtc *this,
-			      const struct intel_initial_plane_config plane_configs[])
-{
-	struct xe_device *xe = to_xe_device(this->base.dev);
-	struct intel_crtc *crtc;
-
-	for_each_intel_crtc(&xe->drm, crtc) {
-		struct intel_plane *plane =
-			to_intel_plane(crtc->base.primary);
-		const struct intel_plane_state *plane_state =
-			to_intel_plane_state(plane->base.state);
-		const struct intel_crtc_state *crtc_state =
-			to_intel_crtc_state(crtc->base.state);
-
-		if (!crtc_state->hw.active)
-			continue;
-
-		if (!plane_state->ggtt_vma)
-			continue;
-
-		if (plane_configs[this->pipe].base == plane_configs[crtc->pipe].base)
-			return plane_state;
-	}
-
-	return NULL;
-}
-
 static struct xe_bo *
 initial_plane_bo(struct xe_device *xe,
 		 struct intel_initial_plane_config *plane_config)
@@ -152,10 +124,11 @@ initial_plane_bo(struct xe_device *xe,
 	return bo;
 }
 
-static bool
-intel_alloc_initial_plane_obj(struct intel_crtc *crtc,
-			      struct intel_initial_plane_config *plane_config)
+static struct drm_gem_object *
+xe_alloc_initial_plane_obj(struct drm_crtc *_crtc,
+			   struct intel_initial_plane_config *plane_config)
 {
+	struct intel_crtc *crtc = to_intel_crtc(_crtc);
 	struct xe_device *xe = to_xe_device(crtc->base.dev);
 	struct drm_mode_fb_cmd2 mode_cmd = { 0 };
 	struct drm_framebuffer *fb = &plane_config->fb->base;
@@ -171,7 +144,7 @@ intel_alloc_initial_plane_obj(struct intel_crtc *crtc,
 		drm_dbg_kms(&xe->drm,
 			    "Unsupported modifier for initial FB: 0x%llx\n",
 			    fb->modifier);
-		return false;
+		return NULL;
 	}
 
 	mode_cmd.pixel_format = fb->format->format;
@@ -183,7 +156,7 @@ intel_alloc_initial_plane_obj(struct intel_crtc *crtc,
 
 	bo = initial_plane_bo(xe, plane_config);
 	if (!bo)
-		return false;
+		return NULL;
 
 	if (intel_framebuffer_init(to_intel_framebuffer(fb),
 				   &bo->ttm.base, fb->format, &mode_cmd)) {
@@ -193,39 +166,26 @@ intel_alloc_initial_plane_obj(struct intel_crtc *crtc,
 	/* Reference handed over to fb */
 	xe_bo_put(bo);
 
-	return true;
+	return &bo->ttm.base;
 
 err_bo:
 	xe_bo_unpin_map_no_vm(bo);
-	return false;
+	return NULL;
 }
 
 static int
-xe_find_initial_plane_obj(struct drm_crtc *_crtc,
-			  struct intel_initial_plane_config plane_configs[])
+xe_initial_plane_setup(struct drm_crtc *_crtc,
+		       struct intel_initial_plane_config *plane_config,
+		       struct drm_framebuffer *fb,
+		       struct i915_vma *_unused)
 {
 	struct intel_crtc *crtc = to_intel_crtc(_crtc);
-	struct intel_initial_plane_config *plane_config =
-		&plane_configs[crtc->pipe];
 	struct intel_plane *plane =
 		to_intel_plane(crtc->base.primary);
 	struct intel_plane_state *plane_state =
 		to_intel_plane_state(plane->base.state);
-	struct drm_framebuffer *fb;
 	struct i915_vma *vma;
 
-	if (intel_alloc_initial_plane_obj(crtc, plane_config)) {
-		fb = &plane_config->fb->base;
-	} else {
-		const struct intel_plane_state *other_plane_state;
-
-		other_plane_state = intel_reuse_initial_plane_obj(crtc, plane_configs);
-		if (!other_plane_state)
-			return -EINVAL;
-
-		fb = other_plane_state->hw.fb;
-	}
-
 	plane_state->uapi.rotation = plane_config->rotation;
 	intel_fb_fill_view(to_intel_framebuffer(fb),
 			   plane_state->uapi.rotation, &plane_state->view);
@@ -268,6 +228,7 @@ static void xe_plane_config_fini(struct intel_initial_plane_config *plane_config
 
 const struct intel_display_initial_plane_interface xe_display_initial_plane_interface = {
 	.vblank_wait = xe_initial_plane_vblank_wait,
-	.find_obj = xe_find_initial_plane_obj,
+	.alloc_obj = xe_alloc_initial_plane_obj,
+	.setup = xe_initial_plane_setup,
 	.config_fini = xe_plane_config_fini,
 };
diff --git a/include/drm/intel/display_parent_interface.h b/include/drm/intel/display_parent_interface.h
index 1be6320a5fc4..164f3d473e3a 100644
--- a/include/drm/intel/display_parent_interface.h
+++ b/include/drm/intel/display_parent_interface.h
@@ -9,7 +9,10 @@
 struct dma_fence;
 struct drm_crtc;
 struct drm_device;
+struct drm_framebuffer;
+struct drm_gem_object;
 struct drm_scanout_buffer;
+struct i915_vma;
 struct intel_hdcp_gsc_context;
 struct intel_initial_plane_config;
 struct intel_panic;
@@ -29,7 +32,9 @@ struct intel_display_hdcp_interface {
 
 struct intel_display_initial_plane_interface {
 	void (*vblank_wait)(struct drm_crtc *crtc);
-	int (*find_obj)(struct drm_crtc *crtc, struct intel_initial_plane_config *plane_configs);
+	struct drm_gem_object *(*alloc_obj)(struct drm_crtc *crtc, struct intel_initial_plane_config *plane_config);
+	int (*setup)(struct drm_crtc *crtc, struct intel_initial_plane_config *plane_config,
+		     struct drm_framebuffer *fb, struct i915_vma *vma);
 	void (*config_fini)(struct intel_initial_plane_config *plane_configs);
 };
 
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH 11/15] drm/{i915, xe}: deduplicate intel_alloc_initial_plane_obj() FB modifier checks
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
                   ` (9 preceding siblings ...)
  2025-12-15 15:28 ` [PATCH 10/15] drm/i915: further deduplicate intel_find_initial_plane_obj() Jani Nikula
@ 2025-12-15 15:28 ` Jani Nikula
  2025-12-15 15:28 ` [PATCH 12/15] drm/{i915,xe}: deduplicate initial plane setup Jani Nikula
                   ` (9 subsequent siblings)
  20 siblings, 0 replies; 23+ messages in thread
From: Jani Nikula @ 2025-12-15 15:28 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

Move the modifier checks into common code to deduplicate.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/display/intel_initial_plane.c | 14 ++++++++++++++
 drivers/gpu/drm/i915/i915_initial_plane.c          | 13 -------------
 drivers/gpu/drm/xe/display/xe_initial_plane.c      | 13 -------------
 3 files changed, 14 insertions(+), 26 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_initial_plane.c b/drivers/gpu/drm/i915/display/intel_initial_plane.c
index cbfcc52f180f..e45d2f4e80ad 100644
--- a/drivers/gpu/drm/i915/display/intel_initial_plane.c
+++ b/drivers/gpu/drm/i915/display/intel_initial_plane.c
@@ -1,6 +1,7 @@
 // SPDX-License-Identifier: MIT
 /* Copyright © 2025 Intel Corporation */
 
+#include <drm/drm_print.h>
 #include <drm/intel/display_parent_interface.h>
 
 #include "intel_display_core.h"
@@ -47,6 +48,19 @@ intel_alloc_initial_plane_obj(struct intel_crtc *crtc,
 			      struct intel_initial_plane_config *plane_config)
 {
 	struct intel_display *display = to_intel_display(crtc);
+	struct intel_framebuffer *fb = plane_config->fb;
+
+	switch (fb->base.modifier) {
+	case DRM_FORMAT_MOD_LINEAR:
+	case I915_FORMAT_MOD_X_TILED:
+	case I915_FORMAT_MOD_Y_TILED:
+	case I915_FORMAT_MOD_4_TILED:
+		break;
+	default:
+		drm_dbg_kms(display->drm, "Unsupported modifier for initial FB: 0x%llx\n",
+			    fb->base.modifier);
+		return NULL;
+	}
 
 	return display->parent->initial_plane->alloc_obj(&crtc->base, plane_config);
 }
diff --git a/drivers/gpu/drm/i915/i915_initial_plane.c b/drivers/gpu/drm/i915/i915_initial_plane.c
index 0117f9e115d1..5f7dfe7c6d42 100644
--- a/drivers/gpu/drm/i915/i915_initial_plane.c
+++ b/drivers/gpu/drm/i915/i915_initial_plane.c
@@ -240,19 +240,6 @@ i915_alloc_initial_plane_obj(struct drm_crtc *_crtc,
 	struct drm_framebuffer *fb = &plane_config->fb->base;
 	struct i915_vma *vma;
 
-	switch (fb->modifier) {
-	case DRM_FORMAT_MOD_LINEAR:
-	case I915_FORMAT_MOD_X_TILED:
-	case I915_FORMAT_MOD_Y_TILED:
-	case I915_FORMAT_MOD_4_TILED:
-		break;
-	default:
-		drm_dbg(display->drm,
-			"Unsupported modifier for initial FB: 0x%llx\n",
-			fb->modifier);
-		return NULL;
-	}
-
 	vma = initial_plane_vma(display, plane_config);
 	if (!vma)
 		return NULL;
diff --git a/drivers/gpu/drm/xe/display/xe_initial_plane.c b/drivers/gpu/drm/xe/display/xe_initial_plane.c
index 2ce0f78cd7b1..e819175d5946 100644
--- a/drivers/gpu/drm/xe/display/xe_initial_plane.c
+++ b/drivers/gpu/drm/xe/display/xe_initial_plane.c
@@ -134,19 +134,6 @@ xe_alloc_initial_plane_obj(struct drm_crtc *_crtc,
 	struct drm_framebuffer *fb = &plane_config->fb->base;
 	struct xe_bo *bo;
 
-	switch (fb->modifier) {
-	case DRM_FORMAT_MOD_LINEAR:
-	case I915_FORMAT_MOD_X_TILED:
-	case I915_FORMAT_MOD_Y_TILED:
-	case I915_FORMAT_MOD_4_TILED:
-		break;
-	default:
-		drm_dbg_kms(&xe->drm,
-			    "Unsupported modifier for initial FB: 0x%llx\n",
-			    fb->modifier);
-		return NULL;
-	}
-
 	mode_cmd.pixel_format = fb->format->format;
 	mode_cmd.width = fb->width;
 	mode_cmd.height = fb->height;
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH 12/15] drm/{i915,xe}: deduplicate initial plane setup
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
                   ` (10 preceding siblings ...)
  2025-12-15 15:28 ` [PATCH 11/15] drm/{i915, xe}: deduplicate intel_alloc_initial_plane_obj() FB modifier checks Jani Nikula
@ 2025-12-15 15:28 ` Jani Nikula
  2025-12-15 15:28 ` [PATCH 13/15] drm/{i915, xe}: pass struct drm_plane_state instead of struct drm_crtc to ->setup Jani Nikula
                   ` (8 subsequent siblings)
  20 siblings, 0 replies; 23+ messages in thread
From: Jani Nikula @ 2025-12-15 15:28 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

Deduplicate more of the identical parts of i915 and xe initial plane
setup. This lets us reduce the core dependency on display internals.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 .../drm/i915/display/intel_initial_plane.c    | 26 +++++++++++++++++++
 drivers/gpu/drm/i915/i915_initial_plane.c     | 25 ------------------
 drivers/gpu/drm/xe/display/xe_initial_plane.c | 25 ------------------
 3 files changed, 26 insertions(+), 50 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_initial_plane.c b/drivers/gpu/drm/i915/display/intel_initial_plane.c
index e45d2f4e80ad..6e7bd6c3c02d 100644
--- a/drivers/gpu/drm/i915/display/intel_initial_plane.c
+++ b/drivers/gpu/drm/i915/display/intel_initial_plane.c
@@ -6,7 +6,10 @@
 
 #include "intel_display_core.h"
 #include "intel_display_types.h"
+#include "intel_fb.h"
+#include "intel_frontbuffer.h"
 #include "intel_initial_plane.h"
+#include "intel_plane.h"
 
 void intel_initial_plane_vblank_wait(struct intel_crtc *crtc)
 {
@@ -72,6 +75,7 @@ intel_find_initial_plane_obj(struct intel_crtc *crtc,
 	struct intel_display *display = to_intel_display(crtc);
 	struct intel_initial_plane_config *plane_config = &plane_configs[crtc->pipe];
 	struct intel_plane *plane = to_intel_plane(crtc->base.primary);
+	struct intel_plane_state *plane_state = to_intel_plane_state(plane->base.state);
 	struct drm_framebuffer *fb;
 	struct i915_vma *vma;
 	int ret;
@@ -98,10 +102,32 @@ intel_find_initial_plane_obj(struct intel_crtc *crtc,
 		vma = other_plane_state->ggtt_vma;
 	}
 
+	plane_state->uapi.rotation = plane_config->rotation;
+	intel_fb_fill_view(to_intel_framebuffer(fb),
+			   plane_state->uapi.rotation, &plane_state->view);
+
 	ret = display->parent->initial_plane->setup(&crtc->base, plane_config, fb, vma);
 	if (ret)
 		goto nofb;
 
+	plane_state->uapi.src_x = 0;
+	plane_state->uapi.src_y = 0;
+	plane_state->uapi.src_w = fb->width << 16;
+	plane_state->uapi.src_h = fb->height << 16;
+
+	plane_state->uapi.crtc_x = 0;
+	plane_state->uapi.crtc_y = 0;
+	plane_state->uapi.crtc_w = fb->width;
+	plane_state->uapi.crtc_h = fb->height;
+
+	plane_state->uapi.fb = fb;
+	drm_framebuffer_get(fb);
+
+	plane_state->uapi.crtc = &crtc->base;
+	intel_plane_copy_uapi_to_hw_state(plane_state, plane_state, crtc);
+
+	atomic_or(plane->frontbuffer_bit, &to_intel_frontbuffer(fb)->bits);
+
 	return;
 
 nofb:
diff --git a/drivers/gpu/drm/i915/i915_initial_plane.c b/drivers/gpu/drm/i915/i915_initial_plane.c
index 5f7dfe7c6d42..5fe96b52d2cd 100644
--- a/drivers/gpu/drm/i915/i915_initial_plane.c
+++ b/drivers/gpu/drm/i915/i915_initial_plane.c
@@ -11,9 +11,6 @@
 #include "display/intel_display_core.h"
 #include "display/intel_display_types.h"
 #include "display/intel_fb.h"
-#include "display/intel_frontbuffer.h"
-#include "display/intel_initial_plane.h"
-#include "display/intel_plane.h"
 #include "gem/i915_gem_lmem.h"
 #include "gem/i915_gem_region.h"
 
@@ -279,10 +276,6 @@ i915_initial_plane_setup(struct drm_crtc *_crtc,
 	struct intel_plane_state *plane_state =
 		to_intel_plane_state(plane->base.state);
 
-	plane_state->uapi.rotation = plane_config->rotation;
-	intel_fb_fill_view(to_intel_framebuffer(fb),
-			   plane_state->uapi.rotation, &plane_state->view);
-
 	__i915_vma_pin(vma);
 	plane_state->ggtt_vma = i915_vma_get(vma);
 	if (intel_plane_uses_fence(plane_state) &&
@@ -291,27 +284,9 @@ i915_initial_plane_setup(struct drm_crtc *_crtc,
 
 	plane_state->surf = i915_ggtt_offset(plane_state->ggtt_vma);
 
-	plane_state->uapi.src_x = 0;
-	plane_state->uapi.src_y = 0;
-	plane_state->uapi.src_w = fb->width << 16;
-	plane_state->uapi.src_h = fb->height << 16;
-
-	plane_state->uapi.crtc_x = 0;
-	plane_state->uapi.crtc_y = 0;
-	plane_state->uapi.crtc_w = fb->width;
-	plane_state->uapi.crtc_h = fb->height;
-
 	if (fb->modifier != DRM_FORMAT_MOD_LINEAR)
 		dev_priv->preserve_bios_swizzle = true;
 
-	plane_state->uapi.fb = fb;
-	drm_framebuffer_get(fb);
-
-	plane_state->uapi.crtc = &crtc->base;
-	intel_plane_copy_uapi_to_hw_state(plane_state, plane_state, crtc);
-
-	atomic_or(plane->frontbuffer_bit, &to_intel_frontbuffer(fb)->bits);
-
 	return 0;
 }
 
diff --git a/drivers/gpu/drm/xe/display/xe_initial_plane.c b/drivers/gpu/drm/xe/display/xe_initial_plane.c
index e819175d5946..45513fd79766 100644
--- a/drivers/gpu/drm/xe/display/xe_initial_plane.c
+++ b/drivers/gpu/drm/xe/display/xe_initial_plane.c
@@ -20,9 +20,6 @@
 #include "intel_display_types.h"
 #include "intel_fb.h"
 #include "intel_fb_pin.h"
-#include "intel_frontbuffer.h"
-#include "intel_initial_plane.h"
-#include "intel_plane.h"
 #include "xe_bo.h"
 #include "xe_vram_types.h"
 #include "xe_wa.h"
@@ -173,10 +170,6 @@ xe_initial_plane_setup(struct drm_crtc *_crtc,
 		to_intel_plane_state(plane->base.state);
 	struct i915_vma *vma;
 
-	plane_state->uapi.rotation = plane_config->rotation;
-	intel_fb_fill_view(to_intel_framebuffer(fb),
-			   plane_state->uapi.rotation, &plane_state->view);
-
 	vma = intel_fb_pin_to_ggtt(fb, &plane_state->view.gtt,
 				   0, 0, 0, false, &plane_state->flags);
 	if (IS_ERR(vma))
@@ -186,24 +179,6 @@ xe_initial_plane_setup(struct drm_crtc *_crtc,
 
 	plane_state->surf = i915_ggtt_offset(plane_state->ggtt_vma);
 
-	plane_state->uapi.src_x = 0;
-	plane_state->uapi.src_y = 0;
-	plane_state->uapi.src_w = fb->width << 16;
-	plane_state->uapi.src_h = fb->height << 16;
-
-	plane_state->uapi.crtc_x = 0;
-	plane_state->uapi.crtc_y = 0;
-	plane_state->uapi.crtc_w = fb->width;
-	plane_state->uapi.crtc_h = fb->height;
-
-	plane_state->uapi.fb = fb;
-	drm_framebuffer_get(fb);
-
-	plane_state->uapi.crtc = &crtc->base;
-	intel_plane_copy_uapi_to_hw_state(plane_state, plane_state, crtc);
-
-	atomic_or(plane->frontbuffer_bit, &to_intel_frontbuffer(fb)->bits);
-
 	plane_config->vma = vma;
 
 	return 0;
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH 13/15] drm/{i915, xe}: pass struct drm_plane_state instead of struct drm_crtc to ->setup
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
                   ` (11 preceding siblings ...)
  2025-12-15 15:28 ` [PATCH 12/15] drm/{i915,xe}: deduplicate initial plane setup Jani Nikula
@ 2025-12-15 15:28 ` Jani Nikula
  2025-12-15 15:28 ` [PATCH 14/15] drm/{i915, xe}: pass struct drm_device instead of drm_device to ->alloc_obj Jani Nikula
                   ` (7 subsequent siblings)
  20 siblings, 0 replies; 23+ messages in thread
From: Jani Nikula @ 2025-12-15 15:28 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

The initial plane parent interface ->setup hook no longer needs the crtc
for anything. Pass the struct drm_plane_state instead.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/display/intel_initial_plane.c |  2 +-
 drivers/gpu/drm/i915/i915_initial_plane.c          | 10 +++-------
 drivers/gpu/drm/xe/display/xe_initial_plane.c      |  8 ++------
 include/drm/intel/display_parent_interface.h       |  3 ++-
 4 files changed, 8 insertions(+), 15 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_initial_plane.c b/drivers/gpu/drm/i915/display/intel_initial_plane.c
index 6e7bd6c3c02d..e4d6aa438e66 100644
--- a/drivers/gpu/drm/i915/display/intel_initial_plane.c
+++ b/drivers/gpu/drm/i915/display/intel_initial_plane.c
@@ -106,7 +106,7 @@ intel_find_initial_plane_obj(struct intel_crtc *crtc,
 	intel_fb_fill_view(to_intel_framebuffer(fb),
 			   plane_state->uapi.rotation, &plane_state->view);
 
-	ret = display->parent->initial_plane->setup(&crtc->base, plane_config, fb, vma);
+	ret = display->parent->initial_plane->setup(plane->base.state, plane_config, fb, vma);
 	if (ret)
 		goto nofb;
 
diff --git a/drivers/gpu/drm/i915/i915_initial_plane.c b/drivers/gpu/drm/i915/i915_initial_plane.c
index 5fe96b52d2cd..40d4f990f23c 100644
--- a/drivers/gpu/drm/i915/i915_initial_plane.c
+++ b/drivers/gpu/drm/i915/i915_initial_plane.c
@@ -264,17 +264,13 @@ i915_alloc_initial_plane_obj(struct drm_crtc *_crtc,
 }
 
 static int
-i915_initial_plane_setup(struct drm_crtc *_crtc,
+i915_initial_plane_setup(struct drm_plane_state *_plane_state,
 			 struct intel_initial_plane_config *plane_config,
 			 struct drm_framebuffer *fb,
 			 struct i915_vma *vma)
 {
-	struct intel_crtc *crtc = to_intel_crtc(_crtc);
-	struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
-	struct intel_plane *plane =
-		to_intel_plane(crtc->base.primary);
-	struct intel_plane_state *plane_state =
-		to_intel_plane_state(plane->base.state);
+	struct intel_plane_state *plane_state = to_intel_plane_state(_plane_state);
+	struct drm_i915_private *dev_priv = to_i915(_plane_state->plane->dev);
 
 	__i915_vma_pin(vma);
 	plane_state->ggtt_vma = i915_vma_get(vma);
diff --git a/drivers/gpu/drm/xe/display/xe_initial_plane.c b/drivers/gpu/drm/xe/display/xe_initial_plane.c
index 45513fd79766..9999bc07743b 100644
--- a/drivers/gpu/drm/xe/display/xe_initial_plane.c
+++ b/drivers/gpu/drm/xe/display/xe_initial_plane.c
@@ -158,16 +158,12 @@ xe_alloc_initial_plane_obj(struct drm_crtc *_crtc,
 }
 
 static int
-xe_initial_plane_setup(struct drm_crtc *_crtc,
+xe_initial_plane_setup(struct drm_plane_state *_plane_state,
 		       struct intel_initial_plane_config *plane_config,
 		       struct drm_framebuffer *fb,
 		       struct i915_vma *_unused)
 {
-	struct intel_crtc *crtc = to_intel_crtc(_crtc);
-	struct intel_plane *plane =
-		to_intel_plane(crtc->base.primary);
-	struct intel_plane_state *plane_state =
-		to_intel_plane_state(plane->base.state);
+	struct intel_plane_state *plane_state = to_intel_plane_state(_plane_state);
 	struct i915_vma *vma;
 
 	vma = intel_fb_pin_to_ggtt(fb, &plane_state->view.gtt,
diff --git a/include/drm/intel/display_parent_interface.h b/include/drm/intel/display_parent_interface.h
index 164f3d473e3a..040a014b49ce 100644
--- a/include/drm/intel/display_parent_interface.h
+++ b/include/drm/intel/display_parent_interface.h
@@ -11,6 +11,7 @@ struct drm_crtc;
 struct drm_device;
 struct drm_framebuffer;
 struct drm_gem_object;
+struct drm_plane_state;
 struct drm_scanout_buffer;
 struct i915_vma;
 struct intel_hdcp_gsc_context;
@@ -33,7 +34,7 @@ struct intel_display_hdcp_interface {
 struct intel_display_initial_plane_interface {
 	void (*vblank_wait)(struct drm_crtc *crtc);
 	struct drm_gem_object *(*alloc_obj)(struct drm_crtc *crtc, struct intel_initial_plane_config *plane_config);
-	int (*setup)(struct drm_crtc *crtc, struct intel_initial_plane_config *plane_config,
+	int (*setup)(struct drm_plane_state *plane_state, struct intel_initial_plane_config *plane_config,
 		     struct drm_framebuffer *fb, struct i915_vma *vma);
 	void (*config_fini)(struct intel_initial_plane_config *plane_configs);
 };
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH 14/15] drm/{i915, xe}: pass struct drm_device instead of drm_device to ->alloc_obj
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
                   ` (12 preceding siblings ...)
  2025-12-15 15:28 ` [PATCH 13/15] drm/{i915, xe}: pass struct drm_plane_state instead of struct drm_crtc to ->setup Jani Nikula
@ 2025-12-15 15:28 ` Jani Nikula
  2025-12-15 15:28 ` [PATCH 15/15] drm/i915: drop dependency on struct intel_display from i915 initial plane Jani Nikula
                   ` (6 subsequent siblings)
  20 siblings, 0 replies; 23+ messages in thread
From: Jani Nikula @ 2025-12-15 15:28 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

The initial plane parent interface ->alloc_obj hook no longer needs the
crtc for anything. Pass struct drm_device instead.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/display/intel_initial_plane.c | 7 +++----
 drivers/gpu/drm/i915/i915_initial_plane.c          | 5 ++---
 drivers/gpu/drm/xe/display/xe_initial_plane.c      | 7 ++-----
 include/drm/intel/display_parent_interface.h       | 2 +-
 4 files changed, 8 insertions(+), 13 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_initial_plane.c b/drivers/gpu/drm/i915/display/intel_initial_plane.c
index e4d6aa438e66..ee545c033da6 100644
--- a/drivers/gpu/drm/i915/display/intel_initial_plane.c
+++ b/drivers/gpu/drm/i915/display/intel_initial_plane.c
@@ -47,10 +47,9 @@ intel_reuse_initial_plane_obj(struct intel_crtc *this,
 }
 
 static struct drm_gem_object *
-intel_alloc_initial_plane_obj(struct intel_crtc *crtc,
+intel_alloc_initial_plane_obj(struct intel_display *display,
 			      struct intel_initial_plane_config *plane_config)
 {
-	struct intel_display *display = to_intel_display(crtc);
 	struct intel_framebuffer *fb = plane_config->fb;
 
 	switch (fb->base.modifier) {
@@ -65,7 +64,7 @@ intel_alloc_initial_plane_obj(struct intel_crtc *crtc,
 		return NULL;
 	}
 
-	return display->parent->initial_plane->alloc_obj(&crtc->base, plane_config);
+	return display->parent->initial_plane->alloc_obj(display->drm, plane_config);
 }
 
 static void
@@ -88,7 +87,7 @@ intel_find_initial_plane_obj(struct intel_crtc *crtc,
 	if (!plane_config->fb)
 		return;
 
-	if (intel_alloc_initial_plane_obj(crtc, plane_config)) {
+	if (intel_alloc_initial_plane_obj(display, plane_config)) {
 		fb = &plane_config->fb->base;
 		vma = plane_config->vma;
 	} else {
diff --git a/drivers/gpu/drm/i915/i915_initial_plane.c b/drivers/gpu/drm/i915/i915_initial_plane.c
index 40d4f990f23c..de90ca69e0bb 100644
--- a/drivers/gpu/drm/i915/i915_initial_plane.c
+++ b/drivers/gpu/drm/i915/i915_initial_plane.c
@@ -228,11 +228,10 @@ initial_plane_vma(struct intel_display *display,
 }
 
 static struct drm_gem_object *
-i915_alloc_initial_plane_obj(struct drm_crtc *_crtc,
+i915_alloc_initial_plane_obj(struct drm_device *drm,
 			     struct intel_initial_plane_config *plane_config)
 {
-	struct intel_crtc *crtc = to_intel_crtc(_crtc);
-	struct intel_display *display = to_intel_display(crtc);
+	struct intel_display *display = to_intel_display(drm);
 	struct drm_mode_fb_cmd2 mode_cmd = {};
 	struct drm_framebuffer *fb = &plane_config->fb->base;
 	struct i915_vma *vma;
diff --git a/drivers/gpu/drm/xe/display/xe_initial_plane.c b/drivers/gpu/drm/xe/display/xe_initial_plane.c
index 9999bc07743b..4cfeafcc158d 100644
--- a/drivers/gpu/drm/xe/display/xe_initial_plane.c
+++ b/drivers/gpu/drm/xe/display/xe_initial_plane.c
@@ -14,8 +14,6 @@
 
 #include "i915_vma.h"
 #include "intel_crtc.h"
-#include "intel_display.h"
-#include "intel_display_core.h"
 #include "intel_display_regs.h"
 #include "intel_display_types.h"
 #include "intel_fb.h"
@@ -122,11 +120,10 @@ initial_plane_bo(struct xe_device *xe,
 }
 
 static struct drm_gem_object *
-xe_alloc_initial_plane_obj(struct drm_crtc *_crtc,
+xe_alloc_initial_plane_obj(struct drm_device *drm,
 			   struct intel_initial_plane_config *plane_config)
 {
-	struct intel_crtc *crtc = to_intel_crtc(_crtc);
-	struct xe_device *xe = to_xe_device(crtc->base.dev);
+	struct xe_device *xe = to_xe_device(drm);
 	struct drm_mode_fb_cmd2 mode_cmd = { 0 };
 	struct drm_framebuffer *fb = &plane_config->fb->base;
 	struct xe_bo *bo;
diff --git a/include/drm/intel/display_parent_interface.h b/include/drm/intel/display_parent_interface.h
index 040a014b49ce..b3acb6e49ed8 100644
--- a/include/drm/intel/display_parent_interface.h
+++ b/include/drm/intel/display_parent_interface.h
@@ -33,7 +33,7 @@ struct intel_display_hdcp_interface {
 
 struct intel_display_initial_plane_interface {
 	void (*vblank_wait)(struct drm_crtc *crtc);
-	struct drm_gem_object *(*alloc_obj)(struct drm_crtc *crtc, struct intel_initial_plane_config *plane_config);
+	struct drm_gem_object *(*alloc_obj)(struct drm_device *drm, struct intel_initial_plane_config *plane_config);
 	int (*setup)(struct drm_plane_state *plane_state, struct intel_initial_plane_config *plane_config,
 		     struct drm_framebuffer *fb, struct i915_vma *vma);
 	void (*config_fini)(struct intel_initial_plane_config *plane_configs);
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [PATCH 15/15] drm/i915: drop dependency on struct intel_display from i915 initial plane
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
                   ` (13 preceding siblings ...)
  2025-12-15 15:28 ` [PATCH 14/15] drm/{i915, xe}: pass struct drm_device instead of drm_device to ->alloc_obj Jani Nikula
@ 2025-12-15 15:28 ` Jani Nikula
  2025-12-15 17:31 ` ✗ CI.checkpatch: warning for drm/{i915, xe}: clean up and deduplicate initial plane handling Patchwork
                   ` (5 subsequent siblings)
  20 siblings, 0 replies; 23+ messages in thread
From: Jani Nikula @ 2025-12-15 15:28 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: jani.nikula

The i915 core initial plane handling doesn't actually need struct
intel_display for anything. Switch to i915 specific data structures in
i915 core code.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
 drivers/gpu/drm/i915/i915_initial_plane.c | 43 +++++++++--------------
 1 file changed, 17 insertions(+), 26 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_initial_plane.c b/drivers/gpu/drm/i915/i915_initial_plane.c
index de90ca69e0bb..7fb52d81f7b6 100644
--- a/drivers/gpu/drm/i915/i915_initial_plane.c
+++ b/drivers/gpu/drm/i915/i915_initial_plane.c
@@ -7,8 +7,6 @@
 #include <drm/intel/display_parent_interface.h>
 
 #include "display/intel_crtc.h"
-#include "display/intel_display.h"
-#include "display/intel_display_core.h"
 #include "display/intel_display_types.h"
 #include "display/intel_fb.h"
 #include "gem/i915_gem_lmem.h"
@@ -23,11 +21,9 @@ static void i915_initial_plane_vblank_wait(struct drm_crtc *crtc)
 }
 
 static enum intel_memory_type
-initial_plane_memory_type(struct intel_display *display)
+initial_plane_memory_type(struct drm_i915_private *i915)
 {
-	struct drm_i915_private *i915 = to_i915(display->drm);
-
-	if (display->platform.dgfx)
+	if (IS_DGFX(i915))
 		return INTEL_MEMORY_LOCAL;
 	else if (HAS_LMEMBAR_SMEM_STOLEN(i915))
 		return INTEL_MEMORY_STOLEN_LOCAL;
@@ -36,10 +32,9 @@ initial_plane_memory_type(struct intel_display *display)
 }
 
 static bool
-initial_plane_phys(struct intel_display *display,
+initial_plane_phys(struct drm_i915_private *i915,
 		   struct intel_initial_plane_config *plane_config)
 {
-	struct drm_i915_private *i915 = to_i915(display->drm);
 	struct i915_ggtt *ggtt = to_gt(i915)->ggtt;
 	struct intel_memory_region *mem;
 	enum intel_memory_type mem_type;
@@ -47,10 +42,10 @@ initial_plane_phys(struct intel_display *display,
 	dma_addr_t dma_addr;
 	u32 base;
 
-	mem_type = initial_plane_memory_type(display);
+	mem_type = initial_plane_memory_type(i915);
 	mem = intel_memory_region_by_type(i915, mem_type);
 	if (!mem) {
-		drm_dbg_kms(display->drm,
+		drm_dbg_kms(&i915->drm,
 			    "Initial plane memory region (type %s) not initialized\n",
 			    intel_memory_type_str(mem_type));
 		return false;
@@ -61,27 +56,24 @@ initial_plane_phys(struct intel_display *display,
 	dma_addr = intel_ggtt_read_entry(&ggtt->vm, base, &is_present, &is_local);
 
 	if (!is_present) {
-		drm_err(display->drm,
-			"Initial plane FB PTE not present\n");
+		drm_err(&i915->drm, "Initial plane FB PTE not present\n");
 		return false;
 	}
 
 	if (intel_memory_type_is_local(mem->type) != is_local) {
-		drm_err(display->drm,
-			"Initial plane FB PTE unsuitable for %s\n",
+		drm_err(&i915->drm, "Initial plane FB PTE unsuitable for %s\n",
 			mem->region.name);
 		return false;
 	}
 
 	if (dma_addr < mem->region.start || dma_addr > mem->region.end) {
-		drm_err(display->drm,
+		drm_err(&i915->drm,
 			"Initial plane programming using invalid range, dma_addr=%pa (%s [%pa-%pa])\n",
 			&dma_addr, mem->region.name, &mem->region.start, &mem->region.end);
 		return false;
 	}
 
-	drm_dbg(display->drm,
-		"Using dma_addr=%pa, based on initial plane programming\n",
+	drm_dbg(&i915->drm, "Using dma_addr=%pa, based on initial plane programming\n",
 		&dma_addr);
 
 	plane_config->phys_base = dma_addr - mem->region.start;
@@ -91,10 +83,9 @@ initial_plane_phys(struct intel_display *display,
 }
 
 static struct i915_vma *
-initial_plane_vma(struct intel_display *display,
+initial_plane_vma(struct drm_i915_private *i915,
 		  struct intel_initial_plane_config *plane_config)
 {
-	struct drm_i915_private *i915 = to_i915(display->drm);
 	struct intel_memory_region *mem;
 	struct drm_i915_gem_object *obj;
 	struct drm_mm_node orig_mm = {};
@@ -107,7 +98,7 @@ initial_plane_vma(struct intel_display *display,
 	if (plane_config->size == 0)
 		return NULL;
 
-	if (!initial_plane_phys(display, plane_config))
+	if (!initial_plane_phys(i915, plane_config))
 		return NULL;
 
 	phys_base = plane_config->phys_base;
@@ -126,7 +117,7 @@ initial_plane_vma(struct intel_display *display,
 	if (IS_ENABLED(CONFIG_FRAMEBUFFER_CONSOLE) &&
 	    mem == i915->mm.stolen_region &&
 	    size * 2 > i915->dsm.usable_size) {
-		drm_dbg_kms(display->drm, "Initial FB size exceeds half of stolen, discarding\n");
+		drm_dbg_kms(&i915->drm, "Initial FB size exceeds half of stolen, discarding\n");
 		return NULL;
 	}
 
@@ -134,7 +125,7 @@ initial_plane_vma(struct intel_display *display,
 					       I915_BO_ALLOC_USER |
 					       I915_BO_PREALLOC);
 	if (IS_ERR(obj)) {
-		drm_dbg_kms(display->drm, "Failed to preallocate initial FB in %s\n",
+		drm_dbg_kms(&i915->drm, "Failed to preallocate initial FB in %s\n",
 			    mem->region.name);
 		return NULL;
 	}
@@ -214,7 +205,7 @@ initial_plane_vma(struct intel_display *display,
 	if (drm_mm_node_allocated(&orig_mm))
 		drm_mm_remove_node(&orig_mm);
 
-	drm_dbg_kms(display->drm,
+	drm_dbg_kms(&i915->drm,
 		    "Initial plane fb bound to 0x%x in the ggtt (original 0x%x)\n",
 		    i915_ggtt_offset(vma), plane_config->base);
 
@@ -231,12 +222,12 @@ static struct drm_gem_object *
 i915_alloc_initial_plane_obj(struct drm_device *drm,
 			     struct intel_initial_plane_config *plane_config)
 {
-	struct intel_display *display = to_intel_display(drm);
+	struct drm_i915_private *i915 = to_i915(drm);
 	struct drm_mode_fb_cmd2 mode_cmd = {};
 	struct drm_framebuffer *fb = &plane_config->fb->base;
 	struct i915_vma *vma;
 
-	vma = initial_plane_vma(display, plane_config);
+	vma = initial_plane_vma(i915, plane_config);
 	if (!vma)
 		return NULL;
 
@@ -250,7 +241,7 @@ i915_alloc_initial_plane_obj(struct drm_device *drm,
 	if (intel_framebuffer_init(to_intel_framebuffer(fb),
 				   intel_bo_to_drm_bo(vma->obj),
 				   fb->format, &mode_cmd)) {
-		drm_dbg_kms(display->drm, "intel fb init failed\n");
+		drm_dbg_kms(&i915->drm, "intel fb init failed\n");
 		goto err_vma;
 	}
 
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 23+ messages in thread

* ✗ CI.checkpatch: warning for drm/{i915, xe}: clean up and deduplicate initial plane handling
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
                   ` (14 preceding siblings ...)
  2025-12-15 15:28 ` [PATCH 15/15] drm/i915: drop dependency on struct intel_display from i915 initial plane Jani Nikula
@ 2025-12-15 17:31 ` Patchwork
  2025-12-15 17:32 ` ✓ CI.KUnit: success " Patchwork
                   ` (4 subsequent siblings)
  20 siblings, 0 replies; 23+ messages in thread
From: Patchwork @ 2025-12-15 17:31 UTC (permalink / raw)
  To: Jani Nikula; +Cc: intel-xe

== Series Details ==

Series: drm/{i915, xe}: clean up and deduplicate initial plane handling
URL   : https://patchwork.freedesktop.org/series/159031/
State : warning

== Summary ==

+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
8f50e69d0ce3656564bbdf8b3e213d61470d463f
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 214c2b5a0abf919fc69947ae86b2e72fdfd37bfe
Author: Jani Nikula <jani.nikula@intel.com>
Date:   Mon Dec 15 17:28:29 2025 +0200

    drm/i915: drop dependency on struct intel_display from i915 initial plane
    
    The i915 core initial plane handling doesn't actually need struct
    intel_display for anything. Switch to i915 specific data structures in
    i915 core code.
    
    Signed-off-by: Jani Nikula <jani.nikula@intel.com>
+ /mt/dim checkpatch 342b565445c93d6fb5221aa9c23be20d65fa16f2 drm-intel
3c5e632a3cd2 drm/i915: move display/intel_plane_initial.c to i915_initial_plane.c
-:35: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#35: 
rename from drivers/gpu/drm/i915/display/intel_plane_initial.c

total: 0 errors, 1 warnings, 0 checks, 40 lines checked
ad48ee1ee7de drm/xe/display: rename xe_plane_initial.c to xe_initial_plane.c
-:27: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#27: 
rename from drivers/gpu/drm/xe/display/xe_plane_initial.c

total: 0 errors, 1 warnings, 0 checks, 9 lines checked
52d1f13c13bb drm/i915: rename intel_plane_initial.h to intel_initial_plane.h
-:78: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#78: 
rename from drivers/gpu/drm/i915/display/intel_plane_initial.h

total: 0 errors, 1 warnings, 0 checks, 115 lines checked
e7b6c8e9f73d drm/{i915, xe}: move initial plane calls to parent interface
-:25: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#25: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 195 lines checked
7aa7e6bef831 drm/{i915, xe}: deduplicate intel_initial_plane_config() between i915 and xe
1e82b8803897 drm/{i915, xe}: deduplicate plane_config_fini() between i915 and xe
4a3a74655761 drm/{i915, xe}: start deduplicating intel_find_initial_plane_obj() between i915 and xe
4ac5ee4b427b drm/i915: return plane_state from intel_reuse_initial_plane_obj()
cbd2143fdeb0 drm/xe: return plane_state from intel_reuse_initial_plane_obj()
58d98965a878 drm/i915: further deduplicate intel_find_initial_plane_obj()
-:367: WARNING:LONG_LINE: line length of 116 exceeds 100 columns
#367: FILE: include/drm/intel/display_parent_interface.h:35:
+	struct drm_gem_object *(*alloc_obj)(struct drm_crtc *crtc, struct intel_initial_plane_config *plane_config);

total: 0 errors, 1 warnings, 0 checks, 327 lines checked
f99ce4ccd042 drm/{i915, xe}: deduplicate intel_alloc_initial_plane_obj() FB modifier checks
2689ff7548c1 drm/{i915,xe}: deduplicate initial plane setup
d21420b5dfbb drm/{i915, xe}: pass struct drm_plane_state instead of struct drm_crtc to ->setup
-:90: WARNING:LONG_LINE: line length of 106 exceeds 100 columns
#90: FILE: include/drm/intel/display_parent_interface.h:37:
+	int (*setup)(struct drm_plane_state *plane_state, struct intel_initial_plane_config *plane_config,

total: 0 errors, 1 warnings, 0 checks, 61 lines checked
41ce8c189931 drm/{i915, xe}: pass struct drm_device instead of drm_device to ->alloc_obj
-:100: WARNING:LONG_LINE: line length of 117 exceeds 100 columns
#100: FILE: include/drm/intel/display_parent_interface.h:36:
+	struct drm_gem_object *(*alloc_obj)(struct drm_device *drm, struct intel_initial_plane_config *plane_config);

total: 0 errors, 1 warnings, 0 checks, 69 lines checked
214c2b5a0abf drm/i915: drop dependency on struct intel_display from i915 initial plane



^ permalink raw reply	[flat|nested] 23+ messages in thread

* ✓ CI.KUnit: success for drm/{i915, xe}: clean up and deduplicate initial plane handling
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
                   ` (15 preceding siblings ...)
  2025-12-15 17:31 ` ✗ CI.checkpatch: warning for drm/{i915, xe}: clean up and deduplicate initial plane handling Patchwork
@ 2025-12-15 17:32 ` Patchwork
  2025-12-15 17:48 ` ✗ CI.checksparse: warning " Patchwork
                   ` (3 subsequent siblings)
  20 siblings, 0 replies; 23+ messages in thread
From: Patchwork @ 2025-12-15 17:32 UTC (permalink / raw)
  To: Jani Nikula; +Cc: intel-xe

== Series Details ==

Series: drm/{i915, xe}: clean up and deduplicate initial plane handling
URL   : https://patchwork.freedesktop.org/series/159031/
State : success

== Summary ==

+ trap cleanup EXIT
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/xe/.kunitconfig
[17:31:38] Configuring KUnit Kernel ...
Generating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[17:31:42] Building KUnit Kernel ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
Building with:
$ make all compile_commands.json scripts_gdb ARCH=um O=.kunit --jobs=48
[17:32:14] Starting KUnit Kernel (1/1)...
[17:32:14] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[17:32:14] ================== guc_buf (11 subtests) ===================
[17:32:14] [PASSED] test_smallest
[17:32:14] [PASSED] test_largest
[17:32:14] [PASSED] test_granular
[17:32:14] [PASSED] test_unique
[17:32:14] [PASSED] test_overlap
[17:32:14] [PASSED] test_reusable
[17:32:14] [PASSED] test_too_big
[17:32:14] [PASSED] test_flush
[17:32:14] [PASSED] test_lookup
[17:32:14] [PASSED] test_data
[17:32:14] [PASSED] test_class
[17:32:14] ===================== [PASSED] guc_buf =====================
[17:32:14] =================== guc_dbm (7 subtests) ===================
[17:32:14] [PASSED] test_empty
[17:32:14] [PASSED] test_default
[17:32:14] ======================== test_size  ========================
[17:32:14] [PASSED] 4
[17:32:14] [PASSED] 8
[17:32:14] [PASSED] 32
[17:32:14] [PASSED] 256
[17:32:14] ==================== [PASSED] test_size ====================
[17:32:14] ======================= test_reuse  ========================
[17:32:14] [PASSED] 4
[17:32:14] [PASSED] 8
[17:32:14] [PASSED] 32
[17:32:14] [PASSED] 256
[17:32:14] =================== [PASSED] test_reuse ====================
[17:32:14] =================== test_range_overlap  ====================
[17:32:14] [PASSED] 4
[17:32:14] [PASSED] 8
[17:32:14] [PASSED] 32
[17:32:14] [PASSED] 256
[17:32:14] =============== [PASSED] test_range_overlap ================
[17:32:14] =================== test_range_compact  ====================
[17:32:14] [PASSED] 4
[17:32:14] [PASSED] 8
[17:32:14] [PASSED] 32
[17:32:14] [PASSED] 256
[17:32:14] =============== [PASSED] test_range_compact ================
[17:32:14] ==================== test_range_spare  =====================
[17:32:14] [PASSED] 4
[17:32:14] [PASSED] 8
[17:32:14] [PASSED] 32
[17:32:14] [PASSED] 256
[17:32:14] ================ [PASSED] test_range_spare =================
[17:32:14] ===================== [PASSED] guc_dbm =====================
[17:32:14] =================== guc_idm (6 subtests) ===================
[17:32:14] [PASSED] bad_init
[17:32:14] [PASSED] no_init
[17:32:14] [PASSED] init_fini
[17:32:14] [PASSED] check_used
[17:32:14] [PASSED] check_quota
[17:32:14] [PASSED] check_all
[17:32:14] ===================== [PASSED] guc_idm =====================
[17:32:14] ================== no_relay (3 subtests) ===================
[17:32:14] [PASSED] xe_drops_guc2pf_if_not_ready
[17:32:14] [PASSED] xe_drops_guc2vf_if_not_ready
[17:32:14] [PASSED] xe_rejects_send_if_not_ready
[17:32:14] ==================== [PASSED] no_relay =====================
[17:32:14] ================== pf_relay (14 subtests) ==================
[17:32:14] [PASSED] pf_rejects_guc2pf_too_short
[17:32:14] [PASSED] pf_rejects_guc2pf_too_long
[17:32:14] [PASSED] pf_rejects_guc2pf_no_payload
[17:32:14] [PASSED] pf_fails_no_payload
[17:32:14] [PASSED] pf_fails_bad_origin
[17:32:14] [PASSED] pf_fails_bad_type
[17:32:14] [PASSED] pf_txn_reports_error
[17:32:14] [PASSED] pf_txn_sends_pf2guc
[17:32:14] [PASSED] pf_sends_pf2guc
[17:32:14] [SKIPPED] pf_loopback_nop
[17:32:14] [SKIPPED] pf_loopback_echo
[17:32:14] [SKIPPED] pf_loopback_fail
[17:32:14] [SKIPPED] pf_loopback_busy
[17:32:14] [SKIPPED] pf_loopback_retry
[17:32:14] ==================== [PASSED] pf_relay =====================
[17:32:14] ================== vf_relay (3 subtests) ===================
[17:32:14] [PASSED] vf_rejects_guc2vf_too_short
[17:32:14] [PASSED] vf_rejects_guc2vf_too_long
[17:32:14] [PASSED] vf_rejects_guc2vf_no_payload
[17:32:14] ==================== [PASSED] vf_relay =====================
[17:32:14] ================ pf_gt_config (6 subtests) =================
[17:32:14] [PASSED] fair_contexts_1vf
[17:32:14] [PASSED] fair_doorbells_1vf
[17:32:14] [PASSED] fair_ggtt_1vf
[17:32:14] ====================== fair_contexts  ======================
[17:32:14] [PASSED] 1 VF
[17:32:14] [PASSED] 2 VFs
[17:32:14] [PASSED] 3 VFs
[17:32:14] [PASSED] 4 VFs
[17:32:14] [PASSED] 5 VFs
[17:32:14] [PASSED] 6 VFs
[17:32:14] [PASSED] 7 VFs
[17:32:14] [PASSED] 8 VFs
[17:32:14] [PASSED] 9 VFs
[17:32:14] [PASSED] 10 VFs
[17:32:14] [PASSED] 11 VFs
[17:32:14] [PASSED] 12 VFs
[17:32:14] [PASSED] 13 VFs
[17:32:14] [PASSED] 14 VFs
[17:32:14] [PASSED] 15 VFs
[17:32:14] [PASSED] 16 VFs
[17:32:14] [PASSED] 17 VFs
[17:32:14] [PASSED] 18 VFs
[17:32:14] [PASSED] 19 VFs
[17:32:14] [PASSED] 20 VFs
[17:32:14] [PASSED] 21 VFs
[17:32:14] [PASSED] 22 VFs
[17:32:14] [PASSED] 23 VFs
[17:32:14] [PASSED] 24 VFs
[17:32:14] [PASSED] 25 VFs
[17:32:14] [PASSED] 26 VFs
[17:32:14] [PASSED] 27 VFs
[17:32:14] [PASSED] 28 VFs
[17:32:14] [PASSED] 29 VFs
[17:32:14] [PASSED] 30 VFs
[17:32:14] [PASSED] 31 VFs
[17:32:14] [PASSED] 32 VFs
[17:32:14] [PASSED] 33 VFs
[17:32:14] [PASSED] 34 VFs
[17:32:14] [PASSED] 35 VFs
[17:32:14] [PASSED] 36 VFs
[17:32:14] [PASSED] 37 VFs
[17:32:14] [PASSED] 38 VFs
[17:32:14] [PASSED] 39 VFs
[17:32:14] [PASSED] 40 VFs
[17:32:14] [PASSED] 41 VFs
[17:32:14] [PASSED] 42 VFs
[17:32:14] [PASSED] 43 VFs
[17:32:14] [PASSED] 44 VFs
[17:32:14] [PASSED] 45 VFs
[17:32:14] [PASSED] 46 VFs
[17:32:14] [PASSED] 47 VFs
[17:32:14] [PASSED] 48 VFs
[17:32:14] [PASSED] 49 VFs
[17:32:14] [PASSED] 50 VFs
[17:32:14] [PASSED] 51 VFs
[17:32:14] [PASSED] 52 VFs
[17:32:14] [PASSED] 53 VFs
[17:32:14] [PASSED] 54 VFs
[17:32:14] [PASSED] 55 VFs
[17:32:14] [PASSED] 56 VFs
[17:32:14] [PASSED] 57 VFs
[17:32:14] [PASSED] 58 VFs
[17:32:14] [PASSED] 59 VFs
[17:32:14] [PASSED] 60 VFs
[17:32:14] [PASSED] 61 VFs
[17:32:14] [PASSED] 62 VFs
[17:32:14] [PASSED] 63 VFs
[17:32:14] ================== [PASSED] fair_contexts ==================
[17:32:14] ===================== fair_doorbells  ======================
[17:32:14] [PASSED] 1 VF
[17:32:14] [PASSED] 2 VFs
[17:32:14] [PASSED] 3 VFs
[17:32:14] [PASSED] 4 VFs
[17:32:14] [PASSED] 5 VFs
[17:32:14] [PASSED] 6 VFs
[17:32:14] [PASSED] 7 VFs
[17:32:14] [PASSED] 8 VFs
[17:32:14] [PASSED] 9 VFs
[17:32:14] [PASSED] 10 VFs
[17:32:14] [PASSED] 11 VFs
[17:32:14] [PASSED] 12 VFs
[17:32:14] [PASSED] 13 VFs
[17:32:14] [PASSED] 14 VFs
[17:32:14] [PASSED] 15 VFs
[17:32:14] [PASSED] 16 VFs
[17:32:14] [PASSED] 17 VFs
[17:32:14] [PASSED] 18 VFs
[17:32:14] [PASSED] 19 VFs
[17:32:14] [PASSED] 20 VFs
[17:32:14] [PASSED] 21 VFs
[17:32:14] [PASSED] 22 VFs
[17:32:14] [PASSED] 23 VFs
[17:32:14] [PASSED] 24 VFs
[17:32:14] [PASSED] 25 VFs
[17:32:14] [PASSED] 26 VFs
[17:32:14] [PASSED] 27 VFs
[17:32:14] [PASSED] 28 VFs
[17:32:14] [PASSED] 29 VFs
[17:32:14] [PASSED] 30 VFs
[17:32:14] [PASSED] 31 VFs
[17:32:14] [PASSED] 32 VFs
[17:32:14] [PASSED] 33 VFs
[17:32:14] [PASSED] 34 VFs
[17:32:14] [PASSED] 35 VFs
[17:32:14] [PASSED] 36 VFs
[17:32:14] [PASSED] 37 VFs
[17:32:14] [PASSED] 38 VFs
[17:32:14] [PASSED] 39 VFs
[17:32:14] [PASSED] 40 VFs
[17:32:14] [PASSED] 41 VFs
[17:32:14] [PASSED] 42 VFs
[17:32:14] [PASSED] 43 VFs
[17:32:14] [PASSED] 44 VFs
[17:32:14] [PASSED] 45 VFs
[17:32:14] [PASSED] 46 VFs
[17:32:14] [PASSED] 47 VFs
[17:32:14] [PASSED] 48 VFs
[17:32:14] [PASSED] 49 VFs
[17:32:14] [PASSED] 50 VFs
[17:32:14] [PASSED] 51 VFs
[17:32:14] [PASSED] 52 VFs
[17:32:14] [PASSED] 53 VFs
[17:32:14] [PASSED] 54 VFs
[17:32:14] [PASSED] 55 VFs
[17:32:14] [PASSED] 56 VFs
[17:32:14] [PASSED] 57 VFs
[17:32:14] [PASSED] 58 VFs
[17:32:14] [PASSED] 59 VFs
[17:32:14] [PASSED] 60 VFs
[17:32:14] [PASSED] 61 VFs
[17:32:14] [PASSED] 62 VFs
[17:32:14] [PASSED] 63 VFs
[17:32:14] ================= [PASSED] fair_doorbells ==================
[17:32:14] ======================== fair_ggtt  ========================
[17:32:14] [PASSED] 1 VF
[17:32:14] [PASSED] 2 VFs
[17:32:14] [PASSED] 3 VFs
[17:32:14] [PASSED] 4 VFs
[17:32:14] [PASSED] 5 VFs
[17:32:14] [PASSED] 6 VFs
[17:32:14] [PASSED] 7 VFs
[17:32:14] [PASSED] 8 VFs
[17:32:14] [PASSED] 9 VFs
[17:32:14] [PASSED] 10 VFs
[17:32:14] [PASSED] 11 VFs
[17:32:14] [PASSED] 12 VFs
[17:32:14] [PASSED] 13 VFs
[17:32:14] [PASSED] 14 VFs
[17:32:14] [PASSED] 15 VFs
[17:32:14] [PASSED] 16 VFs
[17:32:14] [PASSED] 17 VFs
[17:32:14] [PASSED] 18 VFs
[17:32:14] [PASSED] 19 VFs
[17:32:14] [PASSED] 20 VFs
[17:32:14] [PASSED] 21 VFs
[17:32:14] [PASSED] 22 VFs
[17:32:14] [PASSED] 23 VFs
[17:32:14] [PASSED] 24 VFs
[17:32:14] [PASSED] 25 VFs
[17:32:14] [PASSED] 26 VFs
[17:32:14] [PASSED] 27 VFs
[17:32:14] [PASSED] 28 VFs
[17:32:14] [PASSED] 29 VFs
[17:32:14] [PASSED] 30 VFs
[17:32:14] [PASSED] 31 VFs
[17:32:14] [PASSED] 32 VFs
[17:32:14] [PASSED] 33 VFs
[17:32:14] [PASSED] 34 VFs
[17:32:14] [PASSED] 35 VFs
[17:32:14] [PASSED] 36 VFs
[17:32:14] [PASSED] 37 VFs
[17:32:14] [PASSED] 38 VFs
[17:32:14] [PASSED] 39 VFs
[17:32:14] [PASSED] 40 VFs
[17:32:14] [PASSED] 41 VFs
[17:32:14] [PASSED] 42 VFs
[17:32:14] [PASSED] 43 VFs
[17:32:14] [PASSED] 44 VFs
[17:32:14] [PASSED] 45 VFs
[17:32:14] [PASSED] 46 VFs
[17:32:14] [PASSED] 47 VFs
[17:32:14] [PASSED] 48 VFs
[17:32:14] [PASSED] 49 VFs
[17:32:14] [PASSED] 50 VFs
[17:32:14] [PASSED] 51 VFs
[17:32:14] [PASSED] 52 VFs
[17:32:14] [PASSED] 53 VFs
[17:32:14] [PASSED] 54 VFs
[17:32:14] [PASSED] 55 VFs
[17:32:14] [PASSED] 56 VFs
[17:32:14] [PASSED] 57 VFs
[17:32:14] [PASSED] 58 VFs
[17:32:14] [PASSED] 59 VFs
[17:32:14] [PASSED] 60 VFs
[17:32:14] [PASSED] 61 VFs
[17:32:14] [PASSED] 62 VFs
[17:32:14] [PASSED] 63 VFs
[17:32:14] ==================== [PASSED] fair_ggtt ====================
[17:32:14] ================== [PASSED] pf_gt_config ===================
[17:32:14] ===================== lmtt (1 subtest) =====================
[17:32:14] ======================== test_ops  =========================
[17:32:14] [PASSED] 2-level
[17:32:14] [PASSED] multi-level
[17:32:14] ==================== [PASSED] test_ops =====================
[17:32:14] ====================== [PASSED] lmtt =======================
[17:32:14] ================= pf_service (11 subtests) =================
[17:32:14] [PASSED] pf_negotiate_any
[17:32:14] [PASSED] pf_negotiate_base_match
[17:32:14] [PASSED] pf_negotiate_base_newer
[17:32:14] [PASSED] pf_negotiate_base_next
[17:32:14] [SKIPPED] pf_negotiate_base_older
[17:32:14] [PASSED] pf_negotiate_base_prev
[17:32:14] [PASSED] pf_negotiate_latest_match
[17:32:14] [PASSED] pf_negotiate_latest_newer
[17:32:14] [PASSED] pf_negotiate_latest_next
[17:32:14] [SKIPPED] pf_negotiate_latest_older
[17:32:14] [SKIPPED] pf_negotiate_latest_prev
[17:32:14] =================== [PASSED] pf_service ====================
[17:32:14] ================= xe_guc_g2g (2 subtests) ==================
[17:32:14] ============== xe_live_guc_g2g_kunit_default  ==============
[17:32:14] ========= [SKIPPED] xe_live_guc_g2g_kunit_default ==========
[17:32:14] ============== xe_live_guc_g2g_kunit_allmem  ===============
[17:32:14] ========== [SKIPPED] xe_live_guc_g2g_kunit_allmem ==========
[17:32:14] =================== [SKIPPED] xe_guc_g2g ===================
[17:32:14] =================== xe_mocs (2 subtests) ===================
[17:32:14] ================ xe_live_mocs_kernel_kunit  ================
[17:32:14] =========== [SKIPPED] xe_live_mocs_kernel_kunit ============
[17:32:14] ================ xe_live_mocs_reset_kunit  =================
[17:32:14] ============ [SKIPPED] xe_live_mocs_reset_kunit ============
[17:32:14] ==================== [SKIPPED] xe_mocs =====================
[17:32:14] ================= xe_migrate (2 subtests) ==================
[17:32:14] ================= xe_migrate_sanity_kunit  =================
[17:32:14] ============ [SKIPPED] xe_migrate_sanity_kunit =============
[17:32:14] ================== xe_validate_ccs_kunit  ==================
[17:32:14] ============= [SKIPPED] xe_validate_ccs_kunit ==============
[17:32:14] =================== [SKIPPED] xe_migrate ===================
[17:32:14] ================== xe_dma_buf (1 subtest) ==================
[17:32:14] ==================== xe_dma_buf_kunit  =====================
[17:32:14] ================ [SKIPPED] xe_dma_buf_kunit ================
[17:32:14] =================== [SKIPPED] xe_dma_buf ===================
[17:32:14] ================= xe_bo_shrink (1 subtest) =================
[17:32:14] =================== xe_bo_shrink_kunit  ====================
[17:32:14] =============== [SKIPPED] xe_bo_shrink_kunit ===============
[17:32:14] ================== [SKIPPED] xe_bo_shrink ==================
[17:32:14] ==================== xe_bo (2 subtests) ====================
[17:32:14] ================== xe_ccs_migrate_kunit  ===================
[17:32:14] ============== [SKIPPED] xe_ccs_migrate_kunit ==============
[17:32:14] ==================== xe_bo_evict_kunit  ====================
[17:32:14] =============== [SKIPPED] xe_bo_evict_kunit ================
[17:32:14] ===================== [SKIPPED] xe_bo ======================
[17:32:14] ==================== args (11 subtests) ====================
[17:32:14] [PASSED] count_args_test
[17:32:14] [PASSED] call_args_example
[17:32:14] [PASSED] call_args_test
[17:32:14] [PASSED] drop_first_arg_example
[17:32:14] [PASSED] drop_first_arg_test
[17:32:14] [PASSED] first_arg_example
[17:32:14] [PASSED] first_arg_test
[17:32:14] [PASSED] last_arg_example
[17:32:14] [PASSED] last_arg_test
[17:32:14] [PASSED] pick_arg_example
[17:32:14] [PASSED] sep_comma_example
[17:32:14] ====================== [PASSED] args =======================
[17:32:14] =================== xe_pci (3 subtests) ====================
[17:32:14] ==================== check_graphics_ip  ====================
[17:32:14] [PASSED] 12.00 Xe_LP
[17:32:14] [PASSED] 12.10 Xe_LP+
[17:32:14] [PASSED] 12.55 Xe_HPG
[17:32:14] [PASSED] 12.60 Xe_HPC
[17:32:14] [PASSED] 12.70 Xe_LPG
[17:32:14] [PASSED] 12.71 Xe_LPG
[17:32:14] [PASSED] 12.74 Xe_LPG+
[17:32:14] [PASSED] 20.01 Xe2_HPG
[17:32:14] [PASSED] 20.02 Xe2_HPG
[17:32:14] [PASSED] 20.04 Xe2_LPG
[17:32:14] [PASSED] 30.00 Xe3_LPG
[17:32:14] [PASSED] 30.01 Xe3_LPG
[17:32:14] [PASSED] 30.03 Xe3_LPG
[17:32:14] [PASSED] 30.04 Xe3_LPG
[17:32:14] [PASSED] 30.05 Xe3_LPG
[17:32:14] [PASSED] 35.11 Xe3p_XPC
[17:32:14] ================ [PASSED] check_graphics_ip ================
[17:32:14] ===================== check_media_ip  ======================
[17:32:14] [PASSED] 12.00 Xe_M
[17:32:14] [PASSED] 12.55 Xe_HPM
[17:32:14] [PASSED] 13.00 Xe_LPM+
[17:32:14] [PASSED] 13.01 Xe2_HPM
[17:32:14] [PASSED] 20.00 Xe2_LPM
[17:32:14] [PASSED] 30.00 Xe3_LPM
[17:32:14] [PASSED] 30.02 Xe3_LPM
[17:32:14] [PASSED] 35.00 Xe3p_LPM
[17:32:14] [PASSED] 35.03 Xe3p_HPM
[17:32:14] ================= [PASSED] check_media_ip ==================
[17:32:14] =================== check_platform_desc  ===================
[17:32:14] [PASSED] 0x9A60 (TIGERLAKE)
[17:32:14] [PASSED] 0x9A68 (TIGERLAKE)
[17:32:14] [PASSED] 0x9A70 (TIGERLAKE)
[17:32:14] [PASSED] 0x9A40 (TIGERLAKE)
[17:32:14] [PASSED] 0x9A49 (TIGERLAKE)
[17:32:14] [PASSED] 0x9A59 (TIGERLAKE)
[17:32:14] [PASSED] 0x9A78 (TIGERLAKE)
[17:32:14] [PASSED] 0x9AC0 (TIGERLAKE)
[17:32:14] [PASSED] 0x9AC9 (TIGERLAKE)
[17:32:14] [PASSED] 0x9AD9 (TIGERLAKE)
[17:32:14] [PASSED] 0x9AF8 (TIGERLAKE)
[17:32:14] [PASSED] 0x4C80 (ROCKETLAKE)
[17:32:14] [PASSED] 0x4C8A (ROCKETLAKE)
[17:32:14] [PASSED] 0x4C8B (ROCKETLAKE)
[17:32:14] [PASSED] 0x4C8C (ROCKETLAKE)
[17:32:14] [PASSED] 0x4C90 (ROCKETLAKE)
[17:32:14] [PASSED] 0x4C9A (ROCKETLAKE)
[17:32:14] [PASSED] 0x4680 (ALDERLAKE_S)
[17:32:14] [PASSED] 0x4682 (ALDERLAKE_S)
[17:32:14] [PASSED] 0x4688 (ALDERLAKE_S)
[17:32:14] [PASSED] 0x468A (ALDERLAKE_S)
[17:32:14] [PASSED] 0x468B (ALDERLAKE_S)
[17:32:14] [PASSED] 0x4690 (ALDERLAKE_S)
[17:32:14] [PASSED] 0x4692 (ALDERLAKE_S)
[17:32:14] [PASSED] 0x4693 (ALDERLAKE_S)
[17:32:14] [PASSED] 0x46A0 (ALDERLAKE_P)
[17:32:14] [PASSED] 0x46A1 (ALDERLAKE_P)
[17:32:14] [PASSED] 0x46A2 (ALDERLAKE_P)
[17:32:14] [PASSED] 0x46A3 (ALDERLAKE_P)
[17:32:14] [PASSED] 0x46A6 (ALDERLAKE_P)
[17:32:14] [PASSED] 0x46A8 (ALDERLAKE_P)
[17:32:14] [PASSED] 0x46AA (ALDERLAKE_P)
[17:32:14] [PASSED] 0x462A (ALDERLAKE_P)
[17:32:14] [PASSED] 0x4626 (ALDERLAKE_P)
[17:32:14] [PASSED] 0x4628 (ALDERLAKE_P)
[17:32:14] [PASSED] 0x46B0 (ALDERLAKE_P)
stty: 'standard input': Inappropriate ioctl for device
[17:32:14] [PASSED] 0x46B1 (ALDERLAKE_P)
[17:32:14] [PASSED] 0x46B2 (ALDERLAKE_P)
[17:32:14] [PASSED] 0x46B3 (ALDERLAKE_P)
[17:32:14] [PASSED] 0x46C0 (ALDERLAKE_P)
[17:32:14] [PASSED] 0x46C1 (ALDERLAKE_P)
[17:32:14] [PASSED] 0x46C2 (ALDERLAKE_P)
[17:32:14] [PASSED] 0x46C3 (ALDERLAKE_P)
[17:32:14] [PASSED] 0x46D0 (ALDERLAKE_N)
[17:32:14] [PASSED] 0x46D1 (ALDERLAKE_N)
[17:32:14] [PASSED] 0x46D2 (ALDERLAKE_N)
[17:32:14] [PASSED] 0x46D3 (ALDERLAKE_N)
[17:32:14] [PASSED] 0x46D4 (ALDERLAKE_N)
[17:32:14] [PASSED] 0xA721 (ALDERLAKE_P)
[17:32:14] [PASSED] 0xA7A1 (ALDERLAKE_P)
[17:32:14] [PASSED] 0xA7A9 (ALDERLAKE_P)
[17:32:14] [PASSED] 0xA7AC (ALDERLAKE_P)
[17:32:14] [PASSED] 0xA7AD (ALDERLAKE_P)
[17:32:14] [PASSED] 0xA720 (ALDERLAKE_P)
[17:32:14] [PASSED] 0xA7A0 (ALDERLAKE_P)
[17:32:14] [PASSED] 0xA7A8 (ALDERLAKE_P)
[17:32:14] [PASSED] 0xA7AA (ALDERLAKE_P)
[17:32:14] [PASSED] 0xA7AB (ALDERLAKE_P)
[17:32:14] [PASSED] 0xA780 (ALDERLAKE_S)
[17:32:14] [PASSED] 0xA781 (ALDERLAKE_S)
[17:32:14] [PASSED] 0xA782 (ALDERLAKE_S)
[17:32:14] [PASSED] 0xA783 (ALDERLAKE_S)
[17:32:14] [PASSED] 0xA788 (ALDERLAKE_S)
[17:32:14] [PASSED] 0xA789 (ALDERLAKE_S)
[17:32:14] [PASSED] 0xA78A (ALDERLAKE_S)
[17:32:14] [PASSED] 0xA78B (ALDERLAKE_S)
[17:32:14] [PASSED] 0x4905 (DG1)
[17:32:14] [PASSED] 0x4906 (DG1)
[17:32:14] [PASSED] 0x4907 (DG1)
[17:32:14] [PASSED] 0x4908 (DG1)
[17:32:14] [PASSED] 0x4909 (DG1)
[17:32:14] [PASSED] 0x56C0 (DG2)
[17:32:14] [PASSED] 0x56C2 (DG2)
[17:32:14] [PASSED] 0x56C1 (DG2)
[17:32:14] [PASSED] 0x7D51 (METEORLAKE)
[17:32:14] [PASSED] 0x7DD1 (METEORLAKE)
[17:32:14] [PASSED] 0x7D41 (METEORLAKE)
[17:32:14] [PASSED] 0x7D67 (METEORLAKE)
[17:32:14] [PASSED] 0xB640 (METEORLAKE)
[17:32:14] [PASSED] 0x56A0 (DG2)
[17:32:14] [PASSED] 0x56A1 (DG2)
[17:32:14] [PASSED] 0x56A2 (DG2)
[17:32:14] [PASSED] 0x56BE (DG2)
[17:32:14] [PASSED] 0x56BF (DG2)
[17:32:14] [PASSED] 0x5690 (DG2)
[17:32:14] [PASSED] 0x5691 (DG2)
[17:32:14] [PASSED] 0x5692 (DG2)
[17:32:14] [PASSED] 0x56A5 (DG2)
[17:32:14] [PASSED] 0x56A6 (DG2)
[17:32:14] [PASSED] 0x56B0 (DG2)
[17:32:14] [PASSED] 0x56B1 (DG2)
[17:32:14] [PASSED] 0x56BA (DG2)
[17:32:14] [PASSED] 0x56BB (DG2)
[17:32:14] [PASSED] 0x56BC (DG2)
[17:32:14] [PASSED] 0x56BD (DG2)
[17:32:14] [PASSED] 0x5693 (DG2)
[17:32:14] [PASSED] 0x5694 (DG2)
[17:32:14] [PASSED] 0x5695 (DG2)
[17:32:14] [PASSED] 0x56A3 (DG2)
[17:32:14] [PASSED] 0x56A4 (DG2)
[17:32:14] [PASSED] 0x56B2 (DG2)
[17:32:14] [PASSED] 0x56B3 (DG2)
[17:32:14] [PASSED] 0x5696 (DG2)
[17:32:14] [PASSED] 0x5697 (DG2)
[17:32:14] [PASSED] 0xB69 (PVC)
[17:32:14] [PASSED] 0xB6E (PVC)
[17:32:14] [PASSED] 0xBD4 (PVC)
[17:32:14] [PASSED] 0xBD5 (PVC)
[17:32:14] [PASSED] 0xBD6 (PVC)
[17:32:14] [PASSED] 0xBD7 (PVC)
[17:32:14] [PASSED] 0xBD8 (PVC)
[17:32:14] [PASSED] 0xBD9 (PVC)
[17:32:14] [PASSED] 0xBDA (PVC)
[17:32:14] [PASSED] 0xBDB (PVC)
[17:32:14] [PASSED] 0xBE0 (PVC)
[17:32:14] [PASSED] 0xBE1 (PVC)
[17:32:14] [PASSED] 0xBE5 (PVC)
[17:32:14] [PASSED] 0x7D40 (METEORLAKE)
[17:32:14] [PASSED] 0x7D45 (METEORLAKE)
[17:32:14] [PASSED] 0x7D55 (METEORLAKE)
[17:32:14] [PASSED] 0x7D60 (METEORLAKE)
[17:32:14] [PASSED] 0x7DD5 (METEORLAKE)
[17:32:14] [PASSED] 0x6420 (LUNARLAKE)
[17:32:14] [PASSED] 0x64A0 (LUNARLAKE)
[17:32:14] [PASSED] 0x64B0 (LUNARLAKE)
[17:32:14] [PASSED] 0xE202 (BATTLEMAGE)
[17:32:14] [PASSED] 0xE209 (BATTLEMAGE)
[17:32:14] [PASSED] 0xE20B (BATTLEMAGE)
[17:32:14] [PASSED] 0xE20C (BATTLEMAGE)
[17:32:14] [PASSED] 0xE20D (BATTLEMAGE)
[17:32:14] [PASSED] 0xE210 (BATTLEMAGE)
[17:32:14] [PASSED] 0xE211 (BATTLEMAGE)
[17:32:14] [PASSED] 0xE212 (BATTLEMAGE)
[17:32:14] [PASSED] 0xE216 (BATTLEMAGE)
[17:32:14] [PASSED] 0xE220 (BATTLEMAGE)
[17:32:14] [PASSED] 0xE221 (BATTLEMAGE)
[17:32:14] [PASSED] 0xE222 (BATTLEMAGE)
[17:32:14] [PASSED] 0xE223 (BATTLEMAGE)
[17:32:14] [PASSED] 0xB080 (PANTHERLAKE)
[17:32:14] [PASSED] 0xB081 (PANTHERLAKE)
[17:32:14] [PASSED] 0xB082 (PANTHERLAKE)
[17:32:14] [PASSED] 0xB083 (PANTHERLAKE)
[17:32:14] [PASSED] 0xB084 (PANTHERLAKE)
[17:32:14] [PASSED] 0xB085 (PANTHERLAKE)
[17:32:14] [PASSED] 0xB086 (PANTHERLAKE)
[17:32:14] [PASSED] 0xB087 (PANTHERLAKE)
[17:32:14] [PASSED] 0xB08F (PANTHERLAKE)
[17:32:14] [PASSED] 0xB090 (PANTHERLAKE)
[17:32:14] [PASSED] 0xB0A0 (PANTHERLAKE)
[17:32:14] [PASSED] 0xB0B0 (PANTHERLAKE)
[17:32:14] [PASSED] 0xFD80 (PANTHERLAKE)
[17:32:14] [PASSED] 0xFD81 (PANTHERLAKE)
[17:32:14] [PASSED] 0xD740 (NOVALAKE_S)
[17:32:14] [PASSED] 0xD741 (NOVALAKE_S)
[17:32:14] [PASSED] 0xD742 (NOVALAKE_S)
[17:32:14] [PASSED] 0xD743 (NOVALAKE_S)
[17:32:14] [PASSED] 0xD744 (NOVALAKE_S)
[17:32:14] [PASSED] 0xD745 (NOVALAKE_S)
[17:32:14] [PASSED] 0x674C (CRESCENTISLAND)
[17:32:14] =============== [PASSED] check_platform_desc ===============
[17:32:14] ===================== [PASSED] xe_pci ======================
[17:32:14] =================== xe_rtp (2 subtests) ====================
[17:32:14] =============== xe_rtp_process_to_sr_tests  ================
[17:32:14] [PASSED] coalesce-same-reg
[17:32:14] [PASSED] no-match-no-add
[17:32:14] [PASSED] match-or
[17:32:14] [PASSED] match-or-xfail
[17:32:14] [PASSED] no-match-no-add-multiple-rules
[17:32:14] [PASSED] two-regs-two-entries
[17:32:14] [PASSED] clr-one-set-other
[17:32:14] [PASSED] set-field
[17:32:14] [PASSED] conflict-duplicate
[17:32:14] [PASSED] conflict-not-disjoint
[17:32:14] [PASSED] conflict-reg-type
[17:32:14] =========== [PASSED] xe_rtp_process_to_sr_tests ============
[17:32:14] ================== xe_rtp_process_tests  ===================
[17:32:14] [PASSED] active1
[17:32:14] [PASSED] active2
[17:32:14] [PASSED] active-inactive
[17:32:14] [PASSED] inactive-active
[17:32:14] [PASSED] inactive-1st_or_active-inactive
[17:32:14] [PASSED] inactive-2nd_or_active-inactive
[17:32:14] [PASSED] inactive-last_or_active-inactive
[17:32:14] [PASSED] inactive-no_or_active-inactive
[17:32:14] ============== [PASSED] xe_rtp_process_tests ===============
[17:32:14] ===================== [PASSED] xe_rtp ======================
[17:32:14] ==================== xe_wa (1 subtest) =====================
[17:32:14] ======================== xe_wa_gt  =========================
[17:32:14] [PASSED] TIGERLAKE B0
[17:32:14] [PASSED] DG1 A0
[17:32:14] [PASSED] DG1 B0
[17:32:14] [PASSED] ALDERLAKE_S A0
[17:32:14] [PASSED] ALDERLAKE_S B0
[17:32:14] [PASSED] ALDERLAKE_S C0
[17:32:14] [PASSED] ALDERLAKE_S D0
[17:32:14] [PASSED] ALDERLAKE_P A0
[17:32:14] [PASSED] ALDERLAKE_P B0
[17:32:14] [PASSED] ALDERLAKE_P C0
[17:32:14] [PASSED] ALDERLAKE_S RPLS D0
[17:32:14] [PASSED] ALDERLAKE_P RPLU E0
[17:32:14] [PASSED] DG2 G10 C0
[17:32:14] [PASSED] DG2 G11 B1
[17:32:14] [PASSED] DG2 G12 A1
[17:32:14] [PASSED] METEORLAKE 12.70(Xe_LPG) A0 13.00(Xe_LPM+) A0
[17:32:14] [PASSED] METEORLAKE 12.71(Xe_LPG) A0 13.00(Xe_LPM+) A0
[17:32:14] [PASSED] METEORLAKE 12.74(Xe_LPG+) A0 13.00(Xe_LPM+) A0
[17:32:14] [PASSED] LUNARLAKE 20.04(Xe2_LPG) A0 20.00(Xe2_LPM) A0
[17:32:14] [PASSED] LUNARLAKE 20.04(Xe2_LPG) B0 20.00(Xe2_LPM) A0
[17:32:14] [PASSED] BATTLEMAGE 20.01(Xe2_HPG) A0 13.01(Xe2_HPM) A1
[17:32:14] [PASSED] PANTHERLAKE 30.00(Xe3_LPG) A0 30.00(Xe3_LPM) A0
[17:32:14] ==================== [PASSED] xe_wa_gt =====================
[17:32:14] ====================== [PASSED] xe_wa ======================
[17:32:14] ============================================================
[17:32:14] Testing complete. Ran 510 tests: passed: 492, skipped: 18
[17:32:14] Elapsed time: 36.202s total, 4.252s configuring, 31.434s building, 0.466s running

+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/tests/.kunitconfig
[17:32:14] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[17:32:16] Building KUnit Kernel ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
Building with:
$ make all compile_commands.json scripts_gdb ARCH=um O=.kunit --jobs=48
[17:32:42] Starting KUnit Kernel (1/1)...
[17:32:42] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[17:32:42] ============ drm_test_pick_cmdline (2 subtests) ============
[17:32:42] [PASSED] drm_test_pick_cmdline_res_1920_1080_60
[17:32:42] =============== drm_test_pick_cmdline_named  ===============
[17:32:42] [PASSED] NTSC
[17:32:42] [PASSED] NTSC-J
[17:32:42] [PASSED] PAL
[17:32:42] [PASSED] PAL-M
[17:32:42] =========== [PASSED] drm_test_pick_cmdline_named ===========
[17:32:42] ============== [PASSED] drm_test_pick_cmdline ==============
[17:32:42] == drm_test_atomic_get_connector_for_encoder (1 subtest) ===
[17:32:42] [PASSED] drm_test_drm_atomic_get_connector_for_encoder
[17:32:42] ==== [PASSED] drm_test_atomic_get_connector_for_encoder ====
[17:32:42] =========== drm_validate_clone_mode (2 subtests) ===========
[17:32:42] ============== drm_test_check_in_clone_mode  ===============
[17:32:42] [PASSED] in_clone_mode
[17:32:42] [PASSED] not_in_clone_mode
[17:32:42] ========== [PASSED] drm_test_check_in_clone_mode ===========
[17:32:42] =============== drm_test_check_valid_clones  ===============
[17:32:42] [PASSED] not_in_clone_mode
[17:32:42] [PASSED] valid_clone
[17:32:42] [PASSED] invalid_clone
[17:32:42] =========== [PASSED] drm_test_check_valid_clones ===========
[17:32:42] ============= [PASSED] drm_validate_clone_mode =============
[17:32:42] ============= drm_validate_modeset (1 subtest) =============
[17:32:42] [PASSED] drm_test_check_connector_changed_modeset
[17:32:42] ============== [PASSED] drm_validate_modeset ===============
[17:32:42] ====== drm_test_bridge_get_current_state (2 subtests) ======
[17:32:42] [PASSED] drm_test_drm_bridge_get_current_state_atomic
[17:32:42] [PASSED] drm_test_drm_bridge_get_current_state_legacy
[17:32:42] ======== [PASSED] drm_test_bridge_get_current_state ========
[17:32:42] ====== drm_test_bridge_helper_reset_crtc (3 subtests) ======
[17:32:42] [PASSED] drm_test_drm_bridge_helper_reset_crtc_atomic
[17:32:42] [PASSED] drm_test_drm_bridge_helper_reset_crtc_atomic_disabled
[17:32:42] [PASSED] drm_test_drm_bridge_helper_reset_crtc_legacy
[17:32:42] ======== [PASSED] drm_test_bridge_helper_reset_crtc ========
[17:32:42] ============== drm_bridge_alloc (2 subtests) ===============
[17:32:42] [PASSED] drm_test_drm_bridge_alloc_basic
[17:32:42] [PASSED] drm_test_drm_bridge_alloc_get_put
[17:32:42] ================ [PASSED] drm_bridge_alloc =================
[17:32:42] ================== drm_buddy (8 subtests) ==================
[17:32:42] [PASSED] drm_test_buddy_alloc_limit
[17:32:42] [PASSED] drm_test_buddy_alloc_optimistic
[17:32:42] [PASSED] drm_test_buddy_alloc_pessimistic
[17:32:42] [PASSED] drm_test_buddy_alloc_pathological
[17:32:42] [PASSED] drm_test_buddy_alloc_contiguous
[17:32:42] [PASSED] drm_test_buddy_alloc_clear
[17:32:42] [PASSED] drm_test_buddy_alloc_range_bias
[17:32:42] [PASSED] drm_test_buddy_fragmentation_performance
[17:32:42] ==================== [PASSED] drm_buddy ====================
[17:32:42] ============= drm_cmdline_parser (40 subtests) =============
[17:32:42] [PASSED] drm_test_cmdline_force_d_only
[17:32:42] [PASSED] drm_test_cmdline_force_D_only_dvi
[17:32:42] [PASSED] drm_test_cmdline_force_D_only_hdmi
[17:32:42] [PASSED] drm_test_cmdline_force_D_only_not_digital
[17:32:42] [PASSED] drm_test_cmdline_force_e_only
[17:32:42] [PASSED] drm_test_cmdline_res
[17:32:42] [PASSED] drm_test_cmdline_res_vesa
[17:32:42] [PASSED] drm_test_cmdline_res_vesa_rblank
[17:32:42] [PASSED] drm_test_cmdline_res_rblank
[17:32:42] [PASSED] drm_test_cmdline_res_bpp
[17:32:42] [PASSED] drm_test_cmdline_res_refresh
[17:32:42] [PASSED] drm_test_cmdline_res_bpp_refresh
[17:32:42] [PASSED] drm_test_cmdline_res_bpp_refresh_interlaced
[17:32:42] [PASSED] drm_test_cmdline_res_bpp_refresh_margins
[17:32:42] [PASSED] drm_test_cmdline_res_bpp_refresh_force_off
[17:32:42] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on
[17:32:42] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on_analog
[17:32:42] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on_digital
[17:32:42] [PASSED] drm_test_cmdline_res_bpp_refresh_interlaced_margins_force_on
[17:32:42] [PASSED] drm_test_cmdline_res_margins_force_on
[17:32:42] [PASSED] drm_test_cmdline_res_vesa_margins
[17:32:42] [PASSED] drm_test_cmdline_name
[17:32:42] [PASSED] drm_test_cmdline_name_bpp
[17:32:42] [PASSED] drm_test_cmdline_name_option
[17:32:42] [PASSED] drm_test_cmdline_name_bpp_option
[17:32:42] [PASSED] drm_test_cmdline_rotate_0
[17:32:42] [PASSED] drm_test_cmdline_rotate_90
[17:32:42] [PASSED] drm_test_cmdline_rotate_180
[17:32:42] [PASSED] drm_test_cmdline_rotate_270
[17:32:42] [PASSED] drm_test_cmdline_hmirror
[17:32:42] [PASSED] drm_test_cmdline_vmirror
[17:32:42] [PASSED] drm_test_cmdline_margin_options
[17:32:42] [PASSED] drm_test_cmdline_multiple_options
[17:32:42] [PASSED] drm_test_cmdline_bpp_extra_and_option
[17:32:42] [PASSED] drm_test_cmdline_extra_and_option
[17:32:42] [PASSED] drm_test_cmdline_freestanding_options
[17:32:42] [PASSED] drm_test_cmdline_freestanding_force_e_and_options
[17:32:42] [PASSED] drm_test_cmdline_panel_orientation
[17:32:42] ================ drm_test_cmdline_invalid  =================
[17:32:42] [PASSED] margin_only
[17:32:42] [PASSED] interlace_only
[17:32:42] [PASSED] res_missing_x
[17:32:42] [PASSED] res_missing_y
[17:32:42] [PASSED] res_bad_y
[17:32:42] [PASSED] res_missing_y_bpp
[17:32:42] [PASSED] res_bad_bpp
[17:32:42] [PASSED] res_bad_refresh
[17:32:42] [PASSED] res_bpp_refresh_force_on_off
[17:32:42] [PASSED] res_invalid_mode
[17:32:42] [PASSED] res_bpp_wrong_place_mode
[17:32:42] [PASSED] name_bpp_refresh
[17:32:42] [PASSED] name_refresh
[17:32:42] [PASSED] name_refresh_wrong_mode
[17:32:42] [PASSED] name_refresh_invalid_mode
[17:32:42] [PASSED] rotate_multiple
[17:32:42] [PASSED] rotate_invalid_val
[17:32:42] [PASSED] rotate_truncated
[17:32:42] [PASSED] invalid_option
[17:32:42] [PASSED] invalid_tv_option
[17:32:42] [PASSED] truncated_tv_option
[17:32:42] ============ [PASSED] drm_test_cmdline_invalid =============
[17:32:42] =============== drm_test_cmdline_tv_options  ===============
[17:32:42] [PASSED] NTSC
[17:32:42] [PASSED] NTSC_443
[17:32:42] [PASSED] NTSC_J
[17:32:42] [PASSED] PAL
[17:32:42] [PASSED] PAL_M
[17:32:42] [PASSED] PAL_N
[17:32:42] [PASSED] SECAM
[17:32:42] [PASSED] MONO_525
[17:32:42] [PASSED] MONO_625
[17:32:42] =========== [PASSED] drm_test_cmdline_tv_options ===========
[17:32:42] =============== [PASSED] drm_cmdline_parser ================
[17:32:42] ========== drmm_connector_hdmi_init (20 subtests) ==========
[17:32:42] [PASSED] drm_test_connector_hdmi_init_valid
[17:32:42] [PASSED] drm_test_connector_hdmi_init_bpc_8
[17:32:42] [PASSED] drm_test_connector_hdmi_init_bpc_10
[17:32:42] [PASSED] drm_test_connector_hdmi_init_bpc_12
[17:32:42] [PASSED] drm_test_connector_hdmi_init_bpc_invalid
[17:32:42] [PASSED] drm_test_connector_hdmi_init_bpc_null
[17:32:42] [PASSED] drm_test_connector_hdmi_init_formats_empty
[17:32:42] [PASSED] drm_test_connector_hdmi_init_formats_no_rgb
[17:32:42] === drm_test_connector_hdmi_init_formats_yuv420_allowed  ===
[17:32:42] [PASSED] supported_formats=0x9 yuv420_allowed=1
[17:32:42] [PASSED] supported_formats=0x9 yuv420_allowed=0
[17:32:42] [PASSED] supported_formats=0x3 yuv420_allowed=1
[17:32:42] [PASSED] supported_formats=0x3 yuv420_allowed=0
[17:32:42] === [PASSED] drm_test_connector_hdmi_init_formats_yuv420_allowed ===
[17:32:42] [PASSED] drm_test_connector_hdmi_init_null_ddc
[17:32:42] [PASSED] drm_test_connector_hdmi_init_null_product
[17:32:42] [PASSED] drm_test_connector_hdmi_init_null_vendor
[17:32:42] [PASSED] drm_test_connector_hdmi_init_product_length_exact
[17:32:42] [PASSED] drm_test_connector_hdmi_init_product_length_too_long
[17:32:42] [PASSED] drm_test_connector_hdmi_init_product_valid
[17:32:42] [PASSED] drm_test_connector_hdmi_init_vendor_length_exact
[17:32:42] [PASSED] drm_test_connector_hdmi_init_vendor_length_too_long
[17:32:42] [PASSED] drm_test_connector_hdmi_init_vendor_valid
[17:32:42] ========= drm_test_connector_hdmi_init_type_valid  =========
[17:32:42] [PASSED] HDMI-A
[17:32:42] [PASSED] HDMI-B
[17:32:42] ===== [PASSED] drm_test_connector_hdmi_init_type_valid =====
[17:32:42] ======== drm_test_connector_hdmi_init_type_invalid  ========
[17:32:42] [PASSED] Unknown
[17:32:42] [PASSED] VGA
[17:32:42] [PASSED] DVI-I
[17:32:42] [PASSED] DVI-D
[17:32:42] [PASSED] DVI-A
[17:32:42] [PASSED] Composite
[17:32:42] [PASSED] SVIDEO
[17:32:42] [PASSED] LVDS
[17:32:42] [PASSED] Component
[17:32:42] [PASSED] DIN
[17:32:42] [PASSED] DP
[17:32:42] [PASSED] TV
[17:32:42] [PASSED] eDP
[17:32:42] [PASSED] Virtual
[17:32:42] [PASSED] DSI
[17:32:42] [PASSED] DPI
[17:32:42] [PASSED] Writeback
[17:32:42] [PASSED] SPI
[17:32:42] [PASSED] USB
[17:32:42] ==== [PASSED] drm_test_connector_hdmi_init_type_invalid ====
[17:32:42] ============ [PASSED] drmm_connector_hdmi_init =============
[17:32:42] ============= drmm_connector_init (3 subtests) =============
[17:32:42] [PASSED] drm_test_drmm_connector_init
[17:32:42] [PASSED] drm_test_drmm_connector_init_null_ddc
[17:32:42] ========= drm_test_drmm_connector_init_type_valid  =========
[17:32:42] [PASSED] Unknown
[17:32:42] [PASSED] VGA
[17:32:42] [PASSED] DVI-I
[17:32:42] [PASSED] DVI-D
[17:32:42] [PASSED] DVI-A
[17:32:42] [PASSED] Composite
[17:32:42] [PASSED] SVIDEO
[17:32:42] [PASSED] LVDS
[17:32:42] [PASSED] Component
[17:32:42] [PASSED] DIN
[17:32:42] [PASSED] DP
[17:32:42] [PASSED] HDMI-A
[17:32:42] [PASSED] HDMI-B
[17:32:42] [PASSED] TV
[17:32:42] [PASSED] eDP
[17:32:42] [PASSED] Virtual
[17:32:42] [PASSED] DSI
[17:32:42] [PASSED] DPI
[17:32:42] [PASSED] Writeback
[17:32:42] [PASSED] SPI
[17:32:42] [PASSED] USB
[17:32:42] ===== [PASSED] drm_test_drmm_connector_init_type_valid =====
[17:32:42] =============== [PASSED] drmm_connector_init ===============
[17:32:42] ========= drm_connector_dynamic_init (6 subtests) ==========
[17:32:42] [PASSED] drm_test_drm_connector_dynamic_init
[17:32:42] [PASSED] drm_test_drm_connector_dynamic_init_null_ddc
[17:32:42] [PASSED] drm_test_drm_connector_dynamic_init_not_added
[17:32:42] [PASSED] drm_test_drm_connector_dynamic_init_properties
[17:32:42] ===== drm_test_drm_connector_dynamic_init_type_valid  ======
[17:32:42] [PASSED] Unknown
[17:32:42] [PASSED] VGA
[17:32:42] [PASSED] DVI-I
[17:32:42] [PASSED] DVI-D
[17:32:42] [PASSED] DVI-A
[17:32:42] [PASSED] Composite
[17:32:42] [PASSED] SVIDEO
[17:32:42] [PASSED] LVDS
[17:32:42] [PASSED] Component
[17:32:42] [PASSED] DIN
[17:32:42] [PASSED] DP
[17:32:42] [PASSED] HDMI-A
[17:32:42] [PASSED] HDMI-B
[17:32:42] [PASSED] TV
[17:32:42] [PASSED] eDP
[17:32:42] [PASSED] Virtual
[17:32:42] [PASSED] DSI
[17:32:42] [PASSED] DPI
[17:32:42] [PASSED] Writeback
[17:32:42] [PASSED] SPI
[17:32:42] [PASSED] USB
[17:32:42] = [PASSED] drm_test_drm_connector_dynamic_init_type_valid ==
[17:32:42] ======== drm_test_drm_connector_dynamic_init_name  =========
[17:32:42] [PASSED] Unknown
[17:32:42] [PASSED] VGA
[17:32:42] [PASSED] DVI-I
[17:32:42] [PASSED] DVI-D
[17:32:42] [PASSED] DVI-A
[17:32:42] [PASSED] Composite
[17:32:42] [PASSED] SVIDEO
[17:32:42] [PASSED] LVDS
[17:32:42] [PASSED] Component
[17:32:42] [PASSED] DIN
[17:32:42] [PASSED] DP
[17:32:42] [PASSED] HDMI-A
[17:32:42] [PASSED] HDMI-B
[17:32:42] [PASSED] TV
[17:32:42] [PASSED] eDP
[17:32:42] [PASSED] Virtual
[17:32:42] [PASSED] DSI
[17:32:42] [PASSED] DPI
[17:32:42] [PASSED] Writeback
[17:32:42] [PASSED] SPI
[17:32:42] [PASSED] USB
[17:32:42] ==== [PASSED] drm_test_drm_connector_dynamic_init_name =====
[17:32:42] =========== [PASSED] drm_connector_dynamic_init ============
[17:32:42] ==== drm_connector_dynamic_register_early (4 subtests) =====
[17:32:42] [PASSED] drm_test_drm_connector_dynamic_register_early_on_list
[17:32:42] [PASSED] drm_test_drm_connector_dynamic_register_early_defer
[17:32:42] [PASSED] drm_test_drm_connector_dynamic_register_early_no_init
[17:32:42] [PASSED] drm_test_drm_connector_dynamic_register_early_no_mode_object
[17:32:42] ====== [PASSED] drm_connector_dynamic_register_early =======
[17:32:42] ======= drm_connector_dynamic_register (7 subtests) ========
[17:32:42] [PASSED] drm_test_drm_connector_dynamic_register_on_list
[17:32:42] [PASSED] drm_test_drm_connector_dynamic_register_no_defer
[17:32:42] [PASSED] drm_test_drm_connector_dynamic_register_no_init
[17:32:42] [PASSED] drm_test_drm_connector_dynamic_register_mode_object
[17:32:42] [PASSED] drm_test_drm_connector_dynamic_register_sysfs
[17:32:42] [PASSED] drm_test_drm_connector_dynamic_register_sysfs_name
[17:32:42] [PASSED] drm_test_drm_connector_dynamic_register_debugfs
[17:32:42] ========= [PASSED] drm_connector_dynamic_register ==========
[17:32:42] = drm_connector_attach_broadcast_rgb_property (2 subtests) =
[17:32:42] [PASSED] drm_test_drm_connector_attach_broadcast_rgb_property
[17:32:42] [PASSED] drm_test_drm_connector_attach_broadcast_rgb_property_hdmi_connector
[17:32:42] === [PASSED] drm_connector_attach_broadcast_rgb_property ===
[17:32:42] ========== drm_get_tv_mode_from_name (2 subtests) ==========
[17:32:42] ========== drm_test_get_tv_mode_from_name_valid  ===========
[17:32:42] [PASSED] NTSC
[17:32:42] [PASSED] NTSC-443
[17:32:42] [PASSED] NTSC-J
[17:32:42] [PASSED] PAL
[17:32:42] [PASSED] PAL-M
[17:32:42] [PASSED] PAL-N
[17:32:42] [PASSED] SECAM
[17:32:42] [PASSED] Mono
[17:32:42] ====== [PASSED] drm_test_get_tv_mode_from_name_valid =======
[17:32:42] [PASSED] drm_test_get_tv_mode_from_name_truncated
[17:32:42] ============ [PASSED] drm_get_tv_mode_from_name ============
[17:32:42] = drm_test_connector_hdmi_compute_mode_clock (12 subtests) =
[17:32:42] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb
[17:32:42] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_10bpc
[17:32:42] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_10bpc_vic_1
[17:32:42] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_12bpc
[17:32:42] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_12bpc_vic_1
[17:32:42] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_double
[17:32:42] = drm_test_connector_hdmi_compute_mode_clock_yuv420_valid  =
[17:32:42] [PASSED] VIC 96
[17:32:42] [PASSED] VIC 97
[17:32:42] [PASSED] VIC 101
[17:32:42] [PASSED] VIC 102
[17:32:42] [PASSED] VIC 106
[17:32:42] [PASSED] VIC 107
[17:32:42] === [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_valid ===
[17:32:42] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_10_bpc
[17:32:42] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_12_bpc
[17:32:42] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_8_bpc
[17:32:42] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_10_bpc
[17:32:42] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_12_bpc
[17:32:42] === [PASSED] drm_test_connector_hdmi_compute_mode_clock ====
[17:32:42] == drm_hdmi_connector_get_broadcast_rgb_name (2 subtests) ==
[17:32:42] === drm_test_drm_hdmi_connector_get_broadcast_rgb_name  ====
[17:32:42] [PASSED] Automatic
[17:32:42] [PASSED] Full
[17:32:42] [PASSED] Limited 16:235
[17:32:42] === [PASSED] drm_test_drm_hdmi_connector_get_broadcast_rgb_name ===
[17:32:42] [PASSED] drm_test_drm_hdmi_connector_get_broadcast_rgb_name_invalid
[17:32:42] ==== [PASSED] drm_hdmi_connector_get_broadcast_rgb_name ====
[17:32:42] == drm_hdmi_connector_get_output_format_name (2 subtests) ==
[17:32:42] === drm_test_drm_hdmi_connector_get_output_format_name  ====
[17:32:42] [PASSED] RGB
[17:32:42] [PASSED] YUV 4:2:0
[17:32:42] [PASSED] YUV 4:2:2
[17:32:42] [PASSED] YUV 4:4:4
[17:32:42] === [PASSED] drm_test_drm_hdmi_connector_get_output_format_name ===
[17:32:42] [PASSED] drm_test_drm_hdmi_connector_get_output_format_name_invalid
[17:32:42] ==== [PASSED] drm_hdmi_connector_get_output_format_name ====
[17:32:42] ============= drm_damage_helper (21 subtests) ==============
[17:32:42] [PASSED] drm_test_damage_iter_no_damage
[17:32:42] [PASSED] drm_test_damage_iter_no_damage_fractional_src
[17:32:42] [PASSED] drm_test_damage_iter_no_damage_src_moved
[17:32:42] [PASSED] drm_test_damage_iter_no_damage_fractional_src_moved
[17:32:42] [PASSED] drm_test_damage_iter_no_damage_not_visible
[17:32:42] [PASSED] drm_test_damage_iter_no_damage_no_crtc
[17:32:42] [PASSED] drm_test_damage_iter_no_damage_no_fb
[17:32:42] [PASSED] drm_test_damage_iter_simple_damage
[17:32:42] [PASSED] drm_test_damage_iter_single_damage
[17:32:42] [PASSED] drm_test_damage_iter_single_damage_intersect_src
[17:32:42] [PASSED] drm_test_damage_iter_single_damage_outside_src
[17:32:42] [PASSED] drm_test_damage_iter_single_damage_fractional_src
[17:32:42] [PASSED] drm_test_damage_iter_single_damage_intersect_fractional_src
[17:32:42] [PASSED] drm_test_damage_iter_single_damage_outside_fractional_src
[17:32:42] [PASSED] drm_test_damage_iter_single_damage_src_moved
[17:32:42] [PASSED] drm_test_damage_iter_single_damage_fractional_src_moved
[17:32:42] [PASSED] drm_test_damage_iter_damage
[17:32:42] [PASSED] drm_test_damage_iter_damage_one_intersect
[17:32:42] [PASSED] drm_test_damage_iter_damage_one_outside
[17:32:42] [PASSED] drm_test_damage_iter_damage_src_moved
[17:32:42] [PASSED] drm_test_damage_iter_damage_not_visible
[17:32:42] ================ [PASSED] drm_damage_helper ================
[17:32:42] ============== drm_dp_mst_helper (3 subtests) ==============
[17:32:42] ============== drm_test_dp_mst_calc_pbn_mode  ==============
[17:32:42] [PASSED] Clock 154000 BPP 30 DSC disabled
[17:32:42] [PASSED] Clock 234000 BPP 30 DSC disabled
[17:32:42] [PASSED] Clock 297000 BPP 24 DSC disabled
[17:32:42] [PASSED] Clock 332880 BPP 24 DSC enabled
[17:32:42] [PASSED] Clock 324540 BPP 24 DSC enabled
[17:32:42] ========== [PASSED] drm_test_dp_mst_calc_pbn_mode ==========
[17:32:42] ============== drm_test_dp_mst_calc_pbn_div  ===============
[17:32:42] [PASSED] Link rate 2000000 lane count 4
[17:32:42] [PASSED] Link rate 2000000 lane count 2
[17:32:42] [PASSED] Link rate 2000000 lane count 1
[17:32:42] [PASSED] Link rate 1350000 lane count 4
[17:32:42] [PASSED] Link rate 1350000 lane count 2
[17:32:42] [PASSED] Link rate 1350000 lane count 1
[17:32:42] [PASSED] Link rate 1000000 lane count 4
[17:32:42] [PASSED] Link rate 1000000 lane count 2
[17:32:42] [PASSED] Link rate 1000000 lane count 1
[17:32:42] [PASSED] Link rate 810000 lane count 4
[17:32:42] [PASSED] Link rate 810000 lane count 2
[17:32:42] [PASSED] Link rate 810000 lane count 1
[17:32:42] [PASSED] Link rate 540000 lane count 4
[17:32:42] [PASSED] Link rate 540000 lane count 2
[17:32:42] [PASSED] Link rate 540000 lane count 1
[17:32:42] [PASSED] Link rate 270000 lane count 4
[17:32:42] [PASSED] Link rate 270000 lane count 2
[17:32:42] [PASSED] Link rate 270000 lane count 1
[17:32:42] [PASSED] Link rate 162000 lane count 4
[17:32:42] [PASSED] Link rate 162000 lane count 2
[17:32:42] [PASSED] Link rate 162000 lane count 1
[17:32:42] ========== [PASSED] drm_test_dp_mst_calc_pbn_div ===========
[17:32:42] ========= drm_test_dp_mst_sideband_msg_req_decode  =========
[17:32:42] [PASSED] DP_ENUM_PATH_RESOURCES with port number
[17:32:42] [PASSED] DP_POWER_UP_PHY with port number
[17:32:42] [PASSED] DP_POWER_DOWN_PHY with port number
[17:32:42] [PASSED] DP_ALLOCATE_PAYLOAD with SDP stream sinks
[17:32:42] [PASSED] DP_ALLOCATE_PAYLOAD with port number
[17:32:42] [PASSED] DP_ALLOCATE_PAYLOAD with VCPI
[17:32:42] [PASSED] DP_ALLOCATE_PAYLOAD with PBN
[17:32:42] [PASSED] DP_QUERY_PAYLOAD with port number
[17:32:42] [PASSED] DP_QUERY_PAYLOAD with VCPI
[17:32:42] [PASSED] DP_REMOTE_DPCD_READ with port number
[17:32:42] [PASSED] DP_REMOTE_DPCD_READ with DPCD address
[17:32:42] [PASSED] DP_REMOTE_DPCD_READ with max number of bytes
[17:32:42] [PASSED] DP_REMOTE_DPCD_WRITE with port number
[17:32:42] [PASSED] DP_REMOTE_DPCD_WRITE with DPCD address
[17:32:42] [PASSED] DP_REMOTE_DPCD_WRITE with data array
[17:32:42] [PASSED] DP_REMOTE_I2C_READ with port number
[17:32:42] [PASSED] DP_REMOTE_I2C_READ with I2C device ID
[17:32:42] [PASSED] DP_REMOTE_I2C_READ with transactions array
[17:32:42] [PASSED] DP_REMOTE_I2C_WRITE with port number
[17:32:42] [PASSED] DP_REMOTE_I2C_WRITE with I2C device ID
[17:32:42] [PASSED] DP_REMOTE_I2C_WRITE with data array
[17:32:42] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream ID
[17:32:42] [PASSED] DP_QUERY_STREAM_ENC_STATUS with client ID
[17:32:42] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream event
[17:32:42] [PASSED] DP_QUERY_STREAM_ENC_STATUS with valid stream event
[17:32:42] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream behavior
[17:32:42] [PASSED] DP_QUERY_STREAM_ENC_STATUS with a valid stream behavior
[17:32:42] ===== [PASSED] drm_test_dp_mst_sideband_msg_req_decode =====
[17:32:42] ================ [PASSED] drm_dp_mst_helper ================
[17:32:42] ================== drm_exec (7 subtests) ===================
[17:32:42] [PASSED] sanitycheck
[17:32:42] [PASSED] test_lock
[17:32:42] [PASSED] test_lock_unlock
[17:32:42] [PASSED] test_duplicates
[17:32:42] [PASSED] test_prepare
[17:32:42] [PASSED] test_prepare_array
[17:32:42] [PASSED] test_multiple_loops
[17:32:42] ==================== [PASSED] drm_exec =====================
[17:32:42] =========== drm_format_helper_test (17 subtests) ===========
[17:32:42] ============== drm_test_fb_xrgb8888_to_gray8  ==============
[17:32:42] [PASSED] single_pixel_source_buffer
[17:32:42] [PASSED] single_pixel_clip_rectangle
[17:32:42] [PASSED] well_known_colors
[17:32:42] [PASSED] destination_pitch
[17:32:42] ========== [PASSED] drm_test_fb_xrgb8888_to_gray8 ==========
[17:32:42] ============= drm_test_fb_xrgb8888_to_rgb332  ==============
[17:32:42] [PASSED] single_pixel_source_buffer
[17:32:42] [PASSED] single_pixel_clip_rectangle
[17:32:42] [PASSED] well_known_colors
[17:32:42] [PASSED] destination_pitch
[17:32:42] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb332 ==========
[17:32:42] ============= drm_test_fb_xrgb8888_to_rgb565  ==============
[17:32:42] [PASSED] single_pixel_source_buffer
[17:32:42] [PASSED] single_pixel_clip_rectangle
[17:32:42] [PASSED] well_known_colors
[17:32:42] [PASSED] destination_pitch
[17:32:42] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb565 ==========
[17:32:42] ============ drm_test_fb_xrgb8888_to_xrgb1555  =============
[17:32:42] [PASSED] single_pixel_source_buffer
[17:32:42] [PASSED] single_pixel_clip_rectangle
[17:32:42] [PASSED] well_known_colors
[17:32:42] [PASSED] destination_pitch
[17:32:42] ======== [PASSED] drm_test_fb_xrgb8888_to_xrgb1555 =========
[17:32:42] ============ drm_test_fb_xrgb8888_to_argb1555  =============
[17:32:42] [PASSED] single_pixel_source_buffer
[17:32:42] [PASSED] single_pixel_clip_rectangle
[17:32:42] [PASSED] well_known_colors
[17:32:42] [PASSED] destination_pitch
[17:32:42] ======== [PASSED] drm_test_fb_xrgb8888_to_argb1555 =========
[17:32:42] ============ drm_test_fb_xrgb8888_to_rgba5551  =============
[17:32:42] [PASSED] single_pixel_source_buffer
[17:32:42] [PASSED] single_pixel_clip_rectangle
[17:32:42] [PASSED] well_known_colors
[17:32:42] [PASSED] destination_pitch
[17:32:42] ======== [PASSED] drm_test_fb_xrgb8888_to_rgba5551 =========
[17:32:42] ============= drm_test_fb_xrgb8888_to_rgb888  ==============
[17:32:42] [PASSED] single_pixel_source_buffer
[17:32:42] [PASSED] single_pixel_clip_rectangle
[17:32:42] [PASSED] well_known_colors
[17:32:42] [PASSED] destination_pitch
[17:32:42] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb888 ==========
[17:32:42] ============= drm_test_fb_xrgb8888_to_bgr888  ==============
[17:32:42] [PASSED] single_pixel_source_buffer
[17:32:42] [PASSED] single_pixel_clip_rectangle
[17:32:42] [PASSED] well_known_colors
[17:32:42] [PASSED] destination_pitch
[17:32:42] ========= [PASSED] drm_test_fb_xrgb8888_to_bgr888 ==========
[17:32:42] ============ drm_test_fb_xrgb8888_to_argb8888  =============
[17:32:42] [PASSED] single_pixel_source_buffer
[17:32:42] [PASSED] single_pixel_clip_rectangle
[17:32:42] [PASSED] well_known_colors
[17:32:42] [PASSED] destination_pitch
[17:32:42] ======== [PASSED] drm_test_fb_xrgb8888_to_argb8888 =========
[17:32:42] =========== drm_test_fb_xrgb8888_to_xrgb2101010  ===========
[17:32:42] [PASSED] single_pixel_source_buffer
[17:32:42] [PASSED] single_pixel_clip_rectangle
[17:32:42] [PASSED] well_known_colors
[17:32:42] [PASSED] destination_pitch
[17:32:42] ======= [PASSED] drm_test_fb_xrgb8888_to_xrgb2101010 =======
[17:32:42] =========== drm_test_fb_xrgb8888_to_argb2101010  ===========
[17:32:42] [PASSED] single_pixel_source_buffer
[17:32:42] [PASSED] single_pixel_clip_rectangle
[17:32:42] [PASSED] well_known_colors
[17:32:42] [PASSED] destination_pitch
[17:32:42] ======= [PASSED] drm_test_fb_xrgb8888_to_argb2101010 =======
[17:32:42] ============== drm_test_fb_xrgb8888_to_mono  ===============
[17:32:42] [PASSED] single_pixel_source_buffer
[17:32:42] [PASSED] single_pixel_clip_rectangle
[17:32:42] [PASSED] well_known_colors
[17:32:42] [PASSED] destination_pitch
[17:32:42] ========== [PASSED] drm_test_fb_xrgb8888_to_mono ===========
[17:32:42] ==================== drm_test_fb_swab  =====================
[17:32:42] [PASSED] single_pixel_source_buffer
[17:32:42] [PASSED] single_pixel_clip_rectangle
[17:32:42] [PASSED] well_known_colors
[17:32:42] [PASSED] destination_pitch
[17:32:42] ================ [PASSED] drm_test_fb_swab =================
[17:32:42] ============ drm_test_fb_xrgb8888_to_xbgr8888  =============
[17:32:42] [PASSED] single_pixel_source_buffer
[17:32:42] [PASSED] single_pixel_clip_rectangle
[17:32:42] [PASSED] well_known_colors
[17:32:42] [PASSED] destination_pitch
[17:32:42] ======== [PASSED] drm_test_fb_xrgb8888_to_xbgr8888 =========
[17:32:42] ============ drm_test_fb_xrgb8888_to_abgr8888  =============
[17:32:42] [PASSED] single_pixel_source_buffer
[17:32:42] [PASSED] single_pixel_clip_rectangle
[17:32:42] [PASSED] well_known_colors
[17:32:42] [PASSED] destination_pitch
[17:32:42] ======== [PASSED] drm_test_fb_xrgb8888_to_abgr8888 =========
[17:32:42] ================= drm_test_fb_clip_offset  =================
[17:32:42] [PASSED] pass through
[17:32:42] [PASSED] horizontal offset
[17:32:42] [PASSED] vertical offset
[17:32:42] [PASSED] horizontal and vertical offset
[17:32:42] [PASSED] horizontal offset (custom pitch)
[17:32:42] [PASSED] vertical offset (custom pitch)
[17:32:42] [PASSED] horizontal and vertical offset (custom pitch)
[17:32:42] ============= [PASSED] drm_test_fb_clip_offset =============
[17:32:42] =================== drm_test_fb_memcpy  ====================
[17:32:42] [PASSED] single_pixel_source_buffer: XR24 little-endian (0x34325258)
[17:32:42] [PASSED] single_pixel_source_buffer: XRA8 little-endian (0x38415258)
[17:32:42] [PASSED] single_pixel_source_buffer: YU24 little-endian (0x34325559)
[17:32:42] [PASSED] single_pixel_clip_rectangle: XB24 little-endian (0x34324258)
[17:32:42] [PASSED] single_pixel_clip_rectangle: XRA8 little-endian (0x38415258)
[17:32:42] [PASSED] single_pixel_clip_rectangle: YU24 little-endian (0x34325559)
[17:32:42] [PASSED] well_known_colors: XB24 little-endian (0x34324258)
[17:32:42] [PASSED] well_known_colors: XRA8 little-endian (0x38415258)
[17:32:42] [PASSED] well_known_colors: YU24 little-endian (0x34325559)
[17:32:42] [PASSED] destination_pitch: XB24 little-endian (0x34324258)
[17:32:42] [PASSED] destination_pitch: XRA8 little-endian (0x38415258)
[17:32:42] [PASSED] destination_pitch: YU24 little-endian (0x34325559)
[17:32:42] =============== [PASSED] drm_test_fb_memcpy ================
[17:32:42] ============= [PASSED] drm_format_helper_test ==============
[17:32:42] ================= drm_format (18 subtests) =================
[17:32:42] [PASSED] drm_test_format_block_width_invalid
[17:32:42] [PASSED] drm_test_format_block_width_one_plane
[17:32:42] [PASSED] drm_test_format_block_width_two_plane
[17:32:42] [PASSED] drm_test_format_block_width_three_plane
[17:32:42] [PASSED] drm_test_format_block_width_tiled
[17:32:42] [PASSED] drm_test_format_block_height_invalid
[17:32:42] [PASSED] drm_test_format_block_height_one_plane
[17:32:42] [PASSED] drm_test_format_block_height_two_plane
[17:32:42] [PASSED] drm_test_format_block_height_three_plane
[17:32:42] [PASSED] drm_test_format_block_height_tiled
[17:32:42] [PASSED] drm_test_format_min_pitch_invalid
[17:32:42] [PASSED] drm_test_format_min_pitch_one_plane_8bpp
[17:32:42] [PASSED] drm_test_format_min_pitch_one_plane_16bpp
[17:32:42] [PASSED] drm_test_format_min_pitch_one_plane_24bpp
[17:32:42] [PASSED] drm_test_format_min_pitch_one_plane_32bpp
[17:32:42] [PASSED] drm_test_format_min_pitch_two_plane
[17:32:42] [PASSED] drm_test_format_min_pitch_three_plane_8bpp
[17:32:42] [PASSED] drm_test_format_min_pitch_tiled
[17:32:42] =================== [PASSED] drm_format ====================
[17:32:42] ============== drm_framebuffer (10 subtests) ===============
[17:32:42] ========== drm_test_framebuffer_check_src_coords  ==========
[17:32:42] [PASSED] Success: source fits into fb
[17:32:42] [PASSED] Fail: overflowing fb with x-axis coordinate
[17:32:42] [PASSED] Fail: overflowing fb with y-axis coordinate
[17:32:42] [PASSED] Fail: overflowing fb with source width
[17:32:42] [PASSED] Fail: overflowing fb with source height
[17:32:42] ====== [PASSED] drm_test_framebuffer_check_src_coords ======
[17:32:42] [PASSED] drm_test_framebuffer_cleanup
[17:32:42] =============== drm_test_framebuffer_create  ===============
[17:32:42] [PASSED] ABGR8888 normal sizes
[17:32:42] [PASSED] ABGR8888 max sizes
[17:32:42] [PASSED] ABGR8888 pitch greater than min required
[17:32:42] [PASSED] ABGR8888 pitch less than min required
[17:32:42] [PASSED] ABGR8888 Invalid width
[17:32:42] [PASSED] ABGR8888 Invalid buffer handle
[17:32:42] [PASSED] No pixel format
[17:32:42] [PASSED] ABGR8888 Width 0
[17:32:42] [PASSED] ABGR8888 Height 0
[17:32:42] [PASSED] ABGR8888 Out of bound height * pitch combination
[17:32:42] [PASSED] ABGR8888 Large buffer offset
[17:32:42] [PASSED] ABGR8888 Buffer offset for inexistent plane
[17:32:42] [PASSED] ABGR8888 Invalid flag
[17:32:42] [PASSED] ABGR8888 Set DRM_MODE_FB_MODIFIERS without modifiers
[17:32:42] [PASSED] ABGR8888 Valid buffer modifier
[17:32:42] [PASSED] ABGR8888 Invalid buffer modifier(DRM_FORMAT_MOD_SAMSUNG_64_32_TILE)
[17:32:42] [PASSED] ABGR8888 Extra pitches without DRM_MODE_FB_MODIFIERS
[17:32:42] [PASSED] ABGR8888 Extra pitches with DRM_MODE_FB_MODIFIERS
[17:32:42] [PASSED] NV12 Normal sizes
[17:32:42] [PASSED] NV12 Max sizes
[17:32:42] [PASSED] NV12 Invalid pitch
[17:32:42] [PASSED] NV12 Invalid modifier/missing DRM_MODE_FB_MODIFIERS flag
[17:32:42] [PASSED] NV12 different  modifier per-plane
[17:32:42] [PASSED] NV12 with DRM_FORMAT_MOD_SAMSUNG_64_32_TILE
[17:32:42] [PASSED] NV12 Valid modifiers without DRM_MODE_FB_MODIFIERS
[17:32:42] [PASSED] NV12 Modifier for inexistent plane
[17:32:42] [PASSED] NV12 Handle for inexistent plane
[17:32:42] [PASSED] NV12 Handle for inexistent plane without DRM_MODE_FB_MODIFIERS
[17:32:42] [PASSED] YVU420 DRM_MODE_FB_MODIFIERS set without modifier
[17:32:42] [PASSED] YVU420 Normal sizes
[17:32:42] [PASSED] YVU420 Max sizes
[17:32:42] [PASSED] YVU420 Invalid pitch
[17:32:42] [PASSED] YVU420 Different pitches
[17:32:42] [PASSED] YVU420 Different buffer offsets/pitches
[17:32:42] [PASSED] YVU420 Modifier set just for plane 0, without DRM_MODE_FB_MODIFIERS
[17:32:42] [PASSED] YVU420 Modifier set just for planes 0, 1, without DRM_MODE_FB_MODIFIERS
[17:32:42] [PASSED] YVU420 Modifier set just for plane 0, 1, with DRM_MODE_FB_MODIFIERS
[17:32:42] [PASSED] YVU420 Valid modifier
[17:32:42] [PASSED] YVU420 Different modifiers per plane
[17:32:42] [PASSED] YVU420 Modifier for inexistent plane
[17:32:42] [PASSED] YUV420_10BIT Invalid modifier(DRM_FORMAT_MOD_LINEAR)
[17:32:42] [PASSED] X0L2 Normal sizes
[17:32:42] [PASSED] X0L2 Max sizes
[17:32:42] [PASSED] X0L2 Invalid pitch
[17:32:42] [PASSED] X0L2 Pitch greater than minimum required
[17:32:42] [PASSED] X0L2 Handle for inexistent plane
[17:32:42] [PASSED] X0L2 Offset for inexistent plane, without DRM_MODE_FB_MODIFIERS set
[17:32:42] [PASSED] X0L2 Modifier without DRM_MODE_FB_MODIFIERS set
[17:32:42] [PASSED] X0L2 Valid modifier
[17:32:42] [PASSED] X0L2 Modifier for inexistent plane
[17:32:42] =========== [PASSED] drm_test_framebuffer_create ===========
[17:32:42] [PASSED] drm_test_framebuffer_free
[17:32:42] [PASSED] drm_test_framebuffer_init
[17:32:42] [PASSED] drm_test_framebuffer_init_bad_format
[17:32:42] [PASSED] drm_test_framebuffer_init_dev_mismatch
[17:32:42] [PASSED] drm_test_framebuffer_lookup
[17:32:42] [PASSED] drm_test_framebuffer_lookup_inexistent
[17:32:42] [PASSED] drm_test_framebuffer_modifiers_not_supported
[17:32:42] ================= [PASSED] drm_framebuffer =================
[17:32:42] ================ drm_gem_shmem (8 subtests) ================
[17:32:42] [PASSED] drm_gem_shmem_test_obj_create
[17:32:42] [PASSED] drm_gem_shmem_test_obj_create_private
[17:32:42] [PASSED] drm_gem_shmem_test_pin_pages
[17:32:42] [PASSED] drm_gem_shmem_test_vmap
[17:32:42] [PASSED] drm_gem_shmem_test_get_pages_sgt
[17:32:42] [PASSED] drm_gem_shmem_test_get_sg_table
[17:32:42] [PASSED] drm_gem_shmem_test_madvise
[17:32:42] [PASSED] drm_gem_shmem_test_purge
[17:32:42] ================== [PASSED] drm_gem_shmem ==================
[17:32:42] === drm_atomic_helper_connector_hdmi_check (27 subtests) ===
[17:32:42] [PASSED] drm_test_check_broadcast_rgb_auto_cea_mode
[17:32:42] [PASSED] drm_test_check_broadcast_rgb_auto_cea_mode_vic_1
[17:32:42] [PASSED] drm_test_check_broadcast_rgb_full_cea_mode
[17:32:42] [PASSED] drm_test_check_broadcast_rgb_full_cea_mode_vic_1
[17:32:42] [PASSED] drm_test_check_broadcast_rgb_limited_cea_mode
[17:32:42] [PASSED] drm_test_check_broadcast_rgb_limited_cea_mode_vic_1
[17:32:42] ====== drm_test_check_broadcast_rgb_cea_mode_yuv420  =======
[17:32:42] [PASSED] Automatic
[17:32:42] [PASSED] Full
[17:32:42] [PASSED] Limited 16:235
[17:32:42] == [PASSED] drm_test_check_broadcast_rgb_cea_mode_yuv420 ===
[17:32:42] [PASSED] drm_test_check_broadcast_rgb_crtc_mode_changed
[17:32:42] [PASSED] drm_test_check_broadcast_rgb_crtc_mode_not_changed
[17:32:42] [PASSED] drm_test_check_disable_connector
[17:32:42] [PASSED] drm_test_check_hdmi_funcs_reject_rate
[17:32:42] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_rgb
[17:32:42] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_yuv420
[17:32:42] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_ignore_yuv422
[17:32:42] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_ignore_yuv420
[17:32:42] [PASSED] drm_test_check_driver_unsupported_fallback_yuv420
[17:32:42] [PASSED] drm_test_check_output_bpc_crtc_mode_changed
[17:32:42] [PASSED] drm_test_check_output_bpc_crtc_mode_not_changed
[17:32:42] [PASSED] drm_test_check_output_bpc_dvi
[17:32:42] [PASSED] drm_test_check_output_bpc_format_vic_1
[17:32:42] [PASSED] drm_test_check_output_bpc_format_display_8bpc_only
[17:32:42] [PASSED] drm_test_check_output_bpc_format_display_rgb_only
[17:32:42] [PASSED] drm_test_check_output_bpc_format_driver_8bpc_only
[17:32:42] [PASSED] drm_test_check_output_bpc_format_driver_rgb_only
[17:32:42] [PASSED] drm_test_check_tmds_char_rate_rgb_8bpc
[17:32:42] [PASSED] drm_test_check_tmds_char_rate_rgb_10bpc
[17:32:42] [PASSED] drm_test_check_tmds_char_rate_rgb_12bpc
[17:32:42] ===== [PASSED] drm_atomic_helper_connector_hdmi_check ======
[17:32:42] === drm_atomic_helper_connector_hdmi_reset (6 subtests) ====
[17:32:42] [PASSED] drm_test_check_broadcast_rgb_value
[17:32:42] [PASSED] drm_test_check_bpc_8_value
[17:32:42] [PASSED] drm_test_check_bpc_10_value
[17:32:42] [PASSED] drm_test_check_bpc_12_value
[17:32:42] [PASSED] drm_test_check_format_value
[17:32:42] [PASSED] drm_test_check_tmds_char_value
[17:32:42] ===== [PASSED] drm_atomic_helper_connector_hdmi_reset ======
[17:32:42] = drm_atomic_helper_connector_hdmi_mode_valid (4 subtests) =
[17:32:42] [PASSED] drm_test_check_mode_valid
[17:32:42] [PASSED] drm_test_check_mode_valid_reject
[17:32:42] [PASSED] drm_test_check_mode_valid_reject_rate
[17:32:42] [PASSED] drm_test_check_mode_valid_reject_max_clock
[17:32:42] === [PASSED] drm_atomic_helper_connector_hdmi_mode_valid ===
[17:32:42] ================= drm_managed (2 subtests) =================
[17:32:42] [PASSED] drm_test_managed_release_action
[17:32:42] [PASSED] drm_test_managed_run_action
[17:32:42] =================== [PASSED] drm_managed ===================
[17:32:42] =================== drm_mm (6 subtests) ====================
[17:32:42] [PASSED] drm_test_mm_init
[17:32:42] [PASSED] drm_test_mm_debug
[17:32:42] [PASSED] drm_test_mm_align32
[17:32:42] [PASSED] drm_test_mm_align64
[17:32:42] [PASSED] drm_test_mm_lowest
[17:32:42] [PASSED] drm_test_mm_highest
[17:32:42] ===================== [PASSED] drm_mm ======================
[17:32:42] ============= drm_modes_analog_tv (5 subtests) =============
[17:32:42] [PASSED] drm_test_modes_analog_tv_mono_576i
[17:32:42] [PASSED] drm_test_modes_analog_tv_ntsc_480i
[17:32:42] [PASSED] drm_test_modes_analog_tv_ntsc_480i_inlined
[17:32:42] [PASSED] drm_test_modes_analog_tv_pal_576i
[17:32:42] [PASSED] drm_test_modes_analog_tv_pal_576i_inlined
[17:32:42] =============== [PASSED] drm_modes_analog_tv ===============
[17:32:42] ============== drm_plane_helper (2 subtests) ===============
[17:32:42] =============== drm_test_check_plane_state  ================
[17:32:42] [PASSED] clipping_simple
[17:32:42] [PASSED] clipping_rotate_reflect
[17:32:42] [PASSED] positioning_simple
[17:32:42] [PASSED] upscaling
[17:32:42] [PASSED] downscaling
[17:32:42] [PASSED] rounding1
[17:32:42] [PASSED] rounding2
[17:32:42] [PASSED] rounding3
[17:32:42] [PASSED] rounding4
[17:32:42] =========== [PASSED] drm_test_check_plane_state ============
[17:32:42] =========== drm_test_check_invalid_plane_state  ============
[17:32:42] [PASSED] positioning_invalid
[17:32:42] [PASSED] upscaling_invalid
[17:32:42] [PASSED] downscaling_invalid
[17:32:42] ======= [PASSED] drm_test_check_invalid_plane_state ========
[17:32:42] ================ [PASSED] drm_plane_helper =================
[17:32:42] ====== drm_connector_helper_tv_get_modes (1 subtest) =======
[17:32:42] ====== drm_test_connector_helper_tv_get_modes_check  =======
[17:32:42] [PASSED] None
[17:32:42] [PASSED] PAL
[17:32:42] [PASSED] NTSC
[17:32:42] [PASSED] Both, NTSC Default
[17:32:42] [PASSED] Both, PAL Default
[17:32:42] [PASSED] Both, NTSC Default, with PAL on command-line
[17:32:42] [PASSED] Both, PAL Default, with NTSC on command-line
[17:32:42] == [PASSED] drm_test_connector_helper_tv_get_modes_check ===
[17:32:42] ======== [PASSED] drm_connector_helper_tv_get_modes ========
[17:32:42] ================== drm_rect (9 subtests) ===================
[17:32:42] [PASSED] drm_test_rect_clip_scaled_div_by_zero
[17:32:42] [PASSED] drm_test_rect_clip_scaled_not_clipped
[17:32:42] [PASSED] drm_test_rect_clip_scaled_clipped
[17:32:42] [PASSED] drm_test_rect_clip_scaled_signed_vs_unsigned
[17:32:42] ================= drm_test_rect_intersect  =================
[17:32:42] [PASSED] top-left x bottom-right: 2x2+1+1 x 2x2+0+0
[17:32:42] [PASSED] top-right x bottom-left: 2x2+0+0 x 2x2+1-1
[17:32:42] [PASSED] bottom-left x top-right: 2x2+1-1 x 2x2+0+0
[17:32:42] [PASSED] bottom-right x top-left: 2x2+0+0 x 2x2+1+1
[17:32:42] [PASSED] right x left: 2x1+0+0 x 3x1+1+0
[17:32:42] [PASSED] left x right: 3x1+1+0 x 2x1+0+0
[17:32:42] [PASSED] up x bottom: 1x2+0+0 x 1x3+0-1
[17:32:42] [PASSED] bottom x up: 1x3+0-1 x 1x2+0+0
[17:32:42] [PASSED] touching corner: 1x1+0+0 x 2x2+1+1
[17:32:42] [PASSED] touching side: 1x1+0+0 x 1x1+1+0
[17:32:42] [PASSED] equal rects: 2x2+0+0 x 2x2+0+0
[17:32:42] [PASSED] inside another: 2x2+0+0 x 1x1+1+1
[17:32:42] [PASSED] far away: 1x1+0+0 x 1x1+3+6
[17:32:42] [PASSED] points intersecting: 0x0+5+10 x 0x0+5+10
[17:32:42] [PASSED] points not intersecting: 0x0+0+0 x 0x0+5+10
[17:32:42] ============= [PASSED] drm_test_rect_intersect =============
[17:32:42] ================ drm_test_rect_calc_hscale  ================
[17:32:42] [PASSED] normal use
[17:32:42] [PASSED] out of max range
[17:32:42] [PASSED] out of min range
[17:32:42] [PASSED] zero dst
[17:32:42] [PASSED] negative src
[17:32:42] [PASSED] negative dst
[17:32:42] ============ [PASSED] drm_test_rect_calc_hscale ============
[17:32:42] ================ drm_test_rect_calc_vscale  ================
[17:32:42] [PASSED] normal use
stty: 'standard input': Inappropriate ioctl for device
[17:32:42] [PASSED] out of max range
[17:32:42] [PASSED] out of min range
[17:32:42] [PASSED] zero dst
[17:32:42] [PASSED] negative src
[17:32:42] [PASSED] negative dst
[17:32:42] ============ [PASSED] drm_test_rect_calc_vscale ============
[17:32:42] ================== drm_test_rect_rotate  ===================
[17:32:42] [PASSED] reflect-x
[17:32:42] [PASSED] reflect-y
[17:32:42] [PASSED] rotate-0
[17:32:42] [PASSED] rotate-90
[17:32:42] [PASSED] rotate-180
[17:32:42] [PASSED] rotate-270
[17:32:42] ============== [PASSED] drm_test_rect_rotate ===============
[17:32:42] ================ drm_test_rect_rotate_inv  =================
[17:32:42] [PASSED] reflect-x
[17:32:42] [PASSED] reflect-y
[17:32:42] [PASSED] rotate-0
[17:32:42] [PASSED] rotate-90
[17:32:42] [PASSED] rotate-180
[17:32:42] [PASSED] rotate-270
[17:32:42] ============ [PASSED] drm_test_rect_rotate_inv =============
[17:32:42] ==================== [PASSED] drm_rect =====================
[17:32:42] ============ drm_sysfb_modeset_test (1 subtest) ============
[17:32:42] ============ drm_test_sysfb_build_fourcc_list  =============
[17:32:42] [PASSED] no native formats
[17:32:42] [PASSED] XRGB8888 as native format
[17:32:42] [PASSED] remove duplicates
[17:32:42] [PASSED] convert alpha formats
[17:32:42] [PASSED] random formats
[17:32:42] ======== [PASSED] drm_test_sysfb_build_fourcc_list =========
[17:32:42] ============= [PASSED] drm_sysfb_modeset_test ==============
[17:32:42] ================== drm_fixp (2 subtests) ===================
[17:32:42] [PASSED] drm_test_int2fixp
[17:32:42] [PASSED] drm_test_sm2fixp
[17:32:42] ==================== [PASSED] drm_fixp =====================
[17:32:42] ============================================================
[17:32:42] Testing complete. Ran 624 tests: passed: 624
[17:32:42] Elapsed time: 27.577s total, 1.683s configuring, 25.473s building, 0.419s running

+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/ttm/tests/.kunitconfig
[17:32:42] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[17:32:44] Building KUnit Kernel ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
Building with:
$ make all compile_commands.json scripts_gdb ARCH=um O=.kunit --jobs=48
[17:32:53] Starting KUnit Kernel (1/1)...
[17:32:53] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[17:32:53] ================= ttm_device (5 subtests) ==================
[17:32:53] [PASSED] ttm_device_init_basic
[17:32:53] [PASSED] ttm_device_init_multiple
[17:32:53] [PASSED] ttm_device_fini_basic
[17:32:53] [PASSED] ttm_device_init_no_vma_man
[17:32:53] ================== ttm_device_init_pools  ==================
[17:32:53] [PASSED] No DMA allocations, no DMA32 required
[17:32:53] [PASSED] DMA allocations, DMA32 required
[17:32:53] [PASSED] No DMA allocations, DMA32 required
[17:32:53] [PASSED] DMA allocations, no DMA32 required
[17:32:53] ============== [PASSED] ttm_device_init_pools ==============
[17:32:53] =================== [PASSED] ttm_device ====================
[17:32:53] ================== ttm_pool (8 subtests) ===================
[17:32:53] ================== ttm_pool_alloc_basic  ===================
[17:32:53] [PASSED] One page
[17:32:53] [PASSED] More than one page
[17:32:53] [PASSED] Above the allocation limit
[17:32:53] [PASSED] One page, with coherent DMA mappings enabled
[17:32:53] [PASSED] Above the allocation limit, with coherent DMA mappings enabled
[17:32:53] ============== [PASSED] ttm_pool_alloc_basic ===============
[17:32:53] ============== ttm_pool_alloc_basic_dma_addr  ==============
[17:32:53] [PASSED] One page
[17:32:53] [PASSED] More than one page
[17:32:53] [PASSED] Above the allocation limit
[17:32:53] [PASSED] One page, with coherent DMA mappings enabled
[17:32:53] [PASSED] Above the allocation limit, with coherent DMA mappings enabled
[17:32:53] ========== [PASSED] ttm_pool_alloc_basic_dma_addr ==========
[17:32:53] [PASSED] ttm_pool_alloc_order_caching_match
[17:32:53] [PASSED] ttm_pool_alloc_caching_mismatch
[17:32:53] [PASSED] ttm_pool_alloc_order_mismatch
[17:32:53] [PASSED] ttm_pool_free_dma_alloc
[17:32:53] [PASSED] ttm_pool_free_no_dma_alloc
[17:32:53] [PASSED] ttm_pool_fini_basic
[17:32:53] ==================== [PASSED] ttm_pool =====================
[17:32:53] ================ ttm_resource (8 subtests) =================
[17:32:53] ================= ttm_resource_init_basic  =================
[17:32:53] [PASSED] Init resource in TTM_PL_SYSTEM
[17:32:53] [PASSED] Init resource in TTM_PL_VRAM
[17:32:53] [PASSED] Init resource in a private placement
[17:32:53] [PASSED] Init resource in TTM_PL_SYSTEM, set placement flags
[17:32:53] ============= [PASSED] ttm_resource_init_basic =============
[17:32:53] [PASSED] ttm_resource_init_pinned
[17:32:53] [PASSED] ttm_resource_fini_basic
[17:32:53] [PASSED] ttm_resource_manager_init_basic
[17:32:53] [PASSED] ttm_resource_manager_usage_basic
[17:32:53] [PASSED] ttm_resource_manager_set_used_basic
[17:32:53] [PASSED] ttm_sys_man_alloc_basic
[17:32:53] [PASSED] ttm_sys_man_free_basic
[17:32:53] ================== [PASSED] ttm_resource ===================
[17:32:53] =================== ttm_tt (15 subtests) ===================
[17:32:53] ==================== ttm_tt_init_basic  ====================
[17:32:53] [PASSED] Page-aligned size
[17:32:53] [PASSED] Extra pages requested
[17:32:53] ================ [PASSED] ttm_tt_init_basic ================
[17:32:53] [PASSED] ttm_tt_init_misaligned
[17:32:53] [PASSED] ttm_tt_fini_basic
[17:32:53] [PASSED] ttm_tt_fini_sg
[17:32:53] [PASSED] ttm_tt_fini_shmem
[17:32:53] [PASSED] ttm_tt_create_basic
[17:32:53] [PASSED] ttm_tt_create_invalid_bo_type
[17:32:53] [PASSED] ttm_tt_create_ttm_exists
[17:32:53] [PASSED] ttm_tt_create_failed
[17:32:53] [PASSED] ttm_tt_destroy_basic
[17:32:53] [PASSED] ttm_tt_populate_null_ttm
[17:32:53] [PASSED] ttm_tt_populate_populated_ttm
[17:32:53] [PASSED] ttm_tt_unpopulate_basic
[17:32:53] [PASSED] ttm_tt_unpopulate_empty_ttm
[17:32:53] [PASSED] ttm_tt_swapin_basic
[17:32:53] ===================== [PASSED] ttm_tt ======================
[17:32:53] =================== ttm_bo (14 subtests) ===================
[17:32:53] =========== ttm_bo_reserve_optimistic_no_ticket  ===========
[17:32:53] [PASSED] Cannot be interrupted and sleeps
[17:32:53] [PASSED] Cannot be interrupted, locks straight away
[17:32:53] [PASSED] Can be interrupted, sleeps
[17:32:53] ======= [PASSED] ttm_bo_reserve_optimistic_no_ticket =======
[17:32:53] [PASSED] ttm_bo_reserve_locked_no_sleep
[17:32:53] [PASSED] ttm_bo_reserve_no_wait_ticket
[17:32:53] [PASSED] ttm_bo_reserve_double_resv
[17:32:53] [PASSED] ttm_bo_reserve_interrupted
[17:32:53] [PASSED] ttm_bo_reserve_deadlock
[17:32:53] [PASSED] ttm_bo_unreserve_basic
[17:32:53] [PASSED] ttm_bo_unreserve_pinned
[17:32:53] [PASSED] ttm_bo_unreserve_bulk
[17:32:53] [PASSED] ttm_bo_fini_basic
[17:32:53] [PASSED] ttm_bo_fini_shared_resv
[17:32:53] [PASSED] ttm_bo_pin_basic
[17:32:53] [PASSED] ttm_bo_pin_unpin_resource
[17:32:53] [PASSED] ttm_bo_multiple_pin_one_unpin
[17:32:53] ===================== [PASSED] ttm_bo ======================
[17:32:53] ============== ttm_bo_validate (21 subtests) ===============
[17:32:53] ============== ttm_bo_init_reserved_sys_man  ===============
[17:32:53] [PASSED] Buffer object for userspace
[17:32:53] [PASSED] Kernel buffer object
[17:32:53] [PASSED] Shared buffer object
[17:32:53] ========== [PASSED] ttm_bo_init_reserved_sys_man ===========
[17:32:53] ============== ttm_bo_init_reserved_mock_man  ==============
[17:32:53] [PASSED] Buffer object for userspace
[17:32:53] [PASSED] Kernel buffer object
[17:32:53] [PASSED] Shared buffer object
[17:32:53] ========== [PASSED] ttm_bo_init_reserved_mock_man ==========
[17:32:53] [PASSED] ttm_bo_init_reserved_resv
[17:32:53] ================== ttm_bo_validate_basic  ==================
[17:32:53] [PASSED] Buffer object for userspace
[17:32:53] [PASSED] Kernel buffer object
[17:32:53] [PASSED] Shared buffer object
[17:32:53] ============== [PASSED] ttm_bo_validate_basic ==============
[17:32:53] [PASSED] ttm_bo_validate_invalid_placement
[17:32:53] ============= ttm_bo_validate_same_placement  ==============
[17:32:53] [PASSED] System manager
[17:32:53] [PASSED] VRAM manager
[17:32:53] ========= [PASSED] ttm_bo_validate_same_placement ==========
[17:32:53] [PASSED] ttm_bo_validate_failed_alloc
[17:32:53] [PASSED] ttm_bo_validate_pinned
[17:32:53] [PASSED] ttm_bo_validate_busy_placement
[17:32:53] ================ ttm_bo_validate_multihop  =================
[17:32:53] [PASSED] Buffer object for userspace
[17:32:53] [PASSED] Kernel buffer object
[17:32:53] [PASSED] Shared buffer object
[17:32:53] ============ [PASSED] ttm_bo_validate_multihop =============
[17:32:53] ========== ttm_bo_validate_no_placement_signaled  ==========
[17:32:53] [PASSED] Buffer object in system domain, no page vector
[17:32:53] [PASSED] Buffer object in system domain with an existing page vector
[17:32:53] ====== [PASSED] ttm_bo_validate_no_placement_signaled ======
[17:32:53] ======== ttm_bo_validate_no_placement_not_signaled  ========
[17:32:53] [PASSED] Buffer object for userspace
[17:32:53] [PASSED] Kernel buffer object
[17:32:53] [PASSED] Shared buffer object
[17:32:53] ==== [PASSED] ttm_bo_validate_no_placement_not_signaled ====
[17:32:53] [PASSED] ttm_bo_validate_move_fence_signaled
[17:32:53] ========= ttm_bo_validate_move_fence_not_signaled  =========
[17:32:53] [PASSED] Waits for GPU
[17:32:53] [PASSED] Tries to lock straight away
[17:32:53] ===== [PASSED] ttm_bo_validate_move_fence_not_signaled =====
[17:32:53] [PASSED] ttm_bo_validate_happy_evict
[17:32:53] [PASSED] ttm_bo_validate_all_pinned_evict
[17:32:53] [PASSED] ttm_bo_validate_allowed_only_evict
[17:32:53] [PASSED] ttm_bo_validate_deleted_evict
[17:32:53] [PASSED] ttm_bo_validate_busy_domain_evict
[17:32:53] [PASSED] ttm_bo_validate_evict_gutting
[17:32:53] [PASSED] ttm_bo_validate_recrusive_evict
stty: 'standard input': Inappropriate ioctl for device
[17:32:53] ================= [PASSED] ttm_bo_validate =================
[17:32:53] ============================================================
[17:32:53] Testing complete. Ran 101 tests: passed: 101
[17:32:53] Elapsed time: 11.273s total, 1.613s configuring, 9.444s building, 0.185s running

+ cleanup
++ stat -c %u:%g /kernel
+ chown -R 1003:1003 /kernel



^ permalink raw reply	[flat|nested] 23+ messages in thread

* ✗ CI.checksparse: warning for drm/{i915, xe}: clean up and deduplicate initial plane handling
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
                   ` (16 preceding siblings ...)
  2025-12-15 17:32 ` ✓ CI.KUnit: success " Patchwork
@ 2025-12-15 17:48 ` Patchwork
  2025-12-15 18:20 ` ✓ Xe.CI.BAT: success " Patchwork
                   ` (2 subsequent siblings)
  20 siblings, 0 replies; 23+ messages in thread
From: Patchwork @ 2025-12-15 17:48 UTC (permalink / raw)
  To: Jani Nikula; +Cc: intel-xe

== Series Details ==

Series: drm/{i915, xe}: clean up and deduplicate initial plane handling
URL   : https://patchwork.freedesktop.org/series/159031/
State : warning

== Summary ==

+ trap cleanup EXIT
+ KERNEL=/kernel
+ MT=/root/linux/maintainer-tools
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools /root/linux/maintainer-tools
Cloning into '/root/linux/maintainer-tools'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ make -C /root/linux/maintainer-tools
make: Entering directory '/root/linux/maintainer-tools'
cc -O2 -g -Wextra -o remap-log remap-log.c
make: Leaving directory '/root/linux/maintainer-tools'
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ /root/linux/maintainer-tools/dim sparse --fast 342b565445c93d6fb5221aa9c23be20d65fa16f2
Sparse version: 0.6.4 (Ubuntu: 0.6.4-4ubuntu3)
Fast mode used, each commit won't be checked separately.
-
+drivers/gpu/drm/i915/display/intel_display.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_display_driver.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/display/intel_initial_plane.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/i915_initial_plane.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+drivers/gpu/drm/i915/i915_irq.c:467:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:467:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:475:16: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:475:16: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:480:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:480:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:480:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:518:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:518:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:526:16: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:526:16: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:531:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:531:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:531:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:575:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:575:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:578:15: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:578:15: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:582:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:582:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:589:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:589:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:589:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_irq.c:589:9: warning: unreplaced symbol '<noident>'
+drivers/gpu/drm/i915/i915_panic.c: note: in included file (through drivers/gpu/drm/i915/display/intel_display_types.h):
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression
+./include/linux/pwm.h:13:1: error: bad constant expression

+ cleanup
++ stat -c %u:%g /kernel
+ chown -R 1003:1003 /kernel



^ permalink raw reply	[flat|nested] 23+ messages in thread

* ✓ Xe.CI.BAT: success for drm/{i915, xe}: clean up and deduplicate initial plane handling
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
                   ` (17 preceding siblings ...)
  2025-12-15 17:48 ` ✗ CI.checksparse: warning " Patchwork
@ 2025-12-15 18:20 ` Patchwork
  2025-12-16  0:07 ` ✗ Xe.CI.Full: failure " Patchwork
  2025-12-19 20:17 ` [PATCH 00/15] " Ville Syrjälä
  20 siblings, 0 replies; 23+ messages in thread
From: Patchwork @ 2025-12-15 18:20 UTC (permalink / raw)
  To: Jani Nikula; +Cc: intel-xe

[-- Attachment #1: Type: text/plain, Size: 886 bytes --]

== Series Details ==

Series: drm/{i915, xe}: clean up and deduplicate initial plane handling
URL   : https://patchwork.freedesktop.org/series/159031/
State : success

== Summary ==

CI Bug Log - changes from xe-4245-53c678ed7aabc9f56529f8f15dde305ba7ac36ba_BAT -> xe-pw-159031v1_BAT
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  

Participating hosts (12 -> 12)
------------------------------

  No changes in participating hosts


Changes
-------

  No changes found


Build changes
-------------

  * Linux: xe-4245-53c678ed7aabc9f56529f8f15dde305ba7ac36ba -> xe-pw-159031v1

  IGT_8666: 8666
  xe-4245-53c678ed7aabc9f56529f8f15dde305ba7ac36ba: 53c678ed7aabc9f56529f8f15dde305ba7ac36ba
  xe-pw-159031v1: 159031v1

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/index.html

[-- Attachment #2: Type: text/html, Size: 1434 bytes --]

^ permalink raw reply	[flat|nested] 23+ messages in thread

* ✗ Xe.CI.Full: failure for drm/{i915, xe}: clean up and deduplicate initial plane handling
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
                   ` (18 preceding siblings ...)
  2025-12-15 18:20 ` ✓ Xe.CI.BAT: success " Patchwork
@ 2025-12-16  0:07 ` Patchwork
  2025-12-19 20:17 ` [PATCH 00/15] " Ville Syrjälä
  20 siblings, 0 replies; 23+ messages in thread
From: Patchwork @ 2025-12-16  0:07 UTC (permalink / raw)
  To: Jani Nikula; +Cc: intel-xe

[-- Attachment #1: Type: text/plain, Size: 25348 bytes --]

== Series Details ==

Series: drm/{i915, xe}: clean up and deduplicate initial plane handling
URL   : https://patchwork.freedesktop.org/series/159031/
State : failure

== Summary ==

CI Bug Log - changes from xe-4245-53c678ed7aabc9f56529f8f15dde305ba7ac36ba_FULL -> xe-pw-159031v1_FULL
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with xe-pw-159031v1_FULL absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in xe-pw-159031v1_FULL, please notify your bug team (I915-ci-infra@lists.freedesktop.org) to allow them
  to document this new failure mode, which will reduce false positives in CI.

  

Participating hosts (2 -> 2)
------------------------------

  No changes in participating hosts

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in xe-pw-159031v1_FULL:

### IGT changes ###

#### Possible regressions ####

  * igt@kms_hdr@bpc-switch@pipe-a-dp-2:
    - shard-bmg:          NOTRUN -> [ABORT][1] +3 other tests abort
   [1]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-8/igt@kms_hdr@bpc-switch@pipe-a-dp-2.html

  
Known issues
------------

  Here are the changes found in xe-pw-159031v1_FULL that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@kms_big_fb@linear-32bpp-rotate-270:
    - shard-bmg:          NOTRUN -> [SKIP][2] ([Intel XE#2327]) +2 other tests skip
   [2]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@kms_big_fb@linear-32bpp-rotate-270.html

  * igt@kms_big_fb@y-tiled-16bpp-rotate-270:
    - shard-bmg:          NOTRUN -> [SKIP][3] ([Intel XE#1124]) +14 other tests skip
   [3]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@kms_big_fb@y-tiled-16bpp-rotate-270.html

  * igt@kms_big_fb@y-tiled-addfb-size-overflow:
    - shard-bmg:          NOTRUN -> [SKIP][4] ([Intel XE#610])
   [4]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_big_fb@y-tiled-addfb-size-overflow.html

  * igt@kms_big_fb@yf-tiled-addfb-size-offset-overflow:
    - shard-bmg:          NOTRUN -> [SKIP][5] ([Intel XE#607])
   [5]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@kms_big_fb@yf-tiled-addfb-size-offset-overflow.html

  * igt@kms_bw@connected-linear-tiling-3-displays-2560x1440p:
    - shard-bmg:          NOTRUN -> [SKIP][6] ([Intel XE#2314] / [Intel XE#2894]) +3 other tests skip
   [6]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_bw@connected-linear-tiling-3-displays-2560x1440p.html

  * igt@kms_bw@linear-tiling-2-displays-1920x1080p:
    - shard-bmg:          NOTRUN -> [SKIP][7] ([Intel XE#367]) +1 other test skip
   [7]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@kms_bw@linear-tiling-2-displays-1920x1080p.html

  * igt@kms_ccs@bad-rotation-90-4-tiled-lnl-ccs@pipe-c-dp-2:
    - shard-bmg:          NOTRUN -> [SKIP][8] ([Intel XE#2652] / [Intel XE#787]) +8 other tests skip
   [8]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@kms_ccs@bad-rotation-90-4-tiled-lnl-ccs@pipe-c-dp-2.html

  * igt@kms_ccs@crc-primary-suspend-4-tiled-dg2-mc-ccs:
    - shard-bmg:          NOTRUN -> [SKIP][9] ([Intel XE#3432]) +3 other tests skip
   [9]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_ccs@crc-primary-suspend-4-tiled-dg2-mc-ccs.html

  * igt@kms_ccs@random-ccs-data-4-tiled-dg2-rc-ccs:
    - shard-bmg:          NOTRUN -> [SKIP][10] ([Intel XE#2887]) +21 other tests skip
   [10]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@kms_ccs@random-ccs-data-4-tiled-dg2-rc-ccs.html

  * igt@kms_cdclk@mode-transition-all-outputs:
    - shard-bmg:          NOTRUN -> [SKIP][11] ([Intel XE#2724]) +2 other tests skip
   [11]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_cdclk@mode-transition-all-outputs.html

  * igt@kms_chamelium_color@ctm-0-50:
    - shard-bmg:          NOTRUN -> [SKIP][12] ([Intel XE#2325]) +1 other test skip
   [12]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-5/igt@kms_chamelium_color@ctm-0-50.html

  * igt@kms_chamelium_edid@dp-edid-change-during-hibernate:
    - shard-bmg:          NOTRUN -> [SKIP][13] ([Intel XE#2252]) +12 other tests skip
   [13]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_chamelium_edid@dp-edid-change-during-hibernate.html

  * igt@kms_content_protection@atomic:
    - shard-bmg:          NOTRUN -> [FAIL][14] ([Intel XE#1178]) +3 other tests fail
   [14]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@kms_content_protection@atomic.html

  * igt@kms_content_protection@content-type-change:
    - shard-bmg:          NOTRUN -> [SKIP][15] ([Intel XE#2341]) +1 other test skip
   [15]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_content_protection@content-type-change.html

  * igt@kms_cursor_crc@cursor-offscreen-128x42:
    - shard-bmg:          NOTRUN -> [SKIP][16] ([Intel XE#2320]) +9 other tests skip
   [16]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@kms_cursor_crc@cursor-offscreen-128x42.html

  * igt@kms_cursor_crc@cursor-sliding-512x512:
    - shard-bmg:          NOTRUN -> [SKIP][17] ([Intel XE#2321])
   [17]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_cursor_crc@cursor-sliding-512x512.html

  * igt@kms_cursor_legacy@cursor-vs-flip-atomic-transitions-varying-size:
    - shard-bmg:          [PASS][18] -> [DMESG-WARN][19] ([Intel XE#5354])
   [18]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4245-53c678ed7aabc9f56529f8f15dde305ba7ac36ba/shard-bmg-3/igt@kms_cursor_legacy@cursor-vs-flip-atomic-transitions-varying-size.html
   [19]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@kms_cursor_legacy@cursor-vs-flip-atomic-transitions-varying-size.html

  * igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions:
    - shard-bmg:          [PASS][20] -> [FAIL][21] ([Intel XE#5299])
   [20]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4245-53c678ed7aabc9f56529f8f15dde305ba7ac36ba/shard-bmg-3/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions.html
   [21]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions.html

  * igt@kms_dither@fb-8bpc-vs-panel-6bpc@pipe-a-hdmi-a-3:
    - shard-bmg:          NOTRUN -> [SKIP][22] ([Intel XE#1340])
   [22]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-5/igt@kms_dither@fb-8bpc-vs-panel-6bpc@pipe-a-hdmi-a-3.html

  * igt@kms_dp_link_training@non-uhbr-mst:
    - shard-bmg:          NOTRUN -> [SKIP][23] ([Intel XE#4354])
   [23]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@kms_dp_link_training@non-uhbr-mst.html

  * igt@kms_dp_linktrain_fallback@dsc-fallback:
    - shard-bmg:          NOTRUN -> [SKIP][24] ([Intel XE#4331])
   [24]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@kms_dp_linktrain_fallback@dsc-fallback.html

  * igt@kms_dsc@dsc-with-output-formats:
    - shard-bmg:          NOTRUN -> [SKIP][25] ([Intel XE#2244]) +2 other tests skip
   [25]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-5/igt@kms_dsc@dsc-with-output-formats.html

  * igt@kms_fbc_dirty_rect@fbc-dirty-rectangle-dirtyfb-tests:
    - shard-bmg:          NOTRUN -> [SKIP][26] ([Intel XE#4422]) +1 other test skip
   [26]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-5/igt@kms_fbc_dirty_rect@fbc-dirty-rectangle-dirtyfb-tests.html

  * igt@kms_fbcon_fbt@psr-suspend:
    - shard-bmg:          NOTRUN -> [SKIP][27] ([Intel XE#776])
   [27]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_fbcon_fbt@psr-suspend.html

  * igt@kms_flip@2x-flip-vs-expired-vblank-interruptible:
    - shard-bmg:          [PASS][28] -> [FAIL][29] ([Intel XE#3321]) +1 other test fail
   [28]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4245-53c678ed7aabc9f56529f8f15dde305ba7ac36ba/shard-bmg-1/igt@kms_flip@2x-flip-vs-expired-vblank-interruptible.html
   [29]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_flip@2x-flip-vs-expired-vblank-interruptible.html

  * igt@kms_flip@flip-vs-expired-vblank-interruptible:
    - shard-lnl:          [PASS][30] -> [FAIL][31] ([Intel XE#301] / [Intel XE#3149]) +1 other test fail
   [30]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4245-53c678ed7aabc9f56529f8f15dde305ba7ac36ba/shard-lnl-8/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
   [31]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-lnl-5/igt@kms_flip@flip-vs-expired-vblank-interruptible.html

  * igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1:
    - shard-lnl:          [PASS][32] -> [FAIL][33] ([Intel XE#301])
   [32]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4245-53c678ed7aabc9f56529f8f15dde305ba7ac36ba/shard-lnl-8/igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1.html
   [33]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-lnl-5/igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1.html

  * igt@kms_flip_scaled_crc@flip-32bpp-yftileccs-to-64bpp-yftile-upscaling:
    - shard-bmg:          NOTRUN -> [SKIP][34] ([Intel XE#2293] / [Intel XE#2380]) +7 other tests skip
   [34]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_flip_scaled_crc@flip-32bpp-yftileccs-to-64bpp-yftile-upscaling.html

  * igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs-downscaling@pipe-a-valid-mode:
    - shard-bmg:          NOTRUN -> [SKIP][35] ([Intel XE#2293]) +7 other tests skip
   [35]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-5/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs-downscaling@pipe-a-valid-mode.html

  * igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-onoff:
    - shard-bmg:          NOTRUN -> [SKIP][36] ([Intel XE#4141]) +22 other tests skip
   [36]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-onoff.html

  * igt@kms_frontbuffer_tracking@fbc-tiling-y:
    - shard-bmg:          NOTRUN -> [SKIP][37] ([Intel XE#2352])
   [37]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_frontbuffer_tracking@fbc-tiling-y.html

  * igt@kms_frontbuffer_tracking@fbcdrrs-2p-scndscrn-pri-shrfb-draw-render:
    - shard-bmg:          NOTRUN -> [SKIP][38] ([Intel XE#2311]) +49 other tests skip
   [38]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_frontbuffer_tracking@fbcdrrs-2p-scndscrn-pri-shrfb-draw-render.html

  * igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-blt:
    - shard-bmg:          NOTRUN -> [SKIP][39] ([Intel XE#2313]) +43 other tests skip
   [39]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-5/igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-blt.html

  * igt@kms_plane_lowres@tiling-yf:
    - shard-bmg:          NOTRUN -> [SKIP][40] ([Intel XE#2393])
   [40]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-5/igt@kms_plane_lowres@tiling-yf.html

  * igt@kms_plane_multiple@tiling-yf:
    - shard-bmg:          NOTRUN -> [SKIP][41] ([Intel XE#5020]) +1 other test skip
   [41]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@kms_plane_multiple@tiling-yf.html

  * igt@kms_pm_backlight@bad-brightness:
    - shard-bmg:          NOTRUN -> [SKIP][42] ([Intel XE#870])
   [42]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_pm_backlight@bad-brightness.html

  * igt@kms_pm_rpm@dpms-mode-unset-lpsp:
    - shard-bmg:          NOTRUN -> [SKIP][43] ([Intel XE#1439] / [Intel XE#836])
   [43]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_pm_rpm@dpms-mode-unset-lpsp.html

  * igt@kms_pm_rpm@modeset-lpsp:
    - shard-bmg:          NOTRUN -> [SKIP][44] ([Intel XE#1439] / [Intel XE#3141] / [Intel XE#836])
   [44]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_pm_rpm@modeset-lpsp.html

  * igt@kms_pm_rpm@package-g7:
    - shard-bmg:          NOTRUN -> [SKIP][45] ([Intel XE#6814])
   [45]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@kms_pm_rpm@package-g7.html

  * igt@kms_psr2_sf@fbc-pr-cursor-plane-move-continuous-exceed-fully-sf:
    - shard-bmg:          NOTRUN -> [SKIP][46] ([Intel XE#1406] / [Intel XE#1489]) +11 other tests skip
   [46]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-5/igt@kms_psr2_sf@fbc-pr-cursor-plane-move-continuous-exceed-fully-sf.html

  * igt@kms_psr2_su@frontbuffer-xrgb8888:
    - shard-bmg:          NOTRUN -> [SKIP][47] ([Intel XE#1406] / [Intel XE#2387])
   [47]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_psr2_su@frontbuffer-xrgb8888.html

  * igt@kms_psr@fbc-psr2-cursor-plane-move:
    - shard-bmg:          NOTRUN -> [SKIP][48] ([Intel XE#1406] / [Intel XE#2234] / [Intel XE#2850]) +18 other tests skip
   [48]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_psr@fbc-psr2-cursor-plane-move.html

  * igt@kms_psr_stress_test@flip-primary-invalidate-overlay:
    - shard-bmg:          NOTRUN -> [SKIP][49] ([Intel XE#1406] / [Intel XE#2414])
   [49]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_psr_stress_test@flip-primary-invalidate-overlay.html

  * igt@kms_rotation_crc@primary-y-tiled-reflect-x-180:
    - shard-bmg:          NOTRUN -> [SKIP][50] ([Intel XE#2330])
   [50]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-8/igt@kms_rotation_crc@primary-y-tiled-reflect-x-180.html

  * igt@kms_rotation_crc@sprite-rotation-90:
    - shard-bmg:          NOTRUN -> [SKIP][51] ([Intel XE#3414] / [Intel XE#3904]) +1 other test skip
   [51]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@kms_rotation_crc@sprite-rotation-90.html

  * igt@kms_setmode@basic-clone-single-crtc:
    - shard-bmg:          NOTRUN -> [SKIP][52] ([Intel XE#1435])
   [52]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-5/igt@kms_setmode@basic-clone-single-crtc.html

  * igt@kms_sharpness_filter@invalid-filter-with-plane:
    - shard-bmg:          NOTRUN -> [SKIP][53] ([Intel XE#6503]) +4 other tests skip
   [53]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@kms_sharpness_filter@invalid-filter-with-plane.html

  * igt@kms_vrr@flip-basic:
    - shard-bmg:          NOTRUN -> [SKIP][54] ([Intel XE#1499]) +3 other tests skip
   [54]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@kms_vrr@flip-basic.html

  * igt@kms_vrr@lobf:
    - shard-bmg:          NOTRUN -> [SKIP][55] ([Intel XE#2168]) +1 other test skip
   [55]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-5/igt@kms_vrr@lobf.html

  * igt@xe_eudebug@basic-vm-bind-metadata-discovery:
    - shard-bmg:          NOTRUN -> [SKIP][56] ([Intel XE#4837]) +14 other tests skip
   [56]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@xe_eudebug@basic-vm-bind-metadata-discovery.html

  * igt@xe_eudebug_online@pagefault-write:
    - shard-bmg:          NOTRUN -> [SKIP][57] ([Intel XE#4837] / [Intel XE#6665]) +8 other tests skip
   [57]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-8/igt@xe_eudebug_online@pagefault-write.html

  * igt@xe_exec_basic@multigpu-once-null-rebind:
    - shard-bmg:          NOTRUN -> [SKIP][58] ([Intel XE#2322]) +16 other tests skip
   [58]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@xe_exec_basic@multigpu-once-null-rebind.html

  * igt@xe_exec_multi_queue@few-execs-preempt-mode-close-fd-smem:
    - shard-bmg:          NOTRUN -> [SKIP][59] ([Intel XE#6874]) +44 other tests skip
   [59]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-5/igt@xe_exec_multi_queue@few-execs-preempt-mode-close-fd-smem.html

  * igt@xe_exec_system_allocator@many-64k-mmap-free-huge:
    - shard-bmg:          NOTRUN -> [SKIP][60] ([Intel XE#5007])
   [60]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-5/igt@xe_exec_system_allocator@many-64k-mmap-free-huge.html

  * igt@xe_exec_system_allocator@threads-many-mmap-new-huge-nomemset:
    - shard-bmg:          NOTRUN -> [SKIP][61] ([Intel XE#4943]) +38 other tests skip
   [61]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@xe_exec_system_allocator@threads-many-mmap-new-huge-nomemset.html

  * igt@xe_module_load@force-load:
    - shard-bmg:          NOTRUN -> [SKIP][62] ([Intel XE#2457])
   [62]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-5/igt@xe_module_load@force-load.html

  * igt@xe_pat@pat-index-xehpc:
    - shard-bmg:          NOTRUN -> [SKIP][63] ([Intel XE#1420])
   [63]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-5/igt@xe_pat@pat-index-xehpc.html

  * igt@xe_pat@pat-index-xelp:
    - shard-bmg:          NOTRUN -> [SKIP][64] ([Intel XE#2245])
   [64]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@xe_pat@pat-index-xelp.html

  * igt@xe_peer2peer@read:
    - shard-bmg:          NOTRUN -> [SKIP][65] ([Intel XE#2427])
   [65]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-1/igt@xe_peer2peer@read.html

  * igt@xe_pm@d3cold-mmap-vram:
    - shard-bmg:          NOTRUN -> [SKIP][66] ([Intel XE#2284])
   [66]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@xe_pm@d3cold-mmap-vram.html

  * igt@xe_pmu@engine-activity-accuracy-90@engine-drm_xe_engine_class_copy0:
    - shard-lnl:          [PASS][67] -> [FAIL][68] ([Intel XE#6251])
   [67]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4245-53c678ed7aabc9f56529f8f15dde305ba7ac36ba/shard-lnl-5/igt@xe_pmu@engine-activity-accuracy-90@engine-drm_xe_engine_class_copy0.html
   [68]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-lnl-4/igt@xe_pmu@engine-activity-accuracy-90@engine-drm_xe_engine_class_copy0.html

  * igt@xe_pxp@pxp-termination-key-update-post-rpm:
    - shard-bmg:          NOTRUN -> [SKIP][69] ([Intel XE#4733]) +2 other tests skip
   [69]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-5/igt@xe_pxp@pxp-termination-key-update-post-rpm.html

  * igt@xe_query@multigpu-query-mem-usage:
    - shard-bmg:          NOTRUN -> [SKIP][70] ([Intel XE#944]) +5 other tests skip
   [70]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-3/igt@xe_query@multigpu-query-mem-usage.html

  
#### Possible fixes ####

  * igt@kms_cursor_legacy@cursor-vs-flip-varying-size:
    - shard-bmg:          [DMESG-WARN][71] ([Intel XE#5354]) -> [PASS][72]
   [71]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4245-53c678ed7aabc9f56529f8f15dde305ba7ac36ba/shard-bmg-5/igt@kms_cursor_legacy@cursor-vs-flip-varying-size.html
   [72]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-bmg-2/igt@kms_cursor_legacy@cursor-vs-flip-varying-size.html

  * igt@kms_vrr@flipline:
    - shard-lnl:          [FAIL][73] ([Intel XE#4227]) -> [PASS][74] +1 other test pass
   [73]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4245-53c678ed7aabc9f56529f8f15dde305ba7ac36ba/shard-lnl-1/igt@kms_vrr@flipline.html
   [74]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/shard-lnl-8/igt@kms_vrr@flipline.html

  
  [Intel XE#1124]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1124
  [Intel XE#1178]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1178
  [Intel XE#1340]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1340
  [Intel XE#1406]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1406
  [Intel XE#1420]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1420
  [Intel XE#1435]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1435
  [Intel XE#1439]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1439
  [Intel XE#1489]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1489
  [Intel XE#1499]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1499
  [Intel XE#2168]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2168
  [Intel XE#2234]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2234
  [Intel XE#2244]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2244
  [Intel XE#2245]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2245
  [Intel XE#2252]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2252
  [Intel XE#2284]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2284
  [Intel XE#2293]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2293
  [Intel XE#2311]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2311
  [Intel XE#2313]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2313
  [Intel XE#2314]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2314
  [Intel XE#2320]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2320
  [Intel XE#2321]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2321
  [Intel XE#2322]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2322
  [Intel XE#2325]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2325
  [Intel XE#2327]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2327
  [Intel XE#2330]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2330
  [Intel XE#2341]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2341
  [Intel XE#2352]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2352
  [Intel XE#2380]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2380
  [Intel XE#2387]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2387
  [Intel XE#2393]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2393
  [Intel XE#2414]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2414
  [Intel XE#2427]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2427
  [Intel XE#2457]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2457
  [Intel XE#2652]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2652
  [Intel XE#2724]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2724
  [Intel XE#2850]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2850
  [Intel XE#2887]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2887
  [Intel XE#2894]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2894
  [Intel XE#301]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/301
  [Intel XE#3141]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3141
  [Intel XE#3149]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3149
  [Intel XE#3321]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3321
  [Intel XE#3414]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3414
  [Intel XE#3432]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3432
  [Intel XE#367]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/367
  [Intel XE#3904]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3904
  [Intel XE#4141]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4141
  [Intel XE#4227]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4227
  [Intel XE#4331]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4331
  [Intel XE#4354]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4354
  [Intel XE#4422]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4422
  [Intel XE#4733]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4733
  [Intel XE#4837]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4837
  [Intel XE#4943]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4943
  [Intel XE#5007]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5007
  [Intel XE#5020]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5020
  [Intel XE#5299]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5299
  [Intel XE#5354]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5354
  [Intel XE#607]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/607
  [Intel XE#610]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/610
  [Intel XE#6251]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6251
  [Intel XE#6503]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6503
  [Intel XE#6665]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6665
  [Intel XE#6814]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6814
  [Intel XE#6874]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6874
  [Intel XE#776]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/776
  [Intel XE#787]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/787
  [Intel XE#836]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/836
  [Intel XE#870]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/870
  [Intel XE#944]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/944


Build changes
-------------

  * Linux: xe-4245-53c678ed7aabc9f56529f8f15dde305ba7ac36ba -> xe-pw-159031v1

  IGT_8666: 8666
  xe-4245-53c678ed7aabc9f56529f8f15dde305ba7ac36ba: 53c678ed7aabc9f56529f8f15dde305ba7ac36ba
  xe-pw-159031v1: 159031v1

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159031v1/index.html

[-- Attachment #2: Type: text/html, Size: 27974 bytes --]

^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling
  2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
                   ` (19 preceding siblings ...)
  2025-12-16  0:07 ` ✗ Xe.CI.Full: failure " Patchwork
@ 2025-12-19 20:17 ` Ville Syrjälä
  2025-12-22 13:36   ` Jani Nikula
  20 siblings, 1 reply; 23+ messages in thread
From: Ville Syrjälä @ 2025-12-19 20:17 UTC (permalink / raw)
  To: Jani Nikula; +Cc: intel-gfx, intel-xe

On Mon, Dec 15, 2025 at 05:28:14PM +0200, Jani Nikula wrote:
> The i915 and xe initial plane handling deviate at a too high level,
> leading to lots of duplication between the two. This series starts to
> clean it up by moving it to display parent interface, and deduplicating
> piecemeal.
> 
> This is intentionally done in small chunks to ease review and catch
> regressions (hopefully none).
> 
> There's still more to be done, e.g. it's pointless to have both i915 and
> xe call intel_framebuffer_init(), but there's some error path stuff to
> figure out before doing this. And I ran out of steam a bit, and the
> series got pretty long already.
> 
> Anyway, by reducing duplication, the series highlights the differences
> between i915 and xe.
> 
> BR,
> Jani.
> 
> 
> Jani Nikula (15):
>   drm/i915: move display/intel_plane_initial.c to i915_initial_plane.c
>   drm/xe/display: rename xe_plane_initial.c to xe_initial_plane.c
>   drm/i915: rename intel_plane_initial.h to intel_initial_plane.h
>   drm/{i915,xe}: move initial plane calls to parent interface
>   drm/{i915,xe}: deduplicate intel_initial_plane_config() between i915
>     and xe
>   drm/{i915,xe}: deduplicate plane_config_fini() between i915 and xe
>   drm/{i915,xe}: start deduplicating intel_find_initial_plane_obj()
>     between i915 and xe
>   drm/i915: return plane_state from intel_reuse_initial_plane_obj()
>   drm/xe: return plane_state from intel_reuse_initial_plane_obj()
>   drm/i915: further deduplicate intel_find_initial_plane_obj()
>   drm/{i915,xe}: deduplicate intel_alloc_initial_plane_obj() FB modifier
>     checks
>   drm/{i915,xe}: deduplicate initial plane setup
>   drm/{i915,xe}: pass struct drm_plane_state instead of struct drm_crtc
>     to ->setup
>   drm/{i915,xe}: pass struct drm_device instead of drm_device to
>     ->alloc_obj
>   drm/i915: drop dependency on struct intel_display from i915 initial
>     plane

Everything looked quite reasonable. Series is
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

> 
>  drivers/gpu/drm/i915/Makefile                 |   3 +-
>  drivers/gpu/drm/i915/display/intel_display.c  |   8 +-
>  .../drm/i915/display/intel_display_driver.c   |   2 +-
>  .../drm/i915/display/intel_initial_plane.c    | 193 ++++++++
>  ..._plane_initial.h => intel_initial_plane.h} |   6 +-
>  .../drm/i915/display/intel_plane_initial.c    | 442 ------------------
>  drivers/gpu/drm/i915/i915_driver.c            |   2 +
>  drivers/gpu/drm/i915/i915_initial_plane.c     | 290 ++++++++++++
>  drivers/gpu/drm/i915/i915_initial_plane.h     |   9 +
>  drivers/gpu/drm/xe/Makefile                   |   3 +-
>  drivers/gpu/drm/xe/display/xe_display.c       |   2 +
>  drivers/gpu/drm/xe/display/xe_initial_plane.c | 189 ++++++++
>  drivers/gpu/drm/xe/display/xe_initial_plane.h |   9 +
>  drivers/gpu/drm/xe/display/xe_plane_initial.c | 321 -------------
>  include/drm/intel/display_parent_interface.h  |  17 +
>  15 files changed, 723 insertions(+), 773 deletions(-)
>  create mode 100644 drivers/gpu/drm/i915/display/intel_initial_plane.c
>  rename drivers/gpu/drm/i915/display/{intel_plane_initial.h => intel_initial_plane.h} (60%)
>  delete mode 100644 drivers/gpu/drm/i915/display/intel_plane_initial.c
>  create mode 100644 drivers/gpu/drm/i915/i915_initial_plane.c
>  create mode 100644 drivers/gpu/drm/i915/i915_initial_plane.h
>  create mode 100644 drivers/gpu/drm/xe/display/xe_initial_plane.c
>  create mode 100644 drivers/gpu/drm/xe/display/xe_initial_plane.h
>  delete mode 100644 drivers/gpu/drm/xe/display/xe_plane_initial.c
> 
> -- 
> 2.47.3

-- 
Ville Syrjälä
Intel

^ permalink raw reply	[flat|nested] 23+ messages in thread

* Re: [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling
  2025-12-19 20:17 ` [PATCH 00/15] " Ville Syrjälä
@ 2025-12-22 13:36   ` Jani Nikula
  0 siblings, 0 replies; 23+ messages in thread
From: Jani Nikula @ 2025-12-22 13:36 UTC (permalink / raw)
  To: Ville Syrjälä; +Cc: intel-gfx, intel-xe

On Fri, 19 Dec 2025, Ville Syrjälä <ville.syrjala@linux.intel.com> wrote:
> On Mon, Dec 15, 2025 at 05:28:14PM +0200, Jani Nikula wrote:
>> The i915 and xe initial plane handling deviate at a too high level,
>> leading to lots of duplication between the two. This series starts to
>> clean it up by moving it to display parent interface, and deduplicating
>> piecemeal.
>> 
>> This is intentionally done in small chunks to ease review and catch
>> regressions (hopefully none).
>> 
>> There's still more to be done, e.g. it's pointless to have both i915 and
>> xe call intel_framebuffer_init(), but there's some error path stuff to
>> figure out before doing this. And I ran out of steam a bit, and the
>> series got pretty long already.
>> 
>> Anyway, by reducing duplication, the series highlights the differences
>> between i915 and xe.
>> 
>> BR,
>> Jani.
>> 
>> 
>> Jani Nikula (15):
>>   drm/i915: move display/intel_plane_initial.c to i915_initial_plane.c
>>   drm/xe/display: rename xe_plane_initial.c to xe_initial_plane.c
>>   drm/i915: rename intel_plane_initial.h to intel_initial_plane.h
>>   drm/{i915,xe}: move initial plane calls to parent interface
>>   drm/{i915,xe}: deduplicate intel_initial_plane_config() between i915
>>     and xe
>>   drm/{i915,xe}: deduplicate plane_config_fini() between i915 and xe
>>   drm/{i915,xe}: start deduplicating intel_find_initial_plane_obj()
>>     between i915 and xe
>>   drm/i915: return plane_state from intel_reuse_initial_plane_obj()
>>   drm/xe: return plane_state from intel_reuse_initial_plane_obj()
>>   drm/i915: further deduplicate intel_find_initial_plane_obj()
>>   drm/{i915,xe}: deduplicate intel_alloc_initial_plane_obj() FB modifier
>>     checks
>>   drm/{i915,xe}: deduplicate initial plane setup
>>   drm/{i915,xe}: pass struct drm_plane_state instead of struct drm_crtc
>>     to ->setup
>>   drm/{i915,xe}: pass struct drm_device instead of drm_device to
>>     ->alloc_obj
>>   drm/i915: drop dependency on struct intel_display from i915 initial
>>     plane
>
> Everything looked quite reasonable. Series is
> Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

Thanks, pushed the lot to din.

BR,
Jani.


>
>> 
>>  drivers/gpu/drm/i915/Makefile                 |   3 +-
>>  drivers/gpu/drm/i915/display/intel_display.c  |   8 +-
>>  .../drm/i915/display/intel_display_driver.c   |   2 +-
>>  .../drm/i915/display/intel_initial_plane.c    | 193 ++++++++
>>  ..._plane_initial.h => intel_initial_plane.h} |   6 +-
>>  .../drm/i915/display/intel_plane_initial.c    | 442 ------------------
>>  drivers/gpu/drm/i915/i915_driver.c            |   2 +
>>  drivers/gpu/drm/i915/i915_initial_plane.c     | 290 ++++++++++++
>>  drivers/gpu/drm/i915/i915_initial_plane.h     |   9 +
>>  drivers/gpu/drm/xe/Makefile                   |   3 +-
>>  drivers/gpu/drm/xe/display/xe_display.c       |   2 +
>>  drivers/gpu/drm/xe/display/xe_initial_plane.c | 189 ++++++++
>>  drivers/gpu/drm/xe/display/xe_initial_plane.h |   9 +
>>  drivers/gpu/drm/xe/display/xe_plane_initial.c | 321 -------------
>>  include/drm/intel/display_parent_interface.h  |  17 +
>>  15 files changed, 723 insertions(+), 773 deletions(-)
>>  create mode 100644 drivers/gpu/drm/i915/display/intel_initial_plane.c
>>  rename drivers/gpu/drm/i915/display/{intel_plane_initial.h => intel_initial_plane.h} (60%)
>>  delete mode 100644 drivers/gpu/drm/i915/display/intel_plane_initial.c
>>  create mode 100644 drivers/gpu/drm/i915/i915_initial_plane.c
>>  create mode 100644 drivers/gpu/drm/i915/i915_initial_plane.h
>>  create mode 100644 drivers/gpu/drm/xe/display/xe_initial_plane.c
>>  create mode 100644 drivers/gpu/drm/xe/display/xe_initial_plane.h
>>  delete mode 100644 drivers/gpu/drm/xe/display/xe_plane_initial.c
>> 
>> -- 
>> 2.47.3

-- 
Jani Nikula, Intel

^ permalink raw reply	[flat|nested] 23+ messages in thread

end of thread, other threads:[~2025-12-22 13:37 UTC | newest]

Thread overview: 23+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-12-15 15:28 [PATCH 00/15] drm/{i915, xe}: clean up and deduplicate initial plane handling Jani Nikula
2025-12-15 15:28 ` [PATCH 01/15] drm/i915: move display/intel_plane_initial.c to i915_initial_plane.c Jani Nikula
2025-12-15 15:28 ` [PATCH 02/15] drm/xe/display: rename xe_plane_initial.c to xe_initial_plane.c Jani Nikula
2025-12-15 15:28 ` [PATCH 03/15] drm/i915: rename intel_plane_initial.h to intel_initial_plane.h Jani Nikula
2025-12-15 15:28 ` [PATCH 04/15] drm/{i915, xe}: move initial plane calls to parent interface Jani Nikula
2025-12-15 15:28 ` [PATCH 05/15] drm/{i915, xe}: deduplicate intel_initial_plane_config() between i915 and xe Jani Nikula
2025-12-15 15:28 ` [PATCH 06/15] drm/{i915, xe}: deduplicate plane_config_fini() " Jani Nikula
2025-12-15 15:28 ` [PATCH 07/15] drm/{i915, xe}: start deduplicating intel_find_initial_plane_obj() " Jani Nikula
2025-12-15 15:28 ` [PATCH 08/15] drm/i915: return plane_state from intel_reuse_initial_plane_obj() Jani Nikula
2025-12-15 15:28 ` [PATCH 09/15] drm/xe: " Jani Nikula
2025-12-15 15:28 ` [PATCH 10/15] drm/i915: further deduplicate intel_find_initial_plane_obj() Jani Nikula
2025-12-15 15:28 ` [PATCH 11/15] drm/{i915, xe}: deduplicate intel_alloc_initial_plane_obj() FB modifier checks Jani Nikula
2025-12-15 15:28 ` [PATCH 12/15] drm/{i915,xe}: deduplicate initial plane setup Jani Nikula
2025-12-15 15:28 ` [PATCH 13/15] drm/{i915, xe}: pass struct drm_plane_state instead of struct drm_crtc to ->setup Jani Nikula
2025-12-15 15:28 ` [PATCH 14/15] drm/{i915, xe}: pass struct drm_device instead of drm_device to ->alloc_obj Jani Nikula
2025-12-15 15:28 ` [PATCH 15/15] drm/i915: drop dependency on struct intel_display from i915 initial plane Jani Nikula
2025-12-15 17:31 ` ✗ CI.checkpatch: warning for drm/{i915, xe}: clean up and deduplicate initial plane handling Patchwork
2025-12-15 17:32 ` ✓ CI.KUnit: success " Patchwork
2025-12-15 17:48 ` ✗ CI.checksparse: warning " Patchwork
2025-12-15 18:20 ` ✓ Xe.CI.BAT: success " Patchwork
2025-12-16  0:07 ` ✗ Xe.CI.Full: failure " Patchwork
2025-12-19 20:17 ` [PATCH 00/15] " Ville Syrjälä
2025-12-22 13:36   ` Jani Nikula

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).