* [PATCH 0/9] drm/xe: Probe info outside of xe_info_init() and xe_info_init_early()
@ 2026-06-09 20:17 Gustavo Sousa
2026-06-09 20:17 ` [PATCH 1/9] drm/xe: Add framework for info probing Gustavo Sousa
` (11 more replies)
0 siblings, 12 replies; 24+ messages in thread
From: Gustavo Sousa @ 2026-06-09 20:17 UTC (permalink / raw)
To: intel-xe; +Cc: Gustavo Sousa
Functions xe_info_init() and xe_info_init_early() are meant to
initialize fields of xe->info, but they also probe the hardware as
part of the process. This mixed responsibilities issue makes kunit
testing need to rely on static stubbing in order to initialize the
fake device info.
To make the design cleaner, let's do the necessary refactors so that
those functions don't try to poke the hardware and receive and info
that needs to be probed as parameters instead. The end result is that
functions xe_info_init() and xe_info_init_early() become better
reusable and we have a cleaner tests/xe_pci.c.
This will also be relevant for an upcoming series that is likely to
attempt to do fake device initialization outside of tests/xe_pci.c.
Nevertheless, even without it the refactor is arguably good in itself,
as observed by the end result of tests/xe_pci.c.
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
Gustavo Sousa (9):
drm/xe: Add framework for info probing
drm/xe/step: Pass xe_step_info to xe_step_*_get() functions
drm/xe: Add devid and revid to xe_probed_info
drm/xe/step: Make xe_step_platform_get() independent from xe->info
drm/xe: Add platform-level step info to xe_probed_info
drm/xe/tests: Set non-GMDID graphics step in xe_pci_fake_device_init()
drm/xe: Add graphics/media IPs and their step info to xe_probed_info
drm/xe: Don't initialize tile_count in xe_info_init_early()
drm/xe: Add tile_count to xe_probed_info
drivers/gpu/drm/xe/tests/xe_pci.c | 53 +++++------
drivers/gpu/drm/xe/tests/xe_wa_test.c | 3 -
drivers/gpu/drm/xe/xe_pci.c | 160 +++++++++++++++++++++++-----------
drivers/gpu/drm/xe/xe_step.c | 38 ++++----
drivers/gpu/drm/xe/xe_step.h | 8 +-
5 files changed, 164 insertions(+), 98 deletions(-)
---
base-commit: 3806de1666bcca0525d737e3849c602cbe98fb1a
change-id: 20260604-xe-probe-info-35f404748ce3
Best regards,
--
Gustavo Sousa <gustavo.sousa@intel.com>
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PATCH 1/9] drm/xe: Add framework for info probing
2026-06-09 20:17 [PATCH 0/9] drm/xe: Probe info outside of xe_info_init() and xe_info_init_early() Gustavo Sousa
@ 2026-06-09 20:17 ` Gustavo Sousa
2026-06-16 18:51 ` Violet Monti
2026-06-16 21:56 ` Matt Roper
2026-06-09 20:17 ` [PATCH 2/9] drm/xe/step: Pass xe_step_info to xe_step_*_get() functions Gustavo Sousa
` (10 subsequent siblings)
11 siblings, 2 replies; 24+ messages in thread
From: Gustavo Sousa @ 2026-06-09 20:17 UTC (permalink / raw)
To: intel-xe; +Cc: Gustavo Sousa
Functions xe_info_init_early() and xe_info_init() currently probe some
information from the hardware while doing initialization of info
fields. Besides mixing responsibilities, another issue from this
approach is that kunit tests need to implement static stubs for the
probing part.
Let's prepare the ground to ensuring that those functions stop probing
the information from the hardware by creating the necessary framework
for extracting the probing bits out of them. Do that by creating a
new struct type called xe_probed_info and the functions responsible
for populating it.
In upcoming changes, we will gradually refactor the code so that all
info needed by xe_info_init_early() and xe_info_init() that is probed
from the hardware is passed to them via struct xe_probed_info.
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
drivers/gpu/drm/xe/tests/xe_pci.c | 16 +++++++++++++--
drivers/gpu/drm/xe/xe_pci.c | 41 +++++++++++++++++++++++++++++++++++----
2 files changed, 51 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/xe/tests/xe_pci.c b/drivers/gpu/drm/xe/tests/xe_pci.c
index 9240aff779da..51d032a9e01a 100644
--- a/drivers/gpu/drm/xe/tests/xe_pci.c
+++ b/drivers/gpu/drm/xe/tests/xe_pci.c
@@ -338,13 +338,21 @@ static void fake_xe_info_probe_tile_count(struct xe_device *xe)
/* Nothing to do, just use the statically defined value. */
}
+static int fake_probe_info(struct xe_device *xe,
+ struct xe_probed_info *probed_info)
+{
+ return 0;
+}
+
int xe_pci_fake_device_init(struct xe_device *xe)
{
struct kunit *test = kunit_get_current_test();
struct xe_pci_fake_data *data = test->priv;
+ struct xe_probed_info probed_info = {};
const struct pci_device_id *ent = pciidlist;
const struct xe_device_desc *desc;
const struct xe_subplatform_desc *subplatform_desc;
+ int err;
if (!data) {
desc = (const void *)ent->driver_data;
@@ -379,8 +387,12 @@ int xe_pci_fake_device_init(struct xe_device *xe)
kunit_activate_static_stub(test, xe_info_probe_tile_count,
fake_xe_info_probe_tile_count);
- xe_info_init_early(xe, desc, subplatform_desc);
- xe_info_init(xe, desc);
+ err = fake_probe_info(xe, &probed_info);
+ if (err)
+ return err;
+
+ xe_info_init_early(xe, desc, subplatform_desc, &probed_info);
+ xe_info_init(xe, desc, &probed_info);
return 0;
}
diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
index 78fc2e4dcfc6..7f1da6d25011 100644
--- a/drivers/gpu/drm/xe/xe_pci.c
+++ b/drivers/gpu/drm/xe/xe_pci.c
@@ -737,13 +737,27 @@ static void init_devid(struct xe_device *xe)
xe->info.revid = pdev->revision;
}
+struct xe_probed_info {
+ /* Nothing for now. */
+};
+
+/*
+ * Probe from the hardware the info required by xe_info_init_early().
+ */
+static int xe_probe_info_early(struct xe_device *xe,
+ struct xe_probed_info *probed_info)
+{
+ return 0;
+}
+
/*
* Initialize device info content that only depends on static driver_data
* passed to the driver at probe time from PCI ID table.
*/
static int xe_info_init_early(struct xe_device *xe,
const struct xe_device_desc *desc,
- const struct xe_subplatform_desc *subplatform_desc)
+ const struct xe_subplatform_desc *subplatform_desc,
+ struct xe_probed_info *probed_info)
{
int err;
@@ -910,6 +924,15 @@ static struct xe_gt *alloc_media_gt(struct xe_tile *tile,
return gt;
}
+/*
+ * Probe from the hardware the info required by xe_info_init().
+ */
+static int xe_probe_info(struct xe_device *xe,
+ struct xe_probed_info *probed_info)
+{
+ return 0;
+}
+
/*
* Initialize device info content that does require knowledge about
* graphics / media IP version.
@@ -917,7 +940,8 @@ static struct xe_gt *alloc_media_gt(struct xe_tile *tile,
* present in device info.
*/
static int xe_info_init(struct xe_device *xe,
- const struct xe_device_desc *desc)
+ const struct xe_device_desc *desc,
+ struct xe_probed_info *probed_info)
{
u32 graphics_gmdid_revid = 0, media_gmdid_revid = 0;
const struct xe_ip *graphics_ip;
@@ -1073,6 +1097,7 @@ static void xe_pci_remove(struct pci_dev *pdev)
*/
static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
{
+ struct xe_probed_info probed_info = {};
const struct xe_device_desc *desc = (const void *)ent->driver_data;
const struct xe_subplatform_desc *subplatform_desc;
struct xe_device *xe;
@@ -1117,7 +1142,11 @@ static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
pci_set_master(pdev);
- err = xe_info_init_early(xe, desc, subplatform_desc);
+ err = xe_probe_info_early(xe, &probed_info);
+ if (err)
+ return err;
+
+ err = xe_info_init_early(xe, desc, subplatform_desc, &probed_info);
if (err)
return err;
@@ -1136,7 +1165,11 @@ static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
if (err)
return err;
- err = xe_info_init(xe, desc);
+ err = xe_probe_info(xe, &probed_info);
+ if (err)
+ return err;
+
+ err = xe_info_init(xe, desc, &probed_info);
if (err)
return err;
--
2.53.0
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 2/9] drm/xe/step: Pass xe_step_info to xe_step_*_get() functions
2026-06-09 20:17 [PATCH 0/9] drm/xe: Probe info outside of xe_info_init() and xe_info_init_early() Gustavo Sousa
2026-06-09 20:17 ` [PATCH 1/9] drm/xe: Add framework for info probing Gustavo Sousa
@ 2026-06-09 20:17 ` Gustavo Sousa
2026-06-16 19:31 ` Violet Monti
2026-06-09 20:17 ` [PATCH 3/9] drm/xe: Add devid and revid to xe_probed_info Gustavo Sousa
` (9 subsequent siblings)
11 siblings, 1 reply; 24+ messages in thread
From: Gustavo Sousa @ 2026-06-09 20:17 UTC (permalink / raw)
To: intel-xe; +Cc: Gustavo Sousa
The xe_step_*_get() functions update the step directly in
xe->info.step and are called by functions xe_info_init_early() and
xe_info_init().
As the stepping info is something probed from the hardware (via PCI
revid and/or GMDID) and we want to move away from probing inside
xe_info_init*() functions, let's make xe_step_*_get() functions modify
a pointer to the step structure instead of modifying xe->info.step
directly: this will allow an upcoming change that will move those
function calls out of the info init functions and will pass a member
of struct xe_probed_info instead of xe->info.step.
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
drivers/gpu/drm/xe/xe_pci.c | 6 +++---
drivers/gpu/drm/xe/xe_step.c | 33 ++++++++++++++++++++-------------
drivers/gpu/drm/xe/xe_step.h | 7 ++++---
3 files changed, 27 insertions(+), 19 deletions(-)
diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
index 7f1da6d25011..646e04c6254f 100644
--- a/drivers/gpu/drm/xe/xe_pci.c
+++ b/drivers/gpu/drm/xe/xe_pci.c
@@ -811,7 +811,7 @@ static int xe_info_init_early(struct xe_device *xe,
xe->info.max_gt_per_tile = desc->max_gt_per_tile;
xe->info.tile_count = 1 + desc->max_remote_tiles;
- xe_step_platform_get(xe);
+ xe_step_platform_get(xe, &xe->info.step);
err = xe_tile_init_early(xe_device_get_root_tile(xe), xe, 0);
if (err)
@@ -963,7 +963,7 @@ static int xe_info_init(struct xe_device *xe,
if (desc->pre_gmdid_graphics_ip) {
graphics_ip = desc->pre_gmdid_graphics_ip;
media_ip = desc->pre_gmdid_media_ip;
- xe_step_pre_gmdid_get(xe);
+ xe_step_pre_gmdid_get(xe, &xe->info.step);
} else {
xe_assert(xe, !desc->pre_gmdid_media_ip);
ret = handle_gmdid(xe, &graphics_ip, &media_ip,
@@ -971,7 +971,7 @@ static int xe_info_init(struct xe_device *xe,
if (ret)
return ret;
- xe_step_gmdid_get(xe, graphics_gmdid_revid, media_gmdid_revid);
+ xe_step_gmdid_get(xe, graphics_gmdid_revid, media_gmdid_revid, &xe->info.step);
}
/*
diff --git a/drivers/gpu/drm/xe/xe_step.c b/drivers/gpu/drm/xe/xe_step.c
index fb9c31613ca7..49dc64f2b363 100644
--- a/drivers/gpu/drm/xe/xe_step.c
+++ b/drivers/gpu/drm/xe/xe_step.c
@@ -111,11 +111,12 @@ __diag_pop();
/**
* xe_step_platform_get - Determine platform-level stepping from PCI revid
* @xe: Xe device
+ * @step: Pointer to the step struct to update
*
* Convert the PCI revid into a platform-level stepping value and store that
- * in the device info.
+ * in @step->platform.
*/
-void xe_step_platform_get(struct xe_device *xe)
+void xe_step_platform_get(struct xe_device *xe, struct xe_step_info *step)
{
/*
* Not all platforms map PCI revid directly into our symbolic stepping
@@ -127,17 +128,20 @@ void xe_step_platform_get(struct xe_device *xe)
*/
if (xe->info.platform == XE_NOVALAKE_P)
- xe->info.step.platform = STEP_A0 + xe->info.revid;
+ step->platform = STEP_A0 + xe->info.revid;
}
/**
* xe_step_pre_gmdid_get - Determine IP steppings from PCI revid
* @xe: Xe device
+ * @step: Pointer to the step struct to update
*
- * Convert the PCI revid into proper IP steppings. This should only be
- * used on platforms that do not have GMD_ID support.
+ * Convert the PCI revid into proper IP steppings and update @step->basedie,
+ * @step->graphics and @step->media accordingly.
+ *
+ * This should only be used on platforms that do not have GMD_ID support.
*/
-void xe_step_pre_gmdid_get(struct xe_device *xe)
+void xe_step_pre_gmdid_get(struct xe_device *xe, struct xe_step_info *step)
{
const struct xe_step_info *revids = NULL;
u16 revid = xe->info.revid;
@@ -234,9 +238,9 @@ void xe_step_pre_gmdid_get(struct xe_device *xe)
}
done:
- xe->info.step.graphics = graphics;
- xe->info.step.media = media;
- xe->info.step.basedie = basedie;
+ step->graphics = graphics;
+ step->media = media;
+ step->basedie = basedie;
}
/**
@@ -244,8 +248,10 @@ void xe_step_pre_gmdid_get(struct xe_device *xe)
* @xe: Xe device
* @graphics_gmdid_revid: value of graphics GMD_ID register's revid field
* @media_gmdid_revid: value of media GMD_ID register's revid field
+ * @step: Poninter to the step struct to update.
*
- * Convert the revid fields of the GMD_ID registers into proper IP steppings.
+ * Convert the revid fields of the GMD_ID registers into proper IP steppings
+ * and update @step->graphics and @step->media accordingly.
*
* GMD_ID revid values are currently expected to have consistent meanings on
* all platforms: major steppings (A0, B0, etc.) are 4 apart, with minor
@@ -253,7 +259,8 @@ void xe_step_pre_gmdid_get(struct xe_device *xe)
*/
void xe_step_gmdid_get(struct xe_device *xe,
u32 graphics_gmdid_revid,
- u32 media_gmdid_revid)
+ u32 media_gmdid_revid,
+ struct xe_step_info *step)
{
u8 graphics = STEP_A0 + graphics_gmdid_revid;
u8 media = STEP_A0 + media_gmdid_revid;
@@ -270,8 +277,8 @@ void xe_step_gmdid_get(struct xe_device *xe,
media_gmdid_revid);
}
- xe->info.step.graphics = graphics;
- xe->info.step.media = media;
+ step->graphics = graphics;
+ step->media = media;
}
#define STEP_NAME_CASE(name) \
diff --git a/drivers/gpu/drm/xe/xe_step.h b/drivers/gpu/drm/xe/xe_step.h
index ea36b22cc297..c6cea95a3727 100644
--- a/drivers/gpu/drm/xe/xe_step.h
+++ b/drivers/gpu/drm/xe/xe_step.h
@@ -12,12 +12,13 @@
struct xe_device;
-void xe_step_platform_get(struct xe_device *xe);
+void xe_step_platform_get(struct xe_device *xe, struct xe_step_info *step);
-void xe_step_pre_gmdid_get(struct xe_device *xe);
+void xe_step_pre_gmdid_get(struct xe_device *xe, struct xe_step_info *step);
void xe_step_gmdid_get(struct xe_device *xe,
u32 graphics_gmdid_revid,
- u32 media_gmdid_revid);
+ u32 media_gmdid_revid,
+ struct xe_step_info *step);
static inline u32 xe_step_to_gmdid(enum intel_step step) { return step - STEP_A0; }
const char *xe_step_name(enum intel_step step);
--
2.53.0
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 3/9] drm/xe: Add devid and revid to xe_probed_info
2026-06-09 20:17 [PATCH 0/9] drm/xe: Probe info outside of xe_info_init() and xe_info_init_early() Gustavo Sousa
2026-06-09 20:17 ` [PATCH 1/9] drm/xe: Add framework for info probing Gustavo Sousa
2026-06-09 20:17 ` [PATCH 2/9] drm/xe/step: Pass xe_step_info to xe_step_*_get() functions Gustavo Sousa
@ 2026-06-09 20:17 ` Gustavo Sousa
2026-06-16 19:33 ` Violet Monti
2026-06-09 20:17 ` [PATCH 4/9] drm/xe/step: Make xe_step_platform_get() independent from xe->info Gustavo Sousa
` (8 subsequent siblings)
11 siblings, 1 reply; 24+ messages in thread
From: Gustavo Sousa @ 2026-06-09 20:17 UTC (permalink / raw)
To: intel-xe; +Cc: Gustavo Sousa
The PCI devid and revid fields are info that we probe from the
hardware (indirectly via the PCI subsystem). Add them to
xe_probed_info and set them via xe_probe_info_early(), since the
respective fields in xe->info are updated in xe_info_init_early().
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
drivers/gpu/drm/xe/tests/xe_pci.c | 6 ------
drivers/gpu/drm/xe/xe_pci.c | 23 ++++++++++-------------
2 files changed, 10 insertions(+), 19 deletions(-)
diff --git a/drivers/gpu/drm/xe/tests/xe_pci.c b/drivers/gpu/drm/xe/tests/xe_pci.c
index 51d032a9e01a..1baf3cd0d381 100644
--- a/drivers/gpu/drm/xe/tests/xe_pci.c
+++ b/drivers/gpu/drm/xe/tests/xe_pci.c
@@ -311,11 +311,6 @@ const void *xe_pci_id_gen_param(struct kunit *test, const void *prev, char *desc
}
EXPORT_SYMBOL_IF_KUNIT(xe_pci_id_gen_param);
-static void fake_init_devid(struct xe_device *xe)
-{
- /* Nothing to do, just keep zero. */
-}
-
static int fake_read_gmdid(struct xe_device *xe, enum xe_gmdid_type type,
u32 *ver, u32 *revid)
{
@@ -382,7 +377,6 @@ int xe_pci_fake_device_init(struct xe_device *xe)
xe->sriov.__mode = data && data->sriov_mode ?
data->sriov_mode : XE_SRIOV_MODE_NONE;
- kunit_activate_static_stub(test, init_devid, fake_init_devid);
kunit_activate_static_stub(test, read_gmdid, fake_read_gmdid);
kunit_activate_static_stub(test, xe_info_probe_tile_count,
fake_xe_info_probe_tile_count);
diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
index 646e04c6254f..382df59260ae 100644
--- a/drivers/gpu/drm/xe/xe_pci.c
+++ b/drivers/gpu/drm/xe/xe_pci.c
@@ -727,18 +727,9 @@ static int handle_gmdid(struct xe_device *xe,
return 0;
}
-static void init_devid(struct xe_device *xe)
-{
- struct pci_dev *pdev = to_pci_dev(xe->drm.dev);
-
- KUNIT_STATIC_STUB_REDIRECT(init_devid, xe);
-
- xe->info.devid = pdev->device;
- xe->info.revid = pdev->revision;
-}
-
struct xe_probed_info {
- /* Nothing for now. */
+ u16 devid;
+ u8 revid;
};
/*
@@ -747,6 +738,11 @@ struct xe_probed_info {
static int xe_probe_info_early(struct xe_device *xe,
struct xe_probed_info *probed_info)
{
+ struct pci_dev *pdev = to_pci_dev(xe->drm.dev);
+
+ probed_info->devid = pdev->device;
+ probed_info->revid = pdev->revision;
+
return 0;
}
@@ -761,13 +757,14 @@ static int xe_info_init_early(struct xe_device *xe,
{
int err;
+ xe->info.devid = probed_info->devid;
+ xe->info.revid = probed_info->revid;
+
xe->info.platform_name = desc->platform_name;
xe->info.platform = desc->platform;
xe->info.subplatform = subplatform_desc ?
subplatform_desc->subplatform : XE_SUBPLATFORM_NONE;
- init_devid(xe);
-
xe->info.dma_mask_size = desc->dma_mask_size;
xe->info.va_bits = desc->va_bits;
xe->info.vm_max_level = desc->vm_max_level;
--
2.53.0
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 4/9] drm/xe/step: Make xe_step_platform_get() independent from xe->info
2026-06-09 20:17 [PATCH 0/9] drm/xe: Probe info outside of xe_info_init() and xe_info_init_early() Gustavo Sousa
` (2 preceding siblings ...)
2026-06-09 20:17 ` [PATCH 3/9] drm/xe: Add devid and revid to xe_probed_info Gustavo Sousa
@ 2026-06-09 20:17 ` Gustavo Sousa
2026-06-16 19:37 ` Violet Monti
2026-06-09 20:17 ` [PATCH 5/9] drm/xe: Add platform-level step info to xe_probed_info Gustavo Sousa
` (7 subsequent siblings)
11 siblings, 1 reply; 24+ messages in thread
From: Gustavo Sousa @ 2026-06-09 20:17 UTC (permalink / raw)
To: intel-xe; +Cc: Gustavo Sousa
Currently xe_step_platform_get() uses info fields from xe->info to
define the platform-level stepping value.
Because the platform-level stepping info depends on the PCI revid, it
should be defined as part of xe_probe_info_early() instead of being
directly probed inside xe_info_init_early().
Let's make sure that xe_step_platform_get() receives the necessary
data as parameters and does not depend on xe->info. That will allow
us to move the call up to xe_probe_info_early() in an upcoming change.
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
drivers/gpu/drm/xe/xe_pci.c | 2 +-
drivers/gpu/drm/xe/xe_step.c | 9 +++++----
drivers/gpu/drm/xe/xe_step.h | 3 ++-
3 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
index 382df59260ae..c5c75fbfa0e2 100644
--- a/drivers/gpu/drm/xe/xe_pci.c
+++ b/drivers/gpu/drm/xe/xe_pci.c
@@ -808,7 +808,7 @@ static int xe_info_init_early(struct xe_device *xe,
xe->info.max_gt_per_tile = desc->max_gt_per_tile;
xe->info.tile_count = 1 + desc->max_remote_tiles;
- xe_step_platform_get(xe, &xe->info.step);
+ xe_step_platform_get(xe->info.platform, xe->info.revid, &xe->info.step);
err = xe_tile_init_early(xe_device_get_root_tile(xe), xe, 0);
if (err)
diff --git a/drivers/gpu/drm/xe/xe_step.c b/drivers/gpu/drm/xe/xe_step.c
index 49dc64f2b363..55c1996f689e 100644
--- a/drivers/gpu/drm/xe/xe_step.c
+++ b/drivers/gpu/drm/xe/xe_step.c
@@ -110,13 +110,14 @@ __diag_pop();
/**
* xe_step_platform_get - Determine platform-level stepping from PCI revid
- * @xe: Xe device
+ * @platform: The Xe platform
+ * @revid: The PCI revid
* @step: Pointer to the step struct to update
*
* Convert the PCI revid into a platform-level stepping value and store that
* in @step->platform.
*/
-void xe_step_platform_get(struct xe_device *xe, struct xe_step_info *step)
+void xe_step_platform_get(enum xe_platform platform, u8 revid, struct xe_step_info *step)
{
/*
* Not all platforms map PCI revid directly into our symbolic stepping
@@ -127,8 +128,8 @@ void xe_step_platform_get(struct xe_device *xe, struct xe_step_info *step)
* checks.
*/
- if (xe->info.platform == XE_NOVALAKE_P)
- step->platform = STEP_A0 + xe->info.revid;
+ if (platform == XE_NOVALAKE_P)
+ step->platform = STEP_A0 + revid;
}
/**
diff --git a/drivers/gpu/drm/xe/xe_step.h b/drivers/gpu/drm/xe/xe_step.h
index c6cea95a3727..5a5845335740 100644
--- a/drivers/gpu/drm/xe/xe_step.h
+++ b/drivers/gpu/drm/xe/xe_step.h
@@ -10,9 +10,10 @@
#include "xe_step_types.h"
+enum xe_platform;
struct xe_device;
-void xe_step_platform_get(struct xe_device *xe, struct xe_step_info *step);
+void xe_step_platform_get(enum xe_platform platform, u8 revid, struct xe_step_info *step);
void xe_step_pre_gmdid_get(struct xe_device *xe, struct xe_step_info *step);
void xe_step_gmdid_get(struct xe_device *xe,
--
2.53.0
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 5/9] drm/xe: Add platform-level step info to xe_probed_info
2026-06-09 20:17 [PATCH 0/9] drm/xe: Probe info outside of xe_info_init() and xe_info_init_early() Gustavo Sousa
` (3 preceding siblings ...)
2026-06-09 20:17 ` [PATCH 4/9] drm/xe/step: Make xe_step_platform_get() independent from xe->info Gustavo Sousa
@ 2026-06-09 20:17 ` Gustavo Sousa
2026-06-16 19:50 ` Violet Monti
2026-06-09 20:17 ` [PATCH 6/9] drm/xe/tests: Set non-GMDID graphics step in xe_pci_fake_device_init() Gustavo Sousa
` (6 subsequent siblings)
11 siblings, 1 reply; 24+ messages in thread
From: Gustavo Sousa @ 2026-06-09 20:17 UTC (permalink / raw)
To: intel-xe; +Cc: Gustavo Sousa
The platform-level step information depends on the PCI revid and, as
such, should be probed in xe_probe_info_early() instead of
xe_info_init_early(). Move the code accordingly.
Note that we currently only update probed_info->step.platform as part
of this change. We will deal with the other fields of
probed_info->step as a follow-up change, which will be tied to the
probing of graphics and media IPs.
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
drivers/gpu/drm/xe/xe_pci.c | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
index c5c75fbfa0e2..1c53a25442a9 100644
--- a/drivers/gpu/drm/xe/xe_pci.c
+++ b/drivers/gpu/drm/xe/xe_pci.c
@@ -730,12 +730,14 @@ static int handle_gmdid(struct xe_device *xe,
struct xe_probed_info {
u16 devid;
u8 revid;
+ struct xe_step_info step;
};
/*
* Probe from the hardware the info required by xe_info_init_early().
*/
static int xe_probe_info_early(struct xe_device *xe,
+ const struct xe_device_desc *desc,
struct xe_probed_info *probed_info)
{
struct pci_dev *pdev = to_pci_dev(xe->drm.dev);
@@ -743,6 +745,8 @@ static int xe_probe_info_early(struct xe_device *xe,
probed_info->devid = pdev->device;
probed_info->revid = pdev->revision;
+ xe_step_platform_get(desc->platform, probed_info->revid, &probed_info->step);
+
return 0;
}
@@ -759,6 +763,7 @@ static int xe_info_init_early(struct xe_device *xe,
xe->info.devid = probed_info->devid;
xe->info.revid = probed_info->revid;
+ xe->info.step.platform = probed_info->step.platform;
xe->info.platform_name = desc->platform_name;
xe->info.platform = desc->platform;
@@ -808,8 +813,6 @@ static int xe_info_init_early(struct xe_device *xe,
xe->info.max_gt_per_tile = desc->max_gt_per_tile;
xe->info.tile_count = 1 + desc->max_remote_tiles;
- xe_step_platform_get(xe->info.platform, xe->info.revid, &xe->info.step);
-
err = xe_tile_init_early(xe_device_get_root_tile(xe), xe, 0);
if (err)
return err;
@@ -1139,7 +1142,7 @@ static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
pci_set_master(pdev);
- err = xe_probe_info_early(xe, &probed_info);
+ err = xe_probe_info_early(xe, desc, &probed_info);
if (err)
return err;
--
2.53.0
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 6/9] drm/xe/tests: Set non-GMDID graphics step in xe_pci_fake_device_init()
2026-06-09 20:17 [PATCH 0/9] drm/xe: Probe info outside of xe_info_init() and xe_info_init_early() Gustavo Sousa
` (4 preceding siblings ...)
2026-06-09 20:17 ` [PATCH 5/9] drm/xe: Add platform-level step info to xe_probed_info Gustavo Sousa
@ 2026-06-09 20:17 ` Gustavo Sousa
2026-06-16 19:58 ` Violet Monti
2026-06-09 20:17 ` [PATCH 7/9] drm/xe: Add graphics/media IPs and their step info to xe_probed_info Gustavo Sousa
` (5 subsequent siblings)
11 siblings, 1 reply; 24+ messages in thread
From: Gustavo Sousa @ 2026-06-09 20:17 UTC (permalink / raw)
To: intel-xe; +Cc: Gustavo Sousa
Currently the logic to set the graphics step for non-GMDID-based
platforms in kunit testing is defined in xe_wa_test_init(). That
logic should rather belong to the helper xe_pci_fake_device_init(), so
move it there.
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
drivers/gpu/drm/xe/tests/xe_pci.c | 3 +++
drivers/gpu/drm/xe/tests/xe_wa_test.c | 3 ---
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/xe/tests/xe_pci.c b/drivers/gpu/drm/xe/tests/xe_pci.c
index 1baf3cd0d381..a665d5dbc472 100644
--- a/drivers/gpu/drm/xe/tests/xe_pci.c
+++ b/drivers/gpu/drm/xe/tests/xe_pci.c
@@ -388,6 +388,9 @@ int xe_pci_fake_device_init(struct xe_device *xe)
xe_info_init_early(xe, desc, subplatform_desc, &probed_info);
xe_info_init(xe, desc, &probed_info);
+ if (data && !data->graphics_verx100)
+ xe->info.step = data->step;
+
return 0;
}
EXPORT_SYMBOL_IF_KUNIT(xe_pci_fake_device_init);
diff --git a/drivers/gpu/drm/xe/tests/xe_wa_test.c b/drivers/gpu/drm/xe/tests/xe_wa_test.c
index ff0e2502b39f..21601e9df353 100644
--- a/drivers/gpu/drm/xe/tests/xe_wa_test.c
+++ b/drivers/gpu/drm/xe/tests/xe_wa_test.c
@@ -43,9 +43,6 @@ static int xe_wa_test_init(struct kunit *test)
xe_gt_mmio_init(gt);
}
- if (!param->graphics_verx100)
- xe->info.step = param->step;
-
/* TODO: init hw engines for engine/LRC WAs */
xe->drm.dev = dev;
test->priv = xe;
--
2.53.0
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 7/9] drm/xe: Add graphics/media IPs and their step info to xe_probed_info
2026-06-09 20:17 [PATCH 0/9] drm/xe: Probe info outside of xe_info_init() and xe_info_init_early() Gustavo Sousa
` (5 preceding siblings ...)
2026-06-09 20:17 ` [PATCH 6/9] drm/xe/tests: Set non-GMDID graphics step in xe_pci_fake_device_init() Gustavo Sousa
@ 2026-06-09 20:17 ` Gustavo Sousa
2026-06-16 20:22 ` Violet Monti
2026-06-09 20:17 ` [PATCH 8/9] drm/xe: Don't initialize tile_count in xe_info_init_early() Gustavo Sousa
` (4 subsequent siblings)
11 siblings, 1 reply; 24+ messages in thread
From: Gustavo Sousa @ 2026-06-09 20:17 UTC (permalink / raw)
To: intel-xe; +Cc: Gustavo Sousa
On GMDID-based platforms, the driver needs to probe the hardware by
reading GMDID registers in order to identify the
graphics/media/display IPs that are present in the platform as well as
their stepping values.
Currently, xe_info_init() has such a probing logic, but that task
should be rather responsibility of xe_probe_info(). As such, move it
to the latter.
For pre-GMDID platforms, the IPs are identified via PCI devid and
revid fields, which is arguably also hardware dependent. So do the
same for those platforms.
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
drivers/gpu/drm/xe/tests/xe_pci.c | 44 ++++++++++----------
drivers/gpu/drm/xe/xe_pci.c | 88 ++++++++++++++++++++++++---------------
2 files changed, 77 insertions(+), 55 deletions(-)
diff --git a/drivers/gpu/drm/xe/tests/xe_pci.c b/drivers/gpu/drm/xe/tests/xe_pci.c
index a665d5dbc472..cd64b1d614c8 100644
--- a/drivers/gpu/drm/xe/tests/xe_pci.c
+++ b/drivers/gpu/drm/xe/tests/xe_pci.c
@@ -311,31 +311,35 @@ const void *xe_pci_id_gen_param(struct kunit *test, const void *prev, char *desc
}
EXPORT_SYMBOL_IF_KUNIT(xe_pci_id_gen_param);
-static int fake_read_gmdid(struct xe_device *xe, enum xe_gmdid_type type,
- u32 *ver, u32 *revid)
-{
- struct kunit *test = kunit_get_current_test();
- struct xe_pci_fake_data *data = test->priv;
-
- if (type == GMDID_MEDIA) {
- *ver = data->media_verx100;
- *revid = xe_step_to_gmdid(data->step.media);
- } else {
- *ver = data->graphics_verx100;
- *revid = xe_step_to_gmdid(data->step.graphics);
- }
-
- return 0;
-}
-
static void fake_xe_info_probe_tile_count(struct xe_device *xe)
{
/* Nothing to do, just use the statically defined value. */
}
static int fake_probe_info(struct xe_device *xe,
+ const struct xe_device_desc *desc,
+ struct xe_pci_fake_data *data,
struct xe_probed_info *probed_info)
{
+ if (!data || desc->pre_gmdid_graphics_ip) {
+ probed_info->graphics_ip = desc->pre_gmdid_graphics_ip;
+ probed_info->media_ip = desc->pre_gmdid_media_ip;
+ } else {
+ probed_info->graphics_ip = find_graphics_ip(data->graphics_verx100);
+
+ if (data->media_verx100) {
+ probed_info->media_ip = find_media_ip(data->media_verx100);
+ xe_assert(xe, probed_info->media_ip);
+ }
+ }
+
+ xe_assert(xe, probed_info->graphics_ip);
+ if (!probed_info->graphics_ip)
+ return -ENODEV;
+
+ if (data)
+ probed_info->step = data->step;
+
return 0;
}
@@ -377,20 +381,16 @@ int xe_pci_fake_device_init(struct xe_device *xe)
xe->sriov.__mode = data && data->sriov_mode ?
data->sriov_mode : XE_SRIOV_MODE_NONE;
- kunit_activate_static_stub(test, read_gmdid, fake_read_gmdid);
kunit_activate_static_stub(test, xe_info_probe_tile_count,
fake_xe_info_probe_tile_count);
- err = fake_probe_info(xe, &probed_info);
+ err = fake_probe_info(xe, desc, data, &probed_info);
if (err)
return err;
xe_info_init_early(xe, desc, subplatform_desc, &probed_info);
xe_info_init(xe, desc, &probed_info);
- if (data && !data->graphics_verx100)
- xe->info.step = data->step;
-
return 0;
}
EXPORT_SYMBOL_IF_KUNIT(xe_pci_fake_device_init);
diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
index 1c53a25442a9..388771ef6a52 100644
--- a/drivers/gpu/drm/xe/xe_pci.c
+++ b/drivers/gpu/drm/xe/xe_pci.c
@@ -600,8 +600,6 @@ static int read_gmdid(struct xe_device *xe, enum xe_gmdid_type type, u32 *ver, u
struct xe_reg gmdid_reg = GMD_ID;
u32 val;
- KUNIT_STATIC_STUB_REDIRECT(read_gmdid, xe, type, ver, revid);
-
if (IS_SRIOV_VF(xe)) {
/*
* To get the value of the GMDID register, VFs must obtain it
@@ -731,6 +729,8 @@ struct xe_probed_info {
u16 devid;
u8 revid;
struct xe_step_info step;
+ const struct xe_ip *graphics_ip;
+ const struct xe_ip *media_ip;
};
/*
@@ -924,12 +924,59 @@ static struct xe_gt *alloc_media_gt(struct xe_tile *tile,
return gt;
}
+static int xe_probe_ips(struct xe_device *xe,
+ const struct xe_device_desc *desc,
+ struct xe_probed_info *probed_info)
+{
+ /*
+ * If this platform supports GMD_ID, we'll detect the proper IP
+ * descriptor to use from hardware registers.
+ * desc->pre_gmdid_graphics_ip will only ever be set at this point for
+ * platforms before GMD_ID. In that case the IP descriptions and
+ * versions are simply derived from that.
+ */
+ if (desc->pre_gmdid_graphics_ip) {
+ probed_info->graphics_ip = desc->pre_gmdid_graphics_ip;
+ probed_info->media_ip = desc->pre_gmdid_media_ip;
+ xe_step_pre_gmdid_get(xe, &probed_info->step);
+ } else {
+ int err;
+ u32 graphics_revid, media_revid;
+
+ xe_assert(xe, !desc->pre_gmdid_media_ip);
+
+ err = handle_gmdid(xe, &probed_info->graphics_ip, &probed_info->media_ip,
+ &graphics_revid, &media_revid);
+ if (err)
+ return err;
+
+ xe_step_gmdid_get(xe, graphics_revid, media_revid, &probed_info->step);
+ }
+
+ /*
+ * If we couldn't detect the graphics IP, that's considered a fatal
+ * error and we should abort driver load. Failing to detect media
+ * IP is non-fatal; we'll just proceed without enabling media support.
+ */
+ if (!probed_info->graphics_ip)
+ return -ENODEV;
+
+ return 0;
+}
+
/*
* Probe from the hardware the info required by xe_info_init().
*/
static int xe_probe_info(struct xe_device *xe,
+ const struct xe_device_desc *desc,
struct xe_probed_info *probed_info)
{
+ int err;
+
+ err = xe_probe_ips(xe, desc, probed_info);
+ if (err)
+ return err;
+
return 0;
}
@@ -943,44 +990,19 @@ static int xe_info_init(struct xe_device *xe,
const struct xe_device_desc *desc,
struct xe_probed_info *probed_info)
{
- u32 graphics_gmdid_revid = 0, media_gmdid_revid = 0;
const struct xe_ip *graphics_ip;
const struct xe_ip *media_ip;
const struct xe_graphics_desc *graphics_desc;
const struct xe_media_desc *media_desc;
struct xe_tile *tile;
struct xe_gt *gt;
- int ret;
u8 id;
- /*
- * If this platform supports GMD_ID, we'll detect the proper IP
- * descriptor to use from hardware registers.
- * desc->pre_gmdid_graphics_ip will only ever be set at this point for
- * platforms before GMD_ID. In that case the IP descriptions and
- * versions are simply derived from that.
- */
- if (desc->pre_gmdid_graphics_ip) {
- graphics_ip = desc->pre_gmdid_graphics_ip;
- media_ip = desc->pre_gmdid_media_ip;
- xe_step_pre_gmdid_get(xe, &xe->info.step);
- } else {
- xe_assert(xe, !desc->pre_gmdid_media_ip);
- ret = handle_gmdid(xe, &graphics_ip, &media_ip,
- &graphics_gmdid_revid, &media_gmdid_revid);
- if (ret)
- return ret;
-
- xe_step_gmdid_get(xe, graphics_gmdid_revid, media_gmdid_revid, &xe->info.step);
- }
-
- /*
- * If we couldn't detect the graphics IP, that's considered a fatal
- * error and we should abort driver load. Failing to detect media
- * IP is non-fatal; we'll just proceed without enabling media support.
- */
- if (!graphics_ip)
- return -ENODEV;
+ graphics_ip = probed_info->graphics_ip;
+ media_ip = probed_info->media_ip;
+ xe->info.step.basedie = probed_info->step.basedie;
+ xe->info.step.graphics = probed_info->step.graphics;
+ xe->info.step.media = probed_info->step.media;
xe->info.graphics_verx100 = graphics_ip->verx100;
xe->info.graphics_name = graphics_ip->name;
@@ -1165,7 +1187,7 @@ static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
if (err)
return err;
- err = xe_probe_info(xe, &probed_info);
+ err = xe_probe_info(xe, desc, &probed_info);
if (err)
return err;
--
2.53.0
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 8/9] drm/xe: Don't initialize tile_count in xe_info_init_early()
2026-06-09 20:17 [PATCH 0/9] drm/xe: Probe info outside of xe_info_init() and xe_info_init_early() Gustavo Sousa
` (6 preceding siblings ...)
2026-06-09 20:17 ` [PATCH 7/9] drm/xe: Add graphics/media IPs and their step info to xe_probed_info Gustavo Sousa
@ 2026-06-09 20:17 ` Gustavo Sousa
2026-06-16 21:07 ` Violet Monti
2026-06-09 20:17 ` [PATCH 9/9] drm/xe: Add tile_count to xe_probed_info Gustavo Sousa
` (3 subsequent siblings)
11 siblings, 1 reply; 24+ messages in thread
From: Gustavo Sousa @ 2026-06-09 20:17 UTC (permalink / raw)
To: intel-xe; +Cc: Gustavo Sousa
The value of xe->info.tile_count is only really valid after
xe_info_probe_tile_count(). Any use of tile_count before that point
is invalid and, consequently, initializing it in xe_info_init_early()
is pointless.
Move the initialization to xe_info_probe_tile_count().
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
drivers/gpu/drm/xe/tests/xe_pci.c | 5 +++--
drivers/gpu/drm/xe/xe_pci.c | 10 ++++++----
2 files changed, 9 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/xe/tests/xe_pci.c b/drivers/gpu/drm/xe/tests/xe_pci.c
index cd64b1d614c8..31ec41aa997d 100644
--- a/drivers/gpu/drm/xe/tests/xe_pci.c
+++ b/drivers/gpu/drm/xe/tests/xe_pci.c
@@ -311,9 +311,10 @@ const void *xe_pci_id_gen_param(struct kunit *test, const void *prev, char *desc
}
EXPORT_SYMBOL_IF_KUNIT(xe_pci_id_gen_param);
-static void fake_xe_info_probe_tile_count(struct xe_device *xe)
+static void fake_xe_info_probe_tile_count(struct xe_device *xe,
+ const struct xe_device_desc *desc)
{
- /* Nothing to do, just use the statically defined value. */
+ xe->info.tile_count = 1 + desc->max_remote_tiles;
}
static int fake_probe_info(struct xe_device *xe,
diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
index 388771ef6a52..6b2daad87316 100644
--- a/drivers/gpu/drm/xe/xe_pci.c
+++ b/drivers/gpu/drm/xe/xe_pci.c
@@ -811,7 +811,6 @@ static int xe_info_init_early(struct xe_device *xe,
xe_assert(xe, desc->max_gt_per_tile > 0);
xe_assert(xe, desc->max_gt_per_tile <= XE_MAX_GT_PER_TILE);
xe->info.max_gt_per_tile = desc->max_gt_per_tile;
- xe->info.tile_count = 1 + desc->max_remote_tiles;
err = xe_tile_init_early(xe_device_get_root_tile(xe), xe, 0);
if (err)
@@ -823,13 +822,16 @@ static int xe_info_init_early(struct xe_device *xe,
/*
* Possibly override number of tile based on configuration register.
*/
-static void xe_info_probe_tile_count(struct xe_device *xe)
+static void xe_info_probe_tile_count(struct xe_device *xe,
+ const struct xe_device_desc *desc)
{
struct xe_mmio *mmio;
u8 tile_count;
u32 mtcfg;
- KUNIT_STATIC_STUB_REDIRECT(xe_info_probe_tile_count, xe);
+ KUNIT_STATIC_STUB_REDIRECT(xe_info_probe_tile_count, xe, desc);
+
+ xe->info.tile_count = 1 + desc->max_remote_tiles;
/*
* Probe for tile count only for platforms that support multiple
@@ -1035,7 +1037,7 @@ static int xe_info_init(struct xe_device *xe,
xe->info.has_soc_remapper_telem = 0;
}
- xe_info_probe_tile_count(xe);
+ xe_info_probe_tile_count(xe, desc);
for_each_remote_tile(tile, xe, id) {
int err;
--
2.53.0
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 9/9] drm/xe: Add tile_count to xe_probed_info
2026-06-09 20:17 [PATCH 0/9] drm/xe: Probe info outside of xe_info_init() and xe_info_init_early() Gustavo Sousa
` (7 preceding siblings ...)
2026-06-09 20:17 ` [PATCH 8/9] drm/xe: Don't initialize tile_count in xe_info_init_early() Gustavo Sousa
@ 2026-06-09 20:17 ` Gustavo Sousa
2026-06-16 21:16 ` Violet Monti
2026-06-09 20:58 ` ✓ CI.KUnit: success for drm/xe: Probe info outside of xe_info_init() and xe_info_init_early() Patchwork
` (2 subsequent siblings)
11 siblings, 1 reply; 24+ messages in thread
From: Gustavo Sousa @ 2026-06-09 20:17 UTC (permalink / raw)
To: intel-xe; +Cc: Gustavo Sousa
On multi-tile platforms, we need to probe the hardware for the number
of tiles that are present in the platform. That means that we should
do that as part of xe_probe_info() instead of xe_info_init(). Do
that.
Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
---
drivers/gpu/drm/xe/tests/xe_pci.c | 11 ++---------
drivers/gpu/drm/xe/xe_pci.c | 27 +++++++++++++--------------
2 files changed, 15 insertions(+), 23 deletions(-)
diff --git a/drivers/gpu/drm/xe/tests/xe_pci.c b/drivers/gpu/drm/xe/tests/xe_pci.c
index 31ec41aa997d..8df9029afcd3 100644
--- a/drivers/gpu/drm/xe/tests/xe_pci.c
+++ b/drivers/gpu/drm/xe/tests/xe_pci.c
@@ -311,17 +311,13 @@ const void *xe_pci_id_gen_param(struct kunit *test, const void *prev, char *desc
}
EXPORT_SYMBOL_IF_KUNIT(xe_pci_id_gen_param);
-static void fake_xe_info_probe_tile_count(struct xe_device *xe,
- const struct xe_device_desc *desc)
-{
- xe->info.tile_count = 1 + desc->max_remote_tiles;
-}
-
static int fake_probe_info(struct xe_device *xe,
const struct xe_device_desc *desc,
struct xe_pci_fake_data *data,
struct xe_probed_info *probed_info)
{
+ probed_info->tile_count = 1 + desc->max_remote_tiles;
+
if (!data || desc->pre_gmdid_graphics_ip) {
probed_info->graphics_ip = desc->pre_gmdid_graphics_ip;
probed_info->media_ip = desc->pre_gmdid_media_ip;
@@ -382,9 +378,6 @@ int xe_pci_fake_device_init(struct xe_device *xe)
xe->sriov.__mode = data && data->sriov_mode ?
data->sriov_mode : XE_SRIOV_MODE_NONE;
- kunit_activate_static_stub(test, xe_info_probe_tile_count,
- fake_xe_info_probe_tile_count);
-
err = fake_probe_info(xe, desc, data, &probed_info);
if (err)
return err;
diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
index 6b2daad87316..1b5df4384689 100644
--- a/drivers/gpu/drm/xe/xe_pci.c
+++ b/drivers/gpu/drm/xe/xe_pci.c
@@ -728,6 +728,7 @@ static int handle_gmdid(struct xe_device *xe,
struct xe_probed_info {
u16 devid;
u8 revid;
+ u8 tile_count;
struct xe_step_info step;
const struct xe_ip *graphics_ip;
const struct xe_ip *media_ip;
@@ -819,25 +820,21 @@ static int xe_info_init_early(struct xe_device *xe,
return 0;
}
-/*
- * Possibly override number of tile based on configuration register.
- */
-static void xe_info_probe_tile_count(struct xe_device *xe,
- const struct xe_device_desc *desc)
+static void xe_probe_tile_count(struct xe_device *xe,
+ const struct xe_device_desc *desc,
+ struct xe_probed_info *probed_info)
{
struct xe_mmio *mmio;
u8 tile_count;
u32 mtcfg;
- KUNIT_STATIC_STUB_REDIRECT(xe_info_probe_tile_count, xe, desc);
-
- xe->info.tile_count = 1 + desc->max_remote_tiles;
+ probed_info->tile_count = 1 + desc->max_remote_tiles;
/*
* Probe for tile count only for platforms that support multiple
* tiles.
*/
- if (xe->info.tile_count == 1)
+ if (probed_info->tile_count == 1)
return;
mmio = xe_root_tile_mmio(xe);
@@ -850,10 +847,10 @@ static void xe_info_probe_tile_count(struct xe_device *xe,
mtcfg = xe_mmio_read32(mmio, XEHP_MTCFG_ADDR);
tile_count = REG_FIELD_GET(TILE_COUNT, mtcfg) + 1;
- if (tile_count < xe->info.tile_count) {
+ if (tile_count < probed_info->tile_count) {
drm_info(&xe->drm, "tile_count: %d, reduced_tile_count %d\n",
- xe->info.tile_count, tile_count);
- xe->info.tile_count = tile_count;
+ probed_info->tile_count, tile_count);
+ probed_info->tile_count = tile_count;
}
}
@@ -975,6 +972,8 @@ static int xe_probe_info(struct xe_device *xe,
{
int err;
+ xe_probe_tile_count(xe, desc, probed_info);
+
err = xe_probe_ips(xe, desc, probed_info);
if (err)
return err;
@@ -1002,6 +1001,8 @@ static int xe_info_init(struct xe_device *xe,
graphics_ip = probed_info->graphics_ip;
media_ip = probed_info->media_ip;
+
+ xe->info.tile_count = probed_info->tile_count;
xe->info.step.basedie = probed_info->step.basedie;
xe->info.step.graphics = probed_info->step.graphics;
xe->info.step.media = probed_info->step.media;
@@ -1037,8 +1038,6 @@ static int xe_info_init(struct xe_device *xe,
xe->info.has_soc_remapper_telem = 0;
}
- xe_info_probe_tile_count(xe, desc);
-
for_each_remote_tile(tile, xe, id) {
int err;
--
2.53.0
^ permalink raw reply related [flat|nested] 24+ messages in thread
* ✓ CI.KUnit: success for drm/xe: Probe info outside of xe_info_init() and xe_info_init_early()
2026-06-09 20:17 [PATCH 0/9] drm/xe: Probe info outside of xe_info_init() and xe_info_init_early() Gustavo Sousa
` (8 preceding siblings ...)
2026-06-09 20:17 ` [PATCH 9/9] drm/xe: Add tile_count to xe_probed_info Gustavo Sousa
@ 2026-06-09 20:58 ` Patchwork
2026-06-09 22:06 ` ✓ Xe.CI.BAT: " Patchwork
2026-06-10 12:22 ` ✗ Xe.CI.FULL: failure " Patchwork
11 siblings, 0 replies; 24+ messages in thread
From: Patchwork @ 2026-06-09 20:58 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-xe
== Series Details ==
Series: drm/xe: Probe info outside of xe_info_init() and xe_info_init_early()
URL : https://patchwork.freedesktop.org/series/168205/
State : success
== Summary ==
+ trap cleanup EXIT
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/xe/.kunitconfig
[20:57:03] Configuring KUnit Kernel ...
Generating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[20:57:10] 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
[20:58:09] Starting KUnit Kernel (1/1)...
[20:58:09] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[20:58:09] ================== guc_buf (11 subtests) ===================
[20:58:09] [PASSED] test_smallest
[20:58:09] [PASSED] test_largest
[20:58:09] [PASSED] test_granular
[20:58:09] [PASSED] test_unique
[20:58:09] [PASSED] test_overlap
[20:58:09] [PASSED] test_reusable
[20:58:09] [PASSED] test_too_big
[20:58:09] [PASSED] test_flush
[20:58:09] [PASSED] test_lookup
[20:58:09] [PASSED] test_data
[20:58:09] [PASSED] test_class
[20:58:09] ===================== [PASSED] guc_buf =====================
[20:58:09] =================== guc_dbm (7 subtests) ===================
[20:58:09] [PASSED] test_empty
[20:58:09] [PASSED] test_default
[20:58:09] ======================== test_size ========================
[20:58:09] [PASSED] 4
[20:58:09] [PASSED] 8
[20:58:09] [PASSED] 32
[20:58:09] [PASSED] 256
[20:58:09] ==================== [PASSED] test_size ====================
[20:58:09] ======================= test_reuse ========================
[20:58:09] [PASSED] 4
[20:58:09] [PASSED] 8
[20:58:09] [PASSED] 32
[20:58:09] [PASSED] 256
[20:58:09] =================== [PASSED] test_reuse ====================
[20:58:09] =================== test_range_overlap ====================
[20:58:09] [PASSED] 4
[20:58:09] [PASSED] 8
[20:58:09] [PASSED] 32
[20:58:09] [PASSED] 256
[20:58:09] =============== [PASSED] test_range_overlap ================
[20:58:09] =================== test_range_compact ====================
[20:58:09] [PASSED] 4
[20:58:09] [PASSED] 8
[20:58:09] [PASSED] 32
[20:58:09] [PASSED] 256
[20:58:09] =============== [PASSED] test_range_compact ================
[20:58:09] ==================== test_range_spare =====================
[20:58:09] [PASSED] 4
[20:58:09] [PASSED] 8
[20:58:09] [PASSED] 32
[20:58:09] [PASSED] 256
[20:58:09] ================ [PASSED] test_range_spare =================
[20:58:09] ===================== [PASSED] guc_dbm =====================
[20:58:09] =================== guc_idm (6 subtests) ===================
[20:58:09] [PASSED] bad_init
[20:58:09] [PASSED] no_init
[20:58:09] [PASSED] init_fini
[20:58:09] [PASSED] check_used
[20:58:09] [PASSED] check_quota
[20:58:09] [PASSED] check_all
[20:58:09] ===================== [PASSED] guc_idm =====================
[20:58:09] ================== no_relay (3 subtests) ===================
[20:58:09] [PASSED] xe_drops_guc2pf_if_not_ready
[20:58:09] [PASSED] xe_drops_guc2vf_if_not_ready
[20:58:09] [PASSED] xe_rejects_send_if_not_ready
[20:58:09] ==================== [PASSED] no_relay =====================
[20:58:09] ================== pf_relay (14 subtests) ==================
[20:58:09] [PASSED] pf_rejects_guc2pf_too_short
[20:58:09] [PASSED] pf_rejects_guc2pf_too_long
[20:58:09] [PASSED] pf_rejects_guc2pf_no_payload
[20:58:09] [PASSED] pf_fails_no_payload
[20:58:09] [PASSED] pf_fails_bad_origin
[20:58:09] [PASSED] pf_fails_bad_type
[20:58:09] [PASSED] pf_txn_reports_error
[20:58:09] [PASSED] pf_txn_sends_pf2guc
[20:58:09] [PASSED] pf_sends_pf2guc
[20:58:09] [SKIPPED] pf_loopback_nop
[20:58:09] [SKIPPED] pf_loopback_echo
[20:58:09] [SKIPPED] pf_loopback_fail
[20:58:09] [SKIPPED] pf_loopback_busy
[20:58:09] [SKIPPED] pf_loopback_retry
[20:58:09] ==================== [PASSED] pf_relay =====================
[20:58:09] ================== vf_relay (3 subtests) ===================
[20:58:09] [PASSED] vf_rejects_guc2vf_too_short
[20:58:09] [PASSED] vf_rejects_guc2vf_too_long
[20:58:09] [PASSED] vf_rejects_guc2vf_no_payload
[20:58:09] ==================== [PASSED] vf_relay =====================
[20:58:09] ================ pf_gt_config (9 subtests) =================
[20:58:09] [PASSED] fair_contexts_1vf
[20:58:09] [PASSED] fair_doorbells_1vf
[20:58:09] [PASSED] fair_ggtt_1vf
[20:58:09] ====================== fair_vram_1vf ======================
[20:58:09] [PASSED] 3.50 GiB
[20:58:09] [PASSED] 11.5 GiB
[20:58:09] [PASSED] 15.5 GiB
[20:58:09] [PASSED] 31.5 GiB
[20:58:09] [PASSED] 63.5 GiB
[20:58:09] [PASSED] 1.91 GiB
[20:58:09] ================== [PASSED] fair_vram_1vf ==================
[20:58:09] ================ fair_vram_1vf_admin_only =================
[20:58:09] [PASSED] 3.50 GiB
[20:58:09] [PASSED] 11.5 GiB
[20:58:09] [PASSED] 15.5 GiB
[20:58:09] [PASSED] 31.5 GiB
[20:58:09] [PASSED] 63.5 GiB
[20:58:09] [PASSED] 1.91 GiB
[20:58:09] ============ [PASSED] fair_vram_1vf_admin_only =============
[20:58:09] ====================== fair_contexts ======================
[20:58:09] [PASSED] 1 VF
[20:58:09] [PASSED] 2 VFs
[20:58:09] [PASSED] 3 VFs
[20:58:09] [PASSED] 4 VFs
[20:58:09] [PASSED] 5 VFs
[20:58:09] [PASSED] 6 VFs
[20:58:09] [PASSED] 7 VFs
[20:58:09] [PASSED] 8 VFs
[20:58:09] [PASSED] 9 VFs
[20:58:09] [PASSED] 10 VFs
[20:58:09] [PASSED] 11 VFs
[20:58:09] [PASSED] 12 VFs
[20:58:09] [PASSED] 13 VFs
[20:58:09] [PASSED] 14 VFs
[20:58:09] [PASSED] 15 VFs
[20:58:09] [PASSED] 16 VFs
[20:58:09] [PASSED] 17 VFs
[20:58:09] [PASSED] 18 VFs
[20:58:09] [PASSED] 19 VFs
[20:58:09] [PASSED] 20 VFs
[20:58:09] [PASSED] 21 VFs
[20:58:09] [PASSED] 22 VFs
[20:58:09] [PASSED] 23 VFs
[20:58:09] [PASSED] 24 VFs
[20:58:09] [PASSED] 25 VFs
[20:58:09] [PASSED] 26 VFs
[20:58:09] [PASSED] 27 VFs
[20:58:09] [PASSED] 28 VFs
[20:58:09] [PASSED] 29 VFs
[20:58:09] [PASSED] 30 VFs
[20:58:09] [PASSED] 31 VFs
[20:58:09] [PASSED] 32 VFs
[20:58:09] [PASSED] 33 VFs
[20:58:10] [PASSED] 34 VFs
[20:58:10] [PASSED] 35 VFs
[20:58:10] [PASSED] 36 VFs
[20:58:10] [PASSED] 37 VFs
[20:58:10] [PASSED] 38 VFs
[20:58:10] [PASSED] 39 VFs
[20:58:10] [PASSED] 40 VFs
[20:58:10] [PASSED] 41 VFs
[20:58:10] [PASSED] 42 VFs
[20:58:10] [PASSED] 43 VFs
[20:58:10] [PASSED] 44 VFs
[20:58:10] [PASSED] 45 VFs
[20:58:10] [PASSED] 46 VFs
[20:58:10] [PASSED] 47 VFs
[20:58:10] [PASSED] 48 VFs
[20:58:10] [PASSED] 49 VFs
[20:58:10] [PASSED] 50 VFs
[20:58:10] [PASSED] 51 VFs
[20:58:10] [PASSED] 52 VFs
[20:58:10] [PASSED] 53 VFs
[20:58:10] [PASSED] 54 VFs
[20:58:10] [PASSED] 55 VFs
[20:58:10] [PASSED] 56 VFs
[20:58:10] [PASSED] 57 VFs
[20:58:10] [PASSED] 58 VFs
[20:58:10] [PASSED] 59 VFs
[20:58:10] [PASSED] 60 VFs
[20:58:10] [PASSED] 61 VFs
[20:58:10] [PASSED] 62 VFs
[20:58:10] [PASSED] 63 VFs
[20:58:10] ================== [PASSED] fair_contexts ==================
[20:58:10] ===================== fair_doorbells ======================
[20:58:10] [PASSED] 1 VF
[20:58:10] [PASSED] 2 VFs
[20:58:10] [PASSED] 3 VFs
[20:58:10] [PASSED] 4 VFs
[20:58:10] [PASSED] 5 VFs
[20:58:10] [PASSED] 6 VFs
[20:58:10] [PASSED] 7 VFs
[20:58:10] [PASSED] 8 VFs
[20:58:10] [PASSED] 9 VFs
[20:58:10] [PASSED] 10 VFs
[20:58:10] [PASSED] 11 VFs
[20:58:10] [PASSED] 12 VFs
[20:58:10] [PASSED] 13 VFs
[20:58:10] [PASSED] 14 VFs
[20:58:10] [PASSED] 15 VFs
[20:58:10] [PASSED] 16 VFs
[20:58:10] [PASSED] 17 VFs
[20:58:10] [PASSED] 18 VFs
[20:58:10] [PASSED] 19 VFs
[20:58:10] [PASSED] 20 VFs
[20:58:10] [PASSED] 21 VFs
[20:58:10] [PASSED] 22 VFs
[20:58:10] [PASSED] 23 VFs
[20:58:10] [PASSED] 24 VFs
[20:58:10] [PASSED] 25 VFs
[20:58:10] [PASSED] 26 VFs
[20:58:10] [PASSED] 27 VFs
[20:58:10] [PASSED] 28 VFs
[20:58:10] [PASSED] 29 VFs
[20:58:10] [PASSED] 30 VFs
[20:58:10] [PASSED] 31 VFs
[20:58:10] [PASSED] 32 VFs
[20:58:10] [PASSED] 33 VFs
[20:58:10] [PASSED] 34 VFs
[20:58:10] [PASSED] 35 VFs
[20:58:10] [PASSED] 36 VFs
[20:58:10] [PASSED] 37 VFs
[20:58:10] [PASSED] 38 VFs
[20:58:10] [PASSED] 39 VFs
[20:58:10] [PASSED] 40 VFs
[20:58:10] [PASSED] 41 VFs
[20:58:10] [PASSED] 42 VFs
[20:58:10] [PASSED] 43 VFs
[20:58:10] [PASSED] 44 VFs
[20:58:10] [PASSED] 45 VFs
[20:58:10] [PASSED] 46 VFs
[20:58:10] [PASSED] 47 VFs
[20:58:10] [PASSED] 48 VFs
[20:58:10] [PASSED] 49 VFs
[20:58:10] [PASSED] 50 VFs
[20:58:10] [PASSED] 51 VFs
[20:58:10] [PASSED] 52 VFs
[20:58:10] [PASSED] 53 VFs
[20:58:10] [PASSED] 54 VFs
[20:58:10] [PASSED] 55 VFs
[20:58:10] [PASSED] 56 VFs
[20:58:10] [PASSED] 57 VFs
[20:58:10] [PASSED] 58 VFs
[20:58:10] [PASSED] 59 VFs
[20:58:10] [PASSED] 60 VFs
[20:58:10] [PASSED] 61 VFs
[20:58:10] [PASSED] 62 VFs
[20:58:10] [PASSED] 63 VFs
[20:58:10] ================= [PASSED] fair_doorbells ==================
[20:58:10] ======================== fair_ggtt ========================
[20:58:10] [PASSED] 1 VF
[20:58:10] [PASSED] 2 VFs
[20:58:10] [PASSED] 3 VFs
[20:58:10] [PASSED] 4 VFs
[20:58:10] [PASSED] 5 VFs
[20:58:10] [PASSED] 6 VFs
[20:58:10] [PASSED] 7 VFs
[20:58:10] [PASSED] 8 VFs
[20:58:10] [PASSED] 9 VFs
[20:58:10] [PASSED] 10 VFs
[20:58:10] [PASSED] 11 VFs
[20:58:10] [PASSED] 12 VFs
[20:58:10] [PASSED] 13 VFs
[20:58:10] [PASSED] 14 VFs
[20:58:10] [PASSED] 15 VFs
[20:58:10] [PASSED] 16 VFs
[20:58:10] [PASSED] 17 VFs
[20:58:10] [PASSED] 18 VFs
[20:58:10] [PASSED] 19 VFs
[20:58:10] [PASSED] 20 VFs
[20:58:10] [PASSED] 21 VFs
[20:58:10] [PASSED] 22 VFs
[20:58:10] [PASSED] 23 VFs
[20:58:10] [PASSED] 24 VFs
[20:58:10] [PASSED] 25 VFs
[20:58:10] [PASSED] 26 VFs
[20:58:10] [PASSED] 27 VFs
[20:58:10] [PASSED] 28 VFs
[20:58:10] [PASSED] 29 VFs
[20:58:10] [PASSED] 30 VFs
[20:58:10] [PASSED] 31 VFs
[20:58:10] [PASSED] 32 VFs
[20:58:10] [PASSED] 33 VFs
[20:58:10] [PASSED] 34 VFs
[20:58:10] [PASSED] 35 VFs
[20:58:10] [PASSED] 36 VFs
[20:58:10] [PASSED] 37 VFs
[20:58:10] [PASSED] 38 VFs
[20:58:10] [PASSED] 39 VFs
[20:58:10] [PASSED] 40 VFs
[20:58:10] [PASSED] 41 VFs
[20:58:10] [PASSED] 42 VFs
[20:58:10] [PASSED] 43 VFs
[20:58:10] [PASSED] 44 VFs
[20:58:10] [PASSED] 45 VFs
[20:58:10] [PASSED] 46 VFs
[20:58:10] [PASSED] 47 VFs
[20:58:10] [PASSED] 48 VFs
[20:58:10] [PASSED] 49 VFs
[20:58:10] [PASSED] 50 VFs
[20:58:10] [PASSED] 51 VFs
[20:58:10] [PASSED] 52 VFs
[20:58:10] [PASSED] 53 VFs
[20:58:10] [PASSED] 54 VFs
[20:58:10] [PASSED] 55 VFs
[20:58:10] [PASSED] 56 VFs
[20:58:10] [PASSED] 57 VFs
[20:58:10] [PASSED] 58 VFs
[20:58:10] [PASSED] 59 VFs
[20:58:10] [PASSED] 60 VFs
[20:58:10] [PASSED] 61 VFs
[20:58:10] [PASSED] 62 VFs
[20:58:10] [PASSED] 63 VFs
[20:58:10] ==================== [PASSED] fair_ggtt ====================
[20:58:10] ======================== fair_vram ========================
[20:58:10] [PASSED] 1 VF
[20:58:10] [PASSED] 2 VFs
[20:58:10] [PASSED] 3 VFs
[20:58:10] [PASSED] 4 VFs
[20:58:10] [PASSED] 5 VFs
[20:58:10] [PASSED] 6 VFs
[20:58:10] [PASSED] 7 VFs
[20:58:10] [PASSED] 8 VFs
[20:58:10] [PASSED] 9 VFs
[20:58:10] [PASSED] 10 VFs
[20:58:10] [PASSED] 11 VFs
[20:58:10] [PASSED] 12 VFs
[20:58:10] [PASSED] 13 VFs
[20:58:10] [PASSED] 14 VFs
[20:58:10] [PASSED] 15 VFs
[20:58:10] [PASSED] 16 VFs
[20:58:10] [PASSED] 17 VFs
[20:58:10] [PASSED] 18 VFs
[20:58:10] [PASSED] 19 VFs
[20:58:10] [PASSED] 20 VFs
[20:58:10] [PASSED] 21 VFs
[20:58:10] [PASSED] 22 VFs
[20:58:10] [PASSED] 23 VFs
[20:58:10] [PASSED] 24 VFs
[20:58:10] [PASSED] 25 VFs
[20:58:10] [PASSED] 26 VFs
[20:58:10] [PASSED] 27 VFs
[20:58:10] [PASSED] 28 VFs
[20:58:10] [PASSED] 29 VFs
[20:58:10] [PASSED] 30 VFs
[20:58:10] [PASSED] 31 VFs
[20:58:10] [PASSED] 32 VFs
[20:58:10] [PASSED] 33 VFs
[20:58:10] [PASSED] 34 VFs
[20:58:10] [PASSED] 35 VFs
[20:58:10] [PASSED] 36 VFs
[20:58:10] [PASSED] 37 VFs
[20:58:10] [PASSED] 38 VFs
[20:58:10] [PASSED] 39 VFs
[20:58:10] [PASSED] 40 VFs
[20:58:10] [PASSED] 41 VFs
[20:58:10] [PASSED] 42 VFs
[20:58:10] [PASSED] 43 VFs
[20:58:10] [PASSED] 44 VFs
[20:58:10] [PASSED] 45 VFs
[20:58:10] [PASSED] 46 VFs
[20:58:10] [PASSED] 47 VFs
[20:58:10] [PASSED] 48 VFs
[20:58:10] [PASSED] 49 VFs
[20:58:10] [PASSED] 50 VFs
[20:58:10] [PASSED] 51 VFs
[20:58:10] [PASSED] 52 VFs
[20:58:10] [PASSED] 53 VFs
[20:58:10] [PASSED] 54 VFs
[20:58:10] [PASSED] 55 VFs
[20:58:10] [PASSED] 56 VFs
[20:58:10] [PASSED] 57 VFs
[20:58:10] [PASSED] 58 VFs
[20:58:10] [PASSED] 59 VFs
[20:58:10] [PASSED] 60 VFs
[20:58:10] [PASSED] 61 VFs
[20:58:10] [PASSED] 62 VFs
[20:58:10] [PASSED] 63 VFs
[20:58:10] ==================== [PASSED] fair_vram ====================
[20:58:10] ================== [PASSED] pf_gt_config ===================
[20:58:10] ===================== lmtt (1 subtest) =====================
[20:58:10] ======================== test_ops =========================
[20:58:10] [PASSED] 2-level
[20:58:10] [PASSED] multi-level
[20:58:10] ==================== [PASSED] test_ops =====================
[20:58:10] ====================== [PASSED] lmtt =======================
[20:58:10] ================= pf_service (11 subtests) =================
[20:58:10] [PASSED] pf_negotiate_any
[20:58:10] [PASSED] pf_negotiate_base_match
[20:58:10] [PASSED] pf_negotiate_base_newer
[20:58:10] [PASSED] pf_negotiate_base_next
[20:58:10] [SKIPPED] pf_negotiate_base_older
[20:58:10] [PASSED] pf_negotiate_base_prev
[20:58:10] [PASSED] pf_negotiate_latest_match
[20:58:10] [PASSED] pf_negotiate_latest_newer
[20:58:10] [PASSED] pf_negotiate_latest_next
[20:58:10] [SKIPPED] pf_negotiate_latest_older
[20:58:10] [SKIPPED] pf_negotiate_latest_prev
[20:58:10] =================== [PASSED] pf_service ====================
[20:58:10] ================= xe_guc_g2g (2 subtests) ==================
[20:58:10] ============== xe_live_guc_g2g_kunit_default ==============
[20:58:10] ========= [SKIPPED] xe_live_guc_g2g_kunit_default ==========
[20:58:10] ============== xe_live_guc_g2g_kunit_allmem ===============
[20:58:10] ========== [SKIPPED] xe_live_guc_g2g_kunit_allmem ==========
[20:58:10] =================== [SKIPPED] xe_guc_g2g ===================
[20:58:10] =================== xe_mocs (2 subtests) ===================
[20:58:10] ================ xe_live_mocs_kernel_kunit ================
[20:58:10] =========== [SKIPPED] xe_live_mocs_kernel_kunit ============
[20:58:10] ================ xe_live_mocs_reset_kunit =================
[20:58:10] ============ [SKIPPED] xe_live_mocs_reset_kunit ============
[20:58:10] ==================== [SKIPPED] xe_mocs =====================
[20:58:10] ================= xe_migrate (2 subtests) ==================
[20:58:10] ================= xe_migrate_sanity_kunit =================
[20:58:10] ============ [SKIPPED] xe_migrate_sanity_kunit =============
[20:58:10] ================== xe_validate_ccs_kunit ==================
[20:58:10] ============= [SKIPPED] xe_validate_ccs_kunit ==============
[20:58:10] =================== [SKIPPED] xe_migrate ===================
[20:58:10] ================== xe_dma_buf (1 subtest) ==================
[20:58:10] ==================== xe_dma_buf_kunit =====================
[20:58:10] ================ [SKIPPED] xe_dma_buf_kunit ================
[20:58:10] =================== [SKIPPED] xe_dma_buf ===================
[20:58:10] ================= xe_bo_shrink (1 subtest) =================
[20:58:10] =================== xe_bo_shrink_kunit ====================
[20:58:10] =============== [SKIPPED] xe_bo_shrink_kunit ===============
[20:58:10] ================== [SKIPPED] xe_bo_shrink ==================
[20:58:10] ==================== xe_bo (2 subtests) ====================
[20:58:10] ================== xe_ccs_migrate_kunit ===================
[20:58:10] ============== [SKIPPED] xe_ccs_migrate_kunit ==============
[20:58:10] ==================== xe_bo_evict_kunit ====================
[20:58:10] =============== [SKIPPED] xe_bo_evict_kunit ================
[20:58:10] ===================== [SKIPPED] xe_bo ======================
[20:58:10] ==================== args (13 subtests) ====================
[20:58:10] [PASSED] count_args_test
[20:58:10] [PASSED] call_args_example
[20:58:10] [PASSED] call_args_test
[20:58:10] [PASSED] drop_first_arg_example
[20:58:10] [PASSED] drop_first_arg_test
[20:58:10] [PASSED] first_arg_example
[20:58:10] [PASSED] first_arg_test
[20:58:10] [PASSED] last_arg_example
[20:58:10] [PASSED] last_arg_test
[20:58:10] [PASSED] pick_arg_example
[20:58:10] [PASSED] if_args_example
[20:58:10] [PASSED] if_args_test
[20:58:10] [PASSED] sep_comma_example
[20:58:10] ====================== [PASSED] args =======================
[20:58:10] =================== xe_pci (3 subtests) ====================
[20:58:10] ==================== check_graphics_ip ====================
[20:58:10] [PASSED] 12.00 Xe_LP
[20:58:10] [PASSED] 12.10 Xe_LP+
[20:58:10] [PASSED] 12.55 Xe_HPG
[20:58:10] [PASSED] 12.60 Xe_HPC
[20:58:10] [PASSED] 12.70 Xe_LPG
[20:58:10] [PASSED] 12.71 Xe_LPG
[20:58:10] [PASSED] 12.74 Xe_LPG+
[20:58:10] [PASSED] 20.01 Xe2_HPG
[20:58:10] [PASSED] 20.02 Xe2_HPG
[20:58:10] [PASSED] 20.04 Xe2_LPG
[20:58:10] [PASSED] 30.00 Xe3_LPG
[20:58:10] [PASSED] 30.01 Xe3_LPG
[20:58:10] [PASSED] 30.03 Xe3_LPG
[20:58:10] [PASSED] 30.04 Xe3_LPG
[20:58:10] [PASSED] 30.05 Xe3_LPG
[20:58:10] [PASSED] 35.10 Xe3p_LPG
[20:58:10] [PASSED] 35.11 Xe3p_XPC
[20:58:10] ================ [PASSED] check_graphics_ip ================
[20:58:10] ===================== check_media_ip ======================
[20:58:10] [PASSED] 12.00 Xe_M
[20:58:10] [PASSED] 12.55 Xe_HPM
[20:58:10] [PASSED] 13.00 Xe_LPM+
[20:58:10] [PASSED] 13.01 Xe2_HPM
[20:58:10] [PASSED] 20.00 Xe2_LPM
[20:58:10] [PASSED] 30.00 Xe3_LPM
[20:58:10] [PASSED] 30.02 Xe3_LPM
[20:58:10] [PASSED] 35.00 Xe3p_LPM
[20:58:10] [PASSED] 35.03 Xe3p_HPM
[20:58:10] ================= [PASSED] check_media_ip ==================
[20:58:10] =================== check_platform_desc ===================
[20:58:10] [PASSED] 0x9A60 (TIGERLAKE)
[20:58:10] [PASSED] 0x9A68 (TIGERLAKE)
[20:58:10] [PASSED] 0x9A70 (TIGERLAKE)
[20:58:10] [PASSED] 0x9A40 (TIGERLAKE)
[20:58:10] [PASSED] 0x9A49 (TIGERLAKE)
[20:58:10] [PASSED] 0x9A59 (TIGERLAKE)
[20:58:10] [PASSED] 0x9A78 (TIGERLAKE)
[20:58:10] [PASSED] 0x9AC0 (TIGERLAKE)
[20:58:10] [PASSED] 0x9AC9 (TIGERLAKE)
[20:58:10] [PASSED] 0x9AD9 (TIGERLAKE)
[20:58:10] [PASSED] 0x9AF8 (TIGERLAKE)
[20:58:10] [PASSED] 0x4C80 (ROCKETLAKE)
[20:58:10] [PASSED] 0x4C8A (ROCKETLAKE)
[20:58:10] [PASSED] 0x4C8B (ROCKETLAKE)
[20:58:10] [PASSED] 0x4C8C (ROCKETLAKE)
[20:58:10] [PASSED] 0x4C90 (ROCKETLAKE)
[20:58:10] [PASSED] 0x4C9A (ROCKETLAKE)
[20:58:10] [PASSED] 0x4680 (ALDERLAKE_S)
[20:58:10] [PASSED] 0x4682 (ALDERLAKE_S)
[20:58:10] [PASSED] 0x4688 (ALDERLAKE_S)
[20:58:10] [PASSED] 0x468A (ALDERLAKE_S)
[20:58:10] [PASSED] 0x468B (ALDERLAKE_S)
[20:58:10] [PASSED] 0x4690 (ALDERLAKE_S)
[20:58:10] [PASSED] 0x4692 (ALDERLAKE_S)
[20:58:10] [PASSED] 0x4693 (ALDERLAKE_S)
[20:58:10] [PASSED] 0x46A0 (ALDERLAKE_P)
[20:58:10] [PASSED] 0x46A1 (ALDERLAKE_P)
[20:58:10] [PASSED] 0x46A2 (ALDERLAKE_P)
[20:58:10] [PASSED] 0x46A3 (ALDERLAKE_P)
[20:58:10] [PASSED] 0x46A6 (ALDERLAKE_P)
[20:58:10] [PASSED] 0x46A8 (ALDERLAKE_P)
[20:58:10] [PASSED] 0x46AA (ALDERLAKE_P)
[20:58:10] [PASSED] 0x462A (ALDERLAKE_P)
[20:58:10] [PASSED] 0x4626 (ALDERLAKE_P)
[20:58:10] [PASSED] 0x4628 (ALDERLAKE_P)
[20:58:10] [PASSED] 0x46B0 (ALDERLAKE_P)
[20:58:10] [PASSED] 0x46B1 (ALDERLAKE_P)
[20:58:10] [PASSED] 0x46B2 (ALDERLAKE_P)
[20:58:10] [PASSED] 0x46B3 (ALDERLAKE_P)
[20:58:10] [PASSED] 0x46C0 (ALDERLAKE_P)
[20:58:10] [PASSED] 0x46C1 (ALDERLAKE_P)
[20:58:10] [PASSED] 0x46C2 (ALDERLAKE_P)
[20:58:10] [PASSED] 0x46C3 (ALDERLAKE_P)
[20:58:10] [PASSED] 0x46D0 (ALDERLAKE_N)
[20:58:10] [PASSED] 0x46D1 (ALDERLAKE_N)
[20:58:10] [PASSED] 0x46D2 (ALDERLAKE_N)
[20:58:10] [PASSED] 0x46D3 (ALDERLAKE_N)
[20:58:10] [PASSED] 0x46D4 (ALDERLAKE_N)
[20:58:10] [PASSED] 0xA721 (ALDERLAKE_P)
[20:58:10] [PASSED] 0xA7A1 (ALDERLAKE_P)
[20:58:10] [PASSED] 0xA7A9 (ALDERLAKE_P)
[20:58:10] [PASSED] 0xA7AC (ALDERLAKE_P)
[20:58:10] [PASSED] 0xA7AD (ALDERLAKE_P)
[20:58:10] [PASSED] 0xA720 (ALDERLAKE_P)
[20:58:10] [PASSED] 0xA7A0 (ALDERLAKE_P)
[20:58:10] [PASSED] 0xA7A8 (ALDERLAKE_P)
[20:58:10] [PASSED] 0xA7AA (ALDERLAKE_P)
[20:58:10] [PASSED] 0xA7AB (ALDERLAKE_P)
[20:58:10] [PASSED] 0xA780 (ALDERLAKE_S)
[20:58:10] [PASSED] 0xA781 (ALDERLAKE_S)
[20:58:10] [PASSED] 0xA782 (ALDERLAKE_S)
[20:58:10] [PASSED] 0xA783 (ALDERLAKE_S)
[20:58:10] [PASSED] 0xA788 (ALDERLAKE_S)
[20:58:10] [PASSED] 0xA789 (ALDERLAKE_S)
[20:58:10] [PASSED] 0xA78A (ALDERLAKE_S)
[20:58:10] [PASSED] 0xA78B (ALDERLAKE_S)
[20:58:10] [PASSED] 0x4905 (DG1)
[20:58:10] [PASSED] 0x4906 (DG1)
[20:58:10] [PASSED] 0x4907 (DG1)
[20:58:10] [PASSED] 0x4908 (DG1)
[20:58:10] [PASSED] 0x4909 (DG1)
[20:58:10] [PASSED] 0x56C0 (DG2)
[20:58:10] [PASSED] 0x56C2 (DG2)
[20:58:10] [PASSED] 0x56C1 (DG2)
[20:58:10] [PASSED] 0x7D51 (METEORLAKE)
[20:58:10] [PASSED] 0x7DD1 (METEORLAKE)
[20:58:10] [PASSED] 0x7D41 (METEORLAKE)
[20:58:10] [PASSED] 0x7D67 (METEORLAKE)
[20:58:10] [PASSED] 0xB640 (METEORLAKE)
[20:58:10] [PASSED] 0x56A0 (DG2)
[20:58:10] [PASSED] 0x56A1 (DG2)
[20:58:10] [PASSED] 0x56A2 (DG2)
[20:58:10] [PASSED] 0x56BE (DG2)
[20:58:10] [PASSED] 0x56BF (DG2)
[20:58:10] [PASSED] 0x5690 (DG2)
[20:58:10] [PASSED] 0x5691 (DG2)
[20:58:10] [PASSED] 0x5692 (DG2)
[20:58:10] [PASSED] 0x56A5 (DG2)
[20:58:10] [PASSED] 0x56A6 (DG2)
[20:58:10] [PASSED] 0x56B0 (DG2)
[20:58:10] [PASSED] 0x56B1 (DG2)
[20:58:10] [PASSED] 0x56BA (DG2)
[20:58:10] [PASSED] 0x56BB (DG2)
[20:58:10] [PASSED] 0x56BC (DG2)
[20:58:10] [PASSED] 0x56BD (DG2)
[20:58:10] [PASSED] 0x5693 (DG2)
[20:58:10] [PASSED] 0x5694 (DG2)
[20:58:10] [PASSED] 0x5695 (DG2)
[20:58:10] [PASSED] 0x56A3 (DG2)
[20:58:10] [PASSED] 0x56A4 (DG2)
[20:58:10] [PASSED] 0x56B2 (DG2)
[20:58:10] [PASSED] 0x56B3 (DG2)
[20:58:10] [PASSED] 0x5696 (DG2)
[20:58:10] [PASSED] 0x5697 (DG2)
[20:58:10] [PASSED] 0xB69 (PVC)
[20:58:10] [PASSED] 0xB6E (PVC)
[20:58:10] [PASSED] 0xBD4 (PVC)
[20:58:10] [PASSED] 0xBD5 (PVC)
[20:58:10] [PASSED] 0xBD6 (PVC)
[20:58:10] [PASSED] 0xBD7 (PVC)
[20:58:10] [PASSED] 0xBD8 (PVC)
[20:58:10] [PASSED] 0xBD9 (PVC)
[20:58:10] [PASSED] 0xBDA (PVC)
[20:58:10] [PASSED] 0xBDB (PVC)
[20:58:10] [PASSED] 0xBE0 (PVC)
[20:58:10] [PASSED] 0xBE1 (PVC)
[20:58:10] [PASSED] 0xBE5 (PVC)
[20:58:10] [PASSED] 0x7D40 (METEORLAKE)
[20:58:10] [PASSED] 0x7D45 (METEORLAKE)
[20:58:10] [PASSED] 0x7D55 (METEORLAKE)
[20:58:10] [PASSED] 0x7D60 (METEORLAKE)
[20:58:10] [PASSED] 0x7DD5 (METEORLAKE)
[20:58:10] [PASSED] 0x6420 (LUNARLAKE)
[20:58:10] [PASSED] 0x64A0 (LUNARLAKE)
[20:58:10] [PASSED] 0x64B0 (LUNARLAKE)
[20:58:10] [PASSED] 0xE202 (BATTLEMAGE)
[20:58:10] [PASSED] 0xE209 (BATTLEMAGE)
[20:58:10] [PASSED] 0xE20B (BATTLEMAGE)
[20:58:10] [PASSED] 0xE20C (BATTLEMAGE)
[20:58:10] [PASSED] 0xE20D (BATTLEMAGE)
[20:58:10] [PASSED] 0xE210 (BATTLEMAGE)
[20:58:10] [PASSED] 0xE211 (BATTLEMAGE)
[20:58:10] [PASSED] 0xE212 (BATTLEMAGE)
[20:58:10] [PASSED] 0xE216 (BATTLEMAGE)
[20:58:10] [PASSED] 0xE220 (BATTLEMAGE)
[20:58:10] [PASSED] 0xE221 (BATTLEMAGE)
[20:58:10] [PASSED] 0xE222 (BATTLEMAGE)
[20:58:10] [PASSED] 0xE223 (BATTLEMAGE)
[20:58:10] [PASSED] 0xB080 (PANTHERLAKE)
[20:58:10] [PASSED] 0xB081 (PANTHERLAKE)
[20:58:10] [PASSED] 0xB082 (PANTHERLAKE)
[20:58:10] [PASSED] 0xB083 (PANTHERLAKE)
[20:58:10] [PASSED] 0xB084 (PANTHERLAKE)
[20:58:10] [PASSED] 0xB085 (PANTHERLAKE)
[20:58:10] [PASSED] 0xB086 (PANTHERLAKE)
[20:58:10] [PASSED] 0xB087 (PANTHERLAKE)
[20:58:10] [PASSED] 0xB08F (PANTHERLAKE)
[20:58:10] [PASSED] 0xB090 (PANTHERLAKE)
[20:58:10] [PASSED] 0xB0A0 (PANTHERLAKE)
[20:58:10] [PASSED] 0xB0B0 (PANTHERLAKE)
[20:58:10] [PASSED] 0xFD80 (PANTHERLAKE)
[20:58:10] [PASSED] 0xFD81 (PANTHERLAKE)
[20:58:10] [PASSED] 0xD740 (NOVALAKE_S)
[20:58:10] [PASSED] 0xD741 (NOVALAKE_S)
[20:58:10] [PASSED] 0xD742 (NOVALAKE_S)
[20:58:10] [PASSED] 0xD743 (NOVALAKE_S)
[20:58:10] [PASSED] 0xD745 (NOVALAKE_S)
[20:58:10] [PASSED] 0xD74A (NOVALAKE_S)
[20:58:10] [PASSED] 0xD74B (NOVALAKE_S)
[20:58:10] [PASSED] 0x674C (CRESCENTISLAND)
[20:58:10] [PASSED] 0x674D (CRESCENTISLAND)
[20:58:10] [PASSED] 0x674E (CRESCENTISLAND)
[20:58:10] [PASSED] 0x674F (CRESCENTISLAND)
[20:58:10] [PASSED] 0x6750 (CRESCENTISLAND)
[20:58:10] [PASSED] 0xD750 (NOVALAKE_P)
[20:58:10] [PASSED] 0xD751 (NOVALAKE_P)
[20:58:10] [PASSED] 0xD752 (NOVALAKE_P)
[20:58:10] [PASSED] 0xD753 (NOVALAKE_P)
[20:58:10] [PASSED] 0xD754 (NOVALAKE_P)
[20:58:10] [PASSED] 0xD755 (NOVALAKE_P)
[20:58:10] [PASSED] 0xD756 (NOVALAKE_P)
[20:58:10] [PASSED] 0xD757 (NOVALAKE_P)
[20:58:10] [PASSED] 0xD75F (NOVALAKE_P)
[20:58:10] =============== [PASSED] check_platform_desc ===============
[20:58:10] ===================== [PASSED] xe_pci ======================
[20:58:10] ============= xe_rtp_tables_test (4 subtests) ==============
[20:58:10] ================== xe_rtp_table_gt_test ===================
[20:58:10] [PASSED] gt_was/14011060649
[20:58:10] [PASSED] gt_was/14011059788
[20:58:10] [PASSED] gt_was/14015795083
[20:58:10] [PASSED] gt_was/16021867713
[20:58:10] [PASSED] gt_was/14019449301
[20:58:10] [PASSED] gt_was/16028005424
[20:58:10] [PASSED] gt_was/14026578760
[20:58:10] [PASSED] gt_was/1409420604
[20:58:10] [PASSED] gt_was/1408615072
[20:58:10] [PASSED] gt_was/22010523718
[20:58:10] [PASSED] gt_was/14011006942
[20:58:10] [PASSED] gt_was/14014830051
[20:58:10] [PASSED] gt_was/18018781329
[20:58:10] [PASSED] gt_was/1509235366
[20:58:10] [PASSED] gt_was/18018781329
[20:58:10] [PASSED] gt_was/16016694945
[20:58:10] [PASSED] gt_was/14018575942
[20:58:10] [PASSED] gt_was/22016670082
[20:58:10] [PASSED] gt_was/22016670082
[20:58:10] [PASSED] gt_was/14017421178
[20:58:10] [PASSED] gt_was/16025250150
[20:58:10] [PASSED] gt_was/14021871409
[20:58:10] [PASSED] gt_was/16021865536
[20:58:10] [PASSED] gt_was/14021486841
[20:58:10] [PASSED] gt_was/14025160223
[20:58:10] [PASSED] gt_was/14026144927, 16029437861
[20:58:10] [PASSED] gt_was/14025635424
[20:58:10] [PASSED] gt_was/16028005424
[20:58:10] ============== [PASSED] xe_rtp_table_gt_test ===============
[20:58:10] ================== xe_rtp_table_gt_test ===================
[20:58:10] [PASSED] gt_tunings/Tuning: Blend Fill Caching Optimization Disable
[20:58:10] [PASSED] gt_tunings/Tuning: 32B Access Enable
[20:58:10] [PASSED] gt_tunings/Tuning: L3 cache
[20:58:10] [PASSED] gt_tunings/Tuning: L3 cache - media
[20:58:10] [PASSED] gt_tunings/Tuning: Compression Overfetch
[20:58:10] [PASSED] gt_tunings/Tuning: Compression Overfetch - media
[20:58:10] [PASSED] gt_tunings/Tuning: Enable compressible partial write overfetch in L3
[20:58:10] [PASSED] gt_tunings/Tuning: Enable compressible partial write overfetch in L3 - media
[20:58:10] [PASSED] gt_tunings/Tuning: L2 Overfetch Compressible Only
[20:58:10] [PASSED] gt_tunings/Tuning: L2 Overfetch Compressible Only - media
[20:58:10] [PASSED] gt_tunings/Tuning: Stateless compression control
[20:58:10] [PASSED] gt_tunings/Tuning: Stateless compression control - media
[20:58:10] [PASSED] gt_tunings/Tuning: L3 RW flush all Cache
[20:58:10] [PASSED] gt_tunings/Tuning: L3 RW flush all cache - media
[20:58:10] [PASSED] gt_tunings/Tuning: Set STLB Bank Hash Mode to 4KB
[20:58:10] ============== [PASSED] xe_rtp_table_gt_test ===============
[20:58:10] ================== xe_rtp_table_oob_test ==================
[20:58:10] [PASSED] oob_was/1607983814
[20:58:10] [PASSED] oob_was/16010904313
[20:58:10] [PASSED] oob_was/18022495364
[20:58:10] [PASSED] oob_was/22012773006
[20:58:10] [PASSED] oob_was/14014475959
[20:58:10] [PASSED] oob_was/22011391025
[20:58:10] [PASSED] oob_was/22012727170
[20:58:10] [PASSED] oob_was/22012727685
[20:58:10] [PASSED] oob_was/22016596838
[20:58:10] [PASSED] oob_was/18020744125
[20:58:10] [PASSED] oob_was/1409600907
[20:58:10] [PASSED] oob_was/22014953428
[20:58:10] [PASSED] oob_was/16017236439
[20:58:10] [PASSED] oob_was/14019821291
[20:58:10] [PASSED] oob_was/14015076503
[20:58:10] [PASSED] oob_was/14018913170
[20:58:10] [PASSED] oob_was/14018094691
[20:58:10] [PASSED] oob_was/18024947630
[20:58:10] [PASSED] oob_was/16022287689
[20:58:10] [PASSED] oob_was/13011645652
[20:58:10] [PASSED] oob_was/14022293748
[20:58:10] [PASSED] oob_was/22019794406
[20:58:10] [PASSED] oob_was/22019338487
[20:58:10] [PASSED] oob_was/16023588340
[20:58:10] [PASSED] oob_was/14019789679
[20:58:10] [PASSED] oob_was/14022866841
[20:58:10] [PASSED] oob_was/16021333562
[20:58:10] [PASSED] oob_was/14016712196
[20:58:10] [PASSED] oob_was/14015568240
[20:58:10] [PASSED] oob_was/18013179988
[20:58:10] [PASSED] oob_was/1508761755
[20:58:10] [PASSED] oob_was/16023105232
[20:58:10] [PASSED] oob_was/16026508708
[20:58:10] [PASSED] oob_was/14020001231
[20:58:10] [PASSED] oob_was/16023683509
[20:58:10] [PASSED] oob_was/14025515070
[20:58:10] [PASSED] oob_was/15015404425_disable
[20:58:10] [PASSED] oob_was/16026007364
[20:58:10] [PASSED] oob_was/14020316580
[20:58:10] [PASSED] oob_was/14025883347
[20:58:10] ============== [PASSED] xe_rtp_table_oob_test ==============
[20:58:10] ================ xe_rtp_table_dev_oob_test ================
[20:58:10] [PASSED] device_oob_was/22010954014
[20:58:10] [PASSED] device_oob_was/15015404425
[20:58:10] [PASSED] device_oob_was/22019338487_display
[20:58:10] [PASSED] device_oob_was/14022085890
[20:58:10] [PASSED] device_oob_was/14026539277
[20:58:10] [PASSED] device_oob_was/14026633728
[20:58:10] [PASSED] device_oob_was/14026746987
[20:58:10] [PASSED] device_oob_was/14026779378
[20:58:10] ============ [PASSED] xe_rtp_table_dev_oob_test ============
[20:58:10] =============== [PASSED] xe_rtp_tables_test ================
[20:58:10] =================== xe_rtp (3 subtests) ====================
[20:58:10] =================== xe_rtp_rules_tests ====================
[20:58:10] [PASSED] no
[20:58:10] [PASSED] yes
[20:58:10] [PASSED] no-and-no
[20:58:10] [PASSED] no-and-yes
[20:58:10] [PASSED] yes-and-no
[20:58:10] [PASSED] yes-and-yes
[20:58:10] [PASSED] no-or-no
[20:58:10] [PASSED] no-or-yes
[20:58:10] [PASSED] yes-or-no
[20:58:10] [PASSED] yes-or-yes
[20:58:10] [PASSED] no-yes-or-yes-no
[20:58:10] [PASSED] no-yes-or-yes-yes
[20:58:10] [PASSED] yes-yes-or-no-yes
[20:58:10] [PASSED] yes-yes-or-yes-yes
[20:58:10] [PASSED] no-no-or-yes-or-no
[20:58:10] [PASSED] or
[20:58:10] [PASSED] or-yes
[20:58:10] [PASSED] or-no
[20:58:10] [PASSED] yes-or
[20:58:10] [PASSED] no-or
[20:58:10] [PASSED] no-or-or-yes
[20:58:10] [PASSED] yes-or-or-no
[20:58:10] [PASSED] no-or-or-no
[20:58:10] [PASSED] missing-context-engine-class
[20:58:10] [PASSED] missing-context-engine-class-or-yes
[20:58:10] [PASSED] missing-context-engine-class-or-or-yes
[20:58:10] =============== [PASSED] xe_rtp_rules_tests ================
[20:58:10] =============== xe_rtp_process_to_sr_tests ================
[20:58:10] [PASSED] coalesce-same-reg
[20:58:10] [PASSED] no-match-no-add
[20:58:10] [PASSED] two-regs-two-entries
[20:58:10] [PASSED] clr-one-set-other
[20:58:10] [PASSED] set-field
[20:58:10] [PASSED] conflict-duplicate
[20:58:10] [PASSED] conflict-not-disjoint
[20:58:10] [PASSED] conflict-reg-type
[20:58:10] [PASSED] bad-mcr-reg-forced-to-regular
[20:58:10] [PASSED] bad-regular-reg-forced-to-mcr
[20:58:10] =========== [PASSED] xe_rtp_process_to_sr_tests ============
[20:58:10] ================== xe_rtp_process_tests ===================
[20:58:10] [PASSED] active1
[20:58:10] [PASSED] active2
[20:58:10] [PASSED] active-inactive
[20:58:10] [PASSED] inactive-active
[20:58:10] [PASSED] inactive-active-inactive
[20:58:10] [PASSED] inactive-inactive-inactive
[20:58:10] ============== [PASSED] xe_rtp_process_tests ===============
[20:58:10] ===================== [PASSED] xe_rtp ======================
[20:58:10] ==================== xe_wa (1 subtest) =====================
[20:58:10] ======================== xe_wa_gt =========================
[20:58:10] [PASSED] TIGERLAKE B0
[20:58:10] [PASSED] DG1 A0
[20:58:10] [PASSED] DG1 B0
[20:58:10] [PASSED] ALDERLAKE_S A0
[20:58:10] [PASSED] ALDERLAKE_S B0
[20:58:10] [PASSED] ALDERLAKE_S C0
[20:58:10] [PASSED] ALDERLAKE_S D0
[20:58:10] [PASSED] ALDERLAKE_P A0
[20:58:10] [PASSED] ALDERLAKE_P B0
[20:58:10] [PASSED] ALDERLAKE_P C0
[20:58:10] [PASSED] ALDERLAKE_S RPLS D0
[20:58:10] [PASSED] ALDERLAKE_P RPLU E0
[20:58:10] [PASSED] DG2 G10 C0
[20:58:10] [PASSED] DG2 G11 B1
[20:58:10] [PASSED] DG2 G12 A1
[20:58:10] [PASSED] METEORLAKE 12.70(Xe_LPG) A0 13.00(Xe_LPM+) A0
[20:58:10] [PASSED] METEORLAKE 12.71(Xe_LPG) A0 13.00(Xe_LPM+) A0
[20:58:10] [PASSED] METEORLAKE 12.74(Xe_LPG+) A0 13.00(Xe_LPM+) A0
[20:58:10] [PASSED] LUNARLAKE 20.04(Xe2_LPG) A0 20.00(Xe2_LPM) A0
[20:58:10] [PASSED] LUNARLAKE 20.04(Xe2_LPG) B0 20.00(Xe2_LPM) A0
[20:58:10] [PASSED] BATTLEMAGE 20.01(Xe2_HPG) A0 13.01(Xe2_HPM) A1
[20:58:10] [PASSED] PANTHERLAKE 30.00(Xe3_LPG) A0 30.00(Xe3_LPM) A0
[20:58:10] ==================== [PASSED] xe_wa_gt =====================
[20:58:10] ====================== [PASSED] xe_wa ======================
[20:58:10] ============================================================
[20:58:10] Testing complete. Ran 716 tests: passed: 698, skipped: 18
[20:58:10] Elapsed time: 66.774s total, 7.314s configuring, 58.793s building, 0.639s running
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/tests/.kunitconfig
[20:58:10] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[20:58:12] 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
In file included from ../drivers/gpu/drm/tests/drm_bridge_test.c:21:
../drivers/gpu/drm/tests/drm_kunit_edid.h:958:28: warning: ‘test_edid_hdmi_4k_rgb_yuv420_dc_max_340mhz’ defined but not used [-Wunused-const-variable=]
958 | static const unsigned char test_edid_hdmi_4k_rgb_yuv420_dc_max_340mhz[] = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../drivers/gpu/drm/tests/drm_kunit_edid.h:726:28: warning: ‘test_edid_hdmi_1080p_rgb_yuv_dc_max_340mhz’ defined but not used [-Wunused-const-variable=]
726 | static const unsigned char test_edid_hdmi_1080p_rgb_yuv_dc_max_340mhz[] = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../drivers/gpu/drm/tests/drm_kunit_edid.h:612:28: warning: ‘test_edid_hdmi_1080p_rgb_yuv_dc_max_200mhz’ defined but not used [-Wunused-const-variable=]
612 | static const unsigned char test_edid_hdmi_1080p_rgb_yuv_dc_max_200mhz[] = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../drivers/gpu/drm/tests/drm_kunit_edid.h:498:28: warning: ‘test_edid_hdmi_1080p_rgb_max_340mhz’ defined but not used [-Wunused-const-variable=]
498 | static const unsigned char test_edid_hdmi_1080p_rgb_max_340mhz[] = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../drivers/gpu/drm/tests/drm_kunit_edid.h:390:28: warning: ‘test_edid_hdmi_1080p_rgb_max_200mhz_hdr’ defined but not used [-Wunused-const-variable=]
390 | static const unsigned char test_edid_hdmi_1080p_rgb_max_200mhz_hdr[] = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../drivers/gpu/drm/tests/drm_kunit_edid.h:271:28: warning: ‘test_edid_hdmi_1080p_rgb_max_200mhz’ defined but not used [-Wunused-const-variable=]
271 | static const unsigned char test_edid_hdmi_1080p_rgb_max_200mhz[] = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../drivers/gpu/drm/tests/drm_kunit_edid.h:163:28: warning: ‘test_edid_hdmi_1080p_rgb_max_100mhz’ defined but not used [-Wunused-const-variable=]
163 | static const unsigned char test_edid_hdmi_1080p_rgb_max_100mhz[] = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../drivers/gpu/drm/tests/drm_kunit_edid.h:57:28: warning: ‘test_edid_dvi_1080p’ defined but not used [-Wunused-const-variable=]
57 | static const unsigned char test_edid_dvi_1080p[] = {
| ^~~~~~~~~~~~~~~~~~~
[20:58:36] Starting KUnit Kernel (1/1)...
[20:58:36] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[20:58:36] ============ drm_test_pick_cmdline (2 subtests) ============
[20:58:36] [PASSED] drm_test_pick_cmdline_res_1920_1080_60
[20:58:36] =============== drm_test_pick_cmdline_named ===============
[20:58:36] [PASSED] NTSC
[20:58:36] [PASSED] NTSC-J
[20:58:36] [PASSED] PAL
[20:58:36] [PASSED] PAL-M
[20:58:36] =========== [PASSED] drm_test_pick_cmdline_named ===========
[20:58:36] ============== [PASSED] drm_test_pick_cmdline ==============
[20:58:36] == drm_test_atomic_get_connector_for_encoder (1 subtest) ===
[20:58:36] [PASSED] drm_test_drm_atomic_get_connector_for_encoder
[20:58:36] ==== [PASSED] drm_test_atomic_get_connector_for_encoder ====
[20:58:36] =========== drm_validate_clone_mode (2 subtests) ===========
[20:58:36] ============== drm_test_check_in_clone_mode ===============
[20:58:36] [PASSED] in_clone_mode
[20:58:36] [PASSED] not_in_clone_mode
[20:58:36] ========== [PASSED] drm_test_check_in_clone_mode ===========
[20:58:36] =============== drm_test_check_valid_clones ===============
[20:58:36] [PASSED] not_in_clone_mode
[20:58:36] [PASSED] valid_clone
[20:58:36] [PASSED] invalid_clone
[20:58:36] =========== [PASSED] drm_test_check_valid_clones ===========
[20:58:36] ============= [PASSED] drm_validate_clone_mode =============
[20:58:36] ============= drm_validate_modeset (1 subtest) =============
[20:58:36] [PASSED] drm_test_check_connector_changed_modeset
[20:58:36] ============== [PASSED] drm_validate_modeset ===============
[20:58:36] ====== drm_test_bridge_get_current_state (2 subtests) ======
[20:58:36] [PASSED] drm_test_drm_bridge_get_current_state_atomic
[20:58:36] [PASSED] drm_test_drm_bridge_get_current_state_legacy
[20:58:36] ======== [PASSED] drm_test_bridge_get_current_state ========
[20:58:36] ====== drm_test_bridge_helper_reset_crtc (4 subtests) ======
[20:58:36] [PASSED] drm_test_drm_bridge_helper_reset_crtc_atomic
[20:58:36] [PASSED] drm_test_drm_bridge_helper_reset_crtc_atomic_disabled
[20:58:36] [PASSED] drm_test_drm_bridge_helper_reset_crtc_legacy
[20:58:36] [PASSED] drm_test_drm_bridge_helper_hdmi_output_bus_fmts
[20:58:36] ======== [PASSED] drm_test_bridge_helper_reset_crtc ========
[20:58:36] ============== drm_bridge_alloc (2 subtests) ===============
[20:58:36] [PASSED] drm_test_drm_bridge_alloc_basic
[20:58:36] [PASSED] drm_test_drm_bridge_alloc_get_put
[20:58:36] ================ [PASSED] drm_bridge_alloc =================
[20:58:36] ============= drm_bridge_bus_fmt (5 subtests) ==============
[20:58:36] [PASSED] drm_test_bridge_rgb_yuv_rgb
[20:58:36] [PASSED] drm_test_bridge_must_convert_to_yuv444
[20:58:36] [PASSED] drm_test_bridge_hdmi_auto_rgb
[20:58:36] [PASSED] drm_test_bridge_auto_first
[20:58:36] [PASSED] drm_test_bridge_rgb_yuv_no_path
[20:58:36] =============== [PASSED] drm_bridge_bus_fmt ================
[20:58:36] ============= drm_cmdline_parser (40 subtests) =============
[20:58:36] [PASSED] drm_test_cmdline_force_d_only
[20:58:36] [PASSED] drm_test_cmdline_force_D_only_dvi
[20:58:36] [PASSED] drm_test_cmdline_force_D_only_hdmi
[20:58:36] [PASSED] drm_test_cmdline_force_D_only_not_digital
[20:58:36] [PASSED] drm_test_cmdline_force_e_only
[20:58:36] [PASSED] drm_test_cmdline_res
[20:58:36] [PASSED] drm_test_cmdline_res_vesa
[20:58:36] [PASSED] drm_test_cmdline_res_vesa_rblank
[20:58:36] [PASSED] drm_test_cmdline_res_rblank
[20:58:36] [PASSED] drm_test_cmdline_res_bpp
[20:58:36] [PASSED] drm_test_cmdline_res_refresh
[20:58:36] [PASSED] drm_test_cmdline_res_bpp_refresh
[20:58:36] [PASSED] drm_test_cmdline_res_bpp_refresh_interlaced
[20:58:36] [PASSED] drm_test_cmdline_res_bpp_refresh_margins
[20:58:36] [PASSED] drm_test_cmdline_res_bpp_refresh_force_off
[20:58:36] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on
[20:58:36] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on_analog
[20:58:36] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on_digital
[20:58:36] [PASSED] drm_test_cmdline_res_bpp_refresh_interlaced_margins_force_on
[20:58:36] [PASSED] drm_test_cmdline_res_margins_force_on
[20:58:36] [PASSED] drm_test_cmdline_res_vesa_margins
[20:58:36] [PASSED] drm_test_cmdline_name
[20:58:36] [PASSED] drm_test_cmdline_name_bpp
[20:58:36] [PASSED] drm_test_cmdline_name_option
[20:58:36] [PASSED] drm_test_cmdline_name_bpp_option
[20:58:36] [PASSED] drm_test_cmdline_rotate_0
[20:58:36] [PASSED] drm_test_cmdline_rotate_90
[20:58:36] [PASSED] drm_test_cmdline_rotate_180
[20:58:36] [PASSED] drm_test_cmdline_rotate_270
[20:58:36] [PASSED] drm_test_cmdline_hmirror
[20:58:36] [PASSED] drm_test_cmdline_vmirror
[20:58:36] [PASSED] drm_test_cmdline_margin_options
[20:58:36] [PASSED] drm_test_cmdline_multiple_options
[20:58:36] [PASSED] drm_test_cmdline_bpp_extra_and_option
[20:58:36] [PASSED] drm_test_cmdline_extra_and_option
[20:58:36] [PASSED] drm_test_cmdline_freestanding_options
[20:58:36] [PASSED] drm_test_cmdline_freestanding_force_e_and_options
[20:58:36] [PASSED] drm_test_cmdline_panel_orientation
[20:58:36] ================ drm_test_cmdline_invalid =================
[20:58:36] [PASSED] margin_only
[20:58:36] [PASSED] interlace_only
[20:58:36] [PASSED] res_missing_x
[20:58:36] [PASSED] res_missing_y
[20:58:36] [PASSED] res_bad_y
[20:58:36] [PASSED] res_missing_y_bpp
[20:58:36] [PASSED] res_bad_bpp
[20:58:36] [PASSED] res_bad_refresh
[20:58:36] [PASSED] res_bpp_refresh_force_on_off
[20:58:36] [PASSED] res_invalid_mode
[20:58:36] [PASSED] res_bpp_wrong_place_mode
[20:58:36] [PASSED] name_bpp_refresh
[20:58:36] [PASSED] name_refresh
[20:58:36] [PASSED] name_refresh_wrong_mode
[20:58:36] [PASSED] name_refresh_invalid_mode
[20:58:36] [PASSED] rotate_multiple
[20:58:36] [PASSED] rotate_invalid_val
[20:58:36] [PASSED] rotate_truncated
[20:58:36] [PASSED] invalid_option
[20:58:36] [PASSED] invalid_tv_option
[20:58:36] [PASSED] truncated_tv_option
[20:58:36] ============ [PASSED] drm_test_cmdline_invalid =============
[20:58:36] =============== drm_test_cmdline_tv_options ===============
[20:58:36] [PASSED] NTSC
[20:58:36] [PASSED] NTSC_443
[20:58:36] [PASSED] NTSC_J
[20:58:36] [PASSED] PAL
[20:58:36] [PASSED] PAL_M
[20:58:36] [PASSED] PAL_N
[20:58:36] [PASSED] SECAM
[20:58:36] [PASSED] MONO_525
[20:58:36] [PASSED] MONO_625
[20:58:36] =========== [PASSED] drm_test_cmdline_tv_options ===========
[20:58:36] =============== [PASSED] drm_cmdline_parser ================
[20:58:36] ========== drmm_connector_hdmi_init (20 subtests) ==========
[20:58:36] [PASSED] drm_test_connector_hdmi_init_valid
[20:58:36] [PASSED] drm_test_connector_hdmi_init_bpc_8
[20:58:36] [PASSED] drm_test_connector_hdmi_init_bpc_10
[20:58:36] [PASSED] drm_test_connector_hdmi_init_bpc_12
[20:58:36] [PASSED] drm_test_connector_hdmi_init_bpc_invalid
[20:58:36] [PASSED] drm_test_connector_hdmi_init_bpc_null
[20:58:36] [PASSED] drm_test_connector_hdmi_init_formats_empty
[20:58:36] [PASSED] drm_test_connector_hdmi_init_formats_no_rgb
[20:58:36] === drm_test_connector_hdmi_init_formats_yuv420_allowed ===
[20:58:36] [PASSED] supported_formats=0x9 yuv420_allowed=1
[20:58:36] [PASSED] supported_formats=0x9 yuv420_allowed=0
[20:58:36] [PASSED] supported_formats=0x5 yuv420_allowed=1
[20:58:36] [PASSED] supported_formats=0x5 yuv420_allowed=0
[20:58:36] === [PASSED] drm_test_connector_hdmi_init_formats_yuv420_allowed ===
[20:58:36] [PASSED] drm_test_connector_hdmi_init_null_ddc
[20:58:36] [PASSED] drm_test_connector_hdmi_init_null_product
[20:58:36] [PASSED] drm_test_connector_hdmi_init_null_vendor
[20:58:36] [PASSED] drm_test_connector_hdmi_init_product_length_exact
[20:58:36] [PASSED] drm_test_connector_hdmi_init_product_length_too_long
[20:58:36] [PASSED] drm_test_connector_hdmi_init_product_valid
[20:58:36] [PASSED] drm_test_connector_hdmi_init_vendor_length_exact
[20:58:36] [PASSED] drm_test_connector_hdmi_init_vendor_length_too_long
[20:58:36] [PASSED] drm_test_connector_hdmi_init_vendor_valid
[20:58:36] ========= drm_test_connector_hdmi_init_type_valid =========
[20:58:36] [PASSED] HDMI-A
[20:58:36] [PASSED] HDMI-B
[20:58:36] ===== [PASSED] drm_test_connector_hdmi_init_type_valid =====
[20:58:36] ======== drm_test_connector_hdmi_init_type_invalid ========
[20:58:36] [PASSED] Unknown
[20:58:36] [PASSED] VGA
[20:58:36] [PASSED] DVI-I
[20:58:36] [PASSED] DVI-D
[20:58:36] [PASSED] DVI-A
[20:58:36] [PASSED] Composite
[20:58:36] [PASSED] SVIDEO
[20:58:36] [PASSED] LVDS
[20:58:36] [PASSED] Component
[20:58:36] [PASSED] DIN
[20:58:36] [PASSED] DP
[20:58:36] [PASSED] TV
[20:58:36] [PASSED] eDP
[20:58:36] [PASSED] Virtual
[20:58:36] [PASSED] DSI
[20:58:36] [PASSED] DPI
[20:58:36] [PASSED] Writeback
[20:58:36] [PASSED] SPI
[20:58:36] [PASSED] USB
[20:58:36] ==== [PASSED] drm_test_connector_hdmi_init_type_invalid ====
[20:58:36] ============ [PASSED] drmm_connector_hdmi_init =============
[20:58:36] ============= drmm_connector_init (3 subtests) =============
[20:58:36] [PASSED] drm_test_drmm_connector_init
[20:58:36] [PASSED] drm_test_drmm_connector_init_null_ddc
[20:58:36] ========= drm_test_drmm_connector_init_type_valid =========
[20:58:36] [PASSED] Unknown
[20:58:36] [PASSED] VGA
[20:58:36] [PASSED] DVI-I
[20:58:36] [PASSED] DVI-D
[20:58:36] [PASSED] DVI-A
[20:58:36] [PASSED] Composite
[20:58:36] [PASSED] SVIDEO
[20:58:36] [PASSED] LVDS
[20:58:36] [PASSED] Component
[20:58:36] [PASSED] DIN
[20:58:36] [PASSED] DP
[20:58:36] [PASSED] HDMI-A
[20:58:36] [PASSED] HDMI-B
[20:58:36] [PASSED] TV
[20:58:36] [PASSED] eDP
[20:58:36] [PASSED] Virtual
[20:58:36] [PASSED] DSI
[20:58:36] [PASSED] DPI
[20:58:36] [PASSED] Writeback
[20:58:36] [PASSED] SPI
[20:58:36] [PASSED] USB
[20:58:36] ===== [PASSED] drm_test_drmm_connector_init_type_valid =====
[20:58:36] =============== [PASSED] drmm_connector_init ===============
[20:58:36] ========= drm_connector_dynamic_init (6 subtests) ==========
[20:58:36] [PASSED] drm_test_drm_connector_dynamic_init
[20:58:36] [PASSED] drm_test_drm_connector_dynamic_init_null_ddc
[20:58:36] [PASSED] drm_test_drm_connector_dynamic_init_not_added
[20:58:36] [PASSED] drm_test_drm_connector_dynamic_init_properties
[20:58:36] ===== drm_test_drm_connector_dynamic_init_type_valid ======
[20:58:36] [PASSED] Unknown
[20:58:36] [PASSED] VGA
[20:58:36] [PASSED] DVI-I
[20:58:36] [PASSED] DVI-D
[20:58:36] [PASSED] DVI-A
[20:58:36] [PASSED] Composite
[20:58:36] [PASSED] SVIDEO
[20:58:36] [PASSED] LVDS
[20:58:36] [PASSED] Component
[20:58:36] [PASSED] DIN
[20:58:36] [PASSED] DP
[20:58:36] [PASSED] HDMI-A
[20:58:36] [PASSED] HDMI-B
[20:58:36] [PASSED] TV
[20:58:36] [PASSED] eDP
[20:58:36] [PASSED] Virtual
[20:58:36] [PASSED] DSI
[20:58:36] [PASSED] DPI
[20:58:36] [PASSED] Writeback
[20:58:36] [PASSED] SPI
[20:58:36] [PASSED] USB
[20:58:36] = [PASSED] drm_test_drm_connector_dynamic_init_type_valid ==
[20:58:36] ======== drm_test_drm_connector_dynamic_init_name =========
[20:58:36] [PASSED] Unknown
[20:58:36] [PASSED] VGA
[20:58:36] [PASSED] DVI-I
[20:58:36] [PASSED] DVI-D
[20:58:36] [PASSED] DVI-A
[20:58:36] [PASSED] Composite
[20:58:36] [PASSED] SVIDEO
[20:58:36] [PASSED] LVDS
[20:58:36] [PASSED] Component
[20:58:36] [PASSED] DIN
[20:58:36] [PASSED] DP
[20:58:36] [PASSED] HDMI-A
[20:58:36] [PASSED] HDMI-B
[20:58:36] [PASSED] TV
[20:58:36] [PASSED] eDP
[20:58:36] [PASSED] Virtual
[20:58:36] [PASSED] DSI
[20:58:36] [PASSED] DPI
[20:58:36] [PASSED] Writeback
[20:58:36] [PASSED] SPI
[20:58:36] [PASSED] USB
[20:58:36] ==== [PASSED] drm_test_drm_connector_dynamic_init_name =====
[20:58:36] =========== [PASSED] drm_connector_dynamic_init ============
[20:58:36] ==== drm_connector_dynamic_register_early (4 subtests) =====
[20:58:36] [PASSED] drm_test_drm_connector_dynamic_register_early_on_list
[20:58:36] [PASSED] drm_test_drm_connector_dynamic_register_early_defer
[20:58:36] [PASSED] drm_test_drm_connector_dynamic_register_early_no_init
[20:58:36] [PASSED] drm_test_drm_connector_dynamic_register_early_no_mode_object
[20:58:36] ====== [PASSED] drm_connector_dynamic_register_early =======
[20:58:36] ======= drm_connector_dynamic_register (7 subtests) ========
[20:58:36] [PASSED] drm_test_drm_connector_dynamic_register_on_list
[20:58:36] [PASSED] drm_test_drm_connector_dynamic_register_no_defer
[20:58:36] [PASSED] drm_test_drm_connector_dynamic_register_no_init
[20:58:36] [PASSED] drm_test_drm_connector_dynamic_register_mode_object
[20:58:36] [PASSED] drm_test_drm_connector_dynamic_register_sysfs
[20:58:36] [PASSED] drm_test_drm_connector_dynamic_register_sysfs_name
[20:58:36] [PASSED] drm_test_drm_connector_dynamic_register_debugfs
[20:58:36] ========= [PASSED] drm_connector_dynamic_register ==========
[20:58:36] = drm_connector_attach_broadcast_rgb_property (2 subtests) =
[20:58:36] [PASSED] drm_test_drm_connector_attach_broadcast_rgb_property
[20:58:36] [PASSED] drm_test_drm_connector_attach_broadcast_rgb_property_hdmi_connector
[20:58:36] === [PASSED] drm_connector_attach_broadcast_rgb_property ===
[20:58:36] ========== drm_get_tv_mode_from_name (2 subtests) ==========
[20:58:36] ========== drm_test_get_tv_mode_from_name_valid ===========
[20:58:36] [PASSED] NTSC
[20:58:36] [PASSED] NTSC-443
[20:58:36] [PASSED] NTSC-J
[20:58:36] [PASSED] PAL
[20:58:36] [PASSED] PAL-M
[20:58:36] [PASSED] PAL-N
[20:58:36] [PASSED] SECAM
[20:58:36] [PASSED] Mono
[20:58:36] ====== [PASSED] drm_test_get_tv_mode_from_name_valid =======
[20:58:36] [PASSED] drm_test_get_tv_mode_from_name_truncated
[20:58:36] ============ [PASSED] drm_get_tv_mode_from_name ============
[20:58:36] = drm_test_connector_hdmi_compute_mode_clock (12 subtests) =
[20:58:36] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb
[20:58:36] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_10bpc
[20:58:36] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_10bpc_vic_1
[20:58:36] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_12bpc
[20:58:36] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_12bpc_vic_1
[20:58:36] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_double
[20:58:36] = drm_test_connector_hdmi_compute_mode_clock_yuv420_valid =
[20:58:36] [PASSED] VIC 96
[20:58:36] [PASSED] VIC 97
[20:58:36] [PASSED] VIC 101
[20:58:36] [PASSED] VIC 102
[20:58:36] [PASSED] VIC 106
[20:58:36] [PASSED] VIC 107
[20:58:36] === [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_valid ===
[20:58:36] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_10_bpc
[20:58:36] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_12_bpc
[20:58:36] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_8_bpc
[20:58:36] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_10_bpc
[20:58:36] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_12_bpc
[20:58:36] === [PASSED] drm_test_connector_hdmi_compute_mode_clock ====
[20:58:36] == drm_hdmi_connector_get_broadcast_rgb_name (2 subtests) ==
[20:58:36] === drm_test_drm_hdmi_connector_get_broadcast_rgb_name ====
[20:58:36] [PASSED] Automatic
[20:58:36] [PASSED] Full
[20:58:36] [PASSED] Limited 16:235
[20:58:36] === [PASSED] drm_test_drm_hdmi_connector_get_broadcast_rgb_name ===
[20:58:36] [PASSED] drm_test_drm_hdmi_connector_get_broadcast_rgb_name_invalid
[20:58:36] ==== [PASSED] drm_hdmi_connector_get_broadcast_rgb_name ====
[20:58:36] == drm_hdmi_connector_get_output_format_name (2 subtests) ==
[20:58:36] === drm_test_drm_hdmi_connector_get_output_format_name ====
[20:58:36] [PASSED] RGB
[20:58:36] [PASSED] YUV 4:2:0
[20:58:36] [PASSED] YUV 4:2:2
[20:58:36] [PASSED] YUV 4:4:4
[20:58:36] === [PASSED] drm_test_drm_hdmi_connector_get_output_format_name ===
[20:58:36] [PASSED] drm_test_drm_hdmi_connector_get_output_format_name_invalid
[20:58:36] ==== [PASSED] drm_hdmi_connector_get_output_format_name ====
[20:58:36] ============= drm_damage_helper (21 subtests) ==============
[20:58:36] [PASSED] drm_test_damage_iter_no_damage
[20:58:36] [PASSED] drm_test_damage_iter_no_damage_fractional_src
[20:58:36] [PASSED] drm_test_damage_iter_no_damage_src_moved
[20:58:36] [PASSED] drm_test_damage_iter_no_damage_fractional_src_moved
[20:58:36] [PASSED] drm_test_damage_iter_no_damage_not_visible
[20:58:36] [PASSED] drm_test_damage_iter_no_damage_no_crtc
[20:58:36] [PASSED] drm_test_damage_iter_no_damage_no_fb
[20:58:36] [PASSED] drm_test_damage_iter_simple_damage
[20:58:36] [PASSED] drm_test_damage_iter_single_damage
[20:58:36] [PASSED] drm_test_damage_iter_single_damage_intersect_src
[20:58:36] [PASSED] drm_test_damage_iter_single_damage_outside_src
[20:58:36] [PASSED] drm_test_damage_iter_single_damage_fractional_src
[20:58:36] [PASSED] drm_test_damage_iter_single_damage_intersect_fractional_src
[20:58:36] [PASSED] drm_test_damage_iter_single_damage_outside_fractional_src
[20:58:36] [PASSED] drm_test_damage_iter_single_damage_src_moved
[20:58:36] [PASSED] drm_test_damage_iter_single_damage_fractional_src_moved
[20:58:36] [PASSED] drm_test_damage_iter_damage
[20:58:36] [PASSED] drm_test_damage_iter_damage_one_intersect
[20:58:36] [PASSED] drm_test_damage_iter_damage_one_outside
[20:58:36] [PASSED] drm_test_damage_iter_damage_src_moved
[20:58:36] [PASSED] drm_test_damage_iter_damage_not_visible
[20:58:36] ================ [PASSED] drm_damage_helper ================
[20:58:36] ============== drm_dp_mst_helper (3 subtests) ==============
[20:58:36] ============== drm_test_dp_mst_calc_pbn_mode ==============
[20:58:36] [PASSED] Clock 154000 BPP 30 DSC disabled
[20:58:36] [PASSED] Clock 234000 BPP 30 DSC disabled
[20:58:36] [PASSED] Clock 297000 BPP 24 DSC disabled
[20:58:36] [PASSED] Clock 332880 BPP 24 DSC enabled
[20:58:36] [PASSED] Clock 324540 BPP 24 DSC enabled
[20:58:36] ========== [PASSED] drm_test_dp_mst_calc_pbn_mode ==========
[20:58:36] ============== drm_test_dp_mst_calc_pbn_div ===============
[20:58:36] [PASSED] Link rate 2000000 lane count 4
[20:58:36] [PASSED] Link rate 2000000 lane count 2
[20:58:36] [PASSED] Link rate 2000000 lane count 1
[20:58:36] [PASSED] Link rate 1350000 lane count 4
[20:58:36] [PASSED] Link rate 1350000 lane count 2
[20:58:36] [PASSED] Link rate 1350000 lane count 1
[20:58:36] [PASSED] Link rate 1000000 lane count 4
[20:58:36] [PASSED] Link rate 1000000 lane count 2
[20:58:36] [PASSED] Link rate 1000000 lane count 1
[20:58:36] [PASSED] Link rate 810000 lane count 4
[20:58:36] [PASSED] Link rate 810000 lane count 2
[20:58:36] [PASSED] Link rate 810000 lane count 1
[20:58:36] [PASSED] Link rate 540000 lane count 4
[20:58:36] [PASSED] Link rate 540000 lane count 2
[20:58:36] [PASSED] Link rate 540000 lane count 1
[20:58:36] [PASSED] Link rate 270000 lane count 4
[20:58:36] [PASSED] Link rate 270000 lane count 2
[20:58:36] [PASSED] Link rate 270000 lane count 1
[20:58:36] [PASSED] Link rate 162000 lane count 4
[20:58:36] [PASSED] Link rate 162000 lane count 2
[20:58:36] [PASSED] Link rate 162000 lane count 1
[20:58:36] ========== [PASSED] drm_test_dp_mst_calc_pbn_div ===========
[20:58:36] ========= drm_test_dp_mst_sideband_msg_req_decode =========
[20:58:36] [PASSED] DP_ENUM_PATH_RESOURCES with port number
[20:58:36] [PASSED] DP_POWER_UP_PHY with port number
[20:58:36] [PASSED] DP_POWER_DOWN_PHY with port number
[20:58:36] [PASSED] DP_ALLOCATE_PAYLOAD with SDP stream sinks
[20:58:36] [PASSED] DP_ALLOCATE_PAYLOAD with port number
[20:58:36] [PASSED] DP_ALLOCATE_PAYLOAD with VCPI
[20:58:36] [PASSED] DP_ALLOCATE_PAYLOAD with PBN
[20:58:36] [PASSED] DP_QUERY_PAYLOAD with port number
[20:58:36] [PASSED] DP_QUERY_PAYLOAD with VCPI
[20:58:36] [PASSED] DP_REMOTE_DPCD_READ with port number
[20:58:36] [PASSED] DP_REMOTE_DPCD_READ with DPCD address
[20:58:36] [PASSED] DP_REMOTE_DPCD_READ with max number of bytes
[20:58:36] [PASSED] DP_REMOTE_DPCD_WRITE with port number
[20:58:36] [PASSED] DP_REMOTE_DPCD_WRITE with DPCD address
[20:58:36] [PASSED] DP_REMOTE_DPCD_WRITE with data array
[20:58:36] [PASSED] DP_REMOTE_I2C_READ with port number
[20:58:36] [PASSED] DP_REMOTE_I2C_READ with I2C device ID
[20:58:36] [PASSED] DP_REMOTE_I2C_READ with transactions array
[20:58:36] [PASSED] DP_REMOTE_I2C_WRITE with port number
[20:58:36] [PASSED] DP_REMOTE_I2C_WRITE with I2C device ID
[20:58:36] [PASSED] DP_REMOTE_I2C_WRITE with data array
[20:58:36] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream ID
[20:58:36] [PASSED] DP_QUERY_STREAM_ENC_STATUS with client ID
[20:58:36] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream event
[20:58:36] [PASSED] DP_QUERY_STREAM_ENC_STATUS with valid stream event
[20:58:36] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream behavior
[20:58:36] [PASSED] DP_QUERY_STREAM_ENC_STATUS with a valid stream behavior
[20:58:36] ===== [PASSED] drm_test_dp_mst_sideband_msg_req_decode =====
[20:58:36] ================ [PASSED] drm_dp_mst_helper ================
[20:58:36] ================== drm_exec (7 subtests) ===================
[20:58:36] [PASSED] sanitycheck
[20:58:36] [PASSED] test_lock
[20:58:36] [PASSED] test_lock_unlock
[20:58:36] [PASSED] test_duplicates
[20:58:36] [PASSED] test_prepare
[20:58:36] [PASSED] test_prepare_array
[20:58:36] [PASSED] test_multiple_loops
[20:58:36] ==================== [PASSED] drm_exec =====================
[20:58:36] =========== drm_format_helper_test (17 subtests) ===========
[20:58:36] ============== drm_test_fb_xrgb8888_to_gray8 ==============
[20:58:36] [PASSED] single_pixel_source_buffer
[20:58:36] [PASSED] single_pixel_clip_rectangle
[20:58:36] [PASSED] well_known_colors
[20:58:36] [PASSED] destination_pitch
[20:58:36] ========== [PASSED] drm_test_fb_xrgb8888_to_gray8 ==========
[20:58:36] ============= drm_test_fb_xrgb8888_to_rgb332 ==============
[20:58:36] [PASSED] single_pixel_source_buffer
[20:58:36] [PASSED] single_pixel_clip_rectangle
[20:58:36] [PASSED] well_known_colors
[20:58:36] [PASSED] destination_pitch
[20:58:36] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb332 ==========
[20:58:36] ============= drm_test_fb_xrgb8888_to_rgb565 ==============
[20:58:36] [PASSED] single_pixel_source_buffer
[20:58:36] [PASSED] single_pixel_clip_rectangle
[20:58:36] [PASSED] well_known_colors
[20:58:36] [PASSED] destination_pitch
[20:58:36] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb565 ==========
[20:58:36] ============ drm_test_fb_xrgb8888_to_xrgb1555 =============
[20:58:36] [PASSED] single_pixel_source_buffer
[20:58:36] [PASSED] single_pixel_clip_rectangle
[20:58:36] [PASSED] well_known_colors
[20:58:36] [PASSED] destination_pitch
[20:58:36] ======== [PASSED] drm_test_fb_xrgb8888_to_xrgb1555 =========
[20:58:36] ============ drm_test_fb_xrgb8888_to_argb1555 =============
[20:58:36] [PASSED] single_pixel_source_buffer
[20:58:36] [PASSED] single_pixel_clip_rectangle
[20:58:36] [PASSED] well_known_colors
[20:58:36] [PASSED] destination_pitch
[20:58:36] ======== [PASSED] drm_test_fb_xrgb8888_to_argb1555 =========
[20:58:36] ============ drm_test_fb_xrgb8888_to_rgba5551 =============
[20:58:36] [PASSED] single_pixel_source_buffer
[20:58:36] [PASSED] single_pixel_clip_rectangle
[20:58:36] [PASSED] well_known_colors
[20:58:36] [PASSED] destination_pitch
[20:58:36] ======== [PASSED] drm_test_fb_xrgb8888_to_rgba5551 =========
[20:58:36] ============= drm_test_fb_xrgb8888_to_rgb888 ==============
[20:58:36] [PASSED] single_pixel_source_buffer
[20:58:36] [PASSED] single_pixel_clip_rectangle
[20:58:36] [PASSED] well_known_colors
[20:58:36] [PASSED] destination_pitch
[20:58:36] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb888 ==========
[20:58:36] ============= drm_test_fb_xrgb8888_to_bgr888 ==============
[20:58:36] [PASSED] single_pixel_source_buffer
[20:58:36] [PASSED] single_pixel_clip_rectangle
[20:58:36] [PASSED] well_known_colors
[20:58:36] [PASSED] destination_pitch
[20:58:36] ========= [PASSED] drm_test_fb_xrgb8888_to_bgr888 ==========
[20:58:36] ============ drm_test_fb_xrgb8888_to_argb8888 =============
[20:58:36] [PASSED] single_pixel_source_buffer
[20:58:36] [PASSED] single_pixel_clip_rectangle
[20:58:36] [PASSED] well_known_colors
[20:58:36] [PASSED] destination_pitch
[20:58:36] ======== [PASSED] drm_test_fb_xrgb8888_to_argb8888 =========
[20:58:36] =========== drm_test_fb_xrgb8888_to_xrgb2101010 ===========
[20:58:36] [PASSED] single_pixel_source_buffer
[20:58:36] [PASSED] single_pixel_clip_rectangle
[20:58:36] [PASSED] well_known_colors
[20:58:36] [PASSED] destination_pitch
[20:58:36] ======= [PASSED] drm_test_fb_xrgb8888_to_xrgb2101010 =======
[20:58:36] =========== drm_test_fb_xrgb8888_to_argb2101010 ===========
[20:58:36] [PASSED] single_pixel_source_buffer
[20:58:36] [PASSED] single_pixel_clip_rectangle
[20:58:36] [PASSED] well_known_colors
[20:58:36] [PASSED] destination_pitch
[20:58:36] ======= [PASSED] drm_test_fb_xrgb8888_to_argb2101010 =======
[20:58:36] ============== drm_test_fb_xrgb8888_to_mono ===============
[20:58:36] [PASSED] single_pixel_source_buffer
[20:58:36] [PASSED] single_pixel_clip_rectangle
[20:58:36] [PASSED] well_known_colors
[20:58:36] [PASSED] destination_pitch
[20:58:36] ========== [PASSED] drm_test_fb_xrgb8888_to_mono ===========
[20:58:36] ==================== drm_test_fb_swab =====================
[20:58:36] [PASSED] single_pixel_source_buffer
[20:58:36] [PASSED] single_pixel_clip_rectangle
[20:58:36] [PASSED] well_known_colors
[20:58:36] [PASSED] destination_pitch
[20:58:36] ================ [PASSED] drm_test_fb_swab =================
[20:58:36] ============ drm_test_fb_xrgb8888_to_xbgr8888 =============
[20:58:36] [PASSED] single_pixel_source_buffer
[20:58:36] [PASSED] single_pixel_clip_rectangle
[20:58:36] [PASSED] well_known_colors
[20:58:36] [PASSED] destination_pitch
[20:58:36] ======== [PASSED] drm_test_fb_xrgb8888_to_xbgr8888 =========
[20:58:36] ============ drm_test_fb_xrgb8888_to_abgr8888 =============
[20:58:36] [PASSED] single_pixel_source_buffer
[20:58:36] [PASSED] single_pixel_clip_rectangle
[20:58:36] [PASSED] well_known_colors
[20:58:36] [PASSED] destination_pitch
[20:58:36] ======== [PASSED] drm_test_fb_xrgb8888_to_abgr8888 =========
[20:58:36] ================= drm_test_fb_clip_offset =================
[20:58:36] [PASSED] pass through
[20:58:36] [PASSED] horizontal offset
[20:58:36] [PASSED] vertical offset
[20:58:36] [PASSED] horizontal and vertical offset
[20:58:36] [PASSED] horizontal offset (custom pitch)
[20:58:36] [PASSED] vertical offset (custom pitch)
[20:58:36] [PASSED] horizontal and vertical offset (custom pitch)
[20:58:36] ============= [PASSED] drm_test_fb_clip_offset =============
[20:58:36] =================== drm_test_fb_memcpy ====================
[20:58:36] [PASSED] single_pixel_source_buffer: XR24 little-endian (0x34325258)
[20:58:36] [PASSED] single_pixel_source_buffer: XRA8 little-endian (0x38415258)
[20:58:36] [PASSED] single_pixel_source_buffer: YU24 little-endian (0x34325559)
[20:58:36] [PASSED] single_pixel_clip_rectangle: XB24 little-endian (0x34324258)
[20:58:36] [PASSED] single_pixel_clip_rectangle: XRA8 little-endian (0x38415258)
[20:58:36] [PASSED] single_pixel_clip_rectangle: YU24 little-endian (0x34325559)
[20:58:36] [PASSED] well_known_colors: XB24 little-endian (0x34324258)
[20:58:36] [PASSED] well_known_colors: XRA8 little-endian (0x38415258)
[20:58:36] [PASSED] well_known_colors: YU24 little-endian (0x34325559)
[20:58:36] [PASSED] destination_pitch: XB24 little-endian (0x34324258)
[20:58:36] [PASSED] destination_pitch: XRA8 little-endian (0x38415258)
[20:58:36] [PASSED] destination_pitch: YU24 little-endian (0x34325559)
[20:58:36] =============== [PASSED] drm_test_fb_memcpy ================
[20:58:36] ============= [PASSED] drm_format_helper_test ==============
[20:58:36] ================= drm_format (18 subtests) =================
[20:58:36] [PASSED] drm_test_format_block_width_invalid
[20:58:36] [PASSED] drm_test_format_block_width_one_plane
[20:58:36] [PASSED] drm_test_format_block_width_two_plane
[20:58:36] [PASSED] drm_test_format_block_width_three_plane
[20:58:36] [PASSED] drm_test_format_block_width_tiled
[20:58:36] [PASSED] drm_test_format_block_height_invalid
[20:58:36] [PASSED] drm_test_format_block_height_one_plane
[20:58:36] [PASSED] drm_test_format_block_height_two_plane
[20:58:36] [PASSED] drm_test_format_block_height_three_plane
[20:58:36] [PASSED] drm_test_format_block_height_tiled
[20:58:36] [PASSED] drm_test_format_min_pitch_invalid
[20:58:36] [PASSED] drm_test_format_min_pitch_one_plane_8bpp
[20:58:36] [PASSED] drm_test_format_min_pitch_one_plane_16bpp
[20:58:36] [PASSED] drm_test_format_min_pitch_one_plane_24bpp
[20:58:36] [PASSED] drm_test_format_min_pitch_one_plane_32bpp
[20:58:36] [PASSED] drm_test_format_min_pitch_two_plane
[20:58:36] [PASSED] drm_test_format_min_pitch_three_plane_8bpp
[20:58:36] [PASSED] drm_test_format_min_pitch_tiled
[20:58:36] =================== [PASSED] drm_format ====================
[20:58:36] ============== drm_framebuffer (10 subtests) ===============
[20:58:36] ========== drm_test_framebuffer_check_src_coords ==========
[20:58:36] [PASSED] Success: source fits into fb
[20:58:36] [PASSED] Fail: overflowing fb with x-axis coordinate
[20:58:36] [PASSED] Fail: overflowing fb with y-axis coordinate
[20:58:36] [PASSED] Fail: overflowing fb with source width
[20:58:36] [PASSED] Fail: overflowing fb with source height
[20:58:36] ====== [PASSED] drm_test_framebuffer_check_src_coords ======
[20:58:36] [PASSED] drm_test_framebuffer_cleanup
[20:58:36] =============== drm_test_framebuffer_create ===============
[20:58:36] [PASSED] ABGR8888 normal sizes
[20:58:36] [PASSED] ABGR8888 max sizes
[20:58:36] [PASSED] ABGR8888 pitch greater than min required
[20:58:36] [PASSED] ABGR8888 pitch less than min required
[20:58:36] [PASSED] ABGR8888 Invalid width
[20:58:36] [PASSED] ABGR8888 Invalid buffer handle
[20:58:36] [PASSED] No pixel format
[20:58:36] [PASSED] ABGR8888 Width 0
[20:58:36] [PASSED] ABGR8888 Height 0
[20:58:36] [PASSED] ABGR8888 Out of bound height * pitch combination
[20:58:36] [PASSED] ABGR8888 Large buffer offset
[20:58:36] [PASSED] ABGR8888 Buffer offset for inexistent plane
[20:58:36] [PASSED] ABGR8888 Invalid flag
[20:58:36] [PASSED] ABGR8888 Set DRM_MODE_FB_MODIFIERS without modifiers
[20:58:36] [PASSED] ABGR8888 Valid buffer modifier
[20:58:36] [PASSED] ABGR8888 Invalid buffer modifier(DRM_FORMAT_MOD_SAMSUNG_64_32_TILE)
[20:58:36] [PASSED] ABGR8888 Extra pitches without DRM_MODE_FB_MODIFIERS
[20:58:36] [PASSED] ABGR8888 Extra pitches with DRM_MODE_FB_MODIFIERS
[20:58:36] [PASSED] NV12 Normal sizes
[20:58:36] [PASSED] NV12 Max sizes
[20:58:36] [PASSED] NV12 Invalid pitch
[20:58:36] [PASSED] NV12 Invalid modifier/missing DRM_MODE_FB_MODIFIERS flag
[20:58:36] [PASSED] NV12 different modifier per-plane
[20:58:36] [PASSED] NV12 with DRM_FORMAT_MOD_SAMSUNG_64_32_TILE
[20:58:36] [PASSED] NV12 Valid modifiers without DRM_MODE_FB_MODIFIERS
[20:58:36] [PASSED] NV12 Modifier for inexistent plane
[20:58:36] [PASSED] NV12 Handle for inexistent plane
[20:58:36] [PASSED] NV12 Handle for inexistent plane without DRM_MODE_FB_MODIFIERS
[20:58:36] [PASSED] YVU420 DRM_MODE_FB_MODIFIERS set without modifier
[20:58:36] [PASSED] YVU420 Normal sizes
[20:58:36] [PASSED] YVU420 Max sizes
[20:58:36] [PASSED] YVU420 Invalid pitch
[20:58:36] [PASSED] YVU420 Different pitches
[20:58:36] [PASSED] YVU420 Different buffer offsets/pitches
[20:58:36] [PASSED] YVU420 Modifier set just for plane 0, without DRM_MODE_FB_MODIFIERS
[20:58:36] [PASSED] YVU420 Modifier set just for planes 0, 1, without DRM_MODE_FB_MODIFIERS
[20:58:36] [PASSED] YVU420 Modifier set just for plane 0, 1, with DRM_MODE_FB_MODIFIERS
[20:58:36] [PASSED] YVU420 Valid modifier
[20:58:36] [PASSED] YVU420 Different modifiers per plane
[20:58:36] [PASSED] YVU420 Modifier for inexistent plane
[20:58:36] [PASSED] YUV420_10BIT Invalid modifier(DRM_FORMAT_MOD_LINEAR)
[20:58:36] [PASSED] X0L2 Normal sizes
[20:58:36] [PASSED] X0L2 Max sizes
[20:58:36] [PASSED] X0L2 Invalid pitch
[20:58:36] [PASSED] X0L2 Pitch greater than minimum required
[20:58:36] [PASSED] X0L2 Handle for inexistent plane
[20:58:36] [PASSED] X0L2 Offset for inexistent plane, without DRM_MODE_FB_MODIFIERS set
[20:58:36] [PASSED] X0L2 Modifier without DRM_MODE_FB_MODIFIERS set
[20:58:36] [PASSED] X0L2 Valid modifier
[20:58:36] [PASSED] X0L2 Modifier for inexistent plane
[20:58:36] =========== [PASSED] drm_test_framebuffer_create ===========
[20:58:36] [PASSED] drm_test_framebuffer_free
[20:58:36] [PASSED] drm_test_framebuffer_init
[20:58:36] [PASSED] drm_test_framebuffer_init_bad_format
[20:58:36] [PASSED] drm_test_framebuffer_init_dev_mismatch
[20:58:36] [PASSED] drm_test_framebuffer_lookup
[20:58:36] [PASSED] drm_test_framebuffer_lookup_inexistent
[20:58:36] [PASSED] drm_test_framebuffer_modifiers_not_supported
[20:58:36] ================= [PASSED] drm_framebuffer =================
[20:58:36] ================ drm_gem_shmem (8 subtests) ================
[20:58:36] [PASSED] drm_gem_shmem_test_obj_create
[20:58:36] [PASSED] drm_gem_shmem_test_obj_create_private
[20:58:36] [PASSED] drm_gem_shmem_test_pin_pages
[20:58:36] [PASSED] drm_gem_shmem_test_vmap
[20:58:36] [PASSED] drm_gem_shmem_test_get_sg_table
[20:58:36] [PASSED] drm_gem_shmem_test_get_pages_sgt
[20:58:36] [PASSED] drm_gem_shmem_test_madvise
[20:58:36] [PASSED] drm_gem_shmem_test_purge
[20:58:36] ================== [PASSED] drm_gem_shmem ==================
[20:58:36] === drm_atomic_helper_connector_hdmi_check (29 subtests) ===
[20:58:36] [PASSED] drm_test_check_broadcast_rgb_auto_cea_mode
[20:58:36] [PASSED] drm_test_check_broadcast_rgb_auto_cea_mode_vic_1
[20:58:36] [PASSED] drm_test_check_broadcast_rgb_full_cea_mode
[20:58:36] [PASSED] drm_test_check_broadcast_rgb_full_cea_mode_vic_1
[20:58:36] [PASSED] drm_test_check_broadcast_rgb_limited_cea_mode
[20:58:36] [PASSED] drm_test_check_broadcast_rgb_limited_cea_mode_vic_1
[20:58:36] ====== drm_test_check_broadcast_rgb_cea_mode_yuv420 =======
[20:58:36] [PASSED] Automatic
[20:58:36] [PASSED] Full
[20:58:36] [PASSED] Limited 16:235
[20:58:36] == [PASSED] drm_test_check_broadcast_rgb_cea_mode_yuv420 ===
[20:58:36] [PASSED] drm_test_check_broadcast_rgb_crtc_mode_changed
[20:58:36] [PASSED] drm_test_check_broadcast_rgb_crtc_mode_not_changed
[20:58:36] [PASSED] drm_test_check_disable_connector
[20:58:36] [PASSED] drm_test_check_hdmi_funcs_reject_rate
[20:58:36] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_rgb
[20:58:36] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_yuv420
[20:58:36] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_ignore_yuv422
[20:58:36] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_ignore_yuv420
[20:58:36] [PASSED] drm_test_check_driver_unsupported_fallback_yuv420
[20:58:36] [PASSED] drm_test_check_output_bpc_crtc_mode_changed
[20:58:36] [PASSED] drm_test_check_output_bpc_crtc_mode_not_changed
[20:58:36] [PASSED] drm_test_check_output_bpc_dvi
[20:58:36] [PASSED] drm_test_check_output_bpc_format_vic_1
[20:58:36] [PASSED] drm_test_check_output_bpc_format_display_8bpc_only
[20:58:36] [PASSED] drm_test_check_output_bpc_format_display_rgb_only
[20:58:36] [PASSED] drm_test_check_output_bpc_format_driver_8bpc_only
[20:58:36] [PASSED] drm_test_check_output_bpc_format_driver_rgb_only
[20:58:36] [PASSED] drm_test_check_tmds_char_rate_rgb_8bpc
[20:58:36] [PASSED] drm_test_check_tmds_char_rate_rgb_10bpc
[20:58:36] [PASSED] drm_test_check_tmds_char_rate_rgb_12bpc
[20:58:36] ============ drm_test_check_hdmi_color_format =============
[20:58:36] [PASSED] AUTO -> RGB
[20:58:36] [PASSED] YCBCR422 -> YUV422
[20:58:36] [PASSED] YCBCR420 -> YUV420
[20:58:36] [PASSED] YCBCR444 -> YUV444
[20:58:36] [PASSED] RGB -> RGB
[20:58:36] ======== [PASSED] drm_test_check_hdmi_color_format =========
[20:58:36] ======== drm_test_check_hdmi_color_format_420_only ========
[20:58:36] [PASSED] RGB should fail
[20:58:36] [PASSED] YUV444 should fail
[20:58:36] [PASSED] YUV422 should fail
[20:58:36] [PASSED] YUV420 should work
[20:58:36] ==== [PASSED] drm_test_check_hdmi_color_format_420_only ====
[20:58:36] ===== [PASSED] drm_atomic_helper_connector_hdmi_check ======
[20:58:36] === drm_atomic_helper_connector_hdmi_reset (6 subtests) ====
[20:58:36] [PASSED] drm_test_check_broadcast_rgb_value
[20:58:36] [PASSED] drm_test_check_bpc_8_value
[20:58:36] [PASSED] drm_test_check_bpc_10_value
[20:58:36] [PASSED] drm_test_check_bpc_12_value
[20:58:36] [PASSED] drm_test_check_format_value
[20:58:36] [PASSED] drm_test_check_tmds_char_value
[20:58:36] ===== [PASSED] drm_atomic_helper_connector_hdmi_reset ======
[20:58:36] = drm_atomic_helper_connector_hdmi_mode_valid (7 subtests) =
[20:58:36] [PASSED] drm_test_check_mode_valid
[20:58:36] [PASSED] drm_test_check_mode_valid_reject
[20:58:36] [PASSED] drm_test_check_mode_valid_reject_rate
[20:58:36] [PASSED] drm_test_check_mode_valid_reject_max_clock
[20:58:36] [PASSED] drm_test_check_mode_valid_yuv420_only_max_clock
[20:58:36] [PASSED] drm_test_check_mode_valid_reject_yuv420_only_connector
[20:58:36] [PASSED] drm_test_check_mode_valid_accept_yuv420_also_connector_rgb
[20:58:36] === [PASSED] drm_atomic_helper_connector_hdmi_mode_valid ===
[20:58:36] = drm_atomic_helper_connector_hdmi_infoframes (5 subtests) =
[20:58:36] [PASSED] drm_test_check_infoframes
[20:58:36] [PASSED] drm_test_check_reject_avi_infoframe
[20:58:36] [PASSED] drm_test_check_reject_hdr_infoframe_bpc_8
[20:58:36] [PASSED] drm_test_check_reject_hdr_infoframe_bpc_10
[20:58:36] [PASSED] drm_test_check_reject_audio_infoframe
[20:58:36] === [PASSED] drm_atomic_helper_connector_hdmi_infoframes ===
[20:58:36] ================= drm_managed (2 subtests) =================
[20:58:36] [PASSED] drm_test_managed_release_action
[20:58:36] [PASSED] drm_test_managed_run_action
[20:58:36] =================== [PASSED] drm_managed ===================
[20:58:36] =================== drm_mm (6 subtests) ====================
[20:58:36] [PASSED] drm_test_mm_init
[20:58:36] [PASSED] drm_test_mm_debug
[20:58:36] [PASSED] drm_test_mm_align32
[20:58:36] [PASSED] drm_test_mm_align64
[20:58:36] [PASSED] drm_test_mm_lowest
[20:58:36] [PASSED] drm_test_mm_highest
[20:58:36] ===================== [PASSED] drm_mm ======================
[20:58:36] ============= drm_modes_analog_tv (5 subtests) =============
[20:58:36] [PASSED] drm_test_modes_analog_tv_mono_576i
[20:58:36] [PASSED] drm_test_modes_analog_tv_ntsc_480i
[20:58:36] [PASSED] drm_test_modes_analog_tv_ntsc_480i_inlined
[20:58:36] [PASSED] drm_test_modes_analog_tv_pal_576i
[20:58:36] [PASSED] drm_test_modes_analog_tv_pal_576i_inlined
[20:58:36] =============== [PASSED] drm_modes_analog_tv ===============
[20:58:36] ============== drm_plane_helper (2 subtests) ===============
[20:58:36] =============== drm_test_check_plane_state ================
[20:58:36] [PASSED] clipping_simple
[20:58:36] [PASSED] clipping_rotate_reflect
[20:58:36] [PASSED] positioning_simple
[20:58:36] [PASSED] upscaling
[20:58:36] [PASSED] downscaling
[20:58:36] [PASSED] rounding1
[20:58:36] [PASSED] rounding2
[20:58:36] [PASSED] rounding3
[20:58:36] [PASSED] rounding4
[20:58:36] =========== [PASSED] drm_test_check_plane_state ============
[20:58:36] =========== drm_test_check_invalid_plane_state ============
[20:58:36] [PASSED] positioning_invalid
[20:58:36] [PASSED] upscaling_invalid
[20:58:36] [PASSED] downscaling_invalid
[20:58:36] ======= [PASSED] drm_test_check_invalid_plane_state ========
[20:58:36] ================ [PASSED] drm_plane_helper =================
[20:58:36] ====== drm_connector_helper_tv_get_modes (1 subtest) =======
[20:58:36] ====== drm_test_connector_helper_tv_get_modes_check =======
[20:58:36] [PASSED] None
[20:58:36] [PASSED] PAL
[20:58:36] [PASSED] NTSC
[20:58:36] [PASSED] Both, NTSC Default
[20:58:36] [PASSED] Both, PAL Default
[20:58:36] [PASSED] Both, NTSC Default, with PAL on command-line
[20:58:36] [PASSED] Both, PAL Default, with NTSC on command-line
[20:58:36] == [PASSED] drm_test_connector_helper_tv_get_modes_check ===
[20:58:36] ======== [PASSED] drm_connector_helper_tv_get_modes ========
[20:58:36] ================== drm_rect (9 subtests) ===================
[20:58:36] [PASSED] drm_test_rect_clip_scaled_div_by_zero
[20:58:36] [PASSED] drm_test_rect_clip_scaled_not_clipped
[20:58:36] [PASSED] drm_test_rect_clip_scaled_clipped
[20:58:36] [PASSED] drm_test_rect_clip_scaled_signed_vs_unsigned
[20:58:36] ================= drm_test_rect_intersect =================
[20:58:36] [PASSED] top-left x bottom-right: 2x2+1+1 x 2x2+0+0
[20:58:36] [PASSED] top-right x bottom-left: 2x2+0+0 x 2x2+1-1
[20:58:36] [PASSED] bottom-left x top-right: 2x2+1-1 x 2x2+0+0
[20:58:36] [PASSED] bottom-right x top-left: 2x2+0+0 x 2x2+1+1
[20:58:36] [PASSED] right x left: 2x1+0+0 x 3x1+1+0
[20:58:36] [PASSED] left x right: 3x1+1+0 x 2x1+0+0
[20:58:36] [PASSED] up x bottom: 1x2+0+0 x 1x3+0-1
[20:58:36] [PASSED] bottom x up: 1x3+0-1 x 1x2+0+0
[20:58:36] [PASSED] touching corner: 1x1+0+0 x 2x2+1+1
[20:58:36] [PASSED] touching side: 1x1+0+0 x 1x1+1+0
[20:58:36] [PASSED] equal rects: 2x2+0+0 x 2x2+0+0
[20:58:36] [PASSED] inside another: 2x2+0+0 x 1x1+1+1
[20:58:36] [PASSED] far away: 1x1+0+0 x 1x1+3+6
[20:58:36] [PASSED] points intersecting: 0x0+5+10 x 0x0+5+10
[20:58:36] [PASSED] points not intersecting: 0x0+0+0 x 0x0+5+10
[20:58:36] ============= [PASSED] drm_test_rect_intersect =============
[20:58:36] ================ drm_test_rect_calc_hscale ================
[20:58:36] [PASSED] normal use
[20:58:36] [PASSED] out of max range
[20:58:36] [PASSED] out of min range
[20:58:36] [PASSED] zero dst
[20:58:36] [PASSED] negative src
[20:58:36] [PASSED] negative dst
[20:58:36] ============ [PASSED] drm_test_rect_calc_hscale ============
[20:58:36] ================ drm_test_rect_calc_vscale ================
[20:58:36] [PASSED] normal use
[20:58:36] [PASSED] out of max range
[20:58:36] [PASSED] out of min range
[20:58:36] [PASSED] zero dst
[20:58:36] [PASSED] negative src
[20:58:36] [PASSED] negative dst
[20:58:36] ============ [PASSED] drm_test_rect_calc_vscale ============
[20:58:36] ================== drm_test_rect_rotate ===================
[20:58:36] [PASSED] reflect-x
[20:58:36] [PASSED] reflect-y
[20:58:36] [PASSED] rotate-0
[20:58:36] [PASSED] rotate-90
[20:58:36] [PASSED] rotate-180
[20:58:36] [PASSED] rotate-270
[20:58:36] ============== [PASSED] drm_test_rect_rotate ===============
[20:58:36] ================ drm_test_rect_rotate_inv =================
[20:58:36] [PASSED] reflect-x
[20:58:36] [PASSED] reflect-y
[20:58:36] [PASSED] rotate-0
[20:58:36] [PASSED] rotate-90
[20:58:36] [PASSED] rotate-180
[20:58:36] [PASSED] rotate-270
[20:58:36] ============ [PASSED] drm_test_rect_rotate_inv =============
[20:58:36] ==================== [PASSED] drm_rect =====================
[20:58:36] ============ drm_sysfb_modeset_test (1 subtest) ============
[20:58:36] ============ drm_test_sysfb_build_fourcc_list =============
[20:58:36] [PASSED] no native formats
[20:58:36] [PASSED] XRGB8888 as native format
[20:58:36] [PASSED] remove duplicates
[20:58:36] [PASSED] convert alpha formats
[20:58:36] [PASSED] random formats
[20:58:36] ======== [PASSED] drm_test_sysfb_build_fourcc_list =========
[20:58:36] ============= [PASSED] drm_sysfb_modeset_test ==============
[20:58:36] ================== drm_fixp (2 subtests) ===================
[20:58:36] [PASSED] drm_test_int2fixp
[20:58:36] [PASSED] drm_test_sm2fixp
[20:58:36] ==================== [PASSED] drm_fixp =====================
[20:58:36] ============================================================
[20:58:36] Testing complete. Ran 639 tests: passed: 639
[20:58:36] Elapsed time: 26.497s total, 1.723s configuring, 24.609s building, 0.145s running
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/ttm/tests/.kunitconfig
[20:58:37] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[20:58:38] 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
[20:58:48] Starting KUnit Kernel (1/1)...
[20:58:48] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[20:58:48] ================= ttm_device (5 subtests) ==================
[20:58:48] [PASSED] ttm_device_init_basic
[20:58:48] [PASSED] ttm_device_init_multiple
[20:58:48] [PASSED] ttm_device_fini_basic
[20:58:48] [PASSED] ttm_device_init_no_vma_man
[20:58:48] ================== ttm_device_init_pools ==================
[20:58:48] [PASSED] No DMA allocations, no DMA32 required
[20:58:48] [PASSED] DMA allocations, DMA32 required
[20:58:48] [PASSED] No DMA allocations, DMA32 required
[20:58:48] [PASSED] DMA allocations, no DMA32 required
[20:58:48] ============== [PASSED] ttm_device_init_pools ==============
[20:58:48] =================== [PASSED] ttm_device ====================
[20:58:48] ================== ttm_pool (8 subtests) ===================
[20:58:48] ================== ttm_pool_alloc_basic ===================
[20:58:48] [PASSED] One page
[20:58:48] [PASSED] More than one page
[20:58:48] [PASSED] Above the allocation limit
[20:58:48] [PASSED] One page, with coherent DMA mappings enabled
[20:58:48] [PASSED] Above the allocation limit, with coherent DMA mappings enabled
[20:58:48] ============== [PASSED] ttm_pool_alloc_basic ===============
[20:58:48] ============== ttm_pool_alloc_basic_dma_addr ==============
[20:58:48] [PASSED] One page
[20:58:48] [PASSED] More than one page
[20:58:48] [PASSED] Above the allocation limit
[20:58:48] [PASSED] One page, with coherent DMA mappings enabled
[20:58:48] [PASSED] Above the allocation limit, with coherent DMA mappings enabled
[20:58:48] ========== [PASSED] ttm_pool_alloc_basic_dma_addr ==========
[20:58:48] [PASSED] ttm_pool_alloc_order_caching_match
[20:58:48] [PASSED] ttm_pool_alloc_caching_mismatch
[20:58:48] [PASSED] ttm_pool_alloc_order_mismatch
[20:58:48] [PASSED] ttm_pool_free_dma_alloc
[20:58:48] [PASSED] ttm_pool_free_no_dma_alloc
[20:58:48] [PASSED] ttm_pool_fini_basic
[20:58:48] ==================== [PASSED] ttm_pool =====================
[20:58:48] ================ ttm_resource (8 subtests) =================
[20:58:48] ================= ttm_resource_init_basic =================
[20:58:48] [PASSED] Init resource in TTM_PL_SYSTEM
[20:58:48] [PASSED] Init resource in TTM_PL_VRAM
[20:58:48] [PASSED] Init resource in a private placement
[20:58:48] [PASSED] Init resource in TTM_PL_SYSTEM, set placement flags
[20:58:48] ============= [PASSED] ttm_resource_init_basic =============
[20:58:48] [PASSED] ttm_resource_init_pinned
[20:58:48] [PASSED] ttm_resource_fini_basic
[20:58:48] [PASSED] ttm_resource_manager_init_basic
[20:58:48] [PASSED] ttm_resource_manager_usage_basic
[20:58:48] [PASSED] ttm_resource_manager_set_used_basic
[20:58:48] [PASSED] ttm_sys_man_alloc_basic
[20:58:48] [PASSED] ttm_sys_man_free_basic
[20:58:48] ================== [PASSED] ttm_resource ===================
[20:58:48] =================== ttm_tt (15 subtests) ===================
[20:58:48] ==================== ttm_tt_init_basic ====================
[20:58:48] [PASSED] Page-aligned size
[20:58:48] [PASSED] Extra pages requested
[20:58:48] ================ [PASSED] ttm_tt_init_basic ================
[20:58:48] [PASSED] ttm_tt_init_misaligned
[20:58:48] [PASSED] ttm_tt_fini_basic
[20:58:48] [PASSED] ttm_tt_fini_sg
[20:58:48] [PASSED] ttm_tt_fini_shmem
[20:58:48] [PASSED] ttm_tt_create_basic
[20:58:48] [PASSED] ttm_tt_create_invalid_bo_type
[20:58:48] [PASSED] ttm_tt_create_ttm_exists
[20:58:48] [PASSED] ttm_tt_create_failed
[20:58:48] [PASSED] ttm_tt_destroy_basic
[20:58:48] [PASSED] ttm_tt_populate_null_ttm
[20:58:48] [PASSED] ttm_tt_populate_populated_ttm
[20:58:48] [PASSED] ttm_tt_unpopulate_basic
[20:58:48] [PASSED] ttm_tt_unpopulate_empty_ttm
[20:58:48] [PASSED] ttm_tt_swapin_basic
[20:58:48] ===================== [PASSED] ttm_tt ======================
[20:58:48] =================== ttm_bo (14 subtests) ===================
[20:58:48] =========== ttm_bo_reserve_optimistic_no_ticket ===========
[20:58:48] [PASSED] Cannot be interrupted and sleeps
[20:58:48] [PASSED] Cannot be interrupted, locks straight away
[20:58:48] [PASSED] Can be interrupted, sleeps
[20:58:48] ======= [PASSED] ttm_bo_reserve_optimistic_no_ticket =======
[20:58:48] [PASSED] ttm_bo_reserve_locked_no_sleep
[20:58:48] [PASSED] ttm_bo_reserve_no_wait_ticket
[20:58:48] [PASSED] ttm_bo_reserve_double_resv
[20:58:48] [PASSED] ttm_bo_reserve_interrupted
[20:58:48] [PASSED] ttm_bo_reserve_deadlock
[20:58:48] [PASSED] ttm_bo_unreserve_basic
[20:58:48] [PASSED] ttm_bo_unreserve_pinned
[20:58:48] [PASSED] ttm_bo_unreserve_bulk
[20:58:48] [PASSED] ttm_bo_fini_basic
[20:58:48] [PASSED] ttm_bo_fini_shared_resv
[20:58:48] [PASSED] ttm_bo_pin_basic
[20:58:48] [PASSED] ttm_bo_pin_unpin_resource
[20:58:48] [PASSED] ttm_bo_multiple_pin_one_unpin
[20:58:48] ===================== [PASSED] ttm_bo ======================
[20:58:48] ============== ttm_bo_validate (22 subtests) ===============
[20:58:48] ============== ttm_bo_init_reserved_sys_man ===============
[20:58:48] [PASSED] Buffer object for userspace
[20:58:48] [PASSED] Kernel buffer object
[20:58:48] [PASSED] Shared buffer object
[20:58:48] ========== [PASSED] ttm_bo_init_reserved_sys_man ===========
[20:58:48] ============== ttm_bo_init_reserved_mock_man ==============
[20:58:48] [PASSED] Buffer object for userspace
[20:58:48] [PASSED] Kernel buffer object
[20:58:48] [PASSED] Shared buffer object
[20:58:48] ========== [PASSED] ttm_bo_init_reserved_mock_man ==========
[20:58:48] [PASSED] ttm_bo_init_reserved_resv
[20:58:48] ================== ttm_bo_validate_basic ==================
[20:58:48] [PASSED] Buffer object for userspace
[20:58:48] [PASSED] Kernel buffer object
[20:58:48] [PASSED] Shared buffer object
[20:58:48] ============== [PASSED] ttm_bo_validate_basic ==============
[20:58:48] [PASSED] ttm_bo_validate_invalid_placement
[20:58:48] ============= ttm_bo_validate_same_placement ==============
[20:58:48] [PASSED] System manager
[20:58:48] [PASSED] VRAM manager
[20:58:48] ========= [PASSED] ttm_bo_validate_same_placement ==========
[20:58:48] [PASSED] ttm_bo_validate_failed_alloc
[20:58:48] [PASSED] ttm_bo_validate_pinned
[20:58:48] [PASSED] ttm_bo_validate_busy_placement
[20:58:48] ================ ttm_bo_validate_multihop =================
[20:58:48] [PASSED] Buffer object for userspace
[20:58:48] [PASSED] Kernel buffer object
[20:58:48] [PASSED] Shared buffer object
[20:58:48] ============ [PASSED] ttm_bo_validate_multihop =============
[20:58:48] ========== ttm_bo_validate_no_placement_signaled ==========
[20:58:48] [PASSED] Buffer object in system domain, no page vector
[20:58:48] [PASSED] Buffer object in system domain with an existing page vector
[20:58:48] ====== [PASSED] ttm_bo_validate_no_placement_signaled ======
[20:58:48] ======== ttm_bo_validate_no_placement_not_signaled ========
[20:58:48] [PASSED] Buffer object for userspace
[20:58:48] [PASSED] Kernel buffer object
[20:58:48] [PASSED] Shared buffer object
[20:58:48] ==== [PASSED] ttm_bo_validate_no_placement_not_signaled ====
[20:58:48] [PASSED] ttm_bo_validate_move_fence_signaled
[20:58:48] ========= ttm_bo_validate_move_fence_not_signaled =========
[20:58:48] [PASSED] Waits for GPU
[20:58:48] [PASSED] Tries to lock straight away
[20:58:48] ===== [PASSED] ttm_bo_validate_move_fence_not_signaled =====
[20:58:48] [PASSED] ttm_bo_validate_swapout
[20:58:48] [PASSED] ttm_bo_validate_happy_evict
[20:58:48] [PASSED] ttm_bo_validate_all_pinned_evict
[20:58:48] [PASSED] ttm_bo_validate_allowed_only_evict
[20:58:48] [PASSED] ttm_bo_validate_deleted_evict
[20:58:48] [PASSED] ttm_bo_validate_busy_domain_evict
[20:58:48] [PASSED] ttm_bo_validate_evict_gutting
[20:58:48] [PASSED] ttm_bo_validate_recrusive_evict
[20:58:48] ================= [PASSED] ttm_bo_validate =================
[20:58:48] ============================================================
[20:58:48] Testing complete. Ran 102 tests: passed: 102
[20:58:48] Elapsed time: 11.673s total, 1.743s configuring, 9.665s building, 0.224s running
+ cleanup
++ stat -c %u:%g /kernel
+ chown -R 1003:1003 /kernel
^ permalink raw reply [flat|nested] 24+ messages in thread
* ✓ Xe.CI.BAT: success for drm/xe: Probe info outside of xe_info_init() and xe_info_init_early()
2026-06-09 20:17 [PATCH 0/9] drm/xe: Probe info outside of xe_info_init() and xe_info_init_early() Gustavo Sousa
` (9 preceding siblings ...)
2026-06-09 20:58 ` ✓ CI.KUnit: success for drm/xe: Probe info outside of xe_info_init() and xe_info_init_early() Patchwork
@ 2026-06-09 22:06 ` Patchwork
2026-06-10 12:22 ` ✗ Xe.CI.FULL: failure " Patchwork
11 siblings, 0 replies; 24+ messages in thread
From: Patchwork @ 2026-06-09 22:06 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-xe
[-- Attachment #1: Type: text/plain, Size: 1738 bytes --]
== Series Details ==
Series: drm/xe: Probe info outside of xe_info_init() and xe_info_init_early()
URL : https://patchwork.freedesktop.org/series/168205/
State : success
== Summary ==
CI Bug Log - changes from xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a_BAT -> xe-pw-168205v1_BAT
====================================================
Summary
-------
**SUCCESS**
No regressions found.
Participating hosts (13 -> 13)
------------------------------
No changes in participating hosts
Known issues
------------
Here are the changes found in xe-pw-168205v1_BAT that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@xe_waitfence@reltime:
- bat-bmg-1: [PASS][1] -> [FAIL][2] ([Intel XE#8017])
[1]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a/bat-bmg-1/igt@xe_waitfence@reltime.html
[2]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/bat-bmg-1/igt@xe_waitfence@reltime.html
- bat-bmg-2: [PASS][3] -> [FAIL][4] ([Intel XE#8017])
[3]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a/bat-bmg-2/igt@xe_waitfence@reltime.html
[4]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/bat-bmg-2/igt@xe_waitfence@reltime.html
[Intel XE#8017]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8017
Build changes
-------------
* Linux: xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a -> xe-pw-168205v1
IGT_8956: 8956
xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a: 3806de1666bcca0525d737e3849c602cbe98fb1a
xe-pw-168205v1: 168205v1
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/index.html
[-- Attachment #2: Type: text/html, Size: 2380 bytes --]
^ permalink raw reply [flat|nested] 24+ messages in thread
* ✗ Xe.CI.FULL: failure for drm/xe: Probe info outside of xe_info_init() and xe_info_init_early()
2026-06-09 20:17 [PATCH 0/9] drm/xe: Probe info outside of xe_info_init() and xe_info_init_early() Gustavo Sousa
` (10 preceding siblings ...)
2026-06-09 22:06 ` ✓ Xe.CI.BAT: " Patchwork
@ 2026-06-10 12:22 ` Patchwork
11 siblings, 0 replies; 24+ messages in thread
From: Patchwork @ 2026-06-10 12:22 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-xe
[-- Attachment #1: Type: text/plain, Size: 16874 bytes --]
== Series Details ==
Series: drm/xe: Probe info outside of xe_info_init() and xe_info_init_early()
URL : https://patchwork.freedesktop.org/series/168205/
State : failure
== Summary ==
CI Bug Log - changes from xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a_FULL -> xe-pw-168205v1_FULL
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with xe-pw-168205v1_FULL absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in xe-pw-168205v1_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-168205v1_FULL:
### IGT changes ###
#### Possible regressions ####
* igt@kms_vblank@ts-continuation-dpms-suspend:
- shard-bmg: NOTRUN -> [INCOMPLETE][1] +1 other test incomplete
[1]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-bmg-8/igt@kms_vblank@ts-continuation-dpms-suspend.html
Known issues
------------
Here are the changes found in xe-pw-168205v1_FULL that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-180-async-flip:
- shard-lnl: NOTRUN -> [SKIP][2] ([Intel XE#1124])
[2]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-180-async-flip.html
* igt@kms_chamelium_hpd@dp-hpd-with-enabled-mode:
- shard-lnl: NOTRUN -> [SKIP][3] ([Intel XE#373])
[3]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@kms_chamelium_hpd@dp-hpd-with-enabled-mode.html
* igt@kms_content_protection@dp-mst-type-1-suspend-resume:
- shard-lnl: NOTRUN -> [SKIP][4] ([Intel XE#6974])
[4]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@kms_content_protection@dp-mst-type-1-suspend-resume.html
* igt@kms_feature_discovery@display-4x:
- shard-lnl: NOTRUN -> [SKIP][5] ([Intel XE#1138] / [Intel XE#7344])
[5]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@kms_feature_discovery@display-4x.html
* igt@kms_flip@2x-flip-vs-dpms-off-vs-modeset-interruptible:
- shard-lnl: NOTRUN -> [SKIP][6] ([Intel XE#1421])
[6]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@kms_flip@2x-flip-vs-dpms-off-vs-modeset-interruptible.html
* igt@kms_frontbuffer_tracking@drrs-2p-scndscrn-cur-indfb-draw-blt:
- shard-lnl: NOTRUN -> [SKIP][7] ([Intel XE#656] / [Intel XE#7905])
[7]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@kms_frontbuffer_tracking@drrs-2p-scndscrn-cur-indfb-draw-blt.html
* igt@kms_frontbuffer_tracking@fbc-abgr161616f-draw-render:
- shard-lnl: NOTRUN -> [SKIP][8] ([Intel XE#7061] / [Intel XE#7356])
[8]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@kms_frontbuffer_tracking@fbc-abgr161616f-draw-render.html
* igt@kms_frontbuffer_tracking@fbcdrrshdr-1p-primscrn-shrfb-plflip-blt:
- shard-lnl: NOTRUN -> [SKIP][9] ([Intel XE#6312])
[9]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@kms_frontbuffer_tracking@fbcdrrshdr-1p-primscrn-shrfb-plflip-blt.html
* igt@kms_frontbuffer_tracking@fbcpsrhdr-2p-primscrn-cur-indfb-draw-mmap-wc:
- shard-bmg: NOTRUN -> [SKIP][10] ([Intel XE#2313]) +2 other tests skip
[10]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-bmg-8/igt@kms_frontbuffer_tracking@fbcpsrhdr-2p-primscrn-cur-indfb-draw-mmap-wc.html
* igt@kms_frontbuffer_tracking@hdr-2p-primscrn-shrfb-pgflip-blt:
- shard-lnl: NOTRUN -> [SKIP][11] ([Intel XE#7905]) +4 other tests skip
[11]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@kms_frontbuffer_tracking@hdr-2p-primscrn-shrfb-pgflip-blt.html
* igt@kms_frontbuffer_tracking@psrhdr-abgr161616f-draw-render:
- shard-lnl: NOTRUN -> [SKIP][12] ([Intel XE#7061])
[12]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@kms_frontbuffer_tracking@psrhdr-abgr161616f-draw-render.html
* igt@kms_hdmi_inject@inject-4k:
- shard-lnl: NOTRUN -> [SKIP][13] ([Intel XE#1470])
[13]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@kms_hdmi_inject@inject-4k.html
* igt@kms_hdr@static-swap@pipe-a-hdmi-a-3-xrgb2101010:
- shard-bmg: [PASS][14] -> [SKIP][15] ([Intel XE#7915]) +3 other tests skip
[14]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a/shard-bmg-5/igt@kms_hdr@static-swap@pipe-a-hdmi-a-3-xrgb2101010.html
[15]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-bmg-6/igt@kms_hdr@static-swap@pipe-a-hdmi-a-3-xrgb2101010.html
* igt@kms_plane@pixel-format-4-tiled-mtl-rc-ccs-cc-modifier-source-clamping:
- shard-lnl: NOTRUN -> [SKIP][16] ([Intel XE#7283])
[16]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@kms_plane@pixel-format-4-tiled-mtl-rc-ccs-cc-modifier-source-clamping.html
* igt@kms_plane_scaling@planes-downscale-factor-0-5-upscale-20x20@pipe-b:
- shard-lnl: NOTRUN -> [SKIP][17] ([Intel XE#2763] / [Intel XE#6886]) +3 other tests skip
[17]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@kms_plane_scaling@planes-downscale-factor-0-5-upscale-20x20@pipe-b.html
* igt@kms_pm_dc@dc5-psr:
- shard-lnl: [PASS][18] -> [FAIL][19] ([Intel XE#7340])
[18]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a/shard-lnl-6/igt@kms_pm_dc@dc5-psr.html
[19]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-2/igt@kms_pm_dc@dc5-psr.html
* igt@xe_eudebug@basic-vm-bind-discovery:
- shard-lnl: NOTRUN -> [SKIP][20] ([Intel XE#7636]) +2 other tests skip
[20]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@xe_eudebug@basic-vm-bind-discovery.html
* igt@xe_evict@evict-large-cm:
- shard-lnl: NOTRUN -> [SKIP][21] ([Intel XE#6540] / [Intel XE#688]) +1 other test skip
[21]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@xe_evict@evict-large-cm.html
* igt@xe_evict@evict-mixed-many-threads-small:
- shard-bmg: [PASS][22] -> [INCOMPLETE][23] ([Intel XE#6321])
[22]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a/shard-bmg-7/igt@xe_evict@evict-mixed-many-threads-small.html
[23]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-bmg-9/igt@xe_evict@evict-mixed-many-threads-small.html
* igt@xe_exec_balancer@many-cm-parallel-userptr-invalidate:
- shard-lnl: NOTRUN -> [SKIP][24] ([Intel XE#7482]) +2 other tests skip
[24]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@xe_exec_balancer@many-cm-parallel-userptr-invalidate.html
* igt@xe_exec_basic@multigpu-many-execqueues-many-vm-rebind:
- shard-lnl: NOTRUN -> [SKIP][25] ([Intel XE#1392])
[25]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@xe_exec_basic@multigpu-many-execqueues-many-vm-rebind.html
* igt@xe_exec_fault_mode@many-multi-queue:
- shard-lnl: NOTRUN -> [SKIP][26] ([Intel XE#7136])
[26]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@xe_exec_fault_mode@many-multi-queue.html
* igt@xe_exec_fault_mode@many-multi-queue-userptr-invalidate-race-prefetch:
- shard-bmg: NOTRUN -> [SKIP][27] ([Intel XE#7136])
[27]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-bmg-8/igt@xe_exec_fault_mode@many-multi-queue-userptr-invalidate-race-prefetch.html
* igt@xe_exec_multi_queue@many-execs-basic-smem:
- shard-lnl: NOTRUN -> [SKIP][28] ([Intel XE#6874])
[28]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@xe_exec_multi_queue@many-execs-basic-smem.html
* igt@xe_exec_multi_queue@one-queue-preempt-mode-priority-smem:
- shard-bmg: NOTRUN -> [ABORT][29] ([Intel XE#8007])
[29]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-bmg-1/igt@xe_exec_multi_queue@one-queue-preempt-mode-priority-smem.html
* igt@xe_exec_reset@long-spin-reuse-many-preempt-gt0-threads:
- shard-bmg: [PASS][30] -> [FAIL][31] ([Intel XE#7850])
[30]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a/shard-bmg-10/igt@xe_exec_reset@long-spin-reuse-many-preempt-gt0-threads.html
[31]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-bmg-9/igt@xe_exec_reset@long-spin-reuse-many-preempt-gt0-threads.html
* igt@xe_multigpu_svm@mgpu-pagefault-conflict:
- shard-lnl: NOTRUN -> [SKIP][32] ([Intel XE#6964])
[32]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@xe_multigpu_svm@mgpu-pagefault-conflict.html
* igt@xe_page_reclaim@binds-null-vma:
- shard-lnl: NOTRUN -> [SKIP][33] ([Intel XE#7793])
[33]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@xe_page_reclaim@binds-null-vma.html
* igt@xe_sriov_auto_provisioning@exclusive-ranges:
- shard-lnl: NOTRUN -> [SKIP][34] ([Intel XE#4130] / [Intel XE#7366])
[34]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@xe_sriov_auto_provisioning@exclusive-ranges.html
* igt@xe_sriov_scheduling@nonpreempt-engine-resets:
- shard-lnl: NOTRUN -> [SKIP][35] ([Intel XE#4351] / [Intel XE#7357])
[35]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@xe_sriov_scheduling@nonpreempt-engine-resets.html
#### Possible fixes ####
* igt@kms_flip@flip-vs-expired-vblank-interruptible:
- shard-bmg: [FAIL][36] ([Intel XE#3321]) -> [PASS][37] +1 other test pass
[36]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a/shard-bmg-7/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
[37]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-bmg-9/igt@kms_flip@flip-vs-expired-vblank-interruptible.html
* igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1:
- shard-lnl: [FAIL][38] ([Intel XE#301]) -> [PASS][39]
[38]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a/shard-lnl-4/igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1.html
[39]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-6/igt@kms_flip@flip-vs-expired-vblank-interruptible@b-edp1.html
* igt@kms_flip@flip-vs-expired-vblank@c-edp1:
- shard-lnl: [FAIL][40] ([Intel XE#301] / [Intel XE#3149]) -> [PASS][41]
[40]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a/shard-lnl-6/igt@kms_flip@flip-vs-expired-vblank@c-edp1.html
[41]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-2/igt@kms_flip@flip-vs-expired-vblank@c-edp1.html
* igt@kms_hdr@invalid-hdr:
- shard-bmg: [SKIP][42] ([Intel XE#1503]) -> [PASS][43]
[42]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a/shard-bmg-5/igt@kms_hdr@invalid-hdr.html
[43]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-bmg-6/igt@kms_hdr@invalid-hdr.html
* igt@kms_hdr@invalid-hdr@pipe-a-hdmi-a-3-xrgb2101010:
- shard-bmg: [SKIP][44] ([Intel XE#7922]) -> [PASS][45] +1 other test pass
[44]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a/shard-bmg-5/igt@kms_hdr@invalid-hdr@pipe-a-hdmi-a-3-xrgb2101010.html
[45]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-bmg-6/igt@kms_hdr@invalid-hdr@pipe-a-hdmi-a-3-xrgb2101010.html
* igt@xe_wedged@wedged-mode-toggle:
- shard-bmg: [ABORT][46] ([Intel XE#8007]) -> [PASS][47]
[46]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a/shard-bmg-3/igt@xe_wedged@wedged-mode-toggle.html
[47]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-bmg-1/igt@xe_wedged@wedged-mode-toggle.html
#### Warnings ####
* igt@kms_flip@flip-vs-expired-vblank:
- shard-lnl: [FAIL][48] ([Intel XE#301] / [Intel XE#3149]) -> [FAIL][49] ([Intel XE#301])
[48]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a/shard-lnl-6/igt@kms_flip@flip-vs-expired-vblank.html
[49]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-2/igt@kms_flip@flip-vs-expired-vblank.html
* igt@xe_exec_multi_queue@one-queue-preempt-mode-priority-smem:
- shard-lnl: [ABORT][50] ([Intel XE#8007]) -> [SKIP][51] ([Intel XE#6874])
[50]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a/shard-lnl-5/igt@xe_exec_multi_queue@one-queue-preempt-mode-priority-smem.html
[51]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/shard-lnl-4/igt@xe_exec_multi_queue@one-queue-preempt-mode-priority-smem.html
[Intel XE#1124]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1124
[Intel XE#1138]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1138
[Intel XE#1392]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1392
[Intel XE#1421]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1421
[Intel XE#1470]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1470
[Intel XE#1503]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1503
[Intel XE#2313]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2313
[Intel XE#2763]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2763
[Intel XE#301]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/301
[Intel XE#3149]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3149
[Intel XE#3321]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3321
[Intel XE#373]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/373
[Intel XE#4130]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4130
[Intel XE#4351]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4351
[Intel XE#6312]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6312
[Intel XE#6321]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6321
[Intel XE#6540]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6540
[Intel XE#656]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/656
[Intel XE#6874]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6874
[Intel XE#688]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/688
[Intel XE#6886]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6886
[Intel XE#6964]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6964
[Intel XE#6974]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6974
[Intel XE#7061]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7061
[Intel XE#7136]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7136
[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#7344]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7344
[Intel XE#7356]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7356
[Intel XE#7357]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7357
[Intel XE#7366]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7366
[Intel XE#7482]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7482
[Intel XE#7636]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7636
[Intel XE#7793]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7793
[Intel XE#7850]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7850
[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#8007]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/8007
Build changes
-------------
* Linux: xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a -> xe-pw-168205v1
IGT_8956: 8956
xe-5228-3806de1666bcca0525d737e3849c602cbe98fb1a: 3806de1666bcca0525d737e3849c602cbe98fb1a
xe-pw-168205v1: 168205v1
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-168205v1/index.html
[-- Attachment #2: Type: text/html, Size: 18999 bytes --]
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/9] drm/xe: Add framework for info probing
2026-06-09 20:17 ` [PATCH 1/9] drm/xe: Add framework for info probing Gustavo Sousa
@ 2026-06-16 18:51 ` Violet Monti
2026-06-16 21:56 ` Matt Roper
1 sibling, 0 replies; 24+ messages in thread
From: Violet Monti @ 2026-06-16 18:51 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-xe
On Tue, Jun 09, 2026 at 05:17:33PM -0300, Gustavo Sousa wrote:
> Functions xe_info_init_early() and xe_info_init() currently probe some
> information from the hardware while doing initialization of info
> fields. Besides mixing responsibilities, another issue from this
> approach is that kunit tests need to implement static stubs for the
> probing part.
>
> Let's prepare the ground to ensuring that those functions stop probing
> the information from the hardware by creating the necessary framework
> for extracting the probing bits out of them. Do that by creating a
> new struct type called xe_probed_info and the functions responsible
> for populating it.
>
> In upcoming changes, we will gradually refactor the code so that all
> info needed by xe_info_init_early() and xe_info_init() that is probed
> from the hardware is passed to them via struct xe_probed_info.
>
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
Reviewed-by: Violet Monti <violet.monti@intel.com>
> ---
> drivers/gpu/drm/xe/tests/xe_pci.c | 16 +++++++++++++--
> drivers/gpu/drm/xe/xe_pci.c | 41 +++++++++++++++++++++++++++++++++++----
> 2 files changed, 51 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/tests/xe_pci.c b/drivers/gpu/drm/xe/tests/xe_pci.c
> index 9240aff779da..51d032a9e01a 100644
> --- a/drivers/gpu/drm/xe/tests/xe_pci.c
> +++ b/drivers/gpu/drm/xe/tests/xe_pci.c
> @@ -338,13 +338,21 @@ static void fake_xe_info_probe_tile_count(struct xe_device *xe)
> /* Nothing to do, just use the statically defined value. */
> }
>
> +static int fake_probe_info(struct xe_device *xe,
> + struct xe_probed_info *probed_info)
> +{
> + return 0;
> +}
> +
> int xe_pci_fake_device_init(struct xe_device *xe)
> {
> struct kunit *test = kunit_get_current_test();
> struct xe_pci_fake_data *data = test->priv;
> + struct xe_probed_info probed_info = {};
> const struct pci_device_id *ent = pciidlist;
> const struct xe_device_desc *desc;
> const struct xe_subplatform_desc *subplatform_desc;
> + int err;
>
> if (!data) {
> desc = (const void *)ent->driver_data;
> @@ -379,8 +387,12 @@ int xe_pci_fake_device_init(struct xe_device *xe)
> kunit_activate_static_stub(test, xe_info_probe_tile_count,
> fake_xe_info_probe_tile_count);
>
> - xe_info_init_early(xe, desc, subplatform_desc);
> - xe_info_init(xe, desc);
> + err = fake_probe_info(xe, &probed_info);
> + if (err)
> + return err;
> +
> + xe_info_init_early(xe, desc, subplatform_desc, &probed_info);
> + xe_info_init(xe, desc, &probed_info);
>
> return 0;
> }
> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
> index 78fc2e4dcfc6..7f1da6d25011 100644
> --- a/drivers/gpu/drm/xe/xe_pci.c
> +++ b/drivers/gpu/drm/xe/xe_pci.c
> @@ -737,13 +737,27 @@ static void init_devid(struct xe_device *xe)
> xe->info.revid = pdev->revision;
> }
>
> +struct xe_probed_info {
> + /* Nothing for now. */
> +};
> +
> +/*
> + * Probe from the hardware the info required by xe_info_init_early().
> + */
> +static int xe_probe_info_early(struct xe_device *xe,
> + struct xe_probed_info *probed_info)
> +{
> + return 0;
> +}
> +
> /*
> * Initialize device info content that only depends on static driver_data
> * passed to the driver at probe time from PCI ID table.
> */
> static int xe_info_init_early(struct xe_device *xe,
> const struct xe_device_desc *desc,
> - const struct xe_subplatform_desc *subplatform_desc)
> + const struct xe_subplatform_desc *subplatform_desc,
> + struct xe_probed_info *probed_info)
> {
> int err;
>
> @@ -910,6 +924,15 @@ static struct xe_gt *alloc_media_gt(struct xe_tile *tile,
> return gt;
> }
>
> +/*
> + * Probe from the hardware the info required by xe_info_init().
> + */
> +static int xe_probe_info(struct xe_device *xe,
> + struct xe_probed_info *probed_info)
> +{
> + return 0;
> +}
> +
> /*
> * Initialize device info content that does require knowledge about
> * graphics / media IP version.
> @@ -917,7 +940,8 @@ static struct xe_gt *alloc_media_gt(struct xe_tile *tile,
> * present in device info.
> */
> static int xe_info_init(struct xe_device *xe,
> - const struct xe_device_desc *desc)
> + const struct xe_device_desc *desc,
> + struct xe_probed_info *probed_info)
> {
> u32 graphics_gmdid_revid = 0, media_gmdid_revid = 0;
> const struct xe_ip *graphics_ip;
> @@ -1073,6 +1097,7 @@ static void xe_pci_remove(struct pci_dev *pdev)
> */
> static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
> {
> + struct xe_probed_info probed_info = {};
> const struct xe_device_desc *desc = (const void *)ent->driver_data;
> const struct xe_subplatform_desc *subplatform_desc;
> struct xe_device *xe;
> @@ -1117,7 +1142,11 @@ static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
>
> pci_set_master(pdev);
>
> - err = xe_info_init_early(xe, desc, subplatform_desc);
> + err = xe_probe_info_early(xe, &probed_info);
> + if (err)
> + return err;
> +
> + err = xe_info_init_early(xe, desc, subplatform_desc, &probed_info);
> if (err)
> return err;
>
> @@ -1136,7 +1165,11 @@ static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
> if (err)
> return err;
>
> - err = xe_info_init(xe, desc);
> + err = xe_probe_info(xe, &probed_info);
> + if (err)
> + return err;
> +
> + err = xe_info_init(xe, desc, &probed_info);
> if (err)
> return err;
>
>
> --
> 2.53.0
>
--
--
Violet Monti
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 2/9] drm/xe/step: Pass xe_step_info to xe_step_*_get() functions
2026-06-09 20:17 ` [PATCH 2/9] drm/xe/step: Pass xe_step_info to xe_step_*_get() functions Gustavo Sousa
@ 2026-06-16 19:31 ` Violet Monti
0 siblings, 0 replies; 24+ messages in thread
From: Violet Monti @ 2026-06-16 19:31 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-xe
On Tue, Jun 09, 2026 at 05:17:34PM -0300, Gustavo Sousa wrote:
> The xe_step_*_get() functions update the step directly in
> xe->info.step and are called by functions xe_info_init_early() and
> xe_info_init().
>
> As the stepping info is something probed from the hardware (via PCI
> revid and/or GMDID) and we want to move away from probing inside
> xe_info_init*() functions, let's make xe_step_*_get() functions modify
> a pointer to the step structure instead of modifying xe->info.step
> directly: this will allow an upcoming change that will move those
> function calls out of the info init functions and will pass a member
> of struct xe_probed_info instead of xe->info.step.
>
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
Reviewed-by: Violet Monti <violet.monti@intel.com>
> ---
> drivers/gpu/drm/xe/xe_pci.c | 6 +++---
> drivers/gpu/drm/xe/xe_step.c | 33 ++++++++++++++++++++-------------
> drivers/gpu/drm/xe/xe_step.h | 7 ++++---
> 3 files changed, 27 insertions(+), 19 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
> index 7f1da6d25011..646e04c6254f 100644
> --- a/drivers/gpu/drm/xe/xe_pci.c
> +++ b/drivers/gpu/drm/xe/xe_pci.c
> @@ -811,7 +811,7 @@ static int xe_info_init_early(struct xe_device *xe,
> xe->info.max_gt_per_tile = desc->max_gt_per_tile;
> xe->info.tile_count = 1 + desc->max_remote_tiles;
>
> - xe_step_platform_get(xe);
> + xe_step_platform_get(xe, &xe->info.step);
>
> err = xe_tile_init_early(xe_device_get_root_tile(xe), xe, 0);
> if (err)
> @@ -963,7 +963,7 @@ static int xe_info_init(struct xe_device *xe,
> if (desc->pre_gmdid_graphics_ip) {
> graphics_ip = desc->pre_gmdid_graphics_ip;
> media_ip = desc->pre_gmdid_media_ip;
> - xe_step_pre_gmdid_get(xe);
> + xe_step_pre_gmdid_get(xe, &xe->info.step);
> } else {
> xe_assert(xe, !desc->pre_gmdid_media_ip);
> ret = handle_gmdid(xe, &graphics_ip, &media_ip,
> @@ -971,7 +971,7 @@ static int xe_info_init(struct xe_device *xe,
> if (ret)
> return ret;
>
> - xe_step_gmdid_get(xe, graphics_gmdid_revid, media_gmdid_revid);
> + xe_step_gmdid_get(xe, graphics_gmdid_revid, media_gmdid_revid, &xe->info.step);
> }
>
> /*
> diff --git a/drivers/gpu/drm/xe/xe_step.c b/drivers/gpu/drm/xe/xe_step.c
> index fb9c31613ca7..49dc64f2b363 100644
> --- a/drivers/gpu/drm/xe/xe_step.c
> +++ b/drivers/gpu/drm/xe/xe_step.c
> @@ -111,11 +111,12 @@ __diag_pop();
> /**
> * xe_step_platform_get - Determine platform-level stepping from PCI revid
> * @xe: Xe device
> + * @step: Pointer to the step struct to update
> *
> * Convert the PCI revid into a platform-level stepping value and store that
> - * in the device info.
> + * in @step->platform.
> */
> -void xe_step_platform_get(struct xe_device *xe)
> +void xe_step_platform_get(struct xe_device *xe, struct xe_step_info *step)
> {
> /*
> * Not all platforms map PCI revid directly into our symbolic stepping
> @@ -127,17 +128,20 @@ void xe_step_platform_get(struct xe_device *xe)
> */
>
> if (xe->info.platform == XE_NOVALAKE_P)
> - xe->info.step.platform = STEP_A0 + xe->info.revid;
> + step->platform = STEP_A0 + xe->info.revid;
> }
>
> /**
> * xe_step_pre_gmdid_get - Determine IP steppings from PCI revid
> * @xe: Xe device
> + * @step: Pointer to the step struct to update
> *
> - * Convert the PCI revid into proper IP steppings. This should only be
> - * used on platforms that do not have GMD_ID support.
> + * Convert the PCI revid into proper IP steppings and update @step->basedie,
> + * @step->graphics and @step->media accordingly.
> + *
> + * This should only be used on platforms that do not have GMD_ID support.
> */
> -void xe_step_pre_gmdid_get(struct xe_device *xe)
> +void xe_step_pre_gmdid_get(struct xe_device *xe, struct xe_step_info *step)
> {
> const struct xe_step_info *revids = NULL;
> u16 revid = xe->info.revid;
> @@ -234,9 +238,9 @@ void xe_step_pre_gmdid_get(struct xe_device *xe)
> }
>
> done:
> - xe->info.step.graphics = graphics;
> - xe->info.step.media = media;
> - xe->info.step.basedie = basedie;
> + step->graphics = graphics;
> + step->media = media;
> + step->basedie = basedie;
> }
>
> /**
> @@ -244,8 +248,10 @@ void xe_step_pre_gmdid_get(struct xe_device *xe)
> * @xe: Xe device
> * @graphics_gmdid_revid: value of graphics GMD_ID register's revid field
> * @media_gmdid_revid: value of media GMD_ID register's revid field
> + * @step: Poninter to the step struct to update.
> *
> - * Convert the revid fields of the GMD_ID registers into proper IP steppings.
> + * Convert the revid fields of the GMD_ID registers into proper IP steppings
> + * and update @step->graphics and @step->media accordingly.
> *
> * GMD_ID revid values are currently expected to have consistent meanings on
> * all platforms: major steppings (A0, B0, etc.) are 4 apart, with minor
> @@ -253,7 +259,8 @@ void xe_step_pre_gmdid_get(struct xe_device *xe)
> */
> void xe_step_gmdid_get(struct xe_device *xe,
> u32 graphics_gmdid_revid,
> - u32 media_gmdid_revid)
> + u32 media_gmdid_revid,
> + struct xe_step_info *step)
> {
> u8 graphics = STEP_A0 + graphics_gmdid_revid;
> u8 media = STEP_A0 + media_gmdid_revid;
> @@ -270,8 +277,8 @@ void xe_step_gmdid_get(struct xe_device *xe,
> media_gmdid_revid);
> }
>
> - xe->info.step.graphics = graphics;
> - xe->info.step.media = media;
> + step->graphics = graphics;
> + step->media = media;
> }
>
> #define STEP_NAME_CASE(name) \
> diff --git a/drivers/gpu/drm/xe/xe_step.h b/drivers/gpu/drm/xe/xe_step.h
> index ea36b22cc297..c6cea95a3727 100644
> --- a/drivers/gpu/drm/xe/xe_step.h
> +++ b/drivers/gpu/drm/xe/xe_step.h
> @@ -12,12 +12,13 @@
>
> struct xe_device;
>
> -void xe_step_platform_get(struct xe_device *xe);
> +void xe_step_platform_get(struct xe_device *xe, struct xe_step_info *step);
>
> -void xe_step_pre_gmdid_get(struct xe_device *xe);
> +void xe_step_pre_gmdid_get(struct xe_device *xe, struct xe_step_info *step);
> void xe_step_gmdid_get(struct xe_device *xe,
> u32 graphics_gmdid_revid,
> - u32 media_gmdid_revid);
> + u32 media_gmdid_revid,
> + struct xe_step_info *step);
> static inline u32 xe_step_to_gmdid(enum intel_step step) { return step - STEP_A0; }
>
> const char *xe_step_name(enum intel_step step);
>
> --
> 2.53.0
>
--
--
Violet Monti
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 3/9] drm/xe: Add devid and revid to xe_probed_info
2026-06-09 20:17 ` [PATCH 3/9] drm/xe: Add devid and revid to xe_probed_info Gustavo Sousa
@ 2026-06-16 19:33 ` Violet Monti
0 siblings, 0 replies; 24+ messages in thread
From: Violet Monti @ 2026-06-16 19:33 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-xe
On Tue, Jun 09, 2026 at 05:17:35PM -0300, Gustavo Sousa wrote:
> The PCI devid and revid fields are info that we probe from the
> hardware (indirectly via the PCI subsystem). Add them to
> xe_probed_info and set them via xe_probe_info_early(), since the
> respective fields in xe->info are updated in xe_info_init_early().
>
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
Reviewed-by: Violet Monti <violet.monti@intel.com>
> ---
> drivers/gpu/drm/xe/tests/xe_pci.c | 6 ------
> drivers/gpu/drm/xe/xe_pci.c | 23 ++++++++++-------------
> 2 files changed, 10 insertions(+), 19 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/tests/xe_pci.c b/drivers/gpu/drm/xe/tests/xe_pci.c
> index 51d032a9e01a..1baf3cd0d381 100644
> --- a/drivers/gpu/drm/xe/tests/xe_pci.c
> +++ b/drivers/gpu/drm/xe/tests/xe_pci.c
> @@ -311,11 +311,6 @@ const void *xe_pci_id_gen_param(struct kunit *test, const void *prev, char *desc
> }
> EXPORT_SYMBOL_IF_KUNIT(xe_pci_id_gen_param);
>
> -static void fake_init_devid(struct xe_device *xe)
> -{
> - /* Nothing to do, just keep zero. */
> -}
> -
> static int fake_read_gmdid(struct xe_device *xe, enum xe_gmdid_type type,
> u32 *ver, u32 *revid)
> {
> @@ -382,7 +377,6 @@ int xe_pci_fake_device_init(struct xe_device *xe)
> xe->sriov.__mode = data && data->sriov_mode ?
> data->sriov_mode : XE_SRIOV_MODE_NONE;
>
> - kunit_activate_static_stub(test, init_devid, fake_init_devid);
> kunit_activate_static_stub(test, read_gmdid, fake_read_gmdid);
> kunit_activate_static_stub(test, xe_info_probe_tile_count,
> fake_xe_info_probe_tile_count);
> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
> index 646e04c6254f..382df59260ae 100644
> --- a/drivers/gpu/drm/xe/xe_pci.c
> +++ b/drivers/gpu/drm/xe/xe_pci.c
> @@ -727,18 +727,9 @@ static int handle_gmdid(struct xe_device *xe,
> return 0;
> }
>
> -static void init_devid(struct xe_device *xe)
> -{
> - struct pci_dev *pdev = to_pci_dev(xe->drm.dev);
> -
> - KUNIT_STATIC_STUB_REDIRECT(init_devid, xe);
> -
> - xe->info.devid = pdev->device;
> - xe->info.revid = pdev->revision;
> -}
> -
> struct xe_probed_info {
> - /* Nothing for now. */
> + u16 devid;
> + u8 revid;
> };
>
> /*
> @@ -747,6 +738,11 @@ struct xe_probed_info {
> static int xe_probe_info_early(struct xe_device *xe,
> struct xe_probed_info *probed_info)
> {
> + struct pci_dev *pdev = to_pci_dev(xe->drm.dev);
> +
> + probed_info->devid = pdev->device;
> + probed_info->revid = pdev->revision;
> +
> return 0;
> }
>
> @@ -761,13 +757,14 @@ static int xe_info_init_early(struct xe_device *xe,
> {
> int err;
>
> + xe->info.devid = probed_info->devid;
> + xe->info.revid = probed_info->revid;
> +
> xe->info.platform_name = desc->platform_name;
> xe->info.platform = desc->platform;
> xe->info.subplatform = subplatform_desc ?
> subplatform_desc->subplatform : XE_SUBPLATFORM_NONE;
>
> - init_devid(xe);
> -
> xe->info.dma_mask_size = desc->dma_mask_size;
> xe->info.va_bits = desc->va_bits;
> xe->info.vm_max_level = desc->vm_max_level;
>
> --
> 2.53.0
>
--
--
Violet Monti
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 4/9] drm/xe/step: Make xe_step_platform_get() independent from xe->info
2026-06-09 20:17 ` [PATCH 4/9] drm/xe/step: Make xe_step_platform_get() independent from xe->info Gustavo Sousa
@ 2026-06-16 19:37 ` Violet Monti
0 siblings, 0 replies; 24+ messages in thread
From: Violet Monti @ 2026-06-16 19:37 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-xe
On Tue, Jun 09, 2026 at 05:17:36PM -0300, Gustavo Sousa wrote:
> Currently xe_step_platform_get() uses info fields from xe->info to
> define the platform-level stepping value.
>
> Because the platform-level stepping info depends on the PCI revid, it
> should be defined as part of xe_probe_info_early() instead of being
> directly probed inside xe_info_init_early().
>
> Let's make sure that xe_step_platform_get() receives the necessary
> data as parameters and does not depend on xe->info. That will allow
> us to move the call up to xe_probe_info_early() in an upcoming change.
>
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
Reviewed-by: Violet Monti <violet.monti@intel.com>
> ---
> drivers/gpu/drm/xe/xe_pci.c | 2 +-
> drivers/gpu/drm/xe/xe_step.c | 9 +++++----
> drivers/gpu/drm/xe/xe_step.h | 3 ++-
> 3 files changed, 8 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
> index 382df59260ae..c5c75fbfa0e2 100644
> --- a/drivers/gpu/drm/xe/xe_pci.c
> +++ b/drivers/gpu/drm/xe/xe_pci.c
> @@ -808,7 +808,7 @@ static int xe_info_init_early(struct xe_device *xe,
> xe->info.max_gt_per_tile = desc->max_gt_per_tile;
> xe->info.tile_count = 1 + desc->max_remote_tiles;
>
> - xe_step_platform_get(xe, &xe->info.step);
> + xe_step_platform_get(xe->info.platform, xe->info.revid, &xe->info.step);
>
> err = xe_tile_init_early(xe_device_get_root_tile(xe), xe, 0);
> if (err)
> diff --git a/drivers/gpu/drm/xe/xe_step.c b/drivers/gpu/drm/xe/xe_step.c
> index 49dc64f2b363..55c1996f689e 100644
> --- a/drivers/gpu/drm/xe/xe_step.c
> +++ b/drivers/gpu/drm/xe/xe_step.c
> @@ -110,13 +110,14 @@ __diag_pop();
>
> /**
> * xe_step_platform_get - Determine platform-level stepping from PCI revid
> - * @xe: Xe device
> + * @platform: The Xe platform
> + * @revid: The PCI revid
> * @step: Pointer to the step struct to update
> *
> * Convert the PCI revid into a platform-level stepping value and store that
> * in @step->platform.
> */
> -void xe_step_platform_get(struct xe_device *xe, struct xe_step_info *step)
> +void xe_step_platform_get(enum xe_platform platform, u8 revid, struct xe_step_info *step)
> {
> /*
> * Not all platforms map PCI revid directly into our symbolic stepping
> @@ -127,8 +128,8 @@ void xe_step_platform_get(struct xe_device *xe, struct xe_step_info *step)
> * checks.
> */
>
> - if (xe->info.platform == XE_NOVALAKE_P)
> - step->platform = STEP_A0 + xe->info.revid;
> + if (platform == XE_NOVALAKE_P)
> + step->platform = STEP_A0 + revid;
> }
>
> /**
> diff --git a/drivers/gpu/drm/xe/xe_step.h b/drivers/gpu/drm/xe/xe_step.h
> index c6cea95a3727..5a5845335740 100644
> --- a/drivers/gpu/drm/xe/xe_step.h
> +++ b/drivers/gpu/drm/xe/xe_step.h
> @@ -10,9 +10,10 @@
>
> #include "xe_step_types.h"
>
> +enum xe_platform;
> struct xe_device;
>
> -void xe_step_platform_get(struct xe_device *xe, struct xe_step_info *step);
> +void xe_step_platform_get(enum xe_platform platform, u8 revid, struct xe_step_info *step);
>
> void xe_step_pre_gmdid_get(struct xe_device *xe, struct xe_step_info *step);
> void xe_step_gmdid_get(struct xe_device *xe,
>
> --
> 2.53.0
>
--
--
Violet Monti
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 5/9] drm/xe: Add platform-level step info to xe_probed_info
2026-06-09 20:17 ` [PATCH 5/9] drm/xe: Add platform-level step info to xe_probed_info Gustavo Sousa
@ 2026-06-16 19:50 ` Violet Monti
0 siblings, 0 replies; 24+ messages in thread
From: Violet Monti @ 2026-06-16 19:50 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-xe
On Tue, Jun 09, 2026 at 05:17:37PM -0300, Gustavo Sousa wrote:
> The platform-level step information depends on the PCI revid and, as
> such, should be probed in xe_probe_info_early() instead of
> xe_info_init_early(). Move the code accordingly.
>
> Note that we currently only update probed_info->step.platform as part
> of this change. We will deal with the other fields of
> probed_info->step as a follow-up change, which will be tied to the
> probing of graphics and media IPs.
>
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
Reviewed-by: Violet Monti <violet.monti@intel.com>
> ---
> drivers/gpu/drm/xe/xe_pci.c | 9 ++++++---
> 1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
> index c5c75fbfa0e2..1c53a25442a9 100644
> --- a/drivers/gpu/drm/xe/xe_pci.c
> +++ b/drivers/gpu/drm/xe/xe_pci.c
> @@ -730,12 +730,14 @@ static int handle_gmdid(struct xe_device *xe,
> struct xe_probed_info {
> u16 devid;
> u8 revid;
> + struct xe_step_info step;
> };
>
> /*
> * Probe from the hardware the info required by xe_info_init_early().
> */
> static int xe_probe_info_early(struct xe_device *xe,
> + const struct xe_device_desc *desc,
> struct xe_probed_info *probed_info)
> {
> struct pci_dev *pdev = to_pci_dev(xe->drm.dev);
> @@ -743,6 +745,8 @@ static int xe_probe_info_early(struct xe_device *xe,
> probed_info->devid = pdev->device;
> probed_info->revid = pdev->revision;
>
> + xe_step_platform_get(desc->platform, probed_info->revid, &probed_info->step);
> +
> return 0;
> }
>
> @@ -759,6 +763,7 @@ static int xe_info_init_early(struct xe_device *xe,
>
> xe->info.devid = probed_info->devid;
> xe->info.revid = probed_info->revid;
> + xe->info.step.platform = probed_info->step.platform;
>
> xe->info.platform_name = desc->platform_name;
> xe->info.platform = desc->platform;
> @@ -808,8 +813,6 @@ static int xe_info_init_early(struct xe_device *xe,
> xe->info.max_gt_per_tile = desc->max_gt_per_tile;
> xe->info.tile_count = 1 + desc->max_remote_tiles;
>
> - xe_step_platform_get(xe->info.platform, xe->info.revid, &xe->info.step);
> -
> err = xe_tile_init_early(xe_device_get_root_tile(xe), xe, 0);
> if (err)
> return err;
> @@ -1139,7 +1142,7 @@ static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
>
> pci_set_master(pdev);
>
> - err = xe_probe_info_early(xe, &probed_info);
> + err = xe_probe_info_early(xe, desc, &probed_info);
> if (err)
> return err;
>
>
> --
> 2.53.0
>
--
--
Violet Monti
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 6/9] drm/xe/tests: Set non-GMDID graphics step in xe_pci_fake_device_init()
2026-06-09 20:17 ` [PATCH 6/9] drm/xe/tests: Set non-GMDID graphics step in xe_pci_fake_device_init() Gustavo Sousa
@ 2026-06-16 19:58 ` Violet Monti
0 siblings, 0 replies; 24+ messages in thread
From: Violet Monti @ 2026-06-16 19:58 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-xe
On Tue, Jun 09, 2026 at 05:17:38PM -0300, Gustavo Sousa wrote:
> Currently the logic to set the graphics step for non-GMDID-based
> platforms in kunit testing is defined in xe_wa_test_init(). That
> logic should rather belong to the helper xe_pci_fake_device_init(), so
> move it there.
>
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
Reviewed-by: Violet Monti <violet.monti@intel.com>
> ---
> drivers/gpu/drm/xe/tests/xe_pci.c | 3 +++
> drivers/gpu/drm/xe/tests/xe_wa_test.c | 3 ---
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/tests/xe_pci.c b/drivers/gpu/drm/xe/tests/xe_pci.c
> index 1baf3cd0d381..a665d5dbc472 100644
> --- a/drivers/gpu/drm/xe/tests/xe_pci.c
> +++ b/drivers/gpu/drm/xe/tests/xe_pci.c
> @@ -388,6 +388,9 @@ int xe_pci_fake_device_init(struct xe_device *xe)
> xe_info_init_early(xe, desc, subplatform_desc, &probed_info);
> xe_info_init(xe, desc, &probed_info);
>
> + if (data && !data->graphics_verx100)
> + xe->info.step = data->step;
> +
> return 0;
> }
> EXPORT_SYMBOL_IF_KUNIT(xe_pci_fake_device_init);
> diff --git a/drivers/gpu/drm/xe/tests/xe_wa_test.c b/drivers/gpu/drm/xe/tests/xe_wa_test.c
> index ff0e2502b39f..21601e9df353 100644
> --- a/drivers/gpu/drm/xe/tests/xe_wa_test.c
> +++ b/drivers/gpu/drm/xe/tests/xe_wa_test.c
> @@ -43,9 +43,6 @@ static int xe_wa_test_init(struct kunit *test)
> xe_gt_mmio_init(gt);
> }
>
> - if (!param->graphics_verx100)
> - xe->info.step = param->step;
> -
> /* TODO: init hw engines for engine/LRC WAs */
> xe->drm.dev = dev;
> test->priv = xe;
>
> --
> 2.53.0
>
--
--
Violet Monti
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 7/9] drm/xe: Add graphics/media IPs and their step info to xe_probed_info
2026-06-09 20:17 ` [PATCH 7/9] drm/xe: Add graphics/media IPs and their step info to xe_probed_info Gustavo Sousa
@ 2026-06-16 20:22 ` Violet Monti
0 siblings, 0 replies; 24+ messages in thread
From: Violet Monti @ 2026-06-16 20:22 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-xe
On Tue, Jun 09, 2026 at 05:17:39PM -0300, Gustavo Sousa wrote:
> On GMDID-based platforms, the driver needs to probe the hardware by
> reading GMDID registers in order to identify the
> graphics/media/display IPs that are present in the platform as well as
> their stepping values.
>
> Currently, xe_info_init() has such a probing logic, but that task
> should be rather responsibility of xe_probe_info(). As such, move it
> to the latter.
>
> For pre-GMDID platforms, the IPs are identified via PCI devid and
> revid fields, which is arguably also hardware dependent. So do the
> same for those platforms.
>
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
Reviewed-by: Violet Monti <violet.monti@intel.com>
> ---
> drivers/gpu/drm/xe/tests/xe_pci.c | 44 ++++++++++----------
> drivers/gpu/drm/xe/xe_pci.c | 88 ++++++++++++++++++++++++---------------
> 2 files changed, 77 insertions(+), 55 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/tests/xe_pci.c b/drivers/gpu/drm/xe/tests/xe_pci.c
> index a665d5dbc472..cd64b1d614c8 100644
> --- a/drivers/gpu/drm/xe/tests/xe_pci.c
> +++ b/drivers/gpu/drm/xe/tests/xe_pci.c
> @@ -311,31 +311,35 @@ const void *xe_pci_id_gen_param(struct kunit *test, const void *prev, char *desc
> }
> EXPORT_SYMBOL_IF_KUNIT(xe_pci_id_gen_param);
>
> -static int fake_read_gmdid(struct xe_device *xe, enum xe_gmdid_type type,
> - u32 *ver, u32 *revid)
> -{
> - struct kunit *test = kunit_get_current_test();
> - struct xe_pci_fake_data *data = test->priv;
> -
> - if (type == GMDID_MEDIA) {
> - *ver = data->media_verx100;
> - *revid = xe_step_to_gmdid(data->step.media);
> - } else {
> - *ver = data->graphics_verx100;
> - *revid = xe_step_to_gmdid(data->step.graphics);
> - }
> -
> - return 0;
> -}
> -
> static void fake_xe_info_probe_tile_count(struct xe_device *xe)
> {
> /* Nothing to do, just use the statically defined value. */
> }
>
> static int fake_probe_info(struct xe_device *xe,
> + const struct xe_device_desc *desc,
> + struct xe_pci_fake_data *data,
> struct xe_probed_info *probed_info)
> {
> + if (!data || desc->pre_gmdid_graphics_ip) {
> + probed_info->graphics_ip = desc->pre_gmdid_graphics_ip;
> + probed_info->media_ip = desc->pre_gmdid_media_ip;
> + } else {
> + probed_info->graphics_ip = find_graphics_ip(data->graphics_verx100);
> +
> + if (data->media_verx100) {
> + probed_info->media_ip = find_media_ip(data->media_verx100);
> + xe_assert(xe, probed_info->media_ip);
> + }
> + }
> +
> + xe_assert(xe, probed_info->graphics_ip);
> + if (!probed_info->graphics_ip)
> + return -ENODEV;
> +
> + if (data)
> + probed_info->step = data->step;
> +
> return 0;
> }
>
> @@ -377,20 +381,16 @@ int xe_pci_fake_device_init(struct xe_device *xe)
> xe->sriov.__mode = data && data->sriov_mode ?
> data->sriov_mode : XE_SRIOV_MODE_NONE;
>
> - kunit_activate_static_stub(test, read_gmdid, fake_read_gmdid);
> kunit_activate_static_stub(test, xe_info_probe_tile_count,
> fake_xe_info_probe_tile_count);
>
> - err = fake_probe_info(xe, &probed_info);
> + err = fake_probe_info(xe, desc, data, &probed_info);
> if (err)
> return err;
>
> xe_info_init_early(xe, desc, subplatform_desc, &probed_info);
> xe_info_init(xe, desc, &probed_info);
>
> - if (data && !data->graphics_verx100)
> - xe->info.step = data->step;
> -
> return 0;
> }
> EXPORT_SYMBOL_IF_KUNIT(xe_pci_fake_device_init);
> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
> index 1c53a25442a9..388771ef6a52 100644
> --- a/drivers/gpu/drm/xe/xe_pci.c
> +++ b/drivers/gpu/drm/xe/xe_pci.c
> @@ -600,8 +600,6 @@ static int read_gmdid(struct xe_device *xe, enum xe_gmdid_type type, u32 *ver, u
> struct xe_reg gmdid_reg = GMD_ID;
> u32 val;
>
> - KUNIT_STATIC_STUB_REDIRECT(read_gmdid, xe, type, ver, revid);
> -
> if (IS_SRIOV_VF(xe)) {
> /*
> * To get the value of the GMDID register, VFs must obtain it
> @@ -731,6 +729,8 @@ struct xe_probed_info {
> u16 devid;
> u8 revid;
> struct xe_step_info step;
> + const struct xe_ip *graphics_ip;
> + const struct xe_ip *media_ip;
> };
>
> /*
> @@ -924,12 +924,59 @@ static struct xe_gt *alloc_media_gt(struct xe_tile *tile,
> return gt;
> }
>
> +static int xe_probe_ips(struct xe_device *xe,
> + const struct xe_device_desc *desc,
> + struct xe_probed_info *probed_info)
> +{
> + /*
> + * If this platform supports GMD_ID, we'll detect the proper IP
> + * descriptor to use from hardware registers.
> + * desc->pre_gmdid_graphics_ip will only ever be set at this point for
> + * platforms before GMD_ID. In that case the IP descriptions and
> + * versions are simply derived from that.
> + */
> + if (desc->pre_gmdid_graphics_ip) {
> + probed_info->graphics_ip = desc->pre_gmdid_graphics_ip;
> + probed_info->media_ip = desc->pre_gmdid_media_ip;
> + xe_step_pre_gmdid_get(xe, &probed_info->step);
> + } else {
> + int err;
> + u32 graphics_revid, media_revid;
> +
> + xe_assert(xe, !desc->pre_gmdid_media_ip);
> +
> + err = handle_gmdid(xe, &probed_info->graphics_ip, &probed_info->media_ip,
> + &graphics_revid, &media_revid);
> + if (err)
> + return err;
> +
> + xe_step_gmdid_get(xe, graphics_revid, media_revid, &probed_info->step);
> + }
> +
> + /*
> + * If we couldn't detect the graphics IP, that's considered a fatal
> + * error and we should abort driver load. Failing to detect media
> + * IP is non-fatal; we'll just proceed without enabling media support.
> + */
> + if (!probed_info->graphics_ip)
> + return -ENODEV;
> +
> + return 0;
> +}
> +
> /*
> * Probe from the hardware the info required by xe_info_init().
> */
> static int xe_probe_info(struct xe_device *xe,
> + const struct xe_device_desc *desc,
> struct xe_probed_info *probed_info)
> {
> + int err;
> +
> + err = xe_probe_ips(xe, desc, probed_info);
> + if (err)
> + return err;
> +
> return 0;
> }
>
> @@ -943,44 +990,19 @@ static int xe_info_init(struct xe_device *xe,
> const struct xe_device_desc *desc,
> struct xe_probed_info *probed_info)
> {
> - u32 graphics_gmdid_revid = 0, media_gmdid_revid = 0;
> const struct xe_ip *graphics_ip;
> const struct xe_ip *media_ip;
> const struct xe_graphics_desc *graphics_desc;
> const struct xe_media_desc *media_desc;
> struct xe_tile *tile;
> struct xe_gt *gt;
> - int ret;
> u8 id;
>
> - /*
> - * If this platform supports GMD_ID, we'll detect the proper IP
> - * descriptor to use from hardware registers.
> - * desc->pre_gmdid_graphics_ip will only ever be set at this point for
> - * platforms before GMD_ID. In that case the IP descriptions and
> - * versions are simply derived from that.
> - */
> - if (desc->pre_gmdid_graphics_ip) {
> - graphics_ip = desc->pre_gmdid_graphics_ip;
> - media_ip = desc->pre_gmdid_media_ip;
> - xe_step_pre_gmdid_get(xe, &xe->info.step);
> - } else {
> - xe_assert(xe, !desc->pre_gmdid_media_ip);
> - ret = handle_gmdid(xe, &graphics_ip, &media_ip,
> - &graphics_gmdid_revid, &media_gmdid_revid);
> - if (ret)
> - return ret;
> -
> - xe_step_gmdid_get(xe, graphics_gmdid_revid, media_gmdid_revid, &xe->info.step);
> - }
> -
> - /*
> - * If we couldn't detect the graphics IP, that's considered a fatal
> - * error and we should abort driver load. Failing to detect media
> - * IP is non-fatal; we'll just proceed without enabling media support.
> - */
> - if (!graphics_ip)
> - return -ENODEV;
> + graphics_ip = probed_info->graphics_ip;
> + media_ip = probed_info->media_ip;
> + xe->info.step.basedie = probed_info->step.basedie;
> + xe->info.step.graphics = probed_info->step.graphics;
> + xe->info.step.media = probed_info->step.media;
>
> xe->info.graphics_verx100 = graphics_ip->verx100;
> xe->info.graphics_name = graphics_ip->name;
> @@ -1165,7 +1187,7 @@ static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
> if (err)
> return err;
>
> - err = xe_probe_info(xe, &probed_info);
> + err = xe_probe_info(xe, desc, &probed_info);
> if (err)
> return err;
>
>
> --
> 2.53.0
>
--
--
Violet Monti
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 8/9] drm/xe: Don't initialize tile_count in xe_info_init_early()
2026-06-09 20:17 ` [PATCH 8/9] drm/xe: Don't initialize tile_count in xe_info_init_early() Gustavo Sousa
@ 2026-06-16 21:07 ` Violet Monti
0 siblings, 0 replies; 24+ messages in thread
From: Violet Monti @ 2026-06-16 21:07 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-xe
On Tue, Jun 09, 2026 at 05:17:40PM -0300, Gustavo Sousa wrote:
> The value of xe->info.tile_count is only really valid after
> xe_info_probe_tile_count(). Any use of tile_count before that point
> is invalid and, consequently, initializing it in xe_info_init_early()
> is pointless.
>
> Move the initialization to xe_info_probe_tile_count().
>
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
Reviewed-by: Violet Monti <violet.monti@intel.com>
> ---
> drivers/gpu/drm/xe/tests/xe_pci.c | 5 +++--
> drivers/gpu/drm/xe/xe_pci.c | 10 ++++++----
> 2 files changed, 9 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/tests/xe_pci.c b/drivers/gpu/drm/xe/tests/xe_pci.c
> index cd64b1d614c8..31ec41aa997d 100644
> --- a/drivers/gpu/drm/xe/tests/xe_pci.c
> +++ b/drivers/gpu/drm/xe/tests/xe_pci.c
> @@ -311,9 +311,10 @@ const void *xe_pci_id_gen_param(struct kunit *test, const void *prev, char *desc
> }
> EXPORT_SYMBOL_IF_KUNIT(xe_pci_id_gen_param);
>
> -static void fake_xe_info_probe_tile_count(struct xe_device *xe)
> +static void fake_xe_info_probe_tile_count(struct xe_device *xe,
> + const struct xe_device_desc *desc)
> {
> - /* Nothing to do, just use the statically defined value. */
> + xe->info.tile_count = 1 + desc->max_remote_tiles;
> }
>
> static int fake_probe_info(struct xe_device *xe,
> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
> index 388771ef6a52..6b2daad87316 100644
> --- a/drivers/gpu/drm/xe/xe_pci.c
> +++ b/drivers/gpu/drm/xe/xe_pci.c
> @@ -811,7 +811,6 @@ static int xe_info_init_early(struct xe_device *xe,
> xe_assert(xe, desc->max_gt_per_tile > 0);
> xe_assert(xe, desc->max_gt_per_tile <= XE_MAX_GT_PER_TILE);
> xe->info.max_gt_per_tile = desc->max_gt_per_tile;
> - xe->info.tile_count = 1 + desc->max_remote_tiles;
>
> err = xe_tile_init_early(xe_device_get_root_tile(xe), xe, 0);
> if (err)
> @@ -823,13 +822,16 @@ static int xe_info_init_early(struct xe_device *xe,
> /*
> * Possibly override number of tile based on configuration register.
> */
> -static void xe_info_probe_tile_count(struct xe_device *xe)
> +static void xe_info_probe_tile_count(struct xe_device *xe,
> + const struct xe_device_desc *desc)
> {
> struct xe_mmio *mmio;
> u8 tile_count;
> u32 mtcfg;
>
> - KUNIT_STATIC_STUB_REDIRECT(xe_info_probe_tile_count, xe);
> + KUNIT_STATIC_STUB_REDIRECT(xe_info_probe_tile_count, xe, desc);
> +
> + xe->info.tile_count = 1 + desc->max_remote_tiles;
>
> /*
> * Probe for tile count only for platforms that support multiple
> @@ -1035,7 +1037,7 @@ static int xe_info_init(struct xe_device *xe,
> xe->info.has_soc_remapper_telem = 0;
> }
>
> - xe_info_probe_tile_count(xe);
> + xe_info_probe_tile_count(xe, desc);
>
> for_each_remote_tile(tile, xe, id) {
> int err;
>
> --
> 2.53.0
>
--
--
Violet Monti
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 9/9] drm/xe: Add tile_count to xe_probed_info
2026-06-09 20:17 ` [PATCH 9/9] drm/xe: Add tile_count to xe_probed_info Gustavo Sousa
@ 2026-06-16 21:16 ` Violet Monti
0 siblings, 0 replies; 24+ messages in thread
From: Violet Monti @ 2026-06-16 21:16 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-xe
On Tue, Jun 09, 2026 at 05:17:41PM -0300, Gustavo Sousa wrote:
> On multi-tile platforms, we need to probe the hardware for the number
> of tiles that are present in the platform. That means that we should
> do that as part of xe_probe_info() instead of xe_info_init(). Do
> that.
>
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
Reviewed-by: Violet Monti <violet.monti@intel.com>
> ---
> drivers/gpu/drm/xe/tests/xe_pci.c | 11 ++---------
> drivers/gpu/drm/xe/xe_pci.c | 27 +++++++++++++--------------
> 2 files changed, 15 insertions(+), 23 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/tests/xe_pci.c b/drivers/gpu/drm/xe/tests/xe_pci.c
> index 31ec41aa997d..8df9029afcd3 100644
> --- a/drivers/gpu/drm/xe/tests/xe_pci.c
> +++ b/drivers/gpu/drm/xe/tests/xe_pci.c
> @@ -311,17 +311,13 @@ const void *xe_pci_id_gen_param(struct kunit *test, const void *prev, char *desc
> }
> EXPORT_SYMBOL_IF_KUNIT(xe_pci_id_gen_param);
>
> -static void fake_xe_info_probe_tile_count(struct xe_device *xe,
> - const struct xe_device_desc *desc)
> -{
> - xe->info.tile_count = 1 + desc->max_remote_tiles;
> -}
> -
> static int fake_probe_info(struct xe_device *xe,
> const struct xe_device_desc *desc,
> struct xe_pci_fake_data *data,
> struct xe_probed_info *probed_info)
> {
> + probed_info->tile_count = 1 + desc->max_remote_tiles;
> +
> if (!data || desc->pre_gmdid_graphics_ip) {
> probed_info->graphics_ip = desc->pre_gmdid_graphics_ip;
> probed_info->media_ip = desc->pre_gmdid_media_ip;
> @@ -382,9 +378,6 @@ int xe_pci_fake_device_init(struct xe_device *xe)
> xe->sriov.__mode = data && data->sriov_mode ?
> data->sriov_mode : XE_SRIOV_MODE_NONE;
>
> - kunit_activate_static_stub(test, xe_info_probe_tile_count,
> - fake_xe_info_probe_tile_count);
> -
> err = fake_probe_info(xe, desc, data, &probed_info);
> if (err)
> return err;
> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
> index 6b2daad87316..1b5df4384689 100644
> --- a/drivers/gpu/drm/xe/xe_pci.c
> +++ b/drivers/gpu/drm/xe/xe_pci.c
> @@ -728,6 +728,7 @@ static int handle_gmdid(struct xe_device *xe,
> struct xe_probed_info {
> u16 devid;
> u8 revid;
> + u8 tile_count;
> struct xe_step_info step;
> const struct xe_ip *graphics_ip;
> const struct xe_ip *media_ip;
> @@ -819,25 +820,21 @@ static int xe_info_init_early(struct xe_device *xe,
> return 0;
> }
>
> -/*
> - * Possibly override number of tile based on configuration register.
> - */
> -static void xe_info_probe_tile_count(struct xe_device *xe,
> - const struct xe_device_desc *desc)
> +static void xe_probe_tile_count(struct xe_device *xe,
> + const struct xe_device_desc *desc,
> + struct xe_probed_info *probed_info)
> {
> struct xe_mmio *mmio;
> u8 tile_count;
> u32 mtcfg;
>
> - KUNIT_STATIC_STUB_REDIRECT(xe_info_probe_tile_count, xe, desc);
> -
> - xe->info.tile_count = 1 + desc->max_remote_tiles;
> + probed_info->tile_count = 1 + desc->max_remote_tiles;
>
> /*
> * Probe for tile count only for platforms that support multiple
> * tiles.
> */
> - if (xe->info.tile_count == 1)
> + if (probed_info->tile_count == 1)
> return;
>
> mmio = xe_root_tile_mmio(xe);
> @@ -850,10 +847,10 @@ static void xe_info_probe_tile_count(struct xe_device *xe,
> mtcfg = xe_mmio_read32(mmio, XEHP_MTCFG_ADDR);
> tile_count = REG_FIELD_GET(TILE_COUNT, mtcfg) + 1;
>
> - if (tile_count < xe->info.tile_count) {
> + if (tile_count < probed_info->tile_count) {
> drm_info(&xe->drm, "tile_count: %d, reduced_tile_count %d\n",
> - xe->info.tile_count, tile_count);
> - xe->info.tile_count = tile_count;
> + probed_info->tile_count, tile_count);
> + probed_info->tile_count = tile_count;
> }
> }
>
> @@ -975,6 +972,8 @@ static int xe_probe_info(struct xe_device *xe,
> {
> int err;
>
> + xe_probe_tile_count(xe, desc, probed_info);
> +
> err = xe_probe_ips(xe, desc, probed_info);
> if (err)
> return err;
> @@ -1002,6 +1001,8 @@ static int xe_info_init(struct xe_device *xe,
>
> graphics_ip = probed_info->graphics_ip;
> media_ip = probed_info->media_ip;
> +
> + xe->info.tile_count = probed_info->tile_count;
> xe->info.step.basedie = probed_info->step.basedie;
> xe->info.step.graphics = probed_info->step.graphics;
> xe->info.step.media = probed_info->step.media;
> @@ -1037,8 +1038,6 @@ static int xe_info_init(struct xe_device *xe,
> xe->info.has_soc_remapper_telem = 0;
> }
>
> - xe_info_probe_tile_count(xe, desc);
> -
> for_each_remote_tile(tile, xe, id) {
> int err;
>
>
> --
> 2.53.0
>
--
--
Violet Monti
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/9] drm/xe: Add framework for info probing
2026-06-09 20:17 ` [PATCH 1/9] drm/xe: Add framework for info probing Gustavo Sousa
2026-06-16 18:51 ` Violet Monti
@ 2026-06-16 21:56 ` Matt Roper
2026-06-17 12:43 ` Gustavo Sousa
1 sibling, 1 reply; 24+ messages in thread
From: Matt Roper @ 2026-06-16 21:56 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-xe
On Tue, Jun 09, 2026 at 05:17:33PM -0300, Gustavo Sousa wrote:
> Functions xe_info_init_early() and xe_info_init() currently probe some
> information from the hardware while doing initialization of info
Is the mention of xe_info_init_early() here correct? The general rule
is supposed to be that *_early() functions are software-only setup
(e.g., kzalloc'ing memory, initializing mutexes, etc.), whereas
non-early functions are the ones that touch hardware in some manner
(either for reading and/or writing). There were a lot of mistakes in
this area early on that we're still cleaning up, but I think
xe_info_init_early() at least is following the rules as far as I can
see?
Matt
> fields. Besides mixing responsibilities, another issue from this
> approach is that kunit tests need to implement static stubs for the
> probing part.
>
> Let's prepare the ground to ensuring that those functions stop probing
> the information from the hardware by creating the necessary framework
> for extracting the probing bits out of them. Do that by creating a
> new struct type called xe_probed_info and the functions responsible
> for populating it.
>
> In upcoming changes, we will gradually refactor the code so that all
> info needed by xe_info_init_early() and xe_info_init() that is probed
> from the hardware is passed to them via struct xe_probed_info.
>
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
> ---
> drivers/gpu/drm/xe/tests/xe_pci.c | 16 +++++++++++++--
> drivers/gpu/drm/xe/xe_pci.c | 41 +++++++++++++++++++++++++++++++++++----
> 2 files changed, 51 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/tests/xe_pci.c b/drivers/gpu/drm/xe/tests/xe_pci.c
> index 9240aff779da..51d032a9e01a 100644
> --- a/drivers/gpu/drm/xe/tests/xe_pci.c
> +++ b/drivers/gpu/drm/xe/tests/xe_pci.c
> @@ -338,13 +338,21 @@ static void fake_xe_info_probe_tile_count(struct xe_device *xe)
> /* Nothing to do, just use the statically defined value. */
> }
>
> +static int fake_probe_info(struct xe_device *xe,
> + struct xe_probed_info *probed_info)
> +{
> + return 0;
> +}
> +
> int xe_pci_fake_device_init(struct xe_device *xe)
> {
> struct kunit *test = kunit_get_current_test();
> struct xe_pci_fake_data *data = test->priv;
> + struct xe_probed_info probed_info = {};
> const struct pci_device_id *ent = pciidlist;
> const struct xe_device_desc *desc;
> const struct xe_subplatform_desc *subplatform_desc;
> + int err;
>
> if (!data) {
> desc = (const void *)ent->driver_data;
> @@ -379,8 +387,12 @@ int xe_pci_fake_device_init(struct xe_device *xe)
> kunit_activate_static_stub(test, xe_info_probe_tile_count,
> fake_xe_info_probe_tile_count);
>
> - xe_info_init_early(xe, desc, subplatform_desc);
> - xe_info_init(xe, desc);
> + err = fake_probe_info(xe, &probed_info);
> + if (err)
> + return err;
> +
> + xe_info_init_early(xe, desc, subplatform_desc, &probed_info);
> + xe_info_init(xe, desc, &probed_info);
>
> return 0;
> }
> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
> index 78fc2e4dcfc6..7f1da6d25011 100644
> --- a/drivers/gpu/drm/xe/xe_pci.c
> +++ b/drivers/gpu/drm/xe/xe_pci.c
> @@ -737,13 +737,27 @@ static void init_devid(struct xe_device *xe)
> xe->info.revid = pdev->revision;
> }
>
> +struct xe_probed_info {
> + /* Nothing for now. */
> +};
> +
> +/*
> + * Probe from the hardware the info required by xe_info_init_early().
> + */
> +static int xe_probe_info_early(struct xe_device *xe,
> + struct xe_probed_info *probed_info)
> +{
> + return 0;
> +}
> +
> /*
> * Initialize device info content that only depends on static driver_data
> * passed to the driver at probe time from PCI ID table.
> */
> static int xe_info_init_early(struct xe_device *xe,
> const struct xe_device_desc *desc,
> - const struct xe_subplatform_desc *subplatform_desc)
> + const struct xe_subplatform_desc *subplatform_desc,
> + struct xe_probed_info *probed_info)
> {
> int err;
>
> @@ -910,6 +924,15 @@ static struct xe_gt *alloc_media_gt(struct xe_tile *tile,
> return gt;
> }
>
> +/*
> + * Probe from the hardware the info required by xe_info_init().
> + */
> +static int xe_probe_info(struct xe_device *xe,
> + struct xe_probed_info *probed_info)
> +{
> + return 0;
> +}
> +
> /*
> * Initialize device info content that does require knowledge about
> * graphics / media IP version.
> @@ -917,7 +940,8 @@ static struct xe_gt *alloc_media_gt(struct xe_tile *tile,
> * present in device info.
> */
> static int xe_info_init(struct xe_device *xe,
> - const struct xe_device_desc *desc)
> + const struct xe_device_desc *desc,
> + struct xe_probed_info *probed_info)
> {
> u32 graphics_gmdid_revid = 0, media_gmdid_revid = 0;
> const struct xe_ip *graphics_ip;
> @@ -1073,6 +1097,7 @@ static void xe_pci_remove(struct pci_dev *pdev)
> */
> static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
> {
> + struct xe_probed_info probed_info = {};
> const struct xe_device_desc *desc = (const void *)ent->driver_data;
> const struct xe_subplatform_desc *subplatform_desc;
> struct xe_device *xe;
> @@ -1117,7 +1142,11 @@ static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
>
> pci_set_master(pdev);
>
> - err = xe_info_init_early(xe, desc, subplatform_desc);
> + err = xe_probe_info_early(xe, &probed_info);
> + if (err)
> + return err;
> +
> + err = xe_info_init_early(xe, desc, subplatform_desc, &probed_info);
> if (err)
> return err;
>
> @@ -1136,7 +1165,11 @@ static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
> if (err)
> return err;
>
> - err = xe_info_init(xe, desc);
> + err = xe_probe_info(xe, &probed_info);
> + if (err)
> + return err;
> +
> + err = xe_info_init(xe, desc, &probed_info);
> if (err)
> return err;
>
>
> --
> 2.53.0
>
--
Matt Roper
Graphics Software Engineer
Linux GPU Platform Enablement
Intel Corporation
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/9] drm/xe: Add framework for info probing
2026-06-16 21:56 ` Matt Roper
@ 2026-06-17 12:43 ` Gustavo Sousa
0 siblings, 0 replies; 24+ messages in thread
From: Gustavo Sousa @ 2026-06-17 12:43 UTC (permalink / raw)
To: Matt Roper; +Cc: intel-xe
Matt Roper <matthew.d.roper@intel.com> writes:
> On Tue, Jun 09, 2026 at 05:17:33PM -0300, Gustavo Sousa wrote:
>> Functions xe_info_init_early() and xe_info_init() currently probe some
>> information from the hardware while doing initialization of info
>
> Is the mention of xe_info_init_early() here correct? The general rule
> is supposed to be that *_early() functions are software-only setup
> (e.g., kzalloc'ing memory, initializing mutexes, etc.), whereas
> non-early functions are the ones that touch hardware in some manner
> (either for reading and/or writing). There were a lot of mistakes in
> this area early on that we're still cleaning up, but I think
> xe_info_init_early() at least is following the rules as far as I can
> see?
Yeah, to be fair, xe_info_init_early() does not directly try to read
information from the hardware during its execution. My argument is that
it "probes indirectly" when it gets the PCI device and revision ids from
the PCI subsystem; that becomes more evident by the fact that we need a
kunit static stub for init_devid().
--
Gustavo Sousa
>
>
> Matt
>
>> fields. Besides mixing responsibilities, another issue from this
>> approach is that kunit tests need to implement static stubs for the
>> probing part.
>>
>> Let's prepare the ground to ensuring that those functions stop probing
>> the information from the hardware by creating the necessary framework
>> for extracting the probing bits out of them. Do that by creating a
>> new struct type called xe_probed_info and the functions responsible
>> for populating it.
>>
>> In upcoming changes, we will gradually refactor the code so that all
>> info needed by xe_info_init_early() and xe_info_init() that is probed
>> from the hardware is passed to them via struct xe_probed_info.
>>
>> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
>> ---
>> drivers/gpu/drm/xe/tests/xe_pci.c | 16 +++++++++++++--
>> drivers/gpu/drm/xe/xe_pci.c | 41 +++++++++++++++++++++++++++++++++++----
>> 2 files changed, 51 insertions(+), 6 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/xe/tests/xe_pci.c b/drivers/gpu/drm/xe/tests/xe_pci.c
>> index 9240aff779da..51d032a9e01a 100644
>> --- a/drivers/gpu/drm/xe/tests/xe_pci.c
>> +++ b/drivers/gpu/drm/xe/tests/xe_pci.c
>> @@ -338,13 +338,21 @@ static void fake_xe_info_probe_tile_count(struct xe_device *xe)
>> /* Nothing to do, just use the statically defined value. */
>> }
>>
>> +static int fake_probe_info(struct xe_device *xe,
>> + struct xe_probed_info *probed_info)
>> +{
>> + return 0;
>> +}
>> +
>> int xe_pci_fake_device_init(struct xe_device *xe)
>> {
>> struct kunit *test = kunit_get_current_test();
>> struct xe_pci_fake_data *data = test->priv;
>> + struct xe_probed_info probed_info = {};
>> const struct pci_device_id *ent = pciidlist;
>> const struct xe_device_desc *desc;
>> const struct xe_subplatform_desc *subplatform_desc;
>> + int err;
>>
>> if (!data) {
>> desc = (const void *)ent->driver_data;
>> @@ -379,8 +387,12 @@ int xe_pci_fake_device_init(struct xe_device *xe)
>> kunit_activate_static_stub(test, xe_info_probe_tile_count,
>> fake_xe_info_probe_tile_count);
>>
>> - xe_info_init_early(xe, desc, subplatform_desc);
>> - xe_info_init(xe, desc);
>> + err = fake_probe_info(xe, &probed_info);
>> + if (err)
>> + return err;
>> +
>> + xe_info_init_early(xe, desc, subplatform_desc, &probed_info);
>> + xe_info_init(xe, desc, &probed_info);
>>
>> return 0;
>> }
>> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
>> index 78fc2e4dcfc6..7f1da6d25011 100644
>> --- a/drivers/gpu/drm/xe/xe_pci.c
>> +++ b/drivers/gpu/drm/xe/xe_pci.c
>> @@ -737,13 +737,27 @@ static void init_devid(struct xe_device *xe)
>> xe->info.revid = pdev->revision;
>> }
>>
>> +struct xe_probed_info {
>> + /* Nothing for now. */
>> +};
>> +
>> +/*
>> + * Probe from the hardware the info required by xe_info_init_early().
>> + */
>> +static int xe_probe_info_early(struct xe_device *xe,
>> + struct xe_probed_info *probed_info)
>> +{
>> + return 0;
>> +}
>> +
>> /*
>> * Initialize device info content that only depends on static driver_data
>> * passed to the driver at probe time from PCI ID table.
>> */
>> static int xe_info_init_early(struct xe_device *xe,
>> const struct xe_device_desc *desc,
>> - const struct xe_subplatform_desc *subplatform_desc)
>> + const struct xe_subplatform_desc *subplatform_desc,
>> + struct xe_probed_info *probed_info)
>> {
>> int err;
>>
>> @@ -910,6 +924,15 @@ static struct xe_gt *alloc_media_gt(struct xe_tile *tile,
>> return gt;
>> }
>>
>> +/*
>> + * Probe from the hardware the info required by xe_info_init().
>> + */
>> +static int xe_probe_info(struct xe_device *xe,
>> + struct xe_probed_info *probed_info)
>> +{
>> + return 0;
>> +}
>> +
>> /*
>> * Initialize device info content that does require knowledge about
>> * graphics / media IP version.
>> @@ -917,7 +940,8 @@ static struct xe_gt *alloc_media_gt(struct xe_tile *tile,
>> * present in device info.
>> */
>> static int xe_info_init(struct xe_device *xe,
>> - const struct xe_device_desc *desc)
>> + const struct xe_device_desc *desc,
>> + struct xe_probed_info *probed_info)
>> {
>> u32 graphics_gmdid_revid = 0, media_gmdid_revid = 0;
>> const struct xe_ip *graphics_ip;
>> @@ -1073,6 +1097,7 @@ static void xe_pci_remove(struct pci_dev *pdev)
>> */
>> static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
>> {
>> + struct xe_probed_info probed_info = {};
>> const struct xe_device_desc *desc = (const void *)ent->driver_data;
>> const struct xe_subplatform_desc *subplatform_desc;
>> struct xe_device *xe;
>> @@ -1117,7 +1142,11 @@ static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
>>
>> pci_set_master(pdev);
>>
>> - err = xe_info_init_early(xe, desc, subplatform_desc);
>> + err = xe_probe_info_early(xe, &probed_info);
>> + if (err)
>> + return err;
>> +
>> + err = xe_info_init_early(xe, desc, subplatform_desc, &probed_info);
>> if (err)
>> return err;
>>
>> @@ -1136,7 +1165,11 @@ static int xe_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
>> if (err)
>> return err;
>>
>> - err = xe_info_init(xe, desc);
>> + err = xe_probe_info(xe, &probed_info);
>> + if (err)
>> + return err;
>> +
>> + err = xe_info_init(xe, desc, &probed_info);
>> if (err)
>> return err;
>>
>>
>> --
>> 2.53.0
>>
>
> --
> Matt Roper
> Graphics Software Engineer
> Linux GPU Platform Enablement
> Intel Corporation
^ permalink raw reply [flat|nested] 24+ messages in thread
end of thread, other threads:[~2026-06-17 12:43 UTC | newest]
Thread overview: 24+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-06-09 20:17 [PATCH 0/9] drm/xe: Probe info outside of xe_info_init() and xe_info_init_early() Gustavo Sousa
2026-06-09 20:17 ` [PATCH 1/9] drm/xe: Add framework for info probing Gustavo Sousa
2026-06-16 18:51 ` Violet Monti
2026-06-16 21:56 ` Matt Roper
2026-06-17 12:43 ` Gustavo Sousa
2026-06-09 20:17 ` [PATCH 2/9] drm/xe/step: Pass xe_step_info to xe_step_*_get() functions Gustavo Sousa
2026-06-16 19:31 ` Violet Monti
2026-06-09 20:17 ` [PATCH 3/9] drm/xe: Add devid and revid to xe_probed_info Gustavo Sousa
2026-06-16 19:33 ` Violet Monti
2026-06-09 20:17 ` [PATCH 4/9] drm/xe/step: Make xe_step_platform_get() independent from xe->info Gustavo Sousa
2026-06-16 19:37 ` Violet Monti
2026-06-09 20:17 ` [PATCH 5/9] drm/xe: Add platform-level step info to xe_probed_info Gustavo Sousa
2026-06-16 19:50 ` Violet Monti
2026-06-09 20:17 ` [PATCH 6/9] drm/xe/tests: Set non-GMDID graphics step in xe_pci_fake_device_init() Gustavo Sousa
2026-06-16 19:58 ` Violet Monti
2026-06-09 20:17 ` [PATCH 7/9] drm/xe: Add graphics/media IPs and their step info to xe_probed_info Gustavo Sousa
2026-06-16 20:22 ` Violet Monti
2026-06-09 20:17 ` [PATCH 8/9] drm/xe: Don't initialize tile_count in xe_info_init_early() Gustavo Sousa
2026-06-16 21:07 ` Violet Monti
2026-06-09 20:17 ` [PATCH 9/9] drm/xe: Add tile_count to xe_probed_info Gustavo Sousa
2026-06-16 21:16 ` Violet Monti
2026-06-09 20:58 ` ✓ CI.KUnit: success for drm/xe: Probe info outside of xe_info_init() and xe_info_init_early() Patchwork
2026-06-09 22:06 ` ✓ Xe.CI.BAT: " Patchwork
2026-06-10 12:22 ` ✗ Xe.CI.FULL: failure " Patchwork
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox