[PATCH 1/9] drm/xe: Remove double pageflip

[PATCH 1/9] drm/xe: Remove double pageflip

[Maarten Lankhorst]

This is already handled below by fixup_initial_plane_config.

Fixes: a8153627520a ("drm/i915: Try to relocate the BIOS fb to the start of ggtt")
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Reviewed-by: Vinod Govindapillai <vinod.govindapillai@intel.com>
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
---
 drivers/gpu/drm/xe/display/xe_plane_initial.c | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/drivers/gpu/drm/xe/display/xe_plane_initial.c b/drivers/gpu/drm/xe/display/xe_plane_initial.c
index 8c113463a3d55..668f544a7ac80 100644
--- a/drivers/gpu/drm/xe/display/xe_plane_initial.c
+++ b/drivers/gpu/drm/xe/display/xe_plane_initial.c
@@ -194,8 +194,6 @@ intel_find_initial_plane_obj(struct intel_crtc *crtc,
 		to_intel_plane(crtc->base.primary);
 	struct intel_plane_state *plane_state =
 		to_intel_plane_state(plane->base.state);
-	struct intel_crtc_state *crtc_state =
-		to_intel_crtc_state(crtc->base.state);
 	struct drm_framebuffer *fb;
 	struct i915_vma *vma;
 
@@ -241,14 +239,6 @@ intel_find_initial_plane_obj(struct intel_crtc *crtc,
 	atomic_or(plane->frontbuffer_bit, &to_intel_frontbuffer(fb)->bits);
 
 	plane_config->vma = vma;
-
-	/*
-	 * Flip to the newly created mapping ASAP, so we can re-use the
-	 * first part of GGTT for WOPCM, prevent flickering, and prevent
-	 * the lookup of sysmem scratch pages.
-	 */
-	plane->check_plane(crtc_state, plane_state);
-	plane->async_flip(NULL, plane, crtc_state, plane_state, true);
 	return;
 
 nofb:
-- 
2.45.2

[PATCH 2/9] drm/xe/display: Add intel_plane_initial_vblank_wait

[Maarten Lankhorst]

We're changing the driver to have no interrupts during early init for
Xe, so we poll the PIPE_FRMSTMSMP counter instead.

Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
---
 drivers/gpu/drm/i915/display/intel_display.c  |  6 +++---
 .../drm/i915/display/intel_plane_initial.c    |  9 ++++++++-
 .../drm/i915/display/intel_plane_initial.h    |  1 +
 drivers/gpu/drm/xe/display/xe_plane_initial.c | 19 ++++++++++++++++++-
 4 files changed, 30 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
index 863927f429aa7..029a8583b0548 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -795,7 +795,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_crtc_wait_for_next_vblank(crtc);
+		intel_plane_initial_vblank_wait(plane);
 	}
 
 	/*
@@ -809,7 +809,7 @@ void intel_plane_disable_noatomic(struct intel_crtc *crtc,
 	 */
 	if (HAS_GMCH(dev_priv) &&
 	    intel_set_memory_cxsr(dev_priv, false))
-		intel_crtc_wait_for_next_vblank(crtc);
+		intel_plane_initial_vblank_wait(plane);
 
 	/*
 	 * Gen2 reports pipe underruns whenever all planes are disabled.
@@ -819,7 +819,7 @@ void intel_plane_disable_noatomic(struct intel_crtc *crtc,
 		intel_set_cpu_fifo_underrun_reporting(dev_priv, crtc->pipe, false);
 
 	intel_plane_disable_arm(NULL, plane, crtc_state);
-	intel_crtc_wait_for_next_vblank(crtc);
+	intel_plane_initial_vblank_wait(plane);
 }
 
 unsigned int
diff --git a/drivers/gpu/drm/i915/display/intel_plane_initial.c b/drivers/gpu/drm/i915/display/intel_plane_initial.c
index 62401f6a04e4f..f0f4ec2b6cc98 100644
--- a/drivers/gpu/drm/i915/display/intel_plane_initial.c
+++ b/drivers/gpu/drm/i915/display/intel_plane_initial.c
@@ -14,6 +14,13 @@
 #include "intel_frontbuffer.h"
 #include "intel_plane_initial.h"
 
+void intel_plane_initial_vblank_wait(plane);(struct intel_plane *plane)
+{
+	struct intel_crtc *crtc = intel_crtc_for_pipe(to_intel_display(plane), plane->pipe);
+
+	intel_crtc_wait_for_next_vblank(crtc);
+}
+
 static bool
 intel_reuse_initial_plane_obj(struct intel_crtc *this,
 			      const struct intel_initial_plane_config plane_configs[],
@@ -438,7 +445,7 @@ void intel_initial_plane_config(struct drm_i915_private *i915)
 		intel_find_initial_plane_obj(crtc, plane_configs);
 
 		if (i915->display.funcs.display->fixup_initial_plane_config(crtc, plane_config))
-			intel_crtc_wait_for_next_vblank(crtc);
+			intel_initial_vblank_wait(crtc);
 
 		plane_config_fini(plane_config);
 	}
diff --git a/drivers/gpu/drm/i915/display/intel_plane_initial.h b/drivers/gpu/drm/i915/display/intel_plane_initial.h
index 64ab95239cd45..3ba20b9e2355f 100644
--- a/drivers/gpu/drm/i915/display/intel_plane_initial.h
+++ b/drivers/gpu/drm/i915/display/intel_plane_initial.h
@@ -8,6 +8,7 @@
 
 struct drm_i915_private;
 
+void intel_plane_initial_vblank_wait(struct intel_plane *plane);
 void intel_initial_plane_config(struct drm_i915_private *i915);
 
 #endif
diff --git a/drivers/gpu/drm/xe/display/xe_plane_initial.c b/drivers/gpu/drm/xe/display/xe_plane_initial.c
index 668f544a7ac80..9252cfeef0a1f 100644
--- a/drivers/gpu/drm/xe/display/xe_plane_initial.c
+++ b/drivers/gpu/drm/xe/display/xe_plane_initial.c
@@ -8,7 +8,9 @@
 
 #include "regs/xe_gtt_defs.h"
 #include "xe_ggtt.h"
+#include "xe_mmio.h"
 
+#include "i915_reg.h"
 #include "intel_atomic_plane.h"
 #include "intel_crtc.h"
 #include "intel_display.h"
@@ -22,6 +24,21 @@
 
 #include <generated/xe_wa_oob.h>
 
+void intel_plane_initial_vblank_wait(struct intel_plane *plane)
+{
+	/* Early xe has no irq */
+	struct xe_device *xe = to_xe_device(plane->base.dev);
+	struct xe_reg pipe_frmtmstmp = XE_REG(i915_mmio_reg_offset(PIPE_FRMTMSTMP(plane->pipe)));
+	u32 timestamp;
+	int ret;
+
+	timestamp = xe_mmio_read32(xe_root_tile_mmio(xe), pipe_frmtmstmp);
+
+	ret = xe_mmio_wait32_not(xe_root_tile_mmio(xe), pipe_frmtmstmp, ~0U, timestamp, 40000U, &timestamp, false);
+	if (ret < 0)
+		drm_warn(&xe->drm, "waiting for early vblank failed with %i\n", ret);
+}
+
 static bool
 intel_reuse_initial_plane_obj(struct intel_crtc *this,
 			      const struct intel_initial_plane_config plane_configs[],
@@ -293,7 +310,7 @@ void intel_initial_plane_config(struct drm_i915_private *i915)
 		intel_find_initial_plane_obj(crtc, plane_configs);
 
 		if (i915->display.funcs.display->fixup_initial_plane_config(crtc, plane_config))
-			intel_crtc_wait_for_next_vblank(crtc);
+			intel_plane_initial_vblank_wait(to_intel_plane(crtc->base.primary));
 
 		plane_config_fini(plane_config);
 	}
-- 
2.45.2

[PATCH 3/9] drm/xe: Move suballocator init to after display init

[Maarten Lankhorst]

No allocations should be done before we have had a chance to preserve
the display fb.

Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
---
 drivers/gpu/drm/xe/xe_device.c |  6 ++++++
 drivers/gpu/drm/xe/xe_tile.c   | 12 ++++++++----
 drivers/gpu/drm/xe/xe_tile.h   |  1 +
 3 files changed, 15 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_device.c b/drivers/gpu/drm/xe/xe_device.c
index a0d29d5cb7b70..cef782f244e1a 100644
--- a/drivers/gpu/drm/xe/xe_device.c
+++ b/drivers/gpu/drm/xe/xe_device.c
@@ -730,6 +730,12 @@ int xe_device_probe(struct xe_device *xe)
 	if (err)
 		goto err;
 
+	for_each_tile(tile, xe, id) {
+		err = xe_tile_init(tile);
+		if (err)
+			goto err;
+	}
+
 	for_each_gt(gt, xe, id) {
 		last_gt = id;
 
diff --git a/drivers/gpu/drm/xe/xe_tile.c b/drivers/gpu/drm/xe/xe_tile.c
index 07cf7cfe4abd5..2825553b568f7 100644
--- a/drivers/gpu/drm/xe/xe_tile.c
+++ b/drivers/gpu/drm/xe/xe_tile.c
@@ -170,10 +170,6 @@ int xe_tile_init_noalloc(struct xe_tile *tile)
 	if (err)
 		return err;
 
-	tile->mem.kernel_bb_pool = xe_sa_bo_manager_init(tile, SZ_1M, 16);
-	if (IS_ERR(tile->mem.kernel_bb_pool))
-		return PTR_ERR(tile->mem.kernel_bb_pool);
-
 	xe_wa_apply_tile_workarounds(tile);
 
 	err = xe_tile_sysfs_init(tile);
@@ -181,6 +177,14 @@ int xe_tile_init_noalloc(struct xe_tile *tile)
 	return 0;
 }
 
