intel-gfx.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
* [Intel-gfx] [PATCH] drm/i915/gsc: Assign a uabi class number to the GSC CS
@ 2023-11-09 23:53 Daniele Ceraolo Spurio
  2023-11-10  4:07 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for " Patchwork
                   ` (3 more replies)
  0 siblings, 4 replies; 10+ messages in thread
From: Daniele Ceraolo Spurio @ 2023-11-09 23:53 UTC (permalink / raw)
  To: intel-gfx; +Cc: Matt Roper, Alan Previn

The GSC CS is not exposed to the user, so we skipped assigning a uabi
class number for it. However, the trace logs use the uabi class and
instance to identify the engine, so leaving uabi class unset makes the
GSC CS show up as the RCS in those logs.
Given that the engine is not exposed to the user, we can't add a new
case in the uabi enum, so we insted internally define a kernel
reserved class using the next free number.

Fixes: 194babe26bdc ("drm/i915/mtl: don't expose GSC command streamer to the user")
Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Cc: Alan Previn <alan.previn.teres.alexis@intel.com>
Cc: Matt Roper <matthew.d.roper@intel.com>
---
 drivers/gpu/drm/i915/gt/intel_engine_user.c | 17 ++++++++---------
 drivers/gpu/drm/i915/gt/intel_engine_user.h |  4 ++++
 drivers/gpu/drm/i915/i915_drm_client.h      |  2 +-
 drivers/gpu/drm/i915/i915_drv.h             |  2 +-
 4 files changed, 14 insertions(+), 11 deletions(-)

diff --git a/drivers/gpu/drm/i915/gt/intel_engine_user.c b/drivers/gpu/drm/i915/gt/intel_engine_user.c
index 118164ddbb2e..3fd32bedd6e7 100644
--- a/drivers/gpu/drm/i915/gt/intel_engine_user.c
+++ b/drivers/gpu/drm/i915/gt/intel_engine_user.c
@@ -47,6 +47,7 @@ static const u8 uabi_classes[] = {
 	[VIDEO_DECODE_CLASS] = I915_ENGINE_CLASS_VIDEO,
 	[VIDEO_ENHANCEMENT_CLASS] = I915_ENGINE_CLASS_VIDEO_ENHANCE,
 	[COMPUTE_CLASS] = I915_ENGINE_CLASS_COMPUTE,
+	[OTHER_CLASS] = I915_KERNEL_RSVD_CLASS,
 };
 
 static int engine_cmp(void *priv, const struct list_head *A,
@@ -138,7 +139,7 @@ const char *intel_engine_class_repr(u8 class)
 		[COPY_ENGINE_CLASS] = "bcs",
 		[VIDEO_DECODE_CLASS] = "vcs",
 		[VIDEO_ENHANCEMENT_CLASS] = "vecs",
-		[OTHER_CLASS] = "other",
+		[OTHER_CLASS] = "gsc",
 		[COMPUTE_CLASS] = "ccs",
 	};
 
@@ -216,14 +217,8 @@ void intel_engines_driver_register(struct drm_i915_private *i915)
 		if (intel_gt_has_unrecoverable_error(engine->gt))
 			continue; /* ignore incomplete engines */
 
-		/*
-		 * We don't want to expose the GSC engine to the users, but we
-		 * still rename it so it is easier to identify in the debug logs
-		 */
-		if (engine->id == GSC0) {
-			engine_rename(engine, "gsc", 0);
-			continue;
-		}
+		/* The only engine we expect in OTHER_CLASS is GSC0 */
+		GEM_WARN_ON(engine->class == OTHER_CLASS && engine->id != GSC0);
 
 		GEM_BUG_ON(engine->class >= ARRAY_SIZE(uabi_classes));
 		engine->uabi_class = uabi_classes[engine->class];
@@ -238,6 +233,10 @@ void intel_engines_driver_register(struct drm_i915_private *i915)
 			      intel_engine_class_repr(engine->class),
 			      engine->uabi_instance);
 
+		/* We don't want to expose the GSC engine to the users */
+		if (engine->id == GSC0)
+			continue;
+
 		rb_link_node(&engine->uabi_node, prev, p);
 		rb_insert_color(&engine->uabi_node, &i915->uabi_engines);
 
diff --git a/drivers/gpu/drm/i915/gt/intel_engine_user.h b/drivers/gpu/drm/i915/gt/intel_engine_user.h
index 3dc7e8ab9fbc..dd31805b2a5a 100644
--- a/drivers/gpu/drm/i915/gt/intel_engine_user.h
+++ b/drivers/gpu/drm/i915/gt/intel_engine_user.h
@@ -11,6 +11,10 @@
 struct drm_i915_private;
 struct intel_engine_cs;
 
+#define I915_LAST_UABI_ENGINE_CLASS I915_ENGINE_CLASS_COMPUTE
+#define I915_KERNEL_RSVD_CLASS (I915_LAST_UABI_ENGINE_CLASS + 1)
+#define I915_MAX_UABI_CLASSES (I915_KERNEL_RSVD_CLASS + 1)
+
 struct intel_engine_cs *
 intel_engine_lookup_user(struct drm_i915_private *i915, u8 class, u8 instance);
 
diff --git a/drivers/gpu/drm/i915/i915_drm_client.h b/drivers/gpu/drm/i915/i915_drm_client.h
index 67816c912bca..c42cb2511348 100644
--- a/drivers/gpu/drm/i915/i915_drm_client.h
+++ b/drivers/gpu/drm/i915/i915_drm_client.h
@@ -12,7 +12,7 @@
 
 #include <uapi/drm/i915_drm.h>
 
-#define I915_LAST_UABI_ENGINE_CLASS I915_ENGINE_CLASS_COMPUTE
+#include "gt/intel_engine_user.h"
 
 struct drm_file;
 struct drm_printer;
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index f3be9033a93f..a718b4cb5a2d 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -238,7 +238,7 @@ struct drm_i915_private {
 		struct list_head uabi_engines_list;
 		struct rb_root uabi_engines;
 	};
-	unsigned int engine_uabi_class_count[I915_LAST_UABI_ENGINE_CLASS + 1];
+	unsigned int engine_uabi_class_count[I915_MAX_UABI_CLASSES];
 
 	/* protects the irq masks */
 	spinlock_t irq_lock;
-- 
2.41.0


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

* [Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/gsc: Assign a uabi class number to the GSC CS
  2023-11-09 23:53 [Intel-gfx] [PATCH] drm/i915/gsc: Assign a uabi class number to the GSC CS Daniele Ceraolo Spurio
@ 2023-11-10  4:07 ` Patchwork
  2023-11-10  4:20 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2023-11-10  4:07 UTC (permalink / raw)
  To: Daniele Ceraolo Spurio; +Cc: intel-gfx

== Series Details ==

Series: drm/i915/gsc: Assign a uabi class number to the GSC CS
URL   : https://patchwork.freedesktop.org/series/126225/
State : warning

== Summary ==

Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.



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

