[PATCH v2 0/8] Fix MCR inconsistencies in RTP tables

[PATCH v2 0/8] Fix MCR inconsistencies in RTP tables

[Gustavo Sousa]

The Xe driver uses struct xe_reg as base type to represent a register
definition, and defines struct xe_reg_mcr as a sort of subtype to
represent multicast replicated (MCR) registers.  It uses this subtyping
to "force" the use of the correct MCR-based API for MMIO operations on
MCR registers.

In regular driver code, usage of those MCR registers usually need some
reasoning about whether the access will need to be steered or a
multicast; and the correct MCR API will be selected.

For RTP tables, on the other hand, that's not the case.  The developer
when defining an action the developer can either select a regular or a
MCR register and the register save/restore logic will make the selection
between regular vs MCR MMIO access depending on the register instance
defined in the RTP action.

That allows some mistakes to go unnoticed: a non-MCR register instance
could be used for an RTP action against a platform where such a register
is actually MCR; and vice-versa.

This series fixes those mistakes.

Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
Changes in v2:
- Incorporated review feedback. Please see individual patches for the
  respective changelogs.
- Dropped inclusion of hw_engines in xe_wa test.  We will implement a more
  focused test to check for MCR inconsistencies in the future, which will
  iterate over all applicable RTP tables.
- As a result of the above, dropped MMIO interception patches, as they became
  unnecessary.
- Link to v1: https://patch.msgid.link/20260116-rtp-mcr-check-v1-0-d420b9c1a327@intel.com

---
Gustavo Sousa (8):
      drm/xe: Define CACHE_MODE_1 as MCR register
      drm/xe: Define and use MCR version of COMMON_SLICE_CHICKEN1
      drm/xe: Define and use MCR version of COMMON_SLICE_CHICKEN4
      drm/xe/kunit: Add xe_kunit_helper_is_live_test()
      drm/xe: Extract xe_hw_engine_setup_reg_lrc()
      drm/xe/kunit: Use KUNIT_EXPECT_EQ() in xe_wa_gt()
      drm/xe/mcr: Extract reg_in_steering_type_ranges()
      drm/xe/reg_sr: Do sanity check for MCR vs non-MCR

 drivers/gpu/drm/xe/regs/xe_gt_regs.h        |  4 +-
 drivers/gpu/drm/xe/tests/xe_kunit_helpers.c | 26 +++++++++++
 drivers/gpu/drm/xe/tests/xe_kunit_helpers.h |  4 ++
 drivers/gpu/drm/xe/tests/xe_rtp_test.c      | 71 ++++++++++++++++++++++++++---
 drivers/gpu/drm/xe/tests/xe_wa_test.c       | 14 +++++-
 drivers/gpu/drm/xe/xe_gt.c                  |  8 ++--
 drivers/gpu/drm/xe/xe_gt_mcr.c              | 67 +++++++++++++++++++--------
 drivers/gpu/drm/xe/xe_gt_mcr.h              |  1 +
 drivers/gpu/drm/xe/xe_hw_engine.c           | 15 +++++-
 drivers/gpu/drm/xe/xe_hw_engine.h           |  2 +-
 drivers/gpu/drm/xe/xe_reg_sr.c              | 36 +++++++++++++++
 drivers/gpu/drm/xe/xe_tuning.c              |  2 +-
 drivers/gpu/drm/xe/xe_wa.c                  |  6 +--
 13 files changed, 216 insertions(+), 40 deletions(-)
---
base-commit: 18d2011d95002966c64980ab06f36ee9211d496c
change-id: 20260112-rtp-mcr-check-f976cb1adf94

Best regards,
--  
Gustavo Sousa <gustavo.sousa@intel.com>

[PATCH v2 1/8] drm/xe: Define CACHE_MODE_1 as MCR register

[Gustavo Sousa]

CACHE_MODE_1 is a MCR register for all platforms that currently use it
in the Xe driver.  Use XE_REG_MCR() when defining it.

Fixes: 8cd7e9759766 ("drm/xe: Add missing DG2 lrc workarounds")
Fixes: ff063430caa8 ("drm/xe/mtl: Add some initial MTL workarounds")
Bspec: 66534, 67788
Reviewed-by: Matt Roper <matthew.d.roper@intel.com>
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
 drivers/gpu/drm/xe/regs/xe_gt_regs.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/xe/regs/xe_gt_regs.h b/drivers/gpu/drm/xe/regs/xe_gt_regs.h
index 16c87ce3f614..408933aee08a 100644
--- a/drivers/gpu/drm/xe/regs/xe_gt_regs.h
+++ b/drivers/gpu/drm/xe/regs/xe_gt_regs.h
@@ -152,7 +152,7 @@
 
 #define XEHPG_INSTDONE_GEOM_SVGUNIT		XE_REG_MCR(0x666c)
 
-#define CACHE_MODE_1				XE_REG(0x7004, XE_REG_OPTION_MASKED)
+#define CACHE_MODE_1				XE_REG_MCR(0x7004, XE_REG_OPTION_MASKED)
 #define   MSAA_OPTIMIZATION_REDUC_DISABLE	REG_BIT(11)
 
 #define COMMON_SLICE_CHICKEN1			XE_REG(0x7010, XE_REG_OPTION_MASKED)

-- 
2.53.0

[PATCH v2 2/8] drm/xe: Define and use MCR version of COMMON_SLICE_CHICKEN1

[Gustavo Sousa]

The register COMMON_SLICE_CHICKEN1 is a MCR register on Xe2.
Let's make sure to define a MCR version of it and use it for the
relevant IP versions.

Use XEHP_ as prefix for the register name, since it is MCR as of Xe_HP.

Fixes: a5d221924e13 ("drm/xe/xe2_hpg: Add set of workarounds")
Fixes: 9f18b55b6d3f ("drm/xe/xe2: Add workaround 18033852989")
Bspec: 66534, 71185
Reviewed-by: Matt Roper <matthew.d.roper@intel.com>
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
 drivers/gpu/drm/xe/regs/xe_gt_regs.h | 1 +
 drivers/gpu/drm/xe/xe_wa.c           | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/xe/regs/xe_gt_regs.h b/drivers/gpu/drm/xe/regs/xe_gt_regs.h
index 408933aee08a..b21c66a1b777 100644
--- a/drivers/gpu/drm/xe/regs/xe_gt_regs.h
+++ b/drivers/gpu/drm/xe/regs/xe_gt_regs.h
@@ -156,6 +156,7 @@
 #define   MSAA_OPTIMIZATION_REDUC_DISABLE	REG_BIT(11)
 
 #define COMMON_SLICE_CHICKEN1			XE_REG(0x7010, XE_REG_OPTION_MASKED)
+#define XEHP_COMMON_SLICE_CHICKEN1		XE_REG_MCR(0x7010, XE_REG_OPTION_MASKED)
 #define   DISABLE_BOTTOM_CLIP_RECTANGLE_TEST	REG_BIT(14)
 
 #define HIZ_CHICKEN					XE_REG(0x7018, XE_REG_OPTION_MASKED)
diff --git a/drivers/gpu/drm/xe/xe_wa.c b/drivers/gpu/drm/xe/xe_wa.c
index 49f5e3e4c7cc..d6f94486673e 100644
--- a/drivers/gpu/drm/xe/xe_wa.c
+++ b/drivers/gpu/drm/xe/xe_wa.c
@@ -664,7 +664,7 @@ static const struct xe_rtp_entry_sr lrc_was[] = {
 	},
 	{ XE_RTP_NAME("18033852989"),
 	  XE_RTP_RULES(GRAPHICS_VERSION_RANGE(2001, 2004), ENGINE_CLASS(RENDER)),
-	  XE_RTP_ACTIONS(SET(COMMON_SLICE_CHICKEN1, DISABLE_BOTTOM_CLIP_RECTANGLE_TEST))
+	  XE_RTP_ACTIONS(SET(XEHP_COMMON_SLICE_CHICKEN1, DISABLE_BOTTOM_CLIP_RECTANGLE_TEST))
 	},
 	{ XE_RTP_NAME("15016589081"),
 	  XE_RTP_RULES(GRAPHICS_VERSION_RANGE(2001, 2004), ENGINE_CLASS(RENDER)),

-- 
2.53.0

[PATCH v2 3/8] drm/xe: Define and use MCR version of COMMON_SLICE_CHICKEN4

[Gustavo Sousa]

The register COMMON_SLICE_CHICKEN4 is a MCR register on both Xe2 and
Xe3. Let's make sure to define a MCR version of it and use it for the
relevant IP versions.

Use XEHP_ as prefix for the register name, since it is MCR as of Xe_HP.

v2:
  - Also change for one entry in lrc_tunnings, which was caught by
    manual testing and add corresponging Fixes tag in commit message.
    (Gustavo)

Fixes: 8d6f16f1f082 ("drm/xe: Extend Wa_22021007897 to Xe3 platforms")
Fixes: e5c13e2c505b ("drm/xe/xe2hpg: Add Wa_22021007897")
Fixes: 8ccf5f6b2295 ("drm/xe/tuning: Apply windower hardware filtering setting on Xe3 and Xe3p")
Bspec: 66534, 71185, 74417
Reviewed-by: Matt Roper <matthew.d.roper@intel.com> # v1
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
 drivers/gpu/drm/xe/regs/xe_gt_regs.h | 1 +
 drivers/gpu/drm/xe/xe_tuning.c       | 2 +-
 drivers/gpu/drm/xe/xe_wa.c           | 4 ++--
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/xe/regs/xe_gt_regs.h b/drivers/gpu/drm/xe/regs/xe_gt_regs.h
index b21c66a1b777..08251c7a1a4b 100644
--- a/drivers/gpu/drm/xe/regs/xe_gt_regs.h
+++ b/drivers/gpu/drm/xe/regs/xe_gt_regs.h
@@ -179,6 +179,7 @@
 #define XEHPG_SC_INSTDONE_EXTRA2		XE_REG_MCR(0x7108)
 
 #define COMMON_SLICE_CHICKEN4			XE_REG(0x7300, XE_REG_OPTION_MASKED)
+#define XEHP_COMMON_SLICE_CHICKEN4		XE_REG_MCR(0x7300, XE_REG_OPTION_MASKED)
 #define   SBE_PUSH_CONSTANT_BEHIND_FIX_ENABLE	REG_BIT(12)
 #define   DISABLE_TDC_LOAD_BALANCING_CALC	REG_BIT(6)
 #define   HW_FILTERING				REG_BIT(5)
diff --git a/drivers/gpu/drm/xe/xe_tuning.c b/drivers/gpu/drm/xe/xe_tuning.c
index ce39b77a084a..9a1b3862e192 100644
--- a/drivers/gpu/drm/xe/xe_tuning.c
+++ b/drivers/gpu/drm/xe/xe_tuning.c
@@ -134,7 +134,7 @@ static const struct xe_rtp_entry_sr engine_tunings[] = {
 static const struct xe_rtp_entry_sr lrc_tunings[] = {
 	{ XE_RTP_NAME("Tuning: Windower HW Filtering"),
 	  XE_RTP_RULES(GRAPHICS_VERSION_RANGE(3000, 3599), ENGINE_CLASS(RENDER)),
-	  XE_RTP_ACTIONS(SET(COMMON_SLICE_CHICKEN4, HW_FILTERING))
+	  XE_RTP_ACTIONS(SET(XEHP_COMMON_SLICE_CHICKEN4, HW_FILTERING))
 	},
 
 	/* DG2 */
diff --git a/drivers/gpu/drm/xe/xe_wa.c b/drivers/gpu/drm/xe/xe_wa.c
index d6f94486673e..cb811f8a7781 100644
--- a/drivers/gpu/drm/xe/xe_wa.c
+++ b/drivers/gpu/drm/xe/xe_wa.c
@@ -767,7 +767,7 @@ static const struct xe_rtp_entry_sr lrc_was[] = {
 	},
 	{ XE_RTP_NAME("22021007897"),
 	  XE_RTP_RULES(GRAPHICS_VERSION_RANGE(2001, 2002), ENGINE_CLASS(RENDER)),
-	  XE_RTP_ACTIONS(SET(COMMON_SLICE_CHICKEN4, SBE_PUSH_CONSTANT_BEHIND_FIX_ENABLE))
+	  XE_RTP_ACTIONS(SET(XEHP_COMMON_SLICE_CHICKEN4, SBE_PUSH_CONSTANT_BEHIND_FIX_ENABLE))
 	},
 
 	/* Xe3_LPG */
@@ -783,7 +783,7 @@ static const struct xe_rtp_entry_sr lrc_was[] = {
 	},
 	{ XE_RTP_NAME("22021007897"),
 	  XE_RTP_RULES(GRAPHICS_VERSION_RANGE(3000, 3005), ENGINE_CLASS(RENDER)),
-	  XE_RTP_ACTIONS(SET(COMMON_SLICE_CHICKEN4, SBE_PUSH_CONSTANT_BEHIND_FIX_ENABLE))
+	  XE_RTP_ACTIONS(SET(XEHP_COMMON_SLICE_CHICKEN4, SBE_PUSH_CONSTANT_BEHIND_FIX_ENABLE))
 	},
 	{ XE_RTP_NAME("14024681466"),
 	  XE_RTP_RULES(GRAPHICS_VERSION_RANGE(3000, 3005), ENGINE_CLASS(RENDER)),

-- 
2.53.0

[PATCH v2 4/8] drm/xe/kunit: Add xe_kunit_helper_is_live_test()

[Gustavo Sousa]

In upcoming changes we will need to differentiate between regular and
live KUnit tests.  Add the function xe_kunit_helper_is_live_test() for
that purpose.

Note that this is implemented in a rather hackish way, by leveraging
KUnit's static stubbing functionality.  A better approach would be to
store a boolean somewhere in test->priv that would tell if a test is a
live one, however that's not quite feasible today given the inconsistent
usage of test->priv (there is not a single uniform type of data assigned
to it across existing test).

v2:
  - s/xe_kunit_helper_is_live_test/xe_kunit_helper_is_live_test/
    (Michal)

Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
 drivers/gpu/drm/xe/tests/xe_kunit_helpers.c | 26 ++++++++++++++++++++++++++
 drivers/gpu/drm/xe/tests/xe_kunit_helpers.h |  4 ++++
 2 files changed, 30 insertions(+)

diff --git a/drivers/gpu/drm/xe/tests/xe_kunit_helpers.c b/drivers/gpu/drm/xe/tests/xe_kunit_helpers.c
index bc5156966ce9..d2f654f53cc7 100644
--- a/drivers/gpu/drm/xe/tests/xe_kunit_helpers.c
+++ b/drivers/gpu/drm/xe/tests/xe_kunit_helpers.c
@@ -16,6 +16,18 @@
 #include "xe_device_types.h"
 #include "xe_pm.h"
 
+/**
+ * xe_kunit_helper_is_live_test - Return true if @test is a live test.
+ * @test: the &kunit test
+ *
+ * Return: True for a live test and false otherwise.
+ */
+bool xe_kunit_helper_is_live_test(struct kunit *test)
+{
+	KUNIT_STATIC_STUB_REDIRECT(xe_kunit_helper_is_live_test, test);
+	return false;
+}
+
 /**
  * xe_kunit_helper_alloc_xe_device - Allocate a &xe_device for a KUnit test.
  * @test: the &kunit where this &xe_device will be used
@@ -93,6 +105,11 @@ EXPORT_SYMBOL_IF_KUNIT(xe_kunit_helper_xe_device_test_init);
 
 KUNIT_DEFINE_ACTION_WRAPPER(put_xe_pm_runtime, xe_pm_runtime_put, struct xe_device *);
 
+static bool xe_kunit_is_live_test_indeed(struct kunit *test)
+{
+	return true;
+}
+
 /**
  * xe_kunit_helper_xe_device_live_test_init - Prepare a &xe_device for
  *                                            use in a live KUnit test.
@@ -116,6 +133,15 @@ int xe_kunit_helper_xe_device_live_test_init(struct kunit *test)
 {
 	struct xe_device *xe = xe_device_const_cast(test->param_value);
 
+	/*
+	 * FIXME: This is a hack and a better solution is to have the "priv"
+	 * member of tests have a boolean to tell if a test is a live one.
+	 * Unfortunately that can't be done today because "priv" does not point
+	 * to a single unified type across existing tests.
+	 */
+	kunit_activate_static_stub(test, xe_kunit_helper_is_live_test,
+				   xe_kunit_is_live_test_indeed);
+
 	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, xe);
 	kunit_info(test, "running on %s device\n", xe->info.platform_name);
 
diff --git a/drivers/gpu/drm/xe/tests/xe_kunit_helpers.h b/drivers/gpu/drm/xe/tests/xe_kunit_helpers.h
index 83665f7b1254..71cfeb2d4efb 100644
--- a/drivers/gpu/drm/xe/tests/xe_kunit_helpers.h
+++ b/drivers/gpu/drm/xe/tests/xe_kunit_helpers.h
@@ -6,10 +6,14 @@
 #ifndef _XE_KUNIT_HELPERS_H_
 #define _XE_KUNIT_HELPERS_H_
 
+#include <linux/types.h>
+
 struct device;
 struct kunit;
 struct xe_device;
 
+bool xe_kunit_helper_is_live_test(struct kunit *test);
+
 struct xe_device *xe_kunit_helper_alloc_xe_device(struct kunit *test,
 						  struct device *dev);
 int xe_kunit_helper_xe_device_test_init(struct kunit *test);

-- 
2.53.0

[PATCH v2 5/8] drm/xe: Extract xe_hw_engine_setup_reg_lrc()

[Gustavo Sousa]

The steps for processing RTP rules that build up an engine's reg_lrc
arguably belongs to xe_hw_engine.c and should be encapsulated into a
function in that unit.

Move that logic to a new function called xe_hw_engine_setup_reg_lrc().

Reviewed-by: Matt Roper <matthew.d.roper@intel.com>
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
With the dropping of patch "drm/xe/kunit: Include hw_engines in xe_wa
test" from v1, this patch is not really required anymore, but it is a
good refactor IMO, so I decided to keep it in the series.
---
 drivers/gpu/drm/xe/xe_gt.c        |  5 +----
 drivers/gpu/drm/xe/xe_hw_engine.c | 15 +++++++++++++--
 drivers/gpu/drm/xe/xe_hw_engine.h |  2 +-
 3 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_gt.c b/drivers/gpu/drm/xe/xe_gt.c
index cdc678d1ae1f..c4b25daad542 100644
--- a/drivers/gpu/drm/xe/xe_gt.c
+++ b/drivers/gpu/drm/xe/xe_gt.c
@@ -393,10 +393,7 @@ int xe_gt_record_default_lrcs(struct xe_gt *gt)
 		if (gt->default_lrc[hwe->class])
 			continue;
 
-		xe_reg_sr_init(&hwe->reg_lrc, hwe->name, xe);
-		xe_wa_process_lrc(hwe);
-		xe_hw_engine_setup_default_lrc_state(hwe);
-		xe_tuning_process_lrc(hwe);
+		xe_hw_engine_setup_reg_lrc(hwe);
 
 		default_lrc = drmm_kzalloc(&xe->drm,
 					   xe_gt_lrc_size(gt, hwe->class),
diff --git a/drivers/gpu/drm/xe/xe_hw_engine.c b/drivers/gpu/drm/xe/xe_hw_engine.c
index 0f0e08bcc182..05f0932dbb94 100644
--- a/drivers/gpu/drm/xe/xe_hw_engine.c
+++ b/drivers/gpu/drm/xe/xe_hw_engine.c
@@ -337,8 +337,8 @@ static bool xe_rtp_cfeg_wmtp_disabled(const struct xe_device *xe,
 	return xe_mmio_read32(&hwe->gt->mmio, XEHP_FUSE4) & CFEG_WMTP_DISABLE;
 }
 
-void
-xe_hw_engine_setup_default_lrc_state(struct xe_hw_engine *hwe)
+static void
+hw_engine_setup_default_lrc_state(struct xe_hw_engine *hwe)
 {
 	struct xe_gt *gt = hwe->gt;
 	const u8 mocs_write_idx = gt->mocs.uc_index;
@@ -375,6 +375,17 @@ xe_hw_engine_setup_default_lrc_state(struct xe_hw_engine *hwe)
 			     &hwe->reg_lrc, true);
 }
 
+void xe_hw_engine_setup_reg_lrc(struct xe_hw_engine *hwe)
+{
+	struct xe_gt *gt = hwe->gt;
+	struct xe_device *xe = gt_to_xe(gt);
+
+	xe_reg_sr_init(&hwe->reg_lrc, hwe->name, xe);
+	xe_wa_process_lrc(hwe);
+	hw_engine_setup_default_lrc_state(hwe);
+	xe_tuning_process_lrc(hwe);
+}
+
 static void
 hw_engine_setup_default_state(struct xe_hw_engine *hwe)
 {
diff --git a/drivers/gpu/drm/xe/xe_hw_engine.h b/drivers/gpu/drm/xe/xe_hw_engine.h
index ee9218773b51..c3ee37f8cfc0 100644
--- a/drivers/gpu/drm/xe/xe_hw_engine.h
+++ b/drivers/gpu/drm/xe/xe_hw_engine.h
@@ -59,7 +59,7 @@ struct xe_hw_engine_snapshot *
 xe_hw_engine_snapshot_capture(struct xe_hw_engine *hwe, struct xe_exec_queue *q);
 void xe_hw_engine_snapshot_free(struct xe_hw_engine_snapshot *snapshot);
 void xe_hw_engine_print(struct xe_hw_engine *hwe, struct drm_printer *p);
-void xe_hw_engine_setup_default_lrc_state(struct xe_hw_engine *hwe);
+void xe_hw_engine_setup_reg_lrc(struct xe_hw_engine *hwe);
 
 bool xe_hw_engine_is_reserved(struct xe_hw_engine *hwe);
 

-- 
2.53.0

[PATCH v2 6/8] drm/xe/kunit: Use KUNIT_EXPECT_EQ() in xe_wa_gt()

[Gustavo Sousa]

Use KUNIT_EXPECT_EQ() in xe_wa_gt() as reg_sr errors in one GT do not
impact the next GT in the test.

Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Reviewed-by: Matt Roper <matthew.d.roper@intel.com>
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
 drivers/gpu/drm/xe/tests/xe_wa_test.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/xe/tests/xe_wa_test.c b/drivers/gpu/drm/xe/tests/xe_wa_test.c
index 49d191043dfa..2bf6fab015cd 100644
--- a/drivers/gpu/drm/xe/tests/xe_wa_test.c
+++ b/drivers/gpu/drm/xe/tests/xe_wa_test.c
@@ -55,7 +55,7 @@ static void xe_wa_gt(struct kunit *test)
 		xe_wa_process_gt(gt);
 		xe_tuning_process_gt(gt);
 
-		KUNIT_ASSERT_EQ(test, gt->reg_sr.errors, 0);
+		KUNIT_EXPECT_EQ(test, gt->reg_sr.errors, 0);
 	}
 }
 

-- 
2.53.0

[PATCH v2 7/8] drm/xe/mcr: Extract reg_in_steering_type_ranges()

[Gustavo Sousa]

The logic to check if a register falls within one of the ranges for a
steering type is already duplicated in
xe_gt_mcr_get_nonterminated_steering().  We will also want to use that
same logic in another upcoming function.  Let's factor out that logic
and put it into a function named reg_in_steering_type_ranges().

Reviewed-by: Matt Roper <matthew.d.roper@intel.com>
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
 drivers/gpu/drm/xe/xe_gt_mcr.c | 43 +++++++++++++++++++++++-------------------
 1 file changed, 24 insertions(+), 19 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_gt_mcr.c b/drivers/gpu/drm/xe/xe_gt_mcr.c
index df281688c617..2b2a4d9c3749 100644
--- a/drivers/gpu/drm/xe/xe_gt_mcr.c
+++ b/drivers/gpu/drm/xe/xe_gt_mcr.c
@@ -600,6 +600,20 @@ void xe_gt_mcr_set_implicit_defaults(struct xe_gt *gt)
 	}
 }
 
+static bool reg_in_steering_type_ranges(struct xe_gt *gt,
+					struct xe_reg reg,
+					int type)
+{
+	if (!gt->steering[type].ranges)
+		return false;
+
+	for (int i = 0; gt->steering[type].ranges[i].end > 0; i++)
+		if (xe_mmio_in_range(&gt->mmio, &gt->steering[type].ranges[i], reg))
+			return true;
+
+	return false;
+}
+
 /*
  * xe_gt_mcr_get_nonterminated_steering - find group/instance values that
  *    will steer a register to a non-terminated instance
@@ -621,30 +635,21 @@ bool xe_gt_mcr_get_nonterminated_steering(struct xe_gt *gt,
 					  u8 *group, u8 *instance)
 {
 	const struct xe_reg reg = to_xe_reg(reg_mcr);
-	const struct xe_mmio_range *implicit_ranges;
 
 	for (int type = 0; type < IMPLICIT_STEERING; type++) {
-		if (!gt->steering[type].ranges)
-			continue;
-
-		for (int i = 0; gt->steering[type].ranges[i].end > 0; i++) {
-			if (xe_mmio_in_range(&gt->mmio, &gt->steering[type].ranges[i], reg)) {
-				drm_WARN(&gt_to_xe(gt)->drm, !gt->steering[type].initialized,
-					 "Uninitialized usage of MCR register %s/%#x\n",
-					 xe_steering_types[type].name, reg.addr);
-
-				*group = gt->steering[type].group_target;
-				*instance = gt->steering[type].instance_target;
-				return true;
-			}
+		if (reg_in_steering_type_ranges(gt, reg, type)) {
+			drm_WARN(&gt_to_xe(gt)->drm, !gt->steering[type].initialized,
+				 "Uninitialized usage of MCR register %s/%#x\n",
+				 xe_steering_types[type].name, reg.addr);
+
+			*group = gt->steering[type].group_target;
+			*instance = gt->steering[type].instance_target;
+			return true;
 		}
 	}
 
-	implicit_ranges = gt->steering[IMPLICIT_STEERING].ranges;
-	if (implicit_ranges)
-		for (int i = 0; implicit_ranges[i].end > 0; i++)
-			if (xe_mmio_in_range(&gt->mmio, &implicit_ranges[i], reg))
-				return false;
+	if (reg_in_steering_type_ranges(gt, reg, IMPLICIT_STEERING))
+		return false;
 
 	/*
 	 * Not found in a steering table and not a register with implicit

-- 
2.53.0

[PATCH v2 8/8] drm/xe/reg_sr: Do sanity check for MCR vs non-MCR

[Gustavo Sousa]

The type struct xe_reg_mcr exists to ensure that the correct API is used
when handling MCR registers.  However, for the register save/restore
functionality, the RTP processing always cast the register to a struct
xe_reg and then apply_one_mmio() selects the MMIO API based on the "mcr"
field of the register instance.

This allows the developer to commit mistakes like passing a MCR register
for an RTP action for a GT where the respective register is not MCR; and
vice-versa.

To capture such scenarios, do a sanity check in xe_reg_sr_add() that,
upon an inconsistency:

- "fixes" the register type by favoring what we have in our MCR range
  tables instead of what the developer selected for the save/restore
  entry;
- raises a notice-level message to inform about the inconsistency.

Note: As a collateral of this change, we need to include MCR
initialization in xe_wa_test.c, otherwise a bunch of test cases end up
failing because xe_gt_mcr_check_reg() will always return false, meaning
that will incorrectly say that a MCR register is not MCR.

v2:
  - Downgrade messages to notice level so as not to block CI execution
    when inconsistencies are found. (Matt)
  - Add missing EXPORT_SYMBOL_IF_KUNIT() calls. (Gustavo)

Cc: Matt Roper <matthew.d.roper@intel.com>
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
 drivers/gpu/drm/xe/tests/xe_rtp_test.c | 71 ++++++++++++++++++++++++++++++----
 drivers/gpu/drm/xe/tests/xe_wa_test.c  | 12 +++++-
 drivers/gpu/drm/xe/xe_gt.c             |  3 ++
 drivers/gpu/drm/xe/xe_gt_mcr.c         | 24 ++++++++++++
 drivers/gpu/drm/xe/xe_gt_mcr.h         |  1 +
 drivers/gpu/drm/xe/xe_reg_sr.c         | 36 +++++++++++++++++
 6 files changed, 139 insertions(+), 8 deletions(-)

diff --git a/drivers/gpu/drm/xe/tests/xe_rtp_test.c b/drivers/gpu/drm/xe/tests/xe_rtp_test.c
index e5a0f985a700..5d78f2283df9 100644
--- a/drivers/gpu/drm/xe/tests/xe_rtp_test.c
+++ b/drivers/gpu/drm/xe/tests/xe_rtp_test.c
@@ -9,24 +9,30 @@
 #include <drm/drm_drv.h>
 #include <drm/drm_kunit_helpers.h>
 
+#include <kunit/static_stub.h>
 #include <kunit/test.h>
 
 #include "regs/xe_gt_regs.h"
 #include "regs/xe_reg_defs.h"
 #include "xe_device.h"
 #include "xe_device_types.h"
+#include "xe_gt_mcr.h"
 #include "xe_kunit_helpers.h"
 #include "xe_pci_test.h"
 #include "xe_reg_sr.h"
 #include "xe_rtp.h"
 
-#define REGULAR_REG1	XE_REG(1)
-#define REGULAR_REG2	XE_REG(2)
-#define REGULAR_REG3	XE_REG(3)
-#define MCR_REG1	XE_REG_MCR(1)
-#define MCR_REG2	XE_REG_MCR(2)
-#define MCR_REG3	XE_REG_MCR(3)
-#define MASKED_REG1	XE_REG(1, XE_REG_OPTION_MASKED)
+#define REGULAR_REG1		XE_REG(1)
+#define REGULAR_REG2		XE_REG(2)
+#define REGULAR_REG3		XE_REG(3)
+#define REGULAR_REG4		XE_REG(4)
+#define BAD_REGULAR_REG5	XE_REG(5)
+#define MCR_REG1		XE_REG_MCR(1)
+#define MCR_REG2		XE_REG_MCR(2)
+#define MCR_REG3		XE_REG_MCR(3)
+#define BAD_MCR_REG4		XE_REG_MCR(4)
+#define MCR_REG5		XE_REG_MCR(5)
+#define MASKED_REG1		XE_REG(1, XE_REG_OPTION_MASKED)
 
 #undef XE_REG_MCR
 #define XE_REG_MCR(...)     XE_REG(__VA_ARGS__, .mcr = 1)
@@ -48,6 +54,23 @@ struct rtp_test_case {
 	const struct xe_rtp_entry *entries;
 };
 
+static bool fake_xe_gt_mcr_check_reg(struct xe_gt *gt, struct xe_reg reg)
+{
+	/*
+	 * All supported platforms in this imaginary setup will always have REG4
+	 * as a non-MCR register and REG5 as MCR, meaning that BAD_MCR_REG4 and
+	 * BAD_REGULAR_REG5 represent programming errors to be captured by our
+	 * tests.
+	 */
+	if (reg.raw == BAD_REGULAR_REG5.raw)
+		return true;
+
+	if (reg.raw == BAD_MCR_REG4.raw)
+		return false;
+
+	return reg.mcr;
+}
+
 static bool match_yes(const struct xe_device *xe, const struct xe_gt *gt,
 		      const struct xe_hw_engine *hwe)
 {
@@ -304,6 +327,38 @@ static const struct rtp_to_sr_test_case rtp_to_sr_cases[] = {
 			{}
 		},
 	},
+	{
+		.name = "bad-mcr-reg-forced-to-regular",
+		.expected_reg = REGULAR_REG4,
+		.expected_set_bits = REG_BIT(0),
+		.expected_clr_bits = REG_BIT(0),
+		.expected_active = BIT(0),
+		.expected_count_sr_entries = 1,
+		.expected_sr_errors = 1,
+		.entries = (const struct xe_rtp_entry_sr[]) {
+			{ XE_RTP_NAME("bad-mcr-regular-reg"),
+			  XE_RTP_RULES(FUNC(match_yes)),
+			  XE_RTP_ACTIONS(SET(BAD_MCR_REG4, REG_BIT(0)))
+			},
+			{}
+		},
+	},
+	{
+		.name = "bad-regular-reg-forced-to-mcr",
+		.expected_reg = MCR_REG5,
+		.expected_set_bits = REG_BIT(0),
+		.expected_clr_bits = REG_BIT(0),
+		.expected_active = BIT(0),
+		.expected_count_sr_entries = 1,
+		.expected_sr_errors = 1,
+		.entries = (const struct xe_rtp_entry_sr[]) {
+			{ XE_RTP_NAME("bad-regular-reg"),
+			  XE_RTP_RULES(FUNC(match_yes)),
+			  XE_RTP_ACTIONS(SET(BAD_REGULAR_REG5, REG_BIT(0)))
+			},
+			{}
+		},
+	},
 };
 
 static void xe_rtp_process_to_sr_tests(struct kunit *test)
@@ -523,6 +578,8 @@ static int xe_rtp_test_init(struct kunit *test)
 	xe->drm.dev = dev;
 	test->priv = xe;
 
+	kunit_activate_static_stub(test, xe_gt_mcr_check_reg, fake_xe_gt_mcr_check_reg);
+
 	return 0;
 }
 
diff --git a/drivers/gpu/drm/xe/tests/xe_wa_test.c b/drivers/gpu/drm/xe/tests/xe_wa_test.c
index 2bf6fab015cd..ff0e2502b39f 100644
--- a/drivers/gpu/drm/xe/tests/xe_wa_test.c
+++ b/drivers/gpu/drm/xe/tests/xe_wa_test.c
@@ -9,6 +9,8 @@
 #include <kunit/test.h>
 
 #include "xe_device.h"
+#include "xe_gt.h"
+#include "xe_gt_mcr.h"
 #include "xe_kunit_helpers.h"
 #include "xe_pci_test.h"
 #include "xe_reg_sr.h"
@@ -19,8 +21,10 @@ static int xe_wa_test_init(struct kunit *test)
 {
 	const struct xe_pci_fake_data *param = test->param_value;
 	struct xe_pci_fake_data data = *param;
-	struct xe_device *xe;
 	struct device *dev;
+	struct xe_device *xe;
+	struct xe_gt *gt;
+	int id;
 	int ret;
 
 	dev = drm_kunit_helper_alloc_device(test);
@@ -33,6 +37,12 @@ static int xe_wa_test_init(struct kunit *test)
 	ret = xe_pci_fake_device_init(xe);
 	KUNIT_ASSERT_EQ(test, ret, 0);
 
+	/* Needed for sanitize_mcr(). */
+	for_each_gt(gt, xe, id) {
+		xe_gt_mcr_init_early(gt);
+		xe_gt_mmio_init(gt);
+	}
+
 	if (!param->graphics_verx100)
 		xe->info.step = param->step;
 
diff --git a/drivers/gpu/drm/xe/xe_gt.c b/drivers/gpu/drm/xe/xe_gt.c
index c4b25daad542..783eb6d631b5 100644
--- a/drivers/gpu/drm/xe/xe_gt.c
+++ b/drivers/gpu/drm/xe/xe_gt.c
@@ -7,6 +7,8 @@
 
 #include <linux/minmax.h>
 
+#include <kunit/visibility.h>
+
 #include <drm/drm_managed.h>
 #include <uapi/drm/xe_drm.h>
 
@@ -785,6 +787,7 @@ void xe_gt_mmio_init(struct xe_gt *gt)
 	if (IS_SRIOV_VF(xe))
 		gt->mmio.sriov_vf_gt = gt;
 }
+EXPORT_SYMBOL_IF_KUNIT(xe_gt_mmio_init);
 
 void xe_gt_record_user_engines(struct xe_gt *gt)
 {
diff --git a/drivers/gpu/drm/xe/xe_gt_mcr.c b/drivers/gpu/drm/xe/xe_gt_mcr.c
index 2b2a4d9c3749..04f0098070a4 100644
--- a/drivers/gpu/drm/xe/xe_gt_mcr.c
+++ b/drivers/gpu/drm/xe/xe_gt_mcr.c
@@ -3,6 +3,9 @@
  * Copyright © 2022 Intel Corporation
  */
 
+#include <kunit/static_stub.h>
+#include <kunit/visibility.h>
+
 #include "xe_gt_mcr.h"
 
 #include "regs/xe_gt_regs.h"
@@ -553,6 +556,7 @@ void xe_gt_mcr_init_early(struct xe_gt *gt)
 	/* Mark instance 0 as initialized, we need this early for VRAM and CCS probe. */
 	gt->steering[INSTANCE0].initialized = true;
 }
+EXPORT_SYMBOL_IF_KUNIT(xe_gt_mcr_init_early);
 
 /**
  * xe_gt_mcr_init - Normal initialization of the MCR support
@@ -614,6 +618,26 @@ static bool reg_in_steering_type_ranges(struct xe_gt *gt,
 	return false;
 }
 
+/*
+ * xe_gt_mcr_check_reg - check if a register is recognized by this GT as MCR
+ * @gt: GT structure
+ * @reg: The register to check
+ *
+ * Returns true if the register offset falls within one of the MMIO ranges
+ * classified as MCR for the GT.
+ */
+bool xe_gt_mcr_check_reg(struct xe_gt *gt, struct xe_reg reg)
+{
+	KUNIT_STATIC_STUB_REDIRECT(xe_gt_mcr_check_reg, gt, reg);
+
+	for (int type = 0; type <= IMPLICIT_STEERING; type++)
+		if (reg_in_steering_type_ranges(gt, reg, type))
+			return true;
+
+	return false;
+}
+EXPORT_SYMBOL_IF_KUNIT(xe_gt_mcr_check_reg);
+
 /*
  * xe_gt_mcr_get_nonterminated_steering - find group/instance values that
  *    will steer a register to a non-terminated instance
diff --git a/drivers/gpu/drm/xe/xe_gt_mcr.h b/drivers/gpu/drm/xe/xe_gt_mcr.h
index 2be9419b8acc..75374662f10d 100644
--- a/drivers/gpu/drm/xe/xe_gt_mcr.h
+++ b/drivers/gpu/drm/xe/xe_gt_mcr.h
@@ -26,6 +26,7 @@ void xe_gt_mcr_unicast_write(struct xe_gt *gt, struct xe_reg_mcr mcr_reg,
 void xe_gt_mcr_multicast_write(struct xe_gt *gt, struct xe_reg_mcr mcr_reg,
 			       u32 value);
 
+bool xe_gt_mcr_check_reg(struct xe_gt *gt, struct xe_reg reg);
 bool xe_gt_mcr_get_nonterminated_steering(struct xe_gt *gt,
 					  struct xe_reg_mcr reg_mcr,
 					  u8 *group, u8 *instance);
diff --git a/drivers/gpu/drm/xe/xe_reg_sr.c b/drivers/gpu/drm/xe/xe_reg_sr.c
index 2df0277efb2f..e328f5072557 100644
--- a/drivers/gpu/drm/xe/xe_reg_sr.c
+++ b/drivers/gpu/drm/xe/xe_reg_sr.c
@@ -70,14 +70,49 @@ static void reg_sr_inc_error(struct xe_reg_sr *sr)
 #endif
 }
 
+static struct xe_reg sanitize_mcr(struct xe_reg_sr *sr,
+				  const struct xe_reg_sr_entry *e,
+				  struct xe_gt *gt)
+{
+	struct xe_reg reg = e->reg;
+	bool is_mcr;
+
+	/*
+	 * We need the gt structure to check MCR ranges.
+	 */
+	if (!gt)
+		return reg;
+
+	is_mcr = xe_gt_mcr_check_reg(gt, reg);
+
+	if (is_mcr && !reg.mcr) {
+		reg.mcr = 1;
+		xe_gt_notice(gt, "xe_reg_sr_entry using non-MCR register for address 0x%x, forcing MCR\n",
+			     reg.addr);
+		reg_sr_inc_error(sr);
+	}
+
+	if (!is_mcr && reg.mcr) {
+		reg.mcr = 0;
+		xe_gt_notice(gt, "xe_reg_sr_entry using MCR register for address 0x%x, forcing non-MCR\n",
+			     reg.addr);
+		reg_sr_inc_error(sr);
+	}
+
+	return reg;
+}
+
 int xe_reg_sr_add(struct xe_reg_sr *sr,
 		  const struct xe_reg_sr_entry *e,
 		  struct xe_gt *gt)
 {
 	unsigned long idx = e->reg.addr;
 	struct xe_reg_sr_entry *pentry = xa_load(&sr->xa, idx);
+	struct xe_reg reg;
 	int ret;
 
+	reg = sanitize_mcr(sr, e, gt);
+
 	if (pentry) {
 		if (!compatible_entries(pentry, e)) {
 			ret = -EINVAL;
@@ -98,6 +133,7 @@ int xe_reg_sr_add(struct xe_reg_sr *sr,
 	}
 
 	*pentry = *e;
+	pentry->reg = reg;
 	ret = xa_err(xa_store(&sr->xa, idx, pentry, GFP_KERNEL));
 	if (ret)
 		goto fail_free;

-- 
2.53.0

✓ CI.KUnit: success for Fix MCR inconsistencies in RTP tables (rev2)

[Patchwork]

== Series Details ==

Series: Fix MCR inconsistencies in RTP tables (rev2)
URL   : https://patchwork.freedesktop.org/series/160223/
State : success

== Summary ==

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

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

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

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

✓ Xe.CI.BAT: success for Fix MCR inconsistencies in RTP tables (rev2)

[Patchwork]

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

== Series Details ==

Series: Fix MCR inconsistencies in RTP tables (rev2)
URL   : https://patchwork.freedesktop.org/series/160223/
State : success

== Summary ==

CI Bug Log - changes from xe-5031-18d2011d95002966c64980ab06f36ee9211d496c_BAT -> xe-pw-160223v2_BAT
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  

Participating hosts (13 -> 13)
------------------------------

  No changes in participating hosts


Changes
-------

  No changes found


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

  * IGT: IGT_8900 -> IGT_8901
  * Linux: xe-5031-18d2011d95002966c64980ab06f36ee9211d496c -> xe-pw-160223v2

  IGT_8900: 529869b7d0fba64ed67819111079ff269bc8159a @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
  IGT_8901: 0a391e6b79fd6c6c7a55f75ea4769f31529821c4 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
  xe-5031-18d2011d95002966c64980ab06f36ee9211d496c: 18d2011d95002966c64980ab06f36ee9211d496c
  xe-pw-160223v2: 160223v2

== Logs ==

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

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

✗ Xe.CI.FULL: failure for Fix MCR inconsistencies in RTP tables (rev2)

[Patchwork]

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

== Series Details ==

Series: Fix MCR inconsistencies in RTP tables (rev2)
URL   : https://patchwork.freedesktop.org/series/160223/
State : failure

== Summary ==

CI Bug Log - changes from xe-5031-18d2011d95002966c64980ab06f36ee9211d496c_FULL -> xe-pw-160223v2_FULL
====================================================

Summary
-------

  **FAILURE**

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

  

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

  No changes in participating hosts

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

  Here are the unknown changes that may have been introduced in xe-pw-160223v2_FULL:

### IGT changes ###

#### Possible regressions ####

  * igt@xe_sriov_admin@exec-quantum-write-readback-vfs-disabled@pf-eq_ms-10:
    - shard-bmg:          [PASS][1] -> [SKIP][2]
   [1]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-1/igt@xe_sriov_admin@exec-quantum-write-readback-vfs-disabled@pf-eq_ms-10.html
   [2]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-6/igt@xe_sriov_admin@exec-quantum-write-readback-vfs-disabled@pf-eq_ms-10.html

  
#### Warnings ####

  * igt@kms_frontbuffer_tracking@fbcdrrshdr-2p-primscrn-pri-shrfb-draw-render:
    - shard-lnl:          [SKIP][3] ([Intel XE#7905]) -> [ABORT][4]
   [3]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-lnl-7/igt@kms_frontbuffer_tracking@fbcdrrshdr-2p-primscrn-pri-shrfb-draw-render.html
   [4]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-lnl-3/igt@kms_frontbuffer_tracking@fbcdrrshdr-2p-primscrn-pri-shrfb-draw-render.html

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

  Here are the changes found in xe-pw-160223v2_FULL that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@kms_big_fb@yf-tiled-32bpp-rotate-270:
    - shard-bmg:          NOTRUN -> [SKIP][5] ([Intel XE#1124])
   [5]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-9/igt@kms_big_fb@yf-tiled-32bpp-rotate-270.html

  * igt@kms_chamelium_color@ctm-red-to-blue:
    - shard-bmg:          NOTRUN -> [SKIP][6] ([Intel XE#2325] / [Intel XE#7358])
   [6]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@kms_chamelium_color@ctm-red-to-blue.html

  * igt@kms_chamelium_hpd@hdmi-hpd-for-each-pipe:
    - shard-bmg:          NOTRUN -> [SKIP][7] ([Intel XE#2252])
   [7]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-1/igt@kms_chamelium_hpd@hdmi-hpd-for-each-pipe.html

  * igt@kms_cursor_crc@cursor-sliding-32x10:
    - shard-bmg:          NOTRUN -> [SKIP][8] ([Intel XE#2320])
   [8]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-10/igt@kms_cursor_crc@cursor-sliding-32x10.html

  * igt@kms_cursor_legacy@flip-vs-cursor-atomic:
    - shard-bmg:          [PASS][9] -> [FAIL][10] ([Intel XE#7571])
   [9]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-6/igt@kms_cursor_legacy@flip-vs-cursor-atomic.html
   [10]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-5/igt@kms_cursor_legacy@flip-vs-cursor-atomic.html

  * igt@kms_fbcon_fbt@psr-suspend:
    - shard-lnl:          [PASS][11] -> [FAIL][12] ([Intel XE#7949])
   [11]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-lnl-2/igt@kms_fbcon_fbt@psr-suspend.html
   [12]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-lnl-4/igt@kms_fbcon_fbt@psr-suspend.html

  * igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1:
    - shard-lnl:          [PASS][13] -> [FAIL][14] ([Intel XE#301]) +2 other tests fail
   [13]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-lnl-5/igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1.html
   [14]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-lnl-8/igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1.html

  * igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-32bpp-4tiledg2rcccs-upscaling:
    - shard-bmg:          NOTRUN -> [SKIP][15] ([Intel XE#7178] / [Intel XE#7349])
   [15]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-9/igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-32bpp-4tiledg2rcccs-upscaling.html

  * igt@kms_frontbuffer_tracking@drrshdr-1p-rte:
    - shard-bmg:          NOTRUN -> [SKIP][16] ([Intel XE#2311]) +5 other tests skip
   [16]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-9/igt@kms_frontbuffer_tracking@drrshdr-1p-rte.html

  * igt@kms_frontbuffer_tracking@fbcpsrhdr-2p-primscrn-shrfb-pgflip-blt:
    - shard-bmg:          NOTRUN -> [SKIP][17] ([Intel XE#2313]) +5 other tests skip
   [17]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-1/igt@kms_frontbuffer_tracking@fbcpsrhdr-2p-primscrn-shrfb-pgflip-blt.html

  * igt@kms_hdr@invalid-hdr:
    - shard-bmg:          [PASS][18] -> [SKIP][19] ([Intel XE#1503])
   [18]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-1/igt@kms_hdr@invalid-hdr.html
   [19]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-5/igt@kms_hdr@invalid-hdr.html

  * igt@kms_hdr@invalid-hdr@pipe-a-hdmi-a-3-xrgb2101010:
    - shard-bmg:          [PASS][20] -> [SKIP][21] ([Intel XE#7922]) +1 other test skip
   [20]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-1/igt@kms_hdr@invalid-hdr@pipe-a-hdmi-a-3-xrgb2101010.html
   [21]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-5/igt@kms_hdr@invalid-hdr@pipe-a-hdmi-a-3-xrgb2101010.html

  * igt@kms_joiner@basic-big-joiner:
    - shard-bmg:          NOTRUN -> [SKIP][22] ([Intel XE#6901])
   [22]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-6/igt@kms_joiner@basic-big-joiner.html

  * igt@kms_plane@pixel-format-y-tiled-gen12-rc-ccs-cc-modifier-source-clamping:
    - shard-bmg:          NOTRUN -> [SKIP][23] ([Intel XE#7283])
   [23]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@kms_plane@pixel-format-y-tiled-gen12-rc-ccs-cc-modifier-source-clamping.html

  * igt@kms_psr@pr-dpms:
    - shard-bmg:          NOTRUN -> [SKIP][24] ([Intel XE#2234] / [Intel XE#2850])
   [24]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-3/igt@kms_psr@pr-dpms.html

  * igt@xe_eudebug_online@pagefault-write-stress:
    - shard-bmg:          NOTRUN -> [SKIP][25] ([Intel XE#7636]) +1 other test skip
   [25]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-10/igt@xe_eudebug_online@pagefault-write-stress.html

  * igt@xe_exec_basic@multigpu-once-userptr-rebind:
    - shard-bmg:          NOTRUN -> [SKIP][26] ([Intel XE#2322] / [Intel XE#7372])
   [26]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-6/igt@xe_exec_basic@multigpu-once-userptr-rebind.html

  * igt@xe_exec_fault_mode@many-execqueues-bindexecqueue-userptr-imm:
    - shard-bmg:          [PASS][27] -> [DMESG-FAIL][28] ([Intel XE#5545])
   [27]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-9/igt@xe_exec_fault_mode@many-execqueues-bindexecqueue-userptr-imm.html
   [28]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@xe_exec_fault_mode@many-execqueues-bindexecqueue-userptr-imm.html

  * igt@xe_exec_fault_mode@once-multi-queue:
    - shard-bmg:          NOTRUN -> [SKIP][29] ([Intel XE#7136])
   [29]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@xe_exec_fault_mode@once-multi-queue.html

  * igt@xe_exec_multi_queue@two-queues-userptr:
    - shard-bmg:          NOTRUN -> [SKIP][30] ([Intel XE#6874]) +3 other tests skip
   [30]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-3/igt@xe_exec_multi_queue@two-queues-userptr.html

  * igt@xe_exec_system_allocator@process-many-large-execqueues-mmap-remap:
    - shard-bmg:          [PASS][31] -> [SKIP][32] ([Intel XE#6703]) +38 other tests skip
   [31]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-2/igt@xe_exec_system_allocator@process-many-large-execqueues-mmap-remap.html
   [32]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@xe_exec_system_allocator@process-many-large-execqueues-mmap-remap.html

  * igt@xe_multigpu_svm@mgpu-concurrent-access-prefetch:
    - shard-bmg:          NOTRUN -> [SKIP][33] ([Intel XE#6964])
   [33]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-10/igt@xe_multigpu_svm@mgpu-concurrent-access-prefetch.html

  * igt@xe_pat@xa-app-transient-media-off:
    - shard-bmg:          NOTRUN -> [SKIP][34] ([Intel XE#7590])
   [34]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-1/igt@xe_pat@xa-app-transient-media-off.html

  * igt@xe_pmu@all-fn-engine-activity-load:
    - shard-bmg:          [PASS][35] -> [FAIL][36] ([Intel XE#5937])
   [35]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-9/igt@xe_pmu@all-fn-engine-activity-load.html
   [36]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-3/igt@xe_pmu@all-fn-engine-activity-load.html

  
#### Possible fixes ####

  * igt@kms_ccs@crc-primary-suspend-4-tiled-bmg-ccs@pipe-c-dp-2:
    - shard-bmg:          [INCOMPLETE][37] ([Intel XE#7084]) -> [PASS][38] +1 other test pass
   [37]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-4/igt@kms_ccs@crc-primary-suspend-4-tiled-bmg-ccs@pipe-c-dp-2.html
   [38]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-4/igt@kms_ccs@crc-primary-suspend-4-tiled-bmg-ccs@pipe-c-dp-2.html

  * igt@kms_flip@flip-vs-expired-vblank@a-edp1:
    - shard-lnl:          [FAIL][39] ([Intel XE#301]) -> [PASS][40]
   [39]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-lnl-8/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html
   [40]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-lnl-8/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html

  * igt@kms_frontbuffer_tracking@fbchdr-2p-scndscrn-pri-indfb-draw-blt:
    - shard-bmg:          [ABORT][41] -> [PASS][42]
   [41]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-2/igt@kms_frontbuffer_tracking@fbchdr-2p-scndscrn-pri-indfb-draw-blt.html
   [42]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@kms_frontbuffer_tracking@fbchdr-2p-scndscrn-pri-indfb-draw-blt.html

  * igt@kms_hdr@static-toggle@pipe-a-hdmi-a-3-xrgb16161616f:
    - shard-bmg:          [SKIP][43] ([Intel XE#7915]) -> [PASS][44] +5 other tests pass
   [43]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-8/igt@kms_hdr@static-toggle@pipe-a-hdmi-a-3-xrgb16161616f.html
   [44]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@kms_hdr@static-toggle@pipe-a-hdmi-a-3-xrgb16161616f.html

  * igt@kms_pm_dc@dc5-dpms:
    - shard-lnl:          [FAIL][45] ([Intel XE#7340] / [Intel XE#7504]) -> [PASS][46]
   [45]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-lnl-5/igt@kms_pm_dc@dc5-dpms.html
   [46]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-lnl-8/igt@kms_pm_dc@dc5-dpms.html

  * igt@xe_evict@evict-beng-mixed-many-threads-small:
    - shard-bmg:          [INCOMPLETE][47] ([Intel XE#6321]) -> [PASS][48]
   [47]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-4/igt@xe_evict@evict-beng-mixed-many-threads-small.html
   [48]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-3/igt@xe_evict@evict-beng-mixed-many-threads-small.html

  * igt@xe_sriov_flr@flr-twice:
    - shard-bmg:          [FAIL][49] ([Intel XE#6569]) -> [PASS][50] +1 other test pass
   [49]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-3/igt@xe_sriov_flr@flr-twice.html
   [50]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-5/igt@xe_sriov_flr@flr-twice.html

  * igt@xe_sriov_vram@vf-access-provisioned:
    - shard-bmg:          [FAIL][51] ([Intel XE#5937]) -> [PASS][52] +1 other test pass
   [51]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-1/igt@xe_sriov_vram@vf-access-provisioned.html
   [52]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-4/igt@xe_sriov_vram@vf-access-provisioned.html

  
#### Warnings ####

  * igt@kms_big_fb@yf-tiled-8bpp-rotate-90:
    - shard-bmg:          [SKIP][53] ([Intel XE#1124]) -> [SKIP][54] ([Intel XE#6703])
   [53]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-4/igt@kms_big_fb@yf-tiled-8bpp-rotate-90.html
   [54]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@kms_big_fb@yf-tiled-8bpp-rotate-90.html

  * igt@kms_ccs@ccs-on-another-bo-y-tiled-gen12-rc-ccs:
    - shard-bmg:          [SKIP][55] ([Intel XE#2887]) -> [SKIP][56] ([Intel XE#6703]) +2 other tests skip
   [55]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-3/igt@kms_ccs@ccs-on-another-bo-y-tiled-gen12-rc-ccs.html
   [56]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@kms_ccs@ccs-on-another-bo-y-tiled-gen12-rc-ccs.html

  * igt@kms_ccs@crc-primary-suspend-4-tiled-mtl-rc-ccs:
    - shard-bmg:          [SKIP][57] ([Intel XE#3432]) -> [SKIP][58] ([Intel XE#6703])
   [57]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-4/igt@kms_ccs@crc-primary-suspend-4-tiled-mtl-rc-ccs.html
   [58]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@kms_ccs@crc-primary-suspend-4-tiled-mtl-rc-ccs.html

  * igt@kms_color_pipeline@plane-lut3d-green-only:
    - shard-bmg:          [SKIP][59] ([Intel XE#6969] / [Intel XE#7006]) -> [SKIP][60] ([Intel XE#6703])
   [59]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-8/igt@kms_color_pipeline@plane-lut3d-green-only.html
   [60]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@kms_color_pipeline@plane-lut3d-green-only.html

  * igt@kms_frontbuffer_tracking@fbc-1p-offscreen-pri-shrfb-draw-mmap-wc:
    - shard-bmg:          [SKIP][61] ([Intel XE#4141]) -> [SKIP][62] ([Intel XE#6703]) +1 other test skip
   [61]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-8/igt@kms_frontbuffer_tracking@fbc-1p-offscreen-pri-shrfb-draw-mmap-wc.html
   [62]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@kms_frontbuffer_tracking@fbc-1p-offscreen-pri-shrfb-draw-mmap-wc.html

  * igt@kms_frontbuffer_tracking@fbcdrrshdr-1p-primscrn-pri-indfb-draw-blt:
    - shard-bmg:          [SKIP][63] ([Intel XE#2311]) -> [SKIP][64] ([Intel XE#6703]) +5 other tests skip
   [63]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-9/igt@kms_frontbuffer_tracking@fbcdrrshdr-1p-primscrn-pri-indfb-draw-blt.html
   [64]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@kms_frontbuffer_tracking@fbcdrrshdr-1p-primscrn-pri-indfb-draw-blt.html

  * igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-spr-indfb-onoff:
    - shard-bmg:          [SKIP][65] ([Intel XE#2313]) -> [SKIP][66] ([Intel XE#6703]) +1 other test skip
   [65]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-8/igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-spr-indfb-onoff.html
   [66]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-spr-indfb-onoff.html

  * igt@kms_frontbuffer_tracking@fbcpsrhdr-abgr161616f-draw-mmap-wc:
    - shard-bmg:          [SKIP][67] ([Intel XE#7061]) -> [SKIP][68] ([Intel XE#6703])
   [67]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-8/igt@kms_frontbuffer_tracking@fbcpsrhdr-abgr161616f-draw-mmap-wc.html
   [68]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@kms_frontbuffer_tracking@fbcpsrhdr-abgr161616f-draw-mmap-wc.html

  * igt@kms_joiner@invalid-modeset-force-ultra-joiner:
    - shard-bmg:          [SKIP][69] ([Intel XE#6911] / [Intel XE#7466]) -> [SKIP][70] ([Intel XE#6703])
   [69]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-10/igt@kms_joiner@invalid-modeset-force-ultra-joiner.html
   [70]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@kms_joiner@invalid-modeset-force-ultra-joiner.html

  * igt@kms_psr2_sf@fbc-pr-cursor-plane-move-continuous-exceed-fully-sf:
    - shard-bmg:          [SKIP][71] ([Intel XE#1489]) -> [SKIP][72] ([Intel XE#6703])
   [71]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-1/igt@kms_psr2_sf@fbc-pr-cursor-plane-move-continuous-exceed-fully-sf.html
   [72]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@kms_psr2_sf@fbc-pr-cursor-plane-move-continuous-exceed-fully-sf.html

  * igt@kms_tiled_display@basic-test-pattern-with-chamelium:
    - shard-bmg:          [SKIP][73] ([Intel XE#2509] / [Intel XE#7437]) -> [SKIP][74] ([Intel XE#2426] / [Intel XE#5848])
   [73]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-4/igt@kms_tiled_display@basic-test-pattern-with-chamelium.html
   [74]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-3/igt@kms_tiled_display@basic-test-pattern-with-chamelium.html

  * igt@xe_eudebug@basic-vm-bind-ufence:
    - shard-bmg:          [SKIP][75] ([Intel XE#7636]) -> [SKIP][76] ([Intel XE#6703]) +1 other test skip
   [75]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-3/igt@xe_eudebug@basic-vm-bind-ufence.html
   [76]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@xe_eudebug@basic-vm-bind-ufence.html

  * igt@xe_exec_basic@multigpu-many-execqueues-many-vm-bindexecqueue-rebind:
    - shard-bmg:          [SKIP][77] ([Intel XE#2322] / [Intel XE#7372]) -> [SKIP][78] ([Intel XE#6703])
   [77]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-4/igt@xe_exec_basic@multigpu-many-execqueues-many-vm-bindexecqueue-rebind.html
   [78]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@xe_exec_basic@multigpu-many-execqueues-many-vm-bindexecqueue-rebind.html

  * igt@xe_exec_fault_mode@many-execqueues-multi-queue-invalid-fault:
    - shard-bmg:          [SKIP][79] ([Intel XE#7136]) -> [SKIP][80] ([Intel XE#6703])
   [79]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-6/igt@xe_exec_fault_mode@many-execqueues-multi-queue-invalid-fault.html
   [80]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@xe_exec_fault_mode@many-execqueues-multi-queue-invalid-fault.html

  * igt@xe_exec_multi_queue@max-queues-preempt-mode-priority-smem:
    - shard-bmg:          [SKIP][81] ([Intel XE#6874]) -> [SKIP][82] ([Intel XE#6703]) +1 other test skip
   [81]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-4/igt@xe_exec_multi_queue@max-queues-preempt-mode-priority-smem.html
   [82]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@xe_exec_multi_queue@max-queues-preempt-mode-priority-smem.html

  * igt@xe_pm@d3cold-i2c:
    - shard-bmg:          [SKIP][83] ([Intel XE#5694] / [Intel XE#7370]) -> [SKIP][84] ([Intel XE#6703])
   [83]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5031-18d2011d95002966c64980ab06f36ee9211d496c/shard-bmg-3/igt@xe_pm@d3cold-i2c.html
   [84]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160223v2/shard-bmg-2/igt@xe_pm@d3cold-i2c.html

  
  [Intel XE#1124]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1124
  [Intel XE#1489]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1489
  [Intel XE#1503]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1503
  [Intel XE#2234]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2234
  [Intel XE#2252]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2252
  [Intel XE#2311]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2311
  [Intel XE#2313]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2313
  [Intel XE#2320]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2320
  [Intel XE#2322]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2322
  [Intel XE#2325]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2325
  [Intel XE#2426]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2426
  [Intel XE#2509]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2509
  [Intel XE#2850]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2850
  [Intel XE#2887]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2887
  [Intel XE#301]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/301
  [Intel XE#3432]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3432
  [Intel XE#4141]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4141
  [Intel XE#5545]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5545
  [Intel XE#5694]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5694
  [Intel XE#5848]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5848
  [Intel XE#5937]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5937
  [Intel XE#6321]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6321
  [Intel XE#6569]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6569
  [Intel XE#6703]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6703
  [Intel XE#6874]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6874
  [Intel XE#6901]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6901
  [Intel XE#6911]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6911
  [Intel XE#6964]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6964
  [Intel XE#6969]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6969
  [Intel XE#7006]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7006
  [Intel XE#7061]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7061
  [Intel XE#7084]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7084
  [Intel XE#7136]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7136
  [Intel XE#7178]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7178
  [Intel XE#7283]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7283
  [Intel XE#7340]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7340
  [Intel XE#7349]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7349
  [Intel XE#7358]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7358
  [Intel XE#7370]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7370
  [Intel XE#7372]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7372
  [Intel XE#7437]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7437
  [Intel XE#7466]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7466
  [Intel XE#7504]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7504
  [Intel XE#7571]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7571
  [Intel XE#7590]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7590
  [Intel XE#7636]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7636
  [Intel XE#7905]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7905
  [Intel XE#7915]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7915
  [Intel XE#7922]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7922
  [Intel XE#7949]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7949


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

  * IGT: IGT_8900 -> IGT_8901
  * Linux: xe-5031-18d2011d95002966c64980ab06f36ee9211d496c -> xe-pw-160223v2

  IGT_8900: 529869b7d0fba64ed67819111079ff269bc8159a @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
  IGT_8901: 0a391e6b79fd6c6c7a55f75ea4769f31529821c4 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
  xe-5031-18d2011d95002966c64980ab06f36ee9211d496c: 18d2011d95002966c64980ab06f36ee9211d496c
  xe-pw-160223v2: 160223v2

== Logs ==

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

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

Re: [PATCH v2 4/8] drm/xe/kunit: Add xe_kunit_helper_is_live_test()

[Jani Nikula]

On Fri, 08 May 2026, Gustavo Sousa <gustavo.sousa@intel.com> wrote:
> +/**
> + * xe_kunit_helper_is_live_test - Return true if @test is a live test.
> + * @test: the &kunit test
> + *
> + * Return: True for a live test and false otherwise.
> + */

Pardon me for being blunt, but I think this is the worst kind of
kernel-doc comment.

It doesn't provide any additional information to what the function name
and signature already convey (which is to say excellent job on naming
the function), but it fails to explain what "live test" means.

The extra bits of useful information people might need after seeing the
function xe_kunit_helper_is_live_test() in code are: What is a live
test, and what is it if it's not live? Dead?


BR,
Jani.

> +bool xe_kunit_helper_is_live_test(struct kunit *test)
> +{
> +	KUNIT_STATIC_STUB_REDIRECT(xe_kunit_helper_is_live_test, test);
> +	return false;
> +}

-- 
Jani Nikula, Intel

Re: [PATCH v2 4/8] drm/xe/kunit: Add xe_kunit_helper_is_live_test()

[Gustavo Sousa]

Jani Nikula <jani.nikula@linux.intel.com> writes:

> On Fri, 08 May 2026, Gustavo Sousa <gustavo.sousa@intel.com> wrote:
>> +/**
>> + * xe_kunit_helper_is_live_test - Return true if @test is a live test.
>> + * @test: the &kunit test
>> + *
>> + * Return: True for a live test and false otherwise.
>> + */
>
> Pardon me for being blunt, but I think this is the worst kind of
> kernel-doc comment.

I appreciate the bluntness! :-)

>
> It doesn't provide any additional information to what the function name
> and signature already convey (which is to say excellent job on naming
> the function), but it fails to explain what "live test" means.

I kind of just added this kernel-doc to fill a hole for "consistency",
but, yeah, it does not provide any new info.

>
> The extra bits of useful information people might need after seeing the
> function xe_kunit_helper_is_live_test() in code are: What is a live
> test, and what is it if it's not live? Dead?

Zombie? ;-)

Joking apart, I personally tend to use "regular" to refer to non-live
tests. I do agree we are missing some documentation on the subject.  I'm
not sure though this function should be the place to do it.  I think we
would be better off with a "DOC:" section for that (and also explain
other bits in there).  I think it would be sensible to rename
xe_kunit_helpers.c to simply xe_kunit.c and add such a section.

With that in place, this function would be kind of self-explanatory,
right? Is this a case we just drop the kernel-doc?

Or should we try to be consistent on "every public function should have
a kernel-doc"?  Is that even a rule or am I imagining things? :-)

--
Gustavo Sousa

>
>
> BR,
> Jani.
>
>> +bool xe_kunit_helper_is_live_test(struct kunit *test)
>> +{
>> +	KUNIT_STATIC_STUB_REDIRECT(xe_kunit_helper_is_live_test, test);
>> +	return false;
>> +}
>
> -- 
> Jani Nikula, Intel

Re: [PATCH v2 4/8] drm/xe/kunit: Add xe_kunit_helper_is_live_test()

[Jani Nikula]

On Mon, 11 May 2026, Gustavo Sousa <gustavo.sousa@intel.com> wrote:
> Jani Nikula <jani.nikula@linux.intel.com> writes:
>
>> On Fri, 08 May 2026, Gustavo Sousa <gustavo.sousa@intel.com> wrote:
>>> +/**
>>> + * xe_kunit_helper_is_live_test - Return true if @test is a live test.
>>> + * @test: the &kunit test
>>> + *
>>> + * Return: True for a live test and false otherwise.
>>> + */
>>
>> Pardon me for being blunt, but I think this is the worst kind of
>> kernel-doc comment.
>
> I appreciate the bluntness! :-)
>
>>
>> It doesn't provide any additional information to what the function name
>> and signature already convey (which is to say excellent job on naming
>> the function), but it fails to explain what "live test" means.
>
> I kind of just added this kernel-doc to fill a hole for "consistency",
> but, yeah, it does not provide any new info.
>
>>
>> The extra bits of useful information people might need after seeing the
>> function xe_kunit_helper_is_live_test() in code are: What is a live
>> test, and what is it if it's not live? Dead?
>
> Zombie? ;-)
>
> Joking apart, I personally tend to use "regular" to refer to non-live
> tests. I do agree we are missing some documentation on the subject.  I'm
> not sure though this function should be the place to do it.  I think we
> would be better off with a "DOC:" section for that (and also explain
> other bits in there).  I think it would be sensible to rename
> xe_kunit_helpers.c to simply xe_kunit.c and add such a section.
>
> With that in place, this function would be kind of self-explanatory,
> right? Is this a case we just drop the kernel-doc?
>
> Or should we try to be consistent on "every public function should have
> a kernel-doc"?  Is that even a rule or am I imagining things? :-)

I believe xe maintainership leans more towards requiring kernel-doc
comments than we do with i915 or display. I think the hard requirement
leads to a lot of unnecessary boilerplate, more geared towards filling
the requirement than being informative and helpful.

Personally, I value overview DOC: comments much more than kernel-doc
comments. If I were to add any hard requirement for documentation, it
would be for DOC: comments for each .c file.

Bottom line, for xe, ask for xe maintainer opinion.


BR,
Jani.

>
> --
> Gustavo Sousa
>
>>
>>
>> BR,
>> Jani.
>>
>>> +bool xe_kunit_helper_is_live_test(struct kunit *test)
>>> +{
>>> +	KUNIT_STATIC_STUB_REDIRECT(xe_kunit_helper_is_live_test, test);
>>> +	return false;
>>> +}
>>
>> -- 
>> Jani Nikula, Intel

-- 
Jani Nikula, Intel

Re: [PATCH v2 4/8] drm/xe/kunit: Add xe_kunit_helper_is_live_test()

[Gustavo Sousa]

Jani Nikula <jani.nikula@linux.intel.com> writes:

> On Mon, 11 May 2026, Gustavo Sousa <gustavo.sousa@intel.com> wrote:
>> Jani Nikula <jani.nikula@linux.intel.com> writes:
>>
>>> On Fri, 08 May 2026, Gustavo Sousa <gustavo.sousa@intel.com> wrote:
>>>> +/**
>>>> + * xe_kunit_helper_is_live_test - Return true if @test is a live test.
>>>> + * @test: the &kunit test
>>>> + *
>>>> + * Return: True for a live test and false otherwise.
>>>> + */
>>>
>>> Pardon me for being blunt, but I think this is the worst kind of
>>> kernel-doc comment.
>>
>> I appreciate the bluntness! :-)
>>
>>>
>>> It doesn't provide any additional information to what the function name
>>> and signature already convey (which is to say excellent job on naming
>>> the function), but it fails to explain what "live test" means.
>>
>> I kind of just added this kernel-doc to fill a hole for "consistency",
>> but, yeah, it does not provide any new info.
>>
>>>
>>> The extra bits of useful information people might need after seeing the
>>> function xe_kunit_helper_is_live_test() in code are: What is a live
>>> test, and what is it if it's not live? Dead?
>>
>> Zombie? ;-)
>>
>> Joking apart, I personally tend to use "regular" to refer to non-live
>> tests. I do agree we are missing some documentation on the subject.  I'm
>> not sure though this function should be the place to do it.  I think we
>> would be better off with a "DOC:" section for that (and also explain
>> other bits in there).  I think it would be sensible to rename
>> xe_kunit_helpers.c to simply xe_kunit.c and add such a section.
>>
>> With that in place, this function would be kind of self-explanatory,
>> right? Is this a case we just drop the kernel-doc?
>>
>> Or should we try to be consistent on "every public function should have
>> a kernel-doc"?  Is that even a rule or am I imagining things? :-)
>
> I believe xe maintainership leans more towards requiring kernel-doc
> comments than we do with i915 or display. I think the hard requirement
> leads to a lot of unnecessary boilerplate, more geared towards filling
> the requirement than being informative and helpful.
>
> Personally, I value overview DOC: comments much more than kernel-doc
> comments. If I were to add any hard requirement for documentation, it
> would be for DOC: comments for each .c file.
>
> Bottom line, for xe, ask for xe maintainer opinion.

Cc:Xe maintainers, in case they want to chime in.

--
Gustavo Sousa

>
>
> BR,
> Jani.
>
>>
>> --
>> Gustavo Sousa
>>
>>>
>>>
>>> BR,
>>> Jani.
>>>
>>>> +bool xe_kunit_helper_is_live_test(struct kunit *test)
>>>> +{
>>>> +	KUNIT_STATIC_STUB_REDIRECT(xe_kunit_helper_is_live_test, test);
>>>> +	return false;
>>>> +}
>>>
>>> -- 
>>> Jani Nikula, Intel
>
> -- 
> Jani Nikula, Intel

Re: [PATCH v2 4/8] drm/xe/kunit: Add xe_kunit_helper_is_live_test()

[Rodrigo Vivi]

On Mon, May 11, 2026 at 09:30:49AM -0300, Gustavo Sousa wrote:
> Jani Nikula <jani.nikula@linux.intel.com> writes:
> 
> > On Mon, 11 May 2026, Gustavo Sousa <gustavo.sousa@intel.com> wrote:
> >> Jani Nikula <jani.nikula@linux.intel.com> writes:
> >>
> >>> On Fri, 08 May 2026, Gustavo Sousa <gustavo.sousa@intel.com> wrote:
> >>>> +/**
> >>>> + * xe_kunit_helper_is_live_test - Return true if @test is a live test.
> >>>> + * @test: the &kunit test
> >>>> + *
> >>>> + * Return: True for a live test and false otherwise.
> >>>> + */
> >>>
> >>> Pardon me for being blunt, but I think this is the worst kind of
> >>> kernel-doc comment.
> >>
> >> I appreciate the bluntness! :-)
> >>
> >>>
> >>> It doesn't provide any additional information to what the function name
> >>> and signature already convey (which is to say excellent job on naming
> >>> the function), but it fails to explain what "live test" means.
> >>
> >> I kind of just added this kernel-doc to fill a hole for "consistency",
> >> but, yeah, it does not provide any new info.
> >>
> >>>
> >>> The extra bits of useful information people might need after seeing the
> >>> function xe_kunit_helper_is_live_test() in code are: What is a live
> >>> test, and what is it if it's not live? Dead?
> >>
> >> Zombie? ;-)
> >>
> >> Joking apart, I personally tend to use "regular" to refer to non-live
> >> tests. I do agree we are missing some documentation on the subject.  I'm
> >> not sure though this function should be the place to do it.  I think we
> >> would be better off with a "DOC:" section for that (and also explain
> >> other bits in there).  I think it would be sensible to rename
> >> xe_kunit_helpers.c to simply xe_kunit.c and add such a section.
> >>
> >> With that in place, this function would be kind of self-explanatory,
> >> right? Is this a case we just drop the kernel-doc?
> >>
> >> Or should we try to be consistent on "every public function should have
> >> a kernel-doc"?  Is that even a rule or am I imagining things? :-)
> >
> > I believe xe maintainership leans more towards requiring kernel-doc
> > comments than we do with i915 or display. I think the hard requirement
> > leads to a lot of unnecessary boilerplate, more geared towards filling
> > the requirement than being informative and helpful.
> >
> > Personally, I value overview DOC: comments much more than kernel-doc
> > comments. If I were to add any hard requirement for documentation, it
> > would be for DOC: comments for each .c file.
> >
> > Bottom line, for xe, ask for xe maintainer opinion.
> 
> Cc:Xe maintainers, in case they want to chime in.

I'm definitely the one to be blamed by requesting docs to every
'public' function in Xe. :)

In my view this forces developer to see the .c,.h pair as a 'component'
with specific entry points and a reason to exist, rather than some
architecture like i915 where .c/.h pairs were only created when some file
was 'too big'. With the component in mind it is easier to identify when
something is abusing the interface and accessing specific internal
types directly rather than having a function entry point to handle it.

But well, the 'Doc: ' is actually part fundamental in this component.
We should definitely have a 'Doc: ' as well that justifies and give
reasoning to the component.

That said, in this patch here specifically I agree with Jani. We are
missing the 'Doc: with the reasoning for the component, and the
'public' function documentation could be bringing more useful information
like Jani pointed out, instead of just stating twice the return value.

Thanks,
Rodrigo.

> 
> --
> Gustavo Sousa
> 
> >
> >
> > BR,
> > Jani.
> >
> >>
> >> --
> >> Gustavo Sousa
> >>
> >>>
> >>>
> >>> BR,
> >>> Jani.
> >>>
> >>>> +bool xe_kunit_helper_is_live_test(struct kunit *test)
> >>>> +{
> >>>> +	KUNIT_STATIC_STUB_REDIRECT(xe_kunit_helper_is_live_test, test);
> >>>> +	return false;
> >>>> +}
> >>>
> >>> -- 
> >>> Jani Nikula, Intel
> >
> > -- 
> > Jani Nikula, Intel

Re: [PATCH v2 4/8] drm/xe/kunit: Add xe_kunit_helper_is_live_test()

[Gustavo Sousa]

Rodrigo Vivi <rodrigo.vivi@intel.com> writes:

> On Mon, May 11, 2026 at 09:30:49AM -0300, Gustavo Sousa wrote:
>> Jani Nikula <jani.nikula@linux.intel.com> writes:
>> 
>> > On Mon, 11 May 2026, Gustavo Sousa <gustavo.sousa@intel.com> wrote:
>> >> Jani Nikula <jani.nikula@linux.intel.com> writes:
>> >>
>> >>> On Fri, 08 May 2026, Gustavo Sousa <gustavo.sousa@intel.com> wrote:
>> >>>> +/**
>> >>>> + * xe_kunit_helper_is_live_test - Return true if @test is a live test.
>> >>>> + * @test: the &kunit test
>> >>>> + *
>> >>>> + * Return: True for a live test and false otherwise.
>> >>>> + */
>> >>>
>> >>> Pardon me for being blunt, but I think this is the worst kind of
>> >>> kernel-doc comment.
>> >>
>> >> I appreciate the bluntness! :-)
>> >>
>> >>>
>> >>> It doesn't provide any additional information to what the function name
>> >>> and signature already convey (which is to say excellent job on naming
>> >>> the function), but it fails to explain what "live test" means.
>> >>
>> >> I kind of just added this kernel-doc to fill a hole for "consistency",
>> >> but, yeah, it does not provide any new info.
>> >>
>> >>>
>> >>> The extra bits of useful information people might need after seeing the
>> >>> function xe_kunit_helper_is_live_test() in code are: What is a live
>> >>> test, and what is it if it's not live? Dead?
>> >>
>> >> Zombie? ;-)
>> >>
>> >> Joking apart, I personally tend to use "regular" to refer to non-live
>> >> tests. I do agree we are missing some documentation on the subject.  I'm
>> >> not sure though this function should be the place to do it.  I think we
>> >> would be better off with a "DOC:" section for that (and also explain
>> >> other bits in there).  I think it would be sensible to rename
>> >> xe_kunit_helpers.c to simply xe_kunit.c and add such a section.
>> >>
>> >> With that in place, this function would be kind of self-explanatory,
>> >> right? Is this a case we just drop the kernel-doc?
>> >>
>> >> Or should we try to be consistent on "every public function should have
>> >> a kernel-doc"?  Is that even a rule or am I imagining things? :-)
>> >
>> > I believe xe maintainership leans more towards requiring kernel-doc
>> > comments than we do with i915 or display. I think the hard requirement
>> > leads to a lot of unnecessary boilerplate, more geared towards filling
>> > the requirement than being informative and helpful.
>> >
>> > Personally, I value overview DOC: comments much more than kernel-doc
>> > comments. If I were to add any hard requirement for documentation, it
>> > would be for DOC: comments for each .c file.
>> >
>> > Bottom line, for xe, ask for xe maintainer opinion.
>> 
>> Cc:Xe maintainers, in case they want to chime in.
>
> I'm definitely the one to be blamed by requesting docs to every
> 'public' function in Xe. :)
>
> In my view this forces developer to see the .c,.h pair as a 'component'
> with specific entry points and a reason to exist, rather than some
> architecture like i915 where .c/.h pairs were only created when some file
> was 'too big'. With the component in mind it is easier to identify when
> something is abusing the interface and accessing specific internal
> types directly rather than having a function entry point to handle it.
>
> But well, the 'Doc: ' is actually part fundamental in this component.
> We should definitely have a 'Doc: ' as well that justifies and give
> reasoning to the component.
>
> That said, in this patch here specifically I agree with Jani. We are
> missing the 'Doc: with the reasoning for the component, and the
> 'public' function documentation could be bringing more useful information
> like Jani pointed out, instead of just stating twice the return value.

One relevant point here is that, once we have a "DOC:" section that will
explain "regular" tests and "live" tests, it will be redundant to add
that detail to the kernel-doc for xe_kunit_helper_is_live_test(), and,
IMO, it would be better not to repeat it.

That brings the question: should "every public function needs a
kernel-doc" be a hard rule?

--
Gustavo Sousa

>
> Thanks,
> Rodrigo.
>
>> 
>> --
>> Gustavo Sousa
>> 
>> >
>> >
>> > BR,
>> > Jani.
>> >
>> >>
>> >> --
>> >> Gustavo Sousa
>> >>
>> >>>
>> >>>
>> >>> BR,
>> >>> Jani.
>> >>>
>> >>>> +bool xe_kunit_helper_is_live_test(struct kunit *test)
>> >>>> +{
>> >>>> +	KUNIT_STATIC_STUB_REDIRECT(xe_kunit_helper_is_live_test, test);
>> >>>> +	return false;
>> >>>> +}
>> >>>
>> >>> -- 
>> >>> Jani Nikula, Intel
>> >
>> > -- 
>> > Jani Nikula, Intel

Re: [PATCH v2 4/8] drm/xe/kunit: Add xe_kunit_helper_is_live_test()

[Rodrigo Vivi]

On Mon, May 11, 2026 at 06:01:59PM -0300, Gustavo Sousa wrote:
> Rodrigo Vivi <rodrigo.vivi@intel.com> writes:
> 
> > On Mon, May 11, 2026 at 09:30:49AM -0300, Gustavo Sousa wrote:
> >> Jani Nikula <jani.nikula@linux.intel.com> writes:
> >> 
> >> > On Mon, 11 May 2026, Gustavo Sousa <gustavo.sousa@intel.com> wrote:
> >> >> Jani Nikula <jani.nikula@linux.intel.com> writes:
> >> >>
> >> >>> On Fri, 08 May 2026, Gustavo Sousa <gustavo.sousa@intel.com> wrote:
> >> >>>> +/**
> >> >>>> + * xe_kunit_helper_is_live_test - Return true if @test is a live test.
> >> >>>> + * @test: the &kunit test
> >> >>>> + *
> >> >>>> + * Return: True for a live test and false otherwise.
> >> >>>> + */
> >> >>>
> >> >>> Pardon me for being blunt, but I think this is the worst kind of
> >> >>> kernel-doc comment.
> >> >>
> >> >> I appreciate the bluntness! :-)
> >> >>
> >> >>>
> >> >>> It doesn't provide any additional information to what the function name
> >> >>> and signature already convey (which is to say excellent job on naming
> >> >>> the function), but it fails to explain what "live test" means.
> >> >>
> >> >> I kind of just added this kernel-doc to fill a hole for "consistency",
> >> >> but, yeah, it does not provide any new info.
> >> >>
> >> >>>
> >> >>> The extra bits of useful information people might need after seeing the
> >> >>> function xe_kunit_helper_is_live_test() in code are: What is a live
> >> >>> test, and what is it if it's not live? Dead?
> >> >>
> >> >> Zombie? ;-)
> >> >>
> >> >> Joking apart, I personally tend to use "regular" to refer to non-live
> >> >> tests. I do agree we are missing some documentation on the subject.  I'm
> >> >> not sure though this function should be the place to do it.  I think we
> >> >> would be better off with a "DOC:" section for that (and also explain
> >> >> other bits in there).  I think it would be sensible to rename
> >> >> xe_kunit_helpers.c to simply xe_kunit.c and add such a section.
> >> >>
> >> >> With that in place, this function would be kind of self-explanatory,
> >> >> right? Is this a case we just drop the kernel-doc?
> >> >>
> >> >> Or should we try to be consistent on "every public function should have
> >> >> a kernel-doc"?  Is that even a rule or am I imagining things? :-)
> >> >
> >> > I believe xe maintainership leans more towards requiring kernel-doc
> >> > comments than we do with i915 or display. I think the hard requirement
> >> > leads to a lot of unnecessary boilerplate, more geared towards filling
> >> > the requirement than being informative and helpful.
> >> >
> >> > Personally, I value overview DOC: comments much more than kernel-doc
> >> > comments. If I were to add any hard requirement for documentation, it
> >> > would be for DOC: comments for each .c file.
> >> >
> >> > Bottom line, for xe, ask for xe maintainer opinion.
> >> 
> >> Cc:Xe maintainers, in case they want to chime in.
> >
> > I'm definitely the one to be blamed by requesting docs to every
> > 'public' function in Xe. :)
> >
> > In my view this forces developer to see the .c,.h pair as a 'component'
> > with specific entry points and a reason to exist, rather than some
> > architecture like i915 where .c/.h pairs were only created when some file
> > was 'too big'. With the component in mind it is easier to identify when
> > something is abusing the interface and accessing specific internal
> > types directly rather than having a function entry point to handle it.
> >
> > But well, the 'Doc: ' is actually part fundamental in this component.
> > We should definitely have a 'Doc: ' as well that justifies and give
> > reasoning to the component.
> >
> > That said, in this patch here specifically I agree with Jani. We are
> > missing the 'Doc: with the reasoning for the component, and the
> > 'public' function documentation could be bringing more useful information
> > like Jani pointed out, instead of just stating twice the return value.
> 
> One relevant point here is that, once we have a "DOC:" section that will
> explain "regular" tests and "live" tests, it will be redundant to add
> that detail to the kernel-doc for xe_kunit_helper_is_live_test(), and,
> IMO, it would be better not to repeat it.
> 
> That brings the question: should "every public function needs a
> kernel-doc" be a hard rule?

I do not like repetitions either. But I like the result of the doc when
we have a doc explain the overall flow and underneath the function inputs
of that component. Gives a good overview.

So, why can't you do in a way that the doc explains most of it and overall
flow and still leaving a room for the function to explain itself?!

But perhaps we can still say that it is case by case and live without the
:internals:... 

> 
> --
> Gustavo Sousa
> 
> >
> > Thanks,
> > Rodrigo.
> >
> >> 
> >> --
> >> Gustavo Sousa
> >> 
> >> >
> >> >
> >> > BR,
> >> > Jani.
> >> >
> >> >>
> >> >> --
> >> >> Gustavo Sousa
> >> >>
> >> >>>
> >> >>>
> >> >>> BR,
> >> >>> Jani.
> >> >>>
> >> >>>> +bool xe_kunit_helper_is_live_test(struct kunit *test)
> >> >>>> +{
> >> >>>> +	KUNIT_STATIC_STUB_REDIRECT(xe_kunit_helper_is_live_test, test);
> >> >>>> +	return false;
> >> >>>> +}
> >> >>>
> >> >>> -- 
> >> >>> Jani Nikula, Intel
> >> >
> >> > -- 
> >> > Jani Nikula, Intel

Re: [PATCH v2 4/8] drm/xe/kunit: Add xe_kunit_helper_is_live_test()

[Michal Wajdeczko]

On 5/8/2026 11:42 PM, Gustavo Sousa wrote:
> In upcoming changes we will need to differentiate between regular and
> live KUnit tests.  Add the function xe_kunit_helper_is_live_test() for
> that purpose.
> 
> Note that this is implemented in a rather hackish way, by leveraging
> KUnit's static stubbing functionality.  A better approach would be to
> store a boolean somewhere in test->priv that would tell if a test is a
> live one, however that's not quite feasible today given the inconsistent
> usage of test->priv (there is not a single uniform type of data assigned
> to it across existing test).
> 
> v2:
>   - s/xe_kunit_helper_is_live_test/xe_kunit_helper_is_live_test/
>     (Michal)
> 
> Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
> ---
>  drivers/gpu/drm/xe/tests/xe_kunit_helpers.c | 26 ++++++++++++++++++++++++++
>  drivers/gpu/drm/xe/tests/xe_kunit_helpers.h |  4 ++++
>  2 files changed, 30 insertions(+)
> 
> diff --git a/drivers/gpu/drm/xe/tests/xe_kunit_helpers.c b/drivers/gpu/drm/xe/tests/xe_kunit_helpers.c
> index bc5156966ce9..d2f654f53cc7 100644
> --- a/drivers/gpu/drm/xe/tests/xe_kunit_helpers.c
> +++ b/drivers/gpu/drm/xe/tests/xe_kunit_helpers.c
> @@ -16,6 +16,18 @@
>  #include "xe_device_types.h"
>  #include "xe_pm.h"
>  
> +/**
> + * xe_kunit_helper_is_live_test - Return true if @test is a live test.
> + * @test: the &kunit test
> + *
> + * Return: True for a live test and false otherwise.
> + */
> +bool xe_kunit_helper_is_live_test(struct kunit *test)
> +{
> +	KUNIT_STATIC_STUB_REDIRECT(xe_kunit_helper_is_live_test, test);
> +	return false;
> +}

or maybe we only need something like this in xe_device.h:

+static inline bool xe_device_is_real(const struct xe_device *xe)
+{
+       return xe->drm.dev->bus == &pci_bus_type;
+}

as all fake kunit xe devices will have kunit_bus_type instead

already tested ;)

> +
>  /**
>   * xe_kunit_helper_alloc_xe_device - Allocate a &xe_device for a KUnit test.
>   * @test: the &kunit where this &xe_device will be used
> @@ -93,6 +105,11 @@ EXPORT_SYMBOL_IF_KUNIT(xe_kunit_helper_xe_device_test_init);
>  
>  KUNIT_DEFINE_ACTION_WRAPPER(put_xe_pm_runtime, xe_pm_runtime_put, struct xe_device *);
>  
> +static bool xe_kunit_is_live_test_indeed(struct kunit *test)
> +{
> +	return true;
> +}
> +
>  /**
>   * xe_kunit_helper_xe_device_live_test_init - Prepare a &xe_device for
>   *                                            use in a live KUnit test.
> @@ -116,6 +133,15 @@ int xe_kunit_helper_xe_device_live_test_init(struct kunit *test)
>  {
>  	struct xe_device *xe = xe_device_const_cast(test->param_value);
>  
> +	/*
> +	 * FIXME: This is a hack and a better solution is to have the "priv"
> +	 * member of tests have a boolean to tell if a test is a live one.
> +	 * Unfortunately that can't be done today because "priv" does not point
> +	 * to a single unified type across existing tests.
> +	 */
> +	kunit_activate_static_stub(test, xe_kunit_helper_is_live_test,
> +				   xe_kunit_is_live_test_indeed);
> +
>  	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, xe);
>  	kunit_info(test, "running on %s device\n", xe->info.platform_name);
>  
> diff --git a/drivers/gpu/drm/xe/tests/xe_kunit_helpers.h b/drivers/gpu/drm/xe/tests/xe_kunit_helpers.h
> index 83665f7b1254..71cfeb2d4efb 100644
> --- a/drivers/gpu/drm/xe/tests/xe_kunit_helpers.h
> +++ b/drivers/gpu/drm/xe/tests/xe_kunit_helpers.h
> @@ -6,10 +6,14 @@
>  #ifndef _XE_KUNIT_HELPERS_H_
>  #define _XE_KUNIT_HELPERS_H_
>  
> +#include <linux/types.h>
> +
>  struct device;
>  struct kunit;
>  struct xe_device;
>  
> +bool xe_kunit_helper_is_live_test(struct kunit *test);
> +
>  struct xe_device *xe_kunit_helper_alloc_xe_device(struct kunit *test,
>  						  struct device *dev);
>  int xe_kunit_helper_xe_device_test_init(struct kunit *test);
> 

Re: [PATCH v2 4/8] drm/xe/kunit: Add xe_kunit_helper_is_live_test()

[Gustavo Sousa]

Gustavo Sousa <gustavo.sousa@intel.com> writes:

> In upcoming changes we will need to differentiate between regular and
> live KUnit tests.  Add the function xe_kunit_helper_is_live_test() for
> that purpose.

Just now I realized that this function is not needed by this series
anymore.  We can simply dismiss it.

--
Gustavo Sousa

>
> Note that this is implemented in a rather hackish way, by leveraging
> KUnit's static stubbing functionality.  A better approach would be to
> store a boolean somewhere in test->priv that would tell if a test is a
> live one, however that's not quite feasible today given the inconsistent
> usage of test->priv (there is not a single uniform type of data assigned
> to it across existing test).
>
> v2:
>   - s/xe_kunit_helper_is_live_test/xe_kunit_helper_is_live_test/
>     (Michal)
>
> Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
> ---
>  drivers/gpu/drm/xe/tests/xe_kunit_helpers.c | 26 ++++++++++++++++++++++++++
>  drivers/gpu/drm/xe/tests/xe_kunit_helpers.h |  4 ++++
>  2 files changed, 30 insertions(+)
>
> diff --git a/drivers/gpu/drm/xe/tests/xe_kunit_helpers.c b/drivers/gpu/drm/xe/tests/xe_kunit_helpers.c
> index bc5156966ce9..d2f654f53cc7 100644
> --- a/drivers/gpu/drm/xe/tests/xe_kunit_helpers.c
> +++ b/drivers/gpu/drm/xe/tests/xe_kunit_helpers.c
> @@ -16,6 +16,18 @@
>  #include "xe_device_types.h"
>  #include "xe_pm.h"
>  
> +/**
> + * xe_kunit_helper_is_live_test - Return true if @test is a live test.
> + * @test: the &kunit test
> + *
> + * Return: True for a live test and false otherwise.
> + */
> +bool xe_kunit_helper_is_live_test(struct kunit *test)
> +{
> +	KUNIT_STATIC_STUB_REDIRECT(xe_kunit_helper_is_live_test, test);
> +	return false;
> +}
> +
>  /**
>   * xe_kunit_helper_alloc_xe_device - Allocate a &xe_device for a KUnit test.
>   * @test: the &kunit where this &xe_device will be used
> @@ -93,6 +105,11 @@ EXPORT_SYMBOL_IF_KUNIT(xe_kunit_helper_xe_device_test_init);
>  
>  KUNIT_DEFINE_ACTION_WRAPPER(put_xe_pm_runtime, xe_pm_runtime_put, struct xe_device *);
>  
> +static bool xe_kunit_is_live_test_indeed(struct kunit *test)
> +{
> +	return true;
> +}
> +
>  /**
>   * xe_kunit_helper_xe_device_live_test_init - Prepare a &xe_device for
>   *                                            use in a live KUnit test.
> @@ -116,6 +133,15 @@ int xe_kunit_helper_xe_device_live_test_init(struct kunit *test)
>  {
>  	struct xe_device *xe = xe_device_const_cast(test->param_value);
>  
> +	/*
> +	 * FIXME: This is a hack and a better solution is to have the "priv"
> +	 * member of tests have a boolean to tell if a test is a live one.
> +	 * Unfortunately that can't be done today because "priv" does not point
> +	 * to a single unified type across existing tests.
> +	 */
> +	kunit_activate_static_stub(test, xe_kunit_helper_is_live_test,
> +				   xe_kunit_is_live_test_indeed);
> +
>  	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, xe);
>  	kunit_info(test, "running on %s device\n", xe->info.platform_name);
>  
> diff --git a/drivers/gpu/drm/xe/tests/xe_kunit_helpers.h b/drivers/gpu/drm/xe/tests/xe_kunit_helpers.h
> index 83665f7b1254..71cfeb2d4efb 100644
> --- a/drivers/gpu/drm/xe/tests/xe_kunit_helpers.h
> +++ b/drivers/gpu/drm/xe/tests/xe_kunit_helpers.h
> @@ -6,10 +6,14 @@
>  #ifndef _XE_KUNIT_HELPERS_H_
>  #define _XE_KUNIT_HELPERS_H_
>  
> +#include <linux/types.h>
> +
>  struct device;
>  struct kunit;
>  struct xe_device;
>  
> +bool xe_kunit_helper_is_live_test(struct kunit *test);
> +
>  struct xe_device *xe_kunit_helper_alloc_xe_device(struct kunit *test,
>  						  struct device *dev);
>  int xe_kunit_helper_xe_device_test_init(struct kunit *test);
>
> -- 
> 2.53.0

Re: [PATCH v2 4/8] drm/xe/kunit: Add xe_kunit_helper_is_live_test()

[Gustavo Sousa]

Michal Wajdeczko <michal.wajdeczko@intel.com> writes:

> On 5/8/2026 11:42 PM, Gustavo Sousa wrote:
>> In upcoming changes we will need to differentiate between regular and
>> live KUnit tests.  Add the function xe_kunit_helper_is_live_test() for
>> that purpose.
>> 
>> Note that this is implemented in a rather hackish way, by leveraging
>> KUnit's static stubbing functionality.  A better approach would be to
>> store a boolean somewhere in test->priv that would tell if a test is a
>> live one, however that's not quite feasible today given the inconsistent
>> usage of test->priv (there is not a single uniform type of data assigned
>> to it across existing test).
>> 
>> v2:
>>   - s/xe_kunit_helper_is_live_test/xe_kunit_helper_is_live_test/
>>     (Michal)
>> 
>> Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
>> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
>> ---
>>  drivers/gpu/drm/xe/tests/xe_kunit_helpers.c | 26 ++++++++++++++++++++++++++
>>  drivers/gpu/drm/xe/tests/xe_kunit_helpers.h |  4 ++++
>>  2 files changed, 30 insertions(+)
>> 
>> diff --git a/drivers/gpu/drm/xe/tests/xe_kunit_helpers.c b/drivers/gpu/drm/xe/tests/xe_kunit_helpers.c
>> index bc5156966ce9..d2f654f53cc7 100644
>> --- a/drivers/gpu/drm/xe/tests/xe_kunit_helpers.c
>> +++ b/drivers/gpu/drm/xe/tests/xe_kunit_helpers.c
>> @@ -16,6 +16,18 @@
>>  #include "xe_device_types.h"
>>  #include "xe_pm.h"
>>  
>> +/**
>> + * xe_kunit_helper_is_live_test - Return true if @test is a live test.
>> + * @test: the &kunit test
>> + *
>> + * Return: True for a live test and false otherwise.
>> + */
>> +bool xe_kunit_helper_is_live_test(struct kunit *test)
>> +{
>> +	KUNIT_STATIC_STUB_REDIRECT(xe_kunit_helper_is_live_test, test);
>> +	return false;
>> +}
>
> or maybe we only need something like this in xe_device.h:
>
> +static inline bool xe_device_is_real(const struct xe_device *xe)
> +{
> +       return xe->drm.dev->bus == &pci_bus_type;
> +}
>
> as all fake kunit xe devices will have kunit_bus_type instead

Yep.  This is much cleaner.  Thanks!

Even though I realized after the fact that this
xe_kunit_helper_is_live_test() is not necessary anymore in this series,
the asserts you included in [1] seem useful, so I provided my r-b there.

[1] https://lore.kernel.org/intel-xe/20260512194514.3414-1-michal.wajdeczko@intel.com/

--
Gustavo Sousa

>
> already tested ;)
>
>> +
>>  /**
>>   * xe_kunit_helper_alloc_xe_device - Allocate a &xe_device for a KUnit test.
>>   * @test: the &kunit where this &xe_device will be used
>> @@ -93,6 +105,11 @@ EXPORT_SYMBOL_IF_KUNIT(xe_kunit_helper_xe_device_test_init);
>>  
>>  KUNIT_DEFINE_ACTION_WRAPPER(put_xe_pm_runtime, xe_pm_runtime_put, struct xe_device *);
>>  
>> +static bool xe_kunit_is_live_test_indeed(struct kunit *test)
>> +{
>> +	return true;
>> +}
>> +
>>  /**
>>   * xe_kunit_helper_xe_device_live_test_init - Prepare a &xe_device for
>>   *                                            use in a live KUnit test.
>> @@ -116,6 +133,15 @@ int xe_kunit_helper_xe_device_live_test_init(struct kunit *test)
>>  {
>>  	struct xe_device *xe = xe_device_const_cast(test->param_value);
>>  
>> +	/*
>> +	 * FIXME: This is a hack and a better solution is to have the "priv"
>> +	 * member of tests have a boolean to tell if a test is a live one.
>> +	 * Unfortunately that can't be done today because "priv" does not point
>> +	 * to a single unified type across existing tests.
>> +	 */
>> +	kunit_activate_static_stub(test, xe_kunit_helper_is_live_test,
>> +				   xe_kunit_is_live_test_indeed);
>> +
>>  	KUNIT_ASSERT_NOT_ERR_OR_NULL(test, xe);
>>  	kunit_info(test, "running on %s device\n", xe->info.platform_name);
>>  
>> diff --git a/drivers/gpu/drm/xe/tests/xe_kunit_helpers.h b/drivers/gpu/drm/xe/tests/xe_kunit_helpers.h
>> index 83665f7b1254..71cfeb2d4efb 100644
>> --- a/drivers/gpu/drm/xe/tests/xe_kunit_helpers.h
>> +++ b/drivers/gpu/drm/xe/tests/xe_kunit_helpers.h
>> @@ -6,10 +6,14 @@
>>  #ifndef _XE_KUNIT_HELPERS_H_
>>  #define _XE_KUNIT_HELPERS_H_
>>  
>> +#include <linux/types.h>
>> +
>>  struct device;
>>  struct kunit;
>>  struct xe_device;
>>  
>> +bool xe_kunit_helper_is_live_test(struct kunit *test);
>> +
>>  struct xe_device *xe_kunit_helper_alloc_xe_device(struct kunit *test,
>>  						  struct device *dev);
>>  int xe_kunit_helper_xe_device_test_init(struct kunit *test);
>> 

Re: [PATCH v2 3/8] drm/xe: Define and use MCR version of COMMON_SLICE_CHICKEN4

[Matt Roper]

On Fri, May 08, 2026 at 06:42:33PM -0300, Gustavo Sousa wrote:
> The register COMMON_SLICE_CHICKEN4 is a MCR register on both Xe2 and
> Xe3. Let's make sure to define a MCR version of it and use it for the
> relevant IP versions.
> 
> Use XEHP_ as prefix for the register name, since it is MCR as of Xe_HP.
> 
> v2:
>   - Also change for one entry in lrc_tunnings, which was caught by
>     manual testing and add corresponging Fixes tag in commit message.
>     (Gustavo)
> 
> Fixes: 8d6f16f1f082 ("drm/xe: Extend Wa_22021007897 to Xe3 platforms")
> Fixes: e5c13e2c505b ("drm/xe/xe2hpg: Add Wa_22021007897")
> Fixes: 8ccf5f6b2295 ("drm/xe/tuning: Apply windower hardware filtering setting on Xe3 and Xe3p")
> Bspec: 66534, 71185, 74417
> Reviewed-by: Matt Roper <matthew.d.roper@intel.com> # v1

Reviewed-by: Matt Roper <matthew.d.roper@intel.com>

for the update in v2 as well.

> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
> ---
>  drivers/gpu/drm/xe/regs/xe_gt_regs.h | 1 +
>  drivers/gpu/drm/xe/xe_tuning.c       | 2 +-
>  drivers/gpu/drm/xe/xe_wa.c           | 4 ++--
>  3 files changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/regs/xe_gt_regs.h b/drivers/gpu/drm/xe/regs/xe_gt_regs.h
> index b21c66a1b777..08251c7a1a4b 100644
> --- a/drivers/gpu/drm/xe/regs/xe_gt_regs.h
> +++ b/drivers/gpu/drm/xe/regs/xe_gt_regs.h
> @@ -179,6 +179,7 @@
>  #define XEHPG_SC_INSTDONE_EXTRA2		XE_REG_MCR(0x7108)
>  
>  #define COMMON_SLICE_CHICKEN4			XE_REG(0x7300, XE_REG_OPTION_MASKED)
> +#define XEHP_COMMON_SLICE_CHICKEN4		XE_REG_MCR(0x7300, XE_REG_OPTION_MASKED)
>  #define   SBE_PUSH_CONSTANT_BEHIND_FIX_ENABLE	REG_BIT(12)
>  #define   DISABLE_TDC_LOAD_BALANCING_CALC	REG_BIT(6)
>  #define   HW_FILTERING				REG_BIT(5)
> diff --git a/drivers/gpu/drm/xe/xe_tuning.c b/drivers/gpu/drm/xe/xe_tuning.c
> index ce39b77a084a..9a1b3862e192 100644
> --- a/drivers/gpu/drm/xe/xe_tuning.c
> +++ b/drivers/gpu/drm/xe/xe_tuning.c
> @@ -134,7 +134,7 @@ static const struct xe_rtp_entry_sr engine_tunings[] = {
>  static const struct xe_rtp_entry_sr lrc_tunings[] = {
>  	{ XE_RTP_NAME("Tuning: Windower HW Filtering"),
>  	  XE_RTP_RULES(GRAPHICS_VERSION_RANGE(3000, 3599), ENGINE_CLASS(RENDER)),
> -	  XE_RTP_ACTIONS(SET(COMMON_SLICE_CHICKEN4, HW_FILTERING))
> +	  XE_RTP_ACTIONS(SET(XEHP_COMMON_SLICE_CHICKEN4, HW_FILTERING))
>  	},
>  
>  	/* DG2 */
> diff --git a/drivers/gpu/drm/xe/xe_wa.c b/drivers/gpu/drm/xe/xe_wa.c
> index d6f94486673e..cb811f8a7781 100644
> --- a/drivers/gpu/drm/xe/xe_wa.c
> +++ b/drivers/gpu/drm/xe/xe_wa.c
> @@ -767,7 +767,7 @@ static const struct xe_rtp_entry_sr lrc_was[] = {
>  	},
>  	{ XE_RTP_NAME("22021007897"),
>  	  XE_RTP_RULES(GRAPHICS_VERSION_RANGE(2001, 2002), ENGINE_CLASS(RENDER)),
> -	  XE_RTP_ACTIONS(SET(COMMON_SLICE_CHICKEN4, SBE_PUSH_CONSTANT_BEHIND_FIX_ENABLE))
> +	  XE_RTP_ACTIONS(SET(XEHP_COMMON_SLICE_CHICKEN4, SBE_PUSH_CONSTANT_BEHIND_FIX_ENABLE))
>  	},
>  
>  	/* Xe3_LPG */
> @@ -783,7 +783,7 @@ static const struct xe_rtp_entry_sr lrc_was[] = {
>  	},
>  	{ XE_RTP_NAME("22021007897"),
>  	  XE_RTP_RULES(GRAPHICS_VERSION_RANGE(3000, 3005), ENGINE_CLASS(RENDER)),
> -	  XE_RTP_ACTIONS(SET(COMMON_SLICE_CHICKEN4, SBE_PUSH_CONSTANT_BEHIND_FIX_ENABLE))
> +	  XE_RTP_ACTIONS(SET(XEHP_COMMON_SLICE_CHICKEN4, SBE_PUSH_CONSTANT_BEHIND_FIX_ENABLE))
>  	},
>  	{ XE_RTP_NAME("14024681466"),
>  	  XE_RTP_RULES(GRAPHICS_VERSION_RANGE(3000, 3005), ENGINE_CLASS(RENDER)),
> 
> -- 
> 2.53.0
> 

-- 
Matt Roper
Graphics Software Engineer
Linux GPU Platform Enablement
Intel Corporation

Re: [PATCH v2 8/8] drm/xe/reg_sr: Do sanity check for MCR vs non-MCR

[Matt Roper]

On Fri, May 08, 2026 at 06:42:38PM -0300, Gustavo Sousa wrote:
> The type struct xe_reg_mcr exists to ensure that the correct API is used
> when handling MCR registers.  However, for the register save/restore
> functionality, the RTP processing always cast the register to a struct
> xe_reg and then apply_one_mmio() selects the MMIO API based on the "mcr"
> field of the register instance.
> 
> This allows the developer to commit mistakes like passing a MCR register
> for an RTP action for a GT where the respective register is not MCR; and
> vice-versa.
> 
> To capture such scenarios, do a sanity check in xe_reg_sr_add() that,
> upon an inconsistency:
> 
> - "fixes" the register type by favoring what we have in our MCR range
>   tables instead of what the developer selected for the save/restore
>   entry;
> - raises a notice-level message to inform about the inconsistency.
> 
> Note: As a collateral of this change, we need to include MCR
> initialization in xe_wa_test.c, otherwise a bunch of test cases end up
> failing because xe_gt_mcr_check_reg() will always return false, meaning
> that will incorrectly say that a MCR register is not MCR.
> 
> v2:
>   - Downgrade messages to notice level so as not to block CI execution
>     when inconsistencies are found. (Matt)
>   - Add missing EXPORT_SYMBOL_IF_KUNIT() calls. (Gustavo)
> 
> Cc: Matt Roper <matthew.d.roper@intel.com>
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>

Reviewed-by: Matt Roper <matthew.d.roper@intel.com>

> ---
>  drivers/gpu/drm/xe/tests/xe_rtp_test.c | 71 ++++++++++++++++++++++++++++++----
>  drivers/gpu/drm/xe/tests/xe_wa_test.c  | 12 +++++-
>  drivers/gpu/drm/xe/xe_gt.c             |  3 ++
>  drivers/gpu/drm/xe/xe_gt_mcr.c         | 24 ++++++++++++
>  drivers/gpu/drm/xe/xe_gt_mcr.h         |  1 +
>  drivers/gpu/drm/xe/xe_reg_sr.c         | 36 +++++++++++++++++
>  6 files changed, 139 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/tests/xe_rtp_test.c b/drivers/gpu/drm/xe/tests/xe_rtp_test.c
> index e5a0f985a700..5d78f2283df9 100644
> --- a/drivers/gpu/drm/xe/tests/xe_rtp_test.c
> +++ b/drivers/gpu/drm/xe/tests/xe_rtp_test.c
> @@ -9,24 +9,30 @@
>  #include <drm/drm_drv.h>
>  #include <drm/drm_kunit_helpers.h>
>  
> +#include <kunit/static_stub.h>
>  #include <kunit/test.h>
>  
>  #include "regs/xe_gt_regs.h"
>  #include "regs/xe_reg_defs.h"
>  #include "xe_device.h"
>  #include "xe_device_types.h"
> +#include "xe_gt_mcr.h"
>  #include "xe_kunit_helpers.h"
>  #include "xe_pci_test.h"
>  #include "xe_reg_sr.h"
>  #include "xe_rtp.h"
>  
> -#define REGULAR_REG1	XE_REG(1)
> -#define REGULAR_REG2	XE_REG(2)
> -#define REGULAR_REG3	XE_REG(3)
> -#define MCR_REG1	XE_REG_MCR(1)
> -#define MCR_REG2	XE_REG_MCR(2)
> -#define MCR_REG3	XE_REG_MCR(3)
> -#define MASKED_REG1	XE_REG(1, XE_REG_OPTION_MASKED)
> +#define REGULAR_REG1		XE_REG(1)
> +#define REGULAR_REG2		XE_REG(2)
> +#define REGULAR_REG3		XE_REG(3)
> +#define REGULAR_REG4		XE_REG(4)
> +#define BAD_REGULAR_REG5	XE_REG(5)
> +#define MCR_REG1		XE_REG_MCR(1)
> +#define MCR_REG2		XE_REG_MCR(2)
> +#define MCR_REG3		XE_REG_MCR(3)
> +#define BAD_MCR_REG4		XE_REG_MCR(4)
> +#define MCR_REG5		XE_REG_MCR(5)
> +#define MASKED_REG1		XE_REG(1, XE_REG_OPTION_MASKED)
>  
>  #undef XE_REG_MCR
>  #define XE_REG_MCR(...)     XE_REG(__VA_ARGS__, .mcr = 1)
> @@ -48,6 +54,23 @@ struct rtp_test_case {
>  	const struct xe_rtp_entry *entries;
>  };
>  
> +static bool fake_xe_gt_mcr_check_reg(struct xe_gt *gt, struct xe_reg reg)
> +{
> +	/*
> +	 * All supported platforms in this imaginary setup will always have REG4
> +	 * as a non-MCR register and REG5 as MCR, meaning that BAD_MCR_REG4 and
> +	 * BAD_REGULAR_REG5 represent programming errors to be captured by our
> +	 * tests.
> +	 */
> +	if (reg.raw == BAD_REGULAR_REG5.raw)
> +		return true;
> +
> +	if (reg.raw == BAD_MCR_REG4.raw)
> +		return false;
> +
> +	return reg.mcr;
> +}
> +
>  static bool match_yes(const struct xe_device *xe, const struct xe_gt *gt,
>  		      const struct xe_hw_engine *hwe)
>  {
> @@ -304,6 +327,38 @@ static const struct rtp_to_sr_test_case rtp_to_sr_cases[] = {
>  			{}
>  		},
>  	},
> +	{
> +		.name = "bad-mcr-reg-forced-to-regular",
> +		.expected_reg = REGULAR_REG4,
> +		.expected_set_bits = REG_BIT(0),
> +		.expected_clr_bits = REG_BIT(0),
> +		.expected_active = BIT(0),
> +		.expected_count_sr_entries = 1,
> +		.expected_sr_errors = 1,
> +		.entries = (const struct xe_rtp_entry_sr[]) {
> +			{ XE_RTP_NAME("bad-mcr-regular-reg"),
> +			  XE_RTP_RULES(FUNC(match_yes)),
> +			  XE_RTP_ACTIONS(SET(BAD_MCR_REG4, REG_BIT(0)))
> +			},
> +			{}
> +		},
> +	},
> +	{
> +		.name = "bad-regular-reg-forced-to-mcr",
> +		.expected_reg = MCR_REG5,
> +		.expected_set_bits = REG_BIT(0),
> +		.expected_clr_bits = REG_BIT(0),
> +		.expected_active = BIT(0),
> +		.expected_count_sr_entries = 1,
> +		.expected_sr_errors = 1,
> +		.entries = (const struct xe_rtp_entry_sr[]) {
> +			{ XE_RTP_NAME("bad-regular-reg"),
> +			  XE_RTP_RULES(FUNC(match_yes)),
> +			  XE_RTP_ACTIONS(SET(BAD_REGULAR_REG5, REG_BIT(0)))
> +			},
> +			{}
> +		},
> +	},
>  };
>  
>  static void xe_rtp_process_to_sr_tests(struct kunit *test)
> @@ -523,6 +578,8 @@ static int xe_rtp_test_init(struct kunit *test)
>  	xe->drm.dev = dev;
>  	test->priv = xe;
>  
> +	kunit_activate_static_stub(test, xe_gt_mcr_check_reg, fake_xe_gt_mcr_check_reg);
> +
>  	return 0;
>  }
>  
> diff --git a/drivers/gpu/drm/xe/tests/xe_wa_test.c b/drivers/gpu/drm/xe/tests/xe_wa_test.c
> index 2bf6fab015cd..ff0e2502b39f 100644
> --- a/drivers/gpu/drm/xe/tests/xe_wa_test.c
> +++ b/drivers/gpu/drm/xe/tests/xe_wa_test.c
> @@ -9,6 +9,8 @@
>  #include <kunit/test.h>
>  
>  #include "xe_device.h"
> +#include "xe_gt.h"
> +#include "xe_gt_mcr.h"
>  #include "xe_kunit_helpers.h"
>  #include "xe_pci_test.h"
>  #include "xe_reg_sr.h"
> @@ -19,8 +21,10 @@ static int xe_wa_test_init(struct kunit *test)
>  {
>  	const struct xe_pci_fake_data *param = test->param_value;
>  	struct xe_pci_fake_data data = *param;
> -	struct xe_device *xe;
>  	struct device *dev;
> +	struct xe_device *xe;
> +	struct xe_gt *gt;
> +	int id;
>  	int ret;
>  
>  	dev = drm_kunit_helper_alloc_device(test);
> @@ -33,6 +37,12 @@ static int xe_wa_test_init(struct kunit *test)
>  	ret = xe_pci_fake_device_init(xe);
>  	KUNIT_ASSERT_EQ(test, ret, 0);
>  
> +	/* Needed for sanitize_mcr(). */
> +	for_each_gt(gt, xe, id) {
> +		xe_gt_mcr_init_early(gt);
> +		xe_gt_mmio_init(gt);
> +	}
> +
>  	if (!param->graphics_verx100)
>  		xe->info.step = param->step;
>  
> diff --git a/drivers/gpu/drm/xe/xe_gt.c b/drivers/gpu/drm/xe/xe_gt.c
> index c4b25daad542..783eb6d631b5 100644
> --- a/drivers/gpu/drm/xe/xe_gt.c
> +++ b/drivers/gpu/drm/xe/xe_gt.c
> @@ -7,6 +7,8 @@
>  
>  #include <linux/minmax.h>
>  
> +#include <kunit/visibility.h>
> +
>  #include <drm/drm_managed.h>
>  #include <uapi/drm/xe_drm.h>
>  
> @@ -785,6 +787,7 @@ void xe_gt_mmio_init(struct xe_gt *gt)
>  	if (IS_SRIOV_VF(xe))
>  		gt->mmio.sriov_vf_gt = gt;
>  }
> +EXPORT_SYMBOL_IF_KUNIT(xe_gt_mmio_init);
>  
>  void xe_gt_record_user_engines(struct xe_gt *gt)
>  {
> diff --git a/drivers/gpu/drm/xe/xe_gt_mcr.c b/drivers/gpu/drm/xe/xe_gt_mcr.c
> index 2b2a4d9c3749..04f0098070a4 100644
> --- a/drivers/gpu/drm/xe/xe_gt_mcr.c
> +++ b/drivers/gpu/drm/xe/xe_gt_mcr.c
> @@ -3,6 +3,9 @@
>   * Copyright © 2022 Intel Corporation
>   */
>  
> +#include <kunit/static_stub.h>
> +#include <kunit/visibility.h>
> +
>  #include "xe_gt_mcr.h"
>  
>  #include "regs/xe_gt_regs.h"
> @@ -553,6 +556,7 @@ void xe_gt_mcr_init_early(struct xe_gt *gt)
>  	/* Mark instance 0 as initialized, we need this early for VRAM and CCS probe. */
>  	gt->steering[INSTANCE0].initialized = true;
>  }
> +EXPORT_SYMBOL_IF_KUNIT(xe_gt_mcr_init_early);
>  
>  /**
>   * xe_gt_mcr_init - Normal initialization of the MCR support
> @@ -614,6 +618,26 @@ static bool reg_in_steering_type_ranges(struct xe_gt *gt,
>  	return false;
>  }
>  
> +/*
> + * xe_gt_mcr_check_reg - check if a register is recognized by this GT as MCR
> + * @gt: GT structure
> + * @reg: The register to check
> + *
> + * Returns true if the register offset falls within one of the MMIO ranges
> + * classified as MCR for the GT.
> + */
> +bool xe_gt_mcr_check_reg(struct xe_gt *gt, struct xe_reg reg)
> +{
> +	KUNIT_STATIC_STUB_REDIRECT(xe_gt_mcr_check_reg, gt, reg);
> +
> +	for (int type = 0; type <= IMPLICIT_STEERING; type++)
> +		if (reg_in_steering_type_ranges(gt, reg, type))
> +			return true;
> +
> +	return false;
> +}
> +EXPORT_SYMBOL_IF_KUNIT(xe_gt_mcr_check_reg);
> +
>  /*
>   * xe_gt_mcr_get_nonterminated_steering - find group/instance values that
>   *    will steer a register to a non-terminated instance
> diff --git a/drivers/gpu/drm/xe/xe_gt_mcr.h b/drivers/gpu/drm/xe/xe_gt_mcr.h
> index 2be9419b8acc..75374662f10d 100644
> --- a/drivers/gpu/drm/xe/xe_gt_mcr.h
> +++ b/drivers/gpu/drm/xe/xe_gt_mcr.h
> @@ -26,6 +26,7 @@ void xe_gt_mcr_unicast_write(struct xe_gt *gt, struct xe_reg_mcr mcr_reg,
>  void xe_gt_mcr_multicast_write(struct xe_gt *gt, struct xe_reg_mcr mcr_reg,
>  			       u32 value);
>  
> +bool xe_gt_mcr_check_reg(struct xe_gt *gt, struct xe_reg reg);
>  bool xe_gt_mcr_get_nonterminated_steering(struct xe_gt *gt,
>  					  struct xe_reg_mcr reg_mcr,
>  					  u8 *group, u8 *instance);
> diff --git a/drivers/gpu/drm/xe/xe_reg_sr.c b/drivers/gpu/drm/xe/xe_reg_sr.c
> index 2df0277efb2f..e328f5072557 100644
> --- a/drivers/gpu/drm/xe/xe_reg_sr.c
> +++ b/drivers/gpu/drm/xe/xe_reg_sr.c
> @@ -70,14 +70,49 @@ static void reg_sr_inc_error(struct xe_reg_sr *sr)
>  #endif
>  }
>  
> +static struct xe_reg sanitize_mcr(struct xe_reg_sr *sr,
> +				  const struct xe_reg_sr_entry *e,
> +				  struct xe_gt *gt)
> +{
> +	struct xe_reg reg = e->reg;
> +	bool is_mcr;
> +
> +	/*
> +	 * We need the gt structure to check MCR ranges.
> +	 */
> +	if (!gt)
> +		return reg;
> +
> +	is_mcr = xe_gt_mcr_check_reg(gt, reg);
> +
> +	if (is_mcr && !reg.mcr) {
> +		reg.mcr = 1;
> +		xe_gt_notice(gt, "xe_reg_sr_entry using non-MCR register for address 0x%x, forcing MCR\n",
> +			     reg.addr);
> +		reg_sr_inc_error(sr);
> +	}
> +
> +	if (!is_mcr && reg.mcr) {
> +		reg.mcr = 0;
> +		xe_gt_notice(gt, "xe_reg_sr_entry using MCR register for address 0x%x, forcing non-MCR\n",
> +			     reg.addr);
> +		reg_sr_inc_error(sr);
> +	}
> +
> +	return reg;
> +}
> +
>  int xe_reg_sr_add(struct xe_reg_sr *sr,
>  		  const struct xe_reg_sr_entry *e,
>  		  struct xe_gt *gt)
>  {
>  	unsigned long idx = e->reg.addr;
>  	struct xe_reg_sr_entry *pentry = xa_load(&sr->xa, idx);
> +	struct xe_reg reg;
>  	int ret;
>  
> +	reg = sanitize_mcr(sr, e, gt);
> +
>  	if (pentry) {
>  		if (!compatible_entries(pentry, e)) {
>  			ret = -EINVAL;
> @@ -98,6 +133,7 @@ int xe_reg_sr_add(struct xe_reg_sr *sr,
>  	}
>  
>  	*pentry = *e;
> +	pentry->reg = reg;
>  	ret = xa_err(xa_store(&sr->xa, idx, pentry, GFP_KERNEL));
>  	if (ret)
>  		goto fail_free;
> 
> -- 
> 2.53.0
> 

-- 
Matt Roper
Graphics Software Engineer
Linux GPU Platform Enablement
Intel Corporation