+int xe_tile_init(struct xe_tile *tile)
+{
+	tile->mem.kernel_bb_pool = xe_sa_bo_manager_init(tile, SZ_1M, 16);
+	if (IS_ERR(tile->mem.kernel_bb_pool))
+		return PTR_ERR(tile->mem.kernel_bb_pool);
+
+	return 0;
+}
 void xe_tile_migrate_wait(struct xe_tile *tile)
 {
 	xe_migrate_wait(tile->migrate);
diff --git a/drivers/gpu/drm/xe/xe_tile.h b/drivers/gpu/drm/xe/xe_tile.h
index 1c9e42ade6b05..eb939316d55b0 100644
--- a/drivers/gpu/drm/xe/xe_tile.h
+++ b/drivers/gpu/drm/xe/xe_tile.h
@@ -12,6 +12,7 @@ struct xe_tile;
 
 int xe_tile_init_early(struct xe_tile *tile, struct xe_device *xe, u8 id);
 int xe_tile_init_noalloc(struct xe_tile *tile);
+int xe_tile_init(struct xe_tile *tile);
 
 void xe_tile_migrate_wait(struct xe_tile *tile);
 
-- 
2.45.2

[PATCH 4/9] drm/xe: Defer irq init until after xe_display_init_noaccel

[Maarten Lankhorst]

Technically, I believe this means that xe_display_init_noirq and
xe_display_init_noaccel can be merged together now.

Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
---
 drivers/gpu/drm/xe/xe_device.c | 12 ++++--------
 drivers/gpu/drm/xe/xe_tile.c   |  7 +++++++
 2 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_device.c b/drivers/gpu/drm/xe/xe_device.c
index cef782f244e1a..b9948b2dc8d1d 100644
--- a/drivers/gpu/drm/xe/xe_device.c
+++ b/drivers/gpu/drm/xe/xe_device.c
@@ -41,7 +41,6 @@
 #include "xe_hw_engine_group.h"
 #include "xe_hwmon.h"
 #include "xe_irq.h"
-#include "xe_memirq.h"
 #include "xe_mmio.h"
 #include "xe_module.h"
 #include "xe_observation.h"
@@ -673,9 +672,6 @@ int xe_device_probe(struct xe_device *xe)
 		err = xe_ggtt_init_early(tile->mem.ggtt);
 		if (err)
 			return err;
-		err = xe_memirq_init(&tile->memirq);
-		if (err)
-			return err;
 	}
 
 	for_each_gt(gt, xe, id) {
@@ -695,10 +691,6 @@ int xe_device_probe(struct xe_device *xe)
 	if (err)
 		return err;
 
-	err = xe_irq_install(xe);
-	if (err)
-		goto err;
-
 	err = probe_has_flat_ccs(xe);
 	if (err)
 		goto err;
@@ -736,6 +728,10 @@ int xe_device_probe(struct xe_device *xe)
 			goto err;
 	}
 
+	err = xe_irq_install(xe);
+	if (err)
+		goto err;
+
 	for_each_gt(gt, xe, id) {
 		last_gt = id;
 
diff --git a/drivers/gpu/drm/xe/xe_tile.c b/drivers/gpu/drm/xe/xe_tile.c
index 2825553b568f7..d07c1fba793ca 100644
--- a/drivers/gpu/drm/xe/xe_tile.c
+++ b/drivers/gpu/drm/xe/xe_tile.c
@@ -10,6 +10,7 @@
 #include "xe_device.h"
 #include "xe_ggtt.h"
 #include "xe_gt.h"
+#include "xe_memirq.h"
 #include "xe_migrate.h"
 #include "xe_pcode.h"
 #include "xe_sa.h"
@@ -179,6 +180,12 @@ int xe_tile_init_noalloc(struct xe_tile *tile)
 
 int xe_tile_init(struct xe_tile *tile)
 {
+	int err;
+
+	err = xe_memirq_init(&tile->memirq);
+	if (err)
+		return err;
+
 	tile->mem.kernel_bb_pool = xe_sa_bo_manager_init(tile, SZ_1M, 16);
 	if (IS_ERR(tile->mem.kernel_bb_pool))
 		return PTR_ERR(tile->mem.kernel_bb_pool);
-- 
2.45.2

[PATCH 5/9] drm/xe/display: Use a single early init call for display

[Maarten Lankhorst]

Instead of 3 different calls, it should be safe to unify to a single
call now. This makes the init sequence cleaner, and display less
tangled.

Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
---
 drivers/gpu/drm/xe/display/xe_display.c | 72 +++++++------------------
 drivers/gpu/drm/xe/display/xe_display.h |  8 +--
 drivers/gpu/drm/xe/xe_device.c          | 10 +---
 3 files changed, 23 insertions(+), 67 deletions(-)

diff --git a/drivers/gpu/drm/xe/display/xe_display.c b/drivers/gpu/drm/xe/display/xe_display.c
index b5502f335f531..a9ce4f561e7aa 100644
--- a/drivers/gpu/drm/xe/display/xe_display.c
+++ b/drivers/gpu/drm/xe/display/xe_display.c
@@ -100,31 +100,7 @@ int xe_display_create(struct xe_device *xe)
 	return drmm_add_action_or_reset(&xe->drm, display_destroy, NULL);
 }
 
-static void xe_display_fini_nommio(struct drm_device *dev, void *dummy)
-{
-	struct xe_device *xe = to_xe_device(dev);
-
-	if (!xe->info.probe_display)
-		return;
-
-	intel_power_domains_cleanup(xe);
-}
-
-int xe_display_init_nommio(struct xe_device *xe)
-{
-	if (!xe->info.probe_display)
-		return 0;
-
-	/* Fake uncore lock */
-	spin_lock_init(&xe->uncore.lock);
-
-	/* This must be called before any calls to HAS_PCH_* */
-	intel_detect_pch(xe);
-
-	return drmm_add_action_or_reset(&xe->drm, xe_display_fini_nommio, xe);
-}
-
-static void xe_display_fini_noirq(void *arg)
+static void xe_display_fini_early(void *arg)
 {
 	struct xe_device *xe = arg;
 	struct intel_display *display = &xe->display;
@@ -132,11 +108,13 @@ static void xe_display_fini_noirq(void *arg)
 	if (!xe->info.probe_display)
 		return;
 
+	intel_display_driver_remove_nogem(xe);
 	intel_display_driver_remove_noirq(xe);
 	intel_opregion_cleanup(display);
+	intel_power_domains_cleanup(xe);
 }
 
-int xe_display_init_noirq(struct xe_device *xe)
+int xe_display_init_early(struct xe_device *xe)
 {
 	struct intel_display *display = &xe->display;
 	int err;
@@ -144,6 +122,12 @@ int xe_display_init_noirq(struct xe_device *xe)
 	if (!xe->info.probe_display)
 		return 0;
 
+	/* Fake uncore lock */
+	spin_lock_init(&xe->uncore.lock);
+
+	/* This must be called before any calls to HAS_PCH_* */
+	intel_detect_pch(xe);
+
 	intel_display_driver_early_probe(xe);
 
 	/* Early display init.. */
@@ -160,36 +144,20 @@ int xe_display_init_noirq(struct xe_device *xe)
 	intel_display_device_info_runtime_init(xe);
 
 	err = intel_display_driver_probe_noirq(xe);
-	if (err) {
-		intel_opregion_cleanup(display);
-		return err;
-	}
-
-	return devm_add_action_or_reset(xe->drm.dev, xe_display_fini_noirq, xe);
-}
-
-static void xe_display_fini_noaccel(void *arg)
-{
-	struct xe_device *xe = arg;
-
-	if (!xe->info.probe_display)
-		return;
-
-	intel_display_driver_remove_nogem(xe);
-}
-
-int xe_display_init_noaccel(struct xe_device *xe)
-{
-	int err;
-
-	if (!xe->info.probe_display)
-		return 0;
+	if (err)
+		goto err_opregion;
 
 	err = intel_display_driver_probe_nogem(xe);
 	if (err)
-		return err;
+		goto err_noirq;
 
-	return devm_add_action_or_reset(xe->drm.dev, xe_display_fini_noaccel, xe);
+	return devm_add_action_or_reset(xe->drm.dev, xe_display_fini_early, xe);
+err_noirq:
+	intel_display_driver_remove_noirq(xe);
+	intel_power_domains_cleanup(xe);
+err_opregion:
+	intel_opregion_cleanup(display);
+	return err;
 }
 
 int xe_display_init(struct xe_device *xe)
diff --git a/drivers/gpu/drm/xe/display/xe_display.h b/drivers/gpu/drm/xe/display/xe_display.h
index 17afa537aee50..255cffa328160 100644
--- a/drivers/gpu/drm/xe/display/xe_display.h
+++ b/drivers/gpu/drm/xe/display/xe_display.h
@@ -20,9 +20,7 @@ int xe_display_create(struct xe_device *xe);
 
 int xe_display_probe(struct xe_device *xe);
 
-int xe_display_init_nommio(struct xe_device *xe);
-int xe_display_init_noirq(struct xe_device *xe);
-int xe_display_init_noaccel(struct xe_device *xe);
+int xe_display_init_early(struct xe_device *xe);
 int xe_display_init(struct xe_device *xe);
 void xe_display_fini(struct xe_device *xe);
 
@@ -53,9 +51,7 @@ static inline int xe_display_create(struct xe_device *xe) { return 0; }
 
 static inline int xe_display_probe(struct xe_device *xe) { return 0; }
 
-static inline int xe_display_init_nommio(struct xe_device *xe) { return 0; }
-static inline int xe_display_init_noirq(struct xe_device *xe) { return 0; }
-static inline int xe_display_init_noaccel(struct xe_device *xe) { return 0; }
+static inline int xe_display_init_early(struct xe_device *xe) { return 0; }
 static inline int xe_display_init(struct xe_device *xe) { return 0; }
 static inline void xe_display_fini(struct xe_device *xe) {}
 
diff --git a/drivers/gpu/drm/xe/xe_device.c b/drivers/gpu/drm/xe/xe_device.c
index b9948b2dc8d1d..f1147ebeeff31 100644
--- a/drivers/gpu/drm/xe/xe_device.c
+++ b/drivers/gpu/drm/xe/xe_device.c
@@ -632,10 +632,6 @@ int xe_device_probe(struct xe_device *xe)
 		return err;
 
 	xe->info.mem_region_mask = 1;
-	err = xe_display_init_nommio(xe);
-	if (err)
-		return err;
-
 	err = xe_set_dma_info(xe);
 	if (err)
 		return err;
@@ -687,10 +683,6 @@ int xe_device_probe(struct xe_device *xe)
 	if (err)
 		return err;
 
-	err = xe_display_init_noirq(xe);
-	if (err)
-		return err;
-
 	err = probe_has_flat_ccs(xe);
 	if (err)
 		goto err;
@@ -718,7 +710,7 @@ int xe_device_probe(struct xe_device *xe)
 	 * This is the reason the first allocation needs to be done
 	 * inside display.
 	 */
-	err = xe_display_init_noaccel(xe);
+	err = xe_display_init_early(xe);
 	if (err)
 		goto err;
 
-- 
2.45.2

[PATCH 6/9] drm/xe/sriov: Move VF bootstrap and query_config to vf_guc_init

[Maarten Lankhorst]

We want to split up GUC init to an alloc and noalloc part to keep the
init path the same for VF and !VF as much as possible.

Everything in vf_guc_init should be done as early as possible, otherwise
VRAM probing becomes impossible.

Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
---
 drivers/gpu/drm/xe/xe_gt_sriov_vf.c |  1 -
 drivers/gpu/drm/xe/xe_guc.c         | 17 +++++++++--------
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_gt_sriov_vf.c b/drivers/gpu/drm/xe/xe_gt_sriov_vf.c
index d3baba50f0851..521aaa3631f06 100644
--- a/drivers/gpu/drm/xe/xe_gt_sriov_vf.c
+++ b/drivers/gpu/drm/xe/xe_gt_sriov_vf.c
@@ -885,7 +885,6 @@ u32 xe_gt_sriov_vf_read32(struct xe_gt *gt, struct xe_reg reg)
 	struct vf_runtime_reg *rr;
 
 	xe_gt_assert(gt, IS_SRIOV_VF(gt_to_xe(gt)));
-	xe_gt_assert(gt, gt->sriov.vf.pf_version.major);
 	xe_gt_assert(gt, !reg.vf);
 
 	if (reg.addr == GMD_ID.addr) {
diff --git a/drivers/gpu/drm/xe/xe_guc.c b/drivers/gpu/drm/xe/xe_guc.c
index 7f704346a8f4d..709c219c6a5b1 100644
--- a/drivers/gpu/drm/xe/xe_guc.c
+++ b/drivers/gpu/drm/xe/xe_guc.c
@@ -301,6 +301,7 @@ static int xe_guc_realloc_post_hwconfig(struct xe_guc *guc)
 
 static int vf_guc_init(struct xe_guc *guc)
 {
+	struct xe_gt *gt = guc_to_gt(guc);
 	int err;
 
 	xe_guc_comm_init_early(guc);
@@ -313,6 +314,14 @@ static int vf_guc_init(struct xe_guc *guc)
 	if (err)
 		return err;
 
+	err = xe_gt_sriov_vf_bootstrap(gt);
+	if (err)
+		return err;
+
+	err = xe_gt_sriov_vf_query_config(gt);
+	if (err)
+		return err;
+
 	return 0;
 }
 
@@ -753,14 +762,6 @@ static int vf_guc_min_load_for_hwconfig(struct xe_guc *guc)
 	struct xe_gt *gt = guc_to_gt(guc);
 	int ret;
 
-	ret = xe_gt_sriov_vf_bootstrap(gt);
-	if (ret)
-		return ret;
-
-	ret = xe_gt_sriov_vf_query_config(gt);
-	if (ret)
-		return ret;
-
 	ret = xe_guc_hwconfig_init(guc);
 	if (ret)
 		return ret;
-- 
2.45.2

[PATCH 7/9] drm/xe: Simplify GuC early initialisation

[Maarten Lankhorst]

Add a 2-stage GuC init. An early one for everything that is needed
for VF, and a full one that loads GuC and is allowed to do allocations.

Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
---
 drivers/gpu/drm/xe/tests/xe_guc_relay_test.c |  2 +-
 drivers/gpu/drm/xe/xe_device.c               | 16 ------
 drivers/gpu/drm/xe/xe_gt.c                   |  9 +++-
 drivers/gpu/drm/xe/xe_guc.c                  | 51 ++++++++++++--------
 drivers/gpu/drm/xe/xe_guc.h                  |  1 +
 drivers/gpu/drm/xe/xe_guc_ct.c               | 28 +++++++----
 drivers/gpu/drm/xe/xe_guc_ct.h               |  1 +
 drivers/gpu/drm/xe/xe_guc_relay.c            |  6 +--
 drivers/gpu/drm/xe/xe_guc_relay.h            |  2 +-
 drivers/gpu/drm/xe/xe_huc.h                  |  1 +
 drivers/gpu/drm/xe/xe_uc.c                   | 21 ++++++++
 drivers/gpu/drm/xe/xe_uc.h                   |  1 +
 12 files changed, 88 insertions(+), 51 deletions(-)

diff --git a/drivers/gpu/drm/xe/tests/xe_guc_relay_test.c b/drivers/gpu/drm/xe/tests/xe_guc_relay_test.c
index 13701451b9235..b951d55ef1b32 100644
--- a/drivers/gpu/drm/xe/tests/xe_guc_relay_test.c
+++ b/drivers/gpu/drm/xe/tests/xe_guc_relay_test.c
@@ -42,7 +42,7 @@ static int relay_test_init(struct kunit *test)
 	kunit_activate_static_stub(test, relay_get_totalvfs,
 				   replacement_relay_get_totalvfs);
 
-	KUNIT_ASSERT_EQ(test, xe_guc_relay_init(relay), 0);
+	KUNIT_ASSERT_EQ(test, xe_guc_relay_init_noalloc(relay), 0);
 	KUNIT_EXPECT_TRUE(test, relay_is_ready(relay));
 	relay->last_rid = TEST_RID - 1;
 
diff --git a/drivers/gpu/drm/xe/xe_device.c b/drivers/gpu/drm/xe/xe_device.c
index f1147ebeeff31..5dafd48d5478b 100644
--- a/drivers/gpu/drm/xe/xe_device.c
+++ b/drivers/gpu/drm/xe/xe_device.c
@@ -646,25 +646,9 @@ int xe_device_probe(struct xe_device *xe)
 		err = xe_gt_init_early(gt);
 		if (err)
 			return err;
-
-		/*
-		 * Only after this point can GT-specific MMIO operations
-		 * (including things like communication with the GuC)
-		 * be performed.
-		 */
-		xe_gt_mmio_init(gt);
 	}
 
 	for_each_tile(tile, xe, id) {
-		if (IS_SRIOV_VF(xe)) {
-			xe_guc_comm_init_early(&tile->primary_gt->uc.guc);
-			err = xe_gt_sriov_vf_bootstrap(tile->primary_gt);
-			if (err)
-				return err;
-			err = xe_gt_sriov_vf_query_config(tile->primary_gt);
-			if (err)
-				return err;
-		}
 		err = xe_ggtt_init_early(tile->mem.ggtt);
 		if (err)
 			return err;
diff --git a/drivers/gpu/drm/xe/xe_gt.c b/drivers/gpu/drm/xe/xe_gt.c
index d6744be01a687..dc6901255dc91 100644
--- a/drivers/gpu/drm/xe/xe_gt.c
+++ b/drivers/gpu/drm/xe/xe_gt.c
@@ -387,7 +387,14 @@ int xe_gt_init_early(struct xe_gt *gt)
 	xe_force_wake_init_gt(gt, gt_to_fw(gt));
 	spin_lock_init(&gt->global_invl_lock);
 
-	return 0;
+	/*
+	 * Only after this point can GT-specific MMIO operations
+	 * (including things like communication with the GuC)
+	 * be performed.
+	 */
+	xe_gt_mmio_init(gt);
+
+	return xe_uc_init_noalloc(&gt->uc);
 }
 
 static void dump_pat_on_error(struct xe_gt *gt)
diff --git a/drivers/gpu/drm/xe/xe_guc.c b/drivers/gpu/drm/xe/xe_guc.c
index 709c219c6a5b1..89aa4598bf3c4 100644
--- a/drivers/gpu/drm/xe/xe_guc.c
+++ b/drivers/gpu/drm/xe/xe_guc.c
@@ -299,21 +299,11 @@ static int xe_guc_realloc_post_hwconfig(struct xe_guc *guc)
 	return 0;
 }
 
-static int vf_guc_init(struct xe_guc *guc)
+static int vf_guc_init_noalloc(struct xe_guc *guc)
 {
 	struct xe_gt *gt = guc_to_gt(guc);
 	int err;
 
-	xe_guc_comm_init_early(guc);
-
-	err = xe_guc_ct_init(&guc->ct);
-	if (err)
-		return err;
-
-	err = xe_guc_relay_init(&guc->relay);
-	if (err)
-		return err;
-
 	err = xe_gt_sriov_vf_bootstrap(gt);
 	if (err)
 		return err;
@@ -325,6 +315,35 @@ static int vf_guc_init(struct xe_guc *guc)
 	return 0;
 }
 
+int xe_guc_init_noalloc(struct xe_guc *guc)
+{
+	struct xe_device *xe = guc_to_xe(guc);
+	struct xe_gt *gt = guc_to_gt(guc);
+	int ret;
+
+	xe_guc_comm_init_early(guc);
+
+	ret = xe_guc_ct_init_noalloc(&guc->ct);
+	if (ret)
+		goto out;
+
+	ret = xe_guc_relay_init_noalloc(&guc->relay);
+	if (ret)
+		goto out;
+
+	if (IS_SRIOV_VF(xe)) {
+		ret = vf_guc_init_noalloc(guc);
+		if (ret)
+			goto out;
+	}
+
+	return 0;
+
+out:
+	xe_gt_err(gt, "GuC init failed with %pe\n", ERR_PTR(ret));
+	return ret;
+}
+
 int xe_guc_init(struct xe_guc *guc)
 {
 	struct xe_device *xe = guc_to_xe(guc);
@@ -334,13 +353,13 @@ int xe_guc_init(struct xe_guc *guc)
 	guc->fw.type = XE_UC_FW_TYPE_GUC;
 	ret = xe_uc_fw_init(&guc->fw);
 	if (ret)
-		goto out;
+		return ret;
 
 	if (!xe_uc_fw_is_enabled(&guc->fw))
 		return 0;
 
 	if (IS_SRIOV_VF(xe)) {
-		ret = vf_guc_init(guc);
+		ret = xe_guc_ct_init(&guc->ct);
 		if (ret)
 			goto out;
 		return 0;
@@ -362,10 +381,6 @@ int xe_guc_init(struct xe_guc *guc)
 	if (ret)
 		goto out;
 
-	ret = xe_guc_relay_init(&guc->relay);
-	if (ret)
-		goto out;
-
 	xe_uc_fw_change_status(&guc->fw, XE_UC_FIRMWARE_LOADABLE);
 
 	ret = devm_add_action_or_reset(xe->drm.dev, guc_fini_hw, guc);
@@ -374,8 +389,6 @@ int xe_guc_init(struct xe_guc *guc)
 
 	guc_init_params(guc);
 
-	xe_guc_comm_init_early(guc);
-
 	return 0;
 
 out:
diff --git a/drivers/gpu/drm/xe/xe_guc.h b/drivers/gpu/drm/xe/xe_guc.h
index 58338be445585..965bf72912009 100644
--- a/drivers/gpu/drm/xe/xe_guc.h
+++ b/drivers/gpu/drm/xe/xe_guc.h
@@ -26,6 +26,7 @@
 struct drm_printer;
 
 void xe_guc_comm_init_early(struct xe_guc *guc);
+int xe_guc_init_noalloc(struct xe_guc *guc);
 int xe_guc_init(struct xe_guc *guc);
 int xe_guc_init_post_hwconfig(struct xe_guc *guc);
 int xe_guc_post_load_init(struct xe_guc *guc);
diff --git a/drivers/gpu/drm/xe/xe_guc_ct.c b/drivers/gpu/drm/xe/xe_guc_ct.c
index a405b9218ad2d..f0bbc6bbf6170 100644
--- a/drivers/gpu/drm/xe/xe_guc_ct.c
+++ b/drivers/gpu/drm/xe/xe_guc_ct.c
@@ -203,12 +203,10 @@ static void primelockdep(struct xe_guc_ct *ct)
 	fs_reclaim_release(GFP_KERNEL);
 }
 
-int xe_guc_ct_init(struct xe_guc_ct *ct)
+int xe_guc_ct_init_noalloc(struct xe_guc_ct *ct)
 {
 	struct xe_device *xe = ct_to_xe(ct);
 	struct xe_gt *gt = ct_to_gt(ct);
-	struct xe_tile *tile = gt_to_tile(gt);
-	struct xe_bo *bo;
 	int err;
 
 	xe_gt_assert(gt, !(guc_ct_size() % PAGE_SIZE));
@@ -234,6 +232,23 @@ int xe_guc_ct_init(struct xe_guc_ct *ct)
 
 	primelockdep(ct);
 
+	err = drmm_add_action_or_reset(&xe->drm, guc_ct_fini, ct);
+	if (err)
+		return err;
+
+	xe_gt_assert(gt, ct->state == XE_GUC_CT_STATE_NOT_INITIALIZED);
+	ct->state = XE_GUC_CT_STATE_DISABLED;
+	return 0;
+}
+ALLOW_ERROR_INJECTION(xe_guc_ct_init_noalloc, ERRNO); /* See xe_pci_probe() */
+
+int xe_guc_ct_init(struct xe_guc_ct *ct)
+{
+	struct xe_device *xe = ct_to_xe(ct);
+	struct xe_gt *gt = ct_to_gt(ct);
+	struct xe_tile *tile = gt_to_tile(gt);
+	struct xe_bo *bo;
+
 	bo = xe_managed_bo_create_pin_map(xe, tile, guc_ct_size(),
 					  XE_BO_FLAG_SYSTEM |
 					  XE_BO_FLAG_GGTT |
@@ -242,13 +257,6 @@ int xe_guc_ct_init(struct xe_guc_ct *ct)
 		return PTR_ERR(bo);
 
 	ct->bo = bo;
-
-	err = drmm_add_action_or_reset(&xe->drm, guc_ct_fini, ct);
-	if (err)
-		return err;
-
-	xe_gt_assert(gt, ct->state == XE_GUC_CT_STATE_NOT_INITIALIZED);
-	ct->state = XE_GUC_CT_STATE_DISABLED;
 	return 0;
 }
 ALLOW_ERROR_INJECTION(xe_guc_ct_init, ERRNO); /* See xe_pci_probe() */
diff --git a/drivers/gpu/drm/xe/xe_guc_ct.h b/drivers/gpu/drm/xe/xe_guc_ct.h
index 82c4ae458dda3..9f28fc135890d 100644
--- a/drivers/gpu/drm/xe/xe_guc_ct.h
+++ b/drivers/gpu/drm/xe/xe_guc_ct.h
@@ -11,6 +11,7 @@
 struct drm_printer;
 struct xe_device;
 
+int xe_guc_ct_init_noalloc(struct xe_guc_ct *ct);
 int xe_guc_ct_init(struct xe_guc_ct *ct);
 int xe_guc_ct_enable(struct xe_guc_ct *ct);
 void xe_guc_ct_disable(struct xe_guc_ct *ct);
diff --git a/drivers/gpu/drm/xe/xe_guc_relay.c b/drivers/gpu/drm/xe/xe_guc_relay.c
index 8f62de026724c..b778d0e0253de 100644
--- a/drivers/gpu/drm/xe/xe_guc_relay.c
+++ b/drivers/gpu/drm/xe/xe_guc_relay.c
@@ -322,7 +322,7 @@ static void __fini_relay(struct drm_device *drm, void *arg)
 }
 
 /**
- * xe_guc_relay_init - Initialize a &xe_guc_relay
+ * xe_guc_relay_init_noalloc - Initialize a &xe_guc_relay
  * @relay: the &xe_guc_relay to initialize
  *
  * Initialize remaining members of &xe_guc_relay that may depend
@@ -330,7 +330,7 @@ static void __fini_relay(struct drm_device *drm, void *arg)
  *
  * Return: 0 on success or a negative error code on failure.
  */
-int xe_guc_relay_init(struct xe_guc_relay *relay)
+int xe_guc_relay_init_noalloc(struct xe_guc_relay *relay)
 {
 	const int XE_RELAY_MEMPOOL_MIN_NUM = 1;
 	struct xe_device *xe = relay_to_xe(relay);
@@ -356,7 +356,7 @@ int xe_guc_relay_init(struct xe_guc_relay *relay)
 
 	return drmm_add_action_or_reset(&xe->drm, __fini_relay, relay);
 }
-ALLOW_ERROR_INJECTION(xe_guc_relay_init, ERRNO); /* See xe_pci_probe() */
+ALLOW_ERROR_INJECTION(xe_guc_relay_init_noalloc, ERRNO); /* See xe_pci_probe() */
 
 static u32 to_relay_error(int err)
 {
diff --git a/drivers/gpu/drm/xe/xe_guc_relay.h b/drivers/gpu/drm/xe/xe_guc_relay.h
index 385429aa188ab..e0afff4542cfc 100644
--- a/drivers/gpu/drm/xe/xe_guc_relay.h
+++ b/drivers/gpu/drm/xe/xe_guc_relay.h
@@ -11,7 +11,7 @@
 
 struct xe_guc_relay;
 
-int xe_guc_relay_init(struct xe_guc_relay *relay);
+int xe_guc_relay_init_noalloc(struct xe_guc_relay *relay);
 
 int xe_guc_relay_send_to_pf(struct xe_guc_relay *relay,
 			    const u32 *msg, u32 len, u32 *buf, u32 buf_size);
diff --git a/drivers/gpu/drm/xe/xe_huc.h b/drivers/gpu/drm/xe/xe_huc.h
index fa1c45e704432..bc6f93028c2b6 100644
--- a/drivers/gpu/drm/xe/xe_huc.h
+++ b/drivers/gpu/drm/xe/xe_huc.h
@@ -17,6 +17,7 @@ enum xe_huc_auth_types {
 	XE_HUC_AUTH_TYPES_COUNT
 };
 
+int xe_huc_init_noalloc(struct xe_huc *huc);
 int xe_huc_init(struct xe_huc *huc);
 int xe_huc_init_post_hwconfig(struct xe_huc *huc);
 int xe_huc_upload(struct xe_huc *huc);
diff --git a/drivers/gpu/drm/xe/xe_uc.c b/drivers/gpu/drm/xe/xe_uc.c
index 0d073a9987c2e..45d0c2f26754c 100644
--- a/drivers/gpu/drm/xe/xe_uc.c
+++ b/drivers/gpu/drm/xe/xe_uc.c
@@ -32,6 +32,27 @@ uc_to_xe(struct xe_uc *uc)
 }
 
 /* Should be called once at driver load only */
+int xe_uc_init_noalloc(struct xe_uc *uc)
+{
+	int ret;
+
+	/*
+	 * We call the GuC/HuC/GSC init functions even if GuC submission is off
+	 * to correctly move our tracking of the FW state to "disabled".
+	 */
+	ret = xe_guc_init_noalloc(&uc->guc);
+	if (ret)
+		goto err;
+
+	/* HuC and GSC have no early dependencies and can be completely initialised during full xe_uc_init(). */
+
+	return 0;
+
+err:
+	xe_gt_err(uc_to_gt(uc), "Failed to early initialize uC (%pe)\n", ERR_PTR(ret));
+	return ret;
+}
+
 int xe_uc_init(struct xe_uc *uc)
 {
 	int ret;
diff --git a/drivers/gpu/drm/xe/xe_uc.h b/drivers/gpu/drm/xe/xe_uc.h
index 506517c113339..650e95585b4ba 100644
--- a/drivers/gpu/drm/xe/xe_uc.h
+++ b/drivers/gpu/drm/xe/xe_uc.h
@@ -8,6 +8,7 @@
 
 struct xe_uc;
 
+int xe_uc_init_noalloc(struct xe_uc *uc);
 int xe_uc_init(struct xe_uc *uc);
 int xe_uc_init_hwconfig(struct xe_uc *uc);
 int xe_uc_init_post_hwconfig(struct xe_uc *uc);
-- 
2.45.2

[PATCH 8/9] drm/xe: Make it possible to read instance0 MCR registers after xe_gt_mcr_init_early

[Maarten Lankhorst]

After mcr_init_early, we need to be able to do VRAM and CCS probing
without hwconfig probe. Fortunately the relevant registers are all
instance 0, which fortunately means no dependencies on further initialisation
is required.

Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
---
 drivers/gpu/drm/xe/xe_gt_mcr.c   | 47 ++++++++++++++++++--------------
 drivers/gpu/drm/xe/xe_gt_types.h |  2 ++
 2 files changed, 29 insertions(+), 20 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_gt_mcr.c b/drivers/gpu/drm/xe/xe_gt_mcr.c
index 5013d674e17da..4b23c96290046 100644
--- a/drivers/gpu/drm/xe/xe_gt_mcr.c
+++ b/drivers/gpu/drm/xe/xe_gt_mcr.c
@@ -413,12 +413,6 @@ static void init_steering_sqidi_psmi(struct xe_gt *gt)
 	gt->steering[SQIDI_PSMI].instance_target = select & 0x1;
 }
 
-static void init_steering_inst0(struct xe_gt *gt)
-{
-	gt->steering[INSTANCE0].group_target = 0;	/* unused */
-	gt->steering[INSTANCE0].instance_target = 0;	/* unused */
-}
-
 static const struct {
 	const char *name;
 	void (*init)(struct xe_gt *gt);
@@ -429,7 +423,7 @@ static const struct {
 	[DSS] =		{ "DSS",	init_steering_dss },
 	[OADDRM] =	{ "OADDRM / GPMXMT", init_steering_oaddrm },
 	[SQIDI_PSMI] =  { "SQIDI_PSMI", init_steering_sqidi_psmi },
-	[INSTANCE0] =	{ "INSTANCE 0",	init_steering_inst0 },
+	[INSTANCE0] =	{ "INSTANCE 0",	NULL },
 	[IMPLICIT_STEERING] = { "IMPLICIT", NULL },
 };
 
@@ -439,25 +433,17 @@ static const struct {
  *
  * Perform early software only initialization of the MCR lock to allow
  * the synchronization on accessing the STEER_SEMAPHORE register and
- * use the xe_gt_mcr_multicast_write() function.
+ * use the xe_gt_mcr_multicast_write() function, plus the minimum
+ * safe MCR registers required for VRAM/CCS probing.
  */
 void xe_gt_mcr_init_early(struct xe_gt *gt)
 {
+	struct xe_device *xe = gt_to_xe(gt);
+
 	BUILD_BUG_ON(IMPLICIT_STEERING + 1 != NUM_STEERING_TYPES);
 	BUILD_BUG_ON(ARRAY_SIZE(xe_steering_types) != NUM_STEERING_TYPES);
 
 	spin_lock_init(&gt->mcr_lock);
-}
-
-/**
- * xe_gt_mcr_init - Normal initialization of the MCR support
- * @gt: GT structure
- *
- * Perform normal initialization of the MCR for all usages.
- */
-void xe_gt_mcr_init(struct xe_gt *gt)
-{
-	struct xe_device *xe = gt_to_xe(gt);
 
 	if (IS_SRIOV_VF(xe))
 		return;
@@ -498,10 +484,27 @@ void xe_gt_mcr_init(struct xe_gt *gt)
 		}
 	}
 
+	/* Mark instance 0 as initialised, we need this early for VRAM and CCS probe. */
+	gt->steering[INSTANCE0].initialised = true;
+}
+
+/**
+ * xe_gt_mcr_init - Normal initialization of the MCR support
+ * @gt: GT structure
+ *
+ * Perform normal initialization of the MCR for all usages.
+ */
+void xe_gt_mcr_init(struct xe_gt *gt)
+{
+	if (IS_SRIOV_VF(gt_to_xe(gt)))
+		return;
+
 	/* Select non-terminated steering target for each type */
-	for (int i = 0; i < NUM_STEERING_TYPES; i++)
+	for (int i = 0; i < NUM_STEERING_TYPES; i++) {
+		gt->steering[i].initialised = true;
 		if (gt->steering[i].ranges && xe_steering_types[i].init)
 			xe_steering_types[i].init(gt);
+	}
 }
 
 /**
@@ -563,6 +566,10 @@ static bool xe_gt_mcr_get_nonterminated_steering(struct xe_gt *gt,
 
 		for (int i = 0; gt->steering[type].ranges[i].end > 0; i++) {
 			if (xe_mmio_in_range(&gt->mmio, &gt->steering[type].ranges[i], reg)) {
+				drm_WARN(&gt_to_xe(gt)->drm, !gt->steering[type].initialised,
+					 "Uninitialised usage of MCR register %s/%#x\n",
+					 xe_steering_types[type].name, reg.addr);
+
 				*group = gt->steering[type].group_target;
 				*instance = gt->steering[type].instance_target;
 				return true;
diff --git a/drivers/gpu/drm/xe/xe_gt_types.h b/drivers/gpu/drm/xe/xe_gt_types.h
index a287b98ee70b4..4254665366ffe 100644
--- a/drivers/gpu/drm/xe/xe_gt_types.h
+++ b/drivers/gpu/drm/xe/xe_gt_types.h
@@ -375,6 +375,8 @@ struct xe_gt {
 		u16 group_target;
 		/** @steering.instance_target: instance to steer accesses to */
 		u16 instance_target;
+		/** @steering.initialised: Whether this steering range is initialised */
+		bool initialised;
 	} steering[NUM_STEERING_TYPES];
 
 	/**
-- 
2.45.2

[PATCH 9/9] drm/xe: Split init of xe_gt_init_hwconfig to xe_gt_init and *_early

[Maarten Lankhorst]

Now that we added the separate step of initialising GUC in
xe_gt_init_early, it should be ok to initialise the minimum during early
init, and the rest after allocations are allowed.

Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
---
 drivers/gpu/drm/xe/xe_device.c |  6 ----
 drivers/gpu/drm/xe/xe_gt.c     | 66 +++++++++++++++++-----------------
 drivers/gpu/drm/xe/xe_gt.h     |  1 -
 3 files changed, 32 insertions(+), 41 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_device.c b/drivers/gpu/drm/xe/xe_device.c
index 5dafd48d5478b..e11acd5475867 100644
--- a/drivers/gpu/drm/xe/xe_device.c
+++ b/drivers/gpu/drm/xe/xe_device.c
@@ -654,12 +654,6 @@ int xe_device_probe(struct xe_device *xe)
 			return err;
 	}
 
-	for_each_gt(gt, xe, id) {
-		err = xe_gt_init_hwconfig(gt);
-		if (err)
-			return err;
-	}
-
 	err = xe_devcoredump_init(xe);
 	if (err)
 		return err;
diff --git a/drivers/gpu/drm/xe/xe_gt.c b/drivers/gpu/drm/xe/xe_gt.c
index dc6901255dc91..a6154d1bad9f0 100644
--- a/drivers/gpu/drm/xe/xe_gt.c
+++ b/drivers/gpu/drm/xe/xe_gt.c
@@ -364,6 +364,21 @@ int xe_gt_record_default_lrcs(struct xe_gt *gt)
 	return err;
 }
 
+static int gt_init_mcr(struct xe_gt *gt)
+{
+	unsigned int fw_ref;
+
+	fw_ref = xe_force_wake_get(gt_to_fw(gt), XE_FW_GT);
+	if (!fw_ref)
+		return -ETIMEDOUT;
+
+	xe_gt_mcr_init_early(gt);
+	xe_pat_init(gt);
+
+	xe_force_wake_put(gt_to_fw(gt), fw_ref);
+	return 0;
+}
+
 int xe_gt_init_early(struct xe_gt *gt)
 {
 	int err;
@@ -394,7 +409,11 @@ int xe_gt_init_early(struct xe_gt *gt)
 	 */
 	xe_gt_mmio_init(gt);
 
-	return xe_uc_init_noalloc(&gt->uc);
+	err = xe_uc_init_noalloc(&gt->uc);
+	if (err)
+		return err;
+
+	return gt_init_mcr(gt);
 }
 
 static void dump_pat_on_error(struct xe_gt *gt)
@@ -419,6 +438,18 @@ static int gt_fw_domain_init(struct xe_gt *gt)
 		goto err_hw_fence_irq;
 	}
 
+	err = xe_uc_init(&gt->uc);
+	if (err)
+		goto err_force_wake;
+
+	err = xe_uc_init_hwconfig(&gt->uc);
+	if (err)
+		goto err_force_wake;
+
+	xe_gt_topology_init(gt);
+	xe_gt_mcr_init(gt);
+	xe_gt_enable_host_l2_vram(gt);
+
 	if (!xe_gt_is_media_type(gt)) {
 		err = xe_ggtt_init(gt_to_tile(gt)->mem.ggtt);
 		if (err)
@@ -550,39 +581,6 @@ static int all_fw_domain_init(struct xe_gt *gt)
 	return err;
 }
 
-/*
- * Initialize enough GT to be able to load GuC in order to obtain hwconfig and
- * enable CTB communication.
- */
-int xe_gt_init_hwconfig(struct xe_gt *gt)
-{
-	unsigned int fw_ref;
-	int err;
-
-	fw_ref = xe_force_wake_get(gt_to_fw(gt), XE_FW_GT);
-	if (!fw_ref)
-		return -ETIMEDOUT;
-
-	xe_gt_mcr_init_early(gt);
-	xe_pat_init(gt);
-
-	err = xe_uc_init(&gt->uc);
-	if (err)
-		goto out_fw;
-
-	err = xe_uc_init_hwconfig(&gt->uc);
-	if (err)
-		goto out_fw;
-
-	xe_gt_topology_init(gt);
-	xe_gt_mcr_init(gt);
-	xe_gt_enable_host_l2_vram(gt);
-
-out_fw:
-	xe_force_wake_put(gt_to_fw(gt), fw_ref);
-	return err;
-}
-
 int xe_gt_init(struct xe_gt *gt)
 {
 	int err;
diff --git a/drivers/gpu/drm/xe/xe_gt.h b/drivers/gpu/drm/xe/xe_gt.h
index 82b9b7f82fcab..0c026cc91ffe9 100644
--- a/drivers/gpu/drm/xe/xe_gt.h
+++ b/drivers/gpu/drm/xe/xe_gt.h
@@ -28,7 +28,6 @@ static inline bool xe_fault_inject_gt_reset(void)
 }
 
 struct xe_gt *xe_gt_alloc(struct xe_tile *tile);
-int xe_gt_init_hwconfig(struct xe_gt *gt);
 int xe_gt_init_early(struct xe_gt *gt);
 int xe_gt_init(struct xe_gt *gt);
 void xe_gt_mmio_init(struct xe_gt *gt);
-- 
2.45.2

✓ CI.Patch_applied: success for series starting with [1/9] drm/xe: Remove double pageflip

[Patchwork]

== Series Details ==

Series: series starting with [1/9] drm/xe: Remove double pageflip
URL   : https://patchwork.freedesktop.org/series/140944/
State : success

== Summary ==

=== Applying kernel patches on branch 'drm-tip' with base: ===
Base commit: b925cb281e79 drm-tip: 2024y-11m-05d-14h-00m-11s UTC integration manifest
=== git am output follows ===
Applying: drm/xe: Remove double pageflip
Applying: drm/xe/display: Add intel_plane_initial_vblank_wait
Applying: drm/xe: Move suballocator init to after display init
Applying: drm/xe: Defer irq init until after xe_display_init_noaccel
Applying: drm/xe/display: Use a single early init call for display
Applying: drm/xe/sriov: Move VF bootstrap and query_config to vf_guc_init
Applying: drm/xe: Simplify GuC early initialisation
Applying: drm/xe: Make it possible to read instance0 MCR registers after xe_gt_mcr_init_early
Applying: drm/xe: Split init of xe_gt_init_hwconfig to xe_gt_init and *_early

✗ CI.checkpatch: warning for series starting with [1/9] drm/xe: Remove double pageflip

[Patchwork]

== Series Details ==

Series: series starting with [1/9] drm/xe: Remove double pageflip
URL   : https://patchwork.freedesktop.org/series/140944/
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
30ab6715fc09baee6cc14cb3c89ad8858688d474
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit a884318ecbac588f18d60cb7f98a672360365e92
Author: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Date:   Tue Nov 5 13:18:57 2024 +0100

    drm/xe: Split init of xe_gt_init_hwconfig to xe_gt_init and *_early
    
    Now that we added the separate step of initialising GUC in
    xe_gt_init_early, it should be ok to initialise the minimum during early
    init, and the rest after allocations are allowed.
    
    Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
+ /mt/dim checkpatch b925cb281e79b763990ed64267ef82a87ace5060 drm-intel
eaf614dd96cf drm/xe: Remove double pageflip
ef4b40882868 drm/xe/display: Add intel_plane_initial_vblank_wait
-:50: ERROR:SPACING: space required after that ';' (ctx:VxV)
#50: FILE: drivers/gpu/drm/i915/display/intel_plane_initial.c:17:
+void intel_plane_initial_vblank_wait(plane);(struct intel_plane *plane)
                                            ^

-:109: WARNING:LONG_LINE: line length of 115 exceeds 100 columns
#109: FILE: drivers/gpu/drm/xe/display/xe_plane_initial.c:37:
+	ret = xe_mmio_wait32_not(xe_root_tile_mmio(xe), pipe_frmtmstmp, ~0U, timestamp, 40000U, &timestamp, false);

total: 1 errors, 1 warnings, 0 checks, 90 lines checked
70d6ce91a6cd drm/xe: Move suballocator init to after display init
eb48a291c9e9 drm/xe: Defer irq init until after xe_display_init_noaccel
7266d0ea3ef8 drm/xe/display: Use a single early init call for display
d3a2ec2cceec drm/xe/sriov: Move VF bootstrap and query_config to vf_guc_init
2ae032402548 drm/xe: Simplify GuC early initialisation
-:329: WARNING:LONG_LINE_COMMENT: line length of 112 exceeds 100 columns
#329: FILE: drivers/gpu/drm/xe/xe_uc.c:47:
+	/* HuC and GSC have no early dependencies and can be completely initialised during full xe_uc_init(). */

total: 0 errors, 1 warnings, 0 checks, 274 lines checked
95596fb77fa0 drm/xe: Make it possible to read instance0 MCR registers after xe_gt_mcr_init_early
-:9: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#9: 
instance 0, which fortunately means no dependencies on further initialisation

total: 0 errors, 1 warnings, 0 checks, 95 lines checked
a884318ecbac drm/xe: Split init of xe_gt_init_hwconfig to xe_gt_init and *_early

✓ CI.KUnit: success for series starting with [1/9] drm/xe: Remove double pageflip

[Patchwork]

== Series Details ==

Series: series starting with [1/9] drm/xe: Remove double pageflip
URL   : https://patchwork.freedesktop.org/series/140944/
State : success

== Summary ==

+ trap cleanup EXIT
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/xe/.kunitconfig
[15:16:34] Configuring KUnit Kernel ...
Generating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[15:16:38] Building KUnit Kernel ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
Building with:
$ make all compile_commands.json ARCH=um O=.kunit --jobs=48
../lib/iomap.c:156:5: warning: no previous prototype for ‘ioread64_lo_hi’ [-Wmissing-prototypes]
  156 | u64 ioread64_lo_hi(const void __iomem *addr)
      |     ^~~~~~~~~~~~~~
../lib/iomap.c:163:5: warning: no previous prototype for ‘ioread64_hi_lo’ [-Wmissing-prototypes]
  163 | u64 ioread64_hi_lo(const void __iomem *addr)
      |     ^~~~~~~~~~~~~~
../lib/iomap.c:170:5: warning: no previous prototype for ‘ioread64be_lo_hi’ [-Wmissing-prototypes]
  170 | u64 ioread64be_lo_hi(const void __iomem *addr)
      |     ^~~~~~~~~~~~~~~~
../lib/iomap.c:178:5: warning: no previous prototype for ‘ioread64be_hi_lo’ [-Wmissing-prototypes]
  178 | u64 ioread64be_hi_lo(const void __iomem *addr)
      |     ^~~~~~~~~~~~~~~~
../lib/iomap.c:264:6: warning: no previous prototype for ‘iowrite64_lo_hi’ [-Wmissing-prototypes]
  264 | void iowrite64_lo_hi(u64 val, void __iomem *addr)
      |      ^~~~~~~~~~~~~~~
../lib/iomap.c:272:6: warning: no previous prototype for ‘iowrite64_hi_lo’ [-Wmissing-prototypes]
  272 | void iowrite64_hi_lo(u64 val, void __iomem *addr)
      |      ^~~~~~~~~~~~~~~
../lib/iomap.c:280:6: warning: no previous prototype for ‘iowrite64be_lo_hi’ [-Wmissing-prototypes]
  280 | void iowrite64be_lo_hi(u64 val, void __iomem *addr)
      |      ^~~~~~~~~~~~~~~~~
../lib/iomap.c:288:6: warning: no previous prototype for ‘iowrite64be_hi_lo’ [-Wmissing-prototypes]
  288 | void iowrite64be_hi_lo(u64 val, void __iomem *addr)
      |      ^~~~~~~~~~~~~~~~~

[15:17:06] Starting KUnit Kernel (1/1)...
[15:17:06] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[15:17:07] =================== guc_dbm (7 subtests) ===================
[15:17:07] [PASSED] test_empty
[15:17:07] [PASSED] test_default
[15:17:07] ======================== test_size  ========================
[15:17:07] [PASSED] 4
[15:17:07] [PASSED] 8
[15:17:07] [PASSED] 32
[15:17:07] [PASSED] 256
[15:17:07] ==================== [PASSED] test_size ====================
[15:17:07] ======================= test_reuse  ========================
[15:17:07] [PASSED] 4
[15:17:07] [PASSED] 8
[15:17:07] [PASSED] 32
[15:17:07] [PASSED] 256
[15:17:07] =================== [PASSED] test_reuse ====================
[15:17:07] =================== test_range_overlap  ====================
[15:17:07] [PASSED] 4
[15:17:07] [PASSED] 8
[15:17:07] [PASSED] 32
[15:17:07] [PASSED] 256
[15:17:07] =============== [PASSED] test_range_overlap ================
[15:17:07] =================== test_range_compact  ====================
[15:17:07] [PASSED] 4
[15:17:07] [PASSED] 8
[15:17:07] [PASSED] 32
[15:17:07] [PASSED] 256
[15:17:07] =============== [PASSED] test_range_compact ================
[15:17:07] ==================== test_range_spare  =====================
[15:17:07] [PASSED] 4
[15:17:07] [PASSED] 8
[15:17:07] [PASSED] 32
[15:17:07] [PASSED] 256
[15:17:07] ================ [PASSED] test_range_spare =================
[15:17:07] ===================== [PASSED] guc_dbm =====================
[15:17:07] =================== guc_idm (6 subtests) ===================
[15:17:07] [PASSED] bad_init
[15:17:07] [PASSED] no_init
[15:17:07] [PASSED] init_fini
[15:17:07] [PASSED] check_used
[15:17:07] [PASSED] check_quota
[15:17:07] [PASSED] check_all
[15:17:07] ===================== [PASSED] guc_idm =====================
[15:17:07] ================== no_relay (3 subtests) ===================
[15:17:07] [PASSED] xe_drops_guc2pf_if_not_ready
[15:17:07] [PASSED] xe_drops_guc2vf_if_not_ready
[15:17:07] [PASSED] xe_rejects_send_if_not_ready
[15:17:07] ==================== [PASSED] no_relay =====================
[15:17:07] ================== pf_relay (14 subtests) ==================
[15:17:07] [PASSED] pf_rejects_guc2pf_too_short
[15:17:07] [PASSED] pf_rejects_guc2pf_too_long
[15:17:07] [PASSED] pf_rejects_guc2pf_no_payload
[15:17:07] [PASSED] pf_fails_no_payload
[15:17:07] [PASSED] pf_fails_bad_origin
[15:17:07] [PASSED] pf_fails_bad_type
[15:17:07] [PASSED] pf_txn_reports_error
[15:17:07] [PASSED] pf_txn_sends_pf2guc
[15:17:07] [PASSED] pf_sends_pf2guc
[15:17:07] [SKIPPED] pf_loopback_nop
[15:17:07] [SKIPPED] pf_loopback_echo
[15:17:07] [SKIPPED] pf_loopback_fail
[15:17:07] [SKIPPED] pf_loopback_busy
[15:17:07] [SKIPPED] pf_loopback_retry
[15:17:07] ==================== [PASSED] pf_relay =====================
[15:17:07] ================== vf_relay (3 subtests) ===================
[15:17:07] [PASSED] vf_rejects_guc2vf_too_short
[15:17:07] [PASSED] vf_rejects_guc2vf_too_long
[15:17:07] [PASSED] vf_rejects_guc2vf_no_payload
[15:17:07] ==================== [PASSED] vf_relay =====================
[15:17:07] ================= pf_service (11 subtests) =================
[15:17:07] [PASSED] pf_negotiate_any
[15:17:07] [PASSED] pf_negotiate_base_match
[15:17:07] [PASSED] pf_negotiate_base_newer
[15:17:07] [PASSED] pf_negotiate_base_next
[15:17:07] [SKIPPED] pf_negotiate_base_older
[15:17:07] [PASSED] pf_negotiate_base_prev
[15:17:07] [PASSED] pf_negotiate_latest_match
[15:17:07] [PASSED] pf_negotiate_latest_newer
[15:17:07] [PASSED] pf_negotiate_latest_next
[15:17:07] [SKIPPED] pf_negotiate_latest_older
[15:17:07] [SKIPPED] pf_negotiate_latest_prev
[15:17:07] =================== [PASSED] pf_service ====================
[15:17:07] ===================== lmtt (1 subtest) =====================
[15:17:07] ======================== test_ops  =========================
[15:17:07] [PASSED] 2-level
[15:17:07] [PASSED] multi-level
[15:17:07] ==================== [PASSED] test_ops =====================
[15:17:07] ====================== [PASSED] lmtt =======================
[15:17:07] =================== xe_mocs (2 subtests) ===================
[15:17:07] ================ xe_live_mocs_kernel_kunit  ================
[15:17:07] =========== [SKIPPED] xe_live_mocs_kernel_kunit ============
[15:17:07] ================ xe_live_mocs_reset_kunit  =================
[15:17:07] ============ [SKIPPED] xe_live_mocs_reset_kunit ============
[15:17:07] ==================== [SKIPPED] xe_mocs =====================
[15:17:07] ================= xe_migrate (2 subtests) ==================
[15:17:07] ================= xe_migrate_sanity_kunit  =================
[15:17:07] ============ [SKIPPED] xe_migrate_sanity_kunit =============
[15:17:07] ================== xe_validate_ccs_kunit  ==================
[15:17:07] ============= [SKIPPED] xe_validate_ccs_kunit ==============
[15:17:07] =================== [SKIPPED] xe_migrate ===================
[15:17:07] ================== xe_dma_buf (1 subtest) ==================
[15:17:07] ==================== xe_dma_buf_kunit  =====================
[15:17:07] ================ [SKIPPED] xe_dma_buf_kunit ================
[15:17:07] =================== [SKIPPED] xe_dma_buf ===================
[15:17:07] ==================== xe_bo (3 subtests) ====================
[15:17:07] ================== xe_ccs_migrate_kunit  ===================
[15:17:07] ============== [SKIPPED] xe_ccs_migrate_kunit ==============
[15:17:07] ==================== xe_bo_evict_kunit  ====================
[15:17:07] =============== [SKIPPED] xe_bo_evict_kunit ================
[15:17:07] =================== xe_bo_shrink_kunit  ====================
[15:17:07] =============== [SKIPPED] xe_bo_shrink_kunit ===============
[15:17:07] ===================== [SKIPPED] xe_bo ======================
[15:17:07] ==================== args (11 subtests) ====================
[15:17:07] [PASSED] count_args_test
[15:17:07] [PASSED] call_args_example
[15:17:07] [PASSED] call_args_test
[15:17:07] [PASSED] drop_first_arg_example
[15:17:07] [PASSED] drop_first_arg_test
[15:17:07] [PASSED] first_arg_example
[15:17:07] [PASSED] first_arg_test
[15:17:07] [PASSED] last_arg_example
[15:17:07] [PASSED] last_arg_test
[15:17:07] [PASSED] pick_arg_example
[15:17:07] [PASSED] sep_comma_examplestty: 'standard input': Inappropriate ioctl for device

[15:17:07] ====================== [PASSED] args =======================
[15:17:07] =================== xe_pci (2 subtests) ====================
[15:17:07] [PASSED] xe_gmdid_graphics_ip
[15:17:07] [PASSED] xe_gmdid_media_ip
[15:17:07] ===================== [PASSED] xe_pci ======================
[15:17:07] =================== xe_rtp (2 subtests) ====================
[15:17:07] =============== xe_rtp_process_to_sr_tests  ================
[15:17:07] [PASSED] coalesce-same-reg
[15:17:07] [PASSED] no-match-no-add
[15:17:07] [PASSED] match-or
[15:17:07] [PASSED] match-or-xfail
[15:17:07] [PASSED] no-match-no-add-multiple-rules
[15:17:07] [PASSED] two-regs-two-entries
[15:17:07] [PASSED] clr-one-set-other
[15:17:07] [PASSED] set-field
[15:17:07] [PASSED] conflict-duplicate
[15:17:07] [PASSED] conflict-not-disjoint
[15:17:07] [PASSED] conflict-reg-type
[15:17:07] =========== [PASSED] xe_rtp_process_to_sr_tests ============
[15:17:07] ================== xe_rtp_process_tests  ===================
[15:17:07] [PASSED] active1
[15:17:07] [PASSED] active2
[15:17:07] [PASSED] active-inactive
[15:17:07] [PASSED] inactive-active
[15:17:07] [PASSED] inactive-1st_or_active-inactive
[15:17:07] [PASSED] inactive-2nd_or_active-inactive
[15:17:07] [PASSED] inactive-last_or_active-inactive
[15:17:07] [PASSED] inactive-no_or_active-inactive
[15:17:07] ============== [PASSED] xe_rtp_process_tests ===============
[15:17:07] ===================== [PASSED] xe_rtp ======================
[15:17:07] ==================== xe_wa (1 subtest) =====================
[15:17:07] ======================== xe_wa_gt  =========================
[15:17:07] [PASSED] TIGERLAKE (B0)
[15:17:07] [PASSED] DG1 (A0)
[15:17:07] [PASSED] DG1 (B0)
[15:17:07] [PASSED] ALDERLAKE_S (A0)
[15:17:07] [PASSED] ALDERLAKE_S (B0)
[15:17:07] [PASSED] ALDERLAKE_S (C0)
[15:17:07] [PASSED] ALDERLAKE_S (D0)
[15:17:07] [PASSED] ALDERLAKE_P (A0)
[15:17:07] [PASSED] ALDERLAKE_P (B0)
[15:17:07] [PASSED] ALDERLAKE_P (C0)
[15:17:07] [PASSED] ALDERLAKE_S_RPLS (D0)
[15:17:07] [PASSED] ALDERLAKE_P_RPLU (E0)
[15:17:07] [PASSED] DG2_G10 (C0)
[15:17:07] [PASSED] DG2_G11 (B1)
[15:17:07] [PASSED] DG2_G12 (A1)
[15:17:07] [PASSED] METEORLAKE (g:A0, m:A0)
[15:17:07] [PASSED] METEORLAKE (g:A0, m:A0)
[15:17:07] [PASSED] METEORLAKE (g:A0, m:A0)
[15:17:07] [PASSED] LUNARLAKE (g:A0, m:A0)
[15:17:07] [PASSED] LUNARLAKE (g:B0, m:A0)
[15:17:07] [PASSED] BATTLEMAGE (g:A0, m:A1)
[15:17:07] ==================== [PASSED] xe_wa_gt =====================
[15:17:07] ====================== [PASSED] xe_wa ======================
[15:17:07] ============================================================
[15:17:07] Testing complete. Ran 122 tests: passed: 106, skipped: 16
[15:17:07] Elapsed time: 32.682s total, 4.445s configuring, 27.971s building, 0.221s running

+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/tests/.kunitconfig
[15:17:07] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[15:17:08] Building KUnit Kernel ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
Building with:
$ make all compile_commands.json ARCH=um O=.kunit --jobs=48
../lib/iomap.c:156:5: warning: no previous prototype for ‘ioread64_lo_hi’ [-Wmissing-prototypes]
  156 | u64 ioread64_lo_hi(const void __iomem *addr)
      |     ^~~~~~~~~~~~~~
../lib/iomap.c:163:5: warning: no previous prototype for ‘ioread64_hi_lo’ [-Wmissing-prototypes]
  163 | u64 ioread64_hi_lo(const void __iomem *addr)
      |     ^~~~~~~~~~~~~~
../lib/iomap.c:170:5: warning: no previous prototype for ‘ioread64be_lo_hi’ [-Wmissing-prototypes]
  170 | u64 ioread64be_lo_hi(const void __iomem *addr)
      |     ^~~~~~~~~~~~~~~~
../lib/iomap.c:178:5: warning: no previous prototype for ‘ioread64be_hi_lo’ [-Wmissing-prototypes]
  178 | u64 ioread64be_hi_lo(const void __iomem *addr)
      |     ^~~~~~~~~~~~~~~~
../lib/iomap.c:264:6: warning: no previous prototype for ‘iowrite64_lo_hi’ [-Wmissing-prototypes]
  264 | void iowrite64_lo_hi(u64 val, void __iomem *addr)
      |      ^~~~~~~~~~~~~~~
../lib/iomap.c:272:6: warning: no previous prototype for ‘iowrite64_hi_lo’ [-Wmissing-prototypes]
  272 | void iowrite64_hi_lo(u64 val, void __iomem *addr)
      |      ^~~~~~~~~~~~~~~
../lib/iomap.c:280:6: warning: no previous prototype for ‘iowrite64be_lo_hi’ [-Wmissing-prototypes]
  280 | void iowrite64be_lo_hi(u64 val, void __iomem *addr)
      |      ^~~~~~~~~~~~~~~~~
../lib/iomap.c:288:6: warning: no previous prototype for ‘iowrite64be_hi_lo’ [-Wmissing-prototypes]
  288 | void iowrite64be_hi_lo(u64 val, void __iomem *addr)
      |      ^~~~~~~~~~~~~~~~~

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

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

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

✗ CI.Build: failure for series starting with [1/9] drm/xe: Remove double pageflip

[Patchwork]

== Series Details ==

Series: series starting with [1/9] drm/xe: Remove double pageflip
URL   : https://patchwork.freedesktop.org/series/140944/
State : failure

== Summary ==

CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml2_dc_resource_mgmt.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml2_mall_phantom.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml_display_rq_dlg_calc.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_top/dml_top.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_top/dml_top_mcache.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_top/dml2_top_optimization.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/inc/dml2_debug.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_core/dml2_core_dcn4.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_core/dml2_core_factory.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_core/dml2_core_dcn4_calcs.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_dpmm/dml2_dpmm_dcn4.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_dpmm/dml2_dpmm_factory.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_mcg/dml2_mcg_dcn4.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_mcg/dml2_mcg_factory.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_factory.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn4_fams2.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_standalone_libraries/lib_float_math.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/dml21_translation_helper.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/dml21_wrapper.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/dml21_utils.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dce120/dce120_timing_generator.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dce112/dce112_compressor.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dce110/dce110_timing_generator.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dce110/dce110_compressor.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dce110/dce110_opp_regamma_v.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dce110/dce110_opp_csc_v.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dce110/dce110_timing_generator_v.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dce110/dce110_mem_input_v.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dce110/dce110_opp_v.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dce110/dce110_transform_v.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dce80/dce80_timing_generator.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dce60/dce60_timing_generator.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dce60/dce60_hw_sequencer.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dce60/dce60_resource.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/hdcp/hdcp_msg.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/spl/dc_spl.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/spl/dc_spl_scl_filters.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/spl/dc_spl_scl_easf_filters.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/spl/dc_spl_isharp_filters.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/spl/dc_spl_filters.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/spl/spl_fixpt31_32.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/spl/spl_custom_float.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_stat.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_resource.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_hw_sequencer.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_sink.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_surface.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_debug.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_stream.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_enc_cfg.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_exports.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_state.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_vm_helper.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dc_helper.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dc_dmub_srv.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dc_edid_parser.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dc_spl_translate.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/modules/freesync/freesync.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/modules/color/color_gamma.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/modules/color/color_table.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/modules/info_packet/info_packet.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/modules/power/power_helpers.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_srv.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_srv_stat.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_reg.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn20.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn21.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn30.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn301.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn302.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn303.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn31.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn314.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn315.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn316.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn32.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn35.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn351.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dmub/src/dmub_dcn401.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/modules/hdcp/hdcp_ddc.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/modules/hdcp/hdcp_log.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/modules/hdcp/hdcp_psp.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/modules/hdcp/hdcp.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/modules/hdcp/hdcp1_execution.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/modules/hdcp/hdcp1_transition.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/modules/hdcp/hdcp2_execution.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/modules/hdcp/hdcp2_transition.o
  LD [M]  drivers/gpu/drm/amd/amdgpu/amdgpu.o
make[5]: *** [../scripts/Makefile.build:478: drivers/gpu/drm] Error 2
make[4]: *** [../scripts/Makefile.build:478: drivers/gpu] Error 2
make[3]: *** [../scripts/Makefile.build:478: drivers] Error 2
make[2]: *** [/kernel/Makefile:1936: .] Error 2
make[1]: Leaving directory '/kernel/build64-default'
make[1]: *** [/kernel/Makefile:224: __sub-make] Error 2
make: *** [Makefile:224: __sub-make] Error 2
+ cleanup
++ stat -c %u:%g /kernel
+ chown -R 1003:1003 /kernel

Re: [PATCH 2/9] drm/xe/display: Add intel_plane_initial_vblank_wait

[kernel test robot]

Hi Maarten,

kernel test robot noticed the following build errors:

[auto build test ERROR on drm-xe/drm-xe-next]
[also build test ERROR on next-20241105]
[cannot apply to linus/master drm-intel/for-linux-next drm-intel/for-linux-next-fixes drm-tip/drm-tip v6.12-rc6]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Maarten-Lankhorst/drm-xe-display-Add-intel_plane_initial_vblank_wait/20241105-202047
base:   https://gitlab.freedesktop.org/drm/xe/kernel.git drm-xe-next
patch link:    https://lore.kernel.org/r/20241105121857.17389-2-maarten.lankhorst%40linux.intel.com
patch subject: [PATCH 2/9] drm/xe/display: Add intel_plane_initial_vblank_wait
config: x86_64-rhel-8.3 (https://download.01.org/0day-ci/archive/20241105/202411052327.1vYXqtrV-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241105/202411052327.1vYXqtrV-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202411052327.1vYXqtrV-lkp@intel.com/

All error/warnings (new ones prefixed by >>):

>> drivers/gpu/drm/i915/display/intel_plane_initial.c:17:1: warning: parameter names (without types) in function declaration
      17 | void intel_plane_initial_vblank_wait(plane);(struct intel_plane *plane)
         | ^~~~
>> drivers/gpu/drm/i915/display/intel_plane_initial.c:17:46: error: expected identifier or '(' before 'struct'
      17 | void intel_plane_initial_vblank_wait(plane);(struct intel_plane *plane)
         |                                              ^~~~~~
   drivers/gpu/drm/i915/display/intel_plane_initial.c: In function 'intel_initial_plane_config':
>> drivers/gpu/drm/i915/display/intel_plane_initial.c:448:25: error: implicit declaration of function 'intel_initial_vblank_wait'; did you mean 'intel_plane_initial_vblank_wait'? [-Werror=implicit-function-declaration]
     448 |                         intel_initial_vblank_wait(crtc);
         |                         ^~~~~~~~~~~~~~~~~~~~~~~~~
         |                         intel_plane_initial_vblank_wait
   cc1: some warnings being treated as errors


vim +17 drivers/gpu/drm/i915/display/intel_plane_initial.c

    16	
  > 17	void intel_plane_initial_vblank_wait(plane);(struct intel_plane *plane)
    18	{
    19		struct intel_crtc *crtc = intel_crtc_for_pipe(to_intel_display(plane), plane->pipe);
    20	
    21		intel_crtc_wait_for_next_vblank(crtc);
    22	}
    23	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

Re: [PATCH 2/9] drm/xe/display: Add intel_plane_initial_vblank_wait

[kernel test robot]

Hi Maarten,

kernel test robot noticed the following build errors:

[auto build test ERROR on drm-xe/drm-xe-next]
[also build test ERROR on next-20241105]
[cannot apply to linus/master drm-intel/for-linux-next drm-intel/for-linux-next-fixes drm-tip/drm-tip v6.12-rc6]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Maarten-Lankhorst/drm-xe-display-Add-intel_plane_initial_vblank_wait/20241105-202047
base:   https://gitlab.freedesktop.org/drm/xe/kernel.git drm-xe-next
patch link:    https://lore.kernel.org/r/20241105121857.17389-2-maarten.lankhorst%40linux.intel.com
patch subject: [PATCH 2/9] drm/xe/display: Add intel_plane_initial_vblank_wait
config: i386-defconfig (https://download.01.org/0day-ci/archive/20241106/202411060402.TB3RmJ09-lkp@intel.com/config)
compiler: clang version 19.1.3 (https://github.com/llvm/llvm-project ab51eccf88f5321e7c60591c5546b254b6afab99)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241106/202411060402.TB3RmJ09-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202411060402.TB3RmJ09-lkp@intel.com/

All errors (new ones prefixed by >>):

   In file included from drivers/gpu/drm/i915/display/intel_plane_initial.c:8:
   In file included from drivers/gpu/drm/i915/i915_drv.h:40:
   In file included from drivers/gpu/drm/i915/display/intel_display_core.h:16:
   In file included from include/drm/drm_connector.h:32:
   In file included from include/drm/drm_util.h:36:
   In file included from include/linux/kgdb.h:19:
   In file included from include/linux/kprobes.h:28:
   In file included from include/linux/ftrace.h:13:
   In file included from include/linux/kallsyms.h:13:
   In file included from include/linux/mm.h:2213:
   include/linux/vmstat.h:518:36: warning: arithmetic between different enumeration types ('enum node_stat_item' and 'enum lru_list') [-Wenum-enum-conversion]
     518 |         return node_stat_name(NR_LRU_BASE + lru) + 3; // skip "nr_"
         |                               ~~~~~~~~~~~ ^ ~~~
>> drivers/gpu/drm/i915/display/intel_plane_initial.c:17:38: error: a parameter list without types is only allowed in a function definition
      17 | void intel_plane_initial_vblank_wait(plane);(struct intel_plane *plane)
         |                                      ^
>> drivers/gpu/drm/i915/display/intel_plane_initial.c:17:46: error: expected identifier or '('
      17 | void intel_plane_initial_vblank_wait(plane);(struct intel_plane *plane)
         |                                              ^
>> drivers/gpu/drm/i915/display/intel_plane_initial.c:17:46: error: expected ')'
   drivers/gpu/drm/i915/display/intel_plane_initial.c:17:45: note: to match this '('
      17 | void intel_plane_initial_vblank_wait(plane);(struct intel_plane *plane)
         |                                             ^
>> drivers/gpu/drm/i915/display/intel_plane_initial.c:448:4: error: call to undeclared function 'intel_initial_vblank_wait'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
     448 |                         intel_initial_vblank_wait(crtc);
         |                         ^
   drivers/gpu/drm/i915/display/intel_plane_initial.c:448:4: note: did you mean 'intel_plane_initial_vblank_wait'?
   drivers/gpu/drm/i915/display/intel_plane_initial.h:11:6: note: 'intel_plane_initial_vblank_wait' declared here
      11 | void intel_plane_initial_vblank_wait(struct intel_plane *plane);
         |      ^
   1 warning and 4 errors generated.


vim +17 drivers/gpu/drm/i915/display/intel_plane_initial.c

    16	
  > 17	void intel_plane_initial_vblank_wait(plane);(struct intel_plane *plane)
    18	{
    19		struct intel_crtc *crtc = intel_crtc_for_pipe(to_intel_display(plane), plane->pipe);
    20	
    21		intel_crtc_wait_for_next_vblank(crtc);
    22	}
    23	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

[PATCH 9/9] drm/xe: Split init of xe_gt_init_hwconfig to xe_gt_init and *_early

[Maarten Lankhorst]

Now that we added the separate step of initialising GUC in
xe_gt_init_early, it should be ok to initialise the minimum during early
init, and the rest after allocations are allowed.

Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20241105121857.17389-9-maarten.lankhorst@linux.intel.com
Signed-off-by: Maarten Lankhorst,,, <dev@lankhorst.se>
---
 drivers/gpu/drm/xe/xe_device.c |  6 ----
 drivers/gpu/drm/xe/xe_gt.c     | 66 +++++++++++++++++-----------------
 drivers/gpu/drm/xe/xe_gt.h     |  1 -
 3 files changed, 32 insertions(+), 41 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_device.c b/drivers/gpu/drm/xe/xe_device.c
index 5dafd48d5478b..e11acd5475867 100644
--- a/drivers/gpu/drm/xe/xe_device.c
+++ b/drivers/gpu/drm/xe/xe_device.c
@@ -654,12 +654,6 @@ int xe_device_probe(struct xe_device *xe)
 			return err;
 	}
 
-	for_each_gt(gt, xe, id) {
-		err = xe_gt_init_hwconfig(gt);
-		if (err)
-			return err;
-	}
-
 	err = xe_devcoredump_init(xe);
 	if (err)
 		return err;
diff --git a/drivers/gpu/drm/xe/xe_gt.c b/drivers/gpu/drm/xe/xe_gt.c
index dc6901255dc91..a6154d1bad9f0 100644
--- a/drivers/gpu/drm/xe/xe_gt.c
+++ b/drivers/gpu/drm/xe/xe_gt.c
@@ -364,6 +364,21 @@ int xe_gt_record_default_lrcs(struct xe_gt *gt)
 	return err;
 }
 
+static int gt_init_mcr(struct xe_gt *gt)
+{
+	unsigned int fw_ref;
+
+	fw_ref = xe_force_wake_get(gt_to_fw(gt), XE_FW_GT);
+	if (!fw_ref)
+		return -ETIMEDOUT;
+
+	xe_gt_mcr_init_early(gt);
+	xe_pat_init(gt);
+
+	xe_force_wake_put(gt_to_fw(gt), fw_ref);
+	return 0;
+}
+
 int xe_gt_init_early(struct xe_gt *gt)
 {
 	int err;
@@ -394,7 +409,11 @@ int xe_gt_init_early(struct xe_gt *gt)
 	 */
 	xe_gt_mmio_init(gt);
 
-	return xe_uc_init_noalloc(&gt->uc);
+	err = xe_uc_init_noalloc(&gt->uc);
+	if (err)
+		return err;
+
+	return gt_init_mcr(gt);
 }
 
 static void dump_pat_on_error(struct xe_gt *gt)
@@ -419,6 +438,18 @@ static int gt_fw_domain_init(struct xe_gt *gt)
 		goto err_hw_fence_irq;
 	}
 
+	err = xe_uc_init(&gt->uc);
+	if (err)
+		goto err_force_wake;
+
+	err = xe_uc_init_hwconfig(&gt->uc);
+	if (err)
+		goto err_force_wake;
+
+	xe_gt_topology_init(gt);
+	xe_gt_mcr_init(gt);
+	xe_gt_enable_host_l2_vram(gt);
+
 	if (!xe_gt_is_media_type(gt)) {
 		err = xe_ggtt_init(gt_to_tile(gt)->mem.ggtt);
 		if (err)
@@ -550,39 +581,6 @@ static int all_fw_domain_init(struct xe_gt *gt)
 	return err;
 }
 
-/*
- * Initialize enough GT to be able to load GuC in order to obtain hwconfig and
- * enable CTB communication.
- */
-int xe_gt_init_hwconfig(struct xe_gt *gt)
-{
-	unsigned int fw_ref;
-	int err;
-
-	fw_ref = xe_force_wake_get(gt_to_fw(gt), XE_FW_GT);
-	if (!fw_ref)
-		return -ETIMEDOUT;
-
-	xe_gt_mcr_init_early(gt);
-	xe_pat_init(gt);
-
-	err = xe_uc_init(&gt->uc);
-	if (err)
-		goto out_fw;
-
-	err = xe_uc_init_hwconfig(&gt->uc);
-	if (err)
-		goto out_fw;
-
-	xe_gt_topology_init(gt);
-	xe_gt_mcr_init(gt);
-	xe_gt_enable_host_l2_vram(gt);
-
-out_fw:
-	xe_force_wake_put(gt_to_fw(gt), fw_ref);
-	return err;
-}
-
 int xe_gt_init(struct xe_gt *gt)
 {
 	int err;
diff --git a/drivers/gpu/drm/xe/xe_gt.h b/drivers/gpu/drm/xe/xe_gt.h
index 82b9b7f82fcab..0c026cc91ffe9 100644
--- a/drivers/gpu/drm/xe/xe_gt.h
+++ b/drivers/gpu/drm/xe/xe_gt.h
@@ -28,7 +28,6 @@ static inline bool xe_fault_inject_gt_reset(void)
 }
 
 struct xe_gt *xe_gt_alloc(struct xe_tile *tile);
-int xe_gt_init_hwconfig(struct xe_gt *gt);
 int xe_gt_init_early(struct xe_gt *gt);
 int xe_gt_init(struct xe_gt *gt);
 void xe_gt_mmio_init(struct xe_gt *gt);
-- 
2.45.2