[PATCH v13 00/16] Crescent Island PMT support

[PATCH v13 00/16] Crescent Island PMT support

[Michael J. Ruhl]

The Crescent Island (CRI) GPU supports PMT via Xe driver
registration.

The CRI PMT MMIO mapping is shared for each PMT feature and it is
necessary to set an index register value to access the correct memory
space.

The PMT driver has a callback infrastructure to access data areas with
driver specific access. It is does not support the PMT control access.

CRI discovery/control spaces are part of the access window so a driver
callback is necessary to allow for the correct window to be exposed.

Add control path callback support to the PMT driver.

Add CRI PMT discovery information.

Update the Xe driver to support the CRI PMT access.

The crashlog access needs to be done with power enabled (telem is only
available when the device is powered).

Determine which access is requested, and do the appropriate power
request.

CRI FW is loaded at device power.  Support the late-binding API to
wait for FW readiness. (pending sysctl patch set implements the API).

v2:
  Address AI feedback:
  - add missing entry->cb check
  - address SRIOV VF concerns
  - use xe_pm_runtime_put instead of _get in error path

  Address review comments:
  - redo power/lock sequencing
  - support device hotplug events

v3:
  - cleanup power manangement flow
  - fixed an out of order power management locking issue
  - add a GUID caching mechanism, for GUID read
  - removed unused parameter (mmio)

v4:
 - address various review comments
 - refactor crashlog to allow for return values
 - for callback usage, decouple PMT remap of data areas
 - use IS_SRIOV_VF() rather than expose soc-remapper internals
 - separate locking fixes intos distinct steps
 - refactor define update patches to seperate BMG and CRI updates
 - separate internal GUID into separate steps

v5:
 - removed callback helpers
 - do not return error message with sysfs
 - fix missed return code check in trigger_store
 - return error value not false
 - use DEFINE_RES_MEM
 - document when base is unused
 - return error on invalid GUID request

v6:
 - fix enabled/disabled inversion
 - fix missing DOC reference
 - change memcpy_to/fromio to readl/writel for register read
 - guid cache comment and update on valid guid

v7:
 - reordered to have BMG updates followed by CRI support
 - separate base_offset usage for clarity
 - added EAGAIN usage to block uninitialized use of the API

v8:
 - fix punit crashlog offset (should be 0xE60 not 0xE78)
 - remove EAGAIN support and re-order probe steps to initialize
   VSEC before other features use it

v9:
 - added appropriate reviewed-bys
 - use dev_err instead of pr_err
 - enforce defined base usage when callbacks are in plcace
 - move SRIOV check to get_platform_info
 - assert SRIOV for API check
 - use scoped_guard() where possible
 - update to use correct sysctrl helper

v10:
 - fix incorrect return type (broken in refactor, int -> bool)
 - fix incorrectly documented error return values
 - doc text cleanup

v11:
 - add reviewed-by
 - fix minor checkpatch issue
 - use xe_assert() for a consistent API check

v12:
 - add survivability mode support
 - add reviewed-by

v13:
 - PMT patches (original commits 1 - 5) are merged upstream
 - resubmitting XeKMD patches

Michael J. Ruhl (16):
  drm/xe/vsec: Do not register BMG PMT for VF
  drm/xe/vsec: Correct locking order
  drm/xe/vsec: Use correct pm state get
  drm/xe/vsec: Add DOC text for VSEC
  drm/xe/vsec: Support possible hotplug exit
  drm/xe/vsec: Refactor BattleMage PMT defines
  drm/xe/vsec: Update VSEC probe order
  drm/xe/vsec: Add base_offset to allow for more flexibilty
  drm/xe/vsec: Support Crescent Island PMT
  drm/xe/vsec: Crescent Island PMT decode
  drm/xe/vsec: Crescent Island PMT callbacks
  drm/xe/vsec: Support late bind fw information
  drm/xe/vsec: Add PMT GUID internal access
  drm/xe/vsec: Update PMT internal access
  drm/xe/vsec: Refactor platform check
  drm/xe/vsec: Support survivability mode for CRI

 Documentation/gpu/xe/index.rst             |   1 +
 Documentation/gpu/xe/xe_vsec.rst           |  14 +
 drivers/gpu/drm/xe/regs/xe_pmt.h           |  12 +-
 drivers/gpu/drm/xe/xe_debugfs.c            |  45 +-
 drivers/gpu/drm/xe/xe_device.c             |   6 +-
 drivers/gpu/drm/xe/xe_device_types.h       |   9 +
 drivers/gpu/drm/xe/xe_hwmon.c              |  10 +-
 drivers/gpu/drm/xe/xe_pcode.c              |  10 +-
 drivers/gpu/drm/xe/xe_survivability_mode.c |   2 +
 drivers/gpu/drm/xe/xe_vsec.c               | 584 +++++++++++++++++++--
 drivers/gpu/drm/xe/xe_vsec.h               |   3 +-
 11 files changed, 619 insertions(+), 77 deletions(-)
 create mode 100644 Documentation/gpu/xe/xe_vsec.rst

-- 
2.43.0

[PATCH v13 01/16] drm/xe/vsec: Do not register BMG PMT for VF

[Michael J. Ruhl]

BMG PMT is not supported for VF nodes.

If IS_SRIOV_VF() is true, do NOT register with PMT.

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com>
---
 drivers/gpu/drm/xe/xe_vsec.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
index a9baf0bfe572..178f22db4046 100644
--- a/drivers/gpu/drm/xe/xe_vsec.c
+++ b/drivers/gpu/drm/xe/xe_vsec.c
@@ -149,6 +149,8 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
 	u32 offset;
 	int ret;
 
+	xe_assert(xe, !IS_SRIOV_VF(xe));
+
 	ret = xe_guid_decode(guid, &mem_region, &offset);
 	if (ret)
 		return ret;
@@ -157,9 +159,6 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
 
 	guard(mutex)(&xe->pmt.lock);
 
-	if (!xe->soc_remapper.set_telem_region)
-		return -ENODEV;
-
 	/* indicate that we are not at an appropriate power level */
 	if (!xe_pm_runtime_get_if_active(xe))
 		return -ENODATA;
@@ -186,6 +185,9 @@ static enum xe_vsec get_platform_info(struct xe_device *xe)
 	if (xe->info.platform > XE_BATTLEMAGE)
 		return XE_VSEC_UNKNOWN;
 
+	if (IS_SRIOV_VF(xe) && xe->info.platform == XE_BATTLEMAGE)
+		return XE_VSEC_UNKNOWN;
+
 	return vsec_platforms[xe->info.platform];
 }
 
-- 
2.43.0

[PATCH v13 02/16] drm/xe/vsec: Correct locking order

[Michael J. Ruhl]

Xe locking order is power state first, and then local locks.

Reorder locking for correct usage.

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com>
---
 drivers/gpu/drm/xe/xe_vsec.c | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
index 178f22db4046..d93c042cf25b 100644
--- a/drivers/gpu/drm/xe/xe_vsec.c
+++ b/drivers/gpu/drm/xe/xe_vsec.c
@@ -157,16 +157,17 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
 
 	telem_addr += offset + user_offset;
 
-	guard(mutex)(&xe->pmt.lock);
-
 	/* indicate that we are not at an appropriate power level */
 	if (!xe_pm_runtime_get_if_active(xe))
 		return -ENODATA;
 
-	/* set SoC re-mapper index register based on GUID memory region */
-	xe->soc_remapper.set_telem_region(xe, mem_region);
+	scoped_guard(mutex, &xe->pmt.lock) {
+		/* set SoC re-mapper index register based on GUID memory region */
+		xe->soc_remapper.set_telem_region(xe, mem_region);
+
+		memcpy_fromio(data, telem_addr, count);
+	}
 
-	memcpy_fromio(data, telem_addr, count);
 	xe_pm_runtime_put(xe);
 
 	return count;
-- 
2.43.0

[PATCH v13 03/16] drm/xe/vsec: Use correct pm state get

[Michael J. Ruhl]

Crashlog needs to be collected at all times.  The current pm check
assumes telemetry only.

Update read path to enable device for crashlog instances.

Include check of the WATCHER CAP_TYPE for completeness.
WATCHER is a possible type and needs to be accounted for, but is not
supported.

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Fixes: 2c402a801c19 ("platform/x86/intel/pmt: support BMG crashlog")
Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com>
---
 drivers/gpu/drm/xe/xe_vsec.c | 16 +++++++++++++---
 1 file changed, 13 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