* [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/gsc: Assign a uabi class number to the GSC CS
  2023-11-09 23:53 [Intel-gfx] [PATCH] drm/i915/gsc: Assign a uabi class number to the GSC CS Daniele Ceraolo Spurio
  2023-11-10  4:07 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for " Patchwork
@ 2023-11-10  4:20 ` Patchwork
  2023-11-10 10:25 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
  2023-11-10 12:00 ` [Intel-gfx] [PATCH] " Tvrtko Ursulin
  3 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2023-11-10  4:20 UTC (permalink / raw)
  To: Daniele Ceraolo Spurio; +Cc: intel-gfx

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

== Series Details ==

Series: drm/i915/gsc: Assign a uabi class number to the GSC CS
URL   : https://patchwork.freedesktop.org/series/126225/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_13859 -> Patchwork_126225v1
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/index.html

Participating hosts (33 -> 34)
------------------------------

  Additional (2): fi-hsw-4770 bat-dg2-9 
  Missing    (1): fi-kbl-soraka 

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

  Here are the changes found in Patchwork_126225v1 that come from known issues:

### CI changes ###

#### Issues hit ####

  * boot:
    - fi-hsw-4770:        NOTRUN -> [FAIL][1] ([i915#8293])
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/fi-hsw-4770/boot.html

  

### IGT changes ###

#### Issues hit ####

  * igt@gem_mmap@basic:
    - bat-dg2-9:          NOTRUN -> [SKIP][2] ([i915#4083])
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-dg2-9/igt@gem_mmap@basic.html

  * igt@gem_mmap_gtt@basic:
    - bat-dg2-9:          NOTRUN -> [SKIP][3] ([i915#4077]) +2 other tests skip
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-dg2-9/igt@gem_mmap_gtt@basic.html

  * igt@gem_render_tiled_blits@basic:
    - bat-dg2-9:          NOTRUN -> [SKIP][4] ([i915#4079]) +1 other test skip
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-dg2-9/igt@gem_render_tiled_blits@basic.html

  * igt@i915_pm_rps@basic-api:
    - bat-dg2-9:          NOTRUN -> [SKIP][5] ([i915#6621])
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-dg2-9/igt@i915_pm_rps@basic-api.html

  * igt@i915_selftest@live@gt_heartbeat:
    - fi-apl-guc:         [PASS][6] -> [DMESG-FAIL][7] ([i915#5334])
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/fi-apl-guc/igt@i915_selftest@live@gt_heartbeat.html
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/fi-apl-guc/igt@i915_selftest@live@gt_heartbeat.html

  * igt@i915_suspend@basic-s2idle-without-i915:
    - fi-apl-guc:         [PASS][8] -> [INCOMPLETE][9] ([i915#1982] / [i915#4528])
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/fi-apl-guc/igt@i915_suspend@basic-s2idle-without-i915.html
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/fi-apl-guc/igt@i915_suspend@basic-s2idle-without-i915.html

  * igt@kms_addfb_basic@addfb25-y-tiled-small-legacy:
    - bat-dg2-9:          NOTRUN -> [SKIP][10] ([i915#5190])
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-dg2-9/igt@kms_addfb_basic@addfb25-y-tiled-small-legacy.html

  * igt@kms_addfb_basic@basic-y-tiled-legacy:
    - bat-dg2-9:          NOTRUN -> [SKIP][11] ([i915#4215] / [i915#5190])
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-dg2-9/igt@kms_addfb_basic@basic-y-tiled-legacy.html

  * igt@kms_addfb_basic@framebuffer-vs-set-tiling:
    - bat-dg2-9:          NOTRUN -> [SKIP][12] ([i915#4212]) +6 other tests skip
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-dg2-9/igt@kms_addfb_basic@framebuffer-vs-set-tiling.html

  * igt@kms_addfb_basic@tile-pitch-mismatch:
    - bat-dg2-9:          NOTRUN -> [SKIP][13] ([i915#4212] / [i915#5608])
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-dg2-9/igt@kms_addfb_basic@tile-pitch-mismatch.html

  * igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
    - bat-dg2-9:          NOTRUN -> [SKIP][14] ([i915#4103] / [i915#4213] / [i915#5608]) +1 other test skip
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-dg2-9/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html

  * igt@kms_flip@basic-flip-vs-wf_vblank@b-dp5:
    - bat-adlp-11:        [PASS][15] -> [DMESG-WARN][16] ([i915#6868])
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/bat-adlp-11/igt@kms_flip@basic-flip-vs-wf_vblank@b-dp5.html
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-adlp-11/igt@kms_flip@basic-flip-vs-wf_vblank@b-dp5.html

  * igt@kms_flip@basic-flip-vs-wf_vblank@d-dp5:
    - bat-adlp-11:        [PASS][17] -> [FAIL][18] ([i915#6121]) +1 other test fail
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/bat-adlp-11/igt@kms_flip@basic-flip-vs-wf_vblank@d-dp5.html
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-adlp-11/igt@kms_flip@basic-flip-vs-wf_vblank@d-dp5.html

  * igt@kms_flip@basic-plain-flip@d-dp6:
    - bat-adlp-11:        NOTRUN -> [DMESG-WARN][19] ([i915#6868])
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-adlp-11/igt@kms_flip@basic-plain-flip@d-dp6.html

  * igt@kms_force_connector_basic@force-load-detect:
    - bat-dg2-9:          NOTRUN -> [SKIP][20] ([fdo#109285])
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-dg2-9/igt@kms_force_connector_basic@force-load-detect.html

  * igt@kms_force_connector_basic@prune-stale-modes:
    - bat-dg2-9:          NOTRUN -> [SKIP][21] ([i915#5274])
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-dg2-9/igt@kms_force_connector_basic@prune-stale-modes.html

  * igt@kms_pipe_crc_basic@nonblocking-crc-frame-sequence:
    - bat-adlp-9:         NOTRUN -> [SKIP][22] ([i915#1845] / [i915#3546]) +2 other tests skip
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-adlp-9/igt@kms_pipe_crc_basic@nonblocking-crc-frame-sequence.html

  * igt@kms_pipe_crc_basic@read-crc-frame-sequence@pipe-d-dp-5:
    - bat-adlp-11:        [PASS][23] -> [DMESG-WARN][24] ([i915#4309] / [i915#7507])
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/bat-adlp-11/igt@kms_pipe_crc_basic@read-crc-frame-sequence@pipe-d-dp-5.html
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-adlp-11/igt@kms_pipe_crc_basic@read-crc-frame-sequence@pipe-d-dp-5.html

  * igt@kms_psr@sprite_plane_onoff:
    - bat-dg2-9:          NOTRUN -> [SKIP][25] ([i915#1072]) +3 other tests skip
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-dg2-9/igt@kms_psr@sprite_plane_onoff.html

  * igt@kms_setmode@basic-clone-single-crtc:
    - bat-dg2-9:          NOTRUN -> [SKIP][26] ([i915#3555] / [i915#4098])
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-dg2-9/igt@kms_setmode@basic-clone-single-crtc.html

  * igt@prime_vgem@basic-fence-flip:
    - bat-dg2-9:          NOTRUN -> [SKIP][27] ([i915#3708])
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-dg2-9/igt@prime_vgem@basic-fence-flip.html

  * igt@prime_vgem@basic-fence-mmap:
    - bat-dg2-9:          NOTRUN -> [SKIP][28] ([i915#3708] / [i915#4077]) +1 other test skip
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-dg2-9/igt@prime_vgem@basic-fence-mmap.html

  * igt@prime_vgem@basic-write:
    - bat-dg2-9:          NOTRUN -> [SKIP][29] ([i915#3291] / [i915#3708]) +2 other tests skip
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-dg2-9/igt@prime_vgem@basic-write.html

  
#### Possible fixes ####

  * igt@i915_selftest@live@hangcheck:
    - bat-dg2-11:         [DMESG-FAIL][30] -> [PASS][31]
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/bat-dg2-11/igt@i915_selftest@live@hangcheck.html
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/bat-dg2-11/igt@i915_selftest@live@hangcheck.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285
  [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
  [i915#1845]: https://gitlab.freedesktop.org/drm/intel/issues/1845
  [i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982
  [i915#3291]: https://gitlab.freedesktop.org/drm/intel/issues/3291
  [i915#3546]: https://gitlab.freedesktop.org/drm/intel/issues/3546
  [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
  [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
  [i915#4077]: https://gitlab.freedesktop.org/drm/intel/issues/4077
  [i915#4079]: https://gitlab.freedesktop.org/drm/intel/issues/4079
  [i915#4083]: https://gitlab.freedesktop.org/drm/intel/issues/4083
  [i915#4098]: https://gitlab.freedesktop.org/drm/intel/issues/4098
  [i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103
  [i915#4212]: https://gitlab.freedesktop.org/drm/intel/issues/4212
  [i915#4213]: https://gitlab.freedesktop.org/drm/intel/issues/4213
  [i915#4215]: https://gitlab.freedesktop.org/drm/intel/issues/4215
  [i915#4309]: https://gitlab.freedesktop.org/drm/intel/issues/4309
  [i915#4528]: https://gitlab.freedesktop.org/drm/intel/issues/4528
  [i915#5190]: https://gitlab.freedesktop.org/drm/intel/issues/5190
  [i915#5274]: https://gitlab.freedesktop.org/drm/intel/issues/5274
  [i915#5334]: https://gitlab.freedesktop.org/drm/intel/issues/5334
  [i915#5354]: https://gitlab.freedesktop.org/drm/intel/issues/5354
  [i915#5608]: https://gitlab.freedesktop.org/drm/intel/issues/5608
  [i915#6121]: https://gitlab.freedesktop.org/drm/intel/issues/6121
  [i915#6621]: https://gitlab.freedesktop.org/drm/intel/issues/6621
  [i915#6868]: https://gitlab.freedesktop.org/drm/intel/issues/6868
  [i915#7359]: https://gitlab.freedesktop.org/drm/intel/issues/7359
  [i915#7507]: https://gitlab.freedesktop.org/drm/intel/issues/7507
  [i915#8293]: https://gitlab.freedesktop.org/drm/intel/issues/8293
  [i915#8981]: https://gitlab.freedesktop.org/drm/intel/issues/8981


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

  * Linux: CI_DRM_13859 -> Patchwork_126225v1

  CI-20190529: 20190529
  CI_DRM_13859: 9155ae0ae05f320d84eaf2c4e81413bf937a5f3c @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_7581: 7581
  Patchwork_126225v1: 9155ae0ae05f320d84eaf2c4e81413bf937a5f3c @ git://anongit.freedesktop.org/gfx-ci/linux


### Linux commits

1a195c1c5286 drm/i915/gsc: Assign a uabi class number to the GSC CS

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/index.html

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

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

* [Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/gsc: Assign a uabi class number to the GSC CS
  2023-11-09 23:53 [Intel-gfx] [PATCH] drm/i915/gsc: Assign a uabi class number to the GSC CS Daniele Ceraolo Spurio
  2023-11-10  4:07 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for " Patchwork
  2023-11-10  4:20 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
@ 2023-11-10 10:25 ` Patchwork
  2023-11-10 12:00 ` [Intel-gfx] [PATCH] " Tvrtko Ursulin
  3 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2023-11-10 10:25 UTC (permalink / raw)
  To: Daniele Ceraolo Spurio; +Cc: intel-gfx

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

== Series Details ==

Series: drm/i915/gsc: Assign a uabi class number to the GSC CS
URL   : https://patchwork.freedesktop.org/series/126225/
State : failure

== Summary ==

CI Bug Log - changes from CI_DRM_13859_full -> Patchwork_126225v1_full
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with Patchwork_126225v1_full absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in Patchwork_126225v1_full, please notify your bug team (lgci.bug.filing@intel.com) to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/index.html

Participating hosts (9 -> 11)
------------------------------

  Additional (2): shard-mtlp0 shard-rkl0 

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

  Here are the unknown changes that may have been introduced in Patchwork_126225v1_full:

### IGT changes ###

#### Possible regressions ####

  * igt@perf@oa-formats@0-rcs0:
    - shard-apl:          NOTRUN -> [FAIL][1]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-apl2/igt@perf@oa-formats@0-rcs0.html

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

  Here are the changes found in Patchwork_126225v1_full that come from known issues:

### CI changes ###

#### Issues hit ####

  * boot:
    - shard-glk:          ([PASS][2], [PASS][3], [PASS][4], [PASS][5], [PASS][6], [PASS][7], [PASS][8], [PASS][9], [PASS][10], [PASS][11], [PASS][12], [PASS][13], [PASS][14], [PASS][15], [PASS][16], [PASS][17], [PASS][18], [PASS][19], [PASS][20], [PASS][21], [PASS][22], [PASS][23], [PASS][24], [PASS][25], [PASS][26]) -> ([PASS][27], [PASS][28], [PASS][29], [PASS][30], [PASS][31], [PASS][32], [PASS][33], [PASS][34], [PASS][35], [FAIL][36], [PASS][37], [PASS][38], [PASS][39], [PASS][40], [PASS][41], [PASS][42], [PASS][43], [PASS][44], [PASS][45], [PASS][46], [PASS][47], [PASS][48], [PASS][49], [PASS][50], [PASS][51]) ([i915#8293])
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk5/boot.html
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk5/boot.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk4/boot.html
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk4/boot.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk4/boot.html
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk3/boot.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk3/boot.html
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk3/boot.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk2/boot.html
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk2/boot.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk2/boot.html
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk1/boot.html
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk1/boot.html
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk1/boot.html
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk1/boot.html
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk9/boot.html
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk9/boot.html
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk9/boot.html
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk8/boot.html
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk8/boot.html
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk8/boot.html
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk6/boot.html
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk6/boot.html
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk6/boot.html
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk5/boot.html
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk6/boot.html
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk8/boot.html
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk8/boot.html
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk8/boot.html
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk9/boot.html
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk9/boot.html
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk9/boot.html
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk9/boot.html
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk1/boot.html
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk1/boot.html
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk1/boot.html
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk2/boot.html
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk2/boot.html
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk2/boot.html
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk3/boot.html
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk3/boot.html
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk3/boot.html
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk4/boot.html
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk4/boot.html
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk4/boot.html
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk5/boot.html
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk5/boot.html
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk5/boot.html
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk6/boot.html
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk6/boot.html

  

### IGT changes ###

#### Issues hit ####

  * igt@device_reset@unbind-reset-rebind:
    - shard-dg2:          NOTRUN -> [INCOMPLETE][52] ([i915#5507])
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@device_reset@unbind-reset-rebind.html

  * igt@drm_fdinfo@busy-hang@bcs0:
    - shard-dg2:          NOTRUN -> [SKIP][53] ([i915#8414]) +20 other tests skip
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@drm_fdinfo@busy-hang@bcs0.html

  * igt@drm_fdinfo@virtual-idle:
    - shard-rkl:          [PASS][54] -> [FAIL][55] ([i915#7742])
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@drm_fdinfo@virtual-idle.html
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@drm_fdinfo@virtual-idle.html

  * igt@drm_read@short-buffer-nonblock:
    - shard-rkl:          [PASS][56] -> [SKIP][57] ([i915#4098])
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@drm_read@short-buffer-nonblock.html
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@drm_read@short-buffer-nonblock.html

  * igt@gem_basic@multigpu-create-close:
    - shard-dg2:          NOTRUN -> [SKIP][58] ([i915#7697])
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@gem_basic@multigpu-create-close.html

  * igt@gem_busy@semaphore:
    - shard-dg2:          NOTRUN -> [SKIP][59] ([i915#3936])
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@gem_busy@semaphore.html

  * igt@gem_create@create-ext-cpu-access-sanity-check:
    - shard-mtlp:         NOTRUN -> [SKIP][60] ([i915#6335])
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@gem_create@create-ext-cpu-access-sanity-check.html

  * igt@gem_ctx_persistence@heartbeat-hang:
    - shard-dg2:          NOTRUN -> [SKIP][61] ([i915#8555])
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@gem_ctx_persistence@heartbeat-hang.html

  * igt@gem_eio@kms:
    - shard-dg2:          [PASS][62] -> [FAIL][63] ([i915#5784])
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-dg2-6/igt@gem_eio@kms.html
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@gem_eio@kms.html

  * igt@gem_exec_balancer@bonded-true-hang:
    - shard-dg2:          NOTRUN -> [SKIP][64] ([i915#4812]) +2 other tests skip
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@gem_exec_balancer@bonded-true-hang.html

  * igt@gem_exec_balancer@parallel-contexts:
    - shard-rkl:          NOTRUN -> [SKIP][65] ([i915#4525])
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@gem_exec_balancer@parallel-contexts.html

  * igt@gem_exec_capture@many-4k-incremental:
    - shard-apl:          NOTRUN -> [FAIL][66] ([i915#9606])
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-apl2/igt@gem_exec_capture@many-4k-incremental.html

  * igt@gem_exec_fair@basic-flow:
    - shard-mtlp:         NOTRUN -> [SKIP][67] ([i915#4473] / [i915#4771])
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@gem_exec_fair@basic-flow.html

  * igt@gem_exec_fair@basic-pace-share@rcs0:
    - shard-tglu:         [PASS][68] -> [FAIL][69] ([i915#2842])
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-tglu-2/igt@gem_exec_fair@basic-pace-share@rcs0.html
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-tglu-2/igt@gem_exec_fair@basic-pace-share@rcs0.html

  * igt@gem_exec_fair@basic-pace@rcs0:
    - shard-glk:          [PASS][70] -> [FAIL][71] ([i915#2842]) +2 other tests fail
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk4/igt@gem_exec_fair@basic-pace@rcs0.html
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk5/igt@gem_exec_fair@basic-pace@rcs0.html
    - shard-rkl:          [PASS][72] -> [FAIL][73] ([i915#2842])
   [72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-1/igt@gem_exec_fair@basic-pace@rcs0.html
   [73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-1/igt@gem_exec_fair@basic-pace@rcs0.html

  * igt@gem_exec_fair@basic-throttle:
    - shard-dg2:          NOTRUN -> [SKIP][74] ([i915#3539]) +1 other test skip
   [74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@gem_exec_fair@basic-throttle.html

  * igt@gem_exec_flush@basic-batch-kernel-default-cmd:
    - shard-dg2:          NOTRUN -> [SKIP][75] ([i915#3539] / [i915#4852]) +2 other tests skip
   [75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@gem_exec_flush@basic-batch-kernel-default-cmd.html

  * igt@gem_exec_params@secure-non-master:
    - shard-dg2:          NOTRUN -> [SKIP][76] ([fdo#112283]) +1 other test skip
   [76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@gem_exec_params@secure-non-master.html

  * igt@gem_exec_reloc@basic-concurrent0:
    - shard-mtlp:         NOTRUN -> [SKIP][77] ([i915#3281])
   [77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@gem_exec_reloc@basic-concurrent0.html

  * igt@gem_exec_reloc@basic-concurrent16:
    - shard-rkl:          NOTRUN -> [SKIP][78] ([i915#3281]) +4 other tests skip
   [78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@gem_exec_reloc@basic-concurrent16.html

  * igt@gem_exec_reloc@basic-wc-cpu:
    - shard-rkl:          [PASS][79] -> [SKIP][80] ([i915#3281])
   [79]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@gem_exec_reloc@basic-wc-cpu.html
   [80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@gem_exec_reloc@basic-wc-cpu.html

  * igt@gem_exec_reloc@basic-write-read-active:
    - shard-dg2:          NOTRUN -> [SKIP][81] ([i915#3281]) +12 other tests skip
   [81]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@gem_exec_reloc@basic-write-read-active.html

  * igt@gem_exec_schedule@preempt-queue:
    - shard-mtlp:         NOTRUN -> [SKIP][82] ([i915#4537] / [i915#4812])
   [82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@gem_exec_schedule@preempt-queue.html

  * igt@gem_fenced_exec_thrash@no-spare-fences-busy:
    - shard-dg2:          NOTRUN -> [SKIP][83] ([i915#4860]) +2 other tests skip
   [83]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-2/igt@gem_fenced_exec_thrash@no-spare-fences-busy.html

  * igt@gem_lmem_swapping@basic:
    - shard-apl:          NOTRUN -> [SKIP][84] ([fdo#109271] / [i915#4613]) +2 other tests skip
   [84]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-apl2/igt@gem_lmem_swapping@basic.html

  * igt@gem_lmem_swapping@parallel-multi:
    - shard-mtlp:         NOTRUN -> [SKIP][85] ([i915#4613])
   [85]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@gem_lmem_swapping@parallel-multi.html

  * igt@gem_media_vme:
    - shard-mtlp:         NOTRUN -> [SKIP][86] ([i915#284])
   [86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@gem_media_vme.html

  * igt@gem_mmap_gtt@bad-object:
    - shard-dg2:          NOTRUN -> [SKIP][87] ([i915#4077]) +12 other tests skip
   [87]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@gem_mmap_gtt@bad-object.html

  * igt@gem_mmap_gtt@medium-copy-xy:
    - shard-mtlp:         NOTRUN -> [SKIP][88] ([i915#4077])
   [88]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@gem_mmap_gtt@medium-copy-xy.html

  * igt@gem_mmap_wc@close:
    - shard-dg2:          NOTRUN -> [SKIP][89] ([i915#4083]) +4 other tests skip
   [89]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@gem_mmap_wc@close.html

  * igt@gem_mmap_wc@read:
    - shard-mtlp:         NOTRUN -> [SKIP][90] ([i915#4083]) +1 other test skip
   [90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@gem_mmap_wc@read.html

  * igt@gem_partial_pwrite_pread@writes-after-reads:
    - shard-rkl:          NOTRUN -> [SKIP][91] ([i915#3282]) +4 other tests skip
   [91]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@gem_partial_pwrite_pread@writes-after-reads.html

  * igt@gem_pread@display:
    - shard-rkl:          [PASS][92] -> [SKIP][93] ([i915#3282]) +1 other test skip
   [92]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@gem_pread@display.html
   [93]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@gem_pread@display.html

  * igt@gem_pread@snoop:
    - shard-dg2:          NOTRUN -> [SKIP][94] ([i915#3282]) +7 other tests skip
   [94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@gem_pread@snoop.html

  * igt@gem_pxp@verify-pxp-key-change-after-suspend-resume:
    - shard-dg2:          NOTRUN -> [SKIP][95] ([i915#4270]) +3 other tests skip
   [95]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@gem_pxp@verify-pxp-key-change-after-suspend-resume.html

  * igt@gem_render_copy@yf-tiled-ccs-to-yf-tiled-ccs:
    - shard-mtlp:         NOTRUN -> [SKIP][96] ([i915#8428]) +1 other test skip
   [96]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@gem_render_copy@yf-tiled-ccs-to-yf-tiled-ccs.html

  * igt@gem_set_tiling_vs_blt@tiled-to-untiled:
    - shard-dg2:          NOTRUN -> [SKIP][97] ([i915#4079]) +2 other tests skip
   [97]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-2/igt@gem_set_tiling_vs_blt@tiled-to-untiled.html
    - shard-rkl:          NOTRUN -> [SKIP][98] ([i915#8411])
   [98]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@gem_set_tiling_vs_blt@tiled-to-untiled.html

  * igt@gem_softpin@evict-snoop:
    - shard-rkl:          NOTRUN -> [SKIP][99] ([fdo#109312])
   [99]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@gem_softpin@evict-snoop.html

  * igt@gem_softpin@evict-snoop-interruptible:
    - shard-dg2:          NOTRUN -> [SKIP][100] ([i915#4885])
   [100]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-11/igt@gem_softpin@evict-snoop-interruptible.html

  * igt@gem_spin_batch@spin-all-new:
    - shard-dg2:          NOTRUN -> [FAIL][101] ([i915#5889])
   [101]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@gem_spin_batch@spin-all-new.html

  * igt@gem_userptr_blits@coherency-sync:
    - shard-rkl:          NOTRUN -> [SKIP][102] ([fdo#110542])
   [102]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@gem_userptr_blits@coherency-sync.html

  * igt@gem_userptr_blits@create-destroy-unsync:
    - shard-dg2:          NOTRUN -> [SKIP][103] ([i915#3297]) +3 other tests skip
   [103]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-11/igt@gem_userptr_blits@create-destroy-unsync.html

  * igt@gem_userptr_blits@map-fixed-invalidate-busy:
    - shard-dg2:          NOTRUN -> [SKIP][104] ([i915#3297] / [i915#4880]) +2 other tests skip
   [104]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-6/igt@gem_userptr_blits@map-fixed-invalidate-busy.html

  * igt@gem_userptr_blits@unsync-unmap:
    - shard-rkl:          NOTRUN -> [SKIP][105] ([i915#3297])
   [105]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@gem_userptr_blits@unsync-unmap.html

  * igt@gen7_exec_parse@bitmasks:
    - shard-dg2:          NOTRUN -> [SKIP][106] ([fdo#109289]) +4 other tests skip
   [106]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-11/igt@gen7_exec_parse@bitmasks.html

  * igt@gen9_exec_parse@bb-chained:
    - shard-mtlp:         NOTRUN -> [SKIP][107] ([i915#2856])
   [107]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@gen9_exec_parse@bb-chained.html

  * igt@gen9_exec_parse@bb-start-param:
    - shard-dg2:          NOTRUN -> [SKIP][108] ([i915#2856]) +2 other tests skip
   [108]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-2/igt@gen9_exec_parse@bb-start-param.html
    - shard-rkl:          NOTRUN -> [SKIP][109] ([i915#2527]) +1 other test skip
   [109]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@gen9_exec_parse@bb-start-param.html

  * igt@gen9_exec_parse@cmd-crossing-page:
    - shard-rkl:          [PASS][110] -> [SKIP][111] ([i915#2527])
   [110]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@gen9_exec_parse@cmd-crossing-page.html
   [111]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@gen9_exec_parse@cmd-crossing-page.html

  * igt@i915_hangman@engine-engine-error@bcs0:
    - shard-rkl:          [PASS][112] -> [SKIP][113] ([i915#9588])
   [112]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@i915_hangman@engine-engine-error@bcs0.html
   [113]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@i915_hangman@engine-engine-error@bcs0.html

  * igt@i915_module_load@reload-with-fault-injection:
    - shard-dg2:          NOTRUN -> [DMESG-WARN][114] ([i915#9559])
   [114]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@i915_module_load@reload-with-fault-injection.html

  * igt@i915_pipe_stress@stress-xrgb8888-untiled:
    - shard-apl:          NOTRUN -> [FAIL][115] ([i915#7036])
   [115]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-apl2/igt@i915_pipe_stress@stress-xrgb8888-untiled.html

  * igt@i915_pm_freq_api@freq-reset:
    - shard-rkl:          NOTRUN -> [SKIP][116] ([i915#8399])
   [116]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@i915_pm_freq_api@freq-reset.html

  * igt@i915_pm_rc6_residency@media-rc6-accuracy:
    - shard-rkl:          NOTRUN -> [SKIP][117] ([fdo#109289]) +3 other tests skip
   [117]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@i915_pm_rc6_residency@media-rc6-accuracy.html

  * igt@i915_pm_rc6_residency@rc6-idle@vecs0:
    - shard-dg1:          [PASS][118] -> [FAIL][119] ([i915#3591])
   [118]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-dg1-13/igt@i915_pm_rc6_residency@rc6-idle@vecs0.html
   [119]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg1-15/igt@i915_pm_rc6_residency@rc6-idle@vecs0.html

  * igt@i915_pm_rpm@gem-execbuf-stress-pc8:
    - shard-dg2:          NOTRUN -> [SKIP][120] ([fdo#109293] / [fdo#109506])
   [120]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@i915_pm_rpm@gem-execbuf-stress-pc8.html

  * igt@i915_pm_rps@thresholds-park@gt0:
    - shard-mtlp:         NOTRUN -> [SKIP][121] ([i915#8925])
   [121]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@i915_pm_rps@thresholds-park@gt0.html

  * igt@i915_pm_rps@thresholds-park@gt1:
    - shard-mtlp:         NOTRUN -> [SKIP][122] ([i915#3555] / [i915#8925])
   [122]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@i915_pm_rps@thresholds-park@gt1.html

  * igt@i915_pm_sseu@full-enable:
    - shard-dg2:          NOTRUN -> [SKIP][123] ([i915#4387])
   [123]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@i915_pm_sseu@full-enable.html

  * igt@i915_query@query-topology-coherent-slice-mask:
    - shard-dg2:          NOTRUN -> [SKIP][124] ([i915#6188])
   [124]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-11/igt@i915_query@query-topology-coherent-slice-mask.html

  * igt@i915_selftest@live@gt_heartbeat:
    - shard-apl:          [PASS][125] -> [DMESG-FAIL][126] ([i915#5334])
   [125]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-apl2/igt@i915_selftest@live@gt_heartbeat.html
   [126]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-apl3/igt@i915_selftest@live@gt_heartbeat.html

  * igt@kms_addfb_basic@addfb25-framebuffer-vs-set-tiling:
    - shard-mtlp:         NOTRUN -> [SKIP][127] ([i915#4212])
   [127]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@kms_addfb_basic@addfb25-framebuffer-vs-set-tiling.html

  * igt@kms_addfb_basic@basic-y-tiled-legacy:
    - shard-dg2:          NOTRUN -> [SKIP][128] ([i915#4215] / [i915#5190])
   [128]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-2/igt@kms_addfb_basic@basic-y-tiled-legacy.html

  * igt@kms_async_flips@crc@pipe-a-hdmi-a-3:
    - shard-dg2:          NOTRUN -> [FAIL][129] ([i915#8247]) +3 other tests fail
   [129]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-6/igt@kms_async_flips@crc@pipe-a-hdmi-a-3.html

  * igt@kms_async_flips@crc@pipe-d-hdmi-a-4:
    - shard-dg1:          NOTRUN -> [FAIL][130] ([i915#8247]) +3 other tests fail
   [130]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg1-16/igt@kms_async_flips@crc@pipe-d-hdmi-a-4.html

  * igt@kms_atomic@plane-primary-overlay-mutable-zpos:
    - shard-dg2:          NOTRUN -> [SKIP][131] ([i915#9531])
   [131]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_atomic@plane-primary-overlay-mutable-zpos.html

  * igt@kms_atomic_transition@plane-all-modeset-transition-internal-panels:
    - shard-dg2:          NOTRUN -> [SKIP][132] ([i915#1769] / [i915#3555])
   [132]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_atomic_transition@plane-all-modeset-transition-internal-panels.html

  * igt@kms_big_fb@4-tiled-32bpp-rotate-270:
    - shard-rkl:          NOTRUN -> [SKIP][133] ([i915#5286]) +2 other tests skip
   [133]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_big_fb@4-tiled-32bpp-rotate-270.html

  * igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-0:
    - shard-mtlp:         [PASS][134] -> [FAIL][135] ([i915#5138])
   [134]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-mtlp-2/igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-0.html
   [135]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-0.html

  * igt@kms_big_fb@linear-64bpp-rotate-270:
    - shard-rkl:          NOTRUN -> [SKIP][136] ([fdo#111614] / [i915#3638])
   [136]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_big_fb@linear-64bpp-rotate-270.html

  * igt@kms_big_fb@x-tiled-32bpp-rotate-270:
    - shard-dg2:          NOTRUN -> [SKIP][137] ([fdo#111614]) +5 other tests skip
   [137]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-11/igt@kms_big_fb@x-tiled-32bpp-rotate-270.html

  * igt@kms_big_fb@y-tiled-64bpp-rotate-0:
    - shard-dg2:          NOTRUN -> [SKIP][138] ([i915#5190]) +17 other tests skip
   [138]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-11/igt@kms_big_fb@y-tiled-64bpp-rotate-0.html

  * igt@kms_big_fb@yf-tiled-64bpp-rotate-180:
    - shard-mtlp:         NOTRUN -> [SKIP][139] ([fdo#111615]) +1 other test skip
   [139]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@kms_big_fb@yf-tiled-64bpp-rotate-180.html

  * igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-180-hflip-async-flip:
    - shard-dg2:          NOTRUN -> [SKIP][140] ([i915#4538] / [i915#5190]) +5 other tests skip
   [140]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-2/igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-180-hflip-async-flip.html
    - shard-rkl:          NOTRUN -> [SKIP][141] ([fdo#110723]) +2 other tests skip
   [141]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-180-hflip-async-flip.html

  * igt@kms_cdclk@mode-transition@pipe-d-hdmi-a-3:
    - shard-dg2:          NOTRUN -> [SKIP][142] ([i915#4087] / [i915#7213]) +3 other tests skip
   [142]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-6/igt@kms_cdclk@mode-transition@pipe-d-hdmi-a-3.html

  * igt@kms_cdclk@plane-scaling:
    - shard-rkl:          NOTRUN -> [SKIP][143] ([i915#3742])
   [143]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_cdclk@plane-scaling.html

  * igt@kms_cdclk@plane-scaling@pipe-a-hdmi-a-2:
    - shard-dg2:          NOTRUN -> [SKIP][144] ([i915#4087]) +3 other tests skip
   [144]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-2/igt@kms_cdclk@plane-scaling@pipe-a-hdmi-a-2.html

  * igt@kms_chamelium_color@gamma:
    - shard-dg2:          NOTRUN -> [SKIP][145] ([fdo#111827]) +1 other test skip
   [145]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_chamelium_color@gamma.html

  * igt@kms_chamelium_frames@hdmi-crc-multiple:
    - shard-mtlp:         NOTRUN -> [SKIP][146] ([i915#7828]) +1 other test skip
   [146]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@kms_chamelium_frames@hdmi-crc-multiple.html

  * igt@kms_chamelium_hpd@common-hpd-after-suspend:
    - shard-apl:          NOTRUN -> [SKIP][147] ([fdo#109271]) +135 other tests skip
   [147]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-apl2/igt@kms_chamelium_hpd@common-hpd-after-suspend.html

  * igt@kms_chamelium_hpd@hdmi-hpd-for-each-pipe:
    - shard-dg2:          NOTRUN -> [SKIP][148] ([i915#7828]) +9 other tests skip
   [148]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-2/igt@kms_chamelium_hpd@hdmi-hpd-for-each-pipe.html
    - shard-rkl:          NOTRUN -> [SKIP][149] ([i915#7828]) +4 other tests skip
   [149]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_chamelium_hpd@hdmi-hpd-for-each-pipe.html

  * igt@kms_content_protection@dp-mst-lic-type-0:
    - shard-dg2:          NOTRUN -> [SKIP][150] ([i915#3299])
   [150]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_content_protection@dp-mst-lic-type-0.html

  * igt@kms_content_protection@lic@pipe-a-dp-4:
    - shard-dg2:          NOTRUN -> [TIMEOUT][151] ([i915#7173]) +1 other test timeout
   [151]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-11/igt@kms_content_protection@lic@pipe-a-dp-4.html

  * igt@kms_content_protection@srm:
    - shard-dg2:          NOTRUN -> [SKIP][152] ([i915#7118]) +1 other test skip
   [152]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_content_protection@srm.html

  * igt@kms_content_protection@uevent@pipe-a-dp-1:
    - shard-apl:          NOTRUN -> [FAIL][153] ([i915#1339])
   [153]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-apl2/igt@kms_content_protection@uevent@pipe-a-dp-1.html

  * igt@kms_cursor_crc@cursor-onscreen-256x256:
    - shard-rkl:          NOTRUN -> [SKIP][154] ([i915#1845] / [i915#4098]) +3 other tests skip
   [154]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@kms_cursor_crc@cursor-onscreen-256x256.html

  * igt@kms_cursor_crc@cursor-onscreen-32x32:
    - shard-rkl:          NOTRUN -> [SKIP][155] ([i915#3555]) +5 other tests skip
   [155]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_cursor_crc@cursor-onscreen-32x32.html

  * igt@kms_cursor_crc@cursor-rapid-movement-512x170:
    - shard-dg2:          NOTRUN -> [SKIP][156] ([i915#3359]) +2 other tests skip
   [156]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_cursor_crc@cursor-rapid-movement-512x170.html

  * igt@kms_cursor_edge_walk@256x256-right-edge:
    - shard-rkl:          NOTRUN -> [SKIP][157] ([i915#4098]) +3 other tests skip
   [157]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@kms_cursor_edge_walk@256x256-right-edge.html

  * igt@kms_cursor_legacy@2x-flip-vs-cursor-atomic:
    - shard-apl:          NOTRUN -> [SKIP][158] ([fdo#109271] / [fdo#111767])
   [158]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-apl6/igt@kms_cursor_legacy@2x-flip-vs-cursor-atomic.html

  * igt@kms_cursor_legacy@2x-long-cursor-vs-flip-atomic:
    - shard-mtlp:         NOTRUN -> [SKIP][159] ([i915#3546]) +1 other test skip
   [159]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@kms_cursor_legacy@2x-long-cursor-vs-flip-atomic.html

  * igt@kms_cursor_legacy@cursora-vs-flipa-toggle:
    - shard-rkl:          [PASS][160] -> [SKIP][161] ([i915#1845] / [i915#4098]) +4 other tests skip
   [160]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@kms_cursor_legacy@cursora-vs-flipa-toggle.html
   [161]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@kms_cursor_legacy@cursora-vs-flipa-toggle.html

  * igt@kms_cursor_legacy@cursorb-vs-flipb-varying-size:
    - shard-dg2:          NOTRUN -> [SKIP][162] ([fdo#109274] / [i915#5354]) +3 other tests skip
   [162]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-11/igt@kms_cursor_legacy@cursorb-vs-flipb-varying-size.html

  * igt@kms_cursor_legacy@short-busy-flip-before-cursor-atomic-transitions-varying-size:
    - shard-dg2:          NOTRUN -> [SKIP][163] ([i915#4103] / [i915#4213]) +1 other test skip
   [163]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_cursor_legacy@short-busy-flip-before-cursor-atomic-transitions-varying-size.html

  * igt@kms_dirtyfb@dirtyfb-ioctl@fbc-hdmi-a-1:
    - shard-dg1:          NOTRUN -> [SKIP][164] ([i915#9227])
   [164]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg1-19/igt@kms_dirtyfb@dirtyfb-ioctl@fbc-hdmi-a-1.html

  * igt@kms_dirtyfb@dirtyfb-ioctl@psr-hdmi-a-1:
    - shard-dg1:          NOTRUN -> [SKIP][165] ([i915#9226] / [i915#9261]) +1 other test skip
   [165]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg1-19/igt@kms_dirtyfb@dirtyfb-ioctl@psr-hdmi-a-1.html

  * igt@kms_display_modes@mst-extended-mode-negative:
    - shard-dg2:          NOTRUN -> [SKIP][166] ([i915#8588])
   [166]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_display_modes@mst-extended-mode-negative.html

  * igt@kms_dither@fb-8bpc-vs-panel-8bpc:
    - shard-dg2:          NOTRUN -> [SKIP][167] ([i915#3555]) +3 other tests skip
   [167]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-6/igt@kms_dither@fb-8bpc-vs-panel-8bpc.html

  * igt@kms_dsc@dsc-basic:
    - shard-dg2:          NOTRUN -> [SKIP][168] ([i915#3555] / [i915#3840]) +1 other test skip
   [168]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-11/igt@kms_dsc@dsc-basic.html

  * igt@kms_fence_pin_leak:
    - shard-dg2:          NOTRUN -> [SKIP][169] ([i915#4881]) +1 other test skip
   [169]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_fence_pin_leak.html

  * igt@kms_flip@2x-flip-vs-blocking-wf-vblank:
    - shard-dg2:          NOTRUN -> [SKIP][170] ([fdo#109274] / [fdo#111767]) +1 other test skip
   [170]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-11/igt@kms_flip@2x-flip-vs-blocking-wf-vblank.html

  * igt@kms_flip@2x-flip-vs-panning-vs-hang:
    - shard-rkl:          NOTRUN -> [SKIP][171] ([fdo#111825]) +5 other tests skip
   [171]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_flip@2x-flip-vs-panning-vs-hang.html

  * igt@kms_flip@2x-flip-vs-suspend:
    - shard-mtlp:         NOTRUN -> [SKIP][172] ([i915#3637])
   [172]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@kms_flip@2x-flip-vs-suspend.html

  * igt@kms_flip@2x-modeset-vs-vblank-race:
    - shard-dg2:          NOTRUN -> [SKIP][173] ([fdo#109274]) +7 other tests skip
   [173]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_flip@2x-modeset-vs-vblank-race.html

  * igt@kms_flip@plain-flip-interruptible:
    - shard-rkl:          NOTRUN -> [SKIP][174] ([i915#3637] / [i915#4098]) +3 other tests skip
   [174]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@kms_flip@plain-flip-interruptible.html

  * igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-64bpp-ytile-upscaling@pipe-a-default-mode:
    - shard-mtlp:         NOTRUN -> [SKIP][175] ([i915#2672])
   [175]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-64bpp-ytile-upscaling@pipe-a-default-mode.html

  * igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-16bpp-yftile-upscaling@pipe-a-valid-mode:
    - shard-rkl:          NOTRUN -> [SKIP][176] ([i915#2672]) +1 other test skip
   [176]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-16bpp-yftile-upscaling@pipe-a-valid-mode.html

  * igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytile-upscaling@pipe-a-valid-mode:
    - shard-dg2:          NOTRUN -> [SKIP][177] ([i915#2672]) +4 other tests skip
   [177]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-11/igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytile-upscaling@pipe-a-valid-mode.html

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-fullscreen:
    - shard-rkl:          [PASS][178] -> [SKIP][179] ([i915#1849] / [i915#4098]) +1 other test skip
   [178]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-fullscreen.html
   [179]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-fullscreen.html

  * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-cur-indfb-draw-mmap-wc:
    - shard-dg2:          NOTRUN -> [SKIP][180] ([i915#8708]) +25 other tests skip
   [180]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-6/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-cur-indfb-draw-mmap-wc.html

  * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-shrfb-draw-mmap-gtt:
    - shard-rkl:          NOTRUN -> [SKIP][181] ([fdo#111825] / [i915#1825]) +18 other tests skip
   [181]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-shrfb-draw-mmap-gtt.html

  * igt@kms_frontbuffer_tracking@fbc-rgb565-draw-mmap-gtt:
    - shard-mtlp:         NOTRUN -> [SKIP][182] ([i915#8708])
   [182]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@kms_frontbuffer_tracking@fbc-rgb565-draw-mmap-gtt.html

  * igt@kms_frontbuffer_tracking@fbcpsr-suspend:
    - shard-rkl:          NOTRUN -> [SKIP][183] ([i915#3023]) +7 other tests skip
   [183]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_frontbuffer_tracking@fbcpsr-suspend.html

  * igt@kms_frontbuffer_tracking@psr-2p-primscrn-shrfb-plflip-blt:
    - shard-mtlp:         NOTRUN -> [SKIP][184] ([i915#1825]) +6 other tests skip
   [184]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@kms_frontbuffer_tracking@psr-2p-primscrn-shrfb-plflip-blt.html

  * igt@kms_frontbuffer_tracking@psr-2p-scndscrn-indfb-plflip-blt:
    - shard-dg2:          NOTRUN -> [SKIP][185] ([i915#5354]) +32 other tests skip
   [185]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-6/igt@kms_frontbuffer_tracking@psr-2p-scndscrn-indfb-plflip-blt.html

  * igt@kms_frontbuffer_tracking@psr-indfb-scaledprimary:
    - shard-dg2:          NOTRUN -> [SKIP][186] ([i915#3458]) +20 other tests skip
   [186]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_frontbuffer_tracking@psr-indfb-scaledprimary.html

  * igt@kms_hdr@bpc-switch-suspend:
    - shard-rkl:          NOTRUN -> [SKIP][187] ([i915#3555] / [i915#8228])
   [187]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-2/igt@kms_hdr@bpc-switch-suspend.html

  * igt@kms_hdr@static-toggle-suspend:
    - shard-dg2:          NOTRUN -> [SKIP][188] ([i915#3555] / [i915#8228]) +2 other tests skip
   [188]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-6/igt@kms_hdr@static-toggle-suspend.html

  * igt@kms_panel_fitting@legacy:
    - shard-dg2:          NOTRUN -> [SKIP][189] ([i915#6301])
   [189]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_panel_fitting@legacy.html

  * igt@kms_pipe_b_c_ivb@disable-pipe-b-enable-pipe-c:
    - shard-mtlp:         NOTRUN -> [SKIP][190] ([fdo#109289])
   [190]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@kms_pipe_b_c_ivb@disable-pipe-b-enable-pipe-c.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-dp-1:
    - shard-apl:          [PASS][191] -> [INCOMPLETE][192] ([i915#180] / [i915#9392])
   [191]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-apl7/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-dp-1.html
   [192]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-apl2/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-dp-1.html

  * igt@kms_plane_alpha_blend@alpha-opaque-fb@pipe-a-dp-1:
    - shard-apl:          NOTRUN -> [FAIL][193] ([i915#4573]) +1 other test fail
   [193]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-apl2/igt@kms_plane_alpha_blend@alpha-opaque-fb@pipe-a-dp-1.html

  * igt@kms_plane_multiple@tiling-y:
    - shard-dg2:          NOTRUN -> [SKIP][194] ([i915#8806])
   [194]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-2/igt@kms_plane_multiple@tiling-y.html

  * igt@kms_plane_scaling@intel-max-src-size@pipe-a-hdmi-a-3:
    - shard-dg1:          NOTRUN -> [FAIL][195] ([i915#8292])
   [195]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg1-13/igt@kms_plane_scaling@intel-max-src-size@pipe-a-hdmi-a-3.html

  * igt@kms_plane_scaling@invalid-parameters:
    - shard-rkl:          NOTRUN -> [SKIP][196] ([i915#8152])
   [196]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@kms_plane_scaling@invalid-parameters.html

  * igt@kms_plane_scaling@plane-scaler-with-clipping-clamping-rotation@pipe-a-hdmi-a-1:
    - shard-rkl:          NOTRUN -> [SKIP][197] ([i915#5176] / [i915#9423]) +1 other test skip
   [197]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-7/igt@kms_plane_scaling@plane-scaler-with-clipping-clamping-rotation@pipe-a-hdmi-a-1.html

  * igt@kms_plane_scaling@plane-scaler-with-clipping-clamping-rotation@pipe-a-hdmi-a-3:
    - shard-dg1:          NOTRUN -> [SKIP][198] ([i915#5176] / [i915#9423]) +3 other tests skip
   [198]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg1-13/igt@kms_plane_scaling@plane-scaler-with-clipping-clamping-rotation@pipe-a-hdmi-a-3.html

  * igt@kms_plane_scaling@planes-downscale-factor-0-25-unity-scaling@pipe-c-hdmi-a-1:
    - shard-dg1:          NOTRUN -> [SKIP][199] ([i915#5235]) +15 other tests skip
   [199]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg1-19/igt@kms_plane_scaling@planes-downscale-factor-0-25-unity-scaling@pipe-c-hdmi-a-1.html

  * igt@kms_plane_scaling@planes-downscale-factor-0-25-upscale-factor-0-25@pipe-b-hdmi-a-2:
    - shard-rkl:          NOTRUN -> [SKIP][200] ([i915#5235]) +3 other tests skip
   [200]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_plane_scaling@planes-downscale-factor-0-25-upscale-factor-0-25@pipe-b-hdmi-a-2.html

  * igt@kms_plane_scaling@planes-downscale-factor-0-75-upscale-factor-0-25:
    - shard-rkl:          NOTRUN -> [SKIP][201] ([i915#6953] / [i915#8152])
   [201]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@kms_plane_scaling@planes-downscale-factor-0-75-upscale-factor-0-25.html

  * igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-25@pipe-a-hdmi-a-3:
    - shard-dg2:          NOTRUN -> [SKIP][202] ([i915#5235]) +15 other tests skip
   [202]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-25@pipe-a-hdmi-a-3.html

  * igt@kms_prime@basic-crc-vgem:
    - shard-dg2:          NOTRUN -> [SKIP][203] ([i915#6524] / [i915#6805])
   [203]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_prime@basic-crc-vgem.html

  * igt@kms_prime@basic-modeset-hybrid:
    - shard-rkl:          NOTRUN -> [SKIP][204] ([i915#6524])
   [204]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_prime@basic-modeset-hybrid.html

  * igt@kms_psr2_sf@cursor-plane-move-continuous-exceed-sf:
    - shard-dg2:          NOTRUN -> [SKIP][205] ([i915#658]) +3 other tests skip
   [205]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_psr2_sf@cursor-plane-move-continuous-exceed-sf.html

  * igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area:
    - shard-apl:          NOTRUN -> [SKIP][206] ([fdo#109271] / [i915#658]) +2 other tests skip
   [206]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-apl2/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area.html

  * igt@kms_psr2_su@page_flip-xrgb8888:
    - shard-rkl:          NOTRUN -> [SKIP][207] ([fdo#111068] / [i915#658])
   [207]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_psr2_su@page_flip-xrgb8888.html

  * igt@kms_psr@primary_render:
    - shard-rkl:          NOTRUN -> [SKIP][208] ([i915#1072]) +1 other test skip
   [208]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_psr@primary_render.html

  * igt@kms_psr@psr2_sprite_mmap_gtt:
    - shard-dg2:          NOTRUN -> [SKIP][209] ([i915#1072]) +13 other tests skip
   [209]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-11/igt@kms_psr@psr2_sprite_mmap_gtt.html

  * igt@kms_psr_stress_test@invalidate-primary-flip-overlay:
    - shard-dg2:          NOTRUN -> [SKIP][210] ([i915#5461] / [i915#658])
   [210]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-11/igt@kms_psr_stress_test@invalidate-primary-flip-overlay.html

  * igt@kms_rotation_crc@primary-rotation-90:
    - shard-dg2:          NOTRUN -> [SKIP][211] ([i915#4235])
   [211]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_rotation_crc@primary-rotation-90.html

  * igt@kms_setmode@clone-exclusive-crtc:
    - shard-dg2:          NOTRUN -> [SKIP][212] ([i915#3555] / [i915#4098]) +1 other test skip
   [212]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_setmode@clone-exclusive-crtc.html

  * igt@kms_setmode@invalid-clone-single-crtc:
    - shard-mtlp:         NOTRUN -> [SKIP][213] ([i915#3555] / [i915#8809])
   [213]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@kms_setmode@invalid-clone-single-crtc.html

  * igt@kms_setmode@invalid-clone-single-crtc-stealing:
    - shard-rkl:          NOTRUN -> [SKIP][214] ([i915#3555] / [i915#4098])
   [214]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_setmode@invalid-clone-single-crtc-stealing.html

  * igt@kms_tiled_display@basic-test-pattern:
    - shard-dg2:          NOTRUN -> [SKIP][215] ([i915#8623])
   [215]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_tiled_display@basic-test-pattern.html

  * igt@kms_universal_plane@cursor-fb-leak@pipe-b-hdmi-a-2:
    - shard-rkl:          [PASS][216] -> [FAIL][217] ([i915#9196])
   [216]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-2/igt@kms_universal_plane@cursor-fb-leak@pipe-b-hdmi-a-2.html
   [217]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-1/igt@kms_universal_plane@cursor-fb-leak@pipe-b-hdmi-a-2.html

  * igt@kms_universal_plane@cursor-fb-leak@pipe-c-hdmi-a-1:
    - shard-tglu:         [PASS][218] -> [FAIL][219] ([i915#9196]) +1 other test fail
   [218]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-tglu-10/igt@kms_universal_plane@cursor-fb-leak@pipe-c-hdmi-a-1.html
   [219]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-tglu-10/igt@kms_universal_plane@cursor-fb-leak@pipe-c-hdmi-a-1.html

  * igt@kms_vblank@ts-continuation-modeset-rpm@pipe-a-hdmi-a-1:
    - shard-snb:          NOTRUN -> [SKIP][220] ([fdo#109271]) +3 other tests skip
   [220]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-snb1/igt@kms_vblank@ts-continuation-modeset-rpm@pipe-a-hdmi-a-1.html

  * igt@kms_writeback@writeback-fb-id:
    - shard-apl:          NOTRUN -> [SKIP][221] ([fdo#109271] / [i915#2437])
   [221]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-apl6/igt@kms_writeback@writeback-fb-id.html

  * igt@kms_writeback@writeback-invalid-parameters:
    - shard-dg2:          NOTRUN -> [SKIP][222] ([i915#2437])
   [222]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@kms_writeback@writeback-invalid-parameters.html

  * igt@perf_pmu@busy-double-start@bcs0:
    - shard-mtlp:         [PASS][223] -> [FAIL][224] ([i915#4349])
   [223]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-mtlp-5/igt@perf_pmu@busy-double-start@bcs0.html
   [224]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@perf_pmu@busy-double-start@bcs0.html

  * igt@perf_pmu@frequency@gt0:
    - shard-dg2:          NOTRUN -> [FAIL][225] ([i915#6806])
   [225]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-2/igt@perf_pmu@frequency@gt0.html

  * igt@perf_pmu@rc6@other-idle-gt0:
    - shard-dg2:          NOTRUN -> [SKIP][226] ([i915#8516])
   [226]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@perf_pmu@rc6@other-idle-gt0.html

  * igt@prime_vgem@basic-fence-read:
    - shard-dg2:          NOTRUN -> [SKIP][227] ([i915#3291] / [i915#3708]) +1 other test skip
   [227]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-11/igt@prime_vgem@basic-fence-read.html

  * igt@prime_vgem@basic-gtt:
    - shard-dg2:          NOTRUN -> [SKIP][228] ([i915#3708] / [i915#4077])
   [228]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-2/igt@prime_vgem@basic-gtt.html

  * igt@prime_vgem@fence-read-hang:
    - shard-dg2:          NOTRUN -> [SKIP][229] ([i915#3708])
   [229]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@prime_vgem@fence-read-hang.html

  * igt@syncobj_timeline@invalid-multi-wait-all-available-unsubmitted:
    - shard-dg2:          NOTRUN -> [FAIL][230] ([i915#9583])
   [230]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@syncobj_timeline@invalid-multi-wait-all-available-unsubmitted.html

  * igt@syncobj_timeline@invalid-multi-wait-all-available-unsubmitted-submitted:
    - shard-rkl:          NOTRUN -> [FAIL][231] ([i915#9583])
   [231]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@syncobj_timeline@invalid-multi-wait-all-available-unsubmitted-submitted.html

  * igt@tools_test@sysfs_l3_parity:
    - shard-dg2:          NOTRUN -> [SKIP][232] ([i915#4818])
   [232]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@tools_test@sysfs_l3_parity.html

  * igt@v3d/v3d_job_submission@array-job-submission:
    - shard-dg2:          NOTRUN -> [SKIP][233] ([i915#2575]) +11 other tests skip
   [233]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-1/igt@v3d/v3d_job_submission@array-job-submission.html

  * igt@v3d/v3d_submit_cl@simple-flush-cache:
    - shard-mtlp:         NOTRUN -> [SKIP][234] ([i915#2575]) +1 other test skip
   [234]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@v3d/v3d_submit_cl@simple-flush-cache.html

  * igt@v3d/v3d_submit_csd@bad-multisync-in-sync:
    - shard-rkl:          NOTRUN -> [SKIP][235] ([fdo#109315]) +3 other tests skip
   [235]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@v3d/v3d_submit_csd@bad-multisync-in-sync.html

  * igt@vc4/vc4_create_bo@create-bo-0:
    - shard-rkl:          NOTRUN -> [SKIP][236] ([i915#7711]) +1 other test skip
   [236]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@vc4/vc4_create_bo@create-bo-0.html

  * igt@vc4/vc4_label_bo@set-kernel-name:
    - shard-mtlp:         NOTRUN -> [SKIP][237] ([i915#7711]) +2 other tests skip
   [237]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@vc4/vc4_label_bo@set-kernel-name.html

  * igt@vc4/vc4_wait_seqno@bad-seqno-1ns:
    - shard-dg2:          NOTRUN -> [SKIP][238] ([i915#7711]) +9 other tests skip
   [238]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-2/igt@vc4/vc4_wait_seqno@bad-seqno-1ns.html

  
#### Possible fixes ####

  * igt@api_intel_bb@blit-reloc-purge-cache:
    - shard-rkl:          [SKIP][239] ([i915#8411]) -> [PASS][240]
   [239]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@api_intel_bb@blit-reloc-purge-cache.html
   [240]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@api_intel_bb@blit-reloc-purge-cache.html

  * igt@drm_fdinfo@most-busy-idle-check-all@rcs0:
    - shard-rkl:          [FAIL][241] ([i915#7742]) -> [PASS][242]
   [241]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-1/igt@drm_fdinfo@most-busy-idle-check-all@rcs0.html
   [242]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-7/igt@drm_fdinfo@most-busy-idle-check-all@rcs0.html

  * igt@drm_read@short-buffer-block:
    - shard-rkl:          [SKIP][243] ([i915#1845] / [i915#4098]) -> [PASS][244] +3 other tests pass
   [243]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@drm_read@short-buffer-block.html
   [244]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@drm_read@short-buffer-block.html

  * igt@gem_ccs@suspend-resume@tile4-compressed-compfmt0-smem-lmem0:
    - shard-dg2:          [INCOMPLETE][245] ([i915#7297]) -> [PASS][246]
   [245]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-dg2-2/igt@gem_ccs@suspend-resume@tile4-compressed-compfmt0-smem-lmem0.html
   [246]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-6/igt@gem_ccs@suspend-resume@tile4-compressed-compfmt0-smem-lmem0.html

  * igt@gem_ctx_persistence@engines-hang@bcs0:
    - shard-rkl:          [SKIP][247] ([i915#6252]) -> [PASS][248]
   [247]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@gem_ctx_persistence@engines-hang@bcs0.html
   [248]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@gem_ctx_persistence@engines-hang@bcs0.html

  * igt@gem_eio@hibernate:
    - shard-dg2:          [ABORT][249] ([i915#7975] / [i915#8213]) -> [PASS][250]
   [249]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-dg2-6/igt@gem_eio@hibernate.html
   [250]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-11/igt@gem_eio@hibernate.html

  * igt@gem_eio@kms:
    - shard-mtlp:         [ABORT][251] ([i915#7892] / [i915#9414]) -> [PASS][252]
   [251]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-mtlp-4/igt@gem_eio@kms.html
   [252]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@gem_eio@kms.html

  * igt@gem_exec_fair@basic-pace-share@rcs0:
    - shard-glk:          [FAIL][253] ([i915#2842]) -> [PASS][254]
   [253]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-glk8/igt@gem_exec_fair@basic-pace-share@rcs0.html
   [254]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-glk9/igt@gem_exec_fair@basic-pace-share@rcs0.html

  * igt@gem_exec_reloc@basic-write-read:
    - shard-rkl:          [SKIP][255] ([i915#3281]) -> [PASS][256] +2 other tests pass
   [255]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@gem_exec_reloc@basic-write-read.html
   [256]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@gem_exec_reloc@basic-write-read.html

  * igt@gem_exec_schedule@semaphore-power:
    - shard-rkl:          [SKIP][257] ([i915#7276]) -> [PASS][258]
   [257]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@gem_exec_schedule@semaphore-power.html
   [258]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@gem_exec_schedule@semaphore-power.html

  * igt@gem_lmem_swapping@smem-oom@lmem0:
    - shard-dg2:          [DMESG-WARN][259] ([i915#4936] / [i915#5493]) -> [PASS][260]
   [259]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-dg2-2/igt@gem_lmem_swapping@smem-oom@lmem0.html
   [260]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-11/igt@gem_lmem_swapping@smem-oom@lmem0.html
    - shard-dg1:          [TIMEOUT][261] ([i915#5493]) -> [PASS][262]
   [261]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-dg1-15/igt@gem_lmem_swapping@smem-oom@lmem0.html
   [262]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg1-17/igt@gem_lmem_swapping@smem-oom@lmem0.html

  * igt@gem_mmap_gtt@coherency:
    - shard-rkl:          [SKIP][263] ([fdo#111656]) -> [PASS][264]
   [263]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@gem_mmap_gtt@coherency.html
   [264]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@gem_mmap_gtt@coherency.html

  * igt@gem_readwrite@beyond-eob:
    - shard-rkl:          [SKIP][265] ([i915#3282]) -> [PASS][266]
   [265]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@gem_readwrite@beyond-eob.html
   [266]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@gem_readwrite@beyond-eob.html

  * igt@gem_userptr_blits@stress-purge:
    - shard-dg1:          [DMESG-WARN][267] ([i915#4423]) -> [PASS][268]
   [267]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-dg1-18/igt@gem_userptr_blits@stress-purge.html
   [268]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg1-17/igt@gem_userptr_blits@stress-purge.html

  * igt@gen9_exec_parse@allowed-all:
    - shard-apl:          [INCOMPLETE][269] ([i915#5566]) -> [PASS][270]
   [269]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-apl1/igt@gen9_exec_parse@allowed-all.html
   [270]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-apl7/igt@gen9_exec_parse@allowed-all.html

  * igt@i915_pm_rc6_residency@rc6-idle@vcs0:
    - shard-dg1:          [FAIL][271] ([i915#3591]) -> [PASS][272]
   [271]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-dg1-13/igt@i915_pm_rc6_residency@rc6-idle@vcs0.html
   [272]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg1-15/igt@i915_pm_rc6_residency@rc6-idle@vcs0.html

  * igt@i915_power@sanity:
    - shard-rkl:          [SKIP][273] ([i915#7984]) -> [PASS][274]
   [273]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@i915_power@sanity.html
   [274]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@i915_power@sanity.html

  * igt@kms_big_fb@4-tiled-64bpp-rotate-180:
    - shard-mtlp:         [FAIL][275] ([i915#5138]) -> [PASS][276]
   [275]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-mtlp-4/igt@kms_big_fb@4-tiled-64bpp-rotate-180.html
   [276]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-8/igt@kms_big_fb@4-tiled-64bpp-rotate-180.html

  * {igt@kms_ccs@pipe-a-crc-primary-rotation-180-y-tiled-gen12-rc-ccs}:
    - shard-rkl:          [SKIP][277] ([i915#4098]) -> [PASS][278]
   [277]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@kms_ccs@pipe-a-crc-primary-rotation-180-y-tiled-gen12-rc-ccs.html
   [278]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_ccs@pipe-a-crc-primary-rotation-180-y-tiled-gen12-rc-ccs.html

  * igt@kms_flip@wf_vblank-ts-check@c-edp1:
    - shard-mtlp:         [DMESG-WARN][279] ([i915#9157]) -> [PASS][280]
   [279]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-mtlp-2/igt@kms_flip@wf_vblank-ts-check@c-edp1.html
   [280]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-mtlp-5/igt@kms_flip@wf_vblank-ts-check@c-edp1.html

  * igt@kms_frontbuffer_tracking@fbc-rgb101010-draw-mmap-gtt:
    - shard-rkl:          [SKIP][281] ([i915#1849] / [i915#4098]) -> [PASS][282] +2 other tests pass
   [281]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@kms_frontbuffer_tracking@fbc-rgb101010-draw-mmap-gtt.html
   [282]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_frontbuffer_tracking@fbc-rgb101010-draw-mmap-gtt.html

  * igt@kms_plane_multiple@tiling-y@pipe-a-hdmi-a-2:
    - shard-rkl:          [INCOMPLETE][283] -> [PASS][284]
   [283]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-2/igt@kms_plane_multiple@tiling-y@pipe-a-hdmi-a-2.html
   [284]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_plane_multiple@tiling-y@pipe-a-hdmi-a-2.html

  * {igt@kms_pm_rpm@dpms-lpsp}:
    - shard-rkl:          [SKIP][285] ([i915#9519]) -> [PASS][286]
   [285]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-1/igt@kms_pm_rpm@dpms-lpsp.html
   [286]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-7/igt@kms_pm_rpm@dpms-lpsp.html

  * {igt@kms_pm_rpm@i2c}:
    - shard-rkl:          [SKIP][287] ([fdo#109308]) -> [PASS][288]
   [287]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@kms_pm_rpm@i2c.html
   [288]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_pm_rpm@i2c.html

  * igt@kms_rotation_crc@primary-y-tiled-reflect-x-180:
    - shard-rkl:          [INCOMPLETE][289] ([i915#8875] / [i915#9569]) -> [PASS][290]
   [289]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-2/igt@kms_rotation_crc@primary-y-tiled-reflect-x-180.html
   [290]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_rotation_crc@primary-y-tiled-reflect-x-180.html

  * igt@perf@mi-rpc:
    - shard-rkl:          [SKIP][291] ([i915#2434]) -> [PASS][292]
   [291]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@perf@mi-rpc.html
   [292]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@perf@mi-rpc.html

  * igt@prime_vgem@basic-fence-flip:
    - shard-rkl:          [SKIP][293] ([fdo#109295] / [i915#3708] / [i915#4098]) -> [PASS][294]
   [293]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@prime_vgem@basic-fence-flip.html
   [294]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@prime_vgem@basic-fence-flip.html

  
#### Warnings ####

  * igt@gem_ccs@block-multicopy-compressed:
    - shard-rkl:          [SKIP][295] ([i915#7957]) -> [SKIP][296] ([i915#9323])
   [295]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@gem_ccs@block-multicopy-compressed.html
   [296]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@gem_ccs@block-multicopy-compressed.html

  * igt@gem_ccs@ctrl-surf-copy:
    - shard-rkl:          [SKIP][297] ([i915#3555]) -> [SKIP][298] ([i915#7957])
   [297]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@gem_ccs@ctrl-surf-copy.html
   [298]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@gem_ccs@ctrl-surf-copy.html

  * igt@kms_atomic_transition@plane-all-modeset-transition-fencing-internal-panels:
    - shard-rkl:          [SKIP][299] ([i915#1769] / [i915#3555]) -> [SKIP][300] ([i915#1845] / [i915#4098])
   [299]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@kms_atomic_transition@plane-all-modeset-transition-fencing-internal-panels.html
   [300]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@kms_atomic_transition@plane-all-modeset-transition-fencing-internal-panels.html

  * igt@kms_big_fb@4-tiled-16bpp-rotate-270:
    - shard-rkl:          [SKIP][301] ([i915#1845] / [i915#4098]) -> [SKIP][302] ([i915#5286]) +1 other test skip
   [301]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@kms_big_fb@4-tiled-16bpp-rotate-270.html
   [302]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_big_fb@4-tiled-16bpp-rotate-270.html

  * igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-0:
    - shard-rkl:          [SKIP][303] ([i915#5286]) -> [SKIP][304] ([i915#1845] / [i915#4098]) +1 other test skip
   [303]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-0.html
   [304]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-0.html

  * igt@kms_big_fb@linear-64bpp-rotate-90:
    - shard-rkl:          [SKIP][305] ([fdo#111614] / [i915#3638]) -> [SKIP][306] ([i915#1845] / [i915#4098])
   [305]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@kms_big_fb@linear-64bpp-rotate-90.html
   [306]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@kms_big_fb@linear-64bpp-rotate-90.html

  * igt@kms_big_fb@linear-8bpp-rotate-270:
    - shard-rkl:          [SKIP][307] ([i915#1845] / [i915#4098]) -> [SKIP][308] ([fdo#111614] / [i915#3638]) +1 other test skip
   [307]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@kms_big_fb@linear-8bpp-rotate-270.html
   [308]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_big_fb@linear-8bpp-rotate-270.html

  * igt@kms_big_fb@yf-tiled-16bpp-rotate-180:
    - shard-rkl:          [SKIP][309] ([fdo#110723]) -> [SKIP][310] ([i915#1845] / [i915#4098])
   [309]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@kms_big_fb@yf-tiled-16bpp-rotate-180.html
   [310]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@kms_big_fb@yf-tiled-16bpp-rotate-180.html

  * igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-180-async-flip:
    - shard-rkl:          [SKIP][311] ([i915#1845] / [i915#4098]) -> [SKIP][312] ([fdo#110723])
   [311]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-180-async-flip.html
   [312]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-180-async-flip.html

  * igt@kms_content_protection@dp-mst-lic-type-1:
    - shard-rkl:          [SKIP][313] ([i915#1845] / [i915#4098]) -> [SKIP][314] ([i915#3116])
   [313]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@kms_content_protection@dp-mst-lic-type-1.html
   [314]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_content_protection@dp-mst-lic-type-1.html

  * igt@kms_content_protection@type1:
    - shard-dg2:          [SKIP][315] ([i915#7118] / [i915#7162]) -> [SKIP][316] ([i915#7118])
   [315]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-dg2-11/igt@kms_content_protection@type1.html
   [316]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-dg2-6/igt@kms_content_protection@type1.html

  * igt@kms_cursor_crc@cursor-sliding-32x32:
    - shard-rkl:          [SKIP][317] ([i915#1845] / [i915#4098]) -> [SKIP][318] ([i915#3555])
   [317]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@kms_cursor_crc@cursor-sliding-32x32.html
   [318]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_cursor_crc@cursor-sliding-32x32.html

  * igt@kms_cursor_legacy@2x-flip-vs-cursor-atomic:
    - shard-rkl:          [SKIP][319] ([i915#1845] / [i915#4098]) -> [SKIP][320] ([fdo#111767] / [fdo#111825])
   [319]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@kms_cursor_legacy@2x-flip-vs-cursor-atomic.html
   [320]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_cursor_legacy@2x-flip-vs-cursor-atomic.html

  * igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic:
    - shard-rkl:          [SKIP][321] ([i915#4103]) -> [SKIP][322] ([i915#1845] / [i915#4098])
   [321]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic.html
   [322]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic.html

  * igt@kms_cursor_legacy@cursorb-vs-flipa-legacy:
    - shard-rkl:          [SKIP][323] ([i915#1845] / [i915#4098]) -> [SKIP][324] ([fdo#111825])
   [323]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@kms_cursor_legacy@cursorb-vs-flipa-legacy.html
   [324]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_cursor_legacy@cursorb-vs-flipa-legacy.html

  * igt@kms_fbcon_fbt@psr:
    - shard-rkl:          [SKIP][325] ([fdo#110189] / [i915#3955]) -> [SKIP][326] ([i915#3955])
   [325]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-1/igt@kms_fbcon_fbt@psr.html
   [326]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-7/igt@kms_fbcon_fbt@psr.html

  * igt@kms_fbcon_fbt@psr-suspend:
    - shard-rkl:          [SKIP][327] ([i915#3955]) -> [SKIP][328] ([fdo#110189] / [i915#3955])
   [327]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-4/igt@kms_fbcon_fbt@psr-suspend.html
   [328]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-2/igt@kms_fbcon_fbt@psr-suspend.html

  * igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-shrfb-draw-mmap-gtt:
    - shard-rkl:          [SKIP][329] ([i915#1849] / [i915#4098]) -> [SKIP][330] ([fdo#111825] / [i915#1825]) +9 other tests skip
   [329]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-shrfb-draw-mmap-gtt.html
   [330]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-shrfb-draw-mmap-gtt.html

  * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-mmap-cpu:
    - shard-rkl:          [SKIP][331] ([i915#3023]) -> [SKIP][332] ([i915#1849] / [i915#4098]) +7 other tests skip
   [331]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-mmap-cpu.html
   [332]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-mmap-cpu.html

  * igt@kms_frontbuffer_tracking@fbcpsr-tiling-4:
    - shard-rkl:          [SKIP][333] ([i915#1849] / [i915#4098]) -> [SKIP][334] ([i915#5439])
   [333]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@kms_frontbuffer_tracking@fbcpsr-tiling-4.html
   [334]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_frontbuffer_tracking@fbcpsr-tiling-4.html

  * igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-shrfb-draw-blt:
    - shard-rkl:          [SKIP][335] ([i915#1849] / [i915#4098]) -> [SKIP][336] ([i915#3023]) +3 other tests skip
   [335]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-shrfb-draw-blt.html
   [336]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_frontbuffer_tracking@psr-1p-primscrn-pri-shrfb-draw-blt.html

  * igt@kms_frontbuffer_tracking@psr-2p-primscrn-spr-indfb-draw-mmap-wc:
    - shard-rkl:          [SKIP][337] ([fdo#111825] / [i915#1825]) -> [SKIP][338] ([i915#1849] / [i915#4098]) +7 other tests skip
   [337]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@kms_frontbuffer_tracking@psr-2p-primscrn-spr-indfb-draw-mmap-wc.html
   [338]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@kms_frontbuffer_tracking@psr-2p-primscrn-spr-indfb-draw-mmap-wc.html

  * igt@kms_hdr@invalid-metadata-sizes:
    - shard-rkl:          [SKIP][339] ([i915#3555] / [i915#8228]) -> [SKIP][340] ([i915#4098])
   [339]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@kms_hdr@invalid-metadata-sizes.html
   [340]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@kms_hdr@invalid-metadata-sizes.html

  * igt@kms_hdr@static-toggle-suspend:
    - shard-rkl:          [SKIP][341] ([i915#1845] / [i915#4098]) -> [SKIP][342] ([i915#3555] / [i915#8228])
   [341]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-5/igt@kms_hdr@static-toggle-suspend.html
   [342]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-4/igt@kms_hdr@static-toggle-suspend.html

  * igt@kms_multipipe_modeset@basic-max-pipe-crc-check:
    - shard-rkl:          [SKIP][343] ([i915#4070] / [i915#4816]) -> [SKIP][344] ([i915#4816])
   [343]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-1/igt@kms_multipipe_modeset@basic-max-pipe-crc-check.html
   [344]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-7/igt@kms_multipipe_modeset@basic-max-pipe-crc-check.html

  * igt@kms_vrr@flip-suspend:
    - shard-rkl:          [SKIP][345] ([i915#3555]) -> [SKIP][346] ([i915#1845] / [i915#4098])
   [345]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13859/shard-rkl-7/igt@kms_vrr@flip-suspend.html
   [346]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/shard-rkl-5/igt@kms_vrr@flip-suspend.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [fdo#109274]: https://bugs.freedesktop.org/show_bug.cgi?id=109274
  [fdo#109289]: https://bugs.freedesktop.org/show_bug.cgi?id=109289
  [fdo#109293]: https://bugs.freedesktop.org/show_bug.cgi?id=109293
  [fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295
  [fdo#109308]: https://bugs.freedesktop.org/show_bug.cgi?id=109308
  [fdo#109312]: https://bugs.freedesktop.org/show_bug.cgi?id=109312
  [fdo#109315]: https://bugs.freedesktop.org/show_bug.cgi?id=109315
  [fdo#109506]: https://bugs.freedesktop.org/show_bug.cgi?id=109506
  [fdo#110189]: https://bugs.freedesktop.org/show_bug.cgi?id=110189
  [fdo#110542]: https://bugs.freedesktop.org/show_bug.cgi?id=110542
  [fdo#110723]: https://bugs.freedesktop.org/show_bug.cgi?id=110723
  [fdo#111068]: https://bugs.freedesktop.org/show_bug.cgi?id=111068
  [fdo#111614]: https://bugs.freedesktop.org/show_bug.cgi?id=111614
  [fdo#111615]: https://bugs.freedesktop.org/show_bug.cgi?id=111615
  [fdo#111656]: https://bugs.freedesktop.org/show_bug.cgi?id=111656
  [fdo#111767]: https://bugs.freedesktop.org/show_bug.cgi?id=111767
  [fdo#111825]: https://bugs.freedesktop.org/show_bug.cgi?id=111825
  [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
  [fdo#112283]: https://bugs.freedesktop.org/show_bug.cgi?id=112283
  [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
  [i915#1339]: https://gitlab.freedesktop.org/drm/intel/issues/1339
  [i915#1769]: https://gitlab.freedesktop.org/drm/intel/issues/1769
  [i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180
  [i915#1825]: https://gitlab.freedesktop.org/drm/intel/issues/1825
  [i915#1839]: https://gitlab.freedesktop.org/drm/intel/issues/1839
  [i915#1845]: https://gitlab.freedesktop.org/drm/intel/issues/1845
  [i915#1849]: https://gitlab.freedesktop.org/drm/intel/issues/1849
  [i915#2434]: https://gitlab.freedesktop.org/drm/intel/issues/2434
  [i915#2437]: https://gitlab.freedesktop.org/drm/intel/issues/2437
  [i915#2527]: https://gitlab.freedesktop.org/drm/intel/issues/2527
  [i915#2575]: https://gitlab.freedesktop.org/drm/intel/issues/2575
  [i915#2672]: https://gitlab.freedesktop.org/drm/intel/issues/2672
  [i915#284]: https://gitlab.freedesktop.org/drm/intel/issues/284
  [i915#2842]: https://gitlab.freedesktop.org/drm/intel/issues/2842
  [i915#2856]: https://gitlab.freedesktop.org/drm/intel/issues/2856
  [i915#3023]: https://gitlab.freedesktop.org/drm/intel/issues/3023
  [i915#3116]: https://gitlab.freedesktop.org/drm/intel/issues/3116
  [i915#3281]: https://gitlab.freedesktop.org/drm/intel/issues/3281
  [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282
  [i915#3291]: https://gitlab.freedesktop.org/drm/intel/issues/3291
  [i915#3297]: https://gitlab.freedesktop.org/drm/intel/issues/3297
  [i915#3299]: https://gitlab.freedesktop.org/drm/intel/issues/3299
  [i915#3359]: https://gitlab.freedesktop.org/drm/intel/issues/3359
  [i915#3458]: https://gitlab.freedesktop.org/drm/intel/issues/3458
  [i915#3539]: https://gitlab.freedesktop.org/drm/intel/issues/3539
  [i915#3546]: https://gitlab.freedesktop.org/drm/intel/issues/3546
  [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
  [i915#3591]: https://gitlab.freedesktop.org/drm/intel/issues/3591
  [i915#3637]: https://gitlab.freedesktop.org/drm/intel/issues/3637
  [i915#3638]: https://gitlab.freedesktop.org/drm/intel/issues/3638
  [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
  [i915#3742]: https://gitlab.freedesktop.org/drm/intel/issues/3742
  [i915#3840]: https://gitlab.freedesktop.org/drm/intel/issues/3840
  [i915#3936]: https://gitlab.freedesktop.org/drm/intel/issues/3936
  [i915#3955]: https://gitlab.freedesktop.org/drm/intel/issues/3955
  [i915#4070]: https://gitlab.freedesktop.org/drm/intel/issues/4070
  [i915#4077]: https://gitlab.freedesktop.org/drm/intel/issues/4077
  [i915#4079]: https://gitlab.freedesktop.org/drm/intel/issues/4079
  [i915#4083]: https://gitlab.freedesktop.org/drm/intel/issues/4083
  [i915#4087]: https://gitlab.freedesktop.org/drm/intel/issues/4087
  [i915#4098]: https://gitlab.freedesktop.org/drm/intel/issues/4098
  [i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103
  [i915#4212]: https://gitlab.freedesktop.org/drm/intel/issues/4212
  [i915#4213]: https://gitlab.freedesktop.org/drm/intel/issues/4213
  [i915#4215]: https://gitlab.freedesktop.org/drm/intel/issues/4215
  [i915#4235]: https://gitlab.freedesktop.org/drm/intel/issues/4235
  [i915#4270]: https://gitlab.freedesktop.org/drm/intel/issues/4270
  [i915#4349]: https://gitlab.freedesktop.org/drm/intel/issues/4349
  [i915#4387]: https://gitlab.freedesktop.org/drm/intel/issues/4387
  [i915#4423]: https://gitlab.freedesktop.org/drm/intel/issues/4423
  [i915#4473]: https://gitlab.freedesktop.org/drm/intel/issues/4473
  [i915#4525]: https://gitlab.freedesktop.org/drm/intel/issues/4525
  [i915#4537]: https://gitlab.freedesktop.org/drm/intel/issues/4537
  [i915#4538]: https://gitlab.freedesktop.org/drm/intel/issues/4538
  [i915#4573]: https://gitlab.freedesktop.org/drm/intel/issues/4573
  [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
  [i915#4771]: https://gitlab.freedesktop.org/drm/intel/issues/4771
  [i915#4812]: https://gitlab.freedesktop.org/drm/intel/issues/4812
  [i915#4816]: https://gitlab.freedesktop.org/drm/intel/issues/4816
  [i915#4818]: https://gitlab.freedesktop.org/drm/intel/issues/4818
  [i915#4852]: https://gitlab.freedesktop.org/drm/intel/issues/4852
  [i915#4854]: https://gitlab.freedesktop.org/drm/intel/issues/4854
  [i915#4860]: https://gitlab.freedesktop.org/drm/intel/issues/4860
  [i915#4880]: https://gitlab.freedesktop.org/drm/intel/issues/4880
  [i915#4881]: https://gitlab.freedesktop.org/drm/intel/issues/4881
  [i915#4885]: https://gitlab.freedesktop.org/drm/intel/issues/4885
  [i915#4936]: https://gitlab.freedesktop.org/drm/intel/issues/4936
  [i915#5138]: https://gitlab.freedesktop.org/drm/intel/issues/5138
  [i915#5176]: https://gitlab.freedesktop.org/drm/intel/issues/5176
  [i915#5190]: https://gitlab.freedesktop.org/drm/intel/issues/5190
  [i915#5235]: https://gitlab.freedesktop.org/drm/intel/issues/5235
  [i915#5286]: https://gitlab.freedesktop.org/drm/intel/issues/5286
  [i915#5334]: https://gitlab.freedesktop.org/drm/intel/issues/5334
  [i915#5354]: https://gitlab.freedesktop.org/drm/intel/issues/5354
  [i915#5439]: https://gitlab.freedesktop.org/drm/intel/issues/5439
  [i915#5461]: https://gitlab.freedesktop.org/drm/intel/issues/5461
  [i915#5493]: https://gitlab.freedesktop.org/drm/intel/issues/5493
  [i915#5507]: https://gitlab.freedesktop.org/drm/intel/issues/5507
  [i915#5566]: https://gitlab.freedesktop.org/drm/intel/issues/5566
  [i915#5784]: https://gitlab.freedesktop.org/drm/intel/issues/5784
  [i915#5889]: https://gitlab.freedesktop.org/drm/intel/issues/5889
  [i915#6095]: https://gitlab.freedesktop.org/drm/intel/issues/6095
  [i915#6188]: https://gitlab.freedesktop.org/drm/intel/issues/6188
  [i915#6252]: https://gitlab.freedesktop.org/drm/intel/issues/6252
  [i915#6301]: https://gitlab.freedesktop.org/drm/intel/issues/6301
  [i915#6335]: https://gitlab.freedesktop.org/drm/intel/issues/6335
  [i915#6524]: https://gitlab.freedesktop.org/drm/intel/issues/6524
  [i915#658]: https://gitlab.freedesktop.org/drm/intel/issues/658
  [i915#6805]: https://gitlab.freedesktop.org/drm/intel/issues/6805
  [i915#6806]: https://gitlab.freedesktop.org/drm/intel/issues/6806
  [i915#6953]: https://gitlab.freedesktop.org/drm/intel/issues/6953
  [i915#7036]: https://gitlab.freedesktop.org/drm/intel/issues/7036
  [i915#7118]: https://gitlab.freedesktop.org/drm/intel/issues/7118
  [i915#7162]: https://gitlab.freedesktop.org/drm/intel/issues/7162
  [i915#7173]: https://gitlab.freedesktop.org/drm/intel/issues/7173
  [i915#7213]: https://gitlab.freedesktop.org/drm/intel/issues/7213
  [i915#7276]: https://gitlab.freedesktop.org/drm/intel/issues/7276
  [i915#7297]: https://gitlab.freedesktop.org/drm/intel/issues/7297
  [i915#7697]: https://gitlab.freedesktop.org/drm/intel/issues/7697
  [i915#7711]: https://gitlab.freedesktop.org/drm/intel/issues/7711
  [i915#7742]: https://gitlab.freedesktop.org/drm/intel/issues/7742
  [i915#7828]: https://gitlab.freedesktop.org/drm/intel/issues/7828
  [i915#7892]: https://gitlab.freedesktop.org/drm/intel/issues/7892
  [i915#7957]: https://gitlab.freedesktop.org/drm/intel/issues/7957
  [i915#7975]: https://gitlab.freedesktop.org/drm/intel/issues/7975
  [i915#7984]: https://gitlab.freedesktop.org/drm/intel/issues/7984
  [i915#8152]: https://gitlab.freedesktop.org/drm/intel/issues/8152
  [i915#8213]: https://gitlab.freedesktop.org/drm/intel/issues/8213
  [i915#8228]: https://gitlab.freedesktop.org/drm/intel/issues/8228
  [i915#8247]: https://gitlab.freedesktop.org/drm/intel/issues/8247
  [i915#8292]: https://gitlab.freedesktop.org/drm/intel/issues/8292
  [i915#8293]: https://gitlab.freedesktop.org/drm/intel/issues/8293
  [i915#8399]: https://gitlab.freedesktop.org/drm/intel/issues/8399
  [i915#8411]: https://gitlab.freedesktop.org/drm/intel/issues/8411
  [i915#8414]: https://gitlab.freedesktop.org/drm/intel/issues/8414
  [i915#8428]: https://gitlab.freedesktop.org/drm/intel/issues/8428
  [i915#8516]: https://gitlab.freedesktop.org/drm/intel/issues/8516
  [i915#8555]: https://gitlab.freedesktop.org/drm/intel/issues/8555
  [i915#8588]: https://gitlab.freedesktop.org/drm/intel/issues/8588
  [i915#8623]: https://gitlab.freedesktop.org/drm/intel/issues/8623
  [i915#8708]: https://gitlab.freedesktop.org/drm/intel/issues/8708
  [i915#8806]: https://gitlab.freedesktop.org/drm/intel/issues/8806
  [i915#8809]: https://gitlab.freedesktop.org/drm/intel/issues/8809
  [i915#8875]: https://gitlab.freedesktop.org/drm/intel/issues/8875
  [i915#8925]: https://gitlab.freedesktop.org/drm/intel/issues/8925
  [i915#9053]: https://gitlab.freedesktop.org/drm/intel/issues/9053
  [i915#9157]: https://gitlab.freedesktop.org/drm/intel/issues/9157
  [i915#9196]: https://gitlab.freedesktop.org/drm/intel/issues/9196
  [i915#9226]: https://gitlab.freedesktop.org/drm/intel/issues/9226
  [i915#9227]: https://gitlab.freedesktop.org/drm/intel/issues/9227
  [i915#9261]: https://gitlab.freedesktop.org/drm/intel/issues/9261
  [i915#9323]: https://gitlab.freedesktop.org/drm/intel/issues/9323
  [i915#9340]: https://gitlab.freedesktop.org/drm/intel/issues/9340
  [i915#9392]: https://gitlab.freedesktop.org/drm/intel/issues/9392
  [i915#9412]: https://gitlab.freedesktop.org/drm/intel/issues/9412
  [i915#9414]: https://gitlab.freedesktop.org/drm/intel/issues/9414
  [i915#9423]: https://gitlab.freedesktop.org/drm/intel/issues/9423
  [i915#9519]: https://gitlab.freedesktop.org/drm/intel/issues/9519
  [i915#9531]: https://gitlab.freedesktop.org/drm/intel/issues/9531
  [i915#9559]: https://gitlab.freedesktop.org/drm/intel/issues/9559
  [i915#9569]: https://gitlab.freedesktop.org/drm/intel/issues/9569
  [i915#9583]: https://gitlab.freedesktop.org/drm/intel/issues/9583
  [i915#9588]: https://gitlab.freedesktop.org/drm/intel/issues/9588
  [i915#9606]: https://gitlab.freedesktop.org/drm/intel/issues/9606
  [i915#9638]: https://gitlab.freedesktop.org/drm/intel/issues/9638


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

  * Linux: CI_DRM_13859 -> Patchwork_126225v1

  CI-20190529: 20190529
  CI_DRM_13859: 9155ae0ae05f320d84eaf2c4e81413bf937a5f3c @ git://anongit.freedesktop.org/gfx-ci/linux
  IGT_7581: 7581
  Patchwork_126225v1: 9155ae0ae05f320d84eaf2c4e81413bf937a5f3c @ git://anongit.freedesktop.org/gfx-ci/linux

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_126225v1/index.html

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

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

* Re: [Intel-gfx] [PATCH] drm/i915/gsc: Assign a uabi class number to the GSC CS
  2023-11-09 23:53 [Intel-gfx] [PATCH] drm/i915/gsc: Assign a uabi class number to the GSC CS Daniele Ceraolo Spurio
                   ` (2 preceding siblings ...)
  2023-11-10 10:25 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
@ 2023-11-10 12:00 ` Tvrtko Ursulin
  2023-11-13 15:51   ` Daniele Ceraolo Spurio
  3 siblings, 1 reply; 10+ messages in thread
From: Tvrtko Ursulin @ 2023-11-10 12:00 UTC (permalink / raw)
  To: Daniele Ceraolo Spurio, intel-gfx; +Cc: Matt Roper, Alan Previn


On 09/11/2023 23:53, Daniele Ceraolo Spurio wrote:
> The GSC CS is not exposed to the user, so we skipped assigning a uabi
> class number for it. However, the trace logs use the uabi class and
> instance to identify the engine, so leaving uabi class unset makes the
> GSC CS show up as the RCS in those logs.
> Given that the engine is not exposed to the user, we can't add a new
> case in the uabi enum, so we insted internally define a kernel
> reserved class using the next free number.
> 
> Fixes: 194babe26bdc ("drm/i915/mtl: don't expose GSC command streamer to the user")
> Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
> Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
> Cc: Alan Previn <alan.previn.teres.alexis@intel.com>
> Cc: Matt Roper <matthew.d.roper@intel.com>
> ---
>   drivers/gpu/drm/i915/gt/intel_engine_user.c | 17 ++++++++---------
>   drivers/gpu/drm/i915/gt/intel_engine_user.h |  4 ++++
>   drivers/gpu/drm/i915/i915_drm_client.h      |  2 +-
>   drivers/gpu/drm/i915/i915_drv.h             |  2 +-
>   4 files changed, 14 insertions(+), 11 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/gt/intel_engine_user.c b/drivers/gpu/drm/i915/gt/intel_engine_user.c
> index 118164ddbb2e..3fd32bedd6e7 100644
> --- a/drivers/gpu/drm/i915/gt/intel_engine_user.c
> +++ b/drivers/gpu/drm/i915/gt/intel_engine_user.c
> @@ -47,6 +47,7 @@ static const u8 uabi_classes[] = {
>   	[VIDEO_DECODE_CLASS] = I915_ENGINE_CLASS_VIDEO,
>   	[VIDEO_ENHANCEMENT_CLASS] = I915_ENGINE_CLASS_VIDEO_ENHANCE,
>   	[COMPUTE_CLASS] = I915_ENGINE_CLASS_COMPUTE,
> +	[OTHER_CLASS] = I915_KERNEL_RSVD_CLASS,

Could we set it to -1 (aka no uabi class) to avoid needing to maintain 
the new macros?

And then just teach intel_engines_driver_register to skip -1. Would also 
need teaching engine_rename to handle -1.

Might end up a smaller and more maintainable patch - worth a try do you 
think?

>   };
>   
>   static int engine_cmp(void *priv, const struct list_head *A,
> @@ -138,7 +139,7 @@ const char *intel_engine_class_repr(u8 class)
>   		[COPY_ENGINE_CLASS] = "bcs",
>   		[VIDEO_DECODE_CLASS] = "vcs",
>   		[VIDEO_ENHANCEMENT_CLASS] = "vecs",
> -		[OTHER_CLASS] = "other",
> +		[OTHER_CLASS] = "gsc",

Maybe unrelated?

Regards,

Tvrtko

>   		[COMPUTE_CLASS] = "ccs",
>   	};
>   
> @@ -216,14 +217,8 @@ void intel_engines_driver_register(struct drm_i915_private *i915)
>   		if (intel_gt_has_unrecoverable_error(engine->gt))
>   			continue; /* ignore incomplete engines */
>   
> -		/*
> -		 * We don't want to expose the GSC engine to the users, but we
> -		 * still rename it so it is easier to identify in the debug logs
> -		 */
> -		if (engine->id == GSC0) {
> -			engine_rename(engine, "gsc", 0);
> -			continue;
> -		}
> +		/* The only engine we expect in OTHER_CLASS is GSC0 */
> +		GEM_WARN_ON(engine->class == OTHER_CLASS && engine->id != GSC0);
>   
>   		GEM_BUG_ON(engine->class >= ARRAY_SIZE(uabi_classes));
>   		engine->uabi_class = uabi_classes[engine->class];
> @@ -238,6 +233,10 @@ void intel_engines_driver_register(struct drm_i915_private *i915)
>   			      intel_engine_class_repr(engine->class),
>   			      engine->uabi_instance);
>   
> +		/* We don't want to expose the GSC engine to the users */
> +		if (engine->id == GSC0)
> +			continue;
> +
>   		rb_link_node(&engine->uabi_node, prev, p);
>   		rb_insert_color(&engine->uabi_node, &i915->uabi_engines);
>   
> diff --git a/drivers/gpu/drm/i915/gt/intel_engine_user.h b/drivers/gpu/drm/i915/gt/intel_engine_user.h
> index 3dc7e8ab9fbc..dd31805b2a5a 100644
> --- a/drivers/gpu/drm/i915/gt/intel_engine_user.h
> +++ b/drivers/gpu/drm/i915/gt/intel_engine_user.h
> @@ -11,6 +11,10 @@
>   struct drm_i915_private;
>   struct intel_engine_cs;
>   
> +#define I915_LAST_UABI_ENGINE_CLASS I915_ENGINE_CLASS_COMPUTE
> +#define I915_KERNEL_RSVD_CLASS (I915_LAST_UABI_ENGINE_CLASS + 1)
> +#define I915_MAX_UABI_CLASSES (I915_KERNEL_RSVD_CLASS + 1)
> +
>   struct intel_engine_cs *
>   intel_engine_lookup_user(struct drm_i915_private *i915, u8 class, u8 instance);
>   
> diff --git a/drivers/gpu/drm/i915/i915_drm_client.h b/drivers/gpu/drm/i915/i915_drm_client.h
> index 67816c912bca..c42cb2511348 100644
> --- a/drivers/gpu/drm/i915/i915_drm_client.h
> +++ b/drivers/gpu/drm/i915/i915_drm_client.h
> @@ -12,7 +12,7 @@
>   
>   #include <uapi/drm/i915_drm.h>
>   
> -#define I915_LAST_UABI_ENGINE_CLASS I915_ENGINE_CLASS_COMPUTE
> +#include "gt/intel_engine_user.h"
>   
>   struct drm_file;
>   struct drm_printer;
> diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
> index f3be9033a93f..a718b4cb5a2d 100644
> --- a/drivers/gpu/drm/i915/i915_drv.h
> +++ b/drivers/gpu/drm/i915/i915_drv.h
> @@ -238,7 +238,7 @@ struct drm_i915_private {
>   		struct list_head uabi_engines_list;
>   		struct rb_root uabi_engines;
>   	};
> -	unsigned int engine_uabi_class_count[I915_LAST_UABI_ENGINE_CLASS + 1];
> +	unsigned int engine_uabi_class_count[I915_MAX_UABI_CLASSES];
>   
>   	/* protects the irq masks */
>   	spinlock_t irq_lock;

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

* Re: [Intel-gfx] [PATCH] drm/i915/gsc: Assign a uabi class number to the GSC CS
  2023-11-10 12:00 ` [Intel-gfx] [PATCH] " Tvrtko Ursulin
@ 2023-11-13 15:51   ` Daniele Ceraolo Spurio
  2023-11-13 16:46     ` Tvrtko Ursulin
  0 siblings, 1 reply; 10+ messages in thread
From: Daniele Ceraolo Spurio @ 2023-11-13 15:51 UTC (permalink / raw)
  To: Tvrtko Ursulin, intel-gfx; +Cc: Matt Roper, Alan Previn



On 11/10/2023 4:00 AM, Tvrtko Ursulin wrote:
>
> On 09/11/2023 23:53, Daniele Ceraolo Spurio wrote:
>> The GSC CS is not exposed to the user, so we skipped assigning a uabi
>> class number for it. However, the trace logs use the uabi class and
>> instance to identify the engine, so leaving uabi class unset makes the
>> GSC CS show up as the RCS in those logs.
>> Given that the engine is not exposed to the user, we can't add a new
>> case in the uabi enum, so we insted internally define a kernel
>> reserved class using the next free number.
>>
>> Fixes: 194babe26bdc ("drm/i915/mtl: don't expose GSC command streamer 
>> to the user")
>> Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
>> Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
>> Cc: Alan Previn <alan.previn.teres.alexis@intel.com>
>> Cc: Matt Roper <matthew.d.roper@intel.com>
>> ---
>>   drivers/gpu/drm/i915/gt/intel_engine_user.c | 17 ++++++++---------
>>   drivers/gpu/drm/i915/gt/intel_engine_user.h |  4 ++++
>>   drivers/gpu/drm/i915/i915_drm_client.h      |  2 +-
>>   drivers/gpu/drm/i915/i915_drv.h             |  2 +-
>>   4 files changed, 14 insertions(+), 11 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/gt/intel_engine_user.c 
>> b/drivers/gpu/drm/i915/gt/intel_engine_user.c
>> index 118164ddbb2e..3fd32bedd6e7 100644
>> --- a/drivers/gpu/drm/i915/gt/intel_engine_user.c
>> +++ b/drivers/gpu/drm/i915/gt/intel_engine_user.c
>> @@ -47,6 +47,7 @@ static const u8 uabi_classes[] = {
>>       [VIDEO_DECODE_CLASS] = I915_ENGINE_CLASS_VIDEO,
>>       [VIDEO_ENHANCEMENT_CLASS] = I915_ENGINE_CLASS_VIDEO_ENHANCE,
>>       [COMPUTE_CLASS] = I915_ENGINE_CLASS_COMPUTE,
>> +    [OTHER_CLASS] = I915_KERNEL_RSVD_CLASS,
>
> Could we set it to -1 (aka no uabi class) to avoid needing to maintain 
> the new macros?
>
> And then just teach intel_engines_driver_register to skip -1. Would 
> also need teaching engine_rename to handle -1.
>
> Might end up a smaller and more maintainable patch - worth a try do 
> you think?

That was my initial idea as well, but the issue with this approach is 
the engine_uabi_class_count[] array, which is sized based on the number 
of uabi classes, so having class -1 would needlessly increase its size a 
lot even when using a u8. I thought about limiting the class entry to 3 
bits so the array would max out at 8 entries, but that seemed to be 
getting a bit too convoluted. I can give it a go if you think it's be 
cleaner overall.

Note that this patch does not introduce any new macros that would need 
to be maintained. I915_LAST_UABI_ENGINE_CLASS already existed (I just 
moved it from one file to another) and is the only one that changes when 
a new "real" uabi class is added; the other defines are based on this 
one. This also implies that if a new uabi class is added then 
I915_KERNEL_RSVD_CLASS would be bumped to the next free number, which 
would cause the GSC to show as a different uabi class in newer logs; 
considering that i915 is on its way out, a new class seems very unlikely 
so I thought it'd be an acceptable compromise to keep things simple.

>
>>   };
>>     static int engine_cmp(void *priv, const struct list_head *A,
>> @@ -138,7 +139,7 @@ const char *intel_engine_class_repr(u8 class)
>>           [COPY_ENGINE_CLASS] = "bcs",
>>           [VIDEO_DECODE_CLASS] = "vcs",
>>           [VIDEO_ENHANCEMENT_CLASS] = "vecs",
>> -        [OTHER_CLASS] = "other",
>> +        [OTHER_CLASS] = "gsc",
>
> Maybe unrelated?

no. Before this patch, we hardcoded "gsc" below when calling 
engine_rename() for it. With this patch, we use the name from this 
array, so it needs to be updated. The GEM_WARN_ON below was added to 
make sure we don't get different engines in OTHER_CLASS that might not 
match the "gsc" naming.

Daniele

>
> Regards,
>
> Tvrtko
>
>>           [COMPUTE_CLASS] = "ccs",
>>       };
>>   @@ -216,14 +217,8 @@ void intel_engines_driver_register(struct 
>> drm_i915_private *i915)
>>           if (intel_gt_has_unrecoverable_error(engine->gt))
>>               continue; /* ignore incomplete engines */
>>   -        /*
>> -         * We don't want to expose the GSC engine to the users, but we
>> -         * still rename it so it is easier to identify in the debug 
>> logs
>> -         */
>> -        if (engine->id == GSC0) {
>> -            engine_rename(engine, "gsc", 0);
>> -            continue;
>> -        }
>> +        /* The only engine we expect in OTHER_CLASS is GSC0 */
>> +        GEM_WARN_ON(engine->class == OTHER_CLASS && engine->id != 
>> GSC0);
>>             GEM_BUG_ON(engine->class >= ARRAY_SIZE(uabi_classes));
>>           engine->uabi_class = uabi_classes[engine->class];
>> @@ -238,6 +233,10 @@ void intel_engines_driver_register(struct 
>> drm_i915_private *i915)
>>                     intel_engine_class_repr(engine->class),
>>                     engine->uabi_instance);
>>   +        /* We don't want to expose the GSC engine to the users */
>> +        if (engine->id == GSC0)
>> +            continue;
>> +
>>           rb_link_node(&engine->uabi_node, prev, p);
>>           rb_insert_color(&engine->uabi_node, &i915->uabi_engines);
>>   diff --git a/drivers/gpu/drm/i915/gt/intel_engine_user.h 
>> b/drivers/gpu/drm/i915/gt/intel_engine_user.h
>> index 3dc7e8ab9fbc..dd31805b2a5a 100644
>> --- a/drivers/gpu/drm/i915/gt/intel_engine_user.h
>> +++ b/drivers/gpu/drm/i915/gt/intel_engine_user.h
>> @@ -11,6 +11,10 @@
>>   struct drm_i915_private;
>>   struct intel_engine_cs;
>>   +#define I915_LAST_UABI_ENGINE_CLASS I915_ENGINE_CLASS_COMPUTE
>> +#define I915_KERNEL_RSVD_CLASS (I915_LAST_UABI_ENGINE_CLASS + 1)
>> +#define I915_MAX_UABI_CLASSES (I915_KERNEL_RSVD_CLASS + 1)
>> +
>>   struct intel_engine_cs *
>>   intel_engine_lookup_user(struct drm_i915_private *i915, u8 class, 
>> u8 instance);
>>   diff --git a/drivers/gpu/drm/i915/i915_drm_client.h 
>> b/drivers/gpu/drm/i915/i915_drm_client.h
>> index 67816c912bca..c42cb2511348 100644
>> --- a/drivers/gpu/drm/i915/i915_drm_client.h
>> +++ b/drivers/gpu/drm/i915/i915_drm_client.h
>> @@ -12,7 +12,7 @@
>>     #include <uapi/drm/i915_drm.h>
>>   -#define I915_LAST_UABI_ENGINE_CLASS I915_ENGINE_CLASS_COMPUTE
>> +#include "gt/intel_engine_user.h"
>>     struct drm_file;
>>   struct drm_printer;
>> diff --git a/drivers/gpu/drm/i915/i915_drv.h 
>> b/drivers/gpu/drm/i915/i915_drv.h
>> index f3be9033a93f..a718b4cb5a2d 100644
>> --- a/drivers/gpu/drm/i915/i915_drv.h
>> +++ b/drivers/gpu/drm/i915/i915_drv.h
>> @@ -238,7 +238,7 @@ struct drm_i915_private {
>>           struct list_head uabi_engines_list;
>>           struct rb_root uabi_engines;
>>       };
>> -    unsigned int engine_uabi_class_count[I915_LAST_UABI_ENGINE_CLASS 
>> + 1];
>> +    unsigned int engine_uabi_class_count[I915_MAX_UABI_CLASSES];
>>         /* protects the irq masks */
>>       spinlock_t irq_lock;


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

* Re: [Intel-gfx] [PATCH] drm/i915/gsc: Assign a uabi class number to the GSC CS
  2023-11-13 15:51   ` Daniele Ceraolo Spurio
@ 2023-11-13 16:46     ` Tvrtko Ursulin
  2023-11-14 17:03       ` Daniele Ceraolo Spurio
  0 siblings, 1 reply; 10+ messages in thread
From: Tvrtko Ursulin @ 2023-11-13 16:46 UTC (permalink / raw)
  To: Daniele Ceraolo Spurio, intel-gfx; +Cc: Matt Roper, Alan Previn


On 13/11/2023 15:51, Daniele Ceraolo Spurio wrote:
> On 11/10/2023 4:00 AM, Tvrtko Ursulin wrote:
>>
>> On 09/11/2023 23:53, Daniele Ceraolo Spurio wrote:
>>> The GSC CS is not exposed to the user, so we skipped assigning a uabi
>>> class number for it. However, the trace logs use the uabi class and
>>> instance to identify the engine, so leaving uabi class unset makes the
>>> GSC CS show up as the RCS in those logs.
>>> Given that the engine is not exposed to the user, we can't add a new
>>> case in the uabi enum, so we insted internally define a kernel
>>> reserved class using the next free number.
>>>
>>> Fixes: 194babe26bdc ("drm/i915/mtl: don't expose GSC command streamer 
>>> to the user")
>>> Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
>>> Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
>>> Cc: Alan Previn <alan.previn.teres.alexis@intel.com>
>>> Cc: Matt Roper <matthew.d.roper@intel.com>
>>> ---
>>>   drivers/gpu/drm/i915/gt/intel_engine_user.c | 17 ++++++++---------
>>>   drivers/gpu/drm/i915/gt/intel_engine_user.h |  4 ++++
>>>   drivers/gpu/drm/i915/i915_drm_client.h      |  2 +-
>>>   drivers/gpu/drm/i915/i915_drv.h             |  2 +-
>>>   4 files changed, 14 insertions(+), 11 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/i915/gt/intel_engine_user.c 
>>> b/drivers/gpu/drm/i915/gt/intel_engine_user.c
>>> index 118164ddbb2e..3fd32bedd6e7 100644
>>> --- a/drivers/gpu/drm/i915/gt/intel_engine_user.c
>>> +++ b/drivers/gpu/drm/i915/gt/intel_engine_user.c
>>> @@ -47,6 +47,7 @@ static const u8 uabi_classes[] = {
>>>       [VIDEO_DECODE_CLASS] = I915_ENGINE_CLASS_VIDEO,
>>>       [VIDEO_ENHANCEMENT_CLASS] = I915_ENGINE_CLASS_VIDEO_ENHANCE,
>>>       [COMPUTE_CLASS] = I915_ENGINE_CLASS_COMPUTE,
>>> +    [OTHER_CLASS] = I915_KERNEL_RSVD_CLASS,
>>
>> Could we set it to -1 (aka no uabi class) to avoid needing to maintain 
>> the new macros?
>>
>> And then just teach intel_engines_driver_register to skip -1. Would 
>> also need teaching engine_rename to handle -1.
>>
>> Might end up a smaller and more maintainable patch - worth a try do 
>> you think?
> 
> That was my initial idea as well, but the issue with this approach is 
> the engine_uabi_class_count[] array, which is sized based on the number 
> of uabi classes, so having class -1 would needlessly increase its size a 
> lot even when using a u8. I thought about limiting the class entry to 3 

I was thinking the -1 entry wouldn't be in that array since -1 is not 
uabi class by its very definition. It is not reachable from the outside 
so no need to be there.

> bits so the array would max out at 8 entries, but that seemed to be 
> getting a bit too convoluted. I can give it a go if you think it's be 
> cleaner overall.

I had a feeling it would be, but without trying it out I won't claim 100%.

> Note that this patch does not introduce any new macros that would need 
> to be maintained. I915_LAST_UABI_ENGINE_CLASS already existed (I just 
> moved it from one file to another) and is the only one that changes when 
> a new "real" uabi class is added; the other defines are based on this 
> one. This also implies that if a new uabi class is added then 
> I915_KERNEL_RSVD_CLASS would be bumped to the next free number, which 
> would cause the GSC to show as a different uabi class in newer logs; 
> considering that i915 is on its way out, a new class seems very unlikely 
> so I thought it'd be an acceptable compromise to keep things simple.
> 
>>
>>>   };
>>>     static int engine_cmp(void *priv, const struct list_head *A,
>>> @@ -138,7 +139,7 @@ const char *intel_engine_class_repr(u8 class)
>>>           [COPY_ENGINE_CLASS] = "bcs",
>>>           [VIDEO_DECODE_CLASS] = "vcs",
>>>           [VIDEO_ENHANCEMENT_CLASS] = "vecs",
>>> -        [OTHER_CLASS] = "other",
>>> +        [OTHER_CLASS] = "gsc",
>>
>> Maybe unrelated?
> 
> no. Before this patch, we hardcoded "gsc" below when calling 
> engine_rename() for it. With this patch, we use the name from this 
> array, so it needs to be updated. The GEM_WARN_ON below was added to 
> make sure we don't get different engines in OTHER_CLASS that might not 
> match the "gsc" naming.

Ah okay, one special casing for another, a wash I guess.

Regards,

Tvrtko

>>
>> Regards,
>>
>> Tvrtko
>>
>>>           [COMPUTE_CLASS] = "ccs",
>>>       };
>>>   @@ -216,14 +217,8 @@ void intel_engines_driver_register(struct 
>>> drm_i915_private *i915)
>>>           if (intel_gt_has_unrecoverable_error(engine->gt))
>>>               continue; /* ignore incomplete engines */
>>>   -        /*
>>> -         * We don't want to expose the GSC engine to the users, but we
>>> -         * still rename it so it is easier to identify in the debug 
>>> logs
>>> -         */
>>> -        if (engine->id == GSC0) {
>>> -            engine_rename(engine, "gsc", 0);
>>> -            continue;
>>> -        }
>>> +        /* The only engine we expect in OTHER_CLASS is GSC0 */
>>> +        GEM_WARN_ON(engine->class == OTHER_CLASS && engine->id != 
>>> GSC0);
>>>             GEM_BUG_ON(engine->class >= ARRAY_SIZE(uabi_classes));
>>>           engine->uabi_class = uabi_classes[engine->class];
>>> @@ -238,6 +233,10 @@ void intel_engines_driver_register(struct 
>>> drm_i915_private *i915)
>>>                     intel_engine_class_repr(engine->class),
>>>                     engine->uabi_instance);
>>>   +        /* We don't want to expose the GSC engine to the users */
>>> +        if (engine->id == GSC0)
>>> +            continue;
>>> +
>>>           rb_link_node(&engine->uabi_node, prev, p);
>>>           rb_insert_color(&engine->uabi_node, &i915->uabi_engines);
>>>   diff --git a/drivers/gpu/drm/i915/gt/intel_engine_user.h 
>>> b/drivers/gpu/drm/i915/gt/intel_engine_user.h
>>> index 3dc7e8ab9fbc..dd31805b2a5a 100644
>>> --- a/drivers/gpu/drm/i915/gt/intel_engine_user.h
>>> +++ b/drivers/gpu/drm/i915/gt/intel_engine_user.h
>>> @@ -11,6 +11,10 @@
>>>   struct drm_i915_private;
>>>   struct intel_engine_cs;
>>>   +#define I915_LAST_UABI_ENGINE_CLASS I915_ENGINE_CLASS_COMPUTE
>>> +#define I915_KERNEL_RSVD_CLASS (I915_LAST_UABI_ENGINE_CLASS + 1)
>>> +#define I915_MAX_UABI_CLASSES (I915_KERNEL_RSVD_CLASS + 1)
>>> +
>>>   struct intel_engine_cs *
>>>   intel_engine_lookup_user(struct drm_i915_private *i915, u8 class, 
>>> u8 instance);
>>>   diff --git a/drivers/gpu/drm/i915/i915_drm_client.h 
>>> b/drivers/gpu/drm/i915/i915_drm_client.h
>>> index 67816c912bca..c42cb2511348 100644
>>> --- a/drivers/gpu/drm/i915/i915_drm_client.h
>>> +++ b/drivers/gpu/drm/i915/i915_drm_client.h
>>> @@ -12,7 +12,7 @@
>>>     #include <uapi/drm/i915_drm.h>
>>>   -#define I915_LAST_UABI_ENGINE_CLASS I915_ENGINE_CLASS_COMPUTE
>>> +#include "gt/intel_engine_user.h"
>>>     struct drm_file;
>>>   struct drm_printer;
>>> diff --git a/drivers/gpu/drm/i915/i915_drv.h 
>>> b/drivers/gpu/drm/i915/i915_drv.h
>>> index f3be9033a93f..a718b4cb5a2d 100644
>>> --- a/drivers/gpu/drm/i915/i915_drv.h
>>> +++ b/drivers/gpu/drm/i915/i915_drv.h
>>> @@ -238,7 +238,7 @@ struct drm_i915_private {
>>>           struct list_head uabi_engines_list;
>>>           struct rb_root uabi_engines;
>>>       };
>>> -    unsigned int engine_uabi_class_count[I915_LAST_UABI_ENGINE_CLASS 
>>> + 1];
>>> +    unsigned int engine_uabi_class_count[I915_MAX_UABI_CLASSES];
>>>         /* protects the irq masks */
>>>       spinlock_t irq_lock;
> 

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

* Re: [Intel-gfx] [PATCH] drm/i915/gsc: Assign a uabi class number to the GSC CS
  2023-11-13 16:46     ` Tvrtko Ursulin
@ 2023-11-14 17:03       ` Daniele Ceraolo Spurio
  2023-11-14 17:20         ` Tvrtko Ursulin
  0 siblings, 1 reply; 10+ messages in thread
From: Daniele Ceraolo Spurio @ 2023-11-14 17:03 UTC (permalink / raw)
  To: Tvrtko Ursulin, intel-gfx; +Cc: Matt Roper, Alan Previn



On 11/13/2023 8:46 AM, Tvrtko Ursulin wrote:
>
> On 13/11/2023 15:51, Daniele Ceraolo Spurio wrote:
>> On 11/10/2023 4:00 AM, Tvrtko Ursulin wrote:
>>>
>>> On 09/11/2023 23:53, Daniele Ceraolo Spurio wrote:
>>>> The GSC CS is not exposed to the user, so we skipped assigning a uabi
>>>> class number for it. However, the trace logs use the uabi class and
>>>> instance to identify the engine, so leaving uabi class unset makes the
>>>> GSC CS show up as the RCS in those logs.
>>>> Given that the engine is not exposed to the user, we can't add a new
>>>> case in the uabi enum, so we insted internally define a kernel
>>>> reserved class using the next free number.
>>>>
>>>> Fixes: 194babe26bdc ("drm/i915/mtl: don't expose GSC command 
>>>> streamer to the user")
>>>> Signed-off-by: Daniele Ceraolo Spurio 
>>>> <daniele.ceraolospurio@intel.com>
>>>> Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
>>>> Cc: Alan Previn <alan.previn.teres.alexis@intel.com>
>>>> Cc: Matt Roper <matthew.d.roper@intel.com>
>>>> ---
>>>>   drivers/gpu/drm/i915/gt/intel_engine_user.c | 17 ++++++++---------
>>>>   drivers/gpu/drm/i915/gt/intel_engine_user.h |  4 ++++
>>>>   drivers/gpu/drm/i915/i915_drm_client.h      |  2 +-
>>>>   drivers/gpu/drm/i915/i915_drv.h             |  2 +-
>>>>   4 files changed, 14 insertions(+), 11 deletions(-)
>>>>
>>>> diff --git a/drivers/gpu/drm/i915/gt/intel_engine_user.c 
>>>> b/drivers/gpu/drm/i915/gt/intel_engine_user.c
>>>> index 118164ddbb2e..3fd32bedd6e7 100644
>>>> --- a/drivers/gpu/drm/i915/gt/intel_engine_user.c
>>>> +++ b/drivers/gpu/drm/i915/gt/intel_engine_user.c
>>>> @@ -47,6 +47,7 @@ static const u8 uabi_classes[] = {
>>>>       [VIDEO_DECODE_CLASS] = I915_ENGINE_CLASS_VIDEO,
>>>>       [VIDEO_ENHANCEMENT_CLASS] = I915_ENGINE_CLASS_VIDEO_ENHANCE,
>>>>       [COMPUTE_CLASS] = I915_ENGINE_CLASS_COMPUTE,
>>>> +    [OTHER_CLASS] = I915_KERNEL_RSVD_CLASS,
>>>
>>> Could we set it to -1 (aka no uabi class) to avoid needing to 
>>> maintain the new macros?
>>>
>>> And then just teach intel_engines_driver_register to skip -1. Would 
>>> also need teaching engine_rename to handle -1.
>>>
>>> Might end up a smaller and more maintainable patch - worth a try do 
>>> you think?
>>
>> That was my initial idea as well, but the issue with this approach is 
>> the engine_uabi_class_count[] array, which is sized based on the 
>> number of uabi classes, so having class -1 would needlessly increase 
>> its size a lot even when using a u8. I thought about limiting the 
>> class entry to 3 
>
> I was thinking the -1 entry wouldn't be in that array since -1 is not 
> uabi class by its very definition. It is not reachable from the 
> outside so no need to be there.

The issue there is that the array is used to assign uabi_instance, but I 
can hardcode that to 0 for now since we don't expect any new engines of 
that class. would that work for you?
Otherwise I'll try the 3-bits thing and see how that looks.

Daniele

>
>> bits so the array would max out at 8 entries, but that seemed to be 
>> getting a bit too convoluted. I can give it a go if you think it's be 
>> cleaner overall.
>
> I had a feeling it would be, but without trying it out I won't claim 
> 100%.
>
>> Note that this patch does not introduce any new macros that would 
>> need to be maintained. I915_LAST_UABI_ENGINE_CLASS already existed (I 
>> just moved it from one file to another) and is the only one that 
>> changes when a new "real" uabi class is added; the other defines are 
>> based on this one. This also implies that if a new uabi class is 
>> added then I915_KERNEL_RSVD_CLASS would be bumped to the next free 
>> number, which would cause the GSC to show as a different uabi class 
>> in newer logs; considering that i915 is on its way out, a new class 
>> seems very unlikely so I thought it'd be an acceptable compromise to 
>> keep things simple.
>>
>>>
>>>>   };
>>>>     static int engine_cmp(void *priv, const struct list_head *A,
>>>> @@ -138,7 +139,7 @@ const char *intel_engine_class_repr(u8 class)
>>>>           [COPY_ENGINE_CLASS] = "bcs",
>>>>           [VIDEO_DECODE_CLASS] = "vcs",
>>>>           [VIDEO_ENHANCEMENT_CLASS] = "vecs",
>>>> -        [OTHER_CLASS] = "other",
>>>> +        [OTHER_CLASS] = "gsc",
>>>
>>> Maybe unrelated?
>>
>> no. Before this patch, we hardcoded "gsc" below when calling 
>> engine_rename() for it. With this patch, we use the name from this 
>> array, so it needs to be updated. The GEM_WARN_ON below was added to 
>> make sure we don't get different engines in OTHER_CLASS that might 
>> not match the "gsc" naming.
>
> Ah okay, one special casing for another, a wash I guess.
>
> Regards,
>
> Tvrtko
>
>>>
>>> Regards,
>>>
>>> Tvrtko
>>>
>>>>           [COMPUTE_CLASS] = "ccs",
>>>>       };
>>>>   @@ -216,14 +217,8 @@ void intel_engines_driver_register(struct 
>>>> drm_i915_private *i915)
>>>>           if (intel_gt_has_unrecoverable_error(engine->gt))
>>>>               continue; /* ignore incomplete engines */
>>>>   -        /*
>>>> -         * We don't want to expose the GSC engine to the users, 
>>>> but we
>>>> -         * still rename it so it is easier to identify in the 
>>>> debug logs
>>>> -         */
>>>> -        if (engine->id == GSC0) {
>>>> -            engine_rename(engine, "gsc", 0);
>>>> -            continue;
>>>> -        }
>>>> +        /* The only engine we expect in OTHER_CLASS is GSC0 */
>>>> +        GEM_WARN_ON(engine->class == OTHER_CLASS && engine->id != 
>>>> GSC0);
>>>>             GEM_BUG_ON(engine->class >= ARRAY_SIZE(uabi_classes));
>>>>           engine->uabi_class = uabi_classes[engine->class];
>>>> @@ -238,6 +233,10 @@ void intel_engines_driver_register(struct 
>>>> drm_i915_private *i915)
>>>> intel_engine_class_repr(engine->class),
>>>>                     engine->uabi_instance);
>>>>   +        /* We don't want to expose the GSC engine to the users */
>>>> +        if (engine->id == GSC0)
>>>> +            continue;
>>>> +
>>>>           rb_link_node(&engine->uabi_node, prev, p);
>>>>           rb_insert_color(&engine->uabi_node, &i915->uabi_engines);
>>>>   diff --git a/drivers/gpu/drm/i915/gt/intel_engine_user.h 
>>>> b/drivers/gpu/drm/i915/gt/intel_engine_user.h
>>>> index 3dc7e8ab9fbc..dd31805b2a5a 100644
>>>> --- a/drivers/gpu/drm/i915/gt/intel_engine_user.h
>>>> +++ b/drivers/gpu/drm/i915/gt/intel_engine_user.h
>>>> @@ -11,6 +11,10 @@
>>>>   struct drm_i915_private;
>>>>   struct intel_engine_cs;
>>>>   +#define I915_LAST_UABI_ENGINE_CLASS I915_ENGINE_CLASS_COMPUTE
>>>> +#define I915_KERNEL_RSVD_CLASS (I915_LAST_UABI_ENGINE_CLASS + 1)
>>>> +#define I915_MAX_UABI_CLASSES (I915_KERNEL_RSVD_CLASS + 1)
>>>> +
>>>>   struct intel_engine_cs *
>>>>   intel_engine_lookup_user(struct drm_i915_private *i915, u8 class, 
>>>> u8 instance);
>>>>   diff --git a/drivers/gpu/drm/i915/i915_drm_client.h 
>>>> b/drivers/gpu/drm/i915/i915_drm_client.h
>>>> index 67816c912bca..c42cb2511348 100644
>>>> --- a/drivers/gpu/drm/i915/i915_drm_client.h
>>>> +++ b/drivers/gpu/drm/i915/i915_drm_client.h
>>>> @@ -12,7 +12,7 @@
>>>>     #include <uapi/drm/i915_drm.h>
>>>>   -#define I915_LAST_UABI_ENGINE_CLASS I915_ENGINE_CLASS_COMPUTE
>>>> +#include "gt/intel_engine_user.h"
>>>>     struct drm_file;
>>>>   struct drm_printer;
>>>> diff --git a/drivers/gpu/drm/i915/i915_drv.h 
>>>> b/drivers/gpu/drm/i915/i915_drv.h
>>>> index f3be9033a93f..a718b4cb5a2d 100644
>>>> --- a/drivers/gpu/drm/i915/i915_drv.h
>>>> +++ b/drivers/gpu/drm/i915/i915_drv.h
>>>> @@ -238,7 +238,7 @@ struct drm_i915_private {
>>>>           struct list_head uabi_engines_list;
>>>>           struct rb_root uabi_engines;
>>>>       };
>>>> -    unsigned int 
>>>> engine_uabi_class_count[I915_LAST_UABI_ENGINE_CLASS + 1];
>>>> +    unsigned int engine_uabi_class_count[I915_MAX_UABI_CLASSES];
>>>>         /* protects the irq masks */
>>>>       spinlock_t irq_lock;
>>


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

* Re: [Intel-gfx] [PATCH] drm/i915/gsc: Assign a uabi class number to the GSC CS
  2023-11-14 17:03       ` Daniele Ceraolo Spurio
@ 2023-11-14 17:20         ` Tvrtko Ursulin
  2023-11-14 21:09           ` Daniele Ceraolo Spurio
  0 siblings, 1 reply; 10+ messages in thread
From: Tvrtko Ursulin @ 2023-11-14 17:20 UTC (permalink / raw)
  To: Daniele Ceraolo Spurio, intel-gfx; +Cc: Matt Roper, Alan Previn


On 14/11/2023 17:03, Daniele Ceraolo Spurio wrote:
> On 11/13/2023 8:46 AM, Tvrtko Ursulin wrote:
>> On 13/11/2023 15:51, Daniele Ceraolo Spurio wrote:
>>> On 11/10/2023 4:00 AM, Tvrtko Ursulin wrote:
>>>>
>>>> On 09/11/2023 23:53, Daniele Ceraolo Spurio wrote:
>>>>> The GSC CS is not exposed to the user, so we skipped assigning a uabi
>>>>> class number for it. However, the trace logs use the uabi class and
>>>>> instance to identify the engine, so leaving uabi class unset makes the
>>>>> GSC CS show up as the RCS in those logs.
>>>>> Given that the engine is not exposed to the user, we can't add a new
>>>>> case in the uabi enum, so we insted internally define a kernel
>>>>> reserved class using the next free number.
>>>>>
>>>>> Fixes: 194babe26bdc ("drm/i915/mtl: don't expose GSC command 
>>>>> streamer to the user")
>>>>> Signed-off-by: Daniele Ceraolo Spurio 
>>>>> <daniele.ceraolospurio@intel.com>
>>>>> Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
>>>>> Cc: Alan Previn <alan.previn.teres.alexis@intel.com>
>>>>> Cc: Matt Roper <matthew.d.roper@intel.com>
>>>>> ---
>>>>>   drivers/gpu/drm/i915/gt/intel_engine_user.c | 17 ++++++++---------
>>>>>   drivers/gpu/drm/i915/gt/intel_engine_user.h |  4 ++++
>>>>>   drivers/gpu/drm/i915/i915_drm_client.h      |  2 +-
>>>>>   drivers/gpu/drm/i915/i915_drv.h             |  2 +-
>>>>>   4 files changed, 14 insertions(+), 11 deletions(-)
>>>>>
>>>>> diff --git a/drivers/gpu/drm/i915/gt/intel_engine_user.c 
>>>>> b/drivers/gpu/drm/i915/gt/intel_engine_user.c
>>>>> index 118164ddbb2e..3fd32bedd6e7 100644
>>>>> --- a/drivers/gpu/drm/i915/gt/intel_engine_user.c
>>>>> +++ b/drivers/gpu/drm/i915/gt/intel_engine_user.c
>>>>> @@ -47,6 +47,7 @@ static const u8 uabi_classes[] = {
>>>>>       [VIDEO_DECODE_CLASS] = I915_ENGINE_CLASS_VIDEO,
>>>>>       [VIDEO_ENHANCEMENT_CLASS] = I915_ENGINE_CLASS_VIDEO_ENHANCE,
>>>>>       [COMPUTE_CLASS] = I915_ENGINE_CLASS_COMPUTE,
>>>>> +    [OTHER_CLASS] = I915_KERNEL_RSVD_CLASS,
>>>>
>>>> Could we set it to -1 (aka no uabi class) to avoid needing to 
>>>> maintain the new macros?
>>>>
>>>> And then just teach intel_engines_driver_register to skip -1. Would 
>>>> also need teaching engine_rename to handle -1.
>>>>
>>>> Might end up a smaller and more maintainable patch - worth a try do 
>>>> you think?
>>>
>>> That was my initial idea as well, but the issue with this approach is 
>>> the engine_uabi_class_count[] array, which is sized based on the 
>>> number of uabi classes, so having class -1 would needlessly increase 
>>> its size a lot even when using a u8. I thought about limiting the 
>>> class entry to 3 
>>
>> I was thinking the -1 entry wouldn't be in that array since -1 is not 
>> uabi class by its very definition. It is not reachable from the 
>> outside so no need to be there.
> 
> The issue there is that the array is used to assign uabi_instance, but I 
> can hardcode that to 0 for now since we don't expect any new engines of 
> that class. would that work for you?

My issue is that the starting position here is GSC engine has no uabi 
class, therefore it shouldn't have an uabi instance either, therefore 
the fact you cannot use the array called engine_*uabi*_class_count to 
assign it something which it shouldn't have, shouldn't be an argument. ;)

> Otherwise I'll try the 3-bits thing and see how that looks.

3-bits.. I have to admit I don't know what you mean by that?

I can try to sketch out tomorrow what I have in mind. Lets go from 
there. I might fail and concede there is nothing to be gained by using a 
different approach.

Regards,

Tvrtko

> 
> Daniele
> 
>>
>>> bits so the array would max out at 8 entries, but that seemed to be 
>>> getting a bit too convoluted. I can give it a go if you think it's be 
>>> cleaner overall.
>>
>> I had a feeling it would be, but without trying it out I won't claim 
>> 100%.
>>
>>> Note that this patch does not introduce any new macros that would 
>>> need to be maintained. I915_LAST_UABI_ENGINE_CLASS already existed (I 
>>> just moved it from one file to another) and is the only one that 
>>> changes when a new "real" uabi class is added; the other defines are 
>>> based on this one. This also implies that if a new uabi class is 
>>> added then I915_KERNEL_RSVD_CLASS would be bumped to the next free 
>>> number, which would cause the GSC to show as a different uabi class 
>>> in newer logs; considering that i915 is on its way out, a new class 
>>> seems very unlikely so I thought it'd be an acceptable compromise to 
>>> keep things simple.
>>>
>>>>
>>>>>   };
>>>>>     static int engine_cmp(void *priv, const struct list_head *A,
>>>>> @@ -138,7 +139,7 @@ const char *intel_engine_class_repr(u8 class)
>>>>>           [COPY_ENGINE_CLASS] = "bcs",
>>>>>           [VIDEO_DECODE_CLASS] = "vcs",
>>>>>           [VIDEO_ENHANCEMENT_CLASS] = "vecs",
>>>>> -        [OTHER_CLASS] = "other",
>>>>> +        [OTHER_CLASS] = "gsc",
>>>>
>>>> Maybe unrelated?
>>>
>>> no. Before this patch, we hardcoded "gsc" below when calling 
>>> engine_rename() for it. With this patch, we use the name from this 
>>> array, so it needs to be updated. The GEM_WARN_ON below was added to 
>>> make sure we don't get different engines in OTHER_CLASS that might 
>>> not match the "gsc" naming.
>>
>> Ah okay, one special casing for another, a wash I guess.
>>
>> Regards,
>>
>> Tvrtko
>>
>>>>
>>>> Regards,
>>>>
>>>> Tvrtko
>>>>
>>>>>           [COMPUTE_CLASS] = "ccs",
>>>>>       };
>>>>>   @@ -216,14 +217,8 @@ void intel_engines_driver_register(struct 
>>>>> drm_i915_private *i915)
>>>>>           if (intel_gt_has_unrecoverable_error(engine->gt))
>>>>>               continue; /* ignore incomplete engines */
>>>>>   -        /*
>>>>> -         * We don't want to expose the GSC engine to the users, 
>>>>> but we
>>>>> -         * still rename it so it is easier to identify in the 
>>>>> debug logs
>>>>> -         */
>>>>> -        if (engine->id == GSC0) {
>>>>> -            engine_rename(engine, "gsc", 0);
>>>>> -            continue;
>>>>> -        }
>>>>> +        /* The only engine we expect in OTHER_CLASS is GSC0 */
>>>>> +        GEM_WARN_ON(engine->class == OTHER_CLASS && engine->id != 
>>>>> GSC0);
>>>>>             GEM_BUG_ON(engine->class >= ARRAY_SIZE(uabi_classes));
>>>>>           engine->uabi_class = uabi_classes[engine->class];
>>>>> @@ -238,6 +233,10 @@ void intel_engines_driver_register(struct 
>>>>> drm_i915_private *i915)
>>>>> intel_engine_class_repr(engine->class),
>>>>>                     engine->uabi_instance);
>>>>>   +        /* We don't want to expose the GSC engine to the users */
>>>>> +        if (engine->id == GSC0)
>>>>> +            continue;
>>>>> +
>>>>>           rb_link_node(&engine->uabi_node, prev, p);
>>>>>           rb_insert_color(&engine->uabi_node, &i915->uabi_engines);
>>>>>   diff --git a/drivers/gpu/drm/i915/gt/intel_engine_user.h 
>>>>> b/drivers/gpu/drm/i915/gt/intel_engine_user.h
>>>>> index 3dc7e8ab9fbc..dd31805b2a5a 100644
>>>>> --- a/drivers/gpu/drm/i915/gt/intel_engine_user.h
>>>>> +++ b/drivers/gpu/drm/i915/gt/intel_engine_user.h
>>>>> @@ -11,6 +11,10 @@
>>>>>   struct drm_i915_private;
>>>>>   struct intel_engine_cs;
>>>>>   +#define I915_LAST_UABI_ENGINE_CLASS I915_ENGINE_CLASS_COMPUTE
>>>>> +#define I915_KERNEL_RSVD_CLASS (I915_LAST_UABI_ENGINE_CLASS + 1)
>>>>> +#define I915_MAX_UABI_CLASSES (I915_KERNEL_RSVD_CLASS + 1)
>>>>> +
>>>>>   struct intel_engine_cs *
>>>>>   intel_engine_lookup_user(struct drm_i915_private *i915, u8 class, 
>>>>> u8 instance);
>>>>>   diff --git a/drivers/gpu/drm/i915/i915_drm_client.h 
>>>>> b/drivers/gpu/drm/i915/i915_drm_client.h
>>>>> index 67816c912bca..c42cb2511348 100644
>>>>> --- a/drivers/gpu/drm/i915/i915_drm_client.h
>>>>> +++ b/drivers/gpu/drm/i915/i915_drm_client.h
>>>>> @@ -12,7 +12,7 @@
>>>>>     #include <uapi/drm/i915_drm.h>
>>>>>   -#define I915_LAST_UABI_ENGINE_CLASS I915_ENGINE_CLASS_COMPUTE
>>>>> +#include "gt/intel_engine_user.h"
>>>>>     struct drm_file;
>>>>>   struct drm_printer;
>>>>> diff --git a/drivers/gpu/drm/i915/i915_drv.h 
>>>>> b/drivers/gpu/drm/i915/i915_drv.h
>>>>> index f3be9033a93f..a718b4cb5a2d 100644
>>>>> --- a/drivers/gpu/drm/i915/i915_drv.h
>>>>> +++ b/drivers/gpu/drm/i915/i915_drv.h
>>>>> @@ -238,7 +238,7 @@ struct drm_i915_private {
>>>>>           struct list_head uabi_engines_list;
>>>>>           struct rb_root uabi_engines;
>>>>>       };
>>>>> -    unsigned int 
>>>>> engine_uabi_class_count[I915_LAST_UABI_ENGINE_CLASS + 1];
>>>>> +    unsigned int engine_uabi_class_count[I915_MAX_UABI_CLASSES];
>>>>>         /* protects the irq masks */
>>>>>       spinlock_t irq_lock;
>>>
> 

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

* Re: [Intel-gfx] [PATCH] drm/i915/gsc: Assign a uabi class number to the GSC CS
  2023-11-14 17:20         ` Tvrtko Ursulin
@ 2023-11-14 21:09           ` Daniele Ceraolo Spurio
  0 siblings, 0 replies; 10+ messages in thread
From: Daniele Ceraolo Spurio @ 2023-11-14 21:09 UTC (permalink / raw)
  To: Tvrtko Ursulin, intel-gfx; +Cc: Matt Roper, Alan Previn



On 11/14/2023 9:20 AM, Tvrtko Ursulin wrote:
>
> On 14/11/2023 17:03, Daniele Ceraolo Spurio wrote:
>> On 11/13/2023 8:46 AM, Tvrtko Ursulin wrote:
>>> On 13/11/2023 15:51, Daniele Ceraolo Spurio wrote:
>>>> On 11/10/2023 4:00 AM, Tvrtko Ursulin wrote:
>>>>>
>>>>> On 09/11/2023 23:53, Daniele Ceraolo Spurio wrote:
>>>>>> The GSC CS is not exposed to the user, so we skipped assigning a 
>>>>>> uabi
>>>>>> class number for it. However, the trace logs use the uabi class and
>>>>>> instance to identify the engine, so leaving uabi class unset 
>>>>>> makes the
>>>>>> GSC CS show up as the RCS in those logs.
>>>>>> Given that the engine is not exposed to the user, we can't add a new
>>>>>> case in the uabi enum, so we insted internally define a kernel
>>>>>> reserved class using the next free number.
>>>>>>
>>>>>> Fixes: 194babe26bdc ("drm/i915/mtl: don't expose GSC command 
>>>>>> streamer to the user")
>>>>>> Signed-off-by: Daniele Ceraolo Spurio 
>>>>>> <daniele.ceraolospurio@intel.com>
>>>>>> Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
>>>>>> Cc: Alan Previn <alan.previn.teres.alexis@intel.com>
>>>>>> Cc: Matt Roper <matthew.d.roper@intel.com>
>>>>>> ---
>>>>>>   drivers/gpu/drm/i915/gt/intel_engine_user.c | 17 ++++++++---------
>>>>>>   drivers/gpu/drm/i915/gt/intel_engine_user.h |  4 ++++
>>>>>>   drivers/gpu/drm/i915/i915_drm_client.h      |  2 +-
>>>>>>   drivers/gpu/drm/i915/i915_drv.h             |  2 +-
>>>>>>   4 files changed, 14 insertions(+), 11 deletions(-)
>>>>>>
>>>>>> diff --git a/drivers/gpu/drm/i915/gt/intel_engine_user.c 
>>>>>> b/drivers/gpu/drm/i915/gt/intel_engine_user.c
>>>>>> index 118164ddbb2e..3fd32bedd6e7 100644
>>>>>> --- a/drivers/gpu/drm/i915/gt/intel_engine_user.c
>>>>>> +++ b/drivers/gpu/drm/i915/gt/intel_engine_user.c
>>>>>> @@ -47,6 +47,7 @@ static const u8 uabi_classes[] = {
>>>>>>       [VIDEO_DECODE_CLASS] = I915_ENGINE_CLASS_VIDEO,
>>>>>>       [VIDEO_ENHANCEMENT_CLASS] = I915_ENGINE_CLASS_VIDEO_ENHANCE,
>>>>>>       [COMPUTE_CLASS] = I915_ENGINE_CLASS_COMPUTE,
>>>>>> +    [OTHER_CLASS] = I915_KERNEL_RSVD_CLASS,
>>>>>
>>>>> Could we set it to -1 (aka no uabi class) to avoid needing to 
>>>>> maintain the new macros?
>>>>>
>>>>> And then just teach intel_engines_driver_register to skip -1. 
>>>>> Would also need teaching engine_rename to handle -1.
>>>>>
>>>>> Might end up a smaller and more maintainable patch - worth a try 
>>>>> do you think?
>>>>
>>>> That was my initial idea as well, but the issue with this approach 
>>>> is the engine_uabi_class_count[] array, which is sized based on the 
>>>> number of uabi classes, so having class -1 would needlessly 
>>>> increase its size a lot even when using a u8. I thought about 
>>>> limiting the class entry to 3 
>>>
>>> I was thinking the -1 entry wouldn't be in that array since -1 is 
>>> not uabi class by its very definition. It is not reachable from the 
>>> outside so no need to be there.
>>
>> The issue there is that the array is used to assign uabi_instance, 
>> but I can hardcode that to 0 for now since we don't expect any new 
>> engines of that class. would that work for you?
>
> My issue is that the starting position here is GSC engine has no uabi 
> class, therefore it shouldn't have an uabi instance either, therefore 
> the fact you cannot use the array called engine_*uabi*_class_count to 
> assign it something which it shouldn't have, shouldn't be an argument. ;)

Agreed. The problem is that we use the uabi_class/instance for things 
that aren't strictly uabi (i.e. the traces) so we need to assign a value 
to those even if we don't need it for actual uabi purposes.

>
>> Otherwise I'll try the 3-bits thing and see how that looks.
>
> 3-bits.. I have to admit I don't know what you mean by that?

I was referring to limiting the uabi class index to 3 bits to limit the 
max array size to 8 and use -1 for the GSC uabi_class.

>
> I can try to sketch out tomorrow what I have in mind. Lets go from 
> there. I might fail and concede there is nothing to be gained by using 
> a different approach.

sounds good.

Thanks,
Daniele

>
> Regards,
>
> Tvrtko
>
>>
>> Daniele
>>
>>>
>>>> bits so the array would max out at 8 entries, but that seemed to be 
>>>> getting a bit too convoluted. I can give it a go if you think it's 
>>>> be cleaner overall.
>>>
>>> I had a feeling it would be, but without trying it out I won't claim 
>>> 100%.
>>>
>>>> Note that this patch does not introduce any new macros that would 
>>>> need to be maintained. I915_LAST_UABI_ENGINE_CLASS already existed 
>>>> (I just moved it from one file to another) and is the only one that 
>>>> changes when a new "real" uabi class is added; the other defines 
>>>> are based on this one. This also implies that if a new uabi class 
>>>> is added then I915_KERNEL_RSVD_CLASS would be bumped to the next 
>>>> free number, which would cause the GSC to show as a different uabi 
>>>> class in newer logs; considering that i915 is on its way out, a new 
>>>> class seems very unlikely so I thought it'd be an acceptable 
>>>> compromise to keep things simple.
>>>>
>>>>>
>>>>>>   };
>>>>>>     static int engine_cmp(void *priv, const struct list_head *A,
>>>>>> @@ -138,7 +139,7 @@ const char *intel_engine_class_repr(u8 class)
>>>>>>           [COPY_ENGINE_CLASS] = "bcs",
>>>>>>           [VIDEO_DECODE_CLASS] = "vcs",
>>>>>>           [VIDEO_ENHANCEMENT_CLASS] = "vecs",
>>>>>> -        [OTHER_CLASS] = "other",
>>>>>> +        [OTHER_CLASS] = "gsc",
>>>>>
>>>>> Maybe unrelated?
>>>>
>>>> no. Before this patch, we hardcoded "gsc" below when calling 
>>>> engine_rename() for it. With this patch, we use the name from this 
>>>> array, so it needs to be updated. The GEM_WARN_ON below was added 
>>>> to make sure we don't get different engines in OTHER_CLASS that 
>>>> might not match the "gsc" naming.
>>>
>>> Ah okay, one special casing for another, a wash I guess.
>>>
>>> Regards,
>>>
>>> Tvrtko
>>>
>>>>>
>>>>> Regards,
>>>>>
>>>>> Tvrtko
>>>>>
>>>>>>           [COMPUTE_CLASS] = "ccs",
>>>>>>       };
>>>>>>   @@ -216,14 +217,8 @@ void intel_engines_driver_register(struct 
>>>>>> drm_i915_private *i915)
>>>>>>           if (intel_gt_has_unrecoverable_error(engine->gt))
>>>>>>               continue; /* ignore incomplete engines */
>>>>>>   -        /*
>>>>>> -         * We don't want to expose the GSC engine to the users, 
>>>>>> but we
>>>>>> -         * still rename it so it is easier to identify in the 
>>>>>> debug logs
>>>>>> -         */
>>>>>> -        if (engine->id == GSC0) {
>>>>>> -            engine_rename(engine, "gsc", 0);
>>>>>> -            continue;
>>>>>> -        }
>>>>>> +        /* The only engine we expect in OTHER_CLASS is GSC0 */
>>>>>> +        GEM_WARN_ON(engine->class == OTHER_CLASS && engine->id 
>>>>>> != GSC0);
>>>>>>             GEM_BUG_ON(engine->class >= ARRAY_SIZE(uabi_classes));
>>>>>>           engine->uabi_class = uabi_classes[engine->class];
>>>>>> @@ -238,6 +233,10 @@ void intel_engines_driver_register(struct 
>>>>>> drm_i915_private *i915)
>>>>>> intel_engine_class_repr(engine->class),
>>>>>>                     engine->uabi_instance);
>>>>>>   +        /* We don't want to expose the GSC engine to the users */
>>>>>> +        if (engine->id == GSC0)
>>>>>> +            continue;
>>>>>> +
>>>>>>           rb_link_node(&engine->uabi_node, prev, p);
>>>>>>           rb_insert_color(&engine->uabi_node, &i915->uabi_engines);
>>>>>>   diff --git a/drivers/gpu/drm/i915/gt/intel_engine_user.h 
>>>>>> b/drivers/gpu/drm/i915/gt/intel_engine_user.h
>>>>>> index 3dc7e8ab9fbc..dd31805b2a5a 100644
>>>>>> --- a/drivers/gpu/drm/i915/gt/intel_engine_user.h
>>>>>> +++ b/drivers/gpu/drm/i915/gt/intel_engine_user.h
>>>>>> @@ -11,6 +11,10 @@
>>>>>>   struct drm_i915_private;
>>>>>>   struct intel_engine_cs;
>>>>>>   +#define I915_LAST_UABI_ENGINE_CLASS I915_ENGINE_CLASS_COMPUTE
>>>>>> +#define I915_KERNEL_RSVD_CLASS (I915_LAST_UABI_ENGINE_CLASS + 1)
>>>>>> +#define I915_MAX_UABI_CLASSES (I915_KERNEL_RSVD_CLASS + 1)
>>>>>> +
>>>>>>   struct intel_engine_cs *
>>>>>>   intel_engine_lookup_user(struct drm_i915_private *i915, u8 
>>>>>> class, u8 instance);
>>>>>>   diff --git a/drivers/gpu/drm/i915/i915_drm_client.h 
>>>>>> b/drivers/gpu/drm/i915/i915_drm_client.h
>>>>>> index 67816c912bca..c42cb2511348 100644
>>>>>> --- a/drivers/gpu/drm/i915/i915_drm_client.h
>>>>>> +++ b/drivers/gpu/drm/i915/i915_drm_client.h
>>>>>> @@ -12,7 +12,7 @@
>>>>>>     #include <uapi/drm/i915_drm.h>
>>>>>>   -#define I915_LAST_UABI_ENGINE_CLASS I915_ENGINE_CLASS_COMPUTE
>>>>>> +#include "gt/intel_engine_user.h"
>>>>>>     struct drm_file;
>>>>>>   struct drm_printer;
>>>>>> diff --git a/drivers/gpu/drm/i915/i915_drv.h 
>>>>>> b/drivers/gpu/drm/i915/i915_drv.h
>>>>>> index f3be9033a93f..a718b4cb5a2d 100644
>>>>>> --- a/drivers/gpu/drm/i915/i915_drv.h
>>>>>> +++ b/drivers/gpu/drm/i915/i915_drv.h
>>>>>> @@ -238,7 +238,7 @@ struct drm_i915_private {
>>>>>>           struct list_head uabi_engines_list;
>>>>>>           struct rb_root uabi_engines;
>>>>>>       };
>>>>>> -    unsigned int 
>>>>>> engine_uabi_class_count[I915_LAST_UABI_ENGINE_CLASS + 1];
>>>>>> +    unsigned int engine_uabi_class_count[I915_MAX_UABI_CLASSES];
>>>>>>         /* protects the irq masks */
>>>>>>       spinlock_t irq_lock;
>>>>
>>


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

end of thread, other threads:[~2023-11-14 21:09 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-11-09 23:53 [Intel-gfx] [PATCH] drm/i915/gsc: Assign a uabi class number to the GSC CS Daniele Ceraolo Spurio
2023-11-10  4:07 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for " Patchwork
2023-11-10  4:20 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2023-11-10 10:25 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
2023-11-10 12:00 ` [Intel-gfx] [PATCH] " Tvrtko Ursulin
2023-11-13 15:51   ` Daniele Ceraolo Spurio
2023-11-13 16:46     ` Tvrtko Ursulin
2023-11-14 17:03       ` Daniele Ceraolo Spurio
2023-11-14 17:20         ` Tvrtko Ursulin
2023-11-14 21:09           ` Daniele Ceraolo Spurio

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).