All of lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D
@ 2022-12-30  6:46 bhadanednyaneshwar
  2022-12-30 10:27 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
                   ` (9 more replies)
  0 siblings, 10 replies; 17+ messages in thread
From: bhadanednyaneshwar @ 2022-12-30  6:46 UTC (permalink / raw)
  To: igt-dev; +Cc: suresh.kumar.kurmi, dnyaneshwar.bhadane, chaitanya.kumar.borah

MSO eDP is not supported on pipe C and D. Added a test condition
to prevent tests from execution on pipe C and D.This condition was
missed for cursor-size-change,cursor-alpha-opaque and
cursor-alpha-transparent testcases.

Inside require_cursor_size() checks first for eligiblity to igt commit
using test buffer.For MSO eDP, It is fail to commit for pipe C/pipe D
and require_cursor_size() return non zero value. So it will skip
the dynamic testcase for pipe C and D.

Signed-off-by: bhadanednyaneshwar <dnyaneshwar.bhadane@intel.com>
---
 tests/kms_cursor_crc.c | 21 ++++++++++++++++++---
 1 file changed, 18 insertions(+), 3 deletions(-)

diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c
index 17f294d6..d8fb9c0d 100644
--- a/tests/kms_cursor_crc.c
+++ b/tests/kms_cursor_crc.c
@@ -786,7 +786,12 @@ static void run_tests_on_pipe(data_t *data)
 	igt_subtest_with_dynamic("cursor-size-change") {
 		for_each_pipe(&data->display, pipe) {
 			data->pipe = pipe;
-
+			create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
+			if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) {
+				igt_debug("Cursor size %dx%d not supported by driver\n",
+					  data->cursor_max_w, data->cursor_max_h);
+				continue;
+			}
 			igt_dynamic_f("pipe-%s-%s",
 				      kmstest_pipe_name(pipe),
 				      data->output->name)
@@ -800,7 +805,12 @@ static void run_tests_on_pipe(data_t *data)
 	igt_subtest_with_dynamic("cursor-alpha-opaque") {
 		for_each_pipe(&data->display, pipe) {
 			data->pipe = pipe;
-
+			create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
+			if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) {
+				igt_debug("Cursor size %dx%d not supported by driver\n",
+					  data->cursor_max_w, data->cursor_max_h);
+				continue;
+			}
 			igt_dynamic_f("pipe-%s-%s",
 				      kmstest_pipe_name(pipe),
 				      data->output->name)
@@ -814,7 +824,12 @@ static void run_tests_on_pipe(data_t *data)
 	igt_subtest_with_dynamic("cursor-alpha-transparent") {
 		for_each_pipe(&data->display, pipe) {
 			data->pipe = pipe;
-
+			create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
+			if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) {
+				igt_debug("Cursor size %dx%d not supported by driver\n",
+					  data->cursor_max_w, data->cursor_max_h);
+				continue;
+			}
 			igt_dynamic_f("pipe-%s-%s",
 				      kmstest_pipe_name(pipe),
 				      data->output->name)
-- 
2.35.1

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

* [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D
  2022-12-30  6:46 [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D bhadanednyaneshwar
@ 2022-12-30 10:27 ` Patchwork
  2022-12-30 11:49 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
                   ` (8 subsequent siblings)
  9 siblings, 0 replies; 17+ messages in thread
From: Patchwork @ 2022-12-30 10:27 UTC (permalink / raw)
  To: bhadanednyaneshwar; +Cc: igt-dev

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

== Series Details ==

Series: tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D
URL   : https://patchwork.freedesktop.org/series/112305/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_12530 -> IGTPW_8285
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

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

Participating hosts (43 -> 42)
------------------------------

  Additional (1): fi-bsw-kefka 
  Missing    (2): fi-hsw-4770 fi-snb-2520m 

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

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

### IGT changes ###

#### Issues hit ####

  * igt@fbdev@write:
    - fi-blb-e6850:       [PASS][1] -> [SKIP][2] ([fdo#109271]) +4 similar issues
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/fi-blb-e6850/igt@fbdev@write.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/fi-blb-e6850/igt@fbdev@write.html

  * igt@gem_exec_gttfill@basic:
    - fi-pnv-d510:        [PASS][3] -> [FAIL][4] ([i915#7229])
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/fi-pnv-d510/igt@gem_exec_gttfill@basic.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/fi-pnv-d510/igt@gem_exec_gttfill@basic.html

  * igt@kms_chamelium@hdmi-hpd-fast:
    - fi-bsw-kefka:       NOTRUN -> [SKIP][5] ([fdo#109271] / [fdo#111827]) +8 similar issues
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/fi-bsw-kefka/igt@kms_chamelium@hdmi-hpd-fast.html

  * igt@prime_vgem@basic-fence-flip:
    - fi-bsw-kefka:       NOTRUN -> [SKIP][6] ([fdo#109271]) +17 similar issues
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/fi-bsw-kefka/igt@prime_vgem@basic-fence-flip.html

  
#### Possible fixes ####

  * igt@gem_exec_suspend@basic-s3@smem:
    - {bat-rpls-1}:       [DMESG-WARN][7] ([i915#6687]) -> [PASS][8]
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/bat-rpls-1/igt@gem_exec_suspend@basic-s3@smem.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/bat-rpls-1/igt@gem_exec_suspend@basic-s3@smem.html

  * igt@i915_selftest@live@reset:
    - {bat-rpls-2}:       [DMESG-FAIL][9] ([i915#4983]) -> [PASS][10]
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/bat-rpls-2/igt@i915_selftest@live@reset.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/bat-rpls-2/igt@i915_selftest@live@reset.html

  * igt@i915_selftest@live@slpc:
    - {bat-rpls-1}:       [DMESG-FAIL][11] ([i915#6367]) -> [PASS][12]
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/bat-rpls-1/igt@i915_selftest@live@slpc.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/bat-rpls-1/igt@i915_selftest@live@slpc.html
    - {bat-adln-1}:       [DMESG-FAIL][13] ([i915#6997]) -> [PASS][14]
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/bat-adln-1/igt@i915_selftest@live@slpc.html
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/bat-adln-1/igt@i915_selftest@live@slpc.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#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295
  [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
  [i915#1845]: https://gitlab.freedesktop.org/drm/intel/issues/1845
  [i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
  [i915#4983]: https://gitlab.freedesktop.org/drm/intel/issues/4983
  [i915#5153]: https://gitlab.freedesktop.org/drm/intel/issues/5153
  [i915#6078]: https://gitlab.freedesktop.org/drm/intel/issues/6078
  [i915#6106]: https://gitlab.freedesktop.org/drm/intel/issues/6106
  [i915#6367]: https://gitlab.freedesktop.org/drm/intel/issues/6367
  [i915#6687]: https://gitlab.freedesktop.org/drm/intel/issues/6687
  [i915#6997]: https://gitlab.freedesktop.org/drm/intel/issues/6997
  [i915#7229]: https://gitlab.freedesktop.org/drm/intel/issues/7229


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

  * CI: CI-20190529 -> None
  * IGT: IGT_7104 -> IGTPW_8285

  CI-20190529: 20190529
  CI_DRM_12530: c112aa73981bbf79959a95744af3f3fbbeca2197 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_8285: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/index.html
  IGT_7104: fe5def13049225967770eaaf19ec01ef80e2adc5 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git


Testlist changes
----------------

-igt@v3d/v3d_mmap@mmap-bad-flags
-igt@v3d/v3d_mmap@mmap-bo
-igt@vc4/vc4_mmap@mmap-bad-handle
-igt@vc4/vc4_mmap@mmap-bo

== Logs ==

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

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

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

* [igt-dev] ✓ Fi.CI.IGT: success for tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D
  2022-12-30  6:46 [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D bhadanednyaneshwar
  2022-12-30 10:27 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
@ 2022-12-30 11:49 ` Patchwork
  2022-12-30 19:16 ` [igt-dev] [i-g-t] " Swati Sharma
                   ` (7 subsequent siblings)
  9 siblings, 0 replies; 17+ messages in thread
From: Patchwork @ 2022-12-30 11:49 UTC (permalink / raw)
  To: bhadanednyaneshwar; +Cc: igt-dev

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

== Series Details ==

Series: tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D
URL   : https://patchwork.freedesktop.org/series/112305/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_12530_full -> IGTPW_8285_full
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

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

Participating hosts (14 -> 10)
------------------------------

  Missing    (4): shard-rkl0 pig-kbl-iris pig-glk-j5005 pig-skl-6260u 

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

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

### IGT changes ###

#### Suppressed ####

  The following results come from untrusted machines, tests, or statuses.
  They do not affect the overall result.

  * igt@perf_pmu@busy-double-start@rcs0:
    - {shard-dg1}:        [FAIL][1] ([i915#4349]) -> [FAIL][2]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-dg1-18/igt@perf_pmu@busy-double-start@rcs0.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-dg1-15/igt@perf_pmu@busy-double-start@rcs0.html

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

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

### IGT changes ###

#### Issues hit ####

  * igt@gem_ctx_persistence@legacy-engines-cleanup:
    - shard-snb:          NOTRUN -> [SKIP][3] ([fdo#109271] / [i915#1099])
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-snb2/igt@gem_ctx_persistence@legacy-engines-cleanup.html

  * igt@gem_huc_copy@huc-copy:
    - shard-apl:          NOTRUN -> [SKIP][4] ([fdo#109271] / [i915#2190])
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl6/igt@gem_huc_copy@huc-copy.html

  * igt@gem_lmem_swapping@parallel-random-verify-ccs:
    - shard-apl:          NOTRUN -> [SKIP][5] ([fdo#109271] / [i915#4613])
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl3/igt@gem_lmem_swapping@parallel-random-verify-ccs.html

  * igt@gem_partial_pwrite_pread@writes-after-reads-snoop:
    - shard-apl:          [PASS][6] -> [INCOMPLETE][7] ([i915#7708])
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-apl6/igt@gem_partial_pwrite_pread@writes-after-reads-snoop.html
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl3/igt@gem_partial_pwrite_pread@writes-after-reads-snoop.html

  * igt@i915_selftest@live@gt_heartbeat:
    - shard-apl:          [PASS][8] -> [DMESG-FAIL][9] ([i915#5334])
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-apl3/igt@i915_selftest@live@gt_heartbeat.html
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl3/igt@i915_selftest@live@gt_heartbeat.html

  * igt@kms_ccs@pipe-a-bad-aux-stride-y_tiled_gen12_rc_ccs_cc:
    - shard-glk:          NOTRUN -> [SKIP][10] ([fdo#109271] / [i915#3886])
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-glk8/igt@kms_ccs@pipe-a-bad-aux-stride-y_tiled_gen12_rc_ccs_cc.html

  * igt@kms_ccs@pipe-b-crc-sprite-planes-basic-y_tiled_gen12_mc_ccs:
    - shard-apl:          NOTRUN -> [SKIP][11] ([fdo#109271] / [i915#3886]) +7 similar issues
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl2/igt@kms_ccs@pipe-b-crc-sprite-planes-basic-y_tiled_gen12_mc_ccs.html

  * igt@kms_ccs@pipe-b-random-ccs-data-4_tiled_dg2_rc_ccs:
    - shard-apl:          NOTRUN -> [SKIP][12] ([fdo#109271]) +154 similar issues
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl3/igt@kms_ccs@pipe-b-random-ccs-data-4_tiled_dg2_rc_ccs.html

  * igt@kms_chamelium@dp-edid-stress-resolution-non-4k:
    - shard-glk:          NOTRUN -> [SKIP][13] ([fdo#109271] / [fdo#111827]) +2 similar issues
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-glk5/igt@kms_chamelium@dp-edid-stress-resolution-non-4k.html

  * igt@kms_chamelium@vga-hpd-after-suspend:
    - shard-apl:          NOTRUN -> [SKIP][14] ([fdo#109271] / [fdo#111827]) +6 similar issues
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl3/igt@kms_chamelium@vga-hpd-after-suspend.html
    - shard-snb:          NOTRUN -> [SKIP][15] ([fdo#109271] / [fdo#111827]) +2 similar issues
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-snb2/igt@kms_chamelium@vga-hpd-after-suspend.html

  * igt@kms_cursor_legacy@2x-long-flip-vs-cursor-legacy:
    - shard-glk:          [PASS][16] -> [FAIL][17] ([i915#72])
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-glk6/igt@kms_cursor_legacy@2x-long-flip-vs-cursor-legacy.html
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-glk2/igt@kms_cursor_legacy@2x-long-flip-vs-cursor-legacy.html

  * igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions:
    - shard-apl:          [PASS][18] -> [FAIL][19] ([i915#2346])
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-apl7/igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions.html
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl1/igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions.html

  * igt@kms_psr2_sf@overlay-plane-move-continuous-sf:
    - shard-apl:          NOTRUN -> [SKIP][20] ([fdo#109271] / [i915#658]) +2 similar issues
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl3/igt@kms_psr2_sf@overlay-plane-move-continuous-sf.html

  * igt@kms_psr2_su@frontbuffer-xrgb8888:
    - shard-glk:          NOTRUN -> [SKIP][21] ([fdo#109271] / [i915#658]) +1 similar issue
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-glk4/igt@kms_psr2_su@frontbuffer-xrgb8888.html

  * igt@kms_rotation_crc@sprite-rotation-270:
    - shard-snb:          NOTRUN -> [SKIP][22] ([fdo#109271]) +85 similar issues
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-snb7/igt@kms_rotation_crc@sprite-rotation-270.html

  * igt@kms_vblank@pipe-d-wait-idle:
    - shard-apl:          NOTRUN -> [SKIP][23] ([fdo#109271] / [i915#533])
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl2/igt@kms_vblank@pipe-d-wait-idle.html

  * igt@kms_vrr@negative-basic:
    - shard-glk:          NOTRUN -> [SKIP][24] ([fdo#109271]) +47 similar issues
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-glk7/igt@kms_vrr@negative-basic.html

  * igt@kms_writeback@writeback-invalid-parameters:
    - shard-apl:          NOTRUN -> [SKIP][25] ([fdo#109271] / [i915#2437])
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl1/igt@kms_writeback@writeback-invalid-parameters.html

  * igt@sysfs_clients@recycle-many:
    - shard-apl:          NOTRUN -> [SKIP][26] ([fdo#109271] / [i915#2994]) +1 similar issue
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl1/igt@sysfs_clients@recycle-many.html
    - shard-glk:          NOTRUN -> [SKIP][27] ([fdo#109271] / [i915#2994])
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-glk2/igt@sysfs_clients@recycle-many.html

  
#### Possible fixes ####

  * igt@drm_read@short-buffer-nonblock:
    - {shard-rkl}:        [SKIP][28] ([i915#4098]) -> [PASS][29]
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-rkl-2/igt@drm_read@short-buffer-nonblock.html
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-rkl-6/igt@drm_read@short-buffer-nonblock.html

  * igt@fbdev@read:
    - {shard-dg1}:        [FAIL][30] -> [PASS][31]
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-dg1-15/igt@fbdev@read.html
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-dg1-19/igt@fbdev@read.html
    - shard-apl:          [FAIL][32] -> [PASS][33]
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-apl1/igt@fbdev@read.html
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl3/igt@fbdev@read.html
    - shard-glk:          [FAIL][34] -> [PASS][35]
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-glk7/igt@fbdev@read.html
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-glk1/igt@fbdev@read.html

  * igt@gem_eio@unwedge-stress:
    - {shard-dg1}:        [FAIL][36] ([i915#5784]) -> [PASS][37]
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-dg1-16/igt@gem_eio@unwedge-stress.html
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-dg1-18/igt@gem_eio@unwedge-stress.html

  * igt@gem_exec_fair@basic-deadline:
    - {shard-rkl}:        [FAIL][38] ([i915#2846]) -> [PASS][39]
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-rkl-1/igt@gem_exec_fair@basic-deadline.html
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-rkl-5/igt@gem_exec_fair@basic-deadline.html

  * igt@gem_exec_fair@basic-none-share@rcs0:
    - shard-glk:          [FAIL][40] ([i915#2842]) -> [PASS][41]
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-glk8/igt@gem_exec_fair@basic-none-share@rcs0.html
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-glk8/igt@gem_exec_fair@basic-none-share@rcs0.html

  * igt@gem_exec_fair@basic-pace-share@rcs0:
    - {shard-rkl}:        [FAIL][42] ([i915#2842]) -> [PASS][43]
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-rkl-6/igt@gem_exec_fair@basic-pace-share@rcs0.html
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-rkl-2/igt@gem_exec_fair@basic-pace-share@rcs0.html

  * igt@gem_exec_reloc@basic-write-read-active:
    - {shard-rkl}:        [SKIP][44] ([i915#3281]) -> [PASS][45] +5 similar issues
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-rkl-2/igt@gem_exec_reloc@basic-write-read-active.html
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-rkl-5/igt@gem_exec_reloc@basic-write-read-active.html

  * igt@gem_partial_pwrite_pread@writes-after-reads:
    - shard-apl:          [INCOMPLETE][46] ([i915#7708]) -> [PASS][47] +1 similar issue
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-apl3/igt@gem_partial_pwrite_pread@writes-after-reads.html
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl7/igt@gem_partial_pwrite_pread@writes-after-reads.html

  * igt@gem_partial_pwrite_pread@writes-after-reads-uncached:
    - {shard-rkl}:        [SKIP][48] ([i915#3282]) -> [PASS][49] +3 similar issues
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-rkl-2/igt@gem_partial_pwrite_pread@writes-after-reads-uncached.html
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-rkl-5/igt@gem_partial_pwrite_pread@writes-after-reads-uncached.html

  * igt@gem_workarounds@suspend-resume-fd:
    - {shard-rkl}:        [FAIL][50] ([fdo#103375]) -> [PASS][51] +1 similar issue
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-rkl-4/igt@gem_workarounds@suspend-resume-fd.html
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-rkl-5/igt@gem_workarounds@suspend-resume-fd.html

  * igt@gen9_exec_parse@allowed-single:
    - {shard-rkl}:        [SKIP][52] ([i915#2527]) -> [PASS][53] +1 similar issue
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-rkl-4/igt@gen9_exec_parse@allowed-single.html
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-rkl-5/igt@gen9_exec_parse@allowed-single.html
    - shard-apl:          [DMESG-WARN][54] ([i915#5566] / [i915#716]) -> [PASS][55]
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-apl1/igt@gen9_exec_parse@allowed-single.html
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl2/igt@gen9_exec_parse@allowed-single.html

  * igt@i915_pm_dc@dc9-dpms:
    - shard-apl:          [SKIP][56] ([fdo#109271]) -> [PASS][57]
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-apl3/igt@i915_pm_dc@dc9-dpms.html
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl7/igt@i915_pm_dc@dc9-dpms.html

  * igt@i915_pm_rc6_residency@rc6-idle@vcs0:
    - {shard-dg1}:        [FAIL][58] ([i915#3591]) -> [PASS][59]
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-dg1-15/igt@i915_pm_rc6_residency@rc6-idle@vcs0.html
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-dg1-18/igt@i915_pm_rc6_residency@rc6-idle@vcs0.html

  * igt@i915_pm_rpm@fences-dpms:
    - {shard-rkl}:        [SKIP][60] ([i915#1849]) -> [PASS][61]
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-rkl-2/igt@i915_pm_rpm@fences-dpms.html
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-rkl-6/igt@i915_pm_rpm@fences-dpms.html

  * igt@i915_pm_rpm@modeset-non-lpsp-stress-no-wait:
    - {shard-dg1}:        [SKIP][62] ([i915#1397]) -> [PASS][63]
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-dg1-14/igt@i915_pm_rpm@modeset-non-lpsp-stress-no-wait.html
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-dg1-17/igt@i915_pm_rpm@modeset-non-lpsp-stress-no-wait.html

  * igt@i915_pm_rps@engine-order:
    - shard-apl:          [FAIL][64] ([i915#6537]) -> [PASS][65]
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-apl7/igt@i915_pm_rps@engine-order.html
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl6/igt@i915_pm_rps@engine-order.html

  * igt@kms_ccs@pipe-b-bad-aux-stride-y_tiled_gen12_rc_ccs:
    - {shard-rkl}:        [SKIP][66] ([i915#1845] / [i915#4098]) -> [PASS][67] +14 similar issues
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-rkl-3/igt@kms_ccs@pipe-b-bad-aux-stride-y_tiled_gen12_rc_ccs.html
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-rkl-6/igt@kms_ccs@pipe-b-bad-aux-stride-y_tiled_gen12_rc_ccs.html

  * igt@kms_frontbuffer_tracking@basic:
    - {shard-tglu}:       [SKIP][68] ([i915#1849]) -> [PASS][69] +3 similar issues
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-tglu-6/igt@kms_frontbuffer_tracking@basic.html
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-tglu-8/igt@kms_frontbuffer_tracking@basic.html

  * igt@kms_frontbuffer_tracking@psr-rgb101010-draw-pwrite:
    - {shard-rkl}:        [SKIP][70] ([i915#1849] / [i915#4098]) -> [PASS][71] +8 similar issues
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-rkl-3/igt@kms_frontbuffer_tracking@psr-rgb101010-draw-pwrite.html
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-rkl-6/igt@kms_frontbuffer_tracking@psr-rgb101010-draw-pwrite.html

  * igt@kms_plane@plane-panning-bottom-right-suspend@pipe-a-planes:
    - {shard-rkl}:        [SKIP][72] ([i915#3558]) -> [PASS][73] +1 similar issue
   [72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-rkl-4/igt@kms_plane@plane-panning-bottom-right-suspend@pipe-a-planes.html
   [73]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-rkl-6/igt@kms_plane@plane-panning-bottom-right-suspend@pipe-a-planes.html

  * igt@kms_psr@sprite_plane_move:
    - {shard-rkl}:        [SKIP][74] ([i915#1072]) -> [PASS][75]
   [74]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-rkl-3/igt@kms_psr@sprite_plane_move.html
   [75]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-rkl-6/igt@kms_psr@sprite_plane_move.html

  * igt@kms_universal_plane@disable-primary-vs-flip-pipe-d:
    - {shard-tglu}:       [SKIP][76] ([fdo#109274]) -> [PASS][77]
   [76]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-tglu-6/igt@kms_universal_plane@disable-primary-vs-flip-pipe-d.html
   [77]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-tglu-7/igt@kms_universal_plane@disable-primary-vs-flip-pipe-d.html

  * igt@kms_universal_plane@universal-plane-pipe-b-sanity:
    - {shard-rkl}:        [SKIP][78] ([i915#1845] / [i915#4070] / [i915#4098]) -> [PASS][79]
   [78]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-rkl-2/igt@kms_universal_plane@universal-plane-pipe-b-sanity.html
   [79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-rkl-6/igt@kms_universal_plane@universal-plane-pipe-b-sanity.html

  * igt@kms_vblank@pipe-d-wait-busy:
    - {shard-tglu}:       [SKIP][80] ([i915#7651]) -> [PASS][81] +2 similar issues
   [80]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-tglu-6/igt@kms_vblank@pipe-d-wait-busy.html
   [81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-tglu-4/igt@kms_vblank@pipe-d-wait-busy.html

  * igt@perf_pmu@busy-double-start@vecs0:
    - {shard-dg1}:        [FAIL][82] ([i915#4349]) -> [PASS][83] +2 similar issues
   [82]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-dg1-18/igt@perf_pmu@busy-double-start@vecs0.html
   [83]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-dg1-15/igt@perf_pmu@busy-double-start@vecs0.html

  * igt@perf_pmu@idle@rcs0:
    - {shard-rkl}:        [FAIL][84] ([i915#4349]) -> [PASS][85]
   [84]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-rkl-3/igt@perf_pmu@idle@rcs0.html
   [85]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-rkl-3/igt@perf_pmu@idle@rcs0.html

  * igt@prime_vgem@basic-fence-flip:
    - {shard-rkl}:        [SKIP][86] ([fdo#109295] / [i915#3708] / [i915#4098]) -> [PASS][87]
   [86]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-rkl-3/igt@prime_vgem@basic-fence-flip.html
   [87]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-rkl-6/igt@prime_vgem@basic-fence-flip.html

  * igt@prime_vgem@basic-read:
    - {shard-rkl}:        [SKIP][88] ([fdo#109295] / [i915#3291] / [i915#3708]) -> [PASS][89]
   [88]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-rkl-3/igt@prime_vgem@basic-read.html
   [89]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-rkl-5/igt@prime_vgem@basic-read.html

  * igt@prime_vgem@coherency-gtt:
    - {shard-rkl}:        [SKIP][90] ([fdo#109295] / [fdo#111656] / [i915#3708]) -> [PASS][91]
   [90]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-rkl-1/igt@prime_vgem@coherency-gtt.html
   [91]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-rkl-5/igt@prime_vgem@coherency-gtt.html

  * igt@sysfs_timeslice_duration@timeout@vecs0:
    - {shard-dg1}:        [FAIL][92] ([i915#1755]) -> [PASS][93] +2 similar issues
   [92]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-dg1-17/igt@sysfs_timeslice_duration@timeout@vecs0.html
   [93]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-dg1-12/igt@sysfs_timeslice_duration@timeout@vecs0.html

  
#### Warnings ####

  * igt@kms_plane_alpha_blend@alpha-basic@pipe-c-dp-1:
    - shard-apl:          [FAIL][94] ([i915#4573]) -> [DMESG-FAIL][95] ([IGT#6]) +1 similar issue
   [94]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-apl1/igt@kms_plane_alpha_blend@alpha-basic@pipe-c-dp-1.html
   [95]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl2/igt@kms_plane_alpha_blend@alpha-basic@pipe-c-dp-1.html

  * igt@runner@aborted:
    - shard-apl:          ([FAIL][96], [FAIL][97], [FAIL][98]) ([fdo#109271] / [i915#3002] / [i915#4312]) -> ([FAIL][99], [FAIL][100]) ([i915#3002] / [i915#4312])
   [96]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-apl7/igt@runner@aborted.html
   [97]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-apl1/igt@runner@aborted.html
   [98]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12530/shard-apl2/igt@runner@aborted.html
   [99]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl2/igt@runner@aborted.html
   [100]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/shard-apl6/igt@runner@aborted.html

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

  [IGT#6]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/issues/6
  [fdo#103375]: https://bugs.freedesktop.org/show_bug.cgi?id=103375
  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [fdo#109274]: https://bugs.freedesktop.org/show_bug.cgi?id=109274
  [fdo#109279]: https://bugs.freedesktop.org/show_bug.cgi?id=109279
  [fdo#109280]: https://bugs.freedesktop.org/show_bug.cgi?id=109280
  [fdo#109283]: https://bugs.freedesktop.org/show_bug.cgi?id=109283
  [fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285
  [fdo#109289]: https://bugs.freedesktop.org/show_bug.cgi?id=109289
  [fdo#109291]: https://bugs.freedesktop.org/show_bug.cgi?id=109291
  [fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295
  [fdo#109300]: https://bugs.freedesktop.org/show_bug.cgi?id=109300
  [fdo#109302]: https://bugs.freedesktop.org/show_bug.cgi?id=109302
  [fdo#109303]: https://bugs.freedesktop.org/show_bug.cgi?id=109303
  [fdo#109307]: https://bugs.freedesktop.org/show_bug.cgi?id=109307
  [fdo#109309]: https://bugs.freedesktop.org/show_bug.cgi?id=109309
  [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#109642]: https://bugs.freedesktop.org/show_bug.cgi?id=109642
  [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#111644]: https://bugs.freedesktop.org/show_bug.cgi?id=111644
  [fdo#111656]: https://bugs.freedesktop.org/show_bug.cgi?id=111656
  [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#1099]: https://gitlab.freedesktop.org/drm/intel/issues/1099
  [i915#132]: https://gitlab.freedesktop.org/drm/intel/issues/132
  [i915#1397]: https://gitlab.freedesktop.org/drm/intel/issues/1397
  [i915#1755]: https://gitlab.freedesktop.org/drm/intel/issues/1755
  [i915#1769]: https://gitlab.freedesktop.org/drm/intel/issues/1769
  [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#1850]: https://gitlab.freedesktop.org/drm/intel/issues/1850
  [i915#1902]: https://gitlab.freedesktop.org/drm/intel/issues/1902
  [i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
  [i915#2346]: https://gitlab.freedesktop.org/drm/intel/issues/2346
  [i915#2437]: https://gitlab.freedesktop.org/drm/intel/issues/2437
  [i915#2527]: https://gitlab.freedesktop.org/drm/intel/issues/2527
  [i915#2532]: https://gitlab.freedesktop.org/drm/intel/issues/2532
  [i915#2575]: https://gitlab.freedesktop.org/drm/intel/issues/2575
  [i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582
  [i915#2587]: https://gitlab.freedesktop.org/drm/intel/issues/2587
  [i915#2658]: https://gitlab.freedesktop.org/drm/intel/issues/2658
  [i915#2672]: https://gitlab.freedesktop.org/drm/intel/issues/2672
  [i915#2681]: https://gitlab.freedesktop.org/drm/intel/issues/2681
  [i915#2705]: https://gitlab.freedesktop.org/drm/intel/issues/2705
  [i915#280]: https://gitlab.freedesktop.org/drm/intel/issues/280
  [i915#284]: https://gitlab.freedesktop.org/drm/intel/issues/284
  [i915#2842]: https://gitlab.freedesktop.org/drm/intel/issues/2842
  [i915#2846]: https://gitlab.freedesktop.org/drm/intel/issues/2846
  [i915#2856]: https://gitlab.freedesktop.org/drm/intel/issues/2856
  [i915#2920]: https://gitlab.freedesktop.org/drm/intel/issues/2920
  [i915#2994]: https://gitlab.freedesktop.org/drm/intel/issues/2994
  [i915#3002]: https://gitlab.freedesktop.org/drm/intel/issues/3002
  [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#3318]: https://gitlab.freedesktop.org/drm/intel/issues/3318
  [i915#3323]: https://gitlab.freedesktop.org/drm/intel/issues/3323
  [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#3558]: https://gitlab.freedesktop.org/drm/intel/issues/3558
  [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#3689]: https://gitlab.freedesktop.org/drm/intel/issues/3689
  [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
  [i915#3734]: https://gitlab.freedesktop.org/drm/intel/issues/3734
  [i915#3742]: https://gitlab.freedesktop.org/drm/intel/issues/3742
  [i915#3886]: https://gitlab.freedesktop.org/drm/intel/issues/3886
  [i915#3955]: https://gitlab.freedesktop.org/drm/intel/issues/3955
  [i915#3966]: https://gitlab.freedesktop.org/drm/intel/issues/3966
  [i915#3989]: https://gitlab.freedesktop.org/drm/intel/issues/3989
  [i915#404]: https://gitlab.freedesktop.org/drm/intel/issues/404
  [i915#4070]: https://gitlab.freedesktop.org/drm/intel/issues/4070
  [i915#4077]: https://gitlab.freedesktop.org/drm/intel/issues/4077
  [i915#4078]: https://gitlab.freedesktop.org/drm/intel/issues/4078
  [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#4270]: https://gitlab.freedesktop.org/drm/intel/issues/4270
  [i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
  [i915#4349]: https://gitlab.freedesktop.org/drm/intel/issues/4349
  [i915#4387]: https://gitlab.freedesktop.org/drm/intel/issues/4387
  [i915#4525]: https://gitlab.freedesktop.org/drm/intel/issues/4525
  [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#4833]: https://gitlab.freedesktop.org/drm/intel/issues/4833
  [i915#4852]: https://gitlab.freedesktop.org/drm/intel/issues/4852
  [i915#4854]: https://gitlab.freedesktop.org/drm/intel/issues/4854
  [i915#4877]: https://gitlab.freedesktop.org/drm/intel/issues/4877
  [i915#4880]: https://gitlab.freedesktop.org/drm/intel/issues/4880
  [i915#4881]: https://gitlab.freedesktop.org/drm/intel/issues/4881
  [i915#5030]: https://gitlab.freedesktop.org/drm/intel/issues/5030
  [i915#5176]: https://gitlab.freedesktop.org/drm/intel/issues/5176
  [i915#5235]: https://gitlab.freedesktop.org/drm/intel/issues/5235
  [i915#5286]: https://gitlab.freedesktop.org/drm/intel/issues/5286
  [i915#5289]: https://gitlab.freedesktop.org/drm/intel/issues/5289
  [i915#5325]: https://gitlab.freedesktop.org/drm/intel/issues/5325
  [i915#5327]: https://gitlab.freedesktop.org/drm/intel/issues/5327
  [i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533
  [i915#5334]: https://gitlab.freedesktop.org/drm/intel/issues/5334
  [i915#5439]: https://gitlab.freedesktop.org/drm/intel/issues/5439
  [i915#5563]: https://gitlab.freedesktop.org/drm/intel/issues/5563
  [i915#5566]: https://gitlab.freedesktop.org/drm/intel/issues/5566
  [i915#5784]: https://gitlab.freedesktop.org/drm/intel/issues/5784
  [i915#6095]: https://gitlab.freedesktop.org/drm/intel/issues/6095
  [i915#6117]: https://gitlab.freedesktop.org/drm/intel/issues/6117
  [i915#6248]: https://gitlab.freedesktop.org/drm/intel/issues/6248
  [i915#6301]: https://gitlab.freedesktop.org/drm/intel/issues/6301
  [i915#6334]: https://gitlab.freedesktop.org/drm/intel/issues/6334
  [i915#6335]: https://gitlab.freedesktop.org/drm/intel/issues/6335
  [i915#6344]: https://gitlab.freedesktop.org/drm/intel/issues/6344
  [i915#6412]: https://gitlab.freedesktop.org/drm/intel/issues/6412
  [i915#6433]: https://gitlab.freedesktop.org/drm/intel/issues/6433
  [i915#6497]: https://gitlab.freedesktop.org/drm/intel/issues/6497
  [i915#6524]: https://gitlab.freedesktop.org/drm/intel/issues/6524
  [i915#6537]: https://gitlab.freedesktop.org/drm/intel/issues/6537
  [i915#658]: https://gitlab.freedesktop.org/drm/intel/issues/658
  [i915#6621]: https://gitlab.freedesktop.org/drm/intel/issues/6621
  [i915#6768]: https://gitlab.freedesktop.org/drm/intel/issues/6768
  [i915#6944]: https://gitlab.freedesktop.org/drm/intel/issues/6944
  [i915#6953]: https://gitlab.freedesktop.org/drm/intel/issues/6953
  [i915#7052]: https://gitlab.freedesktop.org/drm/intel/issues/7052
  [i915#7116]: https://gitlab.freedesktop.org/drm/intel/issues/7116
  [i915#7118]: https://gitlab.freedesktop.org/drm/intel/issues/7118
  [i915#7128]: https://gitlab.freedesktop.org/drm/intel/issues/7128
  [i915#716]: https://gitlab.freedesktop.org/drm/intel/issues/716
  [i915#72]: https://gitlab.freedesktop.org/drm/intel/issues/72
  [i915#7561]: https://gitlab.freedesktop.org/drm/intel/issues/7561
  [i915#7651]: https://gitlab.freedesktop.org/drm/intel/issues/7651
  [i915#7697]: https://gitlab.freedesktop.org/drm/intel/issues/7697
  [i915#7701]: https://gitlab.freedesktop.org/drm/intel/issues/7701
  [i915#7707]: https://gitlab.freedesktop.org/drm/intel/issues/7707
  [i915#7708]: https://gitlab.freedesktop.org/drm/intel/issues/7708
  [i915#7711]: https://gitlab.freedesktop.org/drm/intel/issues/7711


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

  * CI: CI-20190529 -> None
  * IGT: IGT_7104 -> IGTPW_8285
  * Piglit: piglit_4509 -> None

  CI-20190529: 20190529
  CI_DRM_12530: c112aa73981bbf79959a95744af3f3fbbeca2197 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_8285: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8285/index.html
  IGT_7104: fe5def13049225967770eaaf19ec01ef80e2adc5 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
  piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit

== Logs ==

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

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

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

* Re: [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D
  2022-12-30  6:46 [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D bhadanednyaneshwar
  2022-12-30 10:27 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
  2022-12-30 11:49 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
@ 2022-12-30 19:16 ` Swati Sharma
  2023-01-03  5:59   ` Bhadane, Dnyaneshwar
  2023-01-03 14:50 ` Kamil Konieczny
                   ` (6 subsequent siblings)
  9 siblings, 1 reply; 17+ messages in thread
From: Swati Sharma @ 2022-12-30 19:16 UTC (permalink / raw)
  To: bhadanednyaneshwar, igt-dev; +Cc: chaitanya.kumar.borah, suresh.kumar.kurmi



On 30-Dec-22 12:16 PM, bhadanednyaneshwar wrote:
> MSO eDP is not supported on pipe C and D. Added a test condition
> to prevent tests from execution on pipe C and D.This condition was
> missed for cursor-size-change,cursor-alpha-opaque and
> cursor-alpha-transparent testcases.
> 
> Inside require_cursor_size() checks first for eligiblity to igt commit
> using test buffer.For MSO eDP, It is fail to commit for pipe C/pipe D
> and require_cursor_size() return non zero value. So it will skip
> the dynamic testcase for pipe C and D.
> 
> Signed-off-by: bhadanednyaneshwar <dnyaneshwar.bhadane@intel.com>
> ---
>   tests/kms_cursor_crc.c | 21 ++++++++++++++++++---
>   1 file changed, 18 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c
> index 17f294d6..d8fb9c0d 100644
> --- a/tests/kms_cursor_crc.c
> +++ b/tests/kms_cursor_crc.c
> @@ -786,7 +786,12 @@ static void run_tests_on_pipe(data_t *data)
>   	igt_subtest_with_dynamic("cursor-size-change") {
>   		for_each_pipe(&data->display, pipe) {
>   			data->pipe = pipe;
> -
> +			create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
 >> Instead of creating cursor_fb in each function, we can have
igt_fixture {
	create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
}
moved up and only have
if (require_cursor_size()) check in each subtest.

> +			if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) {
> +				igt_debug("Cursor size %dx%d not supported by driver\n",
> +					  data->cursor_max_w, data->cursor_max_h);
> +				continue;
> +			}
>   			igt_dynamic_f("pipe-%s-%s",
>   				      kmstest_pipe_name(pipe),
>   				      data->output->name)
> @@ -800,7 +805,12 @@ static void run_tests_on_pipe(data_t *data)
>   	igt_subtest_with_dynamic("cursor-alpha-opaque") {
>   		for_each_pipe(&data->display, pipe) {
>   			data->pipe = pipe;
> -
> +			create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
> +			if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) {
> +				igt_debug("Cursor size %dx%d not supported by driver\n",
> +					  data->cursor_max_w, data->cursor_max_h);
> +				continue;
> +			}
>   			igt_dynamic_f("pipe-%s-%s",
>   				      kmstest_pipe_name(pipe),
>   				      data->output->name)
> @@ -814,7 +824,12 @@ static void run_tests_on_pipe(data_t *data)
>   	igt_subtest_with_dynamic("cursor-alpha-transparent") {
>   		for_each_pipe(&data->display, pipe) {
>   			data->pipe = pipe;
> -
> +			create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
> +			if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) {
> +				igt_debug("Cursor size %dx%d not supported by driver\n",
> +					  data->cursor_max_w, data->cursor_max_h);
> +				continue;
> +			}
>   			igt_dynamic_f("pipe-%s-%s",
>   				      kmstest_pipe_name(pipe),
>   				      data->output->name)

-- 
~Swati Sharma

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

* Re: [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D
  2022-12-30 19:16 ` [igt-dev] [i-g-t] " Swati Sharma
@ 2023-01-03  5:59   ` Bhadane, Dnyaneshwar
  0 siblings, 0 replies; 17+ messages in thread
From: Bhadane, Dnyaneshwar @ 2023-01-03  5:59 UTC (permalink / raw)
  To: Sharma, Swati2, igt-dev; +Cc: Borah, Chaitanya Kumar, Kurmi, Suresh Kumar

Hi Swati,

> -----Original Message-----
> From: Sharma, Swati2 <swati2.sharma@intel.com>
> Sent: Saturday, December 31, 2022 12:46 AM
> To: Bhadane, Dnyaneshwar <dnyaneshwar.bhadane@intel.com>; igt-
> dev@lists.freedesktop.org
> Cc: Kurmi, Suresh Kumar <suresh.kumar.kurmi@intel.com>; Shankar, Uma
> <uma.shankar@intel.com>; Modem, Bhanuprakash
> <bhanuprakash.modem@intel.com>; Borah, Chaitanya Kumar
> <chaitanya.kumar.borah@intel.com>; B, Jeevan <jeevan.b@intel.com>
> Subject: Re: [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C
> and D
>
>
>
> On 30-Dec-22 12:16 PM, bhadanednyaneshwar wrote:
> > MSO eDP is not supported on pipe C and D. Added a test condition to
> > prevent tests from execution on pipe C and D.This condition was missed
> > for cursor-size-change,cursor-alpha-opaque and
> > cursor-alpha-transparent testcases.
> >
> > Inside require_cursor_size() checks first for eligiblity to igt commit
> > using test buffer.For MSO eDP, It is fail to commit for pipe C/pipe D
> > and require_cursor_size() return non zero value. So it will skip the
> > dynamic testcase for pipe C and D.
> >
> > Signed-off-by: bhadanednyaneshwar <dnyaneshwar.bhadane@intel.com>
> > ---
> >   tests/kms_cursor_crc.c | 21 ++++++++++++++++++---
> >   1 file changed, 18 insertions(+), 3 deletions(-)
> >
> > diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c index
> > 17f294d6..d8fb9c0d 100644
> > --- a/tests/kms_cursor_crc.c
> > +++ b/tests/kms_cursor_crc.c
> > @@ -786,7 +786,12 @@ static void run_tests_on_pipe(data_t *data)
> >     igt_subtest_with_dynamic("cursor-size-change") {
> >             for_each_pipe(&data->display, pipe) {
> >                     data->pipe = pipe;
> > -
> > +                   create_cursor_fb(data, data->cursor_max_w, data-
> >cursor_max_h);
>  >> Instead of creating cursor_fb in each function, we can have igt_fixture {
>       create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h); }
> moved up and only have if (require_cursor_size()) check in each subtest.

For every pipe the we require to create_cursor_fb()  because the testcase
closing the buffer using igt_remove_fb() on every pipe in the loop;
So igt_fixture { create_cursor_fb() } will only called once, but we need to
create buffer on every pipe in the loop.
>
> > +                   if (require_cursor_size(data, data->cursor_max_w,
> data->cursor_max_h)) {
> > +                           igt_debug("Cursor size %dx%d not supported
> by driver\n",
> > +                                     data->cursor_max_w, data-
> >cursor_max_h);
> > +                           continue;
> > +                   }
> >                     igt_dynamic_f("pipe-%s-%s",
> >                                   kmstest_pipe_name(pipe),
> >                                   data->output->name)
> > @@ -800,7 +805,12 @@ static void run_tests_on_pipe(data_t *data)
> >     igt_subtest_with_dynamic("cursor-alpha-opaque") {
> >             for_each_pipe(&data->display, pipe) {
> >                     data->pipe = pipe;
> > -
> > +                   create_cursor_fb(data, data->cursor_max_w, data-
> >cursor_max_h);
> > +                   if (require_cursor_size(data, data->cursor_max_w,
> data->cursor_max_h)) {
> > +                           igt_debug("Cursor size %dx%d not supported
> by driver\n",
> > +                                     data->cursor_max_w, data-
> >cursor_max_h);
> > +                           continue;
> > +                   }
> >                     igt_dynamic_f("pipe-%s-%s",
> >                                   kmstest_pipe_name(pipe),
> >                                   data->output->name)
> > @@ -814,7 +824,12 @@ static void run_tests_on_pipe(data_t *data)
> >     igt_subtest_with_dynamic("cursor-alpha-transparent") {
> >             for_each_pipe(&data->display, pipe) {
> >                     data->pipe = pipe;
> > -
> > +                   create_cursor_fb(data, data->cursor_max_w, data-
> >cursor_max_h);
> > +                   if (require_cursor_size(data, data->cursor_max_w,
> data->cursor_max_h)) {
> > +                           igt_debug("Cursor size %dx%d not supported
> by driver\n",
> > +                                     data->cursor_max_w, data-
> >cursor_max_h);
> > +                           continue;
> > +                   }
> >                     igt_dynamic_f("pipe-%s-%s",
> >                                   kmstest_pipe_name(pipe),
> >                                   data->output->name)
>
> --
> ~Swati Sharma

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

* Re: [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D
  2022-12-30  6:46 [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D bhadanednyaneshwar
                   ` (2 preceding siblings ...)
  2022-12-30 19:16 ` [igt-dev] [i-g-t] " Swati Sharma
@ 2023-01-03 14:50 ` Kamil Konieczny
  2023-01-05  6:26   ` Bhadane, Dnyaneshwar
  2023-01-03 17:41 ` Kamil Konieczny
                   ` (5 subsequent siblings)
  9 siblings, 1 reply; 17+ messages in thread
From: Kamil Konieczny @ 2023-01-03 14:50 UTC (permalink / raw)
  To: igt-dev; +Cc: Dnyaneshwar Bhadane, Chaitanya Kumar Borah, Suresh Kumar Kurmi

Hi,

On 2022-12-30 at 12:16:38 +0530, bhadanednyaneshwar wrote:
> MSO eDP is not supported on pipe C and D. Added a test condition
------------------------------------------- ^
> to prevent tests from execution on pipe C and D.This condition was
> missed for cursor-size-change,cursor-alpha-opaque and
> cursor-alpha-transparent testcases.

Please improve a little commit message, you just added skips for
not supported cursors in those subtests.

> 
> Inside require_cursor_size() checks first for eligiblity to igt commit
> using test buffer.For MSO eDP, It is fail to commit for pipe C/pipe D
> and require_cursor_size() return non zero value. So it will skip
> the dynamic testcase for pipe C and D.

The above paragraph is spelling what you done. Will it only skip
for pipe C and D ? Or any pipe on which it cannot create cursor ?

You can add cc here:

Cc: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Cc: Suresh Kumar Kurmi <suresh.kumar.kurmi@intel.com>
Cc: Swati Sharma <swati2.sharma@intel.com>
> 
> Signed-off-by: bhadanednyaneshwar <dnyaneshwar.bhadane@intel.com>
---------------- ^
Please put here your name (look for example for Swati), I guess
it should be

Signed-off-by: Dnyaneshwar Bhadane <dnyaneshwar.bhadane@intel.com>

but if I guess wrong please put correct one (it is best to put this
in global git config).

> ---
>  tests/kms_cursor_crc.c | 21 ++++++++++++++++++---
>  1 file changed, 18 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c
> index 17f294d6..d8fb9c0d 100644
> --- a/tests/kms_cursor_crc.c
> +++ b/tests/kms_cursor_crc.c
> @@ -786,7 +786,12 @@ static void run_tests_on_pipe(data_t *data)
>  	igt_subtest_with_dynamic("cursor-size-change") {
>  		for_each_pipe(&data->display, pipe) {
>  			data->pipe = pipe;
> -
> +			create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
----------------------- ^
Do you really need this here ? It is used elsewhere within fixup
(there is assert in this function), in other tests below
require_cursor_size is called without create_cursor_fb

> +			if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) {
> +				igt_debug("Cursor size %dx%d not supported by driver\n",
> +					  data->cursor_max_w, data->cursor_max_h);
> +				continue;
> +			}

Put newline here.

>  			igt_dynamic_f("pipe-%s-%s",
>  				      kmstest_pipe_name(pipe),
>  				      data->output->name)
> @@ -800,7 +805,12 @@ static void run_tests_on_pipe(data_t *data)
>  	igt_subtest_with_dynamic("cursor-alpha-opaque") {
>  		for_each_pipe(&data->display, pipe) {
>  			data->pipe = pipe;
> -
> +			create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
----------------------- ^
> +			if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) {
> +				igt_debug("Cursor size %dx%d not supported by driver\n",
> +					  data->cursor_max_w, data->cursor_max_h);
> +				continue;
> +			}

Put newline here.

>  			igt_dynamic_f("pipe-%s-%s",
>  				      kmstest_pipe_name(pipe),
>  				      data->output->name)
> @@ -814,7 +824,12 @@ static void run_tests_on_pipe(data_t *data)
>  	igt_subtest_with_dynamic("cursor-alpha-transparent") {
>  		for_each_pipe(&data->display, pipe) {
>  			data->pipe = pipe;
> -
> +			create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
----------------------- ^
> +			if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) {
> +				igt_debug("Cursor size %dx%d not supported by driver\n",
> +					  data->cursor_max_w, data->cursor_max_h);
> +				continue;
> +			}

Put newline here.

Regards,
Kamil

>  			igt_dynamic_f("pipe-%s-%s",
>  				      kmstest_pipe_name(pipe),
>  				      data->output->name)
> -- 
> 2.35.1
> 

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

* Re: [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D
  2022-12-30  6:46 [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D bhadanednyaneshwar
                   ` (3 preceding siblings ...)
  2023-01-03 14:50 ` Kamil Konieczny
@ 2023-01-03 17:41 ` Kamil Konieczny
  2023-01-05  6:22   ` Bhadane, Dnyaneshwar
  2023-01-03 18:31 ` Juha-Pekka Heikkila
                   ` (4 subsequent siblings)
  9 siblings, 1 reply; 17+ messages in thread
From: Kamil Konieczny @ 2023-01-03 17:41 UTC (permalink / raw)
  To: igt-dev; +Cc: dnyaneshwar.bhadane

Hi,

one quick nit for you subject (commit description):

tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D
--------------------- ^ --^ ------------------- ^
s/Add/add/
s/Gaurd/guard/

but imho better say explicitly that you skip, so better:

tests/kms_cursor_crc: skip subtests for MSO eDP on pipe C and D

Regards,
Kamil

On 2022-12-30 at 12:16:38 +0530, bhadanednyaneshwar wrote:
> MSO eDP is not supported on pipe C and D. Added a test condition
> to prevent tests from execution on pipe C and D.This condition was
> missed for cursor-size-change,cursor-alpha-opaque and
> cursor-alpha-transparent testcases.
> 
> Inside require_cursor_size() checks first for eligiblity to igt commit
> using test buffer.For MSO eDP, It is fail to commit for pipe C/pipe D
> and require_cursor_size() return non zero value. So it will skip
> the dynamic testcase for pipe C and D.
> 
> Signed-off-by: bhadanednyaneshwar <dnyaneshwar.bhadane@intel.com>
> ---
>  tests/kms_cursor_crc.c | 21 ++++++++++++++++++---
>  1 file changed, 18 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c
> index 17f294d6..d8fb9c0d 100644
> --- a/tests/kms_cursor_crc.c
> +++ b/tests/kms_cursor_crc.c
> @@ -786,7 +786,12 @@ static void run_tests_on_pipe(data_t *data)
>  	igt_subtest_with_dynamic("cursor-size-change") {
>  		for_each_pipe(&data->display, pipe) {
>  			data->pipe = pipe;
> -
> +			create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
> +			if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) {
> +				igt_debug("Cursor size %dx%d not supported by driver\n",
> +					  data->cursor_max_w, data->cursor_max_h);
> +				continue;
> +			}
>  			igt_dynamic_f("pipe-%s-%s",
>  				      kmstest_pipe_name(pipe),
>  				      data->output->name)
> @@ -800,7 +805,12 @@ static void run_tests_on_pipe(data_t *data)
>  	igt_subtest_with_dynamic("cursor-alpha-opaque") {
>  		for_each_pipe(&data->display, pipe) {
>  			data->pipe = pipe;
> -
> +			create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
> +			if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) {
> +				igt_debug("Cursor size %dx%d not supported by driver\n",
> +					  data->cursor_max_w, data->cursor_max_h);
> +				continue;
> +			}
>  			igt_dynamic_f("pipe-%s-%s",
>  				      kmstest_pipe_name(pipe),
>  				      data->output->name)
> @@ -814,7 +824,12 @@ static void run_tests_on_pipe(data_t *data)
>  	igt_subtest_with_dynamic("cursor-alpha-transparent") {
>  		for_each_pipe(&data->display, pipe) {
>  			data->pipe = pipe;
> -
> +			create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
> +			if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) {
> +				igt_debug("Cursor size %dx%d not supported by driver\n",
> +					  data->cursor_max_w, data->cursor_max_h);
> +				continue;
> +			}
>  			igt_dynamic_f("pipe-%s-%s",
>  				      kmstest_pipe_name(pipe),
>  				      data->output->name)
> -- 
> 2.35.1
> 

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

* Re: [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D
  2022-12-30  6:46 [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D bhadanednyaneshwar
                   ` (4 preceding siblings ...)
  2023-01-03 17:41 ` Kamil Konieczny
@ 2023-01-03 18:31 ` Juha-Pekka Heikkila
  2023-01-05  8:07   ` Bhadane, Dnyaneshwar
  2023-01-06 12:28   ` Bhadane, Dnyaneshwar
  2023-01-09  6:49 ` [igt-dev] [i-g-t v2] tests/kms_cursor_crc: Skip for invalid pipe connector connection Dnyaneshwar Bhadane
                   ` (3 subsequent siblings)
  9 siblings, 2 replies; 17+ messages in thread
From: Juha-Pekka Heikkila @ 2023-01-03 18:31 UTC (permalink / raw)
  To: bhadanednyaneshwar, igt-dev; +Cc: chaitanya.kumar.borah, suresh.kumar.kurmi

Hi Dnyaneshwar,

On 30.12.2022 8.46, bhadanednyaneshwar wrote:
> MSO eDP is not supported on pipe C and D. Added a test condition
> to prevent tests from execution on pipe C and D.This condition was
> missed for cursor-size-change,cursor-alpha-opaque and
> cursor-alpha-transparent testcases.
> 
> Inside require_cursor_size() checks first for eligiblity to igt commit
> using test buffer.For MSO eDP, It is fail to commit for pipe C/pipe D
> and require_cursor_size() return non zero value. So it will skip
> the dynamic testcase for pipe C and D.
> 
> Signed-off-by: bhadanednyaneshwar <dnyaneshwar.bhadane@intel.com>
> ---
>   tests/kms_cursor_crc.c | 21 ++++++++++++++++++---
>   1 file changed, 18 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c
> index 17f294d6..d8fb9c0d 100644
> --- a/tests/kms_cursor_crc.c
> +++ b/tests/kms_cursor_crc.c
> @@ -786,7 +786,12 @@ static void run_tests_on_pipe(data_t *data)
>   	igt_subtest_with_dynamic("cursor-size-change") {
>   		for_each_pipe(&data->display, pipe) {
>   			data->pipe = pipe;
> -
> +			create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
> +			if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) {
> +				igt_debug("Cursor size %dx%d not supported by driver\n",
> +					  data->cursor_max_w, data->cursor_max_h);
> +				continue;
> +			}

Would it work if instead of above checking you would use 
igt_pipe_connector_valid(..)? igt_pipe_connector_valid is using bits 
coming from encoder so I think it might work for this case and it would 
be much faster to check.

/Juha-Pekka

>   			igt_dynamic_f("pipe-%s-%s",
>   				      kmstest_pipe_name(pipe),
>   				      data->output->name)
> @@ -800,7 +805,12 @@ static void run_tests_on_pipe(data_t *data)
>   	igt_subtest_with_dynamic("cursor-alpha-opaque") {
>   		for_each_pipe(&data->display, pipe) {
>   			data->pipe = pipe;
> -
> +			create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
> +			if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) {
> +				igt_debug("Cursor size %dx%d not supported by driver\n",
> +					  data->cursor_max_w, data->cursor_max_h);
> +				continue;
> +			}
>   			igt_dynamic_f("pipe-%s-%s",
>   				      kmstest_pipe_name(pipe),
>   				      data->output->name)
> @@ -814,7 +824,12 @@ static void run_tests_on_pipe(data_t *data)
>   	igt_subtest_with_dynamic("cursor-alpha-transparent") {
>   		for_each_pipe(&data->display, pipe) {
>   			data->pipe = pipe;
> -
> +			create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
> +			if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) {
> +				igt_debug("Cursor size %dx%d not supported by driver\n",
> +					  data->cursor_max_w, data->cursor_max_h);
> +				continue;
> +			}
>   			igt_dynamic_f("pipe-%s-%s",
>   				      kmstest_pipe_name(pipe),
>   				      data->output->name)

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

* Re: [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D
  2023-01-03 17:41 ` Kamil Konieczny
@ 2023-01-05  6:22   ` Bhadane, Dnyaneshwar
  0 siblings, 0 replies; 17+ messages in thread
From: Bhadane, Dnyaneshwar @ 2023-01-05  6:22 UTC (permalink / raw)
  To: Kamil Konieczny, igt-dev

Hi Kamil,
Thanks for the inputs.
I will update the commit message.

> -----Original Message-----
> From: Kamil Konieczny <kamil.konieczny@linux.intel.com>
> Sent: Tuesday, January 3, 2023 11:12 PM
> To: igt-dev@lists.freedesktop.org
> Cc: Bhadane, Dnyaneshwar <dnyaneshwar.bhadane@intel.com>
> Subject: Re: [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for
> Pipe C and D
>
> Hi,
>
> one quick nit for you subject (commit description):
>
> tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D
> --------------------- ^ --^ ------------------- ^ s/Add/add/ s/Gaurd/guard/
>
> but imho better say explicitly that you skip, so better:
>
> tests/kms_cursor_crc: skip subtests for MSO eDP on pipe C and D
>
> Regards,
> Kamil
>
> On 2022-12-30 at 12:16:38 +0530, bhadanednyaneshwar wrote:
> > MSO eDP is not supported on pipe C and D. Added a test condition to
> > prevent tests from execution on pipe C and D.This condition was missed
> > for cursor-size-change,cursor-alpha-opaque and
> > cursor-alpha-transparent testcases.
> >
> > Inside require_cursor_size() checks first for eligiblity to igt commit
> > using test buffer.For MSO eDP, It is fail to commit for pipe C/pipe D
> > and require_cursor_size() return non zero value. So it will skip the
> > dynamic testcase for pipe C and D.
> >
> > Signed-off-by: bhadanednyaneshwar <dnyaneshwar.bhadane@intel.com>
> > ---
> >  tests/kms_cursor_crc.c | 21 ++++++++++++++++++---
> >  1 file changed, 18 insertions(+), 3 deletions(-)
> >
> > diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c index
> > 17f294d6..d8fb9c0d 100644
> > --- a/tests/kms_cursor_crc.c
> > +++ b/tests/kms_cursor_crc.c
> > @@ -786,7 +786,12 @@ static void run_tests_on_pipe(data_t *data)
> >     igt_subtest_with_dynamic("cursor-size-change") {
> >             for_each_pipe(&data->display, pipe) {
> >                     data->pipe = pipe;
> > -
> > +                   create_cursor_fb(data, data->cursor_max_w, data-
> >cursor_max_h);
> > +                   if (require_cursor_size(data, data->cursor_max_w,
> data->cursor_max_h)) {
> > +                           igt_debug("Cursor size %dx%d not supported
> by driver\n",
> > +                                     data->cursor_max_w, data-
> >cursor_max_h);
> > +                           continue;
> > +                   }
> >                     igt_dynamic_f("pipe-%s-%s",
> >                                   kmstest_pipe_name(pipe),
> >                                   data->output->name)
> > @@ -800,7 +805,12 @@ static void run_tests_on_pipe(data_t *data)
> >     igt_subtest_with_dynamic("cursor-alpha-opaque") {
> >             for_each_pipe(&data->display, pipe) {
> >                     data->pipe = pipe;
> > -
> > +                   create_cursor_fb(data, data->cursor_max_w, data-
> >cursor_max_h);
> > +                   if (require_cursor_size(data, data->cursor_max_w,
> data->cursor_max_h)) {
> > +                           igt_debug("Cursor size %dx%d not supported
> by driver\n",
> > +                                     data->cursor_max_w, data-
> >cursor_max_h);
> > +                           continue;
> > +                   }
> >                     igt_dynamic_f("pipe-%s-%s",
> >                                   kmstest_pipe_name(pipe),
> >                                   data->output->name)
> > @@ -814,7 +824,12 @@ static void run_tests_on_pipe(data_t *data)
> >     igt_subtest_with_dynamic("cursor-alpha-transparent") {
> >             for_each_pipe(&data->display, pipe) {
> >                     data->pipe = pipe;
> > -
> > +                   create_cursor_fb(data, data->cursor_max_w, data-
> >cursor_max_h);
> > +                   if (require_cursor_size(data, data->cursor_max_w,
> data->cursor_max_h)) {
> > +                           igt_debug("Cursor size %dx%d not supported
> by driver\n",
> > +                                     data->cursor_max_w, data-
> >cursor_max_h);
> > +                           continue;
> > +                   }
> >                     igt_dynamic_f("pipe-%s-%s",
> >                                   kmstest_pipe_name(pipe),
> >                                   data->output->name)
> > --
> > 2.35.1
> >

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

* Re: [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D
  2023-01-03 14:50 ` Kamil Konieczny
@ 2023-01-05  6:26   ` Bhadane, Dnyaneshwar
  0 siblings, 0 replies; 17+ messages in thread
From: Bhadane, Dnyaneshwar @ 2023-01-05  6:26 UTC (permalink / raw)
  To: Kamil Konieczny, igt-dev; +Cc: Borah, Chaitanya Kumar, Kurmi, Suresh Kumar

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

Hello Kamil,
Thanks for the inputs.
I will keep note this comments and update this patch for the same.

From: Kamil Konieczny<mailto:kamil.konieczny@linux.intel.com>
Sent: Tuesday, January 3, 2023 8:20 PM
To: igt-dev@lists.freedesktop.org<mailto:igt-dev@lists.freedesktop.org>
Cc: Bhadane, Dnyaneshwar<mailto:dnyaneshwar.bhadane@intel.com>; Borah, Chaitanya Kumar<mailto:chaitanya.kumar.borah@intel.com>; Kurmi, Suresh Kumar<mailto:suresh.kumar.kurmi@intel.com>; Sharma, Swati2<mailto:swati2.sharma@intel.com>
Subject: Re: [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D

Hi,

On 2022-12-30 at 12:16:38 +0530, bhadanednyaneshwar wrote:
> MSO eDP is not supported on pipe C and D. Added a test condition
------------------------------------------- ^
> to prevent tests from execution on pipe C and D.This condition was
> missed for cursor-size-change,cursor-alpha-opaque and
> cursor-alpha-transparent testcases.

Please improve a little commit message, you just added skips for
not supported cursors in those subtests.

>
> Inside require_cursor_size() checks first for eligiblity to igt commit
> using test buffer.For MSO eDP, It is fail to commit for pipe C/pipe D
> and require_cursor_size() return non zero value. So it will skip
> the dynamic testcase for pipe C and D.

The above paragraph is spelling what you done. Will it only skip
for pipe C and D ? Or any pipe on which it cannot create cursor ?

You can add cc here:

Cc: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Cc: Suresh Kumar Kurmi <suresh.kumar.kurmi@intel.com>
Cc: Swati Sharma <swati2.sharma@intel.com>
>
> Signed-off-by: bhadanednyaneshwar <dnyaneshwar.bhadane@intel.com>
---------------- ^
Please put here your name (look for example for Swati), I guess
it should be

Signed-off-by: Dnyaneshwar Bhadane <dnyaneshwar.bhadane@intel.com>

but if I guess wrong please put correct one (it is best to put this
in global git config).

> ---
>  tests/kms_cursor_crc.c | 21 ++++++++++++++++++---
>  1 file changed, 18 insertions(+), 3 deletions(-)
>
> diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c
> index 17f294d6..d8fb9c0d 100644
> --- a/tests/kms_cursor_crc.c
> +++ b/tests/kms_cursor_crc.c
> @@ -786,7 +786,12 @@ static void run_tests_on_pipe(data_t *data)
>        igt_subtest_with_dynamic("cursor-size-change") {
>                for_each_pipe(&data->display, pipe) {
>                        data->pipe = pipe;
> -
> +                     create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
----------------------- ^
Do you really need this here ? It is used elsewhere within fixup
(there is assert in this function), in other tests below
require_cursor_size is called without create_cursor_fb

> +                     if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) {
> +                             igt_debug("Cursor size %dx%d not supported by driver\n",
> +                                       data->cursor_max_w, data->cursor_max_h);
> +                             continue;
> +                     }

Put newline here.

>                        igt_dynamic_f("pipe-%s-%s",
>                                      kmstest_pipe_name(pipe),
>                                      data->output->name)
> @@ -800,7 +805,12 @@ static void run_tests_on_pipe(data_t *data)
>        igt_subtest_with_dynamic("cursor-alpha-opaque") {
>                for_each_pipe(&data->display, pipe) {
>                        data->pipe = pipe;
> -
> +                     create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
----------------------- ^
> +                     if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) {
> +                             igt_debug("Cursor size %dx%d not supported by driver\n",
> +                                       data->cursor_max_w, data->cursor_max_h);
> +                             continue;
> +                     }

Put newline here.

>                        igt_dynamic_f("pipe-%s-%s",
>                                      kmstest_pipe_name(pipe),
>                                      data->output->name)
> @@ -814,7 +824,12 @@ static void run_tests_on_pipe(data_t *data)
>        igt_subtest_with_dynamic("cursor-alpha-transparent") {
>                for_each_pipe(&data->display, pipe) {
>                        data->pipe = pipe;
> -
> +                     create_cursor_fb(data, data->cursor_max_w, data->cursor_max_h);
----------------------- ^
> +                     if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) {
> +                             igt_debug("Cursor size %dx%d not supported by driver\n",
> +                                       data->cursor_max_w, data->cursor_max_h);
> +                             continue;
> +                     }

Put newline here.

Regards,
Kamil

>                        igt_dynamic_f("pipe-%s-%s",
>                                      kmstest_pipe_name(pipe),
>                                      data->output->name)
> --
> 2.35.1
>


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

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

* Re: [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D
  2023-01-03 18:31 ` Juha-Pekka Heikkila
@ 2023-01-05  8:07   ` Bhadane, Dnyaneshwar
  2023-01-06 12:28   ` Bhadane, Dnyaneshwar
  1 sibling, 0 replies; 17+ messages in thread
From: Bhadane, Dnyaneshwar @ 2023-01-05  8:07 UTC (permalink / raw)
  To: juhapekka.heikkila, igt-dev; +Cc: Borah, Chaitanya Kumar, Kurmi, Suresh Kumar

Hi Juha-Pekka,

> -----Original Message-----
> From: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>
> Sent: Wednesday, January 4, 2023 12:01 AM
> To: Bhadane, Dnyaneshwar <dnyaneshwar.bhadane@intel.com>; igt-
> dev@lists.freedesktop.org
> Cc: Kurmi, Suresh Kumar <suresh.kumar.kurmi@intel.com>; Borah, Chaitanya
> Kumar <chaitanya.kumar.borah@intel.com>
> Subject: Re: [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for
> Pipe C and D
> 
> Hi Dnyaneshwar,
> 
> On 30.12.2022 8.46, bhadanednyaneshwar wrote:
> > MSO eDP is not supported on pipe C and D. Added a test condition to
> > prevent tests from execution on pipe C and D.This condition was missed
> > for cursor-size-change,cursor-alpha-opaque and
> > cursor-alpha-transparent testcases.
> >
> > Inside require_cursor_size() checks first for eligiblity to igt commit
> > using test buffer.For MSO eDP, It is fail to commit for pipe C/pipe D
> > and require_cursor_size() return non zero value. So it will skip the
> > dynamic testcase for pipe C and D.
> >
> > Signed-off-by: bhadanednyaneshwar <dnyaneshwar.bhadane@intel.com>
> > ---
> >   tests/kms_cursor_crc.c | 21 ++++++++++++++++++---
> >   1 file changed, 18 insertions(+), 3 deletions(-)
> >
> > diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c index
> > 17f294d6..d8fb9c0d 100644
> > --- a/tests/kms_cursor_crc.c
> > +++ b/tests/kms_cursor_crc.c
> > @@ -786,7 +786,12 @@ static void run_tests_on_pipe(data_t *data)
> >   	igt_subtest_with_dynamic("cursor-size-change") {
> >   		for_each_pipe(&data->display, pipe) {
> >   			data->pipe = pipe;
> > -
> > +			create_cursor_fb(data, data->cursor_max_w, data-
> >cursor_max_h);
> > +			if (require_cursor_size(data, data->cursor_max_w,
> data->cursor_max_h)) {
> > +				igt_debug("Cursor size %dx%d not supported
> by driver\n",
> > +					  data->cursor_max_w, data-
> >cursor_max_h);
> > +				continue;
> > +			}
> 
> Would it work if instead of above checking you would use
> igt_pipe_connector_valid(..)? igt_pipe_connector_valid is using bits coming
> from encoder so I think it might work for this case and it would be much faster
> to check.
> 
> /Juha-Pekka

We have checked with igt_pipe_connector_valid() and it is working fine for the testcases.
We are able skip the pipes that are not valid on DRM_MODE_CONNECTED in output config
connection. I will update this patch according igt_pipe_connector_valid() check.
Thanks you for the suggestion.

> 
> >   			igt_dynamic_f("pipe-%s-%s",
> >   				      kmstest_pipe_name(pipe),
> >   				      data->output->name)
> > @@ -800,7 +805,12 @@ static void run_tests_on_pipe(data_t *data)
> >   	igt_subtest_with_dynamic("cursor-alpha-opaque") {
> >   		for_each_pipe(&data->display, pipe) {
> >   			data->pipe = pipe;
> > -
> > +			create_cursor_fb(data, data->cursor_max_w, data-
> >cursor_max_h);
> > +			if (require_cursor_size(data, data->cursor_max_w,
> data->cursor_max_h)) {
> > +				igt_debug("Cursor size %dx%d not supported
> by driver\n",
> > +					  data->cursor_max_w, data-
> >cursor_max_h);
> > +				continue;
> > +			}
> >   			igt_dynamic_f("pipe-%s-%s",
> >   				      kmstest_pipe_name(pipe),
> >   				      data->output->name)
> > @@ -814,7 +824,12 @@ static void run_tests_on_pipe(data_t *data)
> >   	igt_subtest_with_dynamic("cursor-alpha-transparent") {
> >   		for_each_pipe(&data->display, pipe) {
> >   			data->pipe = pipe;
> > -
> > +			create_cursor_fb(data, data->cursor_max_w, data-
> >cursor_max_h);
> > +			if (require_cursor_size(data, data->cursor_max_w,
> data->cursor_max_h)) {
> > +				igt_debug("Cursor size %dx%d not supported
> by driver\n",
> > +					  data->cursor_max_w, data-
> >cursor_max_h);
> > +				continue;
> > +			}
> >   			igt_dynamic_f("pipe-%s-%s",
> >   				      kmstest_pipe_name(pipe),
> >   				      data->output->name)


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

* Re: [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D
  2023-01-03 18:31 ` Juha-Pekka Heikkila
  2023-01-05  8:07   ` Bhadane, Dnyaneshwar
@ 2023-01-06 12:28   ` Bhadane, Dnyaneshwar
  1 sibling, 0 replies; 17+ messages in thread
From: Bhadane, Dnyaneshwar @ 2023-01-06 12:28 UTC (permalink / raw)
  To: juhapekka.heikkila, igt-dev; +Cc: Borah, Chaitanya Kumar, Kurmi, Suresh Kumar

Hi Juha-Pekka,

> -----Original Message-----
> From: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>
> Sent: Wednesday, January 4, 2023 12:01 AM
> To: Bhadane, Dnyaneshwar <dnyaneshwar.bhadane@intel.com>; igt-
> dev@lists.freedesktop.org
> Cc: Kurmi, Suresh Kumar <suresh.kumar.kurmi@intel.com>; Borah, Chaitanya
> Kumar <chaitanya.kumar.borah@intel.com>
> Subject: Re: [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for
> Pipe C and D
> 
> Hi Dnyaneshwar,
> 
> On 30.12.2022 8.46, bhadanednyaneshwar wrote:
> > MSO eDP is not supported on pipe C and D. Added a test condition to
> > prevent tests from execution on pipe C and D.This condition was missed
> > for cursor-size-change,cursor-alpha-opaque and
> > cursor-alpha-transparent testcases.
> >
> > Inside require_cursor_size() checks first for eligiblity to igt commit
> > using test buffer.For MSO eDP, It is fail to commit for pipe C/pipe D
> > and require_cursor_size() return non zero value. So it will skip the
> > dynamic testcase for pipe C and D.
> >
> > Signed-off-by: bhadanednyaneshwar <dnyaneshwar.bhadane@intel.com>
> > ---
> >   tests/kms_cursor_crc.c | 21 ++++++++++++++++++---
> >   1 file changed, 18 insertions(+), 3 deletions(-)
> >
> > diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c index
> > 17f294d6..d8fb9c0d 100644
> > --- a/tests/kms_cursor_crc.c
> > +++ b/tests/kms_cursor_crc.c
> > @@ -786,7 +786,12 @@ static void run_tests_on_pipe(data_t *data)
> >   	igt_subtest_with_dynamic("cursor-size-change") {
> >   		for_each_pipe(&data->display, pipe) {
> >   			data->pipe = pipe;
> > -
> > +			create_cursor_fb(data, data->cursor_max_w, data-
> >cursor_max_h);
> > +			if (require_cursor_size(data, data->cursor_max_w,
> data->cursor_max_h)) {
> > +				igt_debug("Cursor size %dx%d not supported
> by driver\n",
> > +					  data->cursor_max_w, data-
> >cursor_max_h);
> > +				continue;
> > +			}
> 
> Would it work if instead of above checking you would use
> igt_pipe_connector_valid(..)? igt_pipe_connector_valid is using bits coming
> from encoder so I think it might work for this case and it would be much faster
> to check.

Using this igt_pipe_connector_valid, It skip pipe of C and D on these three testcases
(for cursor-size-change,cursor-alpha-opaque and cursor-alpha-transparent) which were
failing for MSO.
For the other testcase cursor-suspend, cursor-dpms and max-size in
kms_cursor_crc.c ,there is already check of require_cursor_size() it avoids their failure 
for the MSO(on Pipe C/D). Do you suggest we should add the igt_pipe_connector_valid()
to these testcases also, Just before the require_cursor_size() ?

Regards,
Dnyaneshwar

> 
> /Juha-Pekka
> 
> >   			igt_dynamic_f("pipe-%s-%s",
> >   				      kmstest_pipe_name(pipe),
> >   				      data->output->name)
> > @@ -800,7 +805,12 @@ static void run_tests_on_pipe(data_t *data)
> >   	igt_subtest_with_dynamic("cursor-alpha-opaque") {
> >   		for_each_pipe(&data->display, pipe) {
> >   			data->pipe = pipe;
> > -
> > +			create_cursor_fb(data, data->cursor_max_w, data-
> >cursor_max_h);
> > +			if (require_cursor_size(data, data->cursor_max_w,
> data->cursor_max_h)) {
> > +				igt_debug("Cursor size %dx%d not supported
> by driver\n",
> > +					  data->cursor_max_w, data-
> >cursor_max_h);
> > +				continue;
> > +			}
> >   			igt_dynamic_f("pipe-%s-%s",
> >   				      kmstest_pipe_name(pipe),
> >   				      data->output->name)
> > @@ -814,7 +824,12 @@ static void run_tests_on_pipe(data_t *data)
> >   	igt_subtest_with_dynamic("cursor-alpha-transparent") {
> >   		for_each_pipe(&data->display, pipe) {
> >   			data->pipe = pipe;
> > -
> > +			create_cursor_fb(data, data->cursor_max_w, data-
> >cursor_max_h);
> > +			if (require_cursor_size(data, data->cursor_max_w,
> data->cursor_max_h)) {
> > +				igt_debug("Cursor size %dx%d not supported
> by driver\n",
> > +					  data->cursor_max_w, data-
> >cursor_max_h);
> > +				continue;
> > +			}
> >   			igt_dynamic_f("pipe-%s-%s",
> >   				      kmstest_pipe_name(pipe),
> >   				      data->output->name)


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

* [igt-dev] [i-g-t v2] tests/kms_cursor_crc: Skip for invalid pipe connector connection
  2022-12-30  6:46 [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D bhadanednyaneshwar
                   ` (5 preceding siblings ...)
  2023-01-03 18:31 ` Juha-Pekka Heikkila
@ 2023-01-09  6:49 ` Dnyaneshwar Bhadane
  2023-01-09 15:19   ` Juha-Pekka Heikkila
  2023-01-09  7:18 ` [igt-dev] ✗ Fi.CI.BAT: failure for tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D (rev2) Patchwork
                   ` (2 subsequent siblings)
  9 siblings, 1 reply; 17+ messages in thread
From: Dnyaneshwar Bhadane @ 2023-01-09  6:49 UTC (permalink / raw)
  To: igt-dev
  Cc: suresh.kumar.kurmi, dnyaneshwar.bhadane, juha-pekka.heikkila,
	chaitanya.kumar.borah

Added a skip condition that checks for the valid pipe connection. This is a 
generic skip on pipe which checks for valid connector connection mask bit. 
This condition helps to skip tests for MSO panels on non supported pipes
(C and D).This condition was missed for cursor-size-change,
cursor-alpha-opaque and cursor-alpha-transparent testcases.

--v2
- Change commit messege  from "Add Gaurd for MSO eDP for Pipe C and D"
to "Skip condition for non valid pipe connector connection" [Kamil]
- skip condition using igt_pipe_connector_valid() [Juha-Pekka]

Signed-off-by: Dnyaneshwar Bhadane <dnyaneshwar.bhadane@intel.com>

---
 tests/kms_cursor_crc.c | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c
index 17f294d6..87497b9e 100644
--- a/tests/kms_cursor_crc.c
+++ b/tests/kms_cursor_crc.c
@@ -786,6 +786,13 @@ static void run_tests_on_pipe(data_t *data)
 	igt_subtest_with_dynamic("cursor-size-change") {
 		for_each_pipe(&data->display, pipe) {
 			data->pipe = pipe;
+			
+			if(!igt_pipe_connector_valid(pipe, data->output)) {
+				igt_debug("Invalid connector on pipe-%s-%s",
+				      kmstest_pipe_name(pipe),
+				      data->output->name);
+				continue;
+			}
 
 			igt_dynamic_f("pipe-%s-%s",
 				      kmstest_pipe_name(pipe),
@@ -800,6 +807,13 @@ static void run_tests_on_pipe(data_t *data)
 	igt_subtest_with_dynamic("cursor-alpha-opaque") {
 		for_each_pipe(&data->display, pipe) {
 			data->pipe = pipe;
+			
+			if(!igt_pipe_connector_valid(pipe, data->output)) {
+				igt_debug("Invalid connector on pipe-%s-%s",
+				      kmstest_pipe_name(pipe),
+				      data->output->name);
+				continue;
+			}
 
 			igt_dynamic_f("pipe-%s-%s",
 				      kmstest_pipe_name(pipe),
@@ -814,6 +828,13 @@ static void run_tests_on_pipe(data_t *data)
 	igt_subtest_with_dynamic("cursor-alpha-transparent") {
 		for_each_pipe(&data->display, pipe) {
 			data->pipe = pipe;
+			
+			if(!igt_pipe_connector_valid(pipe, data->output)) {
+				igt_debug("Invalid connector on pipe-%s-%s",
+				      kmstest_pipe_name(pipe),
+				      data->output->name);
+				continue;
+			}
 
 			igt_dynamic_f("pipe-%s-%s",
 				      kmstest_pipe_name(pipe),
-- 
2.35.1

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

* [igt-dev] ✗ Fi.CI.BAT: failure for tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D (rev2)
  2022-12-30  6:46 [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D bhadanednyaneshwar
                   ` (6 preceding siblings ...)
  2023-01-09  6:49 ` [igt-dev] [i-g-t v2] tests/kms_cursor_crc: Skip for invalid pipe connector connection Dnyaneshwar Bhadane
@ 2023-01-09  7:18 ` Patchwork
  2023-01-09  7:52 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D (rev3) Patchwork
  2023-01-09  9:32 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
  9 siblings, 0 replies; 17+ messages in thread
From: Patchwork @ 2023-01-09  7:18 UTC (permalink / raw)
  To: Dnyaneshwar Bhadane; +Cc: igt-dev

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

== Series Details ==

Series: tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D (rev2)
URL   : https://patchwork.freedesktop.org/series/112305/
State : failure

== Summary ==

CI Bug Log - changes from CI_DRM_12556 -> IGTPW_8312
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with IGTPW_8312 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_8312, please notify your bug team 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/IGTPW_8312/index.html

Participating hosts (39 -> 39)
------------------------------

  Additional (1): bat-rpls-2 
  Missing    (1): fi-snb-2520m 

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

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

### IGT changes ###

#### Possible regressions ####

  * igt@i915_selftest@live@gt_lrc:
    - fi-adl-ddr5:        [PASS][1] -> [DMESG-FAIL][2]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/fi-adl-ddr5/igt@i915_selftest@live@gt_lrc.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8312/fi-adl-ddr5/igt@i915_selftest@live@gt_lrc.html

  
#### Suppressed ####

  The following results come from untrusted machines, tests, or statuses.
  They do not affect the overall result.

  * igt@i915_selftest@live@gt_heartbeat:
    - {bat-kbl-2}:        [PASS][3] -> [DMESG-FAIL][4]
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/bat-kbl-2/igt@i915_selftest@live@gt_heartbeat.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8312/bat-kbl-2/igt@i915_selftest@live@gt_heartbeat.html

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

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

### IGT changes ###

#### Possible fixes ####

  * igt@i915_selftest@live@gt_lrc:
    - {bat-rpls-1}:       [INCOMPLETE][5] ([i915#4983]) -> [PASS][6]
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/bat-rpls-1/igt@i915_selftest@live@gt_lrc.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8312/bat-rpls-1/igt@i915_selftest@live@gt_lrc.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
  [fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295
  [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
  [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
  [i915#1845]: https://gitlab.freedesktop.org/drm/intel/issues/1845
  [i915#1849]: https://gitlab.freedesktop.org/drm/intel/issues/1849
  [i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582
  [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282
  [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
  [i915#3637]: https://gitlab.freedesktop.org/drm/intel/issues/3637
  [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
  [i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
  [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
  [i915#4983]: https://gitlab.freedesktop.org/drm/intel/issues/4983
  [i915#5153]: https://gitlab.freedesktop.org/drm/intel/issues/5153
  [i915#6367]: https://gitlab.freedesktop.org/drm/intel/issues/6367
  [i915#6621]: https://gitlab.freedesktop.org/drm/intel/issues/6621
  [i915#7359]: https://gitlab.freedesktop.org/drm/intel/issues/7359
  [i915#7456]: https://gitlab.freedesktop.org/drm/intel/issues/7456
  [i915#7561]: https://gitlab.freedesktop.org/drm/intel/issues/7561


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

  * CI: CI-20190529 -> None
  * IGT: IGT_7110 -> IGTPW_8312

  CI-20190529: 20190529
  CI_DRM_12556: ac04152253dccfb02dcedfa0c57443122cf79314 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_8312: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8312/index.html
  IGT_7110: db10a19b94d1d7ae5ba62eb48d52c47ccb27766f @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git

== Logs ==

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

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

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

* [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D (rev3)
  2022-12-30  6:46 [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D bhadanednyaneshwar
                   ` (7 preceding siblings ...)
  2023-01-09  7:18 ` [igt-dev] ✗ Fi.CI.BAT: failure for tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D (rev2) Patchwork
@ 2023-01-09  7:52 ` Patchwork
  2023-01-09  9:32 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
  9 siblings, 0 replies; 17+ messages in thread
From: Patchwork @ 2023-01-09  7:52 UTC (permalink / raw)
  To: Dnyaneshwar Bhadane; +Cc: igt-dev

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

== Series Details ==

Series: tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D (rev3)
URL   : https://patchwork.freedesktop.org/series/112305/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_12556 -> IGTPW_8313
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

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

Participating hosts (39 -> 39)
------------------------------

  Additional (2): fi-kbl-soraka bat-rpls-2 
  Missing    (2): fi-bsw-kefka fi-snb-2520m 

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

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

### IGT changes ###

#### Suppressed ####

  The following results come from untrusted machines, tests, or statuses.
  They do not affect the overall result.

  * igt@gem_exec_suspend@basic-s0@smem:
    - {bat-rpls-1}:       NOTRUN -> [DMESG-WARN][1]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/bat-rpls-1/igt@gem_exec_suspend@basic-s0@smem.html

  * igt@i915_module_load@load:
    - {bat-dg2-8}:        [PASS][2] -> [FAIL][3]
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/bat-dg2-8/igt@i915_module_load@load.html
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/bat-dg2-8/igt@i915_module_load@load.html

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

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

### IGT changes ###

#### Issues hit ####

  * igt@gem_exec_gttfill@basic:
    - fi-kbl-soraka:      NOTRUN -> [SKIP][4] ([fdo#109271]) +7 similar issues
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/fi-kbl-soraka/igt@gem_exec_gttfill@basic.html
    - fi-pnv-d510:        [PASS][5] -> [FAIL][6] ([i915#7229])
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/fi-pnv-d510/igt@gem_exec_gttfill@basic.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/fi-pnv-d510/igt@gem_exec_gttfill@basic.html

  * igt@gem_huc_copy@huc-copy:
    - fi-kbl-soraka:      NOTRUN -> [SKIP][7] ([fdo#109271] / [i915#2190])
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/fi-kbl-soraka/igt@gem_huc_copy@huc-copy.html

  * igt@gem_lmem_swapping@basic:
    - fi-kbl-soraka:      NOTRUN -> [SKIP][8] ([fdo#109271] / [i915#4613]) +3 similar issues
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/fi-kbl-soraka/igt@gem_lmem_swapping@basic.html

  * igt@i915_selftest@live@execlists:
    - fi-bsw-n3050:       [PASS][9] -> [INCOMPLETE][10] ([i915#6972])
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/fi-bsw-n3050/igt@i915_selftest@live@execlists.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/fi-bsw-n3050/igt@i915_selftest@live@execlists.html
    - fi-bsw-nick:        [PASS][11] -> [INCOMPLETE][12] ([i915#6972])
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/fi-bsw-nick/igt@i915_selftest@live@execlists.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/fi-bsw-nick/igt@i915_selftest@live@execlists.html

  * igt@i915_selftest@live@gt_heartbeat:
    - fi-kbl-soraka:      NOTRUN -> [DMESG-FAIL][13] ([i915#5334])
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/fi-kbl-soraka/igt@i915_selftest@live@gt_heartbeat.html

  * igt@i915_selftest@live@gt_pm:
    - fi-kbl-soraka:      NOTRUN -> [DMESG-FAIL][14] ([i915#1886])
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/fi-kbl-soraka/igt@i915_selftest@live@gt_pm.html

  * igt@kms_chamelium@hdmi-hpd-fast:
    - fi-kbl-soraka:      NOTRUN -> [SKIP][15] ([fdo#109271] / [fdo#111827]) +7 similar issues
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/fi-kbl-soraka/igt@kms_chamelium@hdmi-hpd-fast.html

  * igt@runner@aborted:
    - fi-bsw-n3050:       NOTRUN -> [FAIL][16] ([fdo#109271] / [i915#4312])
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/fi-bsw-n3050/igt@runner@aborted.html
    - fi-bsw-nick:        NOTRUN -> [FAIL][17] ([fdo#109271] / [i915#4312])
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/fi-bsw-nick/igt@runner@aborted.html

  
#### Possible fixes ####

  * igt@i915_selftest@live@gt_lrc:
    - {bat-rpls-1}:       [INCOMPLETE][18] ([i915#4983]) -> [PASS][19]
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/bat-rpls-1/igt@i915_selftest@live@gt_lrc.html
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/bat-rpls-1/igt@i915_selftest@live@gt_lrc.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#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285
  [fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295
  [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
  [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
  [i915#1845]: https://gitlab.freedesktop.org/drm/intel/issues/1845
  [i915#1849]: https://gitlab.freedesktop.org/drm/intel/issues/1849
  [i915#1886]: https://gitlab.freedesktop.org/drm/intel/issues/1886
  [i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
  [i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582
  [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282
  [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
  [i915#3637]: https://gitlab.freedesktop.org/drm/intel/issues/3637
  [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
  [i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
  [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
  [i915#4983]: https://gitlab.freedesktop.org/drm/intel/issues/4983
  [i915#5334]: https://gitlab.freedesktop.org/drm/intel/issues/5334
  [i915#6367]: https://gitlab.freedesktop.org/drm/intel/issues/6367
  [i915#6621]: https://gitlab.freedesktop.org/drm/intel/issues/6621
  [i915#6972]: https://gitlab.freedesktop.org/drm/intel/issues/6972
  [i915#7229]: https://gitlab.freedesktop.org/drm/intel/issues/7229
  [i915#7456]: https://gitlab.freedesktop.org/drm/intel/issues/7456
  [i915#7561]: https://gitlab.freedesktop.org/drm/intel/issues/7561


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

  * CI: CI-20190529 -> None
  * IGT: IGT_7110 -> IGTPW_8313

  CI-20190529: 20190529
  CI_DRM_12556: ac04152253dccfb02dcedfa0c57443122cf79314 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_8313: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/index.html
  IGT_7110: db10a19b94d1d7ae5ba62eb48d52c47ccb27766f @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git

== Logs ==

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

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

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

* [igt-dev] ✓ Fi.CI.IGT: success for tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D (rev3)
  2022-12-30  6:46 [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D bhadanednyaneshwar
                   ` (8 preceding siblings ...)
  2023-01-09  7:52 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D (rev3) Patchwork
@ 2023-01-09  9:32 ` Patchwork
  9 siblings, 0 replies; 17+ messages in thread
From: Patchwork @ 2023-01-09  9:32 UTC (permalink / raw)
  To: Dnyaneshwar Bhadane; +Cc: igt-dev

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

== Series Details ==

Series: tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D (rev3)
URL   : https://patchwork.freedesktop.org/series/112305/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_12556_full -> IGTPW_8313_full
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

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

Participating hosts (14 -> 11)
------------------------------

  Missing    (3): pig-skl-6260u pig-kbl-iris pig-glk-j5005 

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

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

### IGT changes ###

#### Suppressed ####

  The following results come from untrusted machines, tests, or statuses.
  They do not affect the overall result.

  * igt@kms_ccs@pipe-c-bad-rotation-90-y_tiled_gen12_rc_ccs_cc:
    - {shard-dg1}:        [PASS][1] -> [DMESG-WARN][2] +1 similar issue
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-dg1-14/igt@kms_ccs@pipe-c-bad-rotation-90-y_tiled_gen12_rc_ccs_cc.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-dg1-14/igt@kms_ccs@pipe-c-bad-rotation-90-y_tiled_gen12_rc_ccs_cc.html

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

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

### IGT changes ###

#### Issues hit ####

  * igt@gem_exec_fair@basic-deadline:
    - shard-glk:          [PASS][3] -> [FAIL][4] ([i915#2846])
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-glk2/igt@gem_exec_fair@basic-deadline.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-glk4/igt@gem_exec_fair@basic-deadline.html

  * igt@gem_partial_pwrite_pread@writes-after-reads-uncached:
    - shard-apl:          [PASS][5] -> [INCOMPLETE][6] ([i915#7708]) +2 similar issues
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-apl7/igt@gem_partial_pwrite_pread@writes-after-reads-uncached.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-apl2/igt@gem_partial_pwrite_pread@writes-after-reads-uncached.html

  * igt@kms_ccs@pipe-b-ccs-on-another-bo-y_tiled_gen12_mc_ccs:
    - shard-apl:          NOTRUN -> [SKIP][7] ([fdo#109271] / [i915#3886])
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-apl2/igt@kms_ccs@pipe-b-ccs-on-another-bo-y_tiled_gen12_mc_ccs.html

  * igt@kms_chamelium@dp-hpd-after-suspend:
    - shard-apl:          NOTRUN -> [SKIP][8] ([fdo#109271] / [fdo#111827])
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-apl3/igt@kms_chamelium@dp-hpd-after-suspend.html

  * igt@kms_chamelium@vga-hpd-without-ddc:
    - shard-snb:          NOTRUN -> [SKIP][9] ([fdo#109271] / [fdo#111827]) +1 similar issue
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-snb2/igt@kms_chamelium@vga-hpd-without-ddc.html

  * igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions-varying-size:
    - shard-glk:          [PASS][10] -> [FAIL][11] ([i915#2346]) +1 similar issue
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-glk7/igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions-varying-size.html
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-glk2/igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions-varying-size.html

  * igt@kms_flip@2x-nonexisting-fb:
    - shard-apl:          NOTRUN -> [SKIP][12] ([fdo#109271]) +35 similar issues
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-apl1/igt@kms_flip@2x-nonexisting-fb.html

  * igt@kms_psr2_su@page_flip-xrgb8888:
    - shard-apl:          NOTRUN -> [SKIP][13] ([fdo#109271] / [i915#658])
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-apl1/igt@kms_psr2_su@page_flip-xrgb8888.html

  * igt@kms_vblank@pipe-c-query-busy-hang:
    - shard-snb:          NOTRUN -> [SKIP][14] ([fdo#109271]) +24 similar issues
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-snb5/igt@kms_vblank@pipe-c-query-busy-hang.html

  * igt@sysfs_clients@split-50:
    - shard-apl:          NOTRUN -> [SKIP][15] ([fdo#109271] / [i915#2994])
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-apl1/igt@sysfs_clients@split-50.html

  
#### Possible fixes ####

  * igt@drm_read@invalid-buffer:
    - {shard-tglu}:       [SKIP][16] ([i915#1845]) -> [PASS][17]
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-tglu-6/igt@drm_read@invalid-buffer.html
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-tglu-1/igt@drm_read@invalid-buffer.html

  * igt@drm_read@short-buffer-block:
    - {shard-rkl}:        [SKIP][18] ([i915#4098]) -> [PASS][19]
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-5/igt@drm_read@short-buffer-block.html
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-rkl-6/igt@drm_read@short-buffer-block.html

  * igt@gem_exec_fair@basic-flow@rcs0:
    - {shard-rkl}:        [FAIL][20] ([i915#2842]) -> [PASS][21]
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-2/igt@gem_exec_fair@basic-flow@rcs0.html
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-rkl-3/igt@gem_exec_fair@basic-flow@rcs0.html

  * igt@gem_exec_fair@basic-throttle@rcs0:
    - {shard-tglu}:       [FAIL][22] ([i915#2842]) -> [PASS][23]
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-tglu-1/igt@gem_exec_fair@basic-throttle@rcs0.html
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-tglu-7/igt@gem_exec_fair@basic-throttle@rcs0.html

  * igt@gem_exec_reloc@basic-gtt:
    - {shard-rkl}:        [SKIP][24] ([i915#3281]) -> [PASS][25] +5 similar issues
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-2/igt@gem_exec_reloc@basic-gtt.html
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-rkl-5/igt@gem_exec_reloc@basic-gtt.html

  * igt@gem_exec_schedule@semaphore-power:
    - {shard-rkl}:        [SKIP][26] ([i915#7276]) -> [PASS][27]
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-3/igt@gem_exec_schedule@semaphore-power.html
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-rkl-5/igt@gem_exec_schedule@semaphore-power.html

  * igt@gem_partial_pwrite_pread@writes-after-reads-snoop:
    - shard-apl:          [INCOMPLETE][28] ([i915#7708]) -> [PASS][29]
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-apl3/igt@gem_partial_pwrite_pread@writes-after-reads-snoop.html
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-apl7/igt@gem_partial_pwrite_pread@writes-after-reads-snoop.html

  * igt@gem_pwrite@basic-self:
    - {shard-rkl}:        [SKIP][30] ([i915#3282]) -> [PASS][31] +3 similar issues
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-1/igt@gem_pwrite@basic-self.html
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-rkl-5/igt@gem_pwrite@basic-self.html

  * igt@gen9_exec_parse@unaligned-jump:
    - {shard-rkl}:        [SKIP][32] ([i915#2527]) -> [PASS][33]
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-1/igt@gen9_exec_parse@unaligned-jump.html
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-rkl-5/igt@gen9_exec_parse@unaligned-jump.html

  * igt@i915_pm_dc@dc9-dpms:
    - {shard-tglu}:       [SKIP][34] ([i915#4281]) -> [PASS][35]
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-tglu-8/igt@i915_pm_dc@dc9-dpms.html
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-tglu-2/igt@i915_pm_dc@dc9-dpms.html

  * igt@i915_pm_rpm@dpms-lpsp:
    - {shard-tglu}:       [SKIP][36] ([i915#1397]) -> [PASS][37]
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-tglu-6/igt@i915_pm_rpm@dpms-lpsp.html
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-tglu-3/igt@i915_pm_rpm@dpms-lpsp.html

  * igt@i915_selftest@live@dmabuf:
    - shard-apl:          [DMESG-FAIL][38] ([i915#7562]) -> [PASS][39]
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-apl2/igt@i915_selftest@live@dmabuf.html
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-apl1/igt@i915_selftest@live@dmabuf.html

  * igt@i915_suspend@basic-s3-without-i915:
    - shard-snb:          [INCOMPLETE][40] ([i915#4528] / [i915#4817]) -> [PASS][41]
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-snb7/igt@i915_suspend@basic-s3-without-i915.html
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-snb4/igt@i915_suspend@basic-s3-without-i915.html

  * igt@kms_atomic@plane-overlay-legacy:
    - {shard-rkl}:        [SKIP][42] ([i915#1845] / [i915#4098]) -> [PASS][43] +22 similar issues
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-1/igt@kms_atomic@plane-overlay-legacy.html
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-rkl-6/igt@kms_atomic@plane-overlay-legacy.html

  * igt@kms_ccs@pipe-a-crc-primary-basic-y_tiled_gen12_rc_ccs_cc:
    - {shard-tglu}:       [SKIP][44] ([i915#7651]) -> [PASS][45] +7 similar issues
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-tglu-6/igt@kms_ccs@pipe-a-crc-primary-basic-y_tiled_gen12_rc_ccs_cc.html
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-tglu-3/igt@kms_ccs@pipe-a-crc-primary-basic-y_tiled_gen12_rc_ccs_cc.html

  * igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions:
    - shard-apl:          [FAIL][46] ([i915#2346]) -> [PASS][47]
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-apl2/igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions.html
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-apl7/igt@kms_cursor_legacy@flip-vs-cursor@atomic-transitions.html

  * igt@kms_cursor_legacy@long-nonblocking-modeset-vs-cursor-atomic:
    - shard-snb:          [SKIP][48] ([fdo#109271]) -> [PASS][49]
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-snb2/igt@kms_cursor_legacy@long-nonblocking-modeset-vs-cursor-atomic.html
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-snb2/igt@kms_cursor_legacy@long-nonblocking-modeset-vs-cursor-atomic.html

  * igt@kms_flip@2x-plain-flip-fb-recreate-interruptible@ab-hdmi-a1-hdmi-a2:
    - shard-glk:          [FAIL][50] ([i915#2122]) -> [PASS][51]
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-glk8/igt@kms_flip@2x-plain-flip-fb-recreate-interruptible@ab-hdmi-a1-hdmi-a2.html
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-glk2/igt@kms_flip@2x-plain-flip-fb-recreate-interruptible@ab-hdmi-a1-hdmi-a2.html

  * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-blt:
    - {shard-tglu}:       [SKIP][52] ([i915#1849]) -> [PASS][53] +2 similar issues
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-tglu-6/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-blt.html
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-tglu-2/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-blt.html

  * igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-pwrite:
    - {shard-rkl}:        [SKIP][54] ([i915#1849] / [i915#4098]) -> [PASS][55] +15 similar issues
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-5/igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-pwrite.html
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-rkl-6/igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-pwrite.html

  * igt@kms_plane@pixel-format@pipe-b-planes:
    - {shard-rkl}:        [SKIP][56] ([i915#1849]) -> [PASS][57] +3 similar issues
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-2/igt@kms_plane@pixel-format@pipe-b-planes.html
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-rkl-6/igt@kms_plane@pixel-format@pipe-b-planes.html

  * igt@kms_psr@primary_mmap_cpu:
    - {shard-rkl}:        [SKIP][58] ([i915#1072]) -> [PASS][59]
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-4/igt@kms_psr@primary_mmap_cpu.html
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-rkl-6/igt@kms_psr@primary_mmap_cpu.html

  * igt@kms_universal_plane@universal-plane-pipe-a-functional:
    - {shard-tglu}:       [SKIP][60] ([fdo#109274]) -> [PASS][61]
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-tglu-6/igt@kms_universal_plane@universal-plane-pipe-a-functional.html
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-tglu-5/igt@kms_universal_plane@universal-plane-pipe-a-functional.html

  * igt@kms_vblank@pipe-a-wait-forked-busy-hang:
    - {shard-tglu}:       [SKIP][62] ([i915#1845] / [i915#7651]) -> [PASS][63]
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-tglu-6/igt@kms_vblank@pipe-a-wait-forked-busy-hang.html
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-tglu-2/igt@kms_vblank@pipe-a-wait-forked-busy-hang.html

  * igt@perf@gen12-unprivileged-single-ctx-counters:
    - {shard-rkl}:        [SKIP][64] ([fdo#109289]) -> [PASS][65]
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-rkl-5/igt@perf@gen12-unprivileged-single-ctx-counters.html
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-rkl-1/igt@perf@gen12-unprivileged-single-ctx-counters.html

  * igt@sysfs_timeslice_duration@timeout@rcs0:
    - {shard-dg1}:        [FAIL][66] ([i915#1755]) -> [PASS][67] +4 similar issues
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-dg1-15/igt@sysfs_timeslice_duration@timeout@rcs0.html
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-dg1-13/igt@sysfs_timeslice_duration@timeout@rcs0.html

  
#### Warnings ####

  * igt@gem_exec_capture@capture-invisible@smem0:
    - shard-glk:          [SKIP][68] ([fdo#109271]) -> [SKIP][69] ([fdo#109271] / [i915#6334])
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-glk8/igt@gem_exec_capture@capture-invisible@smem0.html
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-glk5/igt@gem_exec_capture@capture-invisible@smem0.html
    - shard-apl:          [SKIP][70] ([fdo#109271]) -> [SKIP][71] ([fdo#109271] / [i915#6334])
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12556/shard-apl7/igt@gem_exec_capture@capture-invisible@smem0.html
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/shard-apl1/igt@gem_exec_capture@capture-invisible@smem0.html

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

  [fdo#103375]: https://bugs.freedesktop.org/show_bug.cgi?id=103375
  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [fdo#109274]: https://bugs.freedesktop.org/show_bug.cgi?id=109274
  [fdo#109279]: https://bugs.freedesktop.org/show_bug.cgi?id=109279
  [fdo#109280]: https://bugs.freedesktop.org/show_bug.cgi?id=109280
  [fdo#109283]: https://bugs.freedesktop.org/show_bug.cgi?id=109283
  [fdo#109289]: https://bugs.freedesktop.org/show_bug.cgi?id=109289
  [fdo#109291]: https://bugs.freedesktop.org/show_bug.cgi?id=109291
  [fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295
  [fdo#109300]: https://bugs.freedesktop.org/show_bug.cgi?id=109300
  [fdo#109302]: https://bugs.freedesktop.org/show_bug.cgi?id=109302
  [fdo#109307]: https://bugs.freedesktop.org/show_bug.cgi?id=109307
  [fdo#109308]: https://bugs.freedesktop.org/show_bug.cgi?id=109308
  [fdo#109309]: https://bugs.freedesktop.org/show_bug.cgi?id=109309
  [fdo#109313]: https://bugs.freedesktop.org/show_bug.cgi?id=109313
  [fdo#109314]: https://bugs.freedesktop.org/show_bug.cgi?id=109314
  [fdo#109315]: https://bugs.freedesktop.org/show_bug.cgi?id=109315
  [fdo#109506]: https://bugs.freedesktop.org/show_bug.cgi?id=109506
  [fdo#109642]: https://bugs.freedesktop.org/show_bug.cgi?id=109642
  [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#111644]: https://bugs.freedesktop.org/show_bug.cgi?id=111644
  [fdo#111656]: https://bugs.freedesktop.org/show_bug.cgi?id=111656
  [fdo#111825]: https://bugs.freedesktop.org/show_bug.cgi?id=111825
  [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
  [fdo#112054]: https://bugs.freedesktop.org/show_bug.cgi?id=112054
  [fdo#112283]: https://bugs.freedesktop.org/show_bug.cgi?id=112283
  [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
  [i915#132]: https://gitlab.freedesktop.org/drm/intel/issues/132
  [i915#1397]: https://gitlab.freedesktop.org/drm/intel/issues/1397
  [i915#1755]: https://gitlab.freedesktop.org/drm/intel/issues/1755
  [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#2122]: https://gitlab.freedesktop.org/drm/intel/issues/2122
  [i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
  [i915#2346]: https://gitlab.freedesktop.org/drm/intel/issues/2346
  [i915#2434]: https://gitlab.freedesktop.org/drm/intel/issues/2434
  [i915#2435]: https://gitlab.freedesktop.org/drm/intel/issues/2435
  [i915#2436]: https://gitlab.freedesktop.org/drm/intel/issues/2436
  [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#2587]: https://gitlab.freedesktop.org/drm/intel/issues/2587
  [i915#2658]: https://gitlab.freedesktop.org/drm/intel/issues/2658
  [i915#2672]: https://gitlab.freedesktop.org/drm/intel/issues/2672
  [i915#2681]: https://gitlab.freedesktop.org/drm/intel/issues/2681
  [i915#2705]: https://gitlab.freedesktop.org/drm/intel/issues/2705
  [i915#280]: https://gitlab.freedesktop.org/drm/intel/issues/280
  [i915#284]: https://gitlab.freedesktop.org/drm/intel/issues/284
  [i915#2842]: https://gitlab.freedesktop.org/drm/intel/issues/2842
  [i915#2846]: https://gitlab.freedesktop.org/drm/intel/issues/2846
  [i915#2856]: https://gitlab.freedesktop.org/drm/intel/issues/2856
  [i915#2920]: https://gitlab.freedesktop.org/drm/intel/issues/2920
  [i915#2994]: https://gitlab.freedesktop.org/drm/intel/issues/2994
  [i915#3116]: https://gitlab.freedesktop.org/drm/intel/issues/3116
  [i915#315]: https://gitlab.freedesktop.org/drm/intel/issues/315
  [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#3318]: https://gitlab.freedesktop.org/drm/intel/issues/3318
  [i915#3323]: https://gitlab.freedesktop.org/drm/intel/issues/3323
  [i915#3359]: https://gitlab.freedesktop.org/drm/intel/issues/3359
  [i915#3458]: https://gitlab.freedesktop.org/drm/intel/issues/3458
  [i915#3469]: https://gitlab.freedesktop.org/drm/intel/issues/3469
  [i915#3528]: https://gitlab.freedesktop.org/drm/intel/issues/3528
  [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#3689]: https://gitlab.freedesktop.org/drm/intel/issues/3689
  [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
  [i915#3734]: https://gitlab.freedesktop.org/drm/intel/issues/3734
  [i915#3825]: https://gitlab.freedesktop.org/drm/intel/issues/3825
  [i915#3826]: https://gitlab.freedesktop.org/drm/intel/issues/3826
  [i915#3840]: https://gitlab.freedesktop.org/drm/intel/issues/3840
  [i915#3886]: https://gitlab.freedesktop.org/drm/intel/issues/3886
  [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#4078]: https://gitlab.freedesktop.org/drm/intel/issues/4078
  [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#4258]: https://gitlab.freedesktop.org/drm/intel/issues/4258
  [i915#4270]: https://gitlab.freedesktop.org/drm/intel/issues/4270
  [i915#4281]: https://gitlab.freedesktop.org/drm/intel/issues/4281
  [i915#433]: https://gitlab.freedesktop.org/drm/intel/issues/433
  [i915#4387]: https://gitlab.freedesktop.org/drm/intel/issues/4387
  [i915#4525]: https://gitlab.freedesktop.org/drm/intel/issues/4525
  [i915#4528]: https://gitlab.freedesktop.org/drm/intel/issues/4528
  [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
  [i915#4812]: https://gitlab.freedesktop.org/drm/intel/issues/4812
  [i915#4817]: https://gitlab.freedesktop.org/drm/intel/issues/4817
  [i915#4833]: https://gitlab.freedesktop.org/drm/intel/issues/4833
  [i915#4852]: https://gitlab.freedesktop.org/drm/intel/issues/4852
  [i915#4877]: https://gitlab.freedesktop.org/drm/intel/issues/4877
  [i915#5176]: https://gitlab.freedesktop.org/drm/intel/issues/5176
  [i915#5235]: https://gitlab.freedesktop.org/drm/intel/issues/5235
  [i915#5286]: https://gitlab.freedesktop.org/drm/intel/issues/5286
  [i915#5288]: https://gitlab.freedesktop.org/drm/intel/issues/5288
  [i915#5289]: https://gitlab.freedesktop.org/drm/intel/issues/5289
  [i915#5325]: https://gitlab.freedesktop.org/drm/intel/issues/5325
  [i915#5327]: https://gitlab.freedesktop.org/drm/intel/issues/5327
  [i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533
  [i915#5439]: https://gitlab.freedesktop.org/drm/intel/issues/5439
  [i915#5461]: https://gitlab.freedesktop.org/drm/intel/issues/5461
  [i915#5784]: https://gitlab.freedesktop.org/drm/intel/issues/5784
  [i915#6095]: https://gitlab.freedesktop.org/drm/intel/issues/6095
  [i915#6248]: https://gitlab.freedesktop.org/drm/intel/issues/6248
  [i915#6301]: https://gitlab.freedesktop.org/drm/intel/issues/6301
  [i915#6334]: https://gitlab.freedesktop.org/drm/intel/issues/6334
  [i915#6412]: https://gitlab.freedesktop.org/drm/intel/issues/6412
  [i915#6433]: https://gitlab.freedesktop.org/drm/intel/issues/6433
  [i915#6497]: https://gitlab.freedesktop.org/drm/intel/issues/6497
  [i915#6524]: https://gitlab.freedesktop.org/drm/intel/issues/6524
  [i915#658]: https://gitlab.freedesktop.org/drm/intel/issues/658
  [i915#6768]: https://gitlab.freedesktop.org/drm/intel/issues/6768
  [i915#6944]: https://gitlab.freedesktop.org/drm/intel/issues/6944
  [i915#6946]: https://gitlab.freedesktop.org/drm/intel/issues/6946
  [i915#6953]: https://gitlab.freedesktop.org/drm/intel/issues/6953
  [i915#7037]: https://gitlab.freedesktop.org/drm/intel/issues/7037
  [i915#7116]: https://gitlab.freedesktop.org/drm/intel/issues/7116
  [i915#7118]: https://gitlab.freedesktop.org/drm/intel/issues/7118
  [i915#7128]: https://gitlab.freedesktop.org/drm/intel/issues/7128
  [i915#7178]: https://gitlab.freedesktop.org/drm/intel/issues/7178
  [i915#7276]: https://gitlab.freedesktop.org/drm/intel/issues/7276
  [i915#7294]: https://gitlab.freedesktop.org/drm/intel/issues/7294
  [i915#7561]: https://gitlab.freedesktop.org/drm/intel/issues/7561
  [i915#7562]: https://gitlab.freedesktop.org/drm/intel/issues/7562
  [i915#7582]: https://gitlab.freedesktop.org/drm/intel/issues/7582
  [i915#7651]: https://gitlab.freedesktop.org/drm/intel/issues/7651
  [i915#7679]: https://gitlab.freedesktop.org/drm/intel/issues/7679
  [i915#7697]: https://gitlab.freedesktop.org/drm/intel/issues/7697
  [i915#7707]: https://gitlab.freedesktop.org/drm/intel/issues/7707
  [i915#7708]: https://gitlab.freedesktop.org/drm/intel/issues/7708
  [i915#7711]: https://gitlab.freedesktop.org/drm/intel/issues/7711
  [i915#7742]: https://gitlab.freedesktop.org/drm/intel/issues/7742


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

  * CI: CI-20190529 -> None
  * IGT: IGT_7110 -> IGTPW_8313
  * Piglit: piglit_4509 -> None

  CI-20190529: 20190529
  CI_DRM_12556: ac04152253dccfb02dcedfa0c57443122cf79314 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_8313: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8313/index.html
  IGT_7110: db10a19b94d1d7ae5ba62eb48d52c47ccb27766f @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
  piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit

== Logs ==

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

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

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

* Re: [igt-dev] [i-g-t v2] tests/kms_cursor_crc: Skip for invalid pipe connector connection
  2023-01-09  6:49 ` [igt-dev] [i-g-t v2] tests/kms_cursor_crc: Skip for invalid pipe connector connection Dnyaneshwar Bhadane
@ 2023-01-09 15:19   ` Juha-Pekka Heikkila
  0 siblings, 0 replies; 17+ messages in thread
From: Juha-Pekka Heikkila @ 2023-01-09 15:19 UTC (permalink / raw)
  To: Dnyaneshwar Bhadane, igt-dev
  Cc: juha-pekka.heikkila, chaitanya.kumar.borah, suresh.kumar.kurmi

On 9.1.2023 8.49, Dnyaneshwar Bhadane wrote:
> Added a skip condition that checks for the valid pipe connection. This is a
> generic skip on pipe which checks for valid connector connection mask bit.
> This condition helps to skip tests for MSO panels on non supported pipes
> (C and D).This condition was missed for cursor-size-change,
> cursor-alpha-opaque and cursor-alpha-transparent testcases.
> 
> --v2
> - Change commit messege  from "Add Gaurd for MSO eDP for Pipe C and D"
> to "Skip condition for non valid pipe connector connection" [Kamil]
> - skip condition using igt_pipe_connector_valid() [Juha-Pekka]
> 
> Signed-off-by: Dnyaneshwar Bhadane <dnyaneshwar.bhadane@intel.com>
> 
> ---
>   tests/kms_cursor_crc.c | 21 +++++++++++++++++++++
>   1 file changed, 21 insertions(+)
> 
> diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c
> index 17f294d6..87497b9e 100644
> --- a/tests/kms_cursor_crc.c
> +++ b/tests/kms_cursor_crc.c
> @@ -786,6 +786,13 @@ static void run_tests_on_pipe(data_t *data)
>   	igt_subtest_with_dynamic("cursor-size-change") {
>   		for_each_pipe(&data->display, pipe) {
>   			data->pipe = pipe;
> +			
^^
in beginning of these blocks you have extra tab characters which will 
produce notifications about white space errors, leave just new line here 
but no tabs.

With those fixed
Reviewed-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>

> +			if(!igt_pipe_connector_valid(pipe, data->output)) {
> +				igt_debug("Invalid connector on pipe-%s-%s",
> +				      kmstest_pipe_name(pipe),
> +				      data->output->name);
> +				continue;
> +			}
>   
>   			igt_dynamic_f("pipe-%s-%s",
>   				      kmstest_pipe_name(pipe),
> @@ -800,6 +807,13 @@ static void run_tests_on_pipe(data_t *data)
>   	igt_subtest_with_dynamic("cursor-alpha-opaque") {
>   		for_each_pipe(&data->display, pipe) {
>   			data->pipe = pipe;
> +			
> +			if(!igt_pipe_connector_valid(pipe, data->output)) {
> +				igt_debug("Invalid connector on pipe-%s-%s",
> +				      kmstest_pipe_name(pipe),
> +				      data->output->name);
> +				continue;
> +			}
>   
>   			igt_dynamic_f("pipe-%s-%s",
>   				      kmstest_pipe_name(pipe),
> @@ -814,6 +828,13 @@ static void run_tests_on_pipe(data_t *data)
>   	igt_subtest_with_dynamic("cursor-alpha-transparent") {
>   		for_each_pipe(&data->display, pipe) {
>   			data->pipe = pipe;
> +			
> +			if(!igt_pipe_connector_valid(pipe, data->output)) {
> +				igt_debug("Invalid connector on pipe-%s-%s",
> +				      kmstest_pipe_name(pipe),
> +				      data->output->name);
> +				continue;
> +			}
>   
>   			igt_dynamic_f("pipe-%s-%s",
>   				      kmstest_pipe_name(pipe),

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

end of thread, other threads:[~2023-01-09 15:19 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-12-30  6:46 [igt-dev] [i-g-t] tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D bhadanednyaneshwar
2022-12-30 10:27 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2022-12-30 11:49 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2022-12-30 19:16 ` [igt-dev] [i-g-t] " Swati Sharma
2023-01-03  5:59   ` Bhadane, Dnyaneshwar
2023-01-03 14:50 ` Kamil Konieczny
2023-01-05  6:26   ` Bhadane, Dnyaneshwar
2023-01-03 17:41 ` Kamil Konieczny
2023-01-05  6:22   ` Bhadane, Dnyaneshwar
2023-01-03 18:31 ` Juha-Pekka Heikkila
2023-01-05  8:07   ` Bhadane, Dnyaneshwar
2023-01-06 12:28   ` Bhadane, Dnyaneshwar
2023-01-09  6:49 ` [igt-dev] [i-g-t v2] tests/kms_cursor_crc: Skip for invalid pipe connector connection Dnyaneshwar Bhadane
2023-01-09 15:19   ` Juha-Pekka Heikkila
2023-01-09  7:18 ` [igt-dev] ✗ Fi.CI.BAT: failure for tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D (rev2) Patchwork
2023-01-09  7:52 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_cursor_crc: Add Gaurd for MSO eDP for Pipe C and D (rev3) Patchwork
2023-01-09  9:32 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.