index d93c042cf25b..38081dacd307 100644
--- a/drivers/gpu/drm/xe/xe_vsec.c
+++ b/drivers/gpu/drm/xe/xe_vsec.c
@@ -145,6 +145,7 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
 {
 	struct xe_device *xe = kdev_to_xe_device(dev);
 	void __iomem *telem_addr = xe->mmio.regs + BMG_TELEMETRY_OFFSET;
+	u32 cap_type = FIELD_GET(GUID_CAP_TYPE, guid);
 	u32 mem_region;
 	u32 offset;
 	int ret;
@@ -157,9 +158,18 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
 
 	telem_addr += offset + user_offset;
 
-	/* indicate that we are not at an appropriate power level */
-	if (!xe_pm_runtime_get_if_active(xe))
-		return -ENODATA;
+	/* Always allow crashlog. Telemetry, only when powered */
+	switch (cap_type) {
+	case CRASHLOG:
+		xe_pm_runtime_get(xe);
+		break;
+	case TELEMETRY:
+		if (!xe_pm_runtime_get_if_active(xe))
+			return -ENODATA;
+		break;
+	case WATCHER:
+		return -EINVAL;
+	}
 
 	scoped_guard(mutex, &xe->pmt.lock) {
 		/* set SoC re-mapper index register based on GUID memory region */
-- 
2.43.0

[PATCH v13 04/16] drm/xe/vsec: Add DOC text for VSEC

[Michael J. Ruhl]

Add the necessary files and text to use the DOC system.

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com>
---
 Documentation/gpu/xe/index.rst   |  1 +
 Documentation/gpu/xe/xe_vsec.rst | 14 ++++++++++++++
 drivers/gpu/drm/xe/xe_vsec.c     | 24 ++++++++++++++++++++++++
 3 files changed, 39 insertions(+)
 create mode 100644 Documentation/gpu/xe/xe_vsec.rst

diff --git a/Documentation/gpu/xe/index.rst b/Documentation/gpu/xe/index.rst
index 0247a255f7e6..89f013418cf5 100644
--- a/Documentation/gpu/xe/index.rst
+++ b/Documentation/gpu/xe/index.rst
@@ -36,3 +36,4 @@ The display, or :ref:`drm-kms`, support for drm/xe is provided by
    xe_configfs
    xe_gt_stats
    xe_sigid
+   xe_vsec
diff --git a/Documentation/gpu/xe/xe_vsec.rst b/Documentation/gpu/xe/xe_vsec.rst
new file mode 100644
index 000000000000..1a87548489ea
--- /dev/null
+++ b/Documentation/gpu/xe/xe_vsec.rst
@@ -0,0 +1,14 @@
+.. SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+
+=====
+VSEC
+=====
+
+.. kernel-doc:: drivers/gpu/drm/xe/xe_vsec.c
+   :doc: VSEC
+
+Internal API
+============
+
+.. kernel-doc:: drivers/gpu/drm/xe/xe_vsec.c
+   :internal:
diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
index 38081dacd307..082b25b017db 100644
--- a/drivers/gpu/drm/xe/xe_vsec.c
+++ b/drivers/gpu/drm/xe/xe_vsec.c
@@ -19,6 +19,17 @@
 
 #include "regs/xe_pmt.h"
 
+/**
+ * DOC: VSEC
+ *
+ * Some GPUs support the PMT feature.  For those that do, this API provides
+ * probe, discovery and usage (control and access) for GPUs that have PMT.
+ *
+ * PMT is usually accessed through a PCI function.  However, that is not always
+ * available, so the PMT intel_vsec_register() function can be used to set up
+ * PMT usage.
+ */
+
 /* PMT GUID value for BMG devices.  NOTE: this is NOT a PCI id */
 #define BMG_DEVICE_ID 0xE2F8
 
@@ -140,6 +151,19 @@ static int xe_guid_decode(u32 guid, int *index, u32 *offset)
 	return 0;
 }
 
+/**
+ * xe_pmt_telem_read - Given a device and a PMT GUID, read data into a buffer
+ * @dev: valid Xe device
+ * @guid: PMT GUID (from the PMT discovery struct)
+ * @data: buffer to place data in
+ * @user_offset: a user offset relative to the beginning of the data to read
+ * @count: bytes to read
+ *
+ * Return:
+ * * -ENODATA for incorrect power state
+ * * -EINVAL for unsupported PMT feature
+ * * count   number of bytes read on success
+ */
 int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offset,
 		      u32 count)
 {
-- 
2.43.0

[PATCH v13 05/16] drm/xe/vsec: Support possible hotplug exit

[Michael J. Ruhl]

DRM has an API that will verify that a device is valid in the hotplug
context.

Verify device is valid before access in the VSEC callback API.

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com>
---
 drivers/gpu/drm/xe/xe_vsec.c | 29 +++++++++++++++++++++++------
 1 file changed, 23 insertions(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
index 082b25b017db..c0e06b2aeca6 100644
--- a/drivers/gpu/drm/xe/xe_vsec.c
+++ b/drivers/gpu/drm/xe/xe_vsec.c
@@ -10,6 +10,8 @@
 #include <linux/pci.h>
 #include <linux/types.h>
 
+#include <drm/drm_drv.h>
+
 #include "xe_device.h"
 #include "xe_device_types.h"
 #include "xe_mmio.h"
@@ -159,7 +161,12 @@ static int xe_guid_decode(u32 guid, int *index, u32 *offset)
  * @user_offset: a user offset relative to the beginning of the data to read
  * @count: bytes to read
  *
+ * This is a callback API.  I.e this can be accessed external to the XE driver
+ * (PMT driver scope).  Because of this, DRM hotplug needs to be verified
+ * (drm_dev_enter()).
+ *
  * Return:
+ * * -ENODEV device is not available (hotplug)
  * * -ENODATA for incorrect power state
  * * -EINVAL for unsupported PMT feature
  * * count   number of bytes read on success
@@ -172,13 +179,17 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
 	u32 cap_type = FIELD_GET(GUID_CAP_TYPE, guid);
 	u32 mem_region;
 	u32 offset;
-	int ret;
+	int ret = 0;
+	int idx;
 
 	xe_assert(xe, !IS_SRIOV_VF(xe));
 
+	if (!drm_dev_enter(&xe->drm, &idx))
+		return -ENODEV;
+
 	ret = xe_guid_decode(guid, &mem_region, &offset);
 	if (ret)
-		return ret;
+		goto dev_exit;
 
 	telem_addr += offset + user_offset;
 
@@ -188,11 +199,14 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
 		xe_pm_runtime_get(xe);
 		break;
 	case TELEMETRY:
-		if (!xe_pm_runtime_get_if_active(xe))
-			return -ENODATA;
+		if (!xe_pm_runtime_get_if_active(xe)) {
+			ret = -ENODATA;
+			goto dev_exit;
+		}
 		break;
 	case WATCHER:
-		return -EINVAL;
+		ret = -EINVAL;
+		goto dev_exit;
 	}
 
 	scoped_guard(mutex, &xe->pmt.lock) {
@@ -204,7 +218,10 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
 
 	xe_pm_runtime_put(xe);
 
-	return count;
+dev_exit:
+	drm_dev_exit(idx);
+
+	return ret == 0 ? count : ret;
 }
 
 static struct pmt_callbacks xe_pmt_cb = {
-- 
2.43.0

[PATCH v13 06/16] drm/xe/vsec: Refactor BattleMage PMT defines

[Michael J. Ruhl]

Rename generic defines to be BMG specific.
Define several magic numbers.

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com>
---
 drivers/gpu/drm/xe/xe_vsec.c | 29 +++++++++++++++++------------
 1 file changed, 17 insertions(+), 12 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
index c0e06b2aeca6..ebe56c85bc05 100644
--- a/drivers/gpu/drm/xe/xe_vsec.c
+++ b/drivers/gpu/drm/xe/xe_vsec.c
@@ -90,10 +90,15 @@ static struct intel_vsec_platform_info xe_vsec_info[] = {
 #define GUID_CAP_TYPE		GENMASK(29, 28)
 #define GUID_RECORD_ID		GENMASK(31, 30)
 
-#define PUNIT_TELEMETRY_OFFSET		0x0200
-#define PUNIT_WATCHER_OFFSET		0x14A0
-#define OOBMSM_0_WATCHER_OFFSET		0x18D8
-#define OOBMSM_1_TELEMETRY_OFFSET	0x1000
+#define BMG_IDX_TELEM_PUNIT		0x00
+#define BMG_IDX_TELEM_OOBMSM		0x01
+#define BMG_IDX_CRASHLOG_PUNIT		0x02
+#define BMG_IDX_CRASHLOG_OOBMSM		0x04
+
+#define BMG_PUNIT_TELEMETRY_OFFSET	0x0200
+#define BMG_PUNIT_WATCHER_OFFSET	0x14A0
+#define BMG_OOBMSM_0_WATCHER_OFFSET	0x18D8
+#define BMG_OOBMSM_1_TELEMETRY_OFFSET	0x1000
 
 enum record_id {
 	PUNIT,
@@ -122,29 +127,29 @@ static int xe_guid_decode(u32 guid, int *index, u32 *offset)
 	*offset = 0;
 
 	if (cap_type == CRASHLOG) {
-		*index = record_id == PUNIT ? 2 : 4;
+		*index = record_id == PUNIT ? BMG_IDX_CRASHLOG_PUNIT : BMG_IDX_CRASHLOG_OOBMSM;
 		return 0;
 	}
 
 	switch (record_id) {
 	case PUNIT:
-		*index = 0;
+		*index = BMG_IDX_TELEM_PUNIT;
 		if (cap_type == TELEMETRY)
-			*offset = PUNIT_TELEMETRY_OFFSET;
+			*offset = BMG_PUNIT_TELEMETRY_OFFSET;
 		else
-			*offset = PUNIT_WATCHER_OFFSET;
+			*offset = BMG_PUNIT_WATCHER_OFFSET;
 		break;
 
 	case OOBMSM_0:
-		*index = 1;
+		*index = BMG_IDX_TELEM_OOBMSM;
 		if (cap_type == WATCHER)
-			*offset = OOBMSM_0_WATCHER_OFFSET;
+			*offset = BMG_OOBMSM_0_WATCHER_OFFSET;
 		break;
 
 	case OOBMSM_1:
-		*index = 1;
+		*index = BMG_IDX_TELEM_OOBMSM;
 		if (cap_type == TELEMETRY)
-			*offset = OOBMSM_1_TELEMETRY_OFFSET;
+			*offset = BMG_OOBMSM_1_TELEMETRY_OFFSET;
 		break;
 	default:
 		return -EINVAL;
-- 
2.43.0

[PATCH v13 07/16] drm/xe/vsec: Update VSEC probe order

[Michael J. Ruhl]

Both debugfs and hwmon access the PMT data area before PMT is
initialized.

Hardwired offset usage allows this to work by accident.
Upcoming changes will break this usage.

Re-order the probe steps to initialize VSEC before the other
features are initialized.

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com>
---
 drivers/gpu/drm/xe/xe_device.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_device.c b/drivers/gpu/drm/xe/xe_device.c
index 07a7ab6ac083..dfdd83d4150f 100644
--- a/drivers/gpu/drm/xe/xe_device.c
+++ b/drivers/gpu/drm/xe/xe_device.c
@@ -1082,6 +1082,8 @@ int xe_device_probe(struct xe_device *xe)
 	if (err)
 		goto err_unregister_display;
 
+	xe_vsec_init(xe);
+
 	xe_debugfs_register(xe);
 
 	err = xe_hwmon_register(xe);
@@ -1095,8 +1097,6 @@ int xe_device_probe(struct xe_device *xe)
 	for_each_gt(gt, xe, id)
 		xe_gt_sanitize_freq(gt);
 
-	xe_vsec_init(xe);
-
 	err = xe_sriov_init_late(xe);
 	if (err)
 		goto err_unregister_display;
-- 
2.43.0

[PATCH v13 08/16] drm/xe/vsec: Add base_offset to allow for more flexibilty

[Michael J. Ruhl]

Current pmt_telem_read() usage is a fixed (for BMG) offset.
In order to support other devices this must be settable.

Add a base offset for more flexibility.
Add a check to verify that it has been set before allowing
device access.

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com>
---
 drivers/gpu/drm/xe/xe_device_types.h | 2 ++
 drivers/gpu/drm/xe/xe_vsec.c         | 9 ++++++++-
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/xe/xe_device_types.h b/drivers/gpu/drm/xe/xe_device_types.h
index b98418795d5e..b1c7de90002d 100644
--- a/drivers/gpu/drm/xe/xe_device_types.h
+++ b/drivers/gpu/drm/xe/xe_device_types.h
@@ -483,6 +483,8 @@ struct xe_device {
 	struct {
 		/** @pmt.lock: protect access for telemetry data */
 		struct mutex lock;
+		/** @pmt.base_offset: device specific base offset */
+		u64 base_offset;
 	} pmt;
 
 	/** @soc_remapper: SoC remapper object */
diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
index ebe56c85bc05..04aec1e5c15c 100644
--- a/drivers/gpu/drm/xe/xe_vsec.c
+++ b/drivers/gpu/drm/xe/xe_vsec.c
@@ -172,6 +172,7 @@ static int xe_guid_decode(u32 guid, int *index, u32 *offset)
  *
  * Return:
  * * -ENODEV device is not available (hotplug)
+ * * -EAGAIN base_offset has not been set, so init path has not yet been done.
  * * -ENODATA for incorrect power state
  * * -EINVAL for unsupported PMT feature
  * * count   number of bytes read on success
@@ -180,8 +181,8 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
 		      u32 count)
 {
 	struct xe_device *xe = kdev_to_xe_device(dev);
-	void __iomem *telem_addr = xe->mmio.regs + BMG_TELEMETRY_OFFSET;
 	u32 cap_type = FIELD_GET(GUID_CAP_TYPE, guid);
+	void __iomem *telem_addr = xe->mmio.regs + xe->pmt.base_offset;
 	u32 mem_region;
 	u32 offset;
 	int ret = 0;
@@ -192,6 +193,11 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
 	if (!drm_dev_enter(&xe->drm, &idx))
 		return -ENODEV;
 
+	if (!xe->pmt.base_offset) {
+		ret = -EAGAIN;
+		goto dev_exit;
+	}
+
 	ret = xe_guid_decode(guid, &mem_region, &offset);
 	if (ret)
 		goto dev_exit;
@@ -269,6 +275,7 @@ void xe_vsec_init(struct xe_device *xe)
 
 	switch (platform) {
 	case XE_VSEC_BMG:
+		xe->pmt.base_offset = BMG_TELEMETRY_OFFSET;
 		info->priv_data = &xe_pmt_cb;
 		break;
 	default:
-- 
2.43.0

[PATCH v13 09/16] drm/xe/vsec: Support Crescent Island PMT

[Michael J. Ruhl]

Crescent Island (CRI) supports PMT telemetry and crashlog.

Add Crescent Island (CRI) discovery structure (DVSEC) information to
allow for Xe registration.

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com>
---
 drivers/gpu/drm/xe/regs/xe_pmt.h |  8 +++--
 drivers/gpu/drm/xe/xe_vsec.c     | 51 +++++++++++++++++++++++++++++---
 2 files changed, 53 insertions(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/xe/regs/xe_pmt.h b/drivers/gpu/drm/xe/regs/xe_pmt.h
index a62ab05c6b4c..2f6658395587 100644
--- a/drivers/gpu/drm/xe/regs/xe_pmt.h
+++ b/drivers/gpu/drm/xe/regs/xe_pmt.h
@@ -17,8 +17,12 @@
 
 #define PUNIT_VERSION_OFFSET		0xA0
 
-#define BMG_TELEMETRY_BASE_OFFSET	0xE0000
-#define BMG_TELEMETRY_OFFSET		(SOC_BASE + BMG_TELEMETRY_BASE_OFFSET)
+#define XE_PMT_BASE_OFFSET		0xE0000
+
+#define BMG_TELEMETRY_OFFSET		(SOC_BASE + XE_PMT_BASE_OFFSET)
+
+/* for CRI discovery and telemetry are in an indexed window */
+#define CRI_PMT_OFFSET			(SOC_BASE + XE_PMT_BASE_OFFSET)
 
 #define BMG_MODS_RESIDENCY_OFFSET		(0x4D0)
 #define BMG_G2_RESIDENCY_OFFSET		(0x530)
diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
index 04aec1e5c15c..4ecc2d9f3f38 100644
--- a/drivers/gpu/drm/xe/xe_vsec.c
+++ b/drivers/gpu/drm/xe/xe_vsec.c
@@ -8,6 +8,7 @@
 #include <linux/module.h>
 #include <linux/mutex.h>
 #include <linux/pci.h>
+#include <linux/sizes.h>
 #include <linux/types.h>
 
 #include <drm/drm_drv.h>
@@ -32,12 +33,23 @@
  * PMT usage.
  */
 
-/* PMT GUID value for BMG devices.  NOTE: this is NOT a PCI id */
+/* PMT GUID value for BMG and CRI devices.  NOTE: this is NOT a PCI id */
 #define BMG_DEVICE_ID 0xE2F8
+#define CRI_DEVICE_ID 0xE2FA
+
+/*
+ * sizeof(Crashlog Type1 Version2) = 0x18 (24) bytes
+ * For BMG and CRI PUNIT and OOBMSMS crashlogs are consecutive.
+ */
+#define BMG_CRASHLOG_PUNIT_DISC_OFFSET (0x60)
+#define BMG_CRASHLOG_OOBMSM_DISC_OFFSET (BMG_CRASHLOG_PUNIT_DISC_OFFSET + 0x18)
+
+#define CRI_CRASHLOG_PUNIT_DISC_OFFSET (0x80)
+#define CRI_CRASHLOG_OOBMSM_DISC_OFFSET (CRI_CRASHLOG_PUNIT_DISC_OFFSET + 0x18)
 
 static struct intel_vsec_header bmg_telemetry = {
 	.rev = 1,
-	.length = 0x10,
+	.length = SZ_16,
 	.id = VSEC_ID_TELEMETRY,
 	.num_entries = 2,
 	.entry_size = 4,
@@ -47,12 +59,12 @@ static struct intel_vsec_header bmg_telemetry = {
 
 static struct intel_vsec_header bmg_crashlog = {
 	.rev = 1,
-	.length = 0x10,
+	.length = SZ_16,
 	.id = VSEC_ID_CRASHLOG,
 	.num_entries = 2,
 	.entry_size = 6,
 	.tbir = 0,
-	.offset = BMG_DISCOVERY_OFFSET + 0x60,
+	.offset = BMG_DISCOVERY_OFFSET + BMG_CRASHLOG_PUNIT_DISC_OFFSET,
 };
 
 static struct intel_vsec_header *bmg_capabilities[] = {
@@ -61,9 +73,36 @@ static struct intel_vsec_header *bmg_capabilities[] = {
 	NULL
 };
 
+static struct intel_vsec_header cri_telemetry = {
+	.rev = 1,
+	.length = SZ_16,
+	.id = VSEC_ID_TELEMETRY,
+	.num_entries = 3,
+	.entry_size = 4,
+	.tbir = 0,
+	.offset = CRI_PMT_OFFSET,
+};
+
+static struct intel_vsec_header cri_crashlog = {
+	.rev = 1,
+	.length = SZ_16,
+	.id = VSEC_ID_CRASHLOG,
+	.num_entries = 2,
+	.entry_size = 6,
+	.tbir = 0,
+	.offset = CRI_PMT_OFFSET + CRI_CRASHLOG_PUNIT_DISC_OFFSET,
+};
+
+static struct intel_vsec_header *cri_capabilities[] = {
+	&cri_telemetry,
+	&cri_crashlog,
+	NULL
+};
+
 enum xe_vsec {
 	XE_VSEC_UNKNOWN = 0,
 	XE_VSEC_BMG,
+	XE_VSEC_CRI,
 };
 
 static struct intel_vsec_platform_info xe_vsec_info[] = {
@@ -71,6 +110,10 @@ static struct intel_vsec_platform_info xe_vsec_info[] = {
 		.caps = VSEC_CAP_TELEMETRY | VSEC_CAP_CRASHLOG,
 		.headers = bmg_capabilities,
 	},
+	[XE_VSEC_CRI] = {
+		.caps = VSEC_CAP_TELEMETRY | VSEC_CAP_CRASHLOG,
+		.headers = cri_capabilities,
+	},
 	{ }
 };
 
-- 
2.43.0

[PATCH v13 10/16] drm/xe/vsec: Crescent Island PMT decode

[Michael J. Ruhl]

Crescent Island (CRI) has different index and offset values for
accessing the PMT data area.

Update the decode path to support the CRI device.

Update the data read callback so to support the CRI usage.

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com>
---
 drivers/gpu/drm/xe/xe_vsec.c | 100 +++++++++++++++++++++++++++++++----
 1 file changed, 90 insertions(+), 10 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
index 4ecc2d9f3f38..a814d7151eea 100644
--- a/drivers/gpu/drm/xe/xe_vsec.c
+++ b/drivers/gpu/drm/xe/xe_vsec.c
@@ -120,19 +120,25 @@ static struct intel_vsec_platform_info xe_vsec_info[] = {
 /*
  * The GUID will have the following bits to decode:
  *   [0:3]   - {Telemetry space iteration number (0,1,..)}
- *   [4:7]   - Segment (SEGMENT_INDEPENDENT-0, Client-1, Server-2)
+ *   [4:7]   - BMG Segment (SEGMENT_INDEPENDENT-0, Client-1, Server-2)
+ *   [4:5]   - CRI Segment (SEGMENT_INDEPENDENT-0, Client-1, Server-2)
+ *   [6:7]   - CRI Instance
  *   [8:11]  - SOC_SKU
  *   [12:27] – Device ID – changes for each down bin SKU’s
  *   [28:29] - Capability Type (Crashlog-0, Telemetry Aggregator-1, Watcher-2)
  *   [30:31] - Record-ID (0-PUNIT, 1-OOBMSM_0, 2-OOBMSM_1)
  */
 #define GUID_TELEM_ITERATION	GENMASK(3, 0)
-#define GUID_SEGMENT		GENMASK(7, 4)
 #define GUID_SOC_SKU		GENMASK(11, 8)
 #define GUID_DEVICE_ID		GENMASK(27, 12)
 #define GUID_CAP_TYPE		GENMASK(29, 28)
 #define GUID_RECORD_ID		GENMASK(31, 30)
 
+#define BMG_GUID_SEGMENT	GENMASK(7, 4)
+
+#define CRI_GUID_SEGMENT	GENMASK(5, 4)
+#define CRI_GUID_INSTANCE	GENMASK(7, 6)
+
 #define BMG_IDX_TELEM_PUNIT		0x00
 #define BMG_IDX_TELEM_OOBMSM		0x01
 #define BMG_IDX_CRASHLOG_PUNIT		0x02
@@ -143,6 +149,18 @@ static struct intel_vsec_platform_info xe_vsec_info[] = {
 #define BMG_OOBMSM_0_WATCHER_OFFSET	0x18D8
 #define BMG_OOBMSM_1_TELEMETRY_OFFSET	0x1000
 
+#define CRI_IDX_TELEM_DISCOVERY		0x00
+#define CRI_IDX_TELEM_PUNIT		0x01
+#define CRI_IDX_TELEM_OOBMSM		0x02
+#define CRI_IDX_CRASHLOG_PUNIT		0x03
+#define CRI_IDX_WATCHER_OOBMSM		0x03  /* PUNIT and OOBMSM share this index */
+#define CRI_IDX_CRASHLOG_OOBMSM		0x04
+
+#define CRI_PUNIT_WATCHER_OFFSET		0x08A0
+#define CRI_OOBMSM_WATCHER_OFFSET		0x0CF8
+#define CRI_OOBMSM_GFSP_TELEMETRY_OFFSET	0x1600
+#define CRI_PUNIT_CRASHLOG_OFFSET		0x0E60
+
 enum record_id {
 	PUNIT,
 	OOBMSM_0,
@@ -155,17 +173,10 @@ enum capability {
 	WATCHER,
 };
 
-static int xe_guid_decode(u32 guid, int *index, u32 *offset)
+static int bmg_guid_decode(u32 guid, int *index, u32 *offset)
 {
 	u32 record_id = FIELD_GET(GUID_RECORD_ID, guid);
 	u32 cap_type  = FIELD_GET(GUID_CAP_TYPE, guid);
-	u32 device_id = FIELD_GET(GUID_DEVICE_ID, guid);
-
-	if (device_id != BMG_DEVICE_ID)
-		return -ENODEV;
-
-	if (cap_type > WATCHER)
-		return -EINVAL;
 
 	*offset = 0;
 
@@ -201,6 +212,75 @@ static int xe_guid_decode(u32 guid, int *index, u32 *offset)
 	return 0;
 }
 
+static int cri_guid_decode(u32 guid, int *index, u32 *offset)
+{
+	u32 record_id = FIELD_GET(GUID_RECORD_ID, guid);
+	u32 cap_type  = FIELD_GET(GUID_CAP_TYPE, guid);
+	u32 instance  = FIELD_GET(CRI_GUID_INSTANCE, guid);
+
+	*offset = 0;
+
+	if (cap_type == CRASHLOG) {
+		if (record_id == PUNIT) {
+			*index = CRI_IDX_CRASHLOG_PUNIT;
+			*offset = CRI_PUNIT_CRASHLOG_OFFSET;
+		} else {
+			*index = CRI_IDX_CRASHLOG_OOBMSM;
+		}
+		return 0;
+	}
+
+	switch (record_id) {
+	case PUNIT:
+		*index = CRI_IDX_TELEM_PUNIT;
+		if (cap_type == WATCHER)
+			*offset = CRI_PUNIT_WATCHER_OFFSET;
+		break;
+
+	case OOBMSM_0:
+		*index = CRI_IDX_TELEM_OOBMSM;
+		switch (instance) {
+		case 0:
+			if (cap_type == WATCHER) {
+				*index = CRI_IDX_WATCHER_OOBMSM;
+				*offset = CRI_OOBMSM_WATCHER_OFFSET;
+			}
+			break;
+
+		case 1:
+			if (cap_type == TELEMETRY)
+				*offset = CRI_OOBMSM_GFSP_TELEMETRY_OFFSET;
+			break;
+
+		default:
+			return -EINVAL;
+		}
+		break;
+
+	default:
+		return -EINVAL;
+	}
+
+	return 0;
+}
+
+static int xe_guid_decode(u32 guid, int *index, u32 *offset)
+{
+	u32 cap_type  = FIELD_GET(GUID_CAP_TYPE, guid);
+	u32 device_id = FIELD_GET(GUID_DEVICE_ID, guid);
+
+	if (cap_type > WATCHER)
+		return -EINVAL;
+
+	if (device_id == BMG_DEVICE_ID)
+		return bmg_guid_decode(guid, index, offset);
+
+	if (device_id == CRI_DEVICE_ID)
+		return cri_guid_decode(guid, index, offset);
+
+	return -ENODEV;
+}
+
 /**
  * xe_pmt_telem_read - Given a device and a PMT GUID, read data into a buffer
  * @dev: valid Xe device
-- 
2.43.0

[PATCH v13 11/16] drm/xe/vsec: Crescent Island PMT callbacks

[Michael J. Ruhl]

CRI PMT support requires callbacks to access the discovery status and
control areas. Access is a common MMIO area that requires an index to
be set before access is allowed.

Introduce the necessary callbacks to get the status and control
information for CRI PMT usage.

Add the glue logic to register the CRI PMT functionality.

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com>
---
 drivers/gpu/drm/xe/xe_vsec.c | 115 +++++++++++++++++++++++++++++++++--
 1 file changed, 111 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
index a814d7151eea..0f6e3be520c0 100644
--- a/drivers/gpu/drm/xe/xe_vsec.c
+++ b/drivers/gpu/drm/xe/xe_vsec.c
@@ -358,20 +358,120 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
 	return ret == 0 ? count : ret;
 }
 
-static struct pmt_callbacks xe_pmt_cb = {
+/**
+ * xe_pmt_read_reg() - read a crashlog register
+ * @dev: the xe device that registered the callback
+ * @guid: PMT guid of the crashlog instance
+ * @reg: data read from the PMT data structure
+ * @offset: which data to read from the PMT data structure
+ *
+ * Read the requested PMT register based on the pcie device and guid.  The
+ * supported struct is the Crashlog Type1 Version2.
+ *
+ * Currently this is for CRI only.
+ *
+ * Return:
+ * * -ENODEV device is not available (hotplug)
+ * * -EINVAL invalid guid
+ * * 0       success
+ */
+static int xe_pmt_read_reg(struct device *dev, u32 guid, u32 *reg, u32 offset)
+{
+	struct xe_device *xe = kdev_to_xe_device(dev);
+	void __iomem *disc_addr = xe->mmio.regs + xe->pmt.base_offset;
+	int ret = 0;
+	u32 inst;
+	int idx;
+
+	if (!drm_dev_enter(&xe->drm, &idx))
+		return -ENODEV;
+
+	if (FIELD_GET(GUID_DEVICE_ID, guid) != CRI_DEVICE_ID ||
+	    FIELD_GET(GUID_CAP_TYPE, guid) != CRASHLOG) {
+		ret = -EINVAL;
+		goto dev_exit;
+	}
+
+	inst = FIELD_GET(GUID_RECORD_ID, guid) == PUNIT ?
+		CRI_CRASHLOG_PUNIT_DISC_OFFSET : CRI_CRASHLOG_OOBMSM_DISC_OFFSET;
+	disc_addr += inst + offset;
+
+	xe_pm_runtime_get(xe);
+
+	scoped_guard(mutex, &xe->pmt.lock) {
+		xe->soc_remapper.set_telem_region(xe, CRI_IDX_TELEM_DISCOVERY);
+
+		*reg = readl(disc_addr);
+	}
+
+	xe_pm_runtime_put(xe);
+
+dev_exit:
+	drm_dev_exit(idx);
+
+	return ret;
+}
+
+static int xe_pmt_write_reg(struct device *dev, u32 guid, u32 reg, u32 offset)
+{
+	struct xe_device *xe = kdev_to_xe_device(dev);
+	void __iomem *disc_addr = xe->mmio.regs + xe->pmt.base_offset;
+	int ret = 0;
+	u32 inst;
+	int idx;
+
+	if (!drm_dev_enter(&xe->drm, &idx))
+		return -ENODEV;
+
+	if (FIELD_GET(GUID_DEVICE_ID, guid) != CRI_DEVICE_ID ||
+	    FIELD_GET(GUID_CAP_TYPE, guid) != CRASHLOG) {
+		ret = -EINVAL;
+		goto dev_exit;
+	}
+
+	inst = FIELD_GET(GUID_RECORD_ID, guid) == PUNIT ?
+		CRI_CRASHLOG_PUNIT_DISC_OFFSET : CRI_CRASHLOG_OOBMSM_DISC_OFFSET;
+	disc_addr += inst + offset;
+
+	xe_pm_runtime_get(xe);
+
+	scoped_guard(mutex, &xe->pmt.lock) {
+		xe->soc_remapper.set_telem_region(xe, CRI_IDX_TELEM_DISCOVERY);
+
+		writel(reg, disc_addr);
+	}
+
+	xe_pm_runtime_put(xe);
+
+dev_exit:
+	drm_dev_exit(idx);
+
+	return ret;
+}
+
+static struct pmt_callbacks xe_bmg_pmt_cb = {
+	.read_telem = xe_pmt_telem_read,
+};
+
+static struct pmt_callbacks xe_cri_pmt_cb = {
 	.read_telem = xe_pmt_telem_read,
+	.read_reg = xe_pmt_read_reg,
+	.write_reg = xe_pmt_write_reg,
 };
 
 static const int vsec_platforms[] = {
 	[XE_BATTLEMAGE] = XE_VSEC_BMG,
+	[XE_CRESCENTISLAND] = XE_VSEC_CRI,
 };
 
 static enum xe_vsec get_platform_info(struct xe_device *xe)
 {
-	if (xe->info.platform > XE_BATTLEMAGE)
+	if (xe->info.platform > XE_CRESCENTISLAND)
 		return XE_VSEC_UNKNOWN;
 
-	if (IS_SRIOV_VF(xe) && xe->info.platform == XE_BATTLEMAGE)
+	if (IS_SRIOV_VF(xe) &&
+	    (xe->info.platform == XE_BATTLEMAGE ||
+	     xe->info.platform == XE_CRESCENTISLAND))
 		return XE_VSEC_UNKNOWN;
 
 	return vsec_platforms[xe->info.platform];
@@ -399,8 +499,15 @@ void xe_vsec_init(struct xe_device *xe)
 	switch (platform) {
 	case XE_VSEC_BMG:
 		xe->pmt.base_offset = BMG_TELEMETRY_OFFSET;
-		info->priv_data = &xe_pmt_cb;
+		info->priv_data = &xe_bmg_pmt_cb;
 		break;
+
+	case XE_VSEC_CRI:
+		xe->pmt.base_offset = CRI_PMT_OFFSET;
+		info->priv_data = &xe_cri_pmt_cb;
+		xe->soc_remapper.set_telem_region(xe, CRI_IDX_TELEM_DISCOVERY);
+		break;
+
 	default:
 		break;
 	}
-- 
2.43.0

[PATCH v13 12/16] drm/xe/vsec: Support late bind fw information

[Michael J. Ruhl]

CRI FW is loaded on power on.  Because of this, access to the FW cannot
be done until it is running.

Update the XE PMT probe and access to check for late bind devices,
verify, and wait for the appropriate FW state before probe or access.

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com>
---
 drivers/gpu/drm/xe/xe_device.c       |   4 +-
 drivers/gpu/drm/xe/xe_device_types.h |   5 ++
 drivers/gpu/drm/xe/xe_vsec.c         | 129 ++++++++++++++++++++++++++-
 drivers/gpu/drm/xe/xe_vsec.h         |   2 +-
 4 files changed, 134 insertions(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_device.c b/drivers/gpu/drm/xe/xe_device.c
index dfdd83d4150f..e8a270838666 100644
--- a/drivers/gpu/drm/xe/xe_device.c
+++ b/drivers/gpu/drm/xe/xe_device.c
@@ -1082,7 +1082,9 @@ int xe_device_probe(struct xe_device *xe)
 	if (err)
 		goto err_unregister_display;
 
-	xe_vsec_init(xe);
+	err = xe_vsec_init(xe);
+	if (err)
+		goto err_unregister_display;
 
 	xe_debugfs_register(xe);
 
diff --git a/drivers/gpu/drm/xe/xe_device_types.h b/drivers/gpu/drm/xe/xe_device_types.h
index b1c7de90002d..cc8e0e6bed9a 100644
--- a/drivers/gpu/drm/xe/xe_device_types.h
+++ b/drivers/gpu/drm/xe/xe_device_types.h
@@ -7,6 +7,7 @@
 #define _XE_DEVICE_TYPES_H_
 
 #include <linux/pci.h>
+#include <linux/workqueue.h>
 
 #include <drm/drm_device.h>
 #include <drm/drm_file.h>
@@ -485,6 +486,10 @@ struct xe_device {
 		struct mutex lock;
 		/** @pmt.base_offset: device specific base offset */
 		u64 base_offset;
+		/** @pmt.work: support late-bind probe */
+		struct delayed_work work;
+		/** @pmt.retry_count: late-bind probe retry */
+		u32 retry_count;
 	} pmt;
 
 	/** @soc_remapper: SoC remapper object */
diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
index 0f6e3be520c0..5384fef5e5ac 100644
--- a/drivers/gpu/drm/xe/xe_vsec.c
+++ b/drivers/gpu/drm/xe/xe_vsec.c
@@ -3,6 +3,7 @@
 #include <linux/bitfield.h>
 #include <linux/bits.h>
 #include <linux/cleanup.h>
+#include <linux/delay.h>
 #include <linux/errno.h>
 #include <linux/intel_vsec.h>
 #include <linux/module.h>
@@ -18,6 +19,7 @@
 #include "xe_mmio.h"
 #include "xe_platform_types.h"
 #include "xe_pm.h"
+#include "xe_sysctrl.h"
 #include "xe_vsec.h"
 
 #include "regs/xe_pmt.h"
@@ -173,6 +175,14 @@ enum capability {
 	WATCHER,
 };
 
+/*
+ * Late bind will delay 100msec for up to 20 seconds
+ */
+#define VSEC_LATE_BIND_DELAY_MSEC	100
+#define VSEC_LATE_BIND_RETRY		200
+
+static void cri_late_bind_probe(struct xe_device *xe);
+
 static int bmg_guid_decode(u32 guid, int *index, u32 *offset)
 {
 	u32 record_id = FIELD_GET(GUID_RECORD_ID, guid);
@@ -281,6 +291,48 @@ static int xe_guid_decode(u32 guid, int *index, u32 *offset)
 	return -ENODEV;
 }
 
+static void cri_late_bind_probe_work(struct work_struct *work)
+{
+	struct xe_device *xe = container_of(work, struct xe_device, pmt.work.work);
+
+	if (xe_sysctrl_is_oobmsm_fw_ready(xe)) {
+		cri_late_bind_probe(xe);
+		xe_pm_runtime_put(xe);
+		return;
+	}
+
+	xe->pmt.retry_count++;
+
+	/* wait up to 20 seconds */
+	if (xe->pmt.retry_count == VSEC_LATE_BIND_RETRY) {
+		drm_warn(&xe->drm, "PMT probe: Late Binding failed to complete\n");
+		xe_pm_runtime_put(xe);
+		return;
+	}
+
+	if (!schedule_delayed_work(&xe->pmt.work, msecs_to_jiffies(VSEC_LATE_BIND_DELAY_MSEC)))
+		xe_pm_runtime_put(xe);
+}
+
+static bool wait_for_fw(struct xe_device *xe)
+{
+	int retries = VSEC_LATE_BIND_RETRY;  /* wait up to 20 secs */
+
+	if (xe->info.platform != XE_CRESCENTISLAND)
+		return true;
+
+	while (retries--) {
+		if (xe_sysctrl_is_oobmsm_fw_ready(xe))
+			return true;
+
+		msleep(VSEC_LATE_BIND_DELAY_MSEC);
+	}
+
+	drm_warn(&xe->drm, "Late Binding failed to complete\n");
+
+	return false;
+}
+
 /**
  * xe_pmt_telem_read - Given a device and a PMT GUID, read data into a buffer
  * @dev: valid Xe device
@@ -343,6 +395,11 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
 		goto dev_exit;
 	}
 
+	if (!wait_for_fw(xe)) {
+		ret = -ENODATA;
+		goto runtime_exit;
+	}
+
 	scoped_guard(mutex, &xe->pmt.lock) {
 		/* set SoC re-mapper index register based on GUID memory region */
 		xe->soc_remapper.set_telem_region(xe, mem_region);
@@ -350,6 +407,7 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
 		memcpy_fromio(data, telem_addr, count);
 	}
 
+runtime_exit:
 	xe_pm_runtime_put(xe);
 
 dev_exit:
@@ -397,6 +455,10 @@ static int xe_pmt_read_reg(struct device *dev, u32 guid, u32 *reg, u32 offset)
 	disc_addr += inst + offset;
 
 	xe_pm_runtime_get(xe);
+	if (!wait_for_fw(xe)) {
+		ret = -ENODATA;
+		goto runtime_exit;
+	}
 
 	scoped_guard(mutex, &xe->pmt.lock) {
 		xe->soc_remapper.set_telem_region(xe, CRI_IDX_TELEM_DISCOVERY);
@@ -404,6 +466,7 @@ static int xe_pmt_read_reg(struct device *dev, u32 guid, u32 *reg, u32 offset)
 		*reg = readl(disc_addr);
 	}
 
+runtime_exit:
 	xe_pm_runtime_put(xe);
 
 dev_exit:
@@ -434,6 +497,10 @@ static int xe_pmt_write_reg(struct device *dev, u32 guid, u32 reg, u32 offset)
 	disc_addr += inst + offset;
 
 	xe_pm_runtime_get(xe);
+	if (!wait_for_fw(xe)) {
+		ret = -ENODATA;
+		goto runtime_exit;
+	}
 
 	scoped_guard(mutex, &xe->pmt.lock) {
 		xe->soc_remapper.set_telem_region(xe, CRI_IDX_TELEM_DISCOVERY);
@@ -441,6 +508,7 @@ static int xe_pmt_write_reg(struct device *dev, u32 guid, u32 reg, u32 offset)
 		writel(reg, disc_addr);
 	}
 
+runtime_exit:
 	xe_pm_runtime_put(xe);
 
 dev_exit:
@@ -477,12 +545,46 @@ static enum xe_vsec get_platform_info(struct xe_device *xe)
 	return vsec_platforms[xe->info.platform];
 }
 
+static void cri_late_bind_probe(struct xe_device *xe)
+{
+	struct intel_vsec_platform_info *info;
+	struct device *dev = xe->drm.dev;
+	enum xe_vsec platform;
+
+	platform = get_platform_info(xe);
+	if (platform != XE_VSEC_CRI)
+		return;
+
+	info = &xe_vsec_info[platform];
+	if (!info->headers)
+		return;
+
+	guard(mutex)(&xe->pmt.lock);
+
+	info->priv_data = &xe_cri_pmt_cb;
+	xe->soc_remapper.set_telem_region(xe, CRI_IDX_TELEM_DISCOVERY);
+
+	intel_vsec_register(dev, info);
+}
+
+static void vsec_disable_late_bind_work(void *arg)
+{
+	struct xe_device *xe = arg;
+
+	/*
+	 * If the queued work is canceled, the runtime reference needs to be
+	 * released here.
+	 */
+	if (disable_delayed_work_sync(&xe->pmt.work))
+		xe_pm_runtime_put(xe);
+}
+
 /**
  * xe_vsec_init - Initialize resources and add intel_vsec auxiliary
  * interface
  * @xe: valid xe instance
  */
-void xe_vsec_init(struct xe_device *xe)
+int xe_vsec_init(struct xe_device *xe)
 {
 	struct intel_vsec_platform_info *info;
 	struct device *dev = xe->drm.dev;
@@ -490,11 +592,11 @@ void xe_vsec_init(struct xe_device *xe)
 
 	platform = get_platform_info(xe);
 	if (platform == XE_VSEC_UNKNOWN)
-		return;
+		return 0;
 
 	info = &xe_vsec_info[platform];
 	if (!info->headers)
-		return;
+		return 0;
 
 	switch (platform) {
 	case XE_VSEC_BMG:
@@ -504,12 +606,26 @@ void xe_vsec_init(struct xe_device *xe)
 
 	case XE_VSEC_CRI:
 		xe->pmt.base_offset = CRI_PMT_OFFSET;
+
+		xe->pmt.retry_count = 0;
+		INIT_DELAYED_WORK(&xe->pmt.work, cri_late_bind_probe_work);
+
+		xe_pm_runtime_get_noresume(xe);
+		if (!xe_sysctrl_is_oobmsm_fw_ready(xe)) {
+			schedule_delayed_work(&xe->pmt.work,
+					      msecs_to_jiffies(VSEC_LATE_BIND_DELAY_MSEC));
+			return devm_add_action_or_reset(xe->drm.dev,
+							vsec_disable_late_bind_work,
+							xe);
+		}
+
 		info->priv_data = &xe_cri_pmt_cb;
 		xe->soc_remapper.set_telem_region(xe, CRI_IDX_TELEM_DISCOVERY);
 		break;
 
 	default:
-		break;
+		drm_err(&xe->drm, "Unsupported platform: %u\n", platform);
+		return 0;
 	}
 
 	/*
@@ -517,5 +633,10 @@ void xe_vsec_init(struct xe_device *xe)
 	 * resources.
 	 */
 	intel_vsec_register(dev, info);
+
+	if (platform == XE_VSEC_CRI)
+		xe_pm_runtime_put(xe);
+
+	return 0;
 }
 MODULE_IMPORT_NS("INTEL_VSEC");
diff --git a/drivers/gpu/drm/xe/xe_vsec.h b/drivers/gpu/drm/xe/xe_vsec.h
index a25b4e6e681b..c4a1e2fc67d8 100644
--- a/drivers/gpu/drm/xe/xe_vsec.h
+++ b/drivers/gpu/drm/xe/xe_vsec.h
@@ -9,7 +9,7 @@
 struct device;
 struct xe_device;
 
-void xe_vsec_init(struct xe_device *xe);
+int xe_vsec_init(struct xe_device *xe);
 int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offset, u32 count);
 
 #endif
-- 
2.43.0

[PATCH v13 13/16] drm/xe/vsec: Add PMT GUID internal access

[Michael J. Ruhl]

Xe accesses the PMT infrastructure directly. The current usage is
supported ONLY by BMG devices.

CRI has further requirements for access.

Add the platform id to the GUID register define and use that for the
register access.

Add an API to allow access the GUID based on the platform.

Reviewed-by: Karthik Poosa <karthik.poosa@intel.com>
Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com>
---
 drivers/gpu/drm/xe/regs/xe_pmt.h     |  4 ++-
 drivers/gpu/drm/xe/xe_debugfs.c      |  2 +-
 drivers/gpu/drm/xe/xe_device_types.h |  2 ++
 drivers/gpu/drm/xe/xe_hwmon.c        |  2 +-
 drivers/gpu/drm/xe/xe_pcode.c        |  2 +-
 drivers/gpu/drm/xe/xe_vsec.c         | 45 ++++++++++++++++++++++++++++
 drivers/gpu/drm/xe/xe_vsec.h         |  1 +
 7 files changed, 54 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/xe/regs/xe_pmt.h b/drivers/gpu/drm/xe/regs/xe_pmt.h
index 2f6658395587..fb5b5a8648ba 100644
--- a/drivers/gpu/drm/xe/regs/xe_pmt.h
+++ b/drivers/gpu/drm/xe/regs/xe_pmt.h
@@ -10,7 +10,7 @@
 #define BMG_PMT_BASE_OFFSET		0xDB000
 #define BMG_DISCOVERY_OFFSET		(SOC_BASE + BMG_PMT_BASE_OFFSET)
 
-#define PUNIT_TELEMETRY_GUID		XE_REG(BMG_DISCOVERY_OFFSET + 0x4)
+#define BMG_PUNIT_TELEMETRY_GUID	XE_REG(BMG_DISCOVERY_OFFSET + 0x4)
 #define BMG_ENERGY_STATUS_PMT_OFFSET	(0x30)
 #define	ENERGY_PKG			REG_GENMASK64(31, 0)
 #define	ENERGY_CARD			REG_GENMASK64(63, 32)
@@ -24,6 +24,8 @@
 /* for CRI discovery and telemetry are in an indexed window */
 #define CRI_PMT_OFFSET			(SOC_BASE + XE_PMT_BASE_OFFSET)
 
+#define CRI_PUNIT_TELEMETRY_GUID	XE_REG(CRI_PMT_OFFSET + 0x4)
+
 #define BMG_MODS_RESIDENCY_OFFSET		(0x4D0)
 #define BMG_G2_RESIDENCY_OFFSET		(0x530)
 #define BMG_G6_RESIDENCY_OFFSET		(0x538)
diff --git a/drivers/gpu/drm/xe/xe_debugfs.c b/drivers/gpu/drm/xe/xe_debugfs.c
index e59d4964d811..8a470fe967d5 100644
--- a/drivers/gpu/drm/xe/xe_debugfs.c
+++ b/drivers/gpu/drm/xe/xe_debugfs.c
@@ -154,7 +154,7 @@ static void read_residency_counter(struct xe_device *xe, struct xe_mmio *mmio,
 	int ret;
 
 	ret = xe_pmt_telem_read(xe->drm.dev,
-				xe_mmio_read32(mmio, PUNIT_TELEMETRY_GUID),
+				xe_mmio_read32(mmio, BMG_PUNIT_TELEMETRY_GUID),
 				&residency, offset, sizeof(residency));
 	if (ret != sizeof(residency)) {
 		drm_warn(&xe->drm, "%s counter failed to read, ret %d\n", name, ret);
diff --git a/drivers/gpu/drm/xe/xe_device_types.h b/drivers/gpu/drm/xe/xe_device_types.h
index cc8e0e6bed9a..7430185ad41a 100644
--- a/drivers/gpu/drm/xe/xe_device_types.h
+++ b/drivers/gpu/drm/xe/xe_device_types.h
@@ -490,6 +490,8 @@ struct xe_device {
 		struct delayed_work work;
 		/** @pmt.retry_count: late-bind probe retry */
 		u32 retry_count;
+		/** @pmt.punit_guid_cache: cache of the PUINT GUID */
+		u32 punit_guid_cache;
 	} pmt;
 
 	/** @soc_remapper: SoC remapper object */
diff --git a/drivers/gpu/drm/xe/xe_hwmon.c b/drivers/gpu/drm/xe/xe_hwmon.c
index 515b288f18e5..e1d86719ff67 100644
--- a/drivers/gpu/drm/xe/xe_hwmon.c
+++ b/drivers/gpu/drm/xe/xe_hwmon.c
@@ -605,7 +605,7 @@ xe_hwmon_energy_get(struct xe_hwmon *hwmon, int channel, long *energy)
 		u64 pmt_val;
 
 		ret = xe_pmt_telem_read(hwmon->xe->drm.dev,
-					xe_mmio_read32(mmio, PUNIT_TELEMETRY_GUID),
+					xe_mmio_read32(mmio, BMG_PUNIT_TELEMETRY_GUID),
 					&pmt_val, BMG_ENERGY_STATUS_PMT_OFFSET,	sizeof(pmt_val));
 		if (ret != sizeof(pmt_val)) {
 			drm_warn(&hwmon->xe->drm, "energy read from pmt failed, ret %d\n", ret);
diff --git a/drivers/gpu/drm/xe/xe_pcode.c b/drivers/gpu/drm/xe/xe_pcode.c
index 266deecbb100..8c901e54f384 100644
--- a/drivers/gpu/drm/xe/xe_pcode.c
+++ b/drivers/gpu/drm/xe/xe_pcode.c
@@ -369,7 +369,7 @@ int xe_get_pcode_version(struct xe_device *xe, struct xe_pcode_version *version)
 	guard(xe_pm_runtime)(xe);
 
 	ret = xe_pmt_telem_read(xe->drm.dev,
-				xe_mmio_read32(xe_root_tile_mmio(xe), PUNIT_TELEMETRY_GUID),
+				xe_mmio_read32(xe_root_tile_mmio(xe), BMG_PUNIT_TELEMETRY_GUID),
 				(u64 *)version, PUNIT_VERSION_OFFSET, sizeof(*version));
 	if (ret != sizeof(*version)) {
 		xe_warn(xe, "pcode version read from PMT failed, ret %pe\n", ERR_PTR(ret));
diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
index 5384fef5e5ac..1ed904ec1496 100644
--- a/drivers/gpu/drm/xe/xe_vsec.c
+++ b/drivers/gpu/drm/xe/xe_vsec.c
@@ -579,6 +579,51 @@ static void vsec_disable_late_bind_work(void *arg)
 		xe_pm_runtime_put(xe);
 }
 
+int xe_vsec_get_guid(struct xe_device *xe, u32 *guid)
+{
+	struct xe_mmio *mmio = xe_root_tile_mmio(xe);
+	int ret = 0;
+
+	xe_assert(xe, !IS_SRIOV_VF(xe));
+
+	if (!xe_pm_runtime_get_if_active(xe))
+		return -ENODATA;
+
+	mutex_lock(&xe->pmt.lock);
+
+	if (xe->pmt.punit_guid_cache) {
+		*guid = xe->pmt.punit_guid_cache;
+		goto unlock;
+	}
+
+	switch (xe->info.platform) {
+	case XE_BATTLEMAGE:
+		*guid = xe_mmio_read32(mmio, BMG_PUNIT_TELEMETRY_GUID);
+		break;
+
+	case XE_CRESCENTISLAND:
+		xe->soc_remapper.set_telem_region(xe, CRI_IDX_TELEM_DISCOVERY);
+		*guid = xe_mmio_read32(mmio, CRI_PUNIT_TELEMETRY_GUID);
+		break;
+
+	default:
+		*guid = 0;
+		drm_err(&xe->drm, "Unsupported platform: %u\n", xe->info.platform);
+		ret = -EINVAL;
+		break;
+	}
+
+	/* the GUID is static, read once and cache for all requests */
+	if (*guid)
+		xe->pmt.punit_guid_cache = *guid;
+
+unlock:
+	mutex_unlock(&xe->pmt.lock);
+	xe_pm_runtime_put(xe);
+
+	return ret;
+}
+
 /**
  * xe_vsec_init - Initialize resources and add intel_vsec auxiliary
  * interface
diff --git a/drivers/gpu/drm/xe/xe_vsec.h b/drivers/gpu/drm/xe/xe_vsec.h
index c4a1e2fc67d8..50187fdc1207 100644
--- a/drivers/gpu/drm/xe/xe_vsec.h
+++ b/drivers/gpu/drm/xe/xe_vsec.h
@@ -10,6 +10,7 @@ struct device;
 struct xe_device;
 
 int xe_vsec_init(struct xe_device *xe);
+int xe_vsec_get_guid(struct xe_device *xe, u32 *guid);
 int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offset, u32 count);
 
 #endif
-- 
2.43.0

[PATCH v13 14/16] drm/xe/vsec: Update PMT internal access

[Michael J. Ruhl]

Current PMT internal access assumes BMG devices.

Use the new PMT GUID API to read the device specific guid.

Minor cleanup for a newly unused parameter (mmio).

Reviewed-by: Karthik Poosa <karthik.poosa@intel.com>
Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com>
---
 drivers/gpu/drm/xe/xe_debugfs.c | 26 ++++++++++++++------------
 drivers/gpu/drm/xe/xe_hwmon.c   | 10 ++++++++--
 drivers/gpu/drm/xe/xe_pcode.c   | 10 ++++++++--
 3 files changed, 30 insertions(+), 16 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_debugfs.c b/drivers/gpu/drm/xe/xe_debugfs.c
index 8a470fe967d5..b56e4aad4069 100644
--- a/drivers/gpu/drm/xe/xe_debugfs.c
+++ b/drivers/gpu/drm/xe/xe_debugfs.c
@@ -5,6 +5,7 @@
 
 #include "xe_debugfs.h"
 
+#include <linux/array_size.h>
 #include <linux/bits.h>
 #include <linux/debugfs.h>
 #include <linux/fault-inject.h>
@@ -19,7 +20,6 @@
 #include "xe_gt.h"
 #include "xe_gt_debugfs.h"
 #include "xe_hw_engine.h"
-#include "xe_mmio.h"
 #include "xe_pagefault.h"
 #include "xe_pcode.h"
 #include "xe_pm.h"
@@ -147,15 +147,20 @@ static void xe_fault_inject_debugfs_register(struct xe_device *xe,
 	}
 }
 
-static void read_residency_counter(struct xe_device *xe, struct xe_mmio *mmio,
-				   u32 offset, const char *name, struct drm_printer *p)
+static void read_residency_counter(struct xe_device *xe, u32 offset, const char *name,
+				   struct drm_printer *p)
 {
 	u64 residency = 0;
+	u32 guid;
 	int ret;
 
-	ret = xe_pmt_telem_read(xe->drm.dev,
-				xe_mmio_read32(mmio, BMG_PUNIT_TELEMETRY_GUID),
-				&residency, offset, sizeof(residency));
+	ret = xe_vsec_get_guid(xe, &guid);
+	if (ret) {
+		drm_warn(&xe->drm, "PMT device is not available: %d\n", ret);
+		return;
+	}
+
+	ret = xe_pmt_telem_read(xe->drm.dev, guid, &residency, offset, sizeof(residency));
 	if (ret != sizeof(residency)) {
 		drm_warn(&xe->drm, "%s counter failed to read, ret %d\n", name, ret);
 		return;
@@ -294,13 +299,12 @@ static int pcode_info(struct seq_file *m, void *data)
 static int dgfx_pkg_residencies_show(struct seq_file *m, void *data)
 {
 	struct xe_device *xe;
-	struct xe_mmio *mmio;
 	struct drm_printer p;
 
 	xe = node_to_xe(m->private);
 	p = drm_seq_file_printer(m);
 	guard(xe_pm_runtime)(xe);
-	mmio = xe_root_tile_mmio(xe);
+
 	static const struct {
 		u32 offset;
 		const char *name;
@@ -314,7 +318,7 @@ static int dgfx_pkg_residencies_show(struct seq_file *m, void *data)
 	};
 
 	for (int i = 0; i < ARRAY_SIZE(residencies); i++)
-		read_residency_counter(xe, mmio, residencies[i].offset, residencies[i].name, &p);
+		read_residency_counter(xe, residencies[i].offset, residencies[i].name, &p);
 
 	return 0;
 }
@@ -322,13 +326,11 @@ static int dgfx_pkg_residencies_show(struct seq_file *m, void *data)
 static int dgfx_pcie_link_residencies_show(struct seq_file *m, void *data)
 {
 	struct xe_device *xe;
-	struct xe_mmio *mmio;
 	struct drm_printer p;
 
 	xe = node_to_xe(m->private);
 	p = drm_seq_file_printer(m);
 	guard(xe_pm_runtime)(xe);
-	mmio = xe_root_tile_mmio(xe);
 
 	static const struct {
 		u32 offset;
@@ -340,7 +342,7 @@ static int dgfx_pcie_link_residencies_show(struct seq_file *m, void *data)
 	};
 
 	for (int i = 0; i < ARRAY_SIZE(residencies); i++)
-		read_residency_counter(xe, mmio, residencies[i].offset, residencies[i].name, &p);
+		read_residency_counter(xe, residencies[i].offset, residencies[i].name, &p);
 
 	return 0;
 }
diff --git a/drivers/gpu/drm/xe/xe_hwmon.c b/drivers/gpu/drm/xe/xe_hwmon.c
index e1d86719ff67..93e3b086a258 100644
--- a/drivers/gpu/drm/xe/xe_hwmon.c
+++ b/drivers/gpu/drm/xe/xe_hwmon.c
@@ -603,9 +603,15 @@ xe_hwmon_energy_get(struct xe_hwmon *hwmon, int channel, long *energy)
 
 	if (hwmon->xe->info.platform == XE_BATTLEMAGE) {
 		u64 pmt_val;
+		u32 guid;
 
-		ret = xe_pmt_telem_read(hwmon->xe->drm.dev,
-					xe_mmio_read32(mmio, BMG_PUNIT_TELEMETRY_GUID),
+		ret = xe_vsec_get_guid(hwmon->xe, &guid);
+		if (ret) {
+			drm_warn(&hwmon->xe->drm, "PMT device is not available: %d\n", ret);
+			*energy = 0;
+			return ret;
+		}
+		ret = xe_pmt_telem_read(hwmon->xe->drm.dev, guid,
 					&pmt_val, BMG_ENERGY_STATUS_PMT_OFFSET,	sizeof(pmt_val));
 		if (ret != sizeof(pmt_val)) {
 			drm_warn(&hwmon->xe->drm, "energy read from pmt failed, ret %d\n", ret);
diff --git a/drivers/gpu/drm/xe/xe_pcode.c b/drivers/gpu/drm/xe/xe_pcode.c
index 8c901e54f384..fa9f45d565cb 100644
--- a/drivers/gpu/drm/xe/xe_pcode.c
+++ b/drivers/gpu/drm/xe/xe_pcode.c
@@ -365,11 +365,17 @@ ALLOW_ERROR_INJECTION(xe_pcode_probe_early, ERRNO); /* See xe_pci_probe */
 int xe_get_pcode_version(struct xe_device *xe, struct xe_pcode_version *version)
 {
 	int ret = 0;
+	u32 guid;
 
 	guard(xe_pm_runtime)(xe);
 
-	ret = xe_pmt_telem_read(xe->drm.dev,
-				xe_mmio_read32(xe_root_tile_mmio(xe), BMG_PUNIT_TELEMETRY_GUID),
+	ret = xe_vsec_get_guid(xe, &guid);
+	if (ret) {
+		xe_warn(xe, "PMT device is not available: %d\n", ret);
+		return ret;
+	}
+
+	ret = xe_pmt_telem_read(xe->drm.dev, guid,
 				(u64 *)version, PUNIT_VERSION_OFFSET, sizeof(*version));
 	if (ret != sizeof(*version)) {
 		xe_warn(xe, "pcode version read from PMT failed, ret %pe\n", ERR_PTR(ret));
-- 
2.43.0

[PATCH v13 15/16] drm/xe/vsec: Refactor platform check

[Michael J. Ruhl]

For debugfs, residencies and pcode_info are available on the BMG and
CRI (!SRIOV_VF) platforms.

Refactor to have one code path for adding the debugfs entries.

Reviewed-by: Karthik Poosa <karthik.poosa@intel.com>
Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com>
---
 drivers/gpu/drm/xe/xe_debugfs.c | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_debugfs.c b/drivers/gpu/drm/xe/xe_debugfs.c
index b56e4aad4069..5a82f8eab2c6 100644
--- a/drivers/gpu/drm/xe/xe_debugfs.c
+++ b/drivers/gpu/drm/xe/xe_debugfs.c
@@ -702,23 +702,22 @@ void xe_debugfs_register(struct xe_device *xe)
 				 ARRAY_SIZE(debugfs_list),
 				 root, minor);
 
-	if (xe->info.platform == XE_BATTLEMAGE && !IS_SRIOV_VF(xe)) {
-		drm_debugfs_create_files(debugfs_residencies,
-					 ARRAY_SIZE(debugfs_residencies),
-					 root, minor);
-	}
-
 	/*
-	 * Pcode version read from PMT is currently only supported on CRI and BMG platforms in PF
-	 * mode, as both platforms support the necessary telemetry read mechanism and have a fixed
-	 * PUNIT_VERSION_OFFSET.
+	 * Residencies and Pcode version read from PMT is currently only supported on CRI and BMG
+	 * platforms in PF mode.  Both platforms support the necessary telemetry read mechanism
+	 * and have a fixed offsets for the required data.
 	 * Attempting this access on other platforms must be verified before enabling support.
 	 */
 	if (!IS_SRIOV_VF(xe) &&
-	    (xe->info.platform == XE_CRESCENTISLAND || xe->info.platform == XE_BATTLEMAGE))
+	    (xe->info.platform == XE_CRESCENTISLAND || xe->info.platform == XE_BATTLEMAGE)) {
+		drm_debugfs_create_files(debugfs_residencies,
+					 ARRAY_SIZE(debugfs_residencies),
+					 root, minor);
+
 		drm_debugfs_create_files(pcode_info_debugfs,
 					 ARRAY_SIZE(pcode_info_debugfs),
 					 root, minor);
+	}
 
 	debugfs_create_file("forcewake_all", 0400, root, xe,
 			    &forcewake_all_fops);
-- 
2.43.0

[PATCH v13 16/16] drm/xe/vsec: Support survivability mode for CRI

[Michael J. Ruhl]

For survivability mode, CRI can only support access to the
PUNIT CRASHLOG PMT function.

Create a special config to support survivability mode.

Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com>
---
 drivers/gpu/drm/xe/xe_survivability_mode.c |  2 ++
 drivers/gpu/drm/xe/xe_vsec.c               | 34 +++++++++++++++++++++-
 2 files changed, 35 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/xe/xe_survivability_mode.c b/drivers/gpu/drm/xe/xe_survivability_mode.c
index d73cdad0b3a9..4ce84bafcbf5 100644
--- a/drivers/gpu/drm/xe/xe_survivability_mode.c
+++ b/drivers/gpu/drm/xe/xe_survivability_mode.c
@@ -19,6 +19,7 @@
 #include "xe_nvm.h"
 #include "xe_pcode_api.h"
 #include "xe_printk.h"
+#include "xe_soc_remapper.h"
 #include "xe_vsec.h"
 #include "xe_wedge.h"
 
@@ -352,6 +353,7 @@ static int enable_boot_survivability_mode(struct pci_dev *pdev)
 
 	xe_heci_gsc_init(xe);
 
+	xe_soc_remapper_init(xe);
 	xe_vsec_init(xe);
 
 	if (survivability->fdo_mode) {
diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
index 1ed904ec1496..6d4ef9cb9f60 100644
--- a/drivers/gpu/drm/xe/xe_vsec.c
+++ b/drivers/gpu/drm/xe/xe_vsec.c
@@ -101,10 +101,26 @@ static struct intel_vsec_header *cri_capabilities[] = {
 	NULL
 };
 
+static struct intel_vsec_header cri_survivability_crashlog = {
+	.rev = 1,
+	.length = SZ_16,
+	.id = VSEC_ID_CRASHLOG,
+	.num_entries = 1,
+	.entry_size = 6,
+	.tbir = 0,
+	.offset = CRI_PMT_OFFSET + CRI_CRASHLOG_PUNIT_DISC_OFFSET,
+};
+
+static struct intel_vsec_header *cri_survivability_capabilities[] = {
+	&cri_survivability_crashlog,
+	NULL
+};
+
 enum xe_vsec {
 	XE_VSEC_UNKNOWN = 0,
 	XE_VSEC_BMG,
 	XE_VSEC_CRI,
+	XE_VSEC_CRI_SURVIVABILITY,
 };
 
 static struct intel_vsec_platform_info xe_vsec_info[] = {
@@ -116,6 +132,10 @@ static struct intel_vsec_platform_info xe_vsec_info[] = {
 		.caps = VSEC_CAP_TELEMETRY | VSEC_CAP_CRASHLOG,
 		.headers = cri_capabilities,
 	},
+	[XE_VSEC_CRI_SURVIVABILITY] = {
+		.caps = VSEC_CAP_CRASHLOG,
+		.headers = cri_survivability_capabilities,
+	},
 	{ }
 };
 
@@ -316,9 +336,10 @@ static void cri_late_bind_probe_work(struct work_struct *work)
 
 static bool wait_for_fw(struct xe_device *xe)
 {
+	struct xe_survivability *survivability = &xe->survivability;
 	int retries = VSEC_LATE_BIND_RETRY;  /* wait up to 20 secs */
 
-	if (xe->info.platform != XE_CRESCENTISLAND)
+	if (xe->info.platform != XE_CRESCENTISLAND || survivability->mode)
 		return true;
 
 	while (retries--) {
@@ -534,6 +555,8 @@ static const int vsec_platforms[] = {
 
 static enum xe_vsec get_platform_info(struct xe_device *xe)
 {
+	struct xe_survivability *survivability = &xe->survivability;
+
 	if (xe->info.platform > XE_CRESCENTISLAND)
 		return XE_VSEC_UNKNOWN;
 
@@ -542,6 +565,9 @@ static enum xe_vsec get_platform_info(struct xe_device *xe)
 	     xe->info.platform == XE_CRESCENTISLAND))
 		return XE_VSEC_UNKNOWN;
 
+	if (survivability->mode && xe->info.platform == XE_CRESCENTISLAND)
+		return XE_VSEC_CRI_SURVIVABILITY;
+
 	return vsec_platforms[xe->info.platform];
 }
 
@@ -668,6 +694,12 @@ int xe_vsec_init(struct xe_device *xe)
 		xe->soc_remapper.set_telem_region(xe, CRI_IDX_TELEM_DISCOVERY);
 		break;
 
+	case XE_VSEC_CRI_SURVIVABILITY:
+		xe->pmt.base_offset = CRI_PMT_OFFSET;
+		info->priv_data = &xe_cri_pmt_cb;
+		xe->soc_remapper.set_telem_region(xe, CRI_IDX_TELEM_DISCOVERY);
+		break;
+
 	default:
 		drm_err(&xe->drm, "Unsupported platform: %u\n", platform);
 		return 0;
-- 
2.43.0

Re: [PATCH v13 01/16] drm/xe/vsec: Do not register BMG PMT for VF

[sashiko-bot]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [High] drm/xe/vsec: NULL pointer dereference in xe_pmt_telem_read() in survivability mode
--

commit ce97359d953d35ffff16565daa34557b5236e227
Author: Michael J. Ruhl <michael.j.ruhl@intel.com>

drm/xe/vsec: Do not register BMG PMT for VF

This commit skips VSEC PMT registration on Battlemage SR-IOV VF devices,
adds a VF assertion to xe_pmt_telem_read(), and removes the NULL check on
xe->soc_remapper.set_telem_region.

> diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
> index a9baf0bfe572c..178f22db40461 100644
> --- a/drivers/gpu/drm/xe/xe_vsec.c
> +++ b/drivers/gpu/drm/xe/xe_vsec.c
> @@ -149,6 +149,8 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
>  	u32 offset;
>  	int ret;
>  
> +	xe_assert(xe, !IS_SRIOV_VF(xe));
> +
>  	ret = xe_guid_decode(guid, &mem_region, &offset);
>  	if (ret)
>  		return ret;
> @@ -157,9 +159,6 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
>  
>  	guard(mutex)(&xe->pmt.lock);
>  
> -	if (!xe->soc_remapper.set_telem_region)
> -		return -ENODEV;
> -

[Severity: High]
Can removing the xe->soc_remapper.set_telem_region check here cause a NULL
function pointer dereference in xe_pmt_telem_read() when a Battlemage PF
enters boot survivability mode?

When boot survivability mode is enabled during xe_device_probe_early(),
enable_boot_survivability_mode() initializes VSEC and registers the PMT
callbacks:

drivers/gpu/drm/xe/xe_survivability_mode.c:enable_boot_survivability_mode() {
	...
	xe_heci_gsc_init(xe);

	xe_vsec_init(xe);
	...
}

Then __xe_pci_probe() returns early before calling xe_device_probe():

drivers/gpu/drm/xe/xe_pci.c:__xe_pci_probe() {
	...
	err = xe_device_probe_early(xe);
	if (xe_survivability_mode_is_boot_enabled(xe))
		return 0;
	...
	err = xe_device_probe(xe);
}

Because xe_soc_remapper_init(xe) is only called inside xe_device_probe(),
xe->soc_remapper.set_telem_region remains NULL in boot survivability mode.
Meanwhile, local_pci_probe() holds a runtime PM reference, so
xe_pm_runtime_get_if_active(xe) returns true. If userspace reads the
exposed PMT telemetry or crashlog sysfs attributes, xe_pmt_telem_read()
will call xe->soc_remapper.set_telem_region(xe, mem_region) when it is
NULL.

It looks like a later patch in the series, commit 6e126caaac7b
("drm/xe/vsec: Support survivability mode for CRI"), adds a call to
xe_soc_remapper_init(xe) in enable_boot_survivability_mode(), but at this
point in the series (and if that change is not backported together with
this one), reading PMT sysfs files in boot survivability mode will crash.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261102221314.119028-18-michael.j.ruhl@intel.com?part=1

Re: [PATCH v13 14/16] drm/xe/vsec: Update PMT internal access

[sashiko-bot]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] drm/xe/vsec: missing wait_for_fw() and invalid caching in xe_vsec_get_guid()
--

commit ff1f060a60a0a4bf5c3f7818992f8347fb84bca2
Author: Michael J. Ruhl <michael.j.ruhl@intel.com>

drm/xe/vsec: Update PMT internal access

This commit updates read_residency_counter(), xe_hwmon_energy_get(), and
xe_get_pcode_version() to retrieve the device-specific PMT telemetry GUID via
xe_vsec_get_guid() instead of reading BMG_PUNIT_TELEMETRY_GUID directly.

> diff --git a/drivers/gpu/drm/xe/xe_pcode.c b/drivers/gpu/drm/xe/xe_pcode.c
> index 8c901e54f3847..fa9f45d565cb9 100644
> --- a/drivers/gpu/drm/xe/xe_pcode.c
> +++ b/drivers/gpu/drm/xe/xe_pcode.c
> @@ -365,11 +365,17 @@ ALLOW_ERROR_INJECTION(xe_pcode_probe_early, ERRNO); /* See xe_pci_probe */
>  int xe_get_pcode_version(struct xe_device *xe, struct xe_pcode_version *version)
>  {
>  	int ret = 0;
> +	u32 guid;
>  
>  	guard(xe_pm_runtime)(xe);
>  
> -	ret = xe_pmt_telem_read(xe->drm.dev,
> -				xe_mmio_read32(xe_root_tile_mmio(xe), BMG_PUNIT_TELEMETRY_GUID),
> +	ret = xe_vsec_get_guid(xe, &guid);
> +	if (ret) {
> +		xe_warn(xe, "PMT device is not available: %d\n", ret);
> +		return ret;
> +	}
> +
> +	ret = xe_pmt_telem_read(xe->drm.dev, guid,
>  				(u64 *)version, PUNIT_VERSION_OFFSET, sizeof(*version));

[Severity: Medium]
On XE_CRESCENTISLAND, can calling xe_vsec_get_guid() before xe_pmt_telem_read()
access the CRI PMT discovery window before OOBMSM firmware is ready and
corrupt punit_guid_cache?

In xe_debugfs_register(), pcode_info is registered on both XE_BATTLEMAGE and
XE_CRESCENTISLAND, and reading it invokes:

  pcode_info() -> xe_get_pcode_version() -> xe_vsec_get_guid()

Unlike xe_pmt_telem_read(), xe_pmt_read_reg(), and xe_pmt_write_reg(),
xe_vsec_get_guid() in drivers/gpu/drm/xe/xe_vsec.c does not call wait_for_fw()
before programming the SoC remapper and reading CRI_PUNIT_TELEMETRY_GUID:

drivers/gpu/drm/xe/xe_vsec.c:xe_vsec_get_guid() {
	...
	case XE_CRESCENTISLAND:
		xe->soc_remapper.set_telem_region(xe, CRI_IDX_TELEM_DISCOVERY);
		*guid = xe_mmio_read32(mmio, CRI_PUNIT_TELEMETRY_GUID);
		break;
	...
	/* the GUID is static, read once and cache for all requests */
	if (*guid)
		xe->pmt.punit_guid_cache = *guid;
}

If pcode_info (or read_residency_counter()) is read while late-bound OOBMSM
firmware is not yet ready (for example during initial probe while
cri_late_bind_probe_work is polling, or after runtime resume from D3cold),
reading CRI_PUNIT_TELEMETRY_GUID before wait_for_fw() has two failure modes:

1. If the premature read returns 0, xe_vsec_get_guid() still returns 0 with
   *guid == 0, and xe_pmt_telem_read() then fails immediately in
   xe_guid_decode() with -ENODEV before reaching its own wait_for_fw() check:

drivers/gpu/drm/xe/xe_vsec.c:xe_pmt_telem_read() {
	...
	ret = xe_guid_decode(guid, &mem_region, &offset);
	if (ret)
		goto dev_exit;
	...
	if (!wait_for_fw(xe)) {
		ret = -ENODATA;
		goto runtime_exit;
	}
}

2. If the premature read returns a non-zero uninitialized or all-ones value
   (such as 0xffffffff), xe_vsec_get_guid() caches it in
   xe->pmt.punit_guid_cache permanently, causing all subsequent calls to
   xe_vsec_get_guid(), xe_get_pcode_version(), and read_residency_counter() to
   fail even after OOBMSM firmware becomes ready.

Should xe_vsec_get_guid() call wait_for_fw(xe) before reading the GUID (and
validate the GUID before caching it)?

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261102221314.119028-18-michael.j.ruhl@intel.com?part=14

Re: [PATCH v13 11/16] drm/xe/vsec: Crescent Island PMT callbacks

[sashiko-bot]

Thank you for your contribution! Sashiko AI review found 2 potential issue(s) to consider:
- [Medium] drm/xe/vsec: xe_pmt_telem_read() leaves CRI discovery window remapped
- [Low] drm/xe/vsec: mixed goto and scoped_guard() cleanup in reg callbacks
--

commit dc4cd399324f5200a61e70ed93f8ba5b2a90435c
Author: Michael J. Ruhl <michael.j.ruhl@intel.com>

drm/xe/vsec: Crescent Island PMT callbacks

This commit introduces Crescent Island PMT register read and write callbacks
to access the discovery status and control areas via the indexed MMIO window
and registers the CRI PMT functionality.

> diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
> index a814d7151eeae..0f6e3be520c09 100644
> --- a/drivers/gpu/drm/xe/xe_vsec.c
> +++ b/drivers/gpu/drm/xe/xe_vsec.c
> @@ -358,20 +358,120 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
>  	return ret == 0 ? count : ret;
>  }
>  
> -static struct pmt_callbacks xe_pmt_cb = {
> +/**
> + * xe_pmt_read_reg() - read a crashlog register
> + * @dev: the xe device that registered the callback
> + * @guid: PMT guid of the crashlog instance
> + * @reg: data read from the PMT data structure
> + * @offset: which data to read from the PMT data structure
> + *
> + * Read the requested PMT register based on the pcie device and guid.  The
> + * supported struct is the Crashlog Type1 Version2.
> + *
> + * Currently this is for CRI only.
> + *
> + * Return:
> + * * -ENODEV device is not available (hotplug)
> + * * -EINVAL invalid guid
> + * * 0       success
> + */
> +static int xe_pmt_read_reg(struct device *dev, u32 guid, u32 *reg, u32 offset)
> +{
> +	struct xe_device *xe = kdev_to_xe_device(dev);
> +	void __iomem *disc_addr = xe->mmio.regs + xe->pmt.base_offset;
> +	int ret = 0;
> +	u32 inst;
> +	int idx;
> +
> +	if (!drm_dev_enter(&xe->drm, &idx))
> +		return -ENODEV;
> +
> +	if (FIELD_GET(GUID_DEVICE_ID, guid) != CRI_DEVICE_ID ||
> +	    FIELD_GET(GUID_CAP_TYPE, guid) != CRASHLOG) {
> +		ret = -EINVAL;
> +		goto dev_exit;
> +	}
> +
> +	inst = FIELD_GET(GUID_RECORD_ID, guid) == PUNIT ?
> +		CRI_CRASHLOG_PUNIT_DISC_OFFSET : CRI_CRASHLOG_OOBMSM_DISC_OFFSET;
> +	disc_addr += inst + offset;
> +
> +	xe_pm_runtime_get(xe);
> +
> +	scoped_guard(mutex, &xe->pmt.lock) {
> +		xe->soc_remapper.set_telem_region(xe, CRI_IDX_TELEM_DISCOVERY);
> +
> +		*reg = readl(disc_addr);
> +	}
> +
> +	xe_pm_runtime_put(xe);
> +
> +dev_exit:
> +	drm_dev_exit(idx);
> +
> +	return ret;
> +}
> +
> +static int xe_pmt_write_reg(struct device *dev, u32 guid, u32 reg, u32 offset)
> +{
> +	struct xe_device *xe = kdev_to_xe_device(dev);
> +	void __iomem *disc_addr = xe->mmio.regs + xe->pmt.base_offset;
> +	int ret = 0;
> +	u32 inst;
> +	int idx;
> +
> +	if (!drm_dev_enter(&xe->drm, &idx))
> +		return -ENODEV;
> +
> +	if (FIELD_GET(GUID_DEVICE_ID, guid) != CRI_DEVICE_ID ||
> +	    FIELD_GET(GUID_CAP_TYPE, guid) != CRASHLOG) {
> +		ret = -EINVAL;
> +		goto dev_exit;
> +	}
> +
> +	inst = FIELD_GET(GUID_RECORD_ID, guid) == PUNIT ?
> +		CRI_CRASHLOG_PUNIT_DISC_OFFSET : CRI_CRASHLOG_OOBMSM_DISC_OFFSET;
> +	disc_addr += inst + offset;
> +
> +	xe_pm_runtime_get(xe);
> +
> +	scoped_guard(mutex, &xe->pmt.lock) {
> +		xe->soc_remapper.set_telem_region(xe, CRI_IDX_TELEM_DISCOVERY);
> +
> +		writel(reg, disc_addr);
> +	}
> +
> +	xe_pm_runtime_put(xe);
> +
> +dev_exit:
> +	drm_dev_exit(idx);
> +
> +	return ret;
> +}

[Severity: Low]
As reported by Sashiko on v11 [1], this isn't a bug, but both
xe_pmt_read_reg() and xe_pmt_write_reg() in drivers/gpu/drm/xe/xe_vsec.c mix
goto-based cleanup (goto dev_exit) with scope-based cleanup
(scoped_guard(mutex, &xe->pmt.lock)) in the same function, and duplicate
nearly identical bodies differing only in readl() vs writel().

Since the guid validation only inspects the u32 guid argument, could that
check be performed before drm_dev_enter() to avoid the dev_exit label (or
convert the remaining cleanup in these functions to a consistent pattern),
and could the shared setup between xe_pmt_read_reg() and xe_pmt_write_reg()
be factored into a common helper?

[ ... ]

> @@ -399,8 +499,15 @@ void xe_vsec_init(struct xe_device *xe)
>  	switch (platform) {
>  	case XE_VSEC_BMG:
>  		xe->pmt.base_offset = BMG_TELEMETRY_OFFSET;
> -		info->priv_data = &xe_pmt_cb;
> +		info->priv_data = &xe_bmg_pmt_cb;
>  		break;
> +
> +	case XE_VSEC_CRI:
> +		xe->pmt.base_offset = CRI_PMT_OFFSET;
> +		info->priv_data = &xe_cri_pmt_cb;
> +		xe->soc_remapper.set_telem_region(xe, CRI_IDX_TELEM_DISCOVERY);
> +		break;

[Severity: Medium]
Can a call to xe_pmt_telem_read() leave the shared CRI_PMT_OFFSET MMIO window
banked to a telemetry or crashlog payload region instead of
CRI_IDX_TELEM_DISCOVERY when PMT discovery headers are read during probe?

On Crescent Island, both the discovery tables and the telemetry/crashlog data
banks share the CRI_PMT_OFFSET window controlled by set_telem_region().
During auxiliary driver probe in pmt_telem_probe() and pmt_crashlog_probe(),
the Intel PMT drivers read the discovery headers directly from the ioremapped
CRI_PMT_OFFSET window without calling read_reg() or holding xe->pmt.lock:

drivers/platform/x86/intel/pmt/class.c:pmt_get_headers() {
    ...
	memset(entry->disc_header, 0, header_bytes);
	memcpy_fromio(entry->disc_header, disc_table,
		      min(header_bytes, resource_size(disc_res)));

	/* Used by crashlog driver */
	entry->disc_table = disc_table;
    ...
}

and:

drivers/platform/x86/intel/pmt/crashlog.c:pmt_crashlog_header_decode() {
    ...
	header->access_type = GET_ACCESS(readl(disc_table));
	header->guid = readl(disc_table + GUID_OFFSET);
	header->base_offset = readl(disc_table + BASE_OFFSET);

	/* Size is measured in DWORDS, but accessor returns bytes */
	header->size = GET_SIZE(readl(disc_table + SIZE_OFFSET));
    ...
}

However, in drivers/gpu/drm/xe/xe_vsec.c, xe_pmt_telem_read() switches the
remapper index to mem_region and does not restore CRI_IDX_TELEM_DISCOVERY
before releasing xe->pmt.lock:

	scoped_guard(mutex, &xe->pmt.lock) {
		/* set SoC re-mapper index register based on GUID memory region */
		xe->soc_remapper.set_telem_region(xe, mem_region);

		memcpy_fromio(data, telem_addr, count);
	}

In addition, xe_vsec_init() sets CRI_IDX_TELEM_DISCOVERY without holding
xe->pmt.lock.

If pmt_telemetry or pmt_crashlog probes asynchronously as a module or is
rebound via sysfs after any xe_pmt_telem_read() call (such as from debugfs
residency reads or an earlier telemetry sysfs read), wouldn't
pmt_get_headers() and pmt_crashlog_header_decode() read telemetry or crashlog
payload data instead of the discovery headers?

[1] https://lore.kernel.org/r/20260923181115.2514193-38-michael.j.ruhl@intel.com

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261102221314.119028-18-michael.j.ruhl@intel.com?part=11

Re: [PATCH v13 16/16] drm/xe/vsec: Support survivability mode for CRI

[sashiko-bot]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] drm/xe/vsec: NULL sc->mmio dereference in wait_for_fw() on survivability error
--

commit 6e126caaac7b556b33b3c1891921257efd2a418a
Author: Michael J. Ruhl <michael.j.ruhl@intel.com>

drm/xe/vsec: Support survivability mode for CRI

This commit adds a VSEC configuration for Crescent Island in survivability
mode to expose the PUNIT crashlog PMT capability and initializes the SoC
remapper during boot survivability mode.

> diff --git a/drivers/gpu/drm/xe/xe_survivability_mode.c b/drivers/gpu/drm/xe/xe_survivability_mode.c
> index d73cdad0b3a9c..4ce84bafcbf57 100644
> --- a/drivers/gpu/drm/xe/xe_survivability_mode.c
> +++ b/drivers/gpu/drm/xe/xe_survivability_mode.c
[ ... ]
> @@ -352,6 +353,7 @@ static int enable_boot_survivability_mode(struct pci_dev *pdev)
>  
>  	xe_heci_gsc_init(xe);
>  
> +	xe_soc_remapper_init(xe);
>  	xe_vsec_init(xe);
>  
>  	if (survivability->fdo_mode) {
> diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
> index 1ed904ec14965..6d4ef9cb9f60f 100644
> --- a/drivers/gpu/drm/xe/xe_vsec.c
> +++ b/drivers/gpu/drm/xe/xe_vsec.c
[ ... ]
> @@ -316,9 +336,10 @@ static void cri_late_bind_probe_work(struct work_struct *work)
>  
>  static bool wait_for_fw(struct xe_device *xe)
>  {
> +	struct xe_survivability *survivability = &xe->survivability;
>  	int retries = VSEC_LATE_BIND_RETRY;  /* wait up to 20 secs */
>  
> -	if (xe->info.platform != XE_CRESCENTISLAND)
> +	if (xe->info.platform != XE_CRESCENTISLAND || survivability->mode)
>  		return true;
>  
>  	while (retries--) {

[Severity: Medium]
Can this dereference a NULL xe->sc.mmio pointer and lock an uninitialized
xe->sc.cmd_lock mutex if enable_boot_survivability_mode() fails after
xe_vsec_init()?

During early probe in xe_device_probe_early(), xe_sysctrl_init() has not run
yet, so xe->sc.cmd_lock is uninitialized and xe->sc.mmio is NULL.
enable_boot_survivability_mode() sets survivability->mode = true and calls
xe_vsec_init(xe), which registers the crashlog auxiliary device via
intel_vsec_register() with devm-managed cleanup:

drivers/gpu/drm/xe/xe_survivability_mode.c:enable_boot_survivability_mode() {
	...
	survivability->mode = true;

	xe_heci_gsc_init(xe);

	xe_soc_remapper_init(xe);
	xe_vsec_init(xe);

	if (survivability->fdo_mode) {
		ret = xe_nvm_init(xe);
		if (ret)
			goto failed;
	}

	ret = xe_i2c_probe(xe);
	if (ret)
		goto failed;
	...
failed:
	xe_log_err_fatal(pdev, SURVIVABILITY, ret, "Failed to enable Boot Mode!\n");
	survivability->mode = false;
	return ret;
}

If xe_nvm_init() or xe_i2c_probe() fails, survivability->mode is reset to
false while the VSEC crashlog device and its sysfs attributes remain
registered until PCI probe teardown runs devres_release_all().

If userspace reads the crashlog sysfs attributes during that teardown window,
xe_pmt_telem_read() or xe_pmt_read_reg() calls wait_for_fw(xe), which sees
survivability->mode == false and calls:

  wait_for_fw(xe)
    -> xe_sysctrl_is_oobmsm_fw_ready(xe)
      -> xe_sysctrl_send_command(xe, ...)
        -> guard(mutex)(&sc->cmd_lock)
        -> sysctrl_send_command(sc, ...)

Because xe_sysctrl_init() was never called, would this lock the uninitialized
sc->cmd_lock and dereference the NULL sc->mmio pointer?

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261102221314.119028-18-michael.j.ruhl@intel.com?part=16

Re: [PATCH v13 13/16] drm/xe/vsec: Add PMT GUID internal access

[sashiko-bot]

Thank you for your contribution! Sashiko AI review found 2 potential issue(s) to consider:
- [Medium] drm/xe/vsec: missing wait_for_fw() and zero-GUID check in xe_vsec_get_guid()
- [Low] drm/xe/vsec: missing kernel-doc for xe_vsec_get_guid()
--

commit f5d795abace88c0eb050818c49e569b4c3a02f52
Author: Michael J. Ruhl <michael.j.ruhl@intel.com>

drm/xe/vsec: Add PMT GUID internal access

This commit renames PUNIT_TELEMETRY_GUID to BMG_PUNIT_TELEMETRY_GUID, adds
CRI_PUNIT_TELEMETRY_GUID for Crescent Island, and introduces
xe_vsec_get_guid() to read and cache the PUNIT telemetry GUID based on the
platform.

> diff --git a/drivers/gpu/drm/xe/xe_device_types.h b/drivers/gpu/drm/xe/xe_device_types.h
> index cc8e0e6bed9a8..7430185ad41a9 100644
> --- a/drivers/gpu/drm/xe/xe_device_types.h
> +++ b/drivers/gpu/drm/xe/xe_device_types.h
> @@ -490,6 +490,8 @@ struct xe_device {
>  		struct delayed_work work;
>  		/** @pmt.retry_count: late-bind probe retry */
>  		u32 retry_count;
> +		/** @pmt.punit_guid_cache: cache of the PUINT GUID */
> +		u32 punit_guid_cache;

[Severity: Low]
This isn't a bug, but there is a minor typo in the kernel-doc comment here
(PUINT instead of PUNIT).

[ ... ]

> diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
> index 5384fef5e5ac5..1ed904ec14965 100644
> --- a/drivers/gpu/drm/xe/xe_vsec.c
> +++ b/drivers/gpu/drm/xe/xe_vsec.c
> @@ -579,6 +579,51 @@ static void vsec_disable_late_bind_work(void *arg)
>  		xe_pm_runtime_put(xe);
>  }
>  
> +int xe_vsec_get_guid(struct xe_device *xe, u32 *guid)
> +{

[Severity: Low]
This isn't a bug, but since Documentation/gpu/xe/xe_vsec.rst pulls
kernel-doc from drivers/gpu/drm/xe/xe_vsec.c and the other exported VSEC
entry points (xe_pmt_telem_read() and xe_vsec_init()) are documented,
should xe_vsec_get_guid() also have a kernel-doc comment describing its
parameters, runtime PM and xe->pmt.lock locking expectations, and return
values?

> +	struct xe_mmio *mmio = xe_root_tile_mmio(xe);
> +	int ret = 0;
> +
> +	xe_assert(xe, !IS_SRIOV_VF(xe));
> +
> +	if (!xe_pm_runtime_get_if_active(xe))
> +		return -ENODATA;
> +
> +	mutex_lock(&xe->pmt.lock);
> +
> +	if (xe->pmt.punit_guid_cache) {
> +		*guid = xe->pmt.punit_guid_cache;
> +		goto unlock;
> +	}
> +
> +	switch (xe->info.platform) {
> +	case XE_BATTLEMAGE:
> +		*guid = xe_mmio_read32(mmio, BMG_PUNIT_TELEMETRY_GUID);
> +		break;
> +
> +	case XE_CRESCENTISLAND:
> +		xe->soc_remapper.set_telem_region(xe, CRI_IDX_TELEM_DISCOVERY);
> +		*guid = xe_mmio_read32(mmio, CRI_PUNIT_TELEMETRY_GUID);
> +		break;
> +
> +	default:
> +		*guid = 0;
> +		drm_err(&xe->drm, "Unsupported platform: %u\n", xe->info.platform);
> +		ret = -EINVAL;
> +		break;
> +	}
> +
> +	/* the GUID is static, read once and cache for all requests */
> +	if (*guid)
> +		xe->pmt.punit_guid_cache = *guid;
> +
> +unlock:
> +	mutex_unlock(&xe->pmt.lock);
> +	xe_pm_runtime_put(xe);
> +
> +	return ret;
> +}

[Severity: Medium]
On XE_CRESCENTISLAND, can xe_vsec_get_guid() be called before the late-bound
OOBMSM firmware has finished initializing (for example, via debugfs or hwmon
after xe_vsec_init() defers discovery to cri_late_bind_probe_work(), or right
after runtime/system resume)?

The other PMT access helpers in drivers/gpu/drm/xe/xe_vsec.c (such as
xe_pmt_telem_read() and xe_pmt_read_reg()) check wait_for_fw(xe) before
taking xe->pmt.lock, programming CRI_IDX_TELEM_DISCOVERY, and reading the
discovery window:

drivers/gpu/drm/xe/xe_vsec.c:xe_pmt_read_reg() {
	...
	xe_pm_runtime_get(xe);
	if (!wait_for_fw(xe)) {
		ret = -ENODATA;
		goto runtime_exit;
	}

	scoped_guard(mutex, &xe->pmt.lock) {
		xe->soc_remapper.set_telem_region(xe, CRI_IDX_TELEM_DISCOVERY);

		*reg = readl(disc_addr);
	}
	...
}

Without wait_for_fw(xe) in xe_vsec_get_guid(), if reading
CRI_PUNIT_TELEMETRY_GUID before firmware is ready returns a non-zero bogus
value (such as 0xffffffff), won't xe->pmt.punit_guid_cache permanently cache
that invalid value for the lifetime of the device?

Conversely, if xe_mmio_read32() returns 0 (either before firmware is ready
or when !mmio_available(mmio)), xe_vsec_get_guid() skips caching *guid, but
ret remains 0. Should xe_vsec_get_guid() wait for firmware readiness via
wait_for_fw(xe) and return an error code when *guid is 0 (or invalid) rather
than returning 0 with *guid == 0?

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261102221314.119028-18-michael.j.ruhl@intel.com?part=13

Re: [PATCH v13 12/16] drm/xe/vsec: Support late bind fw information

[sashiko-bot]

Thank you for your contribution! Sashiko AI review found 6 potential issue(s) to consider:
- [High] drm/xe/vsec: devres race and leak in cri_late_bind_probe_work() on remove
- [High] drm/xe/vsec: cri_late_bind_probe_work() accesses MMIO during system suspend
- [Medium] drm/xe/vsec: lock inversion holding pmt.lock in cri_late_bind_probe()
- [Medium] drm/xe/vsec: 20s uninterruptible msleep() in wait_for_fw() under SRCU
- [Medium] drm/xe/vsec: inconsistent locking and CRI_IDX_TELEM_DISCOVERY race on probe
- [Low] drm/xe/vsec: missing kernel-doc Return description for xe_vsec_init()
--

commit de390f347a84cad7bb93f5a65e0c5583258f4dc3
Author: Michael J. Ruhl <michael.j.ruhl@intel.com>

drm/xe/vsec: Support late bind fw information

This commit updates the Xe PMT probe and access paths to check for
Crescent Island late-bind firmware readiness, scheduling delayed work
when firmware is not yet initialized at probe time and waiting for
firmware readiness before PMT telemetry or register access.

> diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
> index 0f6e3be520c09..5384fef5e5ac5 100644
> --- a/drivers/gpu/drm/xe/xe_vsec.c
> +++ b/drivers/gpu/drm/xe/xe_vsec.c

[ ... ]

> @@ -281,6 +291,48 @@ static int xe_guid_decode(u32 guid, int *index, u32 *offset)
>  	return -ENODEV;
>  }
>  
> +static void cri_late_bind_probe_work(struct work_struct *work)
> +{
> +	struct xe_device *xe = container_of(work, struct xe_device, pmt.work.work);
> +
> +	if (xe_sysctrl_is_oobmsm_fw_ready(xe)) {
> +		cri_late_bind_probe(xe);
> +		xe_pm_runtime_put(xe);
> +		return;
> +	}
> +
> +	xe->pmt.retry_count++;
> +
> +	/* wait up to 20 seconds */
> +	if (xe->pmt.retry_count == VSEC_LATE_BIND_RETRY) {
> +		drm_warn(&xe->drm, "PMT probe: Late Binding failed to complete\n");
> +		xe_pm_runtime_put(xe);
> +		return;
> +	}
> +
> +	if (!schedule_delayed_work(&xe->pmt.work, msecs_to_jiffies(VSEC_LATE_BIND_DELAY_MSEC)))
> +		xe_pm_runtime_put(xe);
> +}

[Severity: High]
As reported by Sashiko on v5 [1], can cri_late_bind_probe_work() wake up
during system suspend (such as S2idle, S3, or hibernation) and perform
MMIO accesses on a powered-down PCI device?

In xe_vsec_init(), when xe_sysctrl_is_oobmsm_fw_ready() is false, the
driver takes a runtime PM reference via xe_pm_runtime_get_noresume() and
queues xe->pmt.work onto system_percpu_wq via schedule_delayed_work(),
re-arming every 100ms for up to 20 seconds.

While the runtime PM reference prevents runtime suspend, it does not
prevent system suspend (xe_pci_suspend() -> xe_pm_suspend()) because
DPM_FLAG_NO_DIRECT_COMPLETE is set on discrete graphics devices. Unlike
xe->late_bind, which xe_pm_suspend() flushes via
xe_late_bind_wait_for_worker_completion(), neither xe_pm_suspend() nor
xe_pci_suspend() cancels or flushes xe->pmt.work before disabling the
PCI device and placing it into PCI_D3cold.

Since system_percpu_wq is not freezable, won't cri_late_bind_probe_work()
continue running while the device is in D3cold and touch MMIO in
xe_sysctrl_is_oobmsm_fw_ready(), xe->soc_remapper.set_telem_region(), and
intel_vsec_register()?

> +static bool wait_for_fw(struct xe_device *xe)
> +{
> +	int retries = VSEC_LATE_BIND_RETRY;  /* wait up to 20 secs */
> +
> +	if (xe->info.platform != XE_CRESCENTISLAND)
> +		return true;
> +
> +	while (retries--) {
> +		if (xe_sysctrl_is_oobmsm_fw_ready(xe))
> +			return true;
> +
> +		msleep(VSEC_LATE_BIND_DELAY_MSEC);
> +	}
> +
> +	drm_warn(&xe->drm, "Late Binding failed to complete\n");
> +
> +	return false;
> +}
> +
>  /**
>   * xe_pmt_telem_read - Given a device and a PMT GUID, read data into a buffer
>   * @dev: valid Xe device
> @@ -343,6 +395,11 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
>  		goto dev_exit;
>  	}
>  
> +	if (!wait_for_fw(xe)) {
> +		ret = -ENODATA;
> +		goto runtime_exit;
> +	}

[Severity: Medium]
As reported by Sashiko on v2 [2], does calling wait_for_fw() inside
xe_pmt_telem_read(), xe_pmt_read_reg(), and xe_pmt_write_reg() stall
device unplug and leave userspace tasks in uninterruptible sleep for up
to 20 seconds?

Each of those functions enters the DRM SRCU read-side critical section
via drm_dev_enter(&xe->drm, &idx) before calling wait_for_fw(). When
xe_sysctrl_is_oobmsm_fw_ready() returns false on Crescent Island (for
instance while firmware is reloading after D3cold exit, or if late
binding failed while xe->pmt.base_offset remains set), wait_for_fw()
loops up to 200 times calling msleep(100) without checking for pending
signals, task freezing, or device unplug.

Holding the drm_dev_enter() SRCU lock across 20 seconds of msleep() will
block drm_dev_unplug() in synchronize_srcu() for up to 20 seconds during
hot-unplug or driver unbind. In addition, sleeping in TASK_UNINTERRUPTIBLE
for 20 seconds makes userspace readers of PMT sysfs or debugfs files (such
as pcode_info) unkillable and can cause try_to_freeze_tasks() to time out
and abort system suspend.

>  	scoped_guard(mutex, &xe->pmt.lock) {
>  		/* set SoC re-mapper index register based on GUID memory region */
>  		xe->soc_remapper.set_telem_region(xe, mem_region);
> @@ -350,6 +407,7 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
>  		memcpy_fromio(data, telem_addr, count);
>  	}
>  
> +runtime_exit:
>  	xe_pm_runtime_put(xe);
>  
>  dev_exit:
> @@ -397,6 +455,10 @@ static int xe_pmt_read_reg(struct device *dev, u32 guid, u32 *reg, u32 offset)
>  	disc_addr += inst + offset;
>  
>  	xe_pm_runtime_get(xe);
> +	if (!wait_for_fw(xe)) {
> +		ret = -ENODATA;
> +		goto runtime_exit;
> +	}

[ ... ]

> @@ -477,12 +545,46 @@ static enum xe_vsec get_platform_info(struct xe_device *xe)
>  	return vsec_platforms[xe->info.platform];
>  }
>  
> +static void cri_late_bind_probe(struct xe_device *xe)
> +{
> +	struct intel_vsec_platform_info *info;
> +	struct device *dev = xe->drm.dev;
> +	enum xe_vsec platform;
> +
> +	platform = get_platform_info(xe);
> +	if (platform != XE_VSEC_CRI)
> +		return;
> +
> +	info = &xe_vsec_info[platform];
> +	if (!info->headers)
> +		return;
> +
> +	guard(mutex)(&xe->pmt.lock);
> +
> +	info->priv_data = &xe_cri_pmt_cb;
> +	xe->soc_remapper.set_telem_region(xe, CRI_IDX_TELEM_DISCOVERY);
> +
> +	intel_vsec_register(dev, info);
> +}

[Severity: Medium]
As reported by Sashiko on v11 [3], can holding xe->pmt.lock across
intel_vsec_register() in cri_late_bind_probe() cause a lock inversion
with the driver core and kernfs locks?

In cri_late_bind_probe(), xe->pmt.lock is held while calling
intel_vsec_register() -> intel_vsec_add_aux() -> auxiliary_device_add().
That path acquires device_lock(&auxdev->dev) and synchronously runs
pmt_telem_probe() (acquiring ep_lock) and pmt_crashlog_probe(), which
register sysfs attributes for each discovery entry:

  xe->pmt.lock -> &auxdev->dev.mutex -> ep_lock

Conversely, when userspace accesses PMT sysfs attributes via
intel_pmt_read(), clear_show(), or trigger_store(), kernfs holds an
active reference (kn->dep_map) and calls xe_pmt_telem_read(),
xe_pmt_read_reg(), or xe_pmt_write_reg(), which acquires xe->pmt.lock:

  kn->dep_map -> xe->pmt.lock

If a multi-entry probe fails mid-way on the abort_probe or
fail_add_endpoint error path while cri_late_bind_probe() holds
xe->pmt.lock (or when pmt_telem_remove() / pmt_crashlog_remove() later
runs under &auxdev->dev.mutex and ep_lock), intel_pmt_dev_destroy()
removes the sysfs attributes and waits in kernfs_drain() on kn->dep_map,
inverting the lock order against any concurrent sysfs reader waiting on
xe->pmt.lock.

[Severity: Medium]
Also, why does cri_late_bind_probe() duplicate the CRI setup from
xe_vsec_init() with different locking, and can xe_pmt_telem_read()
corrupt CRI discovery reads by changing the SoC remapper index?

On Crescent Island, discovery headers and telemetry data share the same
MMIO window at CRI_PMT_OFFSET indexed by
xe->soc_remapper.set_telem_region(). In xe_vsec_init(), CRI_IDX_TELEM_DISCOVERY
is set and intel_vsec_register() is called without holding xe->pmt.lock.
Once pmt_telem_probe() registers the first telemetry endpoint, a
concurrent xe_pmt_telem_read() (from udev or debugfs) can acquire
xe->pmt.lock and change the remapper index to mem_region while subsequent
telemetry entries and cri_crashlog discovery headers are still being read
from CRI_PMT_OFFSET.

Furthermore, because xe_pmt_telem_read() leaves the remapper index set to
mem_region instead of restoring CRI_IDX_TELEM_DISCOVERY, if the
intel_pmt_telemetry or intel_pmt_crashlog auxiliary drivers bind after any
xe_pmt_telem_read() call (for example when loaded as modules), won't their
discovery reads at CRI_PMT_OFFSET read telemetry data instead of discovery
headers?

> +
> +static void vsec_disable_late_bind_work(void *arg)
> +{
> +	struct xe_device *xe = arg;
> +
> +	/*
> +	 * If the queued work is canceled, the runtime reference needs to be
> +	 * released here.
> +	 */
> +	if (disable_delayed_work_sync(&xe->pmt.work))
> +		xe_pm_runtime_put(xe);
> +}
> +
>  /**
>   * xe_vsec_init - Initialize resources and add intel_vsec auxiliary
>   * interface
>   * @xe: valid xe instance
>   */
> -void xe_vsec_init(struct xe_device *xe)
> +int xe_vsec_init(struct xe_device *xe)

[Severity: Low]
Since xe_vsec_init() now returns an int and drivers/gpu/drm/xe/xe_vsec.c
is included in Documentation/gpu/xe/xe_vsec.rst, should a Return: section
be added to the xe_vsec_init() kernel-doc comment (and -ENODATA added to
the Return: list of xe_pmt_read_reg()) to avoid kernel-doc warnings?

>  {
>  	struct intel_vsec_platform_info *info;
>  	struct device *dev = xe->drm.dev;
> @@ -490,11 +592,11 @@ void xe_vsec_init(struct xe_device *xe)
>  
>  	platform = get_platform_info(xe);
>  	if (platform == XE_VSEC_UNKNOWN)
> -		return;
> +		return 0;
>  
>  	info = &xe_vsec_info[platform];
>  	if (!info->headers)
> -		return;
> +		return 0;
>  
>  	switch (platform) {
>  	case XE_VSEC_BMG:
> @@ -504,12 +606,26 @@ void xe_vsec_init(struct xe_device *xe)
>  
>  	case XE_VSEC_CRI:
>  		xe->pmt.base_offset = CRI_PMT_OFFSET;
> +
> +		xe->pmt.retry_count = 0;
> +		INIT_DELAYED_WORK(&xe->pmt.work, cri_late_bind_probe_work);
> +
> +		xe_pm_runtime_get_noresume(xe);
> +		if (!xe_sysctrl_is_oobmsm_fw_ready(xe)) {
> +			schedule_delayed_work(&xe->pmt.work,
> +					      msecs_to_jiffies(VSEC_LATE_BIND_DELAY_MSEC));
> +			return devm_add_action_or_reset(xe->drm.dev,
> +							vsec_disable_late_bind_work,
> +							xe);
> +		}

[Severity: High]
As reported by Sashiko on v2 [2], can cancelling xe->pmt.work via a devm
action race with devres_release_all() and leak the PMT auxiliary devices
and their MMIO resources on driver unbind or probe failure?

When xe_pci_remove() runs (or if xe_device_probe() fails after
xe_vsec_init(), or during xe_pci_error_slot_reset()), xe_device_remove()
and xe_pm_fini() run before devres_release_all(dev) without stopping
xe->pmt.work.

Inside devres_release_all(dev), remove_nodes() splices all currently
registered devres entries (including vsec_disable_late_bind_work) from
dev->devres_head onto a local todo list before release_nodes() executes
any callbacks:

  devres_release_all(dev)
    remove_nodes(dev, ..., &todo)  /* moves dev->devres_head to todo */
    release_nodes(dev, &todo)
      ...
      vsec_disable_late_bind_work(xe)
        disable_delayed_work_sync(&xe->pmt.work)

If cri_late_bind_probe_work() is already running on another CPU when
remove_nodes() drains dev->devres_head (while release_nodes() is running
earlier actions or waiting in disable_delayed_work_sync()) and sees
xe_sysctrl_is_oobmsm_fw_ready(xe) == true, it calls:

  cri_late_bind_probe(xe)
    intel_vsec_register(dev, info)
      intel_vsec_add_aux(dev, ...)
        devm_add_action_or_reset(parent, intel_vsec_remove_aux, auxdev)

Because dev->devres_head was already moved to todo, the newly appended
intel_vsec_remove_aux devres actions are left on dev->devres_head and
never executed. Doesn't this leave the intel_vsec.telemetry and
intel_vsec.crashlog auxiliary devices and their resource reservations
registered on the unbound device (causing subsequent driver re-probes to
fail with -EBUSY and exposing dangling sysfs callbacks after xe_device is
freed), as well as calling xe_pm_runtime_put() after xe_pm_fini()?

[1] https://lore.kernel.org/r/20260903202808.A8A5A1F000E9@smtp.kernel.org
[2] https://lore.kernel.org/r/20260812195421.15EE31F000E9@smtp.kernel.org
[3] https://lore.kernel.org/r/20260923181115.2514193-39-michael.j.ruhl@intel.com

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261102221314.119028-18-michael.j.ruhl@intel.com?part=12

Re: [PATCH v13 15/16] drm/xe/vsec: Refactor platform check

[sashiko-bot]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] drm/xe/vsec: missing wait_for_fw() and GUID validation in xe_vsec_get_guid()
--

commit fd76168b193bc56ba83b5c95e6f8be66f5597b26
Author: Michael J. Ruhl <michael.j.ruhl@intel.com>

drm/xe/vsec: Refactor platform check

This commit combines the platform checks in xe_debugfs_register() so that
both debugfs_residencies and pcode_info_debugfs are registered on Battlemage
and Crescent Island PF devices.

> diff --git a/drivers/gpu/drm/xe/xe_debugfs.c b/drivers/gpu/drm/xe/xe_debugfs.c
> index b56e4aad40696..5a82f8eab2c66 100644
> --- a/drivers/gpu/drm/xe/xe_debugfs.c
> +++ b/drivers/gpu/drm/xe/xe_debugfs.c
> @@ -702,23 +702,22 @@ void xe_debugfs_register(struct xe_device *xe)
>  				 ARRAY_SIZE(debugfs_list),
>  				 root, minor);
>  
> -	if (xe->info.platform == XE_BATTLEMAGE && !IS_SRIOV_VF(xe)) {
> -		drm_debugfs_create_files(debugfs_residencies,
> -					 ARRAY_SIZE(debugfs_residencies),
> -					 root, minor);
> -	}
> -
>  	/*
> -	 * Pcode version read from PMT is currently only supported on CRI and BMG platforms in PF
> -	 * mode, as both platforms support the necessary telemetry read mechanism and have a fixed
> -	 * PUNIT_VERSION_OFFSET.
> +	 * Residencies and Pcode version read from PMT is currently only supported on CRI and BMG
> +	 * platforms in PF mode.  Both platforms support the necessary telemetry read mechanism
> +	 * and have a fixed offsets for the required data.
>  	 * Attempting this access on other platforms must be verified before enabling support.
>  	 */
>  	if (!IS_SRIOV_VF(xe) &&
> -	    (xe->info.platform == XE_CRESCENTISLAND || xe->info.platform == XE_BATTLEMAGE))
> +	    (xe->info.platform == XE_CRESCENTISLAND || xe->info.platform == XE_BATTLEMAGE)) {
> +		drm_debugfs_create_files(debugfs_residencies,
> +					 ARRAY_SIZE(debugfs_residencies),
> +					 root, minor);
> +
>  		drm_debugfs_create_files(pcode_info_debugfs,
>  					 ARRAY_SIZE(pcode_info_debugfs),
>  					 root, minor);
> +	}

[Severity: Medium]
With debugfs_residencies now exposed on XE_CRESCENTISLAND alongside
pcode_info_debugfs, what happens if userspace reads dgfx_pkg_residencies,
dgfx_pcie_link_residencies, or pcode_info while OOBMSM firmware
initialization (cri_late_bind_probe_work) is still pending after
xe_vsec_init()?

In read_residency_counter() (and xe_get_pcode_version()), xe_vsec_get_guid()
is called before xe_pmt_telem_read():

drivers/gpu/drm/xe/xe_debugfs.c:read_residency_counter() {
	ret = xe_vsec_get_guid(xe, &guid);
	if (ret) {
		drm_warn(&xe->drm, "PMT device is not available: %d\n", ret);
		return;
	}

	ret = xe_pmt_telem_read(xe->drm.dev, guid, &residency, offset, sizeof(residency));
}

Unlike xe_pmt_telem_read(), xe_pmt_read_reg(), and xe_pmt_write_reg(), which
all call wait_for_fw(xe) before setting the SoC remapper index and reading
the PMT window, xe_vsec_get_guid() accesses CRI_PUNIT_TELEMETRY_GUID without
waiting for firmware readiness or validating the returned GUID:

drivers/gpu/drm/xe/xe_vsec.c:xe_vsec_get_guid() {
	...
	case XE_CRESCENTISLAND:
		xe->soc_remapper.set_telem_region(xe, CRI_IDX_TELEM_DISCOVERY);
		*guid = xe_mmio_read32(mmio, CRI_PUNIT_TELEMETRY_GUID);
		break;
	...
	/* the GUID is static, read once and cache for all requests */
	if (*guid)
		xe->pmt.punit_guid_cache = *guid;
}

If CRI_PUNIT_TELEMETRY_GUID reads as 0 before firmware is ready,
xe_vsec_get_guid() returns 0 with *guid == 0, and xe_pmt_telem_read() then
fails xe_guid_decode() with -ENODEV before reaching wait_for_fw(xe). If the
unready register returns a non-zero bogus value (such as 0xffffffff),
xe_vsec_get_guid() caches it in xe->pmt.punit_guid_cache, permanently
breaking subsequent telemetry reads for the lifetime of the device.

Should xe_vsec_get_guid() call wait_for_fw(xe) and validate the GUID before
caching it in xe->pmt.punit_guid_cache?

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261102221314.119028-18-michael.j.ruhl@intel.com?part=15

✗ CI.checkpatch: warning for Crescent Island PMT support (rev15)

[Patchwork]

== Series Details ==

Series: Crescent Island PMT support (rev15)
URL   : https://patchwork.freedesktop.org/series/160717/
State : warning

== Summary ==

+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
3009867cd31863246e4afc138520a7a8bd37bb11
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 7118464c02e38583572f0f5551d4e234b4120252
Author: Michael J. Ruhl <michael.j.ruhl@intel.com>
Date:   Mon Nov 2 14:13:28 2026 -0800

    drm/xe/vsec: Support survivability mode for CRI
    
    For survivability mode, CRI can only support access to the
    PUNIT CRASHLOG PMT function.
    
    Create a special config to support survivability mode.
    
    Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com>
+ /mt/dim checkpatch 250a6b22eabe84f32114c55e0ad4c271bfe573b3 drm-intel
5397d7481488 drm/xe/vsec: Do not register BMG PMT for VF
1f73e8c70cbf drm/xe/vsec: Correct locking order
3297c5789cf1 drm/xe/vsec: Use correct pm state get
260ee0dcc2c8 drm/xe/vsec: Add DOC text for VSEC
-:21: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#21: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 54 lines checked
ac4840e09f62 drm/xe/vsec: Support possible hotplug exit
f1b9746117f1 drm/xe/vsec: Refactor BattleMage PMT defines
2ef31832d85d drm/xe/vsec: Update VSEC probe order
cbe9eab54c34 drm/xe/vsec: Add base_offset to allow for more flexibilty
29054e6934af drm/xe/vsec: Support Crescent Island PMT
4457c042482c drm/xe/vsec: Crescent Island PMT decode
b8ed8f40df19 drm/xe/vsec: Crescent Island PMT callbacks
712e68e86fd8 drm/xe/vsec: Support late bind fw information
ad4df6a96ac1 drm/xe/vsec: Add PMT GUID internal access
3b1ea93cc9ee drm/xe/vsec: Update PMT internal access
f7d8b0feb4c3 drm/xe/vsec: Refactor platform check
7118464c02e3 drm/xe/vsec: Support survivability mode for CRI

✓ CI.KUnit: success for Crescent Island PMT support (rev15)

[Patchwork]

== Series Details ==

Series: Crescent Island PMT support (rev15)
URL   : https://patchwork.freedesktop.org/series/160717/
State : success

== Summary ==

+ trap cleanup EXIT
+ kunitconfigs=('/kernel/drivers/gpu/tests/.kunitconfig' '/kernel/drivers/gpu/drm/xe/.kunitconfig' '/kernel/drivers/gpu/drm/tests/.kunitconfig' '/kernel/drivers/gpu/drm/ttm/tests/.kunitconfig' '/kernel/drivers/dma-buf/.kunitconfig')
+ for kcfg in "${kunitconfigs[@]}"
+ [[ ! -f /kernel/drivers/gpu/tests/.kunitconfig ]]
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/tests/.kunitconfig
[22:49:02] Configuring KUnit Kernel ...
Generating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[22:49:07] Building KUnit Kernel ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
Building with:
$ make all compile_commands.json scripts_gdb ARCH=um O=.kunit --jobs=48
[22:49:27] Starting KUnit Kernel (1/1)...
[22:49:27] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[22:49:27] ================= gpu_buddy (14 subtests) ==================
[22:49:27] [PASSED] gpu_test_buddy_alloc_limit
[22:49:27] [PASSED] gpu_test_buddy_alloc_optimistic
[22:49:27] [PASSED] gpu_test_buddy_alloc_pessimistic
[22:49:27] [PASSED] gpu_test_buddy_alloc_pathological
[22:49:27] [PASSED] gpu_test_buddy_alloc_contiguous
[22:49:27] [PASSED] gpu_test_buddy_alloc_clear
[22:49:27] [PASSED] gpu_test_buddy_alloc_range
[22:49:28] [PASSED] gpu_test_buddy_alloc_range_bias
[22:49:29] [PASSED] gpu_test_buddy_fragmentation_performance
[22:49:29] [PASSED] gpu_test_buddy_dirty_tracker_performance
[22:49:29] [PASSED] gpu_test_buddy_alloc_exceeds_max_order
[22:49:29] [PASSED] gpu_test_buddy_offset_aligned_allocation
[22:49:29] [PASSED] gpu_test_buddy_subtree_offset_alignment_stress
[22:49:29] [PASSED] gpu_test_buddy_addr_to_block
[22:49:29] ==================== [PASSED] gpu_buddy ====================
[22:49:29] ============================================================
[22:49:29] Testing complete. Ran 14 tests: passed: 14
[22:49:29] Elapsed time: 27.579s total, 5.083s configuring, 20.629s building, 1.816s running

+ for kcfg in "${kunitconfigs[@]}"
+ [[ ! -f /kernel/drivers/gpu/drm/xe/.kunitconfig ]]
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/xe/.kunitconfig
[22:49:29] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[22:49:32] 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
[22:50:11] Starting KUnit Kernel (1/1)...
[22:50:11] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[22:50:11] ================== guc_buf (11 subtests) ===================
[22:50:11] [PASSED] test_smallest
[22:50:11] [PASSED] test_largest
[22:50:11] [PASSED] test_granular
[22:50:11] [PASSED] test_unique
[22:50:11] [PASSED] test_overlap
[22:50:11] [PASSED] test_reusable
[22:50:11] [PASSED] test_too_big
[22:50:11] [PASSED] test_flush
[22:50:11] [PASSED] test_lookup
[22:50:11] [PASSED] test_data
[22:50:11] [PASSED] test_class
[22:50:11] ===================== [PASSED] guc_buf =====================
[22:50:11] =================== guc_dbm (7 subtests) ===================
[22:50:11] [PASSED] test_empty
[22:50:11] [PASSED] test_default
[22:50:11] ======================== test_size  ========================
[22:50:11] [PASSED] 4
[22:50:11] [PASSED] 8
[22:50:11] [PASSED] 32
[22:50:11] [PASSED] 256
[22:50:11] ==================== [PASSED] test_size ====================
[22:50:11] ======================= test_reuse  ========================
[22:50:11] [PASSED] 4
[22:50:11] [PASSED] 8
[22:50:11] [PASSED] 32
[22:50:11] [PASSED] 256
[22:50:11] =================== [PASSED] test_reuse ====================
[22:50:11] =================== test_range_overlap  ====================
[22:50:11] [PASSED] 4
[22:50:11] [PASSED] 8
[22:50:11] [PASSED] 32
[22:50:11] [PASSED] 256
[22:50:11] =============== [PASSED] test_range_overlap ================
[22:50:11] =================== test_range_compact  ====================
[22:50:11] [PASSED] 4
[22:50:11] [PASSED] 8
[22:50:11] [PASSED] 32
[22:50:11] [PASSED] 256
[22:50:11] =============== [PASSED] test_range_compact ================
[22:50:11] ==================== test_range_spare  =====================
[22:50:11] [PASSED] 4
[22:50:11] [PASSED] 8
[22:50:11] [PASSED] 32
[22:50:11] [PASSED] 256
[22:50:11] ================ [PASSED] test_range_spare =================
[22:50:11] ===================== [PASSED] guc_dbm =====================
[22:50:11] ================== guc_idm (22 subtests) ===================
[22:50:11] [PASSED] bad_init
[22:50:11] [PASSED] no_init
[22:50:11] [PASSED] init_fini
[22:50:11] [PASSED] check_init_small
[22:50:11] [PASSED] check_init_shared
[22:50:11] [PASSED] check_used
[22:50:11] [PASSED] check_quota
[22:50:11] [PASSED] check_all
[22:50:11] [PASSED] check_limits
[22:50:11] [PASSED] check_overlap
[22:50:11] [PASSED] check_overlap_alloc
[22:50:11] [PASSED] check_overlap_release
[22:50:11] [PASSED] check_overlap_partial_init
[22:50:11] [PASSED] check_overlap_partial_shared_range_blocking
[22:50:11] [PASSED] check_used_usable
[22:50:11] [PASSED] check_used_shareable
[22:50:11] [PASSED] check_used_mixed
[22:50:11] [PASSED] check_release_usable_range
[22:50:11] [PASSED] check_release_shareable_range
[22:50:11] [PASSED] check_split_boundaries
[22:50:11] [PASSED] check_shareable_spare
[22:50:11] [PASSED] check_shareable_spare_fragmented_tail
[22:50:11] ===================== [PASSED] guc_idm =====================
[22:50:11] ============== guc_klv_helpers (13 subtests) ===============
[22:50:11] [PASSED] test_count
[22:50:11] [PASSED] test_encode_u32
[22:50:11] [PASSED] test_encode_u64
[22:50:11] [PASSED] test_encode_string
[22:50:11] [PASSED] test_encode_object_raw
[22:50:11] [PASSED] test_encode_object_klv
[22:50:11] [PASSED] test_encode_object_nested
[22:50:11] [PASSED] test_encode_object_basic
[22:50:11] ===================== test_decode_u16  =====================
[22:50:11] [PASSED] empty
[22:50:11] [PASSED] valid
[22:50:11] [PASSED] max
[22:50:11] [PASSED] big
[22:50:11] [PASSED] long
[22:50:11] ================= [PASSED] test_decode_u16 =================
[22:50:11] ===================== test_decode_u32  =====================
[22:50:11] [PASSED] empty
[22:50:11] [PASSED] valid
[22:50:11] [PASSED] max
[22:50:11] [PASSED] long
[22:50:11] ================= [PASSED] test_decode_u32 =================
[22:50:11] ===================== test_decode_u64  =====================
[22:50:11] [PASSED] empty
[22:50:11] [PASSED] short
[22:50:11] [PASSED] valid
[22:50:11] [PASSED] max
[22:50:11] [PASSED] long
[22:50:11] ================= [PASSED] test_decode_u64 =================
[22:50:11] ===================== test_decode_str  =====================
[22:50:11] [PASSED] empty
[22:50:11] [PASSED] one
[22:50:11] [PASSED] one_trash
[22:50:11] [PASSED] two
[22:50:11] [PASSED] two_trash
[22:50:11] [PASSED] three
[22:50:11] [PASSED] dword
[22:50:11] [PASSED] four
[22:50:11] [PASSED] five
[22:50:11] [PASSED] six
[22:50:11] [PASSED] seven
[22:50:11] [PASSED] qword
[22:50:11] ================= [PASSED] test_decode_str =================
[22:50:11] [PASSED] test_print
[22:50:11] ================= [PASSED] guc_klv_helpers =================
[22:50:11] =================== xe_log (4 subtests) ====================
[22:50:11] [PASSED] demo_cper
[22:50:11] [PASSED] demo_dmesg
[22:50:11] ======================= test_dmesg  ========================
[22:50:11] [PASSED] test_fatal
[22:50:11] [PASSED] test_fatal_tile
[22:50:11] [PASSED] test_fatal_gt
[22:50:11] [PASSED] test_fatal_comp
[22:50:11] [PASSED] test_fatal_comp_tile
[22:50:11] [PASSED] test_fatal_comp_gt
[22:50:11] [PASSED] test_fatal_all
[22:50:11] [PASSED] test_recoverable
[22:50:11] [PASSED] test_recoverable_tile
[22:50:11] [PASSED] test_recoverable_gt
[22:50:11] [PASSED] test_recoverable_comp
[22:50:11] [PASSED] test_recoverable_comp_tile
[22:50:11] [PASSED] test_recoverable_comp_gt
[22:50:11] [PASSED] test_recoverable_all
[22:50:11] [PASSED] test_info
[22:50:11] [PASSED] test_info_tile
[22:50:11] [PASSED] test_info_gt
[22:50:11] [PASSED] test_info_err
[22:50:11] [PASSED] test_info_comp
[22:50:11] [PASSED] test_info_comp_tile
[22:50:11] [PASSED] test_info_comp_gt
[22:50:11] [PASSED] test_info_all
[22:50:11] [PASSED] test_hw_fatal
[22:50:11] [PASSED] test_hw_recoverable
[22:50:11] [PASSED] test_hw_corrected
[22:50:11] [PASSED] test_hw_informational
[22:50:11] =================== [PASSED] test_dmesg ====================
[22:50:11] ====================== test_invalid  =======================
[22:50:11] [SKIPPED] no-component no-location no-warn (requires CONFIG_DRM_XE_DEBUG)
[22:50:11] [SKIPPED] reserved location (requires CONFIG_DRM_XE_DEBUG)
[22:50:11] [SKIPPED] unknown location (requires CONFIG_DRM_XE_DEBUG)
[22:50:11] [SKIPPED] nonzero-device-id location (requires CONFIG_DRM_XE_DEBUG)
[22:50:11] [SKIPPED] invalid-tile-id location (requires CONFIG_DRM_XE_DEBUG)
[22:50:11] [SKIPPED] invalid-gt-id location (requires CONFIG_DRM_XE_DEBUG)
[22:50:11] [SKIPPED] unknown component class (requires CONFIG_DRM_XE_DEBUG)
[22:50:11] [SKIPPED] unknown system component (requires CONFIG_DRM_XE_DEBUG)
[22:50:11] [SKIPPED] unknown hardware component (requires CONFIG_DRM_XE_DEBUG)
[22:50:11] [SKIPPED] unknown component and location (requires CONFIG_DRM_XE_DEBUG)
[22:50:11] ================== [SKIPPED] test_invalid ==================
[22:50:11] ===================== [PASSED] xe_log ======================
[22:50:11] ================== no_relay (3 subtests) ===================
[22:50:11] [PASSED] xe_drops_guc2pf_if_not_ready
[22:50:11] [PASSED] xe_drops_guc2vf_if_not_ready
[22:50:11] [PASSED] xe_rejects_send_if_not_ready
[22:50:11] ==================== [PASSED] no_relay =====================
[22:50:11] ================== pf_relay (14 subtests) ==================
[22:50:11] [PASSED] pf_rejects_guc2pf_too_short
[22:50:11] [PASSED] pf_rejects_guc2pf_too_long
[22:50:11] [PASSED] pf_rejects_guc2pf_no_payload
[22:50:11] [PASSED] pf_fails_no_payload
[22:50:11] [PASSED] pf_fails_bad_origin
[22:50:11] [PASSED] pf_fails_bad_type
[22:50:11] [PASSED] pf_txn_reports_error
[22:50:11] [PASSED] pf_txn_sends_pf2guc
[22:50:11] [PASSED] pf_sends_pf2guc
[22:50:11] [SKIPPED] pf_loopback_nop (requires CONFIG_DRM_XE_DEBUG_SRIOV)
[22:50:11] [SKIPPED] pf_loopback_echo (requires CONFIG_DRM_XE_DEBUG_SRIOV)
[22:50:11] [SKIPPED] pf_loopback_fail (requires CONFIG_DRM_XE_DEBUG_SRIOV)
[22:50:11] [SKIPPED] pf_loopback_busy (requires CONFIG_DRM_XE_DEBUG_SRIOV)
[22:50:11] [SKIPPED] pf_loopback_retry (requires CONFIG_DRM_XE_DEBUG_SRIOV)
[22:50:11] ==================== [PASSED] pf_relay =====================
[22:50:11] ================== vf_relay (3 subtests) ===================
[22:50:11] [PASSED] vf_rejects_guc2vf_too_short
[22:50:11] [PASSED] vf_rejects_guc2vf_too_long
[22:50:11] [PASSED] vf_rejects_guc2vf_no_payload
[22:50:11] ==================== [PASSED] vf_relay =====================
[22:50:11] ================ pf_gt_config (9 subtests) =================
[22:50:11] [PASSED] fair_contexts_1vf
[22:50:11] [PASSED] fair_doorbells_1vf
[22:50:11] [PASSED] fair_ggtt_1vf
[22:50:11] ====================== fair_vram_1vf  ======================
[22:50:11] [PASSED] 3.50 GiB
[22:50:11] [PASSED] 11.5 GiB
[22:50:11] [PASSED] 15.5 GiB
[22:50:11] [PASSED] 31.5 GiB
[22:50:11] [PASSED] 63.5 GiB
[22:50:11] [PASSED] 1.91 GiB
[22:50:11] ================== [PASSED] fair_vram_1vf ==================
[22:50:11] ================ fair_vram_1vf_admin_only  =================
[22:50:11] [PASSED] 3.50 GiB
[22:50:11] [PASSED] 11.5 GiB
[22:50:11] [PASSED] 15.5 GiB
[22:50:11] [PASSED] 31.5 GiB
[22:50:11] [PASSED] 63.5 GiB
[22:50:11] [PASSED] 1.91 GiB
[22:50:11] ============ [PASSED] fair_vram_1vf_admin_only =============
[22:50:11] ====================== fair_contexts  ======================
[22:50:11] [PASSED] 1 VF
[22:50:11] [PASSED] 2 VFs
[22:50:11] [PASSED] 3 VFs
[22:50:11] [PASSED] 4 VFs
[22:50:11] [PASSED] 5 VFs
[22:50:11] [PASSED] 6 VFs
[22:50:11] [PASSED] 7 VFs
[22:50:11] [PASSED] 8 VFs
[22:50:11] [PASSED] 9 VFs
[22:50:11] [PASSED] 10 VFs
[22:50:11] [PASSED] 11 VFs
[22:50:11] [PASSED] 12 VFs
[22:50:11] [PASSED] 13 VFs
[22:50:11] [PASSED] 14 VFs
[22:50:11] [PASSED] 15 VFs
[22:50:11] [PASSED] 16 VFs
[22:50:11] [PASSED] 17 VFs
[22:50:11] [PASSED] 18 VFs
[22:50:11] [PASSED] 19 VFs
[22:50:11] [PASSED] 20 VFs
[22:50:11] [PASSED] 21 VFs
[22:50:11] [PASSED] 22 VFs
[22:50:11] [PASSED] 23 VFs
[22:50:11] [PASSED] 24 VFs
[22:50:11] [PASSED] 25 VFs
[22:50:11] [PASSED] 26 VFs
[22:50:11] [PASSED] 27 VFs
[22:50:11] [PASSED] 28 VFs
[22:50:11] [PASSED] 29 VFs
[22:50:11] [PASSED] 30 VFs
[22:50:11] [PASSED] 31 VFs
[22:50:11] [PASSED] 32 VFs
[22:50:11] [PASSED] 33 VFs
[22:50:11] [PASSED] 34 VFs
[22:50:11] [PASSED] 35 VFs
[22:50:11] [PASSED] 36 VFs
[22:50:11] [PASSED] 37 VFs
[22:50:11] [PASSED] 38 VFs
[22:50:11] [PASSED] 39 VFs
[22:50:11] [PASSED] 40 VFs
[22:50:11] [PASSED] 41 VFs
[22:50:11] [PASSED] 42 VFs
[22:50:11] [PASSED] 43 VFs
[22:50:11] [PASSED] 44 VFs
[22:50:11] [PASSED] 45 VFs
[22:50:11] [PASSED] 46 VFs
[22:50:11] [PASSED] 47 VFs
[22:50:11] [PASSED] 48 VFs
[22:50:11] [PASSED] 49 VFs
[22:50:11] [PASSED] 50 VFs
[22:50:11] [PASSED] 51 VFs
[22:50:11] [PASSED] 52 VFs
[22:50:11] [PASSED] 53 VFs
[22:50:11] [PASSED] 54 VFs
[22:50:11] [PASSED] 55 VFs
[22:50:11] [PASSED] 56 VFs
[22:50:11] [PASSED] 57 VFs
[22:50:11] [PASSED] 58 VFs
[22:50:11] [PASSED] 59 VFs
[22:50:11] [PASSED] 60 VFs
[22:50:11] [PASSED] 61 VFs
[22:50:11] [PASSED] 62 VFs
[22:50:11] [PASSED] 63 VFs
[22:50:11] ================== [PASSED] fair_contexts ==================
[22:50:11] ===================== fair_doorbells  ======================
[22:50:11] [PASSED] 1 VF
[22:50:11] [PASSED] 2 VFs
[22:50:11] [PASSED] 3 VFs
[22:50:11] [PASSED] 4 VFs
[22:50:11] [PASSED] 5 VFs
[22:50:11] [PASSED] 6 VFs
[22:50:11] [PASSED] 7 VFs
[22:50:11] [PASSED] 8 VFs
[22:50:11] [PASSED] 9 VFs
[22:50:11] [PASSED] 10 VFs
[22:50:11] [PASSED] 11 VFs
[22:50:11] [PASSED] 12 VFs
[22:50:11] [PASSED] 13 VFs
[22:50:11] [PASSED] 14 VFs
[22:50:11] [PASSED] 15 VFs
[22:50:11] [PASSED] 16 VFs
[22:50:11] [PASSED] 17 VFs
[22:50:11] [PASSED] 18 VFs
[22:50:11] [PASSED] 19 VFs
[22:50:11] [PASSED] 20 VFs
[22:50:11] [PASSED] 21 VFs
[22:50:11] [PASSED] 22 VFs
[22:50:11] [PASSED] 23 VFs
[22:50:11] [PASSED] 24 VFs
[22:50:11] [PASSED] 25 VFs
[22:50:11] [PASSED] 26 VFs
[22:50:11] [PASSED] 27 VFs
[22:50:11] [PASSED] 28 VFs
[22:50:11] [PASSED] 29 VFs
[22:50:11] [PASSED] 30 VFs
[22:50:11] [PASSED] 31 VFs
[22:50:11] [PASSED] 32 VFs
[22:50:11] [PASSED] 33 VFs
[22:50:11] [PASSED] 34 VFs
[22:50:11] [PASSED] 35 VFs
[22:50:11] [PASSED] 36 VFs
[22:50:11] [PASSED] 37 VFs
[22:50:11] [PASSED] 38 VFs
[22:50:11] [PASSED] 39 VFs
[22:50:11] [PASSED] 40 VFs
[22:50:11] [PASSED] 41 VFs
[22:50:11] [PASSED] 42 VFs
[22:50:11] [PASSED] 43 VFs
[22:50:11] [PASSED] 44 VFs
[22:50:11] [PASSED] 45 VFs
[22:50:11] [PASSED] 46 VFs
[22:50:11] [PASSED] 47 VFs
[22:50:11] [PASSED] 48 VFs
[22:50:11] [PASSED] 49 VFs
[22:50:11] [PASSED] 50 VFs
[22:50:11] [PASSED] 51 VFs
[22:50:11] [PASSED] 52 VFs
[22:50:11] [PASSED] 53 VFs
[22:50:11] [PASSED] 54 VFs
[22:50:11] [PASSED] 55 VFs
[22:50:11] [PASSED] 56 VFs
[22:50:11] [PASSED] 57 VFs
[22:50:11] [PASSED] 58 VFs
[22:50:11] [PASSED] 59 VFs
[22:50:11] [PASSED] 60 VFs
[22:50:11] [PASSED] 61 VFs
[22:50:11] [PASSED] 62 VFs
[22:50:11] [PASSED] 63 VFs
[22:50:11] ================= [PASSED] fair_doorbells ==================
[22:50:11] ======================== fair_ggtt  ========================
[22:50:11] [PASSED] 1 VF
[22:50:11] [PASSED] 2 VFs
[22:50:11] [PASSED] 3 VFs
[22:50:11] [PASSED] 4 VFs
[22:50:11] [PASSED] 5 VFs
[22:50:11] [PASSED] 6 VFs
[22:50:11] [PASSED] 7 VFs
[22:50:11] [PASSED] 8 VFs
[22:50:11] [PASSED] 9 VFs
[22:50:11] [PASSED] 10 VFs
[22:50:11] [PASSED] 11 VFs
[22:50:11] [PASSED] 12 VFs
[22:50:11] [PASSED] 13 VFs
[22:50:11] [PASSED] 14 VFs
[22:50:11] [PASSED] 15 VFs
[22:50:11] [PASSED] 16 VFs
[22:50:11] [PASSED] 17 VFs
[22:50:11] [PASSED] 18 VFs
[22:50:11] [PASSED] 19 VFs
[22:50:11] [PASSED] 20 VFs
[22:50:11] [PASSED] 21 VFs
[22:50:11] [PASSED] 22 VFs
[22:50:11] [PASSED] 23 VFs
[22:50:11] [PASSED] 24 VFs
[22:50:11] [PASSED] 25 VFs
[22:50:11] [PASSED] 26 VFs
[22:50:11] [PASSED] 27 VFs
[22:50:11] [PASSED] 28 VFs
[22:50:11] [PASSED] 29 VFs
[22:50:11] [PASSED] 30 VFs
[22:50:11] [PASSED] 31 VFs
[22:50:11] [PASSED] 32 VFs
[22:50:11] [PASSED] 33 VFs
[22:50:11] [PASSED] 34 VFs
[22:50:11] [PASSED] 35 VFs
[22:50:11] [PASSED] 36 VFs
[22:50:11] [PASSED] 37 VFs
[22:50:11] [PASSED] 38 VFs
[22:50:11] [PASSED] 39 VFs
[22:50:11] [PASSED] 40 VFs
[22:50:11] [PASSED] 41 VFs
[22:50:11] [PASSED] 42 VFs
[22:50:11] [PASSED] 43 VFs
[22:50:11] [PASSED] 44 VFs
[22:50:11] [PASSED] 45 VFs
[22:50:11] [PASSED] 46 VFs
[22:50:11] [PASSED] 47 VFs
[22:50:11] [PASSED] 48 VFs
[22:50:11] [PASSED] 49 VFs
[22:50:11] [PASSED] 50 VFs
[22:50:11] [PASSED] 51 VFs
[22:50:11] [PASSED] 52 VFs
[22:50:11] [PASSED] 53 VFs
[22:50:11] [PASSED] 54 VFs
[22:50:11] [PASSED] 55 VFs
[22:50:11] [PASSED] 56 VFs
[22:50:11] [PASSED] 57 VFs
[22:50:11] [PASSED] 58 VFs
[22:50:11] [PASSED] 59 VFs
[22:50:11] [PASSED] 60 VFs
[22:50:11] [PASSED] 61 VFs
[22:50:11] [PASSED] 62 VFs
[22:50:11] [PASSED] 63 VFs
[22:50:11] ==================== [PASSED] fair_ggtt ====================
[22:50:11] ======================== fair_vram  ========================
[22:50:11] [PASSED] 1 VF
[22:50:11] [PASSED] 2 VFs
[22:50:11] [PASSED] 3 VFs
[22:50:11] [PASSED] 4 VFs
[22:50:11] [PASSED] 5 VFs
[22:50:11] [PASSED] 6 VFs
[22:50:11] [PASSED] 7 VFs
[22:50:11] [PASSED] 8 VFs
[22:50:11] [PASSED] 9 VFs
[22:50:11] [PASSED] 10 VFs
[22:50:11] [PASSED] 11 VFs
[22:50:11] [PASSED] 12 VFs
[22:50:11] [PASSED] 13 VFs
[22:50:11] [PASSED] 14 VFs
[22:50:11] [PASSED] 15 VFs
[22:50:11] [PASSED] 16 VFs
[22:50:11] [PASSED] 17 VFs
[22:50:11] [PASSED] 18 VFs
[22:50:11] [PASSED] 19 VFs
[22:50:11] [PASSED] 20 VFs
[22:50:11] [PASSED] 21 VFs
[22:50:11] [PASSED] 22 VFs
[22:50:11] [PASSED] 23 VFs
[22:50:11] [PASSED] 24 VFs
[22:50:11] [PASSED] 25 VFs
[22:50:11] [PASSED] 26 VFs
[22:50:11] [PASSED] 27 VFs
[22:50:11] [PASSED] 28 VFs
[22:50:11] [PASSED] 29 VFs
[22:50:11] [PASSED] 30 VFs
[22:50:11] [PASSED] 31 VFs
[22:50:11] [PASSED] 32 VFs
[22:50:11] [PASSED] 33 VFs
[22:50:11] [PASSED] 34 VFs
[22:50:11] [PASSED] 35 VFs
[22:50:11] [PASSED] 36 VFs
[22:50:11] [PASSED] 37 VFs
[22:50:11] [PASSED] 38 VFs
[22:50:11] [PASSED] 39 VFs
[22:50:11] [PASSED] 40 VFs
[22:50:11] [PASSED] 41 VFs
[22:50:11] [PASSED] 42 VFs
[22:50:11] [PASSED] 43 VFs
[22:50:11] [PASSED] 44 VFs
[22:50:11] [PASSED] 45 VFs
[22:50:11] [PASSED] 46 VFs
[22:50:11] [PASSED] 47 VFs
[22:50:11] [PASSED] 48 VFs
[22:50:11] [PASSED] 49 VFs
[22:50:11] [PASSED] 50 VFs
[22:50:11] [PASSED] 51 VFs
[22:50:11] [PASSED] 52 VFs
[22:50:11] [PASSED] 53 VFs
[22:50:11] [PASSED] 54 VFs
[22:50:11] [PASSED] 55 VFs
[22:50:11] [PASSED] 56 VFs
[22:50:11] [PASSED] 57 VFs
[22:50:11] [PASSED] 58 VFs
[22:50:11] [PASSED] 59 VFs
[22:50:11] [PASSED] 60 VFs
[22:50:11] [PASSED] 61 VFs
[22:50:11] [PASSED] 62 VFs
[22:50:11] [PASSED] 63 VFs
[22:50:11] ==================== [PASSED] fair_vram ====================
[22:50:11] ================== [PASSED] pf_gt_config ===================
[22:50:11] ===================== lmtt (1 subtest) =====================
[22:50:11] ======================== test_ops  =========================
[22:50:11] [PASSED] 2-level
[22:50:11] [PASSED] multi-level
[22:50:11] ==================== [PASSED] test_ops =====================
[22:50:11] ====================== [PASSED] lmtt =======================
[22:50:11] ================= sriov_packet (1 subtest) =================
[22:50:11] [PASSED] test_descriptor_init
[22:50:11] ================== [PASSED] sriov_packet ===================
[22:50:11] ================= pf_service (11 subtests) =================
[22:50:11] [PASSED] pf_negotiate_any
[22:50:11] [PASSED] pf_negotiate_base_match
[22:50:11] [PASSED] pf_negotiate_base_newer
[22:50:11] [PASSED] pf_negotiate_base_next
[22:50:11] [SKIPPED] pf_negotiate_base_older (no older minor)
[22:50:11] [PASSED] pf_negotiate_base_prev
[22:50:11] [PASSED] pf_negotiate_latest_match
[22:50:11] [PASSED] pf_negotiate_latest_newer
[22:50:11] [PASSED] pf_negotiate_latest_next
[22:50:11] [SKIPPED] pf_negotiate_latest_older (no older minor)
[22:50:11] [SKIPPED] pf_negotiate_latest_prev (no prev major)
[22:50:11] =================== [PASSED] pf_service ====================
[22:50:11] ================= xe_guc_g2g (2 subtests) ==================
[22:50:11] ============== xe_live_guc_g2g_kunit_default  ==============
[22:50:11] ========= [SKIPPED] xe_live_guc_g2g_kunit_default ==========
[22:50:11] ============== xe_live_guc_g2g_kunit_allmem  ===============
[22:50:11] ========== [SKIPPED] xe_live_guc_g2g_kunit_allmem ==========
[22:50:11] =================== [SKIPPED] xe_guc_g2g ===================
[22:50:11] =================== xe_mocs (2 subtests) ===================
[22:50:11] ================ xe_live_mocs_kernel_kunit  ================
[22:50:11] =========== [SKIPPED] xe_live_mocs_kernel_kunit ============
[22:50:11] ================ xe_live_mocs_reset_kunit  =================
[22:50:11] ============ [SKIPPED] xe_live_mocs_reset_kunit ============
[22:50:11] ==================== [SKIPPED] xe_mocs =====================
[22:50:11] ================= xe_migrate (3 subtests) ==================
[22:50:11] ================= xe_migrate_sanity_kunit  =================
[22:50:11] ============ [SKIPPED] xe_migrate_sanity_kunit =============
[22:50:11] ================== xe_validate_ccs_kunit  ==================
[22:50:11] ============= [SKIPPED] xe_validate_ccs_kunit ==============
[22:50:11] ============= xe_migrate_access_memory_kunit  ==============
[22:50:11] ========= [SKIPPED] xe_migrate_access_memory_kunit =========
[22:50:11] =================== [SKIPPED] xe_migrate ===================
[22:50:11] ================== xe_dma_buf (1 subtest) ==================
[22:50:11] ==================== xe_dma_buf_kunit  =====================
[22:50:11] ================ [SKIPPED] xe_dma_buf_kunit ================
[22:50:11] =================== [SKIPPED] xe_dma_buf ===================
[22:50:11] ================= xe_bo_shrink (1 subtest) =================
[22:50:11] =================== xe_bo_shrink_kunit  ====================
[22:50:11] =============== [SKIPPED] xe_bo_shrink_kunit ===============
[22:50:11] ================== [SKIPPED] xe_bo_shrink ==================
[22:50:11] ==================== xe_bo (2 subtests) ====================
[22:50:11] ================== xe_ccs_migrate_kunit  ===================
[22:50:11] ============== [SKIPPED] xe_ccs_migrate_kunit ==============
[22:50:11] ==================== xe_bo_evict_kunit  ====================
[22:50:11] =============== [SKIPPED] xe_bo_evict_kunit ================
[22:50:11] ===================== [SKIPPED] xe_bo ======================
[22:50:11] =================== xe_any (9 subtests) ====================
[22:50:11] [PASSED] test_to_xe
[22:50:11] [PASSED] test_to_dev
[22:50:11] [PASSED] test_to_pdev
[22:50:11] [PASSED] test_to_drm
[22:50:11] [PASSED] test_if_pdev
[22:50:11] [PASSED] test_if_xe
[22:50:11] [PASSED] test_if_tile
[22:50:11] [PASSED] test_if_gt
[22:50:11] [PASSED] test_to_id
[22:50:11] ===================== [PASSED] xe_any ======================
[22:50:11] ==================== args (13 subtests) ====================
[22:50:11] [PASSED] count_args_test
[22:50:11] [PASSED] call_args_example
[22:50:11] [PASSED] call_args_test
[22:50:11] [PASSED] drop_first_arg_example
[22:50:11] [PASSED] drop_first_arg_test
[22:50:11] [PASSED] first_arg_example
[22:50:11] [PASSED] first_arg_test
[22:50:11] [PASSED] last_arg_example
[22:50:11] [PASSED] last_arg_test
[22:50:11] [PASSED] pick_arg_example
[22:50:11] [PASSED] if_args_example
[22:50:11] [PASSED] if_args_test
[22:50:11] [PASSED] sep_comma_example
[22:50:11] ====================== [PASSED] args =======================
[22:50:11] =================== xe_pci (3 subtests) ====================
[22:50:11] ==================== check_graphics_ip  ====================
[22:50:11] [PASSED] 12.00 Xe_LP
[22:50:11] [PASSED] 12.10 Xe_LP+
[22:50:11] [PASSED] 12.55 Xe_HPG
[22:50:11] [PASSED] 12.60 Xe_HPC
[22:50:11] [PASSED] 12.70 Xe_LPG
[22:50:11] [PASSED] 12.71 Xe_LPG
[22:50:11] [PASSED] 12.74 Xe_LPG+
[22:50:11] [PASSED] 20.01 Xe2_HPG
[22:50:11] [PASSED] 20.02 Xe2_HPG
[22:50:11] [PASSED] 20.04 Xe2_LPG
[22:50:11] [PASSED] 30.00 Xe3_LPG
[22:50:11] [PASSED] 30.01 Xe3_LPG
[22:50:11] [PASSED] 30.03 Xe3_LPG
[22:50:11] [PASSED] 30.04 Xe3_LPG
[22:50:11] [PASSED] 30.05 Xe3_LPG
[22:50:11] [PASSED] 35.10 Xe3p_LPG
[22:50:11] [PASSED] 35.11 Xe3p_XPC
[22:50:11] ================ [PASSED] check_graphics_ip ================
[22:50:11] ===================== check_media_ip  ======================
[22:50:11] [PASSED] 12.00 Xe_M
[22:50:11] [PASSED] 12.55 Xe_HPM
[22:50:11] [PASSED] 13.00 Xe_LPM+
[22:50:11] [PASSED] 13.01 Xe2_HPM
[22:50:11] [PASSED] 20.00 Xe2_LPM
[22:50:11] [PASSED] 30.00 Xe3_LPM
[22:50:11] [PASSED] 30.02 Xe3_LPM
[22:50:11] [PASSED] 35.00 Xe3p_LPM
[22:50:11] [PASSED] 35.03 Xe3p_HPM
[22:50:11] ================= [PASSED] check_media_ip ==================
[22:50:11] =================== check_platform_desc  ===================
[22:50:11] [PASSED] 0x9A60 (TIGERLAKE)
[22:50:11] [PASSED] 0x9A68 (TIGERLAKE)
[22:50:11] [PASSED] 0x9A70 (TIGERLAKE)
[22:50:11] [PASSED] 0x9A40 (TIGERLAKE)
[22:50:11] [PASSED] 0x9A49 (TIGERLAKE)
[22:50:11] [PASSED] 0x9A59 (TIGERLAKE)
[22:50:11] [PASSED] 0x9A78 (TIGERLAKE)
[22:50:11] [PASSED] 0x9AC0 (TIGERLAKE)
[22:50:11] [PASSED] 0x9AC9 (TIGERLAKE)
[22:50:11] [PASSED] 0x9AD9 (TIGERLAKE)
[22:50:11] [PASSED] 0x9AF8 (TIGERLAKE)
[22:50:11] [PASSED] 0x4C80 (ROCKETLAKE)
[22:50:11] [PASSED] 0x4C8A (ROCKETLAKE)
[22:50:11] [PASSED] 0x4C8B (ROCKETLAKE)
[22:50:11] [PASSED] 0x4C8C (ROCKETLAKE)
[22:50:11] [PASSED] 0x4C90 (ROCKETLAKE)
[22:50:11] [PASSED] 0x4C9A (ROCKETLAKE)
[22:50:11] [PASSED] 0x4680 (ALDERLAKE_S)
[22:50:11] [PASSED] 0x4682 (ALDERLAKE_S)
[22:50:11] [PASSED] 0x4688 (ALDERLAKE_S)
[22:50:11] [PASSED] 0x468A (ALDERLAKE_S)
[22:50:11] [PASSED] 0x468B (ALDERLAKE_S)
[22:50:11] [PASSED] 0x4690 (ALDERLAKE_S)
[22:50:11] [PASSED] 0x4692 (ALDERLAKE_S)
[22:50:11] [PASSED] 0x4693 (ALDERLAKE_S)
[22:50:11] [PASSED] 0x46A0 (ALDERLAKE_P)
[22:50:11] [PASSED] 0x46A1 (ALDERLAKE_P)
[22:50:11] [PASSED] 0x46A2 (ALDERLAKE_P)
[22:50:11] [PASSED] 0x46A3 (ALDERLAKE_P)
[22:50:11] [PASSED] 0x46A6 (ALDERLAKE_P)
[22:50:11] [PASSED] 0x46A8 (ALDERLAKE_P)
[22:50:11] [PASSED] 0x46AA (ALDERLAKE_P)
[22:50:11] [PASSED] 0x462A (ALDERLAKE_P)
[22:50:11] [PASSED] 0x4626 (ALDERLAKE_P)
[22:50:11] [PASSED] 0x4628 (ALDERLAKE_P)
[22:50:11] [PASSED] 0x46B0 (ALDERLAKE_P)
[22:50:11] [PASSED] 0x46B1 (ALDERLAKE_P)
[22:50:11] [PASSED] 0x46B2 (ALDERLAKE_P)
[22:50:11] [PASSED] 0x46B3 (ALDERLAKE_P)
[22:50:11] [PASSED] 0x46C0 (ALDERLAKE_P)
[22:50:11] [PASSED] 0x46C1 (ALDERLAKE_P)
[22:50:11] [PASSED] 0x46C2 (ALDERLAKE_P)
[22:50:11] [PASSED] 0x46C3 (ALDERLAKE_P)
[22:50:11] [PASSED] 0x46D0 (ALDERLAKE_N)
[22:50:11] [PASSED] 0x46D1 (ALDERLAKE_N)
[22:50:11] [PASSED] 0x46D2 (ALDERLAKE_N)
[22:50:11] [PASSED] 0x46D3 (ALDERLAKE_N)
[22:50:11] [PASSED] 0x46D4 (ALDERLAKE_N)
[22:50:11] [PASSED] 0xA721 (ALDERLAKE_P)
[22:50:11] [PASSED] 0xA7A1 (ALDERLAKE_P)
[22:50:11] [PASSED] 0xA7A9 (ALDERLAKE_P)
[22:50:11] [PASSED] 0xA7AC (ALDERLAKE_P)
[22:50:11] [PASSED] 0xA7AD (ALDERLAKE_P)
[22:50:11] [PASSED] 0xA720 (ALDERLAKE_P)
[22:50:11] [PASSED] 0xA7A0 (ALDERLAKE_P)
[22:50:11] [PASSED] 0xA7A8 (ALDERLAKE_P)
[22:50:11] [PASSED] 0xA7AA (ALDERLAKE_P)
[22:50:11] [PASSED] 0xA7AB (ALDERLAKE_P)
[22:50:11] [PASSED] 0xA780 (ALDERLAKE_S)
[22:50:11] [PASSED] 0xA781 (ALDERLAKE_S)
[22:50:11] [PASSED] 0xA782 (ALDERLAKE_S)
[22:50:11] [PASSED] 0xA783 (ALDERLAKE_S)
[22:50:11] [PASSED] 0xA788 (ALDERLAKE_S)
[22:50:11] [PASSED] 0xA789 (ALDERLAKE_S)
[22:50:11] [PASSED] 0xA78A (ALDERLAKE_S)
[22:50:11] [PASSED] 0xA78B (ALDERLAKE_S)
[22:50:11] [PASSED] 0x4905 (DG1)
[22:50:11] [PASSED] 0x4906 (DG1)
[22:50:11] [PASSED] 0x4907 (DG1)
[22:50:11] [PASSED] 0x4908 (DG1)
[22:50:11] [PASSED] 0x4909 (DG1)
[22:50:11] [PASSED] 0x56C0 (DG2)
[22:50:11] [PASSED] 0x56C2 (DG2)
[22:50:11] [PASSED] 0x56C1 (DG2)
[22:50:11] [PASSED] 0x7D51 (METEORLAKE)
[22:50:11] [PASSED] 0x7DD1 (METEORLAKE)
[22:50:11] [PASSED] 0x7D41 (METEORLAKE)
[22:50:11] [PASSED] 0x7D67 (METEORLAKE)
[22:50:11] [PASSED] 0xB640 (METEORLAKE)
[22:50:11] [PASSED] 0x56A0 (DG2)
[22:50:11] [PASSED] 0x56A1 (DG2)
[22:50:11] [PASSED] 0x56A2 (DG2)
[22:50:11] [PASSED] 0x56BE (DG2)
[22:50:11] [PASSED] 0x56BF (DG2)
[22:50:11] [PASSED] 0x5690 (DG2)
[22:50:11] [PASSED] 0x5691 (DG2)
[22:50:11] [PASSED] 0x5692 (DG2)
[22:50:11] [PASSED] 0x56A5 (DG2)
[22:50:11] [PASSED] 0x56A6 (DG2)
[22:50:11] [PASSED] 0x56B0 (DG2)
[22:50:11] [PASSED] 0x56B1 (DG2)
[22:50:11] [PASSED] 0x56BA (DG2)
[22:50:11] [PASSED] 0x56BB (DG2)
[22:50:11] [PASSED] 0x56BC (DG2)
[22:50:11] [PASSED] 0x56BD (DG2)
[22:50:11] [PASSED] 0x5693 (DG2)
[22:50:11] [PASSED] 0x5694 (DG2)
[22:50:11] [PASSED] 0x5695 (DG2)
[22:50:11] [PASSED] 0x56A3 (DG2)
[22:50:11] [PASSED] 0x56A4 (DG2)
[22:50:11] [PASSED] 0x56B2 (DG2)
[22:50:11] [PASSED] 0x56B3 (DG2)
[22:50:11] [PASSED] 0x5696 (DG2)
[22:50:11] [PASSED] 0x5697 (DG2)
[22:50:11] [PASSED] 0xB69 (PVC)
[22:50:11] [PASSED] 0xB6E (PVC)
[22:50:11] [PASSED] 0xBD4 (PVC)
[22:50:11] [PASSED] 0xBD5 (PVC)
[22:50:11] [PASSED] 0xBD6 (PVC)
[22:50:11] [PASSED] 0xBD7 (PVC)
[22:50:11] [PASSED] 0xBD8 (PVC)
[22:50:11] [PASSED] 0xBD9 (PVC)
[22:50:11] [PASSED] 0xBDA (PVC)
[22:50:11] [PASSED] 0xBDB (PVC)
[22:50:11] [PASSED] 0xBE0 (PVC)
[22:50:11] [PASSED] 0xBE1 (PVC)
[22:50:11] [PASSED] 0xBE5 (PVC)
[22:50:11] [PASSED] 0x7D40 (METEORLAKE)
[22:50:11] [PASSED] 0x7D45 (METEORLAKE)
[22:50:11] [PASSED] 0x7D55 (METEORLAKE)
[22:50:11] [PASSED] 0x7D60 (METEORLAKE)
[22:50:11] [PASSED] 0x7DD5 (METEORLAKE)
[22:50:11] [PASSED] 0x6420 (LUNARLAKE)
[22:50:11] [PASSED] 0x64A0 (LUNARLAKE)
[22:50:11] [PASSED] 0x64B0 (LUNARLAKE)
[22:50:11] [PASSED] 0xE202 (BATTLEMAGE)
[22:50:11] [PASSED] 0xE209 (BATTLEMAGE)
[22:50:11] [PASSED] 0xE20B (BATTLEMAGE)
[22:50:11] [PASSED] 0xE20C (BATTLEMAGE)
[22:50:11] [PASSED] 0xE20D (BATTLEMAGE)
[22:50:11] [PASSED] 0xE210 (BATTLEMAGE)
[22:50:11] [PASSED] 0xE211 (BATTLEMAGE)
[22:50:11] [PASSED] 0xE212 (BATTLEMAGE)
[22:50:11] [PASSED] 0xE216 (BATTLEMAGE)
[22:50:11] [PASSED] 0xE220 (BATTLEMAGE)
[22:50:11] [PASSED] 0xE221 (BATTLEMAGE)
[22:50:11] [PASSED] 0xE222 (BATTLEMAGE)
[22:50:11] [PASSED] 0xE223 (BATTLEMAGE)
[22:50:11] [PASSED] 0xB080 (PANTHERLAKE)
[22:50:11] [PASSED] 0xB081 (PANTHERLAKE)
[22:50:11] [PASSED] 0xB082 (PANTHERLAKE)
[22:50:11] [PASSED] 0xB083 (PANTHERLAKE)
[22:50:11] [PASSED] 0xB084 (PANTHERLAKE)
[22:50:11] [PASSED] 0xB085 (PANTHERLAKE)
[22:50:11] [PASSED] 0xB086 (PANTHERLAKE)
[22:50:11] [PASSED] 0xB087 (PANTHERLAKE)
[22:50:11] [PASSED] 0xB08F (PANTHERLAKE)
[22:50:11] [PASSED] 0xB090 (PANTHERLAKE)
[22:50:11] [PASSED] 0xB0A0 (PANTHERLAKE)
[22:50:11] [PASSED] 0xB0A1 (PANTHERLAKE)
[22:50:11] [PASSED] 0xB0B0 (PANTHERLAKE)
[22:50:11] [PASSED] 0xFD80 (PANTHERLAKE)
[22:50:11] [PASSED] 0xFD81 (PANTHERLAKE)
[22:50:11] [PASSED] 0xD740 (NOVALAKE_S)
[22:50:11] [PASSED] 0xD741 (NOVALAKE_S)
[22:50:11] [PASSED] 0xD742 (NOVALAKE_S)
[22:50:11] [PASSED] 0xD743 (NOVALAKE_S)
[22:50:11] [PASSED] 0xD745 (NOVALAKE_S)
[22:50:11] [PASSED] 0xD74A (NOVALAKE_S)
[22:50:11] [PASSED] 0xD74B (NOVALAKE_S)
[22:50:11] [PASSED] 0x674C (CRESCENTISLAND)
[22:50:11] [PASSED] 0x674D (CRESCENTISLAND)
[22:50:11] [PASSED] 0x674E (CRESCENTISLAND)
[22:50:11] [PASSED] 0x674F (CRESCENTISLAND)
[22:50:11] [PASSED] 0x6750 (CRESCENTISLAND)
[22:50:11] [PASSED] 0x6751 (CRESCENTISLAND)
[22:50:11] [PASSED] 0x6752 (CRESCENTISLAND)
[22:50:11] [PASSED] 0xD750 (NOVALAKE_P)
[22:50:11] [PASSED] 0xD751 (NOVALAKE_P)
[22:50:11] [PASSED] 0xD752 (NOVALAKE_P)
[22:50:11] [PASSED] 0xD753 (NOVALAKE_P)
[22:50:11] [PASSED] 0xD754 (NOVALAKE_P)
[22:50:11] [PASSED] 0xD755 (NOVALAKE_P)
[22:50:11] [PASSED] 0xD756 (NOVALAKE_P)
[22:50:11] [PASSED] 0xD757 (NOVALAKE_P)
[22:50:11] [PASSED] 0xD75F (NOVALAKE_P)
[22:50:11] =============== [PASSED] check_platform_desc ===============
[22:50:11] ===================== [PASSED] xe_pci ======================
[22:50:11] ============= xe_rtp_tables_test (5 subtests) ==============
[22:50:11] ================== xe_rtp_table_gt_test  ===================
[22:50:11] [PASSED] gt_was/14011060649
[22:50:11] [PASSED] gt_was/14011059788
[22:50:11] [PASSED] gt_was/14015795083
[22:50:11] [PASSED] gt_was/16021867713
[22:50:11] [PASSED] gt_was/14019449301
[22:50:11] [PASSED] gt_was/16028005424
[22:50:11] [PASSED] gt_was/14026578760
[22:50:11] [PASSED] gt_was/1409420604
[22:50:11] [PASSED] gt_was/1408615072
[22:50:11] [PASSED] gt_was/22010523718
[22:50:11] [PASSED] gt_was/14011006942
[22:50:11] [PASSED] gt_was/14014830051
[22:50:11] [PASSED] gt_was/18018781329
[22:50:11] [PASSED] gt_was/1509235366
[22:50:11] [PASSED] gt_was/18018781329
[22:50:11] [PASSED] gt_was/16016694945
[22:50:11] [PASSED] gt_was/14018575942
[22:50:11] [PASSED] gt_was/22016670082
[22:50:11] [PASSED] gt_was/22016670082
[22:50:11] [PASSED] gt_was/14017421178
[22:50:11] [PASSED] gt_was/16025250150
[22:50:11] [PASSED] gt_was/14021871409
[22:50:11] [PASSED] gt_was/16021865536
[22:50:11] [PASSED] gt_was/14021486841
[22:50:11] [PASSED] gt_was/14025160223
[22:50:11] [PASSED] gt_was/14026144927, 16029437861, 14026127056
[22:50:11] [PASSED] gt_was/14025635424
[22:50:11] [PASSED] gt_was/16028005424
[22:50:11] ============== [PASSED] xe_rtp_table_gt_test ===============
[22:50:11] ================== xe_rtp_table_gt_test  ===================
[22:50:11] [PASSED] gt_tunings/Tuning: Blend Fill Caching Optimization Disable
[22:50:11] [PASSED] gt_tunings/Tuning: 32B Access Enable
[22:50:11] [PASSED] gt_tunings/Tuning: L3 cache
[22:50:11] [PASSED] gt_tunings/Tuning: L3 cache - media
[22:50:11] [PASSED] gt_tunings/Tuning: Compression Overfetch
[22:50:11] [PASSED] gt_tunings/Tuning: Compression Overfetch - media
[22:50:11] [PASSED] gt_tunings/Tuning: Enable compressible partial write overfetch in L3
[22:50:11] [PASSED] gt_tunings/Tuning: Enable compressible partial write overfetch in L3 - media
[22:50:11] [PASSED] gt_tunings/Tuning: L2 Overfetch Compressible Only
[22:50:11] [PASSED] gt_tunings/Tuning: L2 Overfetch Compressible Only - media
[22:50:11] [PASSED] gt_tunings/Tuning: Stateless compression control
[22:50:11] [PASSED] gt_tunings/Tuning: Stateless compression control - media
[22:50:11] [PASSED] gt_tunings/Tuning: L3 RW flush all Cache
[22:50:11] [PASSED] gt_tunings/Tuning: L3 RW flush all cache - media
[22:50:11] [PASSED] gt_tunings/Tuning: Set STLB Bank Hash Mode to 4KB
[22:50:11] [PASSED] gt_tunings/Tuning: Force All Non-Compressible Memory Reads to be 256B Overfetches
[22:50:11] ============== [PASSED] xe_rtp_table_gt_test ===============
[22:50:11] ================== xe_rtp_table_oob_test  ==================
[22:50:11] [PASSED] oob_was/1607983814
[22:50:11] [PASSED] oob_was/16010904313
[22:50:11] [PASSED] oob_was/18022495364
[22:50:11] [PASSED] oob_was/22012773006
[22:50:11] [PASSED] oob_was/14014475959
[22:50:11] [PASSED] oob_was/22011391025
[22:50:11] [PASSED] oob_was/22012727170
[22:50:11] [PASSED] oob_was/22012727685
[22:50:11] [PASSED] oob_was/22016596838
[22:50:11] [PASSED] oob_was/18020744125
[22:50:11] [PASSED] oob_was/1409600907
[22:50:11] [PASSED] oob_was/22014953428
[22:50:11] [PASSED] oob_was/16017236439
[22:50:11] [PASSED] oob_was/14019821291
[22:50:11] [PASSED] oob_was/14015076503
[22:50:11] [PASSED] oob_was/22016122933
[22:50:11] [PASSED] oob_was/14018913170
[22:50:11] [PASSED] oob_was/14018094691
[22:50:11] [PASSED] oob_was/18024947630
[22:50:11] [PASSED] oob_was/16022287689
[22:50:11] [PASSED] oob_was/13011645652
[22:50:11] [PASSED] oob_was/14022293748
[22:50:11] [PASSED] oob_was/22019794406
[22:50:11] [PASSED] oob_was/22019338487
[22:50:11] [PASSED] oob_was/16023588340
[22:50:11] [PASSED] oob_was/14019789679
[22:50:11] [PASSED] oob_was/14022866841
[22:50:11] [PASSED] oob_was/16021333562
[22:50:11] [PASSED] oob_was/14016712196
[22:50:11] [PASSED] oob_was/14015568240
[22:50:11] [PASSED] oob_was/18013179988
[22:50:11] [PASSED] oob_was/1508761755
[22:50:11] [PASSED] oob_was/16023105232
[22:50:11] [PASSED] oob_was/16026508708
[22:50:11] [PASSED] oob_was/14020001231
[22:50:11] [PASSED] oob_was/16023683509
[22:50:11] [PASSED] oob_was/14025515070
[22:50:11] [PASSED] oob_was/15015404425_disable
[22:50:11] [PASSED] oob_was/16026007364
[22:50:11] [PASSED] oob_was/14020316580
[22:50:11] [PASSED] oob_was/14025883347
[22:50:11] [PASSED] oob_was/16029380221
[22:50:11] [PASSED] oob_was/22022079272
[22:50:11] [PASSED] oob_was/16029897822
[22:50:11] [PASSED] oob_was/14026539277
[22:50:11] [PASSED] oob_was/14027054324
[22:50:11] [PASSED] oob_was/14025941587
[22:50:11] [PASSED] oob_was/16030224309
[22:50:11] [PASSED] oob_was/14028709524
[22:50:11] ============== [PASSED] xe_rtp_table_oob_test ==============
[22:50:11] ================ xe_rtp_table_dev_oob_test  ================
[22:50:11] [PASSED] device_oob_was/22010954014
[22:50:11] [PASSED] device_oob_was/15015404425
[22:50:11] [PASSED] device_oob_was/22019338487_display
[22:50:11] [PASSED] device_oob_was/14022085890
[22:50:11] [PASSED] device_oob_was/14026633728
[22:50:11] [PASSED] device_oob_was/14026746987
[22:50:11] [PASSED] device_oob_was/14026779378
[22:50:11] ============ [PASSED] xe_rtp_table_dev_oob_test ============
[22:50:11] ========== xe_rtp_table_missing_upper_bound_test  ==========
[22:50:11] [PASSED] register_whitelist/WaAllowPMDepthAndInvocationCountAccessFromUMD, 1408556865
[22:50:11] [PASSED] register_whitelist/1508744258, 14012131227, 1808121037
[22:50:11] [PASSED] register_whitelist/1806527549
[22:50:11] [PASSED] register_whitelist/allow_read_ctx_timestamp
[22:50:11] [PASSED] register_whitelist/allow_read_queue_timestamp
[22:50:11] [PASSED] register_whitelist/16014440446
[22:50:11] [PASSED] register_whitelist/16017236439
[22:50:11] [PASSED] register_whitelist/16020183090
[22:50:11] [PASSED] register_whitelist/14024997852
[22:50:11] [PASSED] register_whitelist/14024997852
[22:50:11] ====== [PASSED] xe_rtp_table_missing_upper_bound_test ======
[22:50:11] =============== [PASSED] xe_rtp_tables_test ================
[22:50:11] =================== xe_rtp (3 subtests) ====================
[22:50:11] =================== xe_rtp_rules_tests  ====================
[22:50:11] [PASSED] no
[22:50:11] [PASSED] yes
[22:50:11] [PASSED] no-and-no
[22:50:11] [PASSED] no-and-yes
[22:50:11] [PASSED] yes-and-no
[22:50:11] [PASSED] yes-and-yes
[22:50:11] [PASSED] no-or-no
[22:50:11] [PASSED] no-or-yes
[22:50:11] [PASSED] yes-or-no
[22:50:11] [PASSED] yes-or-yes
[22:50:11] [PASSED] no-yes-or-yes-no
[22:50:11] [PASSED] no-yes-or-yes-yes
[22:50:11] [PASSED] yes-yes-or-no-yes
[22:50:11] [PASSED] yes-yes-or-yes-yes
[22:50:11] [PASSED] no-no-or-yes-or-no
[22:50:11] [PASSED] or
[22:50:11] [PASSED] or-yes
[22:50:11] [PASSED] or-no
[22:50:11] [PASSED] yes-or
[22:50:11] [PASSED] no-or
[22:50:11] [PASSED] no-or-or-yes
[22:50:11] [PASSED] yes-or-or-no
[22:50:11] [PASSED] no-or-or-no
[22:50:11] [PASSED] missing-context-engine-class
[22:50:11] [PASSED] missing-context-engine-class-or-yes
[22:50:11] [PASSED] missing-context-engine-class-or-or-yes
[22:50:11] =============== [PASSED] xe_rtp_rules_tests ================
[22:50:11] =============== xe_rtp_process_to_sr_tests  ================
[22:50:11] [PASSED] coalesce-same-reg
[22:50:11] [PASSED] coalesce-same-reg-literal-and-func
[22:50:11] [PASSED] no-match-no-add
[22:50:11] [PASSED] two-regs-two-entries
[22:50:11] [PASSED] clr-one-set-other
[22:50:11] [PASSED] set-field
[22:50:11] [PASSED] conflict-duplicate
[22:50:11] [PASSED] conflict-not-disjoint
[22:50:11] [PASSED] conflict-not-disjoint-literal-and-func
[22:50:11] [PASSED] conflict-reg-type
[22:50:11] [PASSED] bad-mcr-reg-forced-to-regular
[22:50:11] [PASSED] bad-regular-reg-forced-to-mcr
[22:50:11] =========== [PASSED] xe_rtp_process_to_sr_tests ============
[22:50:11] ================== xe_rtp_process_tests  ===================
[22:50:11] [PASSED] active1
[22:50:11] [PASSED] active2
[22:50:11] [PASSED] active-inactive
[22:50:11] [PASSED] inactive-active
[22:50:11] [PASSED] inactive-active-inactive
[22:50:11] [PASSED] inactive-inactive-inactive
[22:50:11] ============== [PASSED] xe_rtp_process_tests ===============
[22:50:11] ===================== [PASSED] xe_rtp ======================
[22:50:11] ==================== xe_wa (1 subtest) =====================
[22:50:11] ======================== xe_wa_gt  =========================
[22:50:11] [PASSED] TIGERLAKE B0
[22:50:11] [PASSED] DG1 A0
[22:50:11] [PASSED] DG1 B0
[22:50:11] [PASSED] ALDERLAKE_S A0
[22:50:11] [PASSED] ALDERLAKE_S B0
[22:50:11] [PASSED] ALDERLAKE_S C0
[22:50:11] [PASSED] ALDERLAKE_S D0
[22:50:11] [PASSED] ALDERLAKE_P A0
[22:50:11] [PASSED] ALDERLAKE_P B0
[22:50:11] [PASSED] ALDERLAKE_P C0
[22:50:11] [PASSED] ALDERLAKE_S RPLS D0
[22:50:11] [PASSED] ALDERLAKE_P RPLU E0
[22:50:11] [PASSED] DG2 G10 C0
[22:50:11] [PASSED] DG2 G11 B1
[22:50:11] [PASSED] DG2 G12 A1
[22:50:11] [PASSED] METEORLAKE 12.70(Xe_LPG) A0 13.00(Xe_LPM+) A0
[22:50:11] [PASSED] METEORLAKE 12.71(Xe_LPG) A0 13.00(Xe_LPM+) A0
[22:50:11] [PASSED] METEORLAKE 12.74(Xe_LPG+) A0 13.00(Xe_LPM+) A0
[22:50:11] [PASSED] LUNARLAKE 20.04(Xe2_LPG) A0 20.00(Xe2_LPM) A0
[22:50:11] [PASSED] LUNARLAKE 20.04(Xe2_LPG) B0 20.00(Xe2_LPM) A0
[22:50:11] [PASSED] BATTLEMAGE 20.01(Xe2_HPG) A0 13.01(Xe2_HPM) A1
[22:50:11] [PASSED] PANTHERLAKE 30.00(Xe3_LPG) A0 30.00(Xe3_LPM) A0
[22:50:11] ==================== [PASSED] xe_wa_gt =====================
[22:50:11] ====================== [PASSED] xe_wa ======================
[22:50:11] ============================================================
[22:50:11] Testing complete. Ran 840 tests: passed: 811, skipped: 29
[22:50:12] Elapsed time: 42.163s total, 2.448s configuring, 38.948s building, 0.712s running

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

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

+ for kcfg in "${kunitconfigs[@]}"
+ [[ ! -f /kernel/drivers/dma-buf/.kunitconfig ]]
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/dma-buf/.kunitconfig
[22:50:53] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[22:50:56] 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
[22:51:04] Starting KUnit Kernel (1/1)...
[22:51:04] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[22:51:05] =============== dma-buf-fence (12 subtests) ================
[22:51:05] [PASSED] test_sanitycheck
[22:51:05] [PASSED] test_signaling
[22:51:05] [PASSED] test_add_callback
[22:51:05] [PASSED] test_late_add_callback
[22:51:05] [PASSED] test_rm_callback
[22:51:05] [PASSED] test_late_rm_callback
[22:51:05] [PASSED] test_status
[22:51:05] [PASSED] test_error
[22:51:05] [PASSED] test_wait
[22:51:05] [PASSED] test_wait_timeout
[22:51:05] [PASSED] test_stub
[22:51:05] [SKIPPED] test_race_signal_callback (requires at least 2 CPUs)
[22:51:05] ================== [PASSED] dma-buf-fence ==================
[22:51:05] ============ dma-buf-fence-chain (11 subtests) =============
[22:51:05] [PASSED] test_sanitycheck
[22:51:05] [PASSED] test_find_seqno
[22:51:05] [PASSED] test_find_signaled
[22:51:05] [PASSED] test_find_out_of_order
[22:51:10] [PASSED] test_find_gap
[22:51:10] [PASSED] test_find_race
[22:51:10] [PASSED] test_signal_forward
[22:51:10] [PASSED] test_signal_backward
[22:51:10] [PASSED] test_wait_forward
[22:51:10] [PASSED] test_wait_backward
[22:51:10] [PASSED] test_wait_random
[22:51:10] =============== [PASSED] dma-buf-fence-chain ===============
[22:51:10] ============ dma-buf-fence-unwrap (11 subtests) ============
[22:51:10] [PASSED] test_sanitycheck
[22:51:10] [PASSED] test_unwrap_array
[22:51:10] [PASSED] test_unwrap_chain
[22:51:10] [PASSED] test_unwrap_chain_array
[22:51:10] [PASSED] test_unwrap_merge
[22:51:10] [PASSED] test_unwrap_merge_duplicate
[22:51:10] [PASSED] test_unwrap_merge_seqno
[22:51:10] [PASSED] test_unwrap_merge_order
[22:51:10] [PASSED] test_unwrap_merge_complex
[22:51:10] [PASSED] test_unwrap_merge_complex_seqno
[22:51:10] [PASSED] test_dedup_empty
[22:51:10] ============== [PASSED] dma-buf-fence-unwrap ===============
[22:51:10] ================ dma-buf-resv (5 subtests) =================
[22:51:10] [PASSED] test_sanitycheck
[22:51:10] ===================== test_signaling  ======================
[22:51:10] [PASSED] kernel
[22:51:10] [PASSED] write
[22:51:10] [PASSED] read
[22:51:10] [PASSED] bookkeep
[22:51:10] ================= [PASSED] test_signaling ==================
[22:51:10] ====================== test_for_each  ======================
[22:51:10] [PASSED] kernel
[22:51:10] [PASSED] write
[22:51:10] [PASSED] read
[22:51:10] [PASSED] bookkeep
[22:51:10] ================== [PASSED] test_for_each ==================
[22:51:10] ================= test_for_each_unlocked  ==================
[22:51:10] [PASSED] kernel
[22:51:10] [PASSED] write
[22:51:10] [PASSED] read
[22:51:10] [PASSED] bookkeep
[22:51:10] ============= [PASSED] test_for_each_unlocked ==============
[22:51:10] ===================== test_get_fences  =====================
[22:51:10] [PASSED] kernel
[22:51:10] [PASSED] write
[22:51:10] [PASSED] read
[22:51:10] [PASSED] bookkeep
[22:51:10] ================= [PASSED] test_get_fences =================
[22:51:10] ================== [PASSED] dma-buf-resv ===================
[22:51:10] ============================================================
[22:51:10] Testing complete. Ran 51 tests: passed: 50, skipped: 1
[22:51:10] Elapsed time: 16.687s total, 2.510s configuring, 8.906s building, 5.262s running

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

✓ Xe.CI.BAT: success for Crescent Island PMT support (rev15)

[Patchwork]

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

== Series Details ==

Series: Crescent Island PMT support (rev15)
URL   : https://patchwork.freedesktop.org/series/160717/
State : success

== Summary ==

CI Bug Log - changes from xe-6047-2cd78713d917a02987f21b437561a5e1e13ddda2_BAT -> xe-pw-160717v15_BAT
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  

Participating hosts (15 -> 15)
------------------------------

  No changes in participating hosts

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

  Here are the changes found in xe-pw-160717v15_BAT that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@runner@aborted:
    - bat-atsm-2:         NOTRUN -> [FAIL][1] ([Intel XE#9717])
   [1]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160717v15/bat-atsm-2/igt@runner@aborted.html

  
  [Intel XE#9717]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/9717


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

  * Linux: xe-6047-2cd78713d917a02987f21b437561a5e1e13ddda2 -> xe-pw-160717v15

  IGT_9175: 9175
  xe-6047-2cd78713d917a02987f21b437561a5e1e13ddda2: 2cd78713d917a02987f21b437561a5e1e13ddda2
  xe-pw-160717v15: 160717v15

== Logs ==

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

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

✓ Xe.CI.FULL: success for Crescent Island PMT support (rev15)

[Patchwork]

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

== Series Details ==

Series: Crescent Island PMT support (rev15)
URL   : https://patchwork.freedesktop.org/series/160717/
State : success

== Summary ==

CI Bug Log - changes from xe-6047-2cd78713d917a02987f21b437561a5e1e13ddda2_FULL -> xe-pw-160717v15_FULL
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  

Participating hosts (3 -> 3)
------------------------------

  No changes in participating hosts


Changes
-------

  No changes found


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

  * Linux: xe-6047-2cd78713d917a02987f21b437561a5e1e13ddda2 -> xe-pw-160717v15

  IGT_9175: 9175
  xe-6047-2cd78713d917a02987f21b437561a5e1e13ddda2: 2cd78713d917a02987f21b437561a5e1e13ddda2
  xe-pw-160717v15: 160717v15

== Logs ==

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

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