Intel-XE Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 3/3] drm/xe: Use i915-display shutdown sequence directly
  2024-11-14 20:22 [PATCH 1/3] drm/i915/display: Move shutdown sequences under display driver Rodrigo Vivi
@ 2024-11-14 20:22 ` Rodrigo Vivi
  0 siblings, 0 replies; 12+ messages in thread
From: Rodrigo Vivi @ 2024-11-14 20:22 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: Rodrigo Vivi, Jonathan Cavitt

Start the xe-i915-display reconciliation by using the same
shutdown sequences.

Reviewed-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
---
 drivers/gpu/drm/xe/display/xe_display.c | 46 +++++++------------------
 drivers/gpu/drm/xe/display/xe_display.h |  5 +--
 drivers/gpu/drm/xe/xe_device.c          |  4 ++-
 3 files changed, 19 insertions(+), 36 deletions(-)

diff --git a/drivers/gpu/drm/xe/display/xe_display.c b/drivers/gpu/drm/xe/display/xe_display.c
index 4759c3cf5915..29676652bc54 100644
--- a/drivers/gpu/drm/xe/display/xe_display.c
+++ b/drivers/gpu/drm/xe/display/xe_display.c
@@ -10,7 +10,6 @@
 
 #include <drm/drm_drv.h>
 #include <drm/drm_managed.h>
-#include <drm/drm_atomic_helper.h>
 #include <drm/drm_probe_helper.h>
 #include <uapi/drm/xe_drm.h>
 
@@ -357,32 +356,26 @@ void xe_display_pm_suspend(struct xe_device *xe)
 
 void xe_display_pm_shutdown(struct xe_device *xe)
 {
-	struct intel_display *display = &xe->display;
-
 	if (!xe->info.probe_display)
 		return;
 
-	intel_power_domains_disable(xe);
-	intel_fbdev_set_suspend(&xe->drm, FBINFO_STATE_SUSPENDED, true);
-	if (has_display(xe)) {
-		drm_kms_helper_poll_disable(&xe->drm);
-		intel_display_driver_disable_user_access(xe);
-
-		drm_atomic_helper_shutdown(&xe->drm);
-	}
-
-	intel_dp_mst_suspend(xe);
-	intel_hpd_cancel_work(xe);
+	intel_display_driver_shutdown(xe);
+}
 
-	if (has_display(xe))
-		intel_display_driver_suspend_access(xe);
+void xe_display_pm_shutdown_noirq(struct xe_device *xe)
+{
+	if (!xe->info.probe_display)
+		return;
 
-	intel_encoder_suspend_all(display);
-	intel_encoder_shutdown_all(display);
+	intel_display_driver_shutdown_noirq(xe);
+}
 
-	intel_opregion_suspend(display, PCI_D3cold);
+void xe_display_pm_shutdown_noaccel(struct xe_device *xe)
+{
+	if (!xe->info.probe_display)
+		return;
 
-	intel_dmc_suspend(display);
+	intel_display_driver_shutdown_nogem(xe);
 }
 
 void xe_display_pm_runtime_suspend(struct xe_device *xe)
@@ -408,19 +401,6 @@ void xe_display_pm_suspend_late(struct xe_device *xe)
 	intel_display_power_suspend_late(xe, s2idle);
 }
 
-void xe_display_pm_shutdown_late(struct xe_device *xe)
-{
-	if (!xe->info.probe_display)
-		return;
-
-	/*
-	 * The only requirement is to reboot with display DC states disabled,
-	 * for now leaving all display power wells in the INIT power domain
-	 * enabled.
-	 */
-	intel_power_domains_driver_remove(xe);
-}
-
 void xe_display_pm_resume_early(struct xe_device *xe)
 {
 	if (!xe->info.probe_display)
diff --git a/drivers/gpu/drm/xe/display/xe_display.h b/drivers/gpu/drm/xe/display/xe_display.h
index 17afa537aee5..a801db19b64f 100644
--- a/drivers/gpu/drm/xe/display/xe_display.h
+++ b/drivers/gpu/drm/xe/display/xe_display.h
@@ -35,9 +35,10 @@ void xe_display_irq_reset(struct xe_device *xe);
 void xe_display_irq_postinstall(struct xe_device *xe, struct xe_gt *gt);
 
 void xe_display_pm_suspend(struct xe_device *xe);
-void xe_display_pm_shutdown(struct xe_device *xe);
 void xe_display_pm_suspend_late(struct xe_device *xe);
-void xe_display_pm_shutdown_late(struct xe_device *xe);
+void xe_display_pm_shutdown(struct xe_device *xe);
+void xe_display_pm_shutdown_noirq(struct xe_device *xe);
+void xe_display_pm_shutdown_noaccel(struct xe_device *xe);
 void xe_display_pm_resume_early(struct xe_device *xe);
 void xe_display_pm_resume(struct xe_device *xe);
 void xe_display_pm_runtime_suspend(struct xe_device *xe);
diff --git a/drivers/gpu/drm/xe/xe_device.c b/drivers/gpu/drm/xe/xe_device.c
index 0e2dd691bdae..f4ae8e11ac53 100644
--- a/drivers/gpu/drm/xe/xe_device.c
+++ b/drivers/gpu/drm/xe/xe_device.c
@@ -819,10 +819,12 @@ void xe_device_shutdown(struct xe_device *xe)
 
 		xe_irq_suspend(xe);
 
+		xe_display_pm_shutdown_noirq(xe);
+
 		for_each_gt(gt, xe, id)
 			xe_gt_shutdown(gt);
 
-		xe_display_pm_shutdown_late(xe);
+		xe_display_pm_shutdown_noaccel(xe);
 	} else {
 		/* BOOM! */
 		__xe_driver_flr(xe);
-- 
2.47.0


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

* [PATCH 1/3] drm/i915/display: Move shutdown sequences under display driver
@ 2025-01-17 22:09 Rodrigo Vivi
  2025-01-17 22:09 ` [PATCH 2/3] drm/xe: At shutdown disable commit helpers instead of flushing Rodrigo Vivi
                   ` (5 more replies)
  0 siblings, 6 replies; 12+ messages in thread
From: Rodrigo Vivi @ 2025-01-17 22:09 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: Rodrigo Vivi, Imre Deak, Jonathan Cavitt

Move display related shutdown sequences from i915_driver to
intel_display_driver.

No functional change. Just taking the right ownership and
start some reconciliation of them between i915 and Xe.

v2: - Add missing _nogem caller (Imre)
    - Fix comment style (Jonathan)
v3: rebase

Cc: Imre Deak <imre.deak@intel.com>
Reviewed-by: Jonathan Cavitt <jonathan.cavitt@intel.com> #v1
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
---
 .../drm/i915/display/intel_display_driver.c   | 40 +++++++++++++++++++
 .../drm/i915/display/intel_display_driver.h   |  3 ++
 drivers/gpu/drm/i915/i915_driver.c            | 27 ++-----------
 3 files changed, 47 insertions(+), 23 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_display_driver.c b/drivers/gpu/drm/i915/display/intel_display_driver.c
index 73ea0e906014..718b8b92276f 100644
--- a/drivers/gpu/drm/i915/display/intel_display_driver.c
+++ b/drivers/gpu/drm/i915/display/intel_display_driver.c
@@ -39,6 +39,7 @@
 #include "intel_dp_tunnel.h"
 #include "intel_dpll.h"
 #include "intel_dpll_mgr.h"
+#include "intel_encoder.h"
 #include "intel_fb.h"
 #include "intel_fbc.h"
 #include "intel_fbdev.h"
@@ -769,3 +770,42 @@ void intel_display_driver_resume(struct intel_display *display)
 	if (state)
 		drm_atomic_state_put(state);
 }
+
+void intel_display_driver_shutdown(struct intel_display *display)
+{
+	intel_power_domains_disable(display);
+
+	intel_fbdev_set_suspend(display->drm, FBINFO_STATE_SUSPENDED, true);
+	if (HAS_DISPLAY(display)) {
+		drm_kms_helper_poll_disable(display->drm);
+		intel_display_driver_disable_user_access(display);
+
+		drm_atomic_helper_shutdown(display->drm);
+	}
+
+	intel_dp_mst_suspend(display);
+}
+
+void intel_display_driver_shutdown_noirq(struct intel_display *display)
+{
+	intel_hpd_cancel_work(display);
+
+	if (HAS_DISPLAY(display))
+		intel_display_driver_suspend_access(display);
+
+	intel_encoder_suspend_all(display);
+	intel_encoder_shutdown_all(display);
+
+	intel_dmc_suspend(display);
+}
+
+void intel_display_driver_shutdown_nogem(struct intel_display *display)
+{
+	/*
+	 * The only requirement is to reboot with display DC states disabled,
+	 * for now leaving all display power wells in the INIT power domain
+	 * enabled.
+	 */
+
+	intel_power_domains_driver_remove(display);
+}
diff --git a/drivers/gpu/drm/i915/display/intel_display_driver.h b/drivers/gpu/drm/i915/display/intel_display_driver.h
index 2966ff91b219..f155a43e2377 100644
--- a/drivers/gpu/drm/i915/display/intel_display_driver.h
+++ b/drivers/gpu/drm/i915/display/intel_display_driver.h
@@ -26,6 +26,9 @@ void intel_display_driver_remove_nogem(struct intel_display *display);
 void intel_display_driver_unregister(struct intel_display *display);
 int intel_display_driver_suspend(struct intel_display *display);
 void intel_display_driver_resume(struct intel_display *display);
+void intel_display_driver_shutdown(struct intel_display *display);
+void intel_display_driver_shutdown_noirq(struct intel_display *display);
+void intel_display_driver_shutdown_nogem(struct intel_display *display);
 
 /* interface for intel_display_reset.c */
 int __intel_display_driver_resume(struct intel_display *display,
diff --git a/drivers/gpu/drm/i915/i915_driver.c b/drivers/gpu/drm/i915/i915_driver.c
index c2ae37d6b94d..cb7453393a21 100644
--- a/drivers/gpu/drm/i915/i915_driver.c
+++ b/drivers/gpu/drm/i915/i915_driver.c
@@ -971,43 +971,24 @@ void i915_driver_shutdown(struct drm_i915_private *i915)
 
 	disable_rpm_wakeref_asserts(&i915->runtime_pm);
 	intel_runtime_pm_disable(&i915->runtime_pm);
-	intel_power_domains_disable(display);
-
-	intel_fbdev_set_suspend(&i915->drm, FBINFO_STATE_SUSPENDED, true);
-	if (HAS_DISPLAY(i915)) {
-		drm_kms_helper_poll_disable(&i915->drm);
-		intel_display_driver_disable_user_access(display);
-
-		drm_atomic_helper_shutdown(&i915->drm);
-	}
 
-	intel_dp_mst_suspend(display);
+	intel_display_driver_shutdown(display);
 
 	intel_irq_suspend(i915);
-	intel_hpd_cancel_work(i915);
 
-	if (HAS_DISPLAY(i915))
-		intel_display_driver_suspend_access(display);
-
-	intel_encoder_suspend_all(&i915->display);
-	intel_encoder_shutdown_all(&i915->display);
-
-	intel_dmc_suspend(&i915->display);
+	intel_display_driver_shutdown_noirq(display);
 
 	i915_gem_suspend(i915);
 
 	/*
-	 * The only requirement is to reboot with display DC states disabled,
-	 * for now leaving all display power wells in the INIT power domain
-	 * enabled.
-	 *
 	 * TODO:
 	 * - unify the pci_driver::shutdown sequence here with the
 	 *   pci_driver.driver.pm.poweroff,poweroff_late sequence.
 	 * - unify the driver remove and system/runtime suspend sequences with
 	 *   the above unified shutdown/poweroff sequence.
 	 */
-	intel_power_domains_driver_remove(display);
+	intel_display_driver_shutdown_nogem(display);
+
 	enable_rpm_wakeref_asserts(&i915->runtime_pm);
 
 	intel_runtime_pm_driver_last_release(&i915->runtime_pm);
-- 
2.48.1


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

* [PATCH 2/3] drm/xe: At shutdown disable commit helpers instead of flushing
  2025-01-17 22:09 [PATCH 1/3] drm/i915/display: Move shutdown sequences under display driver Rodrigo Vivi
@ 2025-01-17 22:09 ` Rodrigo Vivi
  2025-01-17 22:09 ` [PATCH 3/3] drm/xe: Use i915-display shutdown sequence directly Rodrigo Vivi
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 12+ messages in thread
From: Rodrigo Vivi @ 2025-01-17 22:09 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: Rodrigo Vivi, Maarten Lankhort, Jonathan Cavitt

This aligns with the current i915 display sequence.

Cc: Maarten Lankhort <maarten.lankhorst@linux.intel.com>
Reviewed-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
---
 drivers/gpu/drm/xe/display/xe_display.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/xe/display/xe_display.c b/drivers/gpu/drm/xe/display/xe_display.c
index 96ba9595bf2a..4f60d7bd7742 100644
--- a/drivers/gpu/drm/xe/display/xe_display.c
+++ b/drivers/gpu/drm/xe/display/xe_display.c
@@ -10,6 +10,7 @@
 
 #include <drm/drm_drv.h>
 #include <drm/drm_managed.h>
+#include <drm/drm_atomic_helper.h>
 #include <drm/drm_probe_helper.h>
 #include <uapi/drm/xe_drm.h>
 
@@ -378,10 +379,10 @@ void xe_display_pm_shutdown(struct xe_device *xe)
 	if (has_display(xe)) {
 		drm_kms_helper_poll_disable(&xe->drm);
 		intel_display_driver_disable_user_access(display);
-		intel_display_driver_suspend(display);
+
+		drm_atomic_helper_shutdown(display->drm);
 	}
 
-	xe_display_flush_cleanup_work(xe);
 	intel_dp_mst_suspend(display);
 	intel_hpd_cancel_work(xe);
 
-- 
2.48.1


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

* [PATCH 3/3] drm/xe: Use i915-display shutdown sequence directly
  2025-01-17 22:09 [PATCH 1/3] drm/i915/display: Move shutdown sequences under display driver Rodrigo Vivi
  2025-01-17 22:09 ` [PATCH 2/3] drm/xe: At shutdown disable commit helpers instead of flushing Rodrigo Vivi
@ 2025-01-17 22:09 ` Rodrigo Vivi
  2025-01-20 13:42   ` Maarten Lankhorst
  2025-01-17 23:11 ` ✓ CI.Patch_applied: success for series starting with [1/3] drm/i915/display: Move shutdown sequences under display driver Patchwork
                   ` (3 subsequent siblings)
  5 siblings, 1 reply; 12+ messages in thread
From: Rodrigo Vivi @ 2025-01-17 22:09 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: Rodrigo Vivi, Jonathan Cavitt

Start the xe-i915-display reconciliation by using the same
shutdown sequences.

v2: include the stubs for !CONFIG_DRM_XE_DISPLAY (Kunit)

Reviewed-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
---
 drivers/gpu/drm/xe/display/xe_display.c | 48 +++++++------------------
 drivers/gpu/drm/xe/display/xe_display.h | 10 +++---
 drivers/gpu/drm/xe/xe_device.c          |  4 ++-
 3 files changed, 22 insertions(+), 40 deletions(-)

diff --git a/drivers/gpu/drm/xe/display/xe_display.c b/drivers/gpu/drm/xe/display/xe_display.c
index 4f60d7bd7742..e1ce9eb3332d 100644
--- a/drivers/gpu/drm/xe/display/xe_display.c
+++ b/drivers/gpu/drm/xe/display/xe_display.c
@@ -10,7 +10,6 @@
 
 #include <drm/drm_drv.h>
 #include <drm/drm_managed.h>
-#include <drm/drm_atomic_helper.h>
 #include <drm/drm_probe_helper.h>
 #include <uapi/drm/xe_drm.h>
 
@@ -369,32 +368,26 @@ void xe_display_pm_suspend(struct xe_device *xe)
 
 void xe_display_pm_shutdown(struct xe_device *xe)
 {
-	struct intel_display *display = &xe->display;
-
 	if (!xe->info.probe_display)
 		return;
 
-	intel_power_domains_disable(display);
-	intel_fbdev_set_suspend(&xe->drm, FBINFO_STATE_SUSPENDED, true);
-	if (has_display(xe)) {
-		drm_kms_helper_poll_disable(&xe->drm);
-		intel_display_driver_disable_user_access(display);
-
-		drm_atomic_helper_shutdown(display->drm);
-	}
-
-	intel_dp_mst_suspend(display);
-	intel_hpd_cancel_work(xe);
+	intel_display_driver_shutdown(&xe->display);
+}
 
-	if (has_display(xe))
-		intel_display_driver_suspend_access(display);
+void xe_display_pm_shutdown_noirq(struct xe_device *xe)
+{
+	if (!xe->info.probe_display)
+		return;
 
-	intel_encoder_suspend_all(display);
-	intel_encoder_shutdown_all(display);
+	intel_display_driver_shutdown_noirq(&xe->display);
+}
 
-	intel_opregion_suspend(display, PCI_D3cold);
+void xe_display_pm_shutdown_noaccel(struct xe_device *xe)
+{
+	if (!xe->info.probe_display)
+		return;
 
-	intel_dmc_suspend(display);
+	intel_display_driver_shutdown_nogem(&xe->display);
 }
 
 void xe_display_pm_runtime_suspend(struct xe_device *xe)
@@ -439,21 +432,6 @@ void xe_display_pm_runtime_suspend_late(struct xe_device *xe)
 	intel_dmc_wl_flush_release_work(display);
 }
 
-void xe_display_pm_shutdown_late(struct xe_device *xe)
-{
-	struct intel_display *display = &xe->display;
-
-	if (!xe->info.probe_display)
-		return;
-
-	/*
-	 * The only requirement is to reboot with display DC states disabled,
-	 * for now leaving all display power wells in the INIT power domain
-	 * enabled.
-	 */
-	intel_power_domains_driver_remove(display);
-}
-
 void xe_display_pm_resume_early(struct xe_device *xe)
 {
 	struct intel_display *display = &xe->display;
diff --git a/drivers/gpu/drm/xe/display/xe_display.h b/drivers/gpu/drm/xe/display/xe_display.h
index 233f81a26c25..a15ec29b862b 100644
--- a/drivers/gpu/drm/xe/display/xe_display.h
+++ b/drivers/gpu/drm/xe/display/xe_display.h
@@ -35,9 +35,10 @@ void xe_display_irq_reset(struct xe_device *xe);
 void xe_display_irq_postinstall(struct xe_device *xe, struct xe_gt *gt);
 
 void xe_display_pm_suspend(struct xe_device *xe);
-void xe_display_pm_shutdown(struct xe_device *xe);
 void xe_display_pm_suspend_late(struct xe_device *xe);
-void xe_display_pm_shutdown_late(struct xe_device *xe);
+void xe_display_pm_shutdown(struct xe_device *xe);
+void xe_display_pm_shutdown_noirq(struct xe_device *xe);
+void xe_display_pm_shutdown_noaccel(struct xe_device *xe);
 void xe_display_pm_resume_early(struct xe_device *xe);
 void xe_display_pm_resume(struct xe_device *xe);
 void xe_display_pm_runtime_suspend(struct xe_device *xe);
@@ -69,9 +70,10 @@ static inline void xe_display_irq_reset(struct xe_device *xe) {}
 static inline void xe_display_irq_postinstall(struct xe_device *xe, struct xe_gt *gt) {}
 
 static inline void xe_display_pm_suspend(struct xe_device *xe) {}
-static inline void xe_display_pm_shutdown(struct xe_device *xe) {}
 static inline void xe_display_pm_suspend_late(struct xe_device *xe) {}
-static inline void xe_display_pm_shutdown_late(struct xe_device *xe) {}
+static inline void xe_display_pm_shutdown(struct xe_device *xe) {}
+static inline void xe_display_pm_shutdown_noirq(struct xe_device *xe) {}
+static inline void xe_display_pm_shutdown_noaccel(struct xe_device *xe) {}
 static inline void xe_display_pm_resume_early(struct xe_device *xe) {}
 static inline void xe_display_pm_resume(struct xe_device *xe) {}
 static inline void xe_display_pm_runtime_suspend(struct xe_device *xe) {}
diff --git a/drivers/gpu/drm/xe/xe_device.c b/drivers/gpu/drm/xe/xe_device.c
index 0966d9697caf..53cac055a2a9 100644
--- a/drivers/gpu/drm/xe/xe_device.c
+++ b/drivers/gpu/drm/xe/xe_device.c
@@ -934,10 +934,12 @@ void xe_device_shutdown(struct xe_device *xe)
 
 		xe_irq_suspend(xe);
 
+		xe_display_pm_shutdown_noirq(xe);
+
 		for_each_gt(gt, xe, id)
 			xe_gt_shutdown(gt);
 
-		xe_display_pm_shutdown_late(xe);
+		xe_display_pm_shutdown_noaccel(xe);
 	} else {
 		/* BOOM! */
 		__xe_driver_flr(xe);
-- 
2.48.1


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

* ✓ CI.Patch_applied: success for series starting with [1/3] drm/i915/display: Move shutdown sequences under display driver
  2025-01-17 22:09 [PATCH 1/3] drm/i915/display: Move shutdown sequences under display driver Rodrigo Vivi
  2025-01-17 22:09 ` [PATCH 2/3] drm/xe: At shutdown disable commit helpers instead of flushing Rodrigo Vivi
  2025-01-17 22:09 ` [PATCH 3/3] drm/xe: Use i915-display shutdown sequence directly Rodrigo Vivi
@ 2025-01-17 23:11 ` Patchwork
  2025-01-17 23:11 ` ✓ CI.checkpatch: " Patchwork
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 12+ messages in thread
From: Patchwork @ 2025-01-17 23:11 UTC (permalink / raw)
  To: Rodrigo Vivi; +Cc: intel-xe

== Series Details ==

Series: series starting with [1/3] drm/i915/display: Move shutdown sequences under display driver
URL   : https://patchwork.freedesktop.org/series/143697/
State : success

== Summary ==

=== Applying kernel patches on branch 'drm-tip' with base: ===
Base commit: 4fc988de9f5e drm-tip: 2025y-01m-17d-19h-57m-18s UTC integration manifest
=== git am output follows ===
Applying: drm/i915/display: Move shutdown sequences under display driver
Applying: drm/xe: At shutdown disable commit helpers instead of flushing
Applying: drm/xe: Use i915-display shutdown sequence directly



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

* ✓ CI.checkpatch: success for series starting with [1/3] drm/i915/display: Move shutdown sequences under display driver
  2025-01-17 22:09 [PATCH 1/3] drm/i915/display: Move shutdown sequences under display driver Rodrigo Vivi
                   ` (2 preceding siblings ...)
  2025-01-17 23:11 ` ✓ CI.Patch_applied: success for series starting with [1/3] drm/i915/display: Move shutdown sequences under display driver Patchwork
@ 2025-01-17 23:11 ` Patchwork
  2025-01-17 23:13 ` ✓ CI.KUnit: " Patchwork
  2025-01-17 23:20 ` ✗ CI.Build: failure " Patchwork
  5 siblings, 0 replies; 12+ messages in thread
From: Patchwork @ 2025-01-17 23:11 UTC (permalink / raw)
  To: Rodrigo Vivi; +Cc: intel-xe

== Series Details ==

Series: series starting with [1/3] drm/i915/display: Move shutdown sequences under display driver
URL   : https://patchwork.freedesktop.org/series/143697/
State : success

== 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 9cc0349fbbaabf8205abfa7e727f0386d2d98da4
Author: Rodrigo Vivi <rodrigo.vivi@intel.com>
Date:   Fri Jan 17 17:09:43 2025 -0500

    drm/xe: Use i915-display shutdown sequence directly
    
    Start the xe-i915-display reconciliation by using the same
    shutdown sequences.
    
    v2: include the stubs for !CONFIG_DRM_XE_DISPLAY (Kunit)
    
    Reviewed-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
    Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
+ /mt/dim checkpatch 4fc988de9f5e17d19edb5fc0a0fbc15a8fc837f4 drm-intel
1f65f0a41b5f drm/i915/display: Move shutdown sequences under display driver
638db9540b50 drm/xe: At shutdown disable commit helpers instead of flushing
9cc0349fbbaa drm/xe: Use i915-display shutdown sequence directly



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

* ✓ CI.KUnit: success for series starting with [1/3] drm/i915/display: Move shutdown sequences under display driver
  2025-01-17 22:09 [PATCH 1/3] drm/i915/display: Move shutdown sequences under display driver Rodrigo Vivi
                   ` (3 preceding siblings ...)
  2025-01-17 23:11 ` ✓ CI.checkpatch: " Patchwork
@ 2025-01-17 23:13 ` Patchwork
  2025-01-17 23:20 ` ✗ CI.Build: failure " Patchwork
  5 siblings, 0 replies; 12+ messages in thread
From: Patchwork @ 2025-01-17 23:13 UTC (permalink / raw)
  To: Rodrigo Vivi; +Cc: intel-xe

== Series Details ==

Series: series starting with [1/3] drm/i915/display: Move shutdown sequences under display driver
URL   : https://patchwork.freedesktop.org/series/143697/
State : success

== Summary ==

+ trap cleanup EXIT
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/xe/.kunitconfig
[23:11:55] Configuring KUnit Kernel ...
Generating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[23:11:59] 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)
      |      ^~~~~~~~~~~~~~~~~

[23:12:28] Starting KUnit Kernel (1/1)...
[23:12:28] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[23:12:28] =================== guc_dbm (7 subtests) ===================
[23:12:28] [PASSED] test_empty
[23:12:28] [PASSED] test_default
[23:12:28] ======================== test_size  ========================
[23:12:28] [PASSED] 4
[23:12:28] [PASSED] 8
[23:12:28] [PASSED] 32
[23:12:28] [PASSED] 256
[23:12:28] ==================== [PASSED] test_size ====================
[23:12:28] ======================= test_reuse  ========================
[23:12:28] [PASSED] 4
[23:12:28] [PASSED] 8
[23:12:28] [PASSED] 32
[23:12:28] [PASSED] 256
[23:12:28] =================== [PASSED] test_reuse ====================
[23:12:28] =================== test_range_overlap  ====================
[23:12:28] [PASSED] 4
[23:12:28] [PASSED] 8
[23:12:28] [PASSED] 32
[23:12:28] [PASSED] 256
[23:12:28] =============== [PASSED] test_range_overlap ================
[23:12:28] =================== test_range_compact  ====================
[23:12:28] [PASSED] 4
[23:12:28] [PASSED] 8
[23:12:28] [PASSED] 32
[23:12:28] [PASSED] 256
[23:12:28] =============== [PASSED] test_range_compact ================
[23:12:28] ==================== test_range_spare  =====================
[23:12:28] [PASSED] 4
[23:12:28] [PASSED] 8
[23:12:28] [PASSED] 32
[23:12:28] [PASSED] 256
[23:12:28] ================ [PASSED] test_range_spare =================
[23:12:28] ===================== [PASSED] guc_dbm =====================
[23:12:28] =================== guc_idm (6 subtests) ===================
[23:12:28] [PASSED] bad_init
[23:12:28] [PASSED] no_init
[23:12:28] [PASSED] init_fini
[23:12:28] [PASSED] check_used
[23:12:28] [PASSED] check_quota
[23:12:28] [PASSED] check_all
[23:12:28] ===================== [PASSED] guc_idm =====================
[23:12:28] ================== no_relay (3 subtests) ===================
[23:12:28] [PASSED] xe_drops_guc2pf_if_not_ready
[23:12:28] [PASSED] xe_drops_guc2vf_if_not_ready
[23:12:28] [PASSED] xe_rejects_send_if_not_ready
[23:12:28] ==================== [PASSED] no_relay =====================
[23:12:28] ================== pf_relay (14 subtests) ==================
[23:12:28] [PASSED] pf_rejects_guc2pf_too_short
[23:12:28] [PASSED] pf_rejects_guc2pf_too_long
[23:12:28] [PASSED] pf_rejects_guc2pf_no_payload
[23:12:28] [PASSED] pf_fails_no_payload
[23:12:28] [PASSED] pf_fails_bad_origin
[23:12:28] [PASSED] pf_fails_bad_type
[23:12:28] [PASSED] pf_txn_reports_error
[23:12:28] [PASSED] pf_txn_sends_pf2guc
[23:12:28] [PASSED] pf_sends_pf2guc
[23:12:28] [SKIPPED] pf_loopback_nop
[23:12:28] [SKIPPED] pf_loopback_echo
[23:12:28] [SKIPPED] pf_loopback_fail
[23:12:28] [SKIPPED] pf_loopback_busy
[23:12:28] [SKIPPED] pf_loopback_retry
[23:12:28] ==================== [PASSED] pf_relay =====================
[23:12:28] ================== vf_relay (3 subtests) ===================
[23:12:28] [PASSED] vf_rejects_guc2vf_too_short
[23:12:28] [PASSED] vf_rejects_guc2vf_too_long
[23:12:28] [PASSED] vf_rejects_guc2vf_no_payload
[23:12:28] ==================== [PASSED] vf_relay =====================
[23:12:28] ================= pf_service (11 subtests) =================
[23:12:28] [PASSED] pf_negotiate_any
[23:12:28] [PASSED] pf_negotiate_base_match
[23:12:28] [PASSED] pf_negotiate_base_newer
[23:12:28] [PASSED] pf_negotiate_base_next
[23:12:28] [SKIPPED] pf_negotiate_base_older
[23:12:28] [PASSED] pf_negotiate_base_prev
[23:12:28] [PASSED] pf_negotiate_latest_match
[23:12:28] [PASSED] pf_negotiate_latest_newer
[23:12:28] [PASSED] pf_negotiate_latest_next
[23:12:28] [SKIPPED] pf_negotiate_latest_older
[23:12:28] [SKIPPED] pf_negotiate_latest_prev
[23:12:28] =================== [PASSED] pf_service ====================
[23:12:28] ===================== lmtt (1 subtest) =====================
[23:12:28] ======================== test_ops  =========================
[23:12:28] [PASSED] 2-level
[23:12:28] [PASSED] multi-level
[23:12:28] ==================== [PASSED] test_ops =====================
[23:12:28] ====================== [PASSED] lmtt =======================
[23:12:28] =================== xe_mocs (2 subtests) ===================
[23:12:28] ================ xe_live_mocs_kernel_kunit  ================
[23:12:28] =========== [SKIPPED] xe_live_mocs_kernel_kunit ============
[23:12:28] ================ xe_live_mocs_reset_kunit  =================
[23:12:28] ============ [SKIPPED] xe_live_mocs_reset_kunit ============
[23:12:28] ==================== [SKIPPED] xe_mocs =====================
[23:12:28] ================= xe_migrate (2 subtests) ==================
[23:12:28] ================= xe_migrate_sanity_kunit  =================
[23:12:28] ============ [SKIPPED] xe_migrate_sanity_kunit =============
[23:12:28] ================== xe_validate_ccs_kunit  ==================
[23:12:28] ============= [SKIPPED] xe_validate_ccs_kunit ==============
[23:12:28] =================== [SKIPPED] xe_migrate ===================
[23:12:28] ================== xe_dma_buf (1 subtest) ==================
[23:12:28] ==================== xe_dma_buf_kunit  =====================
[23:12:28] ================ [SKIPPED] xe_dma_buf_kunit ================
[23:12:28] =================== [SKIPPED] xe_dma_buf ===================
[23:12:28] ================= xe_bo_shrink (1 subtest) =================
[23:12:28] =================== xe_bo_shrink_kunit  ====================
[23:12:28] =============== [SKIPPED] xe_bo_shrink_kunit ===============
[23:12:28] ================== [SKIPPED] xe_bo_shrink ==================
[23:12:28] ==================== xe_bo (2 subtests) ====================
[23:12:28] ================== xe_ccs_migrate_kunit  ===================
[23:12:28] ============== [SKIPPED] xe_ccs_migrate_kunit ==============
[23:12:28] ==================== xe_bo_evict_kunit  ====================
[23:12:28] =============== [SKIPPED] xe_bo_evict_kunit ================
[23:12:28] ===================== [SKIPPED] xe_bo ======================
[23:12:28] ==================== args (11 subtests) ====================
[23:12:28] [PASSED] count_args_test
[23:12:28] [PASSED] call_args_example
[23:12:28] [PASSED] call_args_test
[23:12:28] [PASSED] drop_first_arg_example
[23:12:28] [PASSED] drop_first_arg_test
[23:12:28] [PASSED] first_arg_example
[23:12:28] [PASSED] first_arg_test
stty: 'standard input': Inappropriate ioctl for device
[23:12:28] [PASSED] last_arg_example
[23:12:28] [PASSED] last_arg_test
[23:12:28] [PASSED] pick_arg_example
[23:12:28] [PASSED] sep_comma_example
[23:12:28] ====================== [PASSED] args =======================
[23:12:28] =================== xe_pci (2 subtests) ====================
[23:12:28] [PASSED] xe_gmdid_graphics_ip
[23:12:28] [PASSED] xe_gmdid_media_ip
[23:12:28] ===================== [PASSED] xe_pci ======================
[23:12:28] =================== xe_rtp (2 subtests) ====================
[23:12:28] =============== xe_rtp_process_to_sr_tests  ================
[23:12:28] [PASSED] coalesce-same-reg
[23:12:28] [PASSED] no-match-no-add
[23:12:28] [PASSED] match-or
[23:12:28] [PASSED] match-or-xfail
[23:12:28] [PASSED] no-match-no-add-multiple-rules
[23:12:28] [PASSED] two-regs-two-entries
[23:12:28] [PASSED] clr-one-set-other
[23:12:28] [PASSED] set-field
[23:12:28] [PASSED] conflict-duplicate
[23:12:28] [PASSED] conflict-not-disjoint
[23:12:28] [PASSED] conflict-reg-type
[23:12:28] =========== [PASSED] xe_rtp_process_to_sr_tests ============
[23:12:28] ================== xe_rtp_process_tests  ===================
[23:12:28] [PASSED] active1
[23:12:28] [PASSED] active2
[23:12:28] [PASSED] active-inactive
[23:12:28] [PASSED] inactive-active
[23:12:28] [PASSED] inactive-1st_or_active-inactive
[23:12:28] [PASSED] inactive-2nd_or_active-inactive
[23:12:28] [PASSED] inactive-last_or_active-inactive
[23:12:28] [PASSED] inactive-no_or_active-inactive
[23:12:28] ============== [PASSED] xe_rtp_process_tests ===============
[23:12:28] ===================== [PASSED] xe_rtp ======================
[23:12:28] ==================== xe_wa (1 subtest) =====================
[23:12:28] ======================== xe_wa_gt  =========================
[23:12:28] [PASSED] TIGERLAKE (B0)
[23:12:28] [PASSED] DG1 (A0)
[23:12:28] [PASSED] DG1 (B0)
[23:12:28] [PASSED] ALDERLAKE_S (A0)
[23:12:28] [PASSED] ALDERLAKE_S (B0)
[23:12:28] [PASSED] ALDERLAKE_S (C0)
[23:12:28] [PASSED] ALDERLAKE_S (D0)
[23:12:28] [PASSED] ALDERLAKE_P (A0)
[23:12:28] [PASSED] ALDERLAKE_P (B0)
[23:12:28] [PASSED] ALDERLAKE_P (C0)
[23:12:28] [PASSED] ALDERLAKE_S_RPLS (D0)
[23:12:28] [PASSED] ALDERLAKE_P_RPLU (E0)
[23:12:28] [PASSED] DG2_G10 (C0)
[23:12:28] [PASSED] DG2_G11 (B1)
[23:12:28] [PASSED] DG2_G12 (A1)
[23:12:28] [PASSED] METEORLAKE (g:A0, m:A0)
[23:12:28] [PASSED] METEORLAKE (g:A0, m:A0)
[23:12:28] [PASSED] METEORLAKE (g:A0, m:A0)
[23:12:28] [PASSED] LUNARLAKE (g:A0, m:A0)
[23:12:28] [PASSED] LUNARLAKE (g:B0, m:A0)
[23:12:28] [PASSED] BATTLEMAGE (g:A0, m:A1)
[23:12:28] ==================== [PASSED] xe_wa_gt =====================
[23:12:28] ====================== [PASSED] xe_wa ======================
[23:12:28] ============================================================
[23:12:28] Testing complete. Ran 122 tests: passed: 106, skipped: 16
[23:12:28] Elapsed time: 33.330s total, 4.483s configuring, 28.581s building, 0.225s running

+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/tests/.kunitconfig
[23:12:28] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[23:12:30] 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)
      |      ^~~~~~~~~~~~~~~~~

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

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

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



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

* ✗ CI.Build: failure for series starting with [1/3] drm/i915/display: Move shutdown sequences under display driver
  2025-01-17 22:09 [PATCH 1/3] drm/i915/display: Move shutdown sequences under display driver Rodrigo Vivi
                   ` (4 preceding siblings ...)
  2025-01-17 23:13 ` ✓ CI.KUnit: " Patchwork
@ 2025-01-17 23:20 ` Patchwork
  5 siblings, 0 replies; 12+ messages in thread
From: Patchwork @ 2025-01-17 23:20 UTC (permalink / raw)
  To: Rodrigo Vivi; +Cc: intel-xe

== Series Details ==

Series: series starting with [1/3] drm/i915/display: Move shutdown sequences under display driver
URL   : https://patchwork.freedesktop.org/series/143697/
State : failure

== Summary ==

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/dml2_top_interfaces.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml2/dml21/src/dml2_top/dml2_top_soc15.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
  CC [M]  drivers/gpu/drm/amd/amdgpu/amdgpu_isp.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/isp_v4_1_0.o
  CC [M]  drivers/gpu/drm/amd/amdgpu/isp_v4_1_1.o
  LD [M]  drivers/gpu/drm/amd/amdgpu/amdgpu.o
make[5]: *** [../scripts/Makefile.build:440: drivers/gpu/drm] Error 2
make[4]: *** [../scripts/Makefile.build:440: drivers/gpu] Error 2
make[3]: *** [../scripts/Makefile.build:440: drivers] Error 2
make[2]: *** [/kernel/Makefile:1989: .] Error 2
make[1]: *** [/kernel/Makefile:251: __sub-make] Error 2
make[1]: Leaving directory '/kernel/build64-default'
make: *** [Makefile:251: __sub-make] Error 2
+ cleanup
++ stat -c %u:%g /kernel
+ chown -R 1003:1003 /kernel



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

* Re: [PATCH 3/3] drm/xe: Use i915-display shutdown sequence directly
  2025-01-17 22:09 ` [PATCH 3/3] drm/xe: Use i915-display shutdown sequence directly Rodrigo Vivi
@ 2025-01-20 13:42   ` Maarten Lankhorst
  2025-01-21 19:37     ` Rodrigo Vivi
  2025-01-28 23:27     ` Rodrigo Vivi
  0 siblings, 2 replies; 12+ messages in thread
From: Maarten Lankhorst @ 2025-01-20 13:42 UTC (permalink / raw)
  To: Rodrigo Vivi, intel-gfx, intel-xe; +Cc: Jonathan Cavitt

Hey,

Den 2025-01-17 kl. 23:09, skrev Rodrigo Vivi:
> Start the xe-i915-display reconciliation by using the same
> shutdown sequences.
> 
> v2: include the stubs for !CONFIG_DRM_XE_DISPLAY (Kunit)
> 
> Reviewed-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> ---
>   drivers/gpu/drm/xe/display/xe_display.c | 48 +++++++------------------
>   drivers/gpu/drm/xe/display/xe_display.h | 10 +++---
>   drivers/gpu/drm/xe/xe_device.c          |  4 ++-
>   3 files changed, 22 insertions(+), 40 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/display/xe_display.c b/drivers/gpu/drm/xe/display/xe_display.c
> index 4f60d7bd7742..e1ce9eb3332d 100644
> --- a/drivers/gpu/drm/xe/display/xe_display.c
> +++ b/drivers/gpu/drm/xe/display/xe_display.c
> @@ -10,7 +10,6 @@
>   
>   #include <drm/drm_drv.h>
>   #include <drm/drm_managed.h>
> -#include <drm/drm_atomic_helper.h>
>   #include <drm/drm_probe_helper.h>
>   #include <uapi/drm/xe_drm.h>
>   
> @@ -369,32 +368,26 @@ void xe_display_pm_suspend(struct xe_device *xe)
>   
>   void xe_display_pm_shutdown(struct xe_device *xe)
>   {
> -	struct intel_display *display = &xe->display;
> -
>   	if (!xe->info.probe_display)
>   		return;
>   
> -	intel_power_domains_disable(display);
> -	intel_fbdev_set_suspend(&xe->drm, FBINFO_STATE_SUSPENDED, true);
> -	if (has_display(xe)) {
> -		drm_kms_helper_poll_disable(&xe->drm);
> -		intel_display_driver_disable_user_access(display);
> -
> -		drm_atomic_helper_shutdown(display->drm);
> -	}
> -
> -	intel_dp_mst_suspend(display);
> -	intel_hpd_cancel_work(xe);
> +	intel_display_driver_shutdown(&xe->display);
> +}
>   
> -	if (has_display(xe))
> -		intel_display_driver_suspend_access(display);
> +void xe_display_pm_shutdown_noirq(struct xe_device *xe)
> +{
> +	if (!xe->info.probe_display)
> +		return;
>   
> -	intel_encoder_suspend_all(display);
> -	intel_encoder_shutdown_all(display);
> +	intel_display_driver_shutdown_noirq(&xe->display);
> +}
>   
> -	intel_opregion_suspend(display, PCI_D3cold);
> +void xe_display_pm_shutdown_noaccel(struct xe_device *xe)
> +{
> +	if (!xe->info.probe_display)
> +		return;
>   
> -	intel_dmc_suspend(display);
> +	intel_display_driver_shutdown_nogem(&xe->display);
>   }
>   
>   void xe_display_pm_runtime_suspend(struct xe_device *xe)
> @@ -439,21 +432,6 @@ void xe_display_pm_runtime_suspend_late(struct xe_device *xe)
>   	intel_dmc_wl_flush_release_work(display);
>   }
>   
> -void xe_display_pm_shutdown_late(struct xe_device *xe)
> -{
> -	struct intel_display *display = &xe->display;
> -
> -	if (!xe->info.probe_display)
> -		return;
> -
> -	/*
> -	 * The only requirement is to reboot with display DC states disabled,
> -	 * for now leaving all display power wells in the INIT power domain
> -	 * enabled.
> -	 */
> -	intel_power_domains_driver_remove(display);
> -}
> -
>   void xe_display_pm_resume_early(struct xe_device *xe)
>   {
>   	struct intel_display *display = &xe->display;
> diff --git a/drivers/gpu/drm/xe/display/xe_display.h b/drivers/gpu/drm/xe/display/xe_display.h
> index 233f81a26c25..a15ec29b862b 100644
> --- a/drivers/gpu/drm/xe/display/xe_display.h
> +++ b/drivers/gpu/drm/xe/display/xe_display.h
> @@ -35,9 +35,10 @@ void xe_display_irq_reset(struct xe_device *xe);
>   void xe_display_irq_postinstall(struct xe_device *xe, struct xe_gt *gt);
>   
>   void xe_display_pm_suspend(struct xe_device *xe);
> -void xe_display_pm_shutdown(struct xe_device *xe);
>   void xe_display_pm_suspend_late(struct xe_device *xe);
> -void xe_display_pm_shutdown_late(struct xe_device *xe);
> +void xe_display_pm_shutdown(struct xe_device *xe);
> +void xe_display_pm_shutdown_noirq(struct xe_device *xe);
> +void xe_display_pm_shutdown_noaccel(struct xe_device *xe);
>   void xe_display_pm_resume_early(struct xe_device *xe);
>   void xe_display_pm_resume(struct xe_device *xe);
>   void xe_display_pm_runtime_suspend(struct xe_device *xe);
> @@ -69,9 +70,10 @@ static inline void xe_display_irq_reset(struct xe_device *xe) {}
>   static inline void xe_display_irq_postinstall(struct xe_device *xe, struct xe_gt *gt) {}
>   
>   static inline void xe_display_pm_suspend(struct xe_device *xe) {}
> -static inline void xe_display_pm_shutdown(struct xe_device *xe) {}
>   static inline void xe_display_pm_suspend_late(struct xe_device *xe) {}
> -static inline void xe_display_pm_shutdown_late(struct xe_device *xe) {}
> +static inline void xe_display_pm_shutdown(struct xe_device *xe) {}
> +static inline void xe_display_pm_shutdown_noirq(struct xe_device *xe) {}
> +static inline void xe_display_pm_shutdown_noaccel(struct xe_device *xe) {}
>   static inline void xe_display_pm_resume_early(struct xe_device *xe) {}
>   static inline void xe_display_pm_resume(struct xe_device *xe) {}
>   static inline void xe_display_pm_runtime_suspend(struct xe_device *xe) {}
> diff --git a/drivers/gpu/drm/xe/xe_device.c b/drivers/gpu/drm/xe/xe_device.c
> index 0966d9697caf..53cac055a2a9 100644
> --- a/drivers/gpu/drm/xe/xe_device.c
> +++ b/drivers/gpu/drm/xe/xe_device.c
> @@ -934,10 +934,12 @@ void xe_device_shutdown(struct xe_device *xe)
>   
>   		xe_irq_suspend(xe);
>   
> +		xe_display_pm_shutdown_noirq(xe);
> +
>   		for_each_gt(gt, xe, id)
>   			xe_gt_shutdown(gt);
>   
> -		xe_display_pm_shutdown_late(xe);
> +		xe_display_pm_shutdown_noaccel(xe);
 From the xe point of view, it shouldn't matter whether we call the 
noirq part before or after gt shutdown. I like the integration into 
xe_device to be as simple as possible, so could we keep the single 
xe_display_pm_shutdown_late() call?

Which reminds me to send out xe_display simplification once more to do 
the same for init..
>   	} else {
>   		/* BOOM! */
>   		__xe_driver_flr(xe);
Completely unrelated, do you happen to know if we need to call 
encoder_suspend/shutdown in the FLR path, whether it affects logic not 
on the chip itself?

Cheers,
~Maarten


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

* Re: [PATCH 3/3] drm/xe: Use i915-display shutdown sequence directly
  2025-01-20 13:42   ` Maarten Lankhorst
@ 2025-01-21 19:37     ` Rodrigo Vivi
  2025-01-28 23:27     ` Rodrigo Vivi
  1 sibling, 0 replies; 12+ messages in thread
From: Rodrigo Vivi @ 2025-01-21 19:37 UTC (permalink / raw)
  To: Maarten Lankhorst; +Cc: intel-gfx, intel-xe, Jonathan Cavitt

On Mon, Jan 20, 2025 at 02:42:14PM +0100, Maarten Lankhorst wrote:
> Hey,
> 
> Den 2025-01-17 kl. 23:09, skrev Rodrigo Vivi:
> > Start the xe-i915-display reconciliation by using the same
> > shutdown sequences.
> > 
> > v2: include the stubs for !CONFIG_DRM_XE_DISPLAY (Kunit)
> > 
> > Reviewed-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
> > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > ---
> >   drivers/gpu/drm/xe/display/xe_display.c | 48 +++++++------------------
> >   drivers/gpu/drm/xe/display/xe_display.h | 10 +++---
> >   drivers/gpu/drm/xe/xe_device.c          |  4 ++-
> >   3 files changed, 22 insertions(+), 40 deletions(-)
> > 
> > diff --git a/drivers/gpu/drm/xe/display/xe_display.c b/drivers/gpu/drm/xe/display/xe_display.c
> > index 4f60d7bd7742..e1ce9eb3332d 100644
> > --- a/drivers/gpu/drm/xe/display/xe_display.c
> > +++ b/drivers/gpu/drm/xe/display/xe_display.c
> > @@ -10,7 +10,6 @@
> >   #include <drm/drm_drv.h>
> >   #include <drm/drm_managed.h>
> > -#include <drm/drm_atomic_helper.h>
> >   #include <drm/drm_probe_helper.h>
> >   #include <uapi/drm/xe_drm.h>
> > @@ -369,32 +368,26 @@ void xe_display_pm_suspend(struct xe_device *xe)
> >   void xe_display_pm_shutdown(struct xe_device *xe)
> >   {
> > -	struct intel_display *display = &xe->display;
> > -
> >   	if (!xe->info.probe_display)
> >   		return;
> > -	intel_power_domains_disable(display);
> > -	intel_fbdev_set_suspend(&xe->drm, FBINFO_STATE_SUSPENDED, true);
> > -	if (has_display(xe)) {
> > -		drm_kms_helper_poll_disable(&xe->drm);
> > -		intel_display_driver_disable_user_access(display);
> > -
> > -		drm_atomic_helper_shutdown(display->drm);
> > -	}
> > -
> > -	intel_dp_mst_suspend(display);
> > -	intel_hpd_cancel_work(xe);
> > +	intel_display_driver_shutdown(&xe->display);
> > +}
> > -	if (has_display(xe))
> > -		intel_display_driver_suspend_access(display);
> > +void xe_display_pm_shutdown_noirq(struct xe_device *xe)
> > +{
> > +	if (!xe->info.probe_display)
> > +		return;
> > -	intel_encoder_suspend_all(display);
> > -	intel_encoder_shutdown_all(display);
> > +	intel_display_driver_shutdown_noirq(&xe->display);
> > +}
> > -	intel_opregion_suspend(display, PCI_D3cold);
> > +void xe_display_pm_shutdown_noaccel(struct xe_device *xe)
> > +{
> > +	if (!xe->info.probe_display)
> > +		return;
> > -	intel_dmc_suspend(display);
> > +	intel_display_driver_shutdown_nogem(&xe->display);
> >   }
> >   void xe_display_pm_runtime_suspend(struct xe_device *xe)
> > @@ -439,21 +432,6 @@ void xe_display_pm_runtime_suspend_late(struct xe_device *xe)
> >   	intel_dmc_wl_flush_release_work(display);
> >   }
> > -void xe_display_pm_shutdown_late(struct xe_device *xe)
> > -{
> > -	struct intel_display *display = &xe->display;
> > -
> > -	if (!xe->info.probe_display)
> > -		return;
> > -
> > -	/*
> > -	 * The only requirement is to reboot with display DC states disabled,
> > -	 * for now leaving all display power wells in the INIT power domain
> > -	 * enabled.
> > -	 */
> > -	intel_power_domains_driver_remove(display);
> > -}
> > -
> >   void xe_display_pm_resume_early(struct xe_device *xe)
> >   {
> >   	struct intel_display *display = &xe->display;
> > diff --git a/drivers/gpu/drm/xe/display/xe_display.h b/drivers/gpu/drm/xe/display/xe_display.h
> > index 233f81a26c25..a15ec29b862b 100644
> > --- a/drivers/gpu/drm/xe/display/xe_display.h
> > +++ b/drivers/gpu/drm/xe/display/xe_display.h
> > @@ -35,9 +35,10 @@ void xe_display_irq_reset(struct xe_device *xe);
> >   void xe_display_irq_postinstall(struct xe_device *xe, struct xe_gt *gt);
> >   void xe_display_pm_suspend(struct xe_device *xe);
> > -void xe_display_pm_shutdown(struct xe_device *xe);
> >   void xe_display_pm_suspend_late(struct xe_device *xe);
> > -void xe_display_pm_shutdown_late(struct xe_device *xe);
> > +void xe_display_pm_shutdown(struct xe_device *xe);
> > +void xe_display_pm_shutdown_noirq(struct xe_device *xe);
> > +void xe_display_pm_shutdown_noaccel(struct xe_device *xe);
> >   void xe_display_pm_resume_early(struct xe_device *xe);
> >   void xe_display_pm_resume(struct xe_device *xe);
> >   void xe_display_pm_runtime_suspend(struct xe_device *xe);
> > @@ -69,9 +70,10 @@ static inline void xe_display_irq_reset(struct xe_device *xe) {}
> >   static inline void xe_display_irq_postinstall(struct xe_device *xe, struct xe_gt *gt) {}
> >   static inline void xe_display_pm_suspend(struct xe_device *xe) {}
> > -static inline void xe_display_pm_shutdown(struct xe_device *xe) {}
> >   static inline void xe_display_pm_suspend_late(struct xe_device *xe) {}
> > -static inline void xe_display_pm_shutdown_late(struct xe_device *xe) {}
> > +static inline void xe_display_pm_shutdown(struct xe_device *xe) {}
> > +static inline void xe_display_pm_shutdown_noirq(struct xe_device *xe) {}
> > +static inline void xe_display_pm_shutdown_noaccel(struct xe_device *xe) {}
> >   static inline void xe_display_pm_resume_early(struct xe_device *xe) {}
> >   static inline void xe_display_pm_resume(struct xe_device *xe) {}
> >   static inline void xe_display_pm_runtime_suspend(struct xe_device *xe) {}
> > diff --git a/drivers/gpu/drm/xe/xe_device.c b/drivers/gpu/drm/xe/xe_device.c
> > index 0966d9697caf..53cac055a2a9 100644
> > --- a/drivers/gpu/drm/xe/xe_device.c
> > +++ b/drivers/gpu/drm/xe/xe_device.c
> > @@ -934,10 +934,12 @@ void xe_device_shutdown(struct xe_device *xe)
> >   		xe_irq_suspend(xe);
> > +		xe_display_pm_shutdown_noirq(xe);
> > +
> >   		for_each_gt(gt, xe, id)
> >   			xe_gt_shutdown(gt);
> > -		xe_display_pm_shutdown_late(xe);
> > +		xe_display_pm_shutdown_noaccel(xe);
> From the xe point of view, it shouldn't matter whether we call the noirq
> part before or after gt shutdown. I like the integration into xe_device to
> be as simple as possible, so could we keep the single
> xe_display_pm_shutdown_late() call?

I prefer that we make the xe_display entirely an wrapper to i915/display,
It should only check for the Xe's display module parameter, and then call
the equivalent function there directly.

Then, whatever differences we might have we move to xe_device itself.

And in this case _noaccel is just a generic name for _nogem which is a name
that would works for both i915 and xe...

> 
> Which reminds me to send out xe_display simplification once more to do the
> same for init..

On that too, I know I reviewed, but Jani also had concerns with that on the
sense that that deviates from removing display differences between drivers.

Although I still believe it is possible to take that patch in, but later
move the differences out of xe_display ?!

> >   	} else {
> >   		/* BOOM! */
> >   		__xe_driver_flr(xe);
> Completely unrelated, do you happen to know if we need to call
> encoder_suspend/shutdown in the FLR path, whether it affects logic not on
> the chip itself?
> 
> Cheers,
> ~Maarten
> 

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

* [PATCH 3/3] drm/xe: Use i915-display shutdown sequence directly
  2025-01-22 10:40 [PATCH 1/3] " Rodrigo Vivi
@ 2025-01-22 10:40 ` Rodrigo Vivi
  0 siblings, 0 replies; 12+ messages in thread
From: Rodrigo Vivi @ 2025-01-22 10:40 UTC (permalink / raw)
  To: intel-gfx, intel-xe; +Cc: Rodrigo Vivi, Jonathan Cavitt

Start the xe-i915-display reconciliation by using the same
shutdown sequences.

v2: include the stubs for !CONFIG_DRM_XE_DISPLAY (Kunit)

Reviewed-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
---
 drivers/gpu/drm/xe/display/xe_display.c | 48 +++++++------------------
 drivers/gpu/drm/xe/display/xe_display.h | 10 +++---
 drivers/gpu/drm/xe/xe_device.c          |  4 ++-
 3 files changed, 22 insertions(+), 40 deletions(-)

diff --git a/drivers/gpu/drm/xe/display/xe_display.c b/drivers/gpu/drm/xe/display/xe_display.c
index 4f60d7bd7742..e1ce9eb3332d 100644
--- a/drivers/gpu/drm/xe/display/xe_display.c
+++ b/drivers/gpu/drm/xe/display/xe_display.c
@@ -10,7 +10,6 @@
 
 #include <drm/drm_drv.h>
 #include <drm/drm_managed.h>
-#include <drm/drm_atomic_helper.h>
 #include <drm/drm_probe_helper.h>
 #include <uapi/drm/xe_drm.h>
 
@@ -369,32 +368,26 @@ void xe_display_pm_suspend(struct xe_device *xe)
 
 void xe_display_pm_shutdown(struct xe_device *xe)
 {
-	struct intel_display *display = &xe->display;
-
 	if (!xe->info.probe_display)
 		return;
 
-	intel_power_domains_disable(display);
-	intel_fbdev_set_suspend(&xe->drm, FBINFO_STATE_SUSPENDED, true);
-	if (has_display(xe)) {
-		drm_kms_helper_poll_disable(&xe->drm);
-		intel_display_driver_disable_user_access(display);
-
-		drm_atomic_helper_shutdown(display->drm);
-	}
-
-	intel_dp_mst_suspend(display);
-	intel_hpd_cancel_work(xe);
+	intel_display_driver_shutdown(&xe->display);
+}
 
-	if (has_display(xe))
-		intel_display_driver_suspend_access(display);
+void xe_display_pm_shutdown_noirq(struct xe_device *xe)
+{
+	if (!xe->info.probe_display)
+		return;
 
-	intel_encoder_suspend_all(display);
-	intel_encoder_shutdown_all(display);
+	intel_display_driver_shutdown_noirq(&xe->display);
+}
 
-	intel_opregion_suspend(display, PCI_D3cold);
+void xe_display_pm_shutdown_noaccel(struct xe_device *xe)
+{
+	if (!xe->info.probe_display)
+		return;
 
-	intel_dmc_suspend(display);
+	intel_display_driver_shutdown_nogem(&xe->display);
 }
 
 void xe_display_pm_runtime_suspend(struct xe_device *xe)
@@ -439,21 +432,6 @@ void xe_display_pm_runtime_suspend_late(struct xe_device *xe)
 	intel_dmc_wl_flush_release_work(display);
 }
 
-void xe_display_pm_shutdown_late(struct xe_device *xe)
-{
-	struct intel_display *display = &xe->display;
-
-	if (!xe->info.probe_display)
-		return;
-
-	/*
-	 * The only requirement is to reboot with display DC states disabled,
-	 * for now leaving all display power wells in the INIT power domain
-	 * enabled.
-	 */
-	intel_power_domains_driver_remove(display);
-}
-
 void xe_display_pm_resume_early(struct xe_device *xe)
 {
 	struct intel_display *display = &xe->display;
diff --git a/drivers/gpu/drm/xe/display/xe_display.h b/drivers/gpu/drm/xe/display/xe_display.h
index 233f81a26c25..a15ec29b862b 100644
--- a/drivers/gpu/drm/xe/display/xe_display.h
+++ b/drivers/gpu/drm/xe/display/xe_display.h
@@ -35,9 +35,10 @@ void xe_display_irq_reset(struct xe_device *xe);
 void xe_display_irq_postinstall(struct xe_device *xe, struct xe_gt *gt);
 
 void xe_display_pm_suspend(struct xe_device *xe);
-void xe_display_pm_shutdown(struct xe_device *xe);
 void xe_display_pm_suspend_late(struct xe_device *xe);
-void xe_display_pm_shutdown_late(struct xe_device *xe);
+void xe_display_pm_shutdown(struct xe_device *xe);
+void xe_display_pm_shutdown_noirq(struct xe_device *xe);
+void xe_display_pm_shutdown_noaccel(struct xe_device *xe);
 void xe_display_pm_resume_early(struct xe_device *xe);
 void xe_display_pm_resume(struct xe_device *xe);
 void xe_display_pm_runtime_suspend(struct xe_device *xe);
@@ -69,9 +70,10 @@ static inline void xe_display_irq_reset(struct xe_device *xe) {}
 static inline void xe_display_irq_postinstall(struct xe_device *xe, struct xe_gt *gt) {}
 
 static inline void xe_display_pm_suspend(struct xe_device *xe) {}
-static inline void xe_display_pm_shutdown(struct xe_device *xe) {}
 static inline void xe_display_pm_suspend_late(struct xe_device *xe) {}
-static inline void xe_display_pm_shutdown_late(struct xe_device *xe) {}
+static inline void xe_display_pm_shutdown(struct xe_device *xe) {}
+static inline void xe_display_pm_shutdown_noirq(struct xe_device *xe) {}
+static inline void xe_display_pm_shutdown_noaccel(struct xe_device *xe) {}
 static inline void xe_display_pm_resume_early(struct xe_device *xe) {}
 static inline void xe_display_pm_resume(struct xe_device *xe) {}
 static inline void xe_display_pm_runtime_suspend(struct xe_device *xe) {}
diff --git a/drivers/gpu/drm/xe/xe_device.c b/drivers/gpu/drm/xe/xe_device.c
index bd6191e1ed3e..b4b8c6582a20 100644
--- a/drivers/gpu/drm/xe/xe_device.c
+++ b/drivers/gpu/drm/xe/xe_device.c
@@ -940,10 +940,12 @@ void xe_device_shutdown(struct xe_device *xe)
 
 		xe_irq_suspend(xe);
 
+		xe_display_pm_shutdown_noirq(xe);
+
 		for_each_gt(gt, xe, id)
 			xe_gt_shutdown(gt);
 
-		xe_display_pm_shutdown_late(xe);
+		xe_display_pm_shutdown_noaccel(xe);
 	} else {
 		/* BOOM! */
 		__xe_driver_flr(xe);
-- 
2.48.1


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

* Re: [PATCH 3/3] drm/xe: Use i915-display shutdown sequence directly
  2025-01-20 13:42   ` Maarten Lankhorst
  2025-01-21 19:37     ` Rodrigo Vivi
@ 2025-01-28 23:27     ` Rodrigo Vivi
  1 sibling, 0 replies; 12+ messages in thread
From: Rodrigo Vivi @ 2025-01-28 23:27 UTC (permalink / raw)
  To: Maarten Lankhorst; +Cc: intel-gfx, intel-xe, Jonathan Cavitt

On Mon, Jan 20, 2025 at 02:42:14PM +0100, Maarten Lankhorst wrote:
> Hey,
> 
> Den 2025-01-17 kl. 23:09, skrev Rodrigo Vivi:
> > Start the xe-i915-display reconciliation by using the same
> > shutdown sequences.
> > 
> > v2: include the stubs for !CONFIG_DRM_XE_DISPLAY (Kunit)
> > 
> > Reviewed-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
> > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > ---
> >   drivers/gpu/drm/xe/display/xe_display.c | 48 +++++++------------------
> >   drivers/gpu/drm/xe/display/xe_display.h | 10 +++---
> >   drivers/gpu/drm/xe/xe_device.c          |  4 ++-
> >   3 files changed, 22 insertions(+), 40 deletions(-)
> > 
> > diff --git a/drivers/gpu/drm/xe/display/xe_display.c b/drivers/gpu/drm/xe/display/xe_display.c
> > index 4f60d7bd7742..e1ce9eb3332d 100644
> > --- a/drivers/gpu/drm/xe/display/xe_display.c
> > +++ b/drivers/gpu/drm/xe/display/xe_display.c
> > @@ -10,7 +10,6 @@
> >   #include <drm/drm_drv.h>
> >   #include <drm/drm_managed.h>
> > -#include <drm/drm_atomic_helper.h>
> >   #include <drm/drm_probe_helper.h>
> >   #include <uapi/drm/xe_drm.h>
> > @@ -369,32 +368,26 @@ void xe_display_pm_suspend(struct xe_device *xe)
> >   void xe_display_pm_shutdown(struct xe_device *xe)
> >   {
> > -	struct intel_display *display = &xe->display;
> > -
> >   	if (!xe->info.probe_display)
> >   		return;
> > -	intel_power_domains_disable(display);
> > -	intel_fbdev_set_suspend(&xe->drm, FBINFO_STATE_SUSPENDED, true);
> > -	if (has_display(xe)) {
> > -		drm_kms_helper_poll_disable(&xe->drm);
> > -		intel_display_driver_disable_user_access(display);
> > -
> > -		drm_atomic_helper_shutdown(display->drm);
> > -	}
> > -
> > -	intel_dp_mst_suspend(display);
> > -	intel_hpd_cancel_work(xe);
> > +	intel_display_driver_shutdown(&xe->display);
> > +}
> > -	if (has_display(xe))
> > -		intel_display_driver_suspend_access(display);
> > +void xe_display_pm_shutdown_noirq(struct xe_device *xe)
> > +{
> > +	if (!xe->info.probe_display)
> > +		return;
> > -	intel_encoder_suspend_all(display);
> > -	intel_encoder_shutdown_all(display);
> > +	intel_display_driver_shutdown_noirq(&xe->display);
> > +}
> > -	intel_opregion_suspend(display, PCI_D3cold);
> > +void xe_display_pm_shutdown_noaccel(struct xe_device *xe)
> > +{
> > +	if (!xe->info.probe_display)
> > +		return;
> > -	intel_dmc_suspend(display);
> > +	intel_display_driver_shutdown_nogem(&xe->display);
> >   }
> >   void xe_display_pm_runtime_suspend(struct xe_device *xe)
> > @@ -439,21 +432,6 @@ void xe_display_pm_runtime_suspend_late(struct xe_device *xe)
> >   	intel_dmc_wl_flush_release_work(display);
> >   }
> > -void xe_display_pm_shutdown_late(struct xe_device *xe)
> > -{
> > -	struct intel_display *display = &xe->display;
> > -
> > -	if (!xe->info.probe_display)
> > -		return;
> > -
> > -	/*
> > -	 * The only requirement is to reboot with display DC states disabled,
> > -	 * for now leaving all display power wells in the INIT power domain
> > -	 * enabled.
> > -	 */
> > -	intel_power_domains_driver_remove(display);
> > -}
> > -
> >   void xe_display_pm_resume_early(struct xe_device *xe)
> >   {
> >   	struct intel_display *display = &xe->display;
> > diff --git a/drivers/gpu/drm/xe/display/xe_display.h b/drivers/gpu/drm/xe/display/xe_display.h
> > index 233f81a26c25..a15ec29b862b 100644
> > --- a/drivers/gpu/drm/xe/display/xe_display.h
> > +++ b/drivers/gpu/drm/xe/display/xe_display.h
> > @@ -35,9 +35,10 @@ void xe_display_irq_reset(struct xe_device *xe);
> >   void xe_display_irq_postinstall(struct xe_device *xe, struct xe_gt *gt);
> >   void xe_display_pm_suspend(struct xe_device *xe);
> > -void xe_display_pm_shutdown(struct xe_device *xe);
> >   void xe_display_pm_suspend_late(struct xe_device *xe);
> > -void xe_display_pm_shutdown_late(struct xe_device *xe);
> > +void xe_display_pm_shutdown(struct xe_device *xe);
> > +void xe_display_pm_shutdown_noirq(struct xe_device *xe);
> > +void xe_display_pm_shutdown_noaccel(struct xe_device *xe);
> >   void xe_display_pm_resume_early(struct xe_device *xe);
> >   void xe_display_pm_resume(struct xe_device *xe);
> >   void xe_display_pm_runtime_suspend(struct xe_device *xe);
> > @@ -69,9 +70,10 @@ static inline void xe_display_irq_reset(struct xe_device *xe) {}
> >   static inline void xe_display_irq_postinstall(struct xe_device *xe, struct xe_gt *gt) {}
> >   static inline void xe_display_pm_suspend(struct xe_device *xe) {}
> > -static inline void xe_display_pm_shutdown(struct xe_device *xe) {}
> >   static inline void xe_display_pm_suspend_late(struct xe_device *xe) {}
> > -static inline void xe_display_pm_shutdown_late(struct xe_device *xe) {}
> > +static inline void xe_display_pm_shutdown(struct xe_device *xe) {}
> > +static inline void xe_display_pm_shutdown_noirq(struct xe_device *xe) {}
> > +static inline void xe_display_pm_shutdown_noaccel(struct xe_device *xe) {}
> >   static inline void xe_display_pm_resume_early(struct xe_device *xe) {}
> >   static inline void xe_display_pm_resume(struct xe_device *xe) {}
> >   static inline void xe_display_pm_runtime_suspend(struct xe_device *xe) {}
> > diff --git a/drivers/gpu/drm/xe/xe_device.c b/drivers/gpu/drm/xe/xe_device.c
> > index 0966d9697caf..53cac055a2a9 100644
> > --- a/drivers/gpu/drm/xe/xe_device.c
> > +++ b/drivers/gpu/drm/xe/xe_device.c
> > @@ -934,10 +934,12 @@ void xe_device_shutdown(struct xe_device *xe)
> >   		xe_irq_suspend(xe);
> > +		xe_display_pm_shutdown_noirq(xe);
> > +
> >   		for_each_gt(gt, xe, id)
> >   			xe_gt_shutdown(gt);
> > -		xe_display_pm_shutdown_late(xe);
> > +		xe_display_pm_shutdown_noaccel(xe);
> From the xe point of view, it shouldn't matter whether we call the noirq
> part before or after gt shutdown. I like the integration into xe_device to
> be as simple as possible, so could we keep the single
> xe_display_pm_shutdown_late() call?

As I had told you offline I was seeing if we could have a naming in i915
more generic like shutdown, shutdown_late, shutdown_early...
However I didn't like that that much... From the i915/display side, the
names still matters... that function can only be called at shutdown when
there's no irq anymore...

And from the xe side, I'm still working towards make the xe_display only
a thin layer for checking xe.enable_display parameter and nothing else.

So I still pretty much prefer the _noirq and _noaccel naming...

> 
> Which reminds me to send out xe_display simplification once more to do the
> same for init..
> >   	} else {
> >   		/* BOOM! */
> >   		__xe_driver_flr(xe);
> Completely unrelated, do you happen to know if we need to call
> encoder_suspend/shutdown in the FLR path, whether it affects logic not on
> the chip itself?
> 
> Cheers,
> ~Maarten
> 

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

end of thread, other threads:[~2025-01-28 23:27 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-01-17 22:09 [PATCH 1/3] drm/i915/display: Move shutdown sequences under display driver Rodrigo Vivi
2025-01-17 22:09 ` [PATCH 2/3] drm/xe: At shutdown disable commit helpers instead of flushing Rodrigo Vivi
2025-01-17 22:09 ` [PATCH 3/3] drm/xe: Use i915-display shutdown sequence directly Rodrigo Vivi
2025-01-20 13:42   ` Maarten Lankhorst
2025-01-21 19:37     ` Rodrigo Vivi
2025-01-28 23:27     ` Rodrigo Vivi
2025-01-17 23:11 ` ✓ CI.Patch_applied: success for series starting with [1/3] drm/i915/display: Move shutdown sequences under display driver Patchwork
2025-01-17 23:11 ` ✓ CI.checkpatch: " Patchwork
2025-01-17 23:13 ` ✓ CI.KUnit: " Patchwork
2025-01-17 23:20 ` ✗ CI.Build: failure " Patchwork
  -- strict thread matches above, loose matches on Subject: below --
2025-01-22 10:40 [PATCH 1/3] " Rodrigo Vivi
2025-01-22 10:40 ` [PATCH 3/3] drm/xe: Use i915-display shutdown sequence directly Rodrigo Vivi
2024-11-14 20:22 [PATCH 1/3] drm/i915/display: Move shutdown sequences under display driver Rodrigo Vivi
2024-11-14 20:22 ` [PATCH 3/3] drm/xe: Use i915-display shutdown sequence directly Rodrigo Vivi

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox