* [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation
@ 2023-10-04 10:25 Mika Kahola
2023-10-04 12:47 ` [Intel-gfx] ✓ Fi.CI.BAT: success for " Patchwork
` (2 more replies)
0 siblings, 3 replies; 12+ messages in thread
From: Mika Kahola @ 2023-10-04 10:25 UTC (permalink / raw)
To: intel-gfx
Every know and then we receive the following error when running
for example IGT test kms_flip.
[drm] *ERROR* PHY G Read 0d80 failed after 3 retries.
[drm] *ERROR* PHY G Write 0d81 failed after 3 retries.
Since the error is sporadic in nature, the patch proposes
to reset the message bus after every successful or unsuccessful
read or write operation. However, testing revealed that this
alone is not sufficient method an additiona delay is also
introduces anything from 200us to 300us. This delay is experimental
value and has no specification to back it up.
Signed-off-by: Mika Kahola <mika.kahola@intel.com>
---
drivers/gpu/drm/i915/display/intel_cx0_phy.c | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/drivers/gpu/drm/i915/display/intel_cx0_phy.c b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
index abd607b564f1..a71b8a29d6b0 100644
--- a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
+++ b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
@@ -220,9 +220,12 @@ static u8 __intel_cx0_read(struct drm_i915_private *i915, enum port port,
/* 3 tries is assumed to be enough to read successfully */
for (i = 0; i < 3; i++) {
status = __intel_cx0_read_once(i915, port, lane, addr);
+ intel_cx0_bus_reset(i915, port, lane);
if (status >= 0)
return status;
+
+ usleep_range(200, 300);
}
drm_err_once(&i915->drm, "PHY %c Read %04x failed after %d retries.\n",
@@ -299,9 +302,12 @@ static void __intel_cx0_write(struct drm_i915_private *i915, enum port port,
/* 3 tries is assumed to be enough to write successfully */
for (i = 0; i < 3; i++) {
status = __intel_cx0_write_once(i915, port, lane, addr, data, committed);
+ intel_cx0_bus_reset(i915, port, lane);
if (status == 0)
return;
+
+ usleep_range(200, 300);
}
drm_err_once(&i915->drm,
--
2.34.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/display: Reset message bus after each read/write operation
2023-10-04 10:25 [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation Mika Kahola
@ 2023-10-04 12:47 ` Patchwork
2023-10-04 12:55 ` [Intel-gfx] [PATCH] " Rodrigo Vivi
2023-10-04 17:46 ` [Intel-gfx] ✗ Fi.CI.IGT: failure for " Patchwork
2 siblings, 0 replies; 12+ messages in thread
From: Patchwork @ 2023-10-04 12:47 UTC (permalink / raw)
To: Mika Kahola; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 16633 bytes --]
== Series Details ==
Series: drm/i915/display: Reset message bus after each read/write operation
URL : https://patchwork.freedesktop.org/series/124602/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_13709 -> Patchwork_124602v1
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/index.html
Participating hosts (26 -> 39)
------------------------------
Additional (13): bat-kbl-2 bat-adlp-11 bat-adlp-9 fi-bsw-n3050 fi-apl-guc fi-kbl-guc fi-glk-j4005 fi-elk-e7500 fi-pnv-d510 bat-rpls-1 fi-blb-e6850 fi-bsw-nick fi-skl-6600u
Known issues
------------
Here are the changes found in Patchwork_124602v1 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@debugfs_test@basic-hwmon:
- bat-adlp-9: NOTRUN -> [SKIP][1] ([i915#9318])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-adlp-9/igt@debugfs_test@basic-hwmon.html
- bat-adlp-11: NOTRUN -> [SKIP][2] ([i915#9318])
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-adlp-11/igt@debugfs_test@basic-hwmon.html
- bat-rpls-1: NOTRUN -> [SKIP][3] ([i915#9318])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-rpls-1/igt@debugfs_test@basic-hwmon.html
* igt@fbdev@info:
- fi-kbl-guc: NOTRUN -> [SKIP][4] ([fdo#109271] / [i915#1849])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-kbl-guc/igt@fbdev@info.html
- bat-rpls-1: NOTRUN -> [SKIP][5] ([i915#1849] / [i915#2582])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-rpls-1/igt@fbdev@info.html
- bat-kbl-2: NOTRUN -> [SKIP][6] ([fdo#109271] / [i915#1849])
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-kbl-2/igt@fbdev@info.html
* igt@fbdev@write:
- bat-rpls-1: NOTRUN -> [SKIP][7] ([i915#2582]) +3 other tests skip
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-rpls-1/igt@fbdev@write.html
* igt@gem_exec_suspend@basic-s0@smem:
- bat-dg2-9: [PASS][8] -> [INCOMPLETE][9] ([i915#9275])
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/bat-dg2-9/igt@gem_exec_suspend@basic-s0@smem.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-dg2-9/igt@gem_exec_suspend@basic-s0@smem.html
* igt@gem_huc_copy@huc-copy:
- fi-skl-6600u: NOTRUN -> [SKIP][10] ([fdo#109271] / [i915#2190])
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-skl-6600u/igt@gem_huc_copy@huc-copy.html
- fi-glk-j4005: NOTRUN -> [SKIP][11] ([fdo#109271] / [i915#2190])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-glk-j4005/igt@gem_huc_copy@huc-copy.html
* igt@gem_lmem_swapping@basic:
- fi-apl-guc: NOTRUN -> [SKIP][12] ([fdo#109271] / [i915#4613]) +3 other tests skip
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-apl-guc/igt@gem_lmem_swapping@basic.html
- fi-glk-j4005: NOTRUN -> [SKIP][13] ([fdo#109271] / [i915#4613]) +3 other tests skip
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-glk-j4005/igt@gem_lmem_swapping@basic.html
- bat-adlp-9: NOTRUN -> [SKIP][14] ([i915#4613]) +3 other tests skip
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-adlp-9/igt@gem_lmem_swapping@basic.html
* igt@gem_lmem_swapping@parallel-random-engines:
- fi-bsw-nick: NOTRUN -> [SKIP][15] ([fdo#109271]) +18 other tests skip
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-bsw-nick/igt@gem_lmem_swapping@parallel-random-engines.html
- bat-kbl-2: NOTRUN -> [SKIP][16] ([fdo#109271]) +39 other tests skip
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-kbl-2/igt@gem_lmem_swapping@parallel-random-engines.html
* igt@gem_lmem_swapping@random-engines:
- fi-bsw-n3050: NOTRUN -> [SKIP][17] ([fdo#109271]) +18 other tests skip
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-bsw-n3050/igt@gem_lmem_swapping@random-engines.html
- bat-rpls-1: NOTRUN -> [SKIP][18] ([i915#4613]) +3 other tests skip
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-rpls-1/igt@gem_lmem_swapping@random-engines.html
- fi-skl-6600u: NOTRUN -> [SKIP][19] ([fdo#109271] / [i915#4613]) +3 other tests skip
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-skl-6600u/igt@gem_lmem_swapping@random-engines.html
* igt@gem_lmem_swapping@verify-random:
- fi-kbl-guc: NOTRUN -> [SKIP][20] ([fdo#109271] / [i915#4613]) +3 other tests skip
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-kbl-guc/igt@gem_lmem_swapping@verify-random.html
* igt@gem_tiled_pread_basic:
- bat-adlp-9: NOTRUN -> [SKIP][21] ([i915#3282])
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-adlp-9/igt@gem_tiled_pread_basic.html
- bat-adlp-11: NOTRUN -> [SKIP][22] ([i915#3282])
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-adlp-11/igt@gem_tiled_pread_basic.html
- bat-rpls-1: NOTRUN -> [SKIP][23] ([i915#3282])
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-rpls-1/igt@gem_tiled_pread_basic.html
* igt@i915_pm_rpm@basic-pci-d3-state:
- fi-elk-e7500: NOTRUN -> [SKIP][24] ([fdo#109271]) +23 other tests skip
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-elk-e7500/igt@i915_pm_rpm@basic-pci-d3-state.html
* igt@i915_pm_rpm@module-reload:
- fi-blb-e6850: NOTRUN -> [SKIP][25] ([fdo#109271]) +31 other tests skip
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-blb-e6850/igt@i915_pm_rpm@module-reload.html
* igt@i915_pm_rps@basic-api:
- bat-adlp-9: NOTRUN -> [SKIP][26] ([i915#6621])
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-adlp-9/igt@i915_pm_rps@basic-api.html
- bat-rpls-1: NOTRUN -> [SKIP][27] ([i915#6621])
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-rpls-1/igt@i915_pm_rps@basic-api.html
* igt@i915_suspend@basic-s2idle-without-i915:
- bat-rpls-1: NOTRUN -> [ABORT][28] ([i915#8668])
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-rpls-1/igt@i915_suspend@basic-s2idle-without-i915.html
* igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic:
- fi-glk-j4005: NOTRUN -> [SKIP][29] ([fdo#109271]) +9 other tests skip
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-glk-j4005/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic.html
- bat-adlp-9: NOTRUN -> [SKIP][30] ([i915#4103] / [i915#5608]) +1 other test skip
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-adlp-9/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic.html
* igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
- bat-adlp-11: NOTRUN -> [SKIP][31] ([i915#4103] / [i915#5608]) +1 other test skip
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-adlp-11/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
* igt@kms_cursor_legacy@basic-flip-after-cursor-legacy:
- bat-rpls-1: NOTRUN -> [SKIP][32] ([i915#1845]) +16 other tests skip
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-rpls-1/igt@kms_cursor_legacy@basic-flip-after-cursor-legacy.html
* igt@kms_cursor_legacy@basic-flip-before-cursor-varying-size:
- fi-kbl-guc: NOTRUN -> [SKIP][33] ([fdo#109271] / [i915#1845]) +8 other tests skip
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-kbl-guc/igt@kms_cursor_legacy@basic-flip-before-cursor-varying-size.html
* igt@kms_dsc@dsc-basic:
- bat-adlp-11: NOTRUN -> [SKIP][34] ([i915#3555] / [i915#3840])
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-adlp-11/igt@kms_dsc@dsc-basic.html
- bat-adlp-9: NOTRUN -> [SKIP][35] ([i915#3555] / [i915#3840])
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-adlp-9/igt@kms_dsc@dsc-basic.html
* igt@kms_flip@basic-flip-vs-modeset:
- bat-rpls-1: NOTRUN -> [SKIP][36] ([i915#3637]) +3 other tests skip
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-rpls-1/igt@kms_flip@basic-flip-vs-modeset.html
* igt@kms_force_connector_basic@force-load-detect:
- bat-rpls-1: NOTRUN -> [SKIP][37] ([fdo#109285])
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-rpls-1/igt@kms_force_connector_basic@force-load-detect.html
- bat-adlp-9: NOTRUN -> [SKIP][38] ([fdo#109285])
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-adlp-9/igt@kms_force_connector_basic@force-load-detect.html
* igt@kms_force_connector_basic@prune-stale-modes:
- bat-adlp-11: NOTRUN -> [SKIP][39] ([i915#4093]) +3 other tests skip
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-adlp-11/igt@kms_force_connector_basic@prune-stale-modes.html
* igt@kms_frontbuffer_tracking@basic:
- bat-rpls-1: NOTRUN -> [SKIP][40] ([i915#1849])
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-rpls-1/igt@kms_frontbuffer_tracking@basic.html
- fi-bsw-nick: NOTRUN -> [FAIL][41] ([i915#9276])
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-bsw-nick/igt@kms_frontbuffer_tracking@basic.html
* igt@kms_hdmi_inject@inject-audio:
- fi-apl-guc: NOTRUN -> [SKIP][42] ([fdo#109271]) +16 other tests skip
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-apl-guc/igt@kms_hdmi_inject@inject-audio.html
- fi-bsw-nick: NOTRUN -> [FAIL][43] ([IGT#3])
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-bsw-nick/igt@kms_hdmi_inject@inject-audio.html
- bat-adlp-11: NOTRUN -> [SKIP][44] ([i915#4369])
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-adlp-11/igt@kms_hdmi_inject@inject-audio.html
- fi-kbl-guc: NOTRUN -> [FAIL][45] ([IGT#3])
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-kbl-guc/igt@kms_hdmi_inject@inject-audio.html
- fi-bsw-n3050: NOTRUN -> [FAIL][46] ([IGT#3])
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-bsw-n3050/igt@kms_hdmi_inject@inject-audio.html
* igt@kms_pipe_crc_basic@compare-crc-sanitycheck-nv12@pipe-c-edp-1:
- fi-skl-6600u: NOTRUN -> [SKIP][47] ([fdo#109271]) +8 other tests skip
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-skl-6600u/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-nv12@pipe-c-edp-1.html
* igt@kms_pipe_crc_basic@read-crc-frame-sequence@pipe-c-dp-5:
- bat-adlp-11: NOTRUN -> [ABORT][48] ([i915#8668] / [i915#9451])
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-adlp-11/igt@kms_pipe_crc_basic@read-crc-frame-sequence@pipe-c-dp-5.html
* igt@kms_pipe_crc_basic@suspend-read-crc:
- fi-kbl-guc: NOTRUN -> [SKIP][49] ([fdo#109271]) +25 other tests skip
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-kbl-guc/igt@kms_pipe_crc_basic@suspend-read-crc.html
* igt@kms_psr@cursor_plane_move:
- bat-rpls-1: NOTRUN -> [SKIP][50] ([i915#1072]) +3 other tests skip
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-rpls-1/igt@kms_psr@cursor_plane_move.html
* igt@kms_psr@primary_page_flip:
- fi-pnv-d510: NOTRUN -> [SKIP][51] ([fdo#109271]) +31 other tests skip
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/fi-pnv-d510/igt@kms_psr@primary_page_flip.html
* igt@kms_psr@sprite_plane_onoff:
- bat-adlp-9: NOTRUN -> [SKIP][52] ([i915#1072]) +3 other tests skip
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-adlp-9/igt@kms_psr@sprite_plane_onoff.html
* igt@kms_setmode@basic-clone-single-crtc:
- bat-adlp-9: NOTRUN -> [SKIP][53] ([i915#3555])
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-adlp-9/igt@kms_setmode@basic-clone-single-crtc.html
- bat-rpls-1: NOTRUN -> [SKIP][54] ([i915#3555])
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-rpls-1/igt@kms_setmode@basic-clone-single-crtc.html
* igt@prime_vgem@basic-fence-flip:
- bat-rpls-1: NOTRUN -> [SKIP][55] ([fdo#109295] / [i915#1845] / [i915#3708])
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-rpls-1/igt@prime_vgem@basic-fence-flip.html
* igt@prime_vgem@basic-fence-read:
- bat-adlp-9: NOTRUN -> [SKIP][56] ([fdo#109295] / [i915#3291] / [i915#3708]) +2 other tests skip
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-adlp-9/igt@prime_vgem@basic-fence-read.html
* igt@prime_vgem@basic-write:
- bat-rpls-1: NOTRUN -> [SKIP][57] ([fdo#109295] / [i915#3708]) +2 other tests skip
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-rpls-1/igt@prime_vgem@basic-write.html
#### Possible fixes ####
* igt@gem_exec_suspend@basic-s3@smem:
- bat-mtlp-8: [ABORT][58] ([i915#8213] / [i915#9262]) -> [PASS][59]
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/bat-mtlp-8/igt@gem_exec_suspend@basic-s3@smem.html
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/bat-mtlp-8/igt@gem_exec_suspend@basic-s3@smem.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[IGT#3]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/issues/3
[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
[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#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#3291]: https://gitlab.freedesktop.org/drm/intel/issues/3291
[i915#3546]: https://gitlab.freedesktop.org/drm/intel/issues/3546
[i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
[i915#3637]: https://gitlab.freedesktop.org/drm/intel/issues/3637
[i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
[i915#3840]: https://gitlab.freedesktop.org/drm/intel/issues/3840
[i915#4093]: https://gitlab.freedesktop.org/drm/intel/issues/4093
[i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103
[i915#4369]: https://gitlab.freedesktop.org/drm/intel/issues/4369
[i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
[i915#5354]: https://gitlab.freedesktop.org/drm/intel/issues/5354
[i915#5608]: https://gitlab.freedesktop.org/drm/intel/issues/5608
[i915#6621]: https://gitlab.freedesktop.org/drm/intel/issues/6621
[i915#8213]: https://gitlab.freedesktop.org/drm/intel/issues/8213
[i915#8668]: https://gitlab.freedesktop.org/drm/intel/issues/8668
[i915#9262]: https://gitlab.freedesktop.org/drm/intel/issues/9262
[i915#9275]: https://gitlab.freedesktop.org/drm/intel/issues/9275
[i915#9276]: https://gitlab.freedesktop.org/drm/intel/issues/9276
[i915#9318]: https://gitlab.freedesktop.org/drm/intel/issues/9318
[i915#9451]: https://gitlab.freedesktop.org/drm/intel/issues/9451
Build changes
-------------
* Linux: CI_DRM_13709 -> Patchwork_124602v1
CI-20190529: 20190529
CI_DRM_13709: 7c6b3bb3b7a2602a399f5f8e6391ab865cad4197 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_7512: 2eb58faf82d3cd5e2e74154a7319cff56eb40762 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_124602v1: 7c6b3bb3b7a2602a399f5f8e6391ab865cad4197 @ git://anongit.freedesktop.org/gfx-ci/linux
### Linux commits
84c23c5da0a8 drm/i915/display: Reset message bus after each read/write operation
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/index.html
[-- Attachment #2: Type: text/html, Size: 21700 bytes --]
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation
2023-10-04 10:25 [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation Mika Kahola
2023-10-04 12:47 ` [Intel-gfx] ✓ Fi.CI.BAT: success for " Patchwork
@ 2023-10-04 12:55 ` Rodrigo Vivi
2023-10-05 7:05 ` Kahola, Mika
2023-10-04 17:46 ` [Intel-gfx] ✗ Fi.CI.IGT: failure for " Patchwork
2 siblings, 1 reply; 12+ messages in thread
From: Rodrigo Vivi @ 2023-10-04 12:55 UTC (permalink / raw)
To: Mika Kahola; +Cc: intel-gfx
On Wed, Oct 04, 2023 at 01:25:04PM +0300, Mika Kahola wrote:
> Every know and then we receive the following error when running
> for example IGT test kms_flip.
>
> [drm] *ERROR* PHY G Read 0d80 failed after 3 retries.
> [drm] *ERROR* PHY G Write 0d81 failed after 3 retries.
>
> Since the error is sporadic in nature, the patch proposes
> to reset the message bus after every successful or unsuccessful
> read or write operation. However, testing revealed that this
> alone is not sufficient method an additiona delay is also
> introduces anything from 200us to 300us. This delay is experimental
> value and has no specification to back it up.
have you tried the delays without the bus_reset?
have you talked to hw architects about this?
I wonder if we should add the delay inside the bus_reset itself?
although the bit 15 clear check should be enough by itself and
it doesn't look like it is a hw/fw reset involved to justify the
extra delay.
well, at least some /* FIXME: */ or /* XXX: */ comments is
desired along with the messages if we are going with this
hack without understanding why...
>
> Signed-off-by: Mika Kahola <mika.kahola@intel.com>
> ---
> drivers/gpu/drm/i915/display/intel_cx0_phy.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_cx0_phy.c b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> index abd607b564f1..a71b8a29d6b0 100644
> --- a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> +++ b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> @@ -220,9 +220,12 @@ static u8 __intel_cx0_read(struct drm_i915_private *i915, enum port port,
> /* 3 tries is assumed to be enough to read successfully */
> for (i = 0; i < 3; i++) {
> status = __intel_cx0_read_once(i915, port, lane, addr);
> + intel_cx0_bus_reset(i915, port, lane);
>
> if (status >= 0)
> return status;
> +
> + usleep_range(200, 300);
> }
>
> drm_err_once(&i915->drm, "PHY %c Read %04x failed after %d retries.\n",
> @@ -299,9 +302,12 @@ static void __intel_cx0_write(struct drm_i915_private *i915, enum port port,
> /* 3 tries is assumed to be enough to write successfully */
> for (i = 0; i < 3; i++) {
> status = __intel_cx0_write_once(i915, port, lane, addr, data, committed);
> + intel_cx0_bus_reset(i915, port, lane);
>
> if (status == 0)
> return;
> +
> + usleep_range(200, 300);
> }
>
> drm_err_once(&i915->drm,
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/display: Reset message bus after each read/write operation
2023-10-04 10:25 [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation Mika Kahola
2023-10-04 12:47 ` [Intel-gfx] ✓ Fi.CI.BAT: success for " Patchwork
2023-10-04 12:55 ` [Intel-gfx] [PATCH] " Rodrigo Vivi
@ 2023-10-04 17:46 ` Patchwork
2 siblings, 0 replies; 12+ messages in thread
From: Patchwork @ 2023-10-04 17:46 UTC (permalink / raw)
To: Mika Kahola; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 71276 bytes --]
== Series Details ==
Series: drm/i915/display: Reset message bus after each read/write operation
URL : https://patchwork.freedesktop.org/series/124602/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_13709_full -> Patchwork_124602v1_full
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_124602v1_full absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_124602v1_full, please notify your bug team (lgci.bug.filing@intel.com) to allow them
to document this new failure mode, which will reduce false positives in CI.
Participating hosts (9 -> 10)
------------------------------
Additional (1): shard-rkl0
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_124602v1_full:
### IGT changes ###
#### Possible regressions ####
* igt@gem_eio@in-flight-contexts-1us:
- shard-snb: [PASS][1] -> [FAIL][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-snb4/igt@gem_eio@in-flight-contexts-1us.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-snb4/igt@gem_eio@in-flight-contexts-1us.html
* igt@gem_exec_suspend@basic-s0@smem:
- shard-dg2: [PASS][3] -> [INCOMPLETE][4]
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-dg2-11/igt@gem_exec_suspend@basic-s0@smem.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-2/igt@gem_exec_suspend@basic-s0@smem.html
Known issues
------------
Here are the changes found in Patchwork_124602v1_full that come from known issues:
### CI changes ###
#### Issues hit ####
* boot:
- shard-glk: ([PASS][5], [PASS][6], [PASS][7], [PASS][8], [PASS][9], [PASS][10], [PASS][11], [PASS][12], [PASS][13], [PASS][14], [PASS][15], [PASS][16], [PASS][17], [PASS][18], [PASS][19], [PASS][20], [PASS][21], [PASS][22], [PASS][23], [PASS][24], [PASS][25], [PASS][26], [PASS][27], [PASS][28], [PASS][29]) -> ([PASS][30], [FAIL][31], [PASS][32], [PASS][33], [PASS][34], [PASS][35], [PASS][36], [PASS][37], [PASS][38], [PASS][39], [PASS][40], [PASS][41], [PASS][42], [PASS][43], [PASS][44], [PASS][45], [PASS][46], [PASS][47], [PASS][48], [PASS][49], [PASS][50], [PASS][51], [PASS][52], [PASS][53], [PASS][54]) ([i915#8293])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk2/boot.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk3/boot.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk3/boot.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk3/boot.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk2/boot.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk1/boot.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk1/boot.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk1/boot.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk1/boot.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk1/boot.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk2/boot.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk2/boot.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk2/boot.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk5/boot.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk5/boot.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk5/boot.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk5/boot.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk5/boot.html
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk4/boot.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk4/boot.html
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk4/boot.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk4/boot.html
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk4/boot.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk3/boot.html
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk3/boot.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk1/boot.html
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk5/boot.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk5/boot.html
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk5/boot.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk5/boot.html
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk4/boot.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk1/boot.html
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk4/boot.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk4/boot.html
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk4/boot.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk4/boot.html
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk3/boot.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk3/boot.html
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk3/boot.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk3/boot.html
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk3/boot.html
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk2/boot.html
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk2/boot.html
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk2/boot.html
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk2/boot.html
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk2/boot.html
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk1/boot.html
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk1/boot.html
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk1/boot.html
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk1/boot.html
### IGT changes ###
#### Issues hit ####
* igt@api_intel_bb@blit-reloc-keep-cache:
- shard-dg2: NOTRUN -> [SKIP][55] ([i915#8411])
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@api_intel_bb@blit-reloc-keep-cache.html
* igt@api_intel_bb@render-ccs:
- shard-dg2: NOTRUN -> [FAIL][56] ([i915#6122])
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@api_intel_bb@render-ccs.html
* igt@debugfs_test@basic-hwmon:
- shard-mtlp: NOTRUN -> [SKIP][57] ([i915#9318])
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@debugfs_test@basic-hwmon.html
- shard-rkl: NOTRUN -> [SKIP][58] ([i915#9318])
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@debugfs_test@basic-hwmon.html
* igt@drm_fdinfo@busy-hang@bcs0:
- shard-dg2: NOTRUN -> [SKIP][59] ([i915#8414]) +10 other tests skip
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-7/igt@drm_fdinfo@busy-hang@bcs0.html
* igt@drm_fdinfo@most-busy-check-all@rcs0:
- shard-rkl: NOTRUN -> [FAIL][60] ([i915#7742])
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@drm_fdinfo@most-busy-check-all@rcs0.html
* igt@drm_fdinfo@most-busy-check-all@vcs0:
- shard-mtlp: NOTRUN -> [SKIP][61] ([i915#8414]) +6 other tests skip
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@drm_fdinfo@most-busy-check-all@vcs0.html
* igt@gem_busy@semaphore:
- shard-dg2: NOTRUN -> [SKIP][62] ([i915#3936])
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@gem_busy@semaphore.html
* igt@gem_create@create-ext-cpu-access-big:
- shard-mtlp: NOTRUN -> [SKIP][63] ([i915#6335])
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@gem_create@create-ext-cpu-access-big.html
- shard-rkl: NOTRUN -> [SKIP][64] ([i915#6335])
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@gem_create@create-ext-cpu-access-big.html
* igt@gem_ctx_persistence@heartbeat-stop:
- shard-dg2: NOTRUN -> [SKIP][65] ([i915#8555]) +1 other test skip
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@gem_ctx_persistence@heartbeat-stop.html
* igt@gem_ctx_persistence@saturated-hostile-nopreempt@ccs0:
- shard-dg2: NOTRUN -> [SKIP][66] ([i915#5882]) +9 other tests skip
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@gem_ctx_persistence@saturated-hostile-nopreempt@ccs0.html
* igt@gem_ctx_sseu@engines:
- shard-rkl: NOTRUN -> [SKIP][67] ([i915#280])
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@gem_ctx_sseu@engines.html
* igt@gem_ctx_sseu@invalid-args:
- shard-mtlp: NOTRUN -> [SKIP][68] ([i915#280]) +1 other test skip
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@gem_ctx_sseu@invalid-args.html
* igt@gem_eio@unwedge-stress:
- shard-mtlp: NOTRUN -> [FAIL][69] ([i915#9436])
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@gem_eio@unwedge-stress.html
* igt@gem_exec_balancer@bonded-semaphore:
- shard-mtlp: NOTRUN -> [SKIP][70] ([i915#4812])
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@gem_exec_balancer@bonded-semaphore.html
* igt@gem_exec_fair@basic-deadline:
- shard-apl: NOTRUN -> [FAIL][71] ([i915#2846])
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-apl4/igt@gem_exec_fair@basic-deadline.html
* igt@gem_exec_fence@submit3:
- shard-dg2: NOTRUN -> [SKIP][72] ([i915#4812]) +1 other test skip
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-7/igt@gem_exec_fence@submit3.html
* igt@gem_exec_flush@basic-wb-ro-before-default:
- shard-dg2: NOTRUN -> [SKIP][73] ([i915#3539] / [i915#4852]) +1 other test skip
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@gem_exec_flush@basic-wb-ro-before-default.html
* igt@gem_exec_gttfill@multigpu-basic:
- shard-dg2: NOTRUN -> [SKIP][74] ([i915#7697])
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@gem_exec_gttfill@multigpu-basic.html
* igt@gem_exec_params@secure-non-root:
- shard-dg2: NOTRUN -> [SKIP][75] ([fdo#112283])
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-11/igt@gem_exec_params@secure-non-root.html
* igt@gem_exec_reloc@basic-concurrent16:
- shard-rkl: NOTRUN -> [SKIP][76] ([i915#3281]) +2 other tests skip
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@gem_exec_reloc@basic-concurrent16.html
* igt@gem_exec_reloc@basic-cpu-noreloc:
- shard-dg2: NOTRUN -> [SKIP][77] ([i915#3281]) +5 other tests skip
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-1/igt@gem_exec_reloc@basic-cpu-noreloc.html
* igt@gem_exec_reloc@basic-wc-noreloc:
- shard-mtlp: NOTRUN -> [SKIP][78] ([i915#3281]) +2 other tests skip
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@gem_exec_reloc@basic-wc-noreloc.html
* igt@gem_exec_schedule@reorder-wide:
- shard-dg2: NOTRUN -> [SKIP][79] ([i915#4537] / [i915#4812]) +1 other test skip
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@gem_exec_schedule@reorder-wide.html
* igt@gem_exec_suspend@basic-s4-devices@lmem0:
- shard-dg2: NOTRUN -> [ABORT][80] ([i915#7975] / [i915#8213])
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-1/igt@gem_exec_suspend@basic-s4-devices@lmem0.html
* igt@gem_fence_thrash@bo-copy:
- shard-mtlp: NOTRUN -> [SKIP][81] ([i915#4860])
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@gem_fence_thrash@bo-copy.html
* igt@gem_fenced_exec_thrash@2-spare-fences:
- shard-dg2: NOTRUN -> [SKIP][82] ([i915#4860]) +1 other test skip
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-11/igt@gem_fenced_exec_thrash@2-spare-fences.html
* igt@gem_lmem_swapping@heavy-verify-multi:
- shard-rkl: NOTRUN -> [SKIP][83] ([i915#4613])
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@gem_lmem_swapping@heavy-verify-multi.html
* igt@gem_lmem_swapping@verify-ccs:
- shard-mtlp: NOTRUN -> [SKIP][84] ([i915#4613])
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-8/igt@gem_lmem_swapping@verify-ccs.html
* igt@gem_mmap@short-mmap:
- shard-dg2: NOTRUN -> [SKIP][85] ([i915#4083]) +5 other tests skip
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-7/igt@gem_mmap@short-mmap.html
* igt@gem_mmap_wc@invalid-flags:
- shard-mtlp: NOTRUN -> [SKIP][86] ([i915#4083])
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@gem_mmap_wc@invalid-flags.html
* igt@gem_pxp@display-protected-crc:
- shard-dg2: NOTRUN -> [SKIP][87] ([i915#4270]) +1 other test skip
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@gem_pxp@display-protected-crc.html
* igt@gem_readwrite@beyond-eob:
- shard-dg2: NOTRUN -> [SKIP][88] ([i915#3282]) +5 other tests skip
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@gem_readwrite@beyond-eob.html
* igt@gem_readwrite@read-write:
- shard-mtlp: NOTRUN -> [SKIP][89] ([i915#3282]) +1 other test skip
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@gem_readwrite@read-write.html
* igt@gem_render_copy@yf-tiled-ccs-to-yf-tiled-ccs:
- shard-mtlp: NOTRUN -> [SKIP][90] ([i915#8428]) +2 other tests skip
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@gem_render_copy@yf-tiled-ccs-to-yf-tiled-ccs.html
* igt@gem_softpin@evict-snoop:
- shard-dg2: NOTRUN -> [SKIP][91] ([i915#4885])
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-7/igt@gem_softpin@evict-snoop.html
* igt@gem_userptr_blits@coherency-unsync:
- shard-dg2: NOTRUN -> [SKIP][92] ([i915#3297])
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@gem_userptr_blits@coherency-unsync.html
* igt@gem_userptr_blits@forbidden-operations:
- shard-rkl: NOTRUN -> [SKIP][93] ([i915#3282])
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@gem_userptr_blits@forbidden-operations.html
* igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy:
- shard-mtlp: NOTRUN -> [SKIP][94] ([i915#3297])
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy.html
* igt@gen9_exec_parse@bb-oversize:
- shard-rkl: NOTRUN -> [SKIP][95] ([i915#2527]) +1 other test skip
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@gen9_exec_parse@bb-oversize.html
* igt@gen9_exec_parse@bb-start-far:
- shard-dg2: NOTRUN -> [SKIP][96] ([i915#2856]) +4 other tests skip
[96]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@gen9_exec_parse@bb-start-far.html
* igt@gen9_exec_parse@bb-start-param:
- shard-mtlp: NOTRUN -> [SKIP][97] ([i915#2856])
[97]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@gen9_exec_parse@bb-start-param.html
* igt@i915_hangman@engine-error-state-capture@vcs1:
- shard-mtlp: [PASS][98] -> [ABORT][99] ([i915#9414])
[98]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-mtlp-6/igt@i915_hangman@engine-error-state-capture@vcs1.html
[99]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-7/igt@i915_hangman@engine-error-state-capture@vcs1.html
* igt@i915_pm_freq_mult@media-freq@gt0:
- shard-rkl: NOTRUN -> [SKIP][100] ([i915#6590])
[100]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@i915_pm_freq_mult@media-freq@gt0.html
* igt@i915_pm_freq_mult@media-freq@gt1:
- shard-mtlp: NOTRUN -> [SKIP][101] ([i915#6590]) +1 other test skip
[101]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@i915_pm_freq_mult@media-freq@gt1.html
* igt@i915_pm_rc6_residency@rc6-idle@vcs0:
- shard-dg1: [PASS][102] -> [FAIL][103] ([i915#3591])
[102]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-dg1-16/igt@i915_pm_rc6_residency@rc6-idle@vcs0.html
[103]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg1-18/igt@i915_pm_rc6_residency@rc6-idle@vcs0.html
* igt@i915_pm_rpm@dpms-lpsp:
- shard-rkl: [PASS][104] -> [SKIP][105] ([i915#1397])
[104]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-rkl-7/igt@i915_pm_rpm@dpms-lpsp.html
[105]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@i915_pm_rpm@dpms-lpsp.html
* igt@i915_pm_rpm@gem-evict-pwrite:
- shard-dg2: NOTRUN -> [SKIP][106] ([i915#4077]) +8 other tests skip
[106]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-7/igt@i915_pm_rpm@gem-evict-pwrite.html
* igt@i915_pm_rpm@modeset-non-lpsp:
- shard-mtlp: NOTRUN -> [SKIP][107] ([i915#1397])
[107]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@i915_pm_rpm@modeset-non-lpsp.html
* igt@i915_pm_rpm@modeset-non-lpsp-stress-no-wait:
- shard-dg1: [PASS][108] -> [SKIP][109] ([i915#1397]) +1 other test skip
[108]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-dg1-14/igt@i915_pm_rpm@modeset-non-lpsp-stress-no-wait.html
[109]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg1-19/igt@i915_pm_rpm@modeset-non-lpsp-stress-no-wait.html
* igt@i915_pm_rps@reset:
- shard-mtlp: NOTRUN -> [FAIL][110] ([i915#8346])
[110]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@i915_pm_rps@reset.html
* igt@i915_selftest@mock@memory_region:
- shard-dg2: NOTRUN -> [DMESG-WARN][111] ([i915#9311])
[111]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-11/igt@i915_selftest@mock@memory_region.html
* igt@i915_suspend@basic-s3-without-i915:
- shard-rkl: [PASS][112] -> [FAIL][113] ([fdo#103375])
[112]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-rkl-1/igt@i915_suspend@basic-s3-without-i915.html
[113]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-4/igt@i915_suspend@basic-s3-without-i915.html
* igt@kms_addfb_basic@addfb25-y-tiled-small-legacy:
- shard-mtlp: NOTRUN -> [SKIP][114] ([i915#5190])
[114]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@kms_addfb_basic@addfb25-y-tiled-small-legacy.html
* igt@kms_addfb_basic@clobberred-modifier:
- shard-mtlp: NOTRUN -> [SKIP][115] ([i915#4212])
[115]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@kms_addfb_basic@clobberred-modifier.html
* igt@kms_async_flips@alternate-sync-async-flip@pipe-c-hdmi-a-2:
- shard-glk: [PASS][116] -> [FAIL][117] ([i915#2521])
[116]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk5/igt@kms_async_flips@alternate-sync-async-flip@pipe-c-hdmi-a-2.html
[117]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk1/igt@kms_async_flips@alternate-sync-async-flip@pipe-c-hdmi-a-2.html
* igt@kms_async_flips@async-flip-with-page-flip-events@pipe-a-edp-1-4-rc_ccs:
- shard-mtlp: NOTRUN -> [SKIP][118] ([i915#8502]) +11 other tests skip
[118]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@kms_async_flips@async-flip-with-page-flip-events@pipe-a-edp-1-4-rc_ccs.html
* igt@kms_async_flips@async-flip-with-page-flip-events@pipe-a-hdmi-a-2-y-rc_ccs:
- shard-rkl: NOTRUN -> [SKIP][119] ([i915#8502]) +3 other tests skip
[119]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_async_flips@async-flip-with-page-flip-events@pipe-a-hdmi-a-2-y-rc_ccs.html
* igt@kms_async_flips@async-flip-with-page-flip-events@pipe-a-hdmi-a-3-y-rc_ccs:
- shard-dg1: NOTRUN -> [SKIP][120] ([i915#8502]) +7 other tests skip
[120]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg1-12/igt@kms_async_flips@async-flip-with-page-flip-events@pipe-a-hdmi-a-3-y-rc_ccs.html
* igt@kms_async_flips@async-flip-with-page-flip-events@pipe-b-hdmi-a-2-4-rc_ccs-cc:
- shard-dg2: NOTRUN -> [SKIP][121] ([i915#8709]) +11 other tests skip
[121]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-2/igt@kms_async_flips@async-flip-with-page-flip-events@pipe-b-hdmi-a-2-4-rc_ccs-cc.html
* igt@kms_async_flips@crc@pipe-d-edp-1:
- shard-mtlp: NOTRUN -> [FAIL][122] ([i915#8247]) +3 other tests fail
[122]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@kms_async_flips@crc@pipe-d-edp-1.html
* igt@kms_atomic_transition@plane-all-modeset-transition-internal-panels:
- shard-rkl: NOTRUN -> [SKIP][123] ([i915#1769] / [i915#3555])
[123]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_atomic_transition@plane-all-modeset-transition-internal-panels.html
* igt@kms_big_fb@4-tiled-16bpp-rotate-270:
- shard-dg2: NOTRUN -> [SKIP][124] ([fdo#111614]) +1 other test skip
[124]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-7/igt@kms_big_fb@4-tiled-16bpp-rotate-270.html
* igt@kms_big_fb@4-tiled-64bpp-rotate-180:
- shard-mtlp: [PASS][125] -> [FAIL][126] ([i915#5138])
[125]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-mtlp-2/igt@kms_big_fb@4-tiled-64bpp-rotate-180.html
[126]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@kms_big_fb@4-tiled-64bpp-rotate-180.html
* igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-0-async-flip:
- shard-rkl: NOTRUN -> [SKIP][127] ([i915#5286])
[127]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-0-async-flip.html
* igt@kms_big_fb@x-tiled-16bpp-rotate-270:
- shard-mtlp: NOTRUN -> [SKIP][128] ([fdo#111614]) +1 other test skip
[128]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-8/igt@kms_big_fb@x-tiled-16bpp-rotate-270.html
* igt@kms_big_fb@y-tiled-addfb-size-offset-overflow:
- shard-dg2: NOTRUN -> [SKIP][129] ([i915#5190]) +12 other tests skip
[129]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-7/igt@kms_big_fb@y-tiled-addfb-size-offset-overflow.html
* igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-180-async-flip:
- shard-tglu: [PASS][130] -> [FAIL][131] ([i915#3743])
[130]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-tglu-5/igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-180-async-flip.html
[131]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-tglu-3/igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-180-async-flip.html
* igt@kms_big_fb@yf-tiled-32bpp-rotate-180:
- shard-rkl: NOTRUN -> [SKIP][132] ([fdo#110723]) +1 other test skip
[132]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_big_fb@yf-tiled-32bpp-rotate-180.html
* igt@kms_big_fb@yf-tiled-max-hw-stride-64bpp-rotate-0:
- shard-dg2: NOTRUN -> [SKIP][133] ([i915#4538] / [i915#5190]) +3 other tests skip
[133]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-7/igt@kms_big_fb@yf-tiled-max-hw-stride-64bpp-rotate-0.html
* igt@kms_big_fb@yf-tiled-max-hw-stride-64bpp-rotate-180-hflip-async-flip:
- shard-mtlp: NOTRUN -> [SKIP][134] ([fdo#111615]) +3 other tests skip
[134]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@kms_big_fb@yf-tiled-max-hw-stride-64bpp-rotate-180-hflip-async-flip.html
* igt@kms_big_joiner@basic:
- shard-mtlp: NOTRUN -> [SKIP][135] ([i915#2705])
[135]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@kms_big_joiner@basic.html
* igt@kms_ccs@pipe-a-bad-rotation-90-y_tiled_ccs:
- shard-rkl: NOTRUN -> [SKIP][136] ([i915#3734] / [i915#5354] / [i915#6095]) +1 other test skip
[136]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_ccs@pipe-a-bad-rotation-90-y_tiled_ccs.html
* igt@kms_ccs@pipe-b-bad-pixel-format-4_tiled_mtl_rc_ccs_cc:
- shard-rkl: NOTRUN -> [SKIP][137] ([i915#5354] / [i915#6095])
[137]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_ccs@pipe-b-bad-pixel-format-4_tiled_mtl_rc_ccs_cc.html
* igt@kms_ccs@pipe-b-bad-pixel-format-y_tiled_gen12_mc_ccs:
- shard-mtlp: NOTRUN -> [SKIP][138] ([i915#3886] / [i915#5354] / [i915#6095]) +3 other tests skip
[138]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@kms_ccs@pipe-b-bad-pixel-format-y_tiled_gen12_mc_ccs.html
- shard-rkl: NOTRUN -> [SKIP][139] ([i915#3886] / [i915#5354] / [i915#6095])
[139]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_ccs@pipe-b-bad-pixel-format-y_tiled_gen12_mc_ccs.html
* igt@kms_ccs@pipe-b-crc-primary-rotation-180-y_tiled_gen12_mc_ccs:
- shard-apl: NOTRUN -> [SKIP][140] ([fdo#109271] / [i915#3886]) +1 other test skip
[140]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-apl4/igt@kms_ccs@pipe-b-crc-primary-rotation-180-y_tiled_gen12_mc_ccs.html
* igt@kms_ccs@pipe-c-bad-aux-stride-4_tiled_mtl_mc_ccs:
- shard-rkl: NOTRUN -> [SKIP][141] ([i915#5354]) +5 other tests skip
[141]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_ccs@pipe-c-bad-aux-stride-4_tiled_mtl_mc_ccs.html
* igt@kms_ccs@pipe-c-crc-primary-rotation-180-y_tiled_gen12_mc_ccs:
- shard-dg2: NOTRUN -> [SKIP][142] ([i915#3689] / [i915#3886] / [i915#5354]) +5 other tests skip
[142]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@kms_ccs@pipe-c-crc-primary-rotation-180-y_tiled_gen12_mc_ccs.html
* igt@kms_ccs@pipe-c-crc-primary-rotation-180-y_tiled_gen12_rc_ccs:
- shard-mtlp: NOTRUN -> [SKIP][143] ([i915#5354] / [i915#6095]) +9 other tests skip
[143]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-8/igt@kms_ccs@pipe-c-crc-primary-rotation-180-y_tiled_gen12_rc_ccs.html
* igt@kms_ccs@pipe-c-crc-sprite-planes-basic-y_tiled_gen12_rc_ccs:
- shard-dg2: NOTRUN -> [SKIP][144] ([i915#3689] / [i915#5354]) +21 other tests skip
[144]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@kms_ccs@pipe-c-crc-sprite-planes-basic-y_tiled_gen12_rc_ccs.html
* igt@kms_cdclk@mode-transition@pipe-b-edp-1:
- shard-mtlp: NOTRUN -> [SKIP][145] ([i915#7213] / [i915#9010]) +3 other tests skip
[145]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@kms_cdclk@mode-transition@pipe-b-edp-1.html
* igt@kms_chamelium_color@ctm-0-75:
- shard-rkl: NOTRUN -> [SKIP][146] ([fdo#111827])
[146]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_chamelium_color@ctm-0-75.html
* igt@kms_chamelium_color@degamma:
- shard-dg2: NOTRUN -> [SKIP][147] ([fdo#111827]) +1 other test skip
[147]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-1/igt@kms_chamelium_color@degamma.html
* igt@kms_chamelium_frames@dp-crc-fast:
- shard-dg2: NOTRUN -> [SKIP][148] ([i915#7828]) +9 other tests skip
[148]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@kms_chamelium_frames@dp-crc-fast.html
* igt@kms_chamelium_hpd@common-hpd-after-suspend:
- shard-apl: NOTRUN -> [SKIP][149] ([fdo#109271]) +25 other tests skip
[149]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-apl4/igt@kms_chamelium_hpd@common-hpd-after-suspend.html
* igt@kms_chamelium_hpd@dp-hpd-storm:
- shard-mtlp: NOTRUN -> [SKIP][150] ([i915#7828]) +1 other test skip
[150]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@kms_chamelium_hpd@dp-hpd-storm.html
* igt@kms_chamelium_hpd@dp-hpd-with-enabled-mode:
- shard-rkl: NOTRUN -> [SKIP][151] ([i915#7828])
[151]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_chamelium_hpd@dp-hpd-with-enabled-mode.html
* igt@kms_content_protection@srm:
- shard-rkl: NOTRUN -> [SKIP][152] ([i915#7118])
[152]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_content_protection@srm.html
- shard-mtlp: NOTRUN -> [SKIP][153] ([i915#6944])
[153]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@kms_content_protection@srm.html
* igt@kms_cursor_crc@cursor-random-512x170:
- shard-dg2: NOTRUN -> [SKIP][154] ([i915#3359])
[154]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-11/igt@kms_cursor_crc@cursor-random-512x170.html
* igt@kms_cursor_crc@cursor-random-max-size:
- shard-mtlp: NOTRUN -> [SKIP][155] ([i915#3555] / [i915#8814]) +1 other test skip
[155]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@kms_cursor_crc@cursor-random-max-size.html
* igt@kms_cursor_legacy@2x-long-cursor-vs-flip-atomic:
- shard-mtlp: NOTRUN -> [SKIP][156] ([i915#3546]) +1 other test skip
[156]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@kms_cursor_legacy@2x-long-cursor-vs-flip-atomic.html
* igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
- shard-dg2: NOTRUN -> [SKIP][157] ([i915#4103] / [i915#4213] / [i915#5608])
[157]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-11/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
* igt@kms_cursor_legacy@cursorb-vs-flipb-atomic:
- shard-dg2: NOTRUN -> [SKIP][158] ([fdo#109274] / [i915#5354]) +2 other tests skip
[158]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-7/igt@kms_cursor_legacy@cursorb-vs-flipb-atomic.html
* igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions:
- shard-dg2: NOTRUN -> [SKIP][159] ([fdo#109274] / [fdo#111767] / [i915#5354])
[159]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions.html
* igt@kms_cursor_legacy@short-busy-flip-before-cursor-atomic-transitions-varying-size:
- shard-rkl: NOTRUN -> [SKIP][160] ([i915#4103])
[160]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_cursor_legacy@short-busy-flip-before-cursor-atomic-transitions-varying-size.html
- shard-mtlp: NOTRUN -> [SKIP][161] ([i915#4213])
[161]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@kms_cursor_legacy@short-busy-flip-before-cursor-atomic-transitions-varying-size.html
* igt@kms_cursor_legacy@short-busy-flip-before-cursor-toggle:
- shard-dg2: NOTRUN -> [SKIP][162] ([i915#4103] / [i915#4213]) +1 other test skip
[162]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@kms_cursor_legacy@short-busy-flip-before-cursor-toggle.html
* igt@kms_cursor_legacy@single-bo@all-pipes:
- shard-mtlp: [PASS][163] -> [DMESG-WARN][164] ([i915#2017])
[163]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-mtlp-5/igt@kms_cursor_legacy@single-bo@all-pipes.html
[164]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-4/igt@kms_cursor_legacy@single-bo@all-pipes.html
* igt@kms_dirtyfb@dirtyfb-ioctl@fbc-hdmi-a-1:
- shard-rkl: NOTRUN -> [SKIP][165] ([i915#9227])
[165]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-7/igt@kms_dirtyfb@dirtyfb-ioctl@fbc-hdmi-a-1.html
* igt@kms_dirtyfb@dirtyfb-ioctl@psr-hdmi-a-1:
- shard-rkl: NOTRUN -> [SKIP][166] ([i915#9226] / [i915#9261]) +1 other test skip
[166]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-7/igt@kms_dirtyfb@dirtyfb-ioctl@psr-hdmi-a-1.html
* igt@kms_fbcon_fbt@psr-suspend:
- shard-dg2: NOTRUN -> [SKIP][167] ([i915#3469])
[167]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@kms_fbcon_fbt@psr-suspend.html
* igt@kms_flip@2x-blocking-wf_vblank:
- shard-rkl: NOTRUN -> [SKIP][168] ([fdo#111825]) +2 other tests skip
[168]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_flip@2x-blocking-wf_vblank.html
* igt@kms_flip@2x-flip-vs-expired-vblank-interruptible:
- shard-snb: NOTRUN -> [SKIP][169] ([fdo#109271] / [fdo#111767]) +1 other test skip
[169]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-snb4/igt@kms_flip@2x-flip-vs-expired-vblank-interruptible.html
* igt@kms_flip@2x-plain-flip:
- shard-mtlp: NOTRUN -> [SKIP][170] ([i915#3637]) +4 other tests skip
[170]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-8/igt@kms_flip@2x-plain-flip.html
* igt@kms_flip@2x-plain-flip-fb-recreate:
- shard-dg2: NOTRUN -> [SKIP][171] ([fdo#109274]) +3 other tests skip
[171]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@kms_flip@2x-plain-flip-fb-recreate.html
* igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs-upscaling@pipe-a-default-mode:
- shard-mtlp: NOTRUN -> [SKIP][172] ([i915#2672]) +1 other test skip
[172]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs-upscaling@pipe-a-default-mode.html
* igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-32bpp-4tile-downscaling@pipe-a-default-mode:
- shard-mtlp: NOTRUN -> [SKIP][173] ([i915#8810])
[173]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-32bpp-4tile-downscaling@pipe-a-default-mode.html
* igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-32bpp-4tile-downscaling@pipe-a-valid-mode:
- shard-rkl: NOTRUN -> [SKIP][174] ([i915#2672])
[174]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-32bpp-4tile-downscaling@pipe-a-valid-mode.html
* igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytile-downscaling@pipe-a-valid-mode:
- shard-dg2: NOTRUN -> [SKIP][175] ([i915#2672]) +2 other tests skip
[175]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytile-downscaling@pipe-a-valid-mode.html
* igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-blt:
- shard-dg2: [PASS][176] -> [FAIL][177] ([i915#6880])
[176]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-dg2-3/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-blt.html
[177]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-6/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-blt.html
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-cur-indfb-draw-blt:
- shard-dg2: NOTRUN -> [SKIP][178] ([i915#5354]) +45 other tests skip
[178]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-7/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-cur-indfb-draw-blt.html
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-cur-indfb-draw-pwrite:
- shard-mtlp: NOTRUN -> [SKIP][179] ([i915#1825]) +9 other tests skip
[179]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-cur-indfb-draw-pwrite.html
* igt@kms_frontbuffer_tracking@fbc-rgb101010-draw-mmap-gtt:
- shard-dg2: NOTRUN -> [SKIP][180] ([i915#8708]) +14 other tests skip
[180]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-11/igt@kms_frontbuffer_tracking@fbc-rgb101010-draw-mmap-gtt.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-pri-indfb-multidraw:
- shard-rkl: NOTRUN -> [SKIP][181] ([i915#3023]) +4 other tests skip
[181]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_frontbuffer_tracking@fbcpsr-1p-pri-indfb-multidraw.html
* igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-cur-indfb-draw-mmap-gtt:
- shard-rkl: NOTRUN -> [SKIP][182] ([fdo#111825] / [i915#1825]) +7 other tests skip
[182]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-cur-indfb-draw-mmap-gtt.html
* igt@kms_frontbuffer_tracking@psr-indfb-scaledprimary:
- shard-dg2: NOTRUN -> [SKIP][183] ([i915#3458]) +16 other tests skip
[183]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@kms_frontbuffer_tracking@psr-indfb-scaledprimary.html
* igt@kms_frontbuffer_tracking@psr-rgb565-draw-mmap-gtt:
- shard-mtlp: NOTRUN -> [SKIP][184] ([i915#8708]) +4 other tests skip
[184]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@kms_frontbuffer_tracking@psr-rgb565-draw-mmap-gtt.html
* igt@kms_hdr@invalid-hdr:
- shard-rkl: NOTRUN -> [SKIP][185] ([i915#3555] / [i915#8228])
[185]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-7/igt@kms_hdr@invalid-hdr.html
* igt@kms_hdr@static-swap:
- shard-dg2: NOTRUN -> [SKIP][186] ([i915#3555] / [i915#8228])
[186]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-7/igt@kms_hdr@static-swap.html
* igt@kms_hdr@static-toggle:
- shard-mtlp: NOTRUN -> [SKIP][187] ([i915#3555] / [i915#8228])
[187]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-8/igt@kms_hdr@static-toggle.html
* igt@kms_pipe_b_c_ivb@from-pipe-c-to-b-with-3-lanes:
- shard-dg2: NOTRUN -> [SKIP][188] ([fdo#109289]) +2 other tests skip
[188]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-11/igt@kms_pipe_b_c_ivb@from-pipe-c-to-b-with-3-lanes.html
* igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-dp-1:
- shard-apl: [PASS][189] -> [INCOMPLETE][190] ([i915#9392])
[189]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-apl4/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-dp-1.html
[190]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-apl1/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-dp-1.html
* igt@kms_plane_scaling@intel-max-src-size:
- shard-dg2: NOTRUN -> [SKIP][191] ([i915#6953])
[191]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-7/igt@kms_plane_scaling@intel-max-src-size.html
* igt@kms_plane_scaling@intel-max-src-size@pipe-a-hdmi-a-1:
- shard-dg1: NOTRUN -> [FAIL][192] ([i915#8292])
[192]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg1-19/igt@kms_plane_scaling@intel-max-src-size@pipe-a-hdmi-a-1.html
* igt@kms_plane_scaling@plane-scaler-with-clipping-clamping-rotation@pipe-b-hdmi-a-2:
- shard-rkl: NOTRUN -> [SKIP][193] ([i915#5176]) +1 other test skip
[193]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_plane_scaling@plane-scaler-with-clipping-clamping-rotation@pipe-b-hdmi-a-2.html
* igt@kms_plane_scaling@planes-downscale-factor-0-25-upscale-factor-0-25@pipe-d-hdmi-a-3:
- shard-dg2: NOTRUN -> [SKIP][194] ([i915#5235]) +15 other tests skip
[194]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-7/igt@kms_plane_scaling@planes-downscale-factor-0-25-upscale-factor-0-25@pipe-d-hdmi-a-3.html
* igt@kms_plane_scaling@planes-downscale-factor-0-5-unity-scaling@pipe-b-vga-1:
- shard-snb: NOTRUN -> [SKIP][195] ([fdo#109271]) +23 other tests skip
[195]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-snb4/igt@kms_plane_scaling@planes-downscale-factor-0-5-unity-scaling@pipe-b-vga-1.html
* igt@kms_plane_scaling@planes-downscale-factor-0-75@pipe-d-edp-1:
- shard-mtlp: NOTRUN -> [SKIP][196] ([i915#5235]) +3 other tests skip
[196]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@kms_plane_scaling@planes-downscale-factor-0-75@pipe-d-edp-1.html
* igt@kms_plane_scaling@planes-unity-scaling-downscale-factor-0-25@pipe-c-hdmi-a-1:
- shard-dg1: NOTRUN -> [SKIP][197] ([i915#5235]) +15 other tests skip
[197]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg1-19/igt@kms_plane_scaling@planes-unity-scaling-downscale-factor-0-25@pipe-c-hdmi-a-1.html
* igt@kms_prime@basic-crc-hybrid:
- shard-dg2: NOTRUN -> [SKIP][198] ([i915#6524] / [i915#6805])
[198]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-11/igt@kms_prime@basic-crc-hybrid.html
* igt@kms_psr2_sf@cursor-plane-move-continuous-exceed-sf:
- shard-dg2: NOTRUN -> [SKIP][199] ([i915#658])
[199]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@kms_psr2_sf@cursor-plane-move-continuous-exceed-sf.html
* igt@kms_psr2_sf@cursor-plane-move-continuous-sf:
- shard-rkl: NOTRUN -> [SKIP][200] ([i915#658])
[200]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_psr2_sf@cursor-plane-move-continuous-sf.html
* igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area:
- shard-apl: NOTRUN -> [SKIP][201] ([fdo#109271] / [i915#658])
[201]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-apl4/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area.html
* igt@kms_psr@dpms:
- shard-dg2: NOTRUN -> [SKIP][202] ([i915#1072]) +8 other tests skip
[202]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@kms_psr@dpms.html
* igt@kms_psr@primary_render:
- shard-rkl: NOTRUN -> [SKIP][203] ([i915#1072]) +2 other tests skip
[203]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_psr@primary_render.html
* igt@kms_psr@psr2_sprite_mmap_gtt:
- shard-mtlp: NOTRUN -> [SKIP][204] ([i915#4077]) +5 other tests skip
[204]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@kms_psr@psr2_sprite_mmap_gtt.html
* igt@kms_rotation_crc@primary-y-tiled-reflect-x-270:
- shard-dg2: NOTRUN -> [SKIP][205] ([i915#4235] / [i915#5190])
[205]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-11/igt@kms_rotation_crc@primary-y-tiled-reflect-x-270.html
* igt@kms_rotation_crc@sprite-rotation-270:
- shard-dg2: NOTRUN -> [SKIP][206] ([i915#4235])
[206]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-11/igt@kms_rotation_crc@sprite-rotation-270.html
* igt@kms_scaling_modes@scaling-mode-none:
- shard-dg2: NOTRUN -> [SKIP][207] ([i915#3555]) +7 other tests skip
[207]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@kms_scaling_modes@scaling-mode-none.html
* igt@kms_setmode@basic@pipe-a-vga-1:
- shard-snb: NOTRUN -> [FAIL][208] ([i915#5465]) +1 other test fail
[208]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-snb6/igt@kms_setmode@basic@pipe-a-vga-1.html
* igt@kms_sysfs_edid_timing:
- shard-dg2: NOTRUN -> [FAIL][209] ([IGT#2])
[209]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-1/igt@kms_sysfs_edid_timing.html
* igt@kms_universal_plane@cursor-fb-leak-pipe-d:
- shard-mtlp: [PASS][210] -> [FAIL][211] ([i915#9196])
[210]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-mtlp-6/igt@kms_universal_plane@cursor-fb-leak-pipe-d.html
[211]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@kms_universal_plane@cursor-fb-leak-pipe-d.html
* igt@kms_universal_plane@universal-plane-pipe-b-sanity:
- shard-dg1: [PASS][212] -> [DMESG-WARN][213] ([i915#4423])
[212]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-dg1-12/igt@kms_universal_plane@universal-plane-pipe-b-sanity.html
[213]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg1-15/igt@kms_universal_plane@universal-plane-pipe-b-sanity.html
* igt@kms_vblank@pipe-c-query-busy:
- shard-rkl: NOTRUN -> [SKIP][214] ([i915#4070] / [i915#6768])
[214]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_vblank@pipe-c-query-busy.html
- shard-mtlp: NOTRUN -> [DMESG-WARN][215] ([i915#2017])
[215]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@kms_vblank@pipe-c-query-busy.html
* igt@kms_vblank@pipe-d-query-forked-busy-hang:
- shard-rkl: NOTRUN -> [SKIP][216] ([i915#4070] / [i915#533] / [i915#6768])
[216]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_vblank@pipe-d-query-forked-busy-hang.html
* igt@kms_vrr@negative-basic:
- shard-rkl: NOTRUN -> [SKIP][217] ([i915#3555]) +1 other test skip
[217]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_vrr@negative-basic.html
* igt@kms_writeback@writeback-pixel-formats:
- shard-mtlp: NOTRUN -> [SKIP][218] ([i915#2437])
[218]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@kms_writeback@writeback-pixel-formats.html
* igt@perf@global-sseu-config:
- shard-dg2: NOTRUN -> [SKIP][219] ([i915#7387])
[219]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-7/igt@perf@global-sseu-config.html
* igt@perf@unprivileged-single-ctx-counters:
- shard-mtlp: NOTRUN -> [SKIP][220] ([fdo#109289]) +1 other test skip
[220]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@perf@unprivileged-single-ctx-counters.html
* igt@perf_pmu@busy-double-start@vecs1:
- shard-dg2: [PASS][221] -> [FAIL][222] ([i915#4349]) +3 other tests fail
[221]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-dg2-6/igt@perf_pmu@busy-double-start@vecs1.html
[222]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@perf_pmu@busy-double-start@vecs1.html
* igt@perf_pmu@frequency@gt0:
- shard-dg2: NOTRUN -> [FAIL][223] ([i915#6806])
[223]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@perf_pmu@frequency@gt0.html
* igt@prime_vgem@basic-fence-mmap:
- shard-mtlp: NOTRUN -> [SKIP][224] ([i915#3708] / [i915#4077])
[224]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@prime_vgem@basic-fence-mmap.html
* igt@prime_vgem@basic-write:
- shard-rkl: NOTRUN -> [SKIP][225] ([fdo#109295] / [i915#3291] / [i915#3708])
[225]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@prime_vgem@basic-write.html
- shard-mtlp: NOTRUN -> [SKIP][226] ([i915#3708])
[226]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-5/igt@prime_vgem@basic-write.html
* igt@v3d/v3d_perfmon@get-values-invalid-perfmon:
- shard-mtlp: NOTRUN -> [SKIP][227] ([i915#2575]) +4 other tests skip
[227]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@v3d/v3d_perfmon@get-values-invalid-perfmon.html
* igt@v3d/v3d_submit_csd@bad-multisync-extension:
- shard-rkl: NOTRUN -> [SKIP][228] ([fdo#109315]) +2 other tests skip
[228]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@v3d/v3d_submit_csd@bad-multisync-extension.html
* igt@v3d/v3d_submit_csd@job-perfmon:
- shard-dg2: NOTRUN -> [SKIP][229] ([i915#2575]) +10 other tests skip
[229]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-7/igt@v3d/v3d_submit_csd@job-perfmon.html
* igt@vc4/vc4_perfmon@create-perfmon-exceed:
- shard-mtlp: NOTRUN -> [SKIP][230] ([i915#7711])
[230]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@vc4/vc4_perfmon@create-perfmon-exceed.html
* igt@vc4/vc4_perfmon@create-two-perfmon:
- shard-rkl: NOTRUN -> [SKIP][231] ([i915#7711])
[231]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@vc4/vc4_perfmon@create-two-perfmon.html
* igt@vc4/vc4_tiling@get-bad-modifier:
- shard-dg2: NOTRUN -> [SKIP][232] ([i915#7711]) +7 other tests skip
[232]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-3/igt@vc4/vc4_tiling@get-bad-modifier.html
#### Possible fixes ####
* igt@drm_fdinfo@most-busy-idle-check-all@rcs0:
- shard-rkl: [FAIL][233] ([i915#7742]) -> [PASS][234]
[233]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-rkl-7/igt@drm_fdinfo@most-busy-idle-check-all@rcs0.html
[234]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@drm_fdinfo@most-busy-idle-check-all@rcs0.html
* igt@gem_eio@in-flight-contexts-10ms:
- shard-mtlp: [FAIL][235] -> [PASS][236]
[235]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-mtlp-2/igt@gem_eio@in-flight-contexts-10ms.html
[236]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@gem_eio@in-flight-contexts-10ms.html
* igt@gem_eio@reset-stress:
- shard-dg1: [FAIL][237] ([i915#5784]) -> [PASS][238]
[237]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-dg1-14/igt@gem_eio@reset-stress.html
[238]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg1-15/igt@gem_eio@reset-stress.html
* igt@gem_exec_schedule@preempt-engines@ccs0:
- shard-mtlp: [FAIL][239] ([i915#9119]) -> [PASS][240] +4 other tests pass
[239]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-mtlp-4/igt@gem_exec_schedule@preempt-engines@ccs0.html
[240]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-7/igt@gem_exec_schedule@preempt-engines@ccs0.html
* igt@gem_exec_schedule@preempt-engines@rcs0:
- shard-mtlp: [DMESG-FAIL][241] ([i915#8962]) -> [PASS][242]
[241]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-mtlp-4/igt@gem_exec_schedule@preempt-engines@rcs0.html
[242]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-7/igt@gem_exec_schedule@preempt-engines@rcs0.html
* igt@gen9_exec_parse@allowed-single:
- shard-apl: [INCOMPLETE][243] ([i915#5566]) -> [PASS][244]
[243]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-apl1/igt@gen9_exec_parse@allowed-single.html
[244]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-apl4/igt@gen9_exec_parse@allowed-single.html
* igt@i915_module_load@reload-with-fault-injection:
- shard-mtlp: [ABORT][245] ([i915#8489] / [i915#8668]) -> [PASS][246]
[245]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-mtlp-2/igt@i915_module_load@reload-with-fault-injection.html
[246]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-2/igt@i915_module_load@reload-with-fault-injection.html
* igt@i915_pm_rpm@dpms-mode-unset-non-lpsp:
- shard-dg1: [SKIP][247] ([i915#1397]) -> [PASS][248] +1 other test pass
[247]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-dg1-19/igt@i915_pm_rpm@dpms-mode-unset-non-lpsp.html
[248]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg1-16/igt@i915_pm_rpm@dpms-mode-unset-non-lpsp.html
* igt@i915_pm_rpm@dpms-non-lpsp:
- shard-rkl: [SKIP][249] ([i915#1397]) -> [PASS][250] +1 other test pass
[249]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-rkl-7/igt@i915_pm_rpm@dpms-non-lpsp.html
[250]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@i915_pm_rpm@dpms-non-lpsp.html
* igt@i915_pm_rpm@i2c:
- shard-dg2: [FAIL][251] ([i915#8717]) -> [PASS][252]
[251]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-dg2-2/igt@i915_pm_rpm@i2c.html
[252]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-11/igt@i915_pm_rpm@i2c.html
* igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-0-hflip:
- shard-mtlp: [FAIL][253] ([i915#5138]) -> [PASS][254]
[253]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-mtlp-4/igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-0-hflip.html
[254]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-4/igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-0-hflip.html
* igt@kms_big_fb@x-tiled-max-hw-stride-32bpp-rotate-0-hflip-async-flip:
- shard-tglu: [FAIL][255] ([i915#3743]) -> [PASS][256] +1 other test pass
[255]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-tglu-3/igt@kms_big_fb@x-tiled-max-hw-stride-32bpp-rotate-0-hflip-async-flip.html
[256]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-tglu-7/igt@kms_big_fb@x-tiled-max-hw-stride-32bpp-rotate-0-hflip-async-flip.html
* igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions:
- shard-glk: [FAIL][257] ([i915#2346]) -> [PASS][258]
[257]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-glk2/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions.html
[258]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-glk3/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions.html
- shard-apl: [FAIL][259] ([i915#2346]) -> [PASS][260]
[259]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-apl6/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions.html
[260]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-apl6/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions.html
* igt@kms_frontbuffer_tracking@fbc-rgb565-draw-blt:
- shard-dg2: [FAIL][261] ([i915#6880]) -> [PASS][262]
[261]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-dg2-11/igt@kms_frontbuffer_tracking@fbc-rgb565-draw-blt.html
[262]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-11/igt@kms_frontbuffer_tracking@fbc-rgb565-draw-blt.html
* igt@kms_plane_cursor@overlay@pipe-d-edp-1-size-128:
- shard-mtlp: [ABORT][263] ([i915#9414]) -> [PASS][264] +1 other test pass
[263]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-mtlp-6/igt@kms_plane_cursor@overlay@pipe-d-edp-1-size-128.html
[264]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-mtlp-8/igt@kms_plane_cursor@overlay@pipe-d-edp-1-size-128.html
* igt@kms_rotation_crc@primary-rotation-270:
- shard-rkl: [INCOMPLETE][265] ([i915#8875]) -> [PASS][266]
[265]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-rkl-4/igt@kms_rotation_crc@primary-rotation-270.html
[266]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-6/igt@kms_rotation_crc@primary-rotation-270.html
* igt@perf@enable-disable@0-rcs0:
- shard-dg2: [FAIL][267] ([i915#8724]) -> [PASS][268]
[267]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-dg2-11/igt@perf@enable-disable@0-rcs0.html
[268]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-7/igt@perf@enable-disable@0-rcs0.html
#### Warnings ####
* igt@gem_exec_fair@basic-pace@bcs0:
- shard-tglu: [FAIL][269] ([i915#2876]) -> [FAIL][270] ([i915#2842])
[269]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-tglu-10/igt@gem_exec_fair@basic-pace@bcs0.html
[270]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-tglu-6/igt@gem_exec_fair@basic-pace@bcs0.html
* igt@gem_lmem_swapping@parallel-random-verify-ccs@lmem0:
- shard-dg1: [SKIP][271] ([i915#4565]) -> [SKIP][272] ([i915#4423] / [i915#4565])
[271]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-dg1-12/igt@gem_lmem_swapping@parallel-random-verify-ccs@lmem0.html
[272]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg1-15/igt@gem_lmem_swapping@parallel-random-verify-ccs@lmem0.html
* igt@gem_lmem_swapping@smem-oom@lmem0:
- shard-dg2: [DMESG-WARN][273] ([i915#4936] / [i915#5493]) -> [TIMEOUT][274] ([i915#5493])
[273]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-dg2-11/igt@gem_lmem_swapping@smem-oom@lmem0.html
[274]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-2/igt@gem_lmem_swapping@smem-oom@lmem0.html
* igt@i915_module_load@reload-with-fault-injection:
- shard-dg2: [DMESG-WARN][275] ([i915#8617]) -> [WARN][276] ([i915#7356])
[275]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-dg2-7/igt@i915_module_load@reload-with-fault-injection.html
[276]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-5/igt@i915_module_load@reload-with-fault-injection.html
* igt@i915_pm_rc6_residency@rc6-idle@vcs0:
- shard-tglu: [FAIL][277] ([i915#2681] / [i915#3591]) -> [WARN][278] ([i915#2681])
[277]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-tglu-2/igt@i915_pm_rc6_residency@rc6-idle@vcs0.html
[278]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-tglu-2/igt@i915_pm_rc6_residency@rc6-idle@vcs0.html
* igt@kms_fbcon_fbt@psr-suspend:
- shard-rkl: [SKIP][279] ([fdo#110189] / [i915#3955]) -> [SKIP][280] ([i915#3955])
[279]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-rkl-1/igt@kms_fbcon_fbt@psr-suspend.html
[280]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-rkl-4/igt@kms_fbcon_fbt@psr-suspend.html
* igt@perf@non-zero-reason@0-rcs0:
- shard-dg2: [FAIL][281] ([i915#7484]) -> [FAIL][282] ([i915#9100])
[281]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13709/shard-dg2-2/igt@perf@non-zero-reason@0-rcs0.html
[282]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/shard-dg2-11/igt@perf@non-zero-reason@0-rcs0.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[IGT#2]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/issues/2
[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#109289]: https://bugs.freedesktop.org/show_bug.cgi?id=109289
[fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295
[fdo#109315]: https://bugs.freedesktop.org/show_bug.cgi?id=109315
[fdo#110189]: https://bugs.freedesktop.org/show_bug.cgi?id=110189
[fdo#110723]: https://bugs.freedesktop.org/show_bug.cgi?id=110723
[fdo#111614]: https://bugs.freedesktop.org/show_bug.cgi?id=111614
[fdo#111615]: https://bugs.freedesktop.org/show_bug.cgi?id=111615
[fdo#111767]: https://bugs.freedesktop.org/show_bug.cgi?id=111767
[fdo#111825]: https://bugs.freedesktop.org/show_bug.cgi?id=111825
[fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
[fdo#112283]: https://bugs.freedesktop.org/show_bug.cgi?id=112283
[i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
[i915#1397]: https://gitlab.freedesktop.org/drm/intel/issues/1397
[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#2017]: https://gitlab.freedesktop.org/drm/intel/issues/2017
[i915#2346]: https://gitlab.freedesktop.org/drm/intel/issues/2346
[i915#2437]: https://gitlab.freedesktop.org/drm/intel/issues/2437
[i915#2521]: https://gitlab.freedesktop.org/drm/intel/issues/2521
[i915#2527]: https://gitlab.freedesktop.org/drm/intel/issues/2527
[i915#2575]: https://gitlab.freedesktop.org/drm/intel/issues/2575
[i915#2672]: https://gitlab.freedesktop.org/drm/intel/issues/2672
[i915#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#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#2876]: https://gitlab.freedesktop.org/drm/intel/issues/2876
[i915#3023]: https://gitlab.freedesktop.org/drm/intel/issues/3023
[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#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#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#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#3743]: https://gitlab.freedesktop.org/drm/intel/issues/3743
[i915#3840]: https://gitlab.freedesktop.org/drm/intel/issues/3840
[i915#3886]: https://gitlab.freedesktop.org/drm/intel/issues/3886
[i915#3936]: https://gitlab.freedesktop.org/drm/intel/issues/3936
[i915#3955]: https://gitlab.freedesktop.org/drm/intel/issues/3955
[i915#4070]: https://gitlab.freedesktop.org/drm/intel/issues/4070
[i915#4077]: https://gitlab.freedesktop.org/drm/intel/issues/4077
[i915#4083]: https://gitlab.freedesktop.org/drm/intel/issues/4083
[i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103
[i915#4212]: https://gitlab.freedesktop.org/drm/intel/issues/4212
[i915#4213]: https://gitlab.freedesktop.org/drm/intel/issues/4213
[i915#4235]: https://gitlab.freedesktop.org/drm/intel/issues/4235
[i915#4270]: https://gitlab.freedesktop.org/drm/intel/issues/4270
[i915#4349]: https://gitlab.freedesktop.org/drm/intel/issues/4349
[i915#4423]: https://gitlab.freedesktop.org/drm/intel/issues/4423
[i915#4537]: https://gitlab.freedesktop.org/drm/intel/issues/4537
[i915#4538]: https://gitlab.freedesktop.org/drm/intel/issues/4538
[i915#4565]: https://gitlab.freedesktop.org/drm/intel/issues/4565
[i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
[i915#4812]: https://gitlab.freedesktop.org/drm/intel/issues/4812
[i915#4852]: https://gitlab.freedesktop.org/drm/intel/issues/4852
[i915#4860]: https://gitlab.freedesktop.org/drm/intel/issues/4860
[i915#4885]: https://gitlab.freedesktop.org/drm/intel/issues/4885
[i915#4936]: https://gitlab.freedesktop.org/drm/intel/issues/4936
[i915#5138]: https://gitlab.freedesktop.org/drm/intel/issues/5138
[i915#5176]: https://gitlab.freedesktop.org/drm/intel/issues/5176
[i915#5190]: https://gitlab.freedesktop.org/drm/intel/issues/5190
[i915#5235]: https://gitlab.freedesktop.org/drm/intel/issues/5235
[i915#5286]: https://gitlab.freedesktop.org/drm/intel/issues/5286
[i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533
[i915#5354]: https://gitlab.freedesktop.org/drm/intel/issues/5354
[i915#5465]: https://gitlab.freedesktop.org/drm/intel/issues/5465
[i915#5493]: https://gitlab.freedesktop.org/drm/intel/issues/5493
[i915#5566]: https://gitlab.freedesktop.org/drm/intel/issues/5566
[i915#5608]: https://gitlab.freedesktop.org/drm/intel/issues/5608
[i915#5784]: https://gitlab.freedesktop.org/drm/intel/issues/5784
[i915#5882]: https://gitlab.freedesktop.org/drm/intel/issues/5882
[i915#6095]: https://gitlab.freedesktop.org/drm/intel/issues/6095
[i915#6122]: https://gitlab.freedesktop.org/drm/intel/issues/6122
[i915#6335]: https://gitlab.freedesktop.org/drm/intel/issues/6335
[i915#6524]: https://gitlab.freedesktop.org/drm/intel/issues/6524
[i915#658]: https://gitlab.freedesktop.org/drm/intel/issues/658
[i915#6590]: https://gitlab.freedesktop.org/drm/intel/issues/6590
[i915#6768]: https://gitlab.freedesktop.org/drm/intel/issues/6768
[i915#6805]: https://gitlab.freedesktop.org/drm/intel/issues/6805
[i915#6806]: https://gitlab.freedesktop.org/drm/intel/issues/6806
[i915#6880]: https://gitlab.freedesktop.org/drm/intel/issues/6880
[i915#6944]: https://gitlab.freedesktop.org/drm/intel/issues/6944
[i915#6953]: https://gitlab.freedesktop.org/drm/intel/issues/6953
[i915#7118]: https://gitlab.freedesktop.org/drm/intel/issues/7118
[i915#7213]: https://gitlab.freedesktop.org/drm/intel/issues/7213
[i915#7356]: https://gitlab.freedesktop.org/drm/intel/issues/7356
[i915#7387]: https://gitlab.freedesktop.org/drm/intel/issues/7387
[i915#7484]: https://gitlab.freedesktop.org/drm/intel/issues/7484
[i915#7697]: https://gitlab.freedesktop.org/drm/intel/issues/7697
[i915#7711]: https://gitlab.freedesktop.org/drm/intel/issues/7711
[i915#7742]: https://gitlab.freedesktop.org/drm/intel/issues/7742
[i915#7828]: https://gitlab.freedesktop.org/drm/intel/issues/7828
[i915#7975]: https://gitlab.freedesktop.org/drm/intel/issues/7975
[i915#8213]: https://gitlab.freedesktop.org/drm/intel/issues/8213
[i915#8228]: https://gitlab.freedesktop.org/drm/intel/issues/8228
[i915#8247]: https://gitlab.freedesktop.org/drm/intel/issues/8247
[i915#8292]: https://gitlab.freedesktop.org/drm/intel/issues/8292
[i915#8293]: https://gitlab.freedesktop.org/drm/intel/issues/8293
[i915#8346]: https://gitlab.freedesktop.org/drm/intel/issues/8346
[i915#8411]: https://gitlab.freedesktop.org/drm/intel/issues/8411
[i915#8414]: https://gitlab.freedesktop.org/drm/intel/issues/8414
[i915#8428]: https://gitlab.freedesktop.org/drm/intel/issues/8428
[i915#8489]: https://gitlab.freedesktop.org/drm/intel/issues/8489
[i915#8502]: https://gitlab.freedesktop.org/drm/intel/issues/8502
[i915#8555]: https://gitlab.freedesktop.org/drm/intel/issues/8555
[i915#8617]: https://gitlab.freedesktop.org/drm/intel/issues/8617
[i915#8668]: https://gitlab.freedesktop.org/drm/intel/issues/8668
[i915#8708]: https://gitlab.freedesktop.org/drm/intel/issues/8708
[i915#8709]: https://gitlab.freedesktop.org/drm/intel/issues/8709
[i915#8717]: https://gitlab.freedesktop.org/drm/intel/issues/8717
[i915#8724]: https://gitlab.freedesktop.org/drm/intel/issues/8724
[i915#8810]: https://gitlab.freedesktop.org/drm/intel/issues/8810
[i915#8814]: https://gitlab.freedesktop.org/drm/intel/issues/8814
[i915#8875]: https://gitlab.freedesktop.org/drm/intel/issues/8875
[i915#8962]: https://gitlab.freedesktop.org/drm/intel/issues/8962
[i915#9010]: https://gitlab.freedesktop.org/drm/intel/issues/9010
[i915#9053]: https://gitlab.freedesktop.org/drm/intel/issues/9053
[i915#9100]: https://gitlab.freedesktop.org/drm/intel/issues/9100
[i915#9119]: https://gitlab.freedesktop.org/drm/intel/issues/9119
[i915#9196]: https://gitlab.freedesktop.org/drm/intel/issues/9196
[i915#9226]: https://gitlab.freedesktop.org/drm/intel/issues/9226
[i915#9227]: https://gitlab.freedesktop.org/drm/intel/issues/9227
[i915#9261]: https://gitlab.freedesktop.org/drm/intel/issues/9261
[i915#9295]: https://gitlab.freedesktop.org/drm/intel/issues/9295
[i915#9311]: https://gitlab.freedesktop.org/drm/intel/issues/9311
[i915#9318]: https://gitlab.freedesktop.org/drm/intel/issues/9318
[i915#9337]: https://gitlab.freedesktop.org/drm/intel/issues/9337
[i915#9392]: https://gitlab.freedesktop.org/drm/intel/issues/9392
[i915#9412]: https://gitlab.freedesktop.org/drm/intel/issues/9412
[i915#9414]: https://gitlab.freedesktop.org/drm/intel/issues/9414
[i915#9423]: https://gitlab.freedesktop.org/drm/intel/issues/9423
[i915#9424]: https://gitlab.freedesktop.org/drm/intel/issues/9424
[i915#9433]: https://gitlab.freedesktop.org/drm/intel/issues/9433
[i915#9436]: https://gitlab.freedesktop.org/drm/intel/issues/9436
Build changes
-------------
* Linux: CI_DRM_13709 -> Patchwork_124602v1
* Piglit: None -> piglit_4509
CI-20190529: 20190529
CI_DRM_13709: 7c6b3bb3b7a2602a399f5f8e6391ab865cad4197 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_7512: 2eb58faf82d3cd5e2e74154a7319cff56eb40762 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_124602v1: 7c6b3bb3b7a2602a399f5f8e6391ab865cad4197 @ git://anongit.freedesktop.org/gfx-ci/linux
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_124602v1/index.html
[-- Attachment #2: Type: text/html, Size: 83623 bytes --]
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation
2023-10-04 12:55 ` [Intel-gfx] [PATCH] " Rodrigo Vivi
@ 2023-10-05 7:05 ` Kahola, Mika
2023-10-05 15:13 ` Rodrigo Vivi
0 siblings, 1 reply; 12+ messages in thread
From: Kahola, Mika @ 2023-10-05 7:05 UTC (permalink / raw)
To: Vivi, Rodrigo; +Cc: intel-gfx@lists.freedesktop.org
> -----Original Message-----
> From: Vivi, Rodrigo <rodrigo.vivi@intel.com>
> Sent: Wednesday, October 4, 2023 3:56 PM
> To: Kahola, Mika <mika.kahola@intel.com>
> Cc: intel-gfx@lists.freedesktop.org
> Subject: Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation
>
> On Wed, Oct 04, 2023 at 01:25:04PM +0300, Mika Kahola wrote:
> > Every know and then we receive the following error when running for
> > example IGT test kms_flip.
> >
> > [drm] *ERROR* PHY G Read 0d80 failed after 3 retries.
> > [drm] *ERROR* PHY G Write 0d81 failed after 3 retries.
> >
> > Since the error is sporadic in nature, the patch proposes to reset the
> > message bus after every successful or unsuccessful read or write
> > operation. However, testing revealed that this alone is not sufficient
> > method an additiona delay is also introduces anything from 200us to
> > 300us. This delay is experimental value and has no specification to
> > back it up.
>
> have you tried the delays without the bus_reset?
Yes, we have bumped up the delay, first from 0x100 to 0x200 and then as per
BSpec change 0xa000 and I have tried 0xf000. Increasing the timeout reduces
the frequency of this error but doesn't solve this issue.
> have you talked to hw architects about this?
Yes, HW guys requested traces which I provided but based on these the sequence we use in i915
is correct.
>
> I wonder if we should add the delay inside the bus_reset itself?
> although the bit 15 clear check should be enough by itself and it doesn't look like it is a hw/fw reset involved to justify the extra
> delay.
That should be enough. To me, it looks like when reading/writing to the bus maybe too fast, the hw cannot handle that and we need
to reset and let things settle down before trying again.
>
> well, at least some /* FIXME: */ or /* XXX: */ comments is desired along with the messages if we are going with this hack without
> understanding why...
True, I will add these the the patch.
Thanks for review!
-Mika-
>
> >
> > Signed-off-by: Mika Kahola <mika.kahola@intel.com>
> > ---
> > drivers/gpu/drm/i915/display/intel_cx0_phy.c | 6 ++++++
> > 1 file changed, 6 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> > b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> > index abd607b564f1..a71b8a29d6b0 100644
> > --- a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> > +++ b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> > @@ -220,9 +220,12 @@ static u8 __intel_cx0_read(struct drm_i915_private *i915, enum port port,
> > /* 3 tries is assumed to be enough to read successfully */
> > for (i = 0; i < 3; i++) {
> > status = __intel_cx0_read_once(i915, port, lane, addr);
> > + intel_cx0_bus_reset(i915, port, lane);
> >
> > if (status >= 0)
> > return status;
> > +
> > + usleep_range(200, 300);
> > }
> >
> > drm_err_once(&i915->drm, "PHY %c Read %04x failed after %d
> > retries.\n", @@ -299,9 +302,12 @@ static void __intel_cx0_write(struct drm_i915_private *i915, enum port port,
> > /* 3 tries is assumed to be enough to write successfully */
> > for (i = 0; i < 3; i++) {
> > status = __intel_cx0_write_once(i915, port, lane, addr, data,
> > committed);
> > + intel_cx0_bus_reset(i915, port, lane);
> >
> > if (status == 0)
> > return;
> > +
> > + usleep_range(200, 300);
> > }
> >
> > drm_err_once(&i915->drm,
> > --
> > 2.34.1
> >
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation
2023-10-05 7:05 ` Kahola, Mika
@ 2023-10-05 15:13 ` Rodrigo Vivi
2023-10-05 15:32 ` Gustavo Sousa
2023-10-05 15:40 ` Gustavo Sousa
0 siblings, 2 replies; 12+ messages in thread
From: Rodrigo Vivi @ 2023-10-05 15:13 UTC (permalink / raw)
To: Kahola, Mika; +Cc: intel-gfx@lists.freedesktop.org
On Thu, Oct 05, 2023 at 03:05:31AM -0400, Kahola, Mika wrote:
> > -----Original Message-----
> > From: Vivi, Rodrigo <rodrigo.vivi@intel.com>
> > Sent: Wednesday, October 4, 2023 3:56 PM
> > To: Kahola, Mika <mika.kahola@intel.com>
> > Cc: intel-gfx@lists.freedesktop.org
> > Subject: Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation
> >
> > On Wed, Oct 04, 2023 at 01:25:04PM +0300, Mika Kahola wrote:
> > > Every know and then we receive the following error when running for
> > > example IGT test kms_flip.
> > >
> > > [drm] *ERROR* PHY G Read 0d80 failed after 3 retries.
> > > [drm] *ERROR* PHY G Write 0d81 failed after 3 retries.
> > >
> > > Since the error is sporadic in nature, the patch proposes to reset the
> > > message bus after every successful or unsuccessful read or write
> > > operation. However, testing revealed that this alone is not sufficient
> > > method an additiona delay is also introduces anything from 200us to
> > > 300us. This delay is experimental value and has no specification to
> > > back it up.
> >
> > have you tried the delays without the bus_reset?
> Yes, we have bumped up the delay, first from 0x100 to 0x200 and then as per
> BSpec change 0xa000 and I have tried 0xf000. Increasing the timeout reduces
> the frequency of this error but doesn't solve this issue.
what is exactly this BSPec's 0xa000? where can I see it? So maybe you can
update the message above removing the 'no specification to back it up'.
Oh, and my english is bad, but it looks to me that 'empirical' might
sound better than 'experimental' for this case, since you really did
a lot of experiments before coming to this final conclusion.
>
> > have you talked to hw architects about this?
> Yes, HW guys requested traces which I provided but based on these the sequence we use in i915
> is correct.
>
> >
> > I wonder if we should add the delay inside the bus_reset itself?
> > although the bit 15 clear check should be enough by itself and it doesn't look like it is a hw/fw reset involved to justify the extra
> > delay.
> That should be enough. To me, it looks like when reading/writing to the bus maybe too fast, the hw cannot handle that and we need
> to reset and let things settle down before trying again.
>
> >
> > well, at least some /* FIXME: */ or /* XXX: */ comments is desired along with the messages if we are going with this hack without
> > understanding why...
> True, I will add these the the patch.
>
> Thanks for review!
>
> -Mika-
> >
> > >
> > > Signed-off-by: Mika Kahola <mika.kahola@intel.com>
> > > ---
> > > drivers/gpu/drm/i915/display/intel_cx0_phy.c | 6 ++++++
> > > 1 file changed, 6 insertions(+)
> > >
> > > diff --git a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> > > b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> > > index abd607b564f1..a71b8a29d6b0 100644
> > > --- a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> > > +++ b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> > > @@ -220,9 +220,12 @@ static u8 __intel_cx0_read(struct drm_i915_private *i915, enum port port,
> > > /* 3 tries is assumed to be enough to read successfully */
> > > for (i = 0; i < 3; i++) {
> > > status = __intel_cx0_read_once(i915, port, lane, addr);
> > > + intel_cx0_bus_reset(i915, port, lane);
> > >
> > > if (status >= 0)
> > > return status;
> > > +
> > > + usleep_range(200, 300);
> > > }
> > >
> > > drm_err_once(&i915->drm, "PHY %c Read %04x failed after %d
> > > retries.\n", @@ -299,9 +302,12 @@ static void __intel_cx0_write(struct drm_i915_private *i915, enum port port,
> > > /* 3 tries is assumed to be enough to write successfully */
> > > for (i = 0; i < 3; i++) {
> > > status = __intel_cx0_write_once(i915, port, lane, addr, data,
> > > committed);
> > > + intel_cx0_bus_reset(i915, port, lane);
> > >
> > > if (status == 0)
> > > return;
> > > +
> > > + usleep_range(200, 300);
> > > }
> > >
> > > drm_err_once(&i915->drm,
> > > --
> > > 2.34.1
> > >
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation
2023-10-05 15:13 ` Rodrigo Vivi
@ 2023-10-05 15:32 ` Gustavo Sousa
2023-10-05 15:40 ` Gustavo Sousa
1 sibling, 0 replies; 12+ messages in thread
From: Gustavo Sousa @ 2023-10-05 15:32 UTC (permalink / raw)
To: Kahola, Mika, Rodrigo Vivi; +Cc: intel-gfx@
Quoting Rodrigo Vivi (2023-10-05 12:13:34-03:00)
>On Thu, Oct 05, 2023 at 03:05:31AM -0400, Kahola, Mika wrote:
>> > -----Original Message-----
>> > From: Vivi, Rodrigo <rodrigo.vivi@intel.com>
>> > Sent: Wednesday, October 4, 2023 3:56 PM
>> > To: Kahola, Mika <mika.kahola@intel.com>
>> > Cc: intel-gfx@lists.freedesktop.org
>> > Subject: Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation
>> >
>> > On Wed, Oct 04, 2023 at 01:25:04PM +0300, Mika Kahola wrote:
>> > > Every know and then we receive the following error when running for
>> > > example IGT test kms_flip.
>> > >
>> > > [drm] *ERROR* PHY G Read 0d80 failed after 3 retries.
>> > > [drm] *ERROR* PHY G Write 0d81 failed after 3 retries.
>> > >
>> > > Since the error is sporadic in nature, the patch proposes to reset the
>> > > message bus after every successful or unsuccessful read or write
>> > > operation. However, testing revealed that this alone is not sufficient
>> > > method an additiona delay is also introduces anything from 200us to
>> > > 300us. This delay is experimental value and has no specification to
>> > > back it up.
>> >
>> > have you tried the delays without the bus_reset?
>> Yes, we have bumped up the delay, first from 0x100 to 0x200 and then as per
>> BSpec change 0xa000 and I have tried 0xf000. Increasing the timeout reduces
>> the frequency of this error but doesn't solve this issue.
>
>what is exactly this BSPec's 0xa000? where can I see it? So maybe you can
>update the message above removing the 'no specification to back it up'.
I think we are confusing "delay" with the "timeout parameter" of the msgbus.
The PHY has a register to control the timeout parameter of msgbus transactions
(BSpec 65156). It's default value is 0x100. With commit e028d7a4235d
("drm/i915/cx0: Check and increase msgbus timeout threshold"), we had integrated
a workaround that bumped the timeout value to 0x200 in case timeouts were
observed. Later on, there was a BSpec update with the formal timeout value to be
programmed to 0xa000, which was incorporated with commit e35628968032
("drm/i915/cx0: Add step for programming msgbus timer").
I *believe* what Rodrigo has asked was about the usleep_range() calls added with
this patch, if we tried to only keep the usleed_range() without the bus reset.
--
Gustavo Sousa
>
>Oh, and my english is bad, but it looks to me that 'empirical' might
>sound better than 'experimental' for this case, since you really did
>a lot of experiments before coming to this final conclusion.
>
>>
>> > have you talked to hw architects about this?
>> Yes, HW guys requested traces which I provided but based on these the sequence we use in i915
>> is correct.
>>
>> >
>> > I wonder if we should add the delay inside the bus_reset itself?
>> > although the bit 15 clear check should be enough by itself and it doesn't look like it is a hw/fw reset involved to justify the extra
>> > delay.
>> That should be enough. To me, it looks like when reading/writing to the bus maybe too fast, the hw cannot handle that and we need
>> to reset and let things settle down before trying again.
>>
>> >
>> > well, at least some /* FIXME: */ or /* XXX: */ comments is desired along with the messages if we are going with this hack without
>> > understanding why...
>> True, I will add these the the patch.
>>
>> Thanks for review!
>>
>> -Mika-
>> >
>> > >
>> > > Signed-off-by: Mika Kahola <mika.kahola@intel.com>
>> > > ---
>> > > drivers/gpu/drm/i915/display/intel_cx0_phy.c | 6 ++++++
>> > > 1 file changed, 6 insertions(+)
>> > >
>> > > diff --git a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
>> > > b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
>> > > index abd607b564f1..a71b8a29d6b0 100644
>> > > --- a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
>> > > +++ b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
>> > > @@ -220,9 +220,12 @@ static u8 __intel_cx0_read(struct drm_i915_private *i915, enum port port,
>> > > /* 3 tries is assumed to be enough to read successfully */
>> > > for (i = 0; i < 3; i++) {
>> > > status = __intel_cx0_read_once(i915, port, lane, addr);
>> > > + intel_cx0_bus_reset(i915, port, lane);
>> > >
>> > > if (status >= 0)
>> > > return status;
>> > > +
>> > > + usleep_range(200, 300);
>> > > }
>> > >
>> > > drm_err_once(&i915->drm, "PHY %c Read %04x failed after %d
>> > > retries.\n", @@ -299,9 +302,12 @@ static void __intel_cx0_write(struct drm_i915_private *i915, enum port port,
>> > > /* 3 tries is assumed to be enough to write successfully */
>> > > for (i = 0; i < 3; i++) {
>> > > status = __intel_cx0_write_once(i915, port, lane, addr, data,
>> > > committed);
>> > > + intel_cx0_bus_reset(i915, port, lane);
>> > >
>> > > if (status == 0)
>> > > return;
>> > > +
>> > > + usleep_range(200, 300);
>> > > }
>> > >
>> > > drm_err_once(&i915->drm,
>> > > --
>> > > 2.34.1
>> > >
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation
2023-10-05 15:13 ` Rodrigo Vivi
2023-10-05 15:32 ` Gustavo Sousa
@ 2023-10-05 15:40 ` Gustavo Sousa
2023-10-05 16:10 ` Rodrigo Vivi
1 sibling, 1 reply; 12+ messages in thread
From: Gustavo Sousa @ 2023-10-05 15:40 UTC (permalink / raw)
To: Kahola, Mika, Rodrigo Vivi; +Cc: intel-gfx
Quoting Rodrigo Vivi (2023-10-05 12:13:34-03:00)
>On Thu, Oct 05, 2023 at 03:05:31AM -0400, Kahola, Mika wrote:
>> > -----Original Message-----
>> > From: Vivi, Rodrigo <rodrigo.vivi@intel.com>
>> > Sent: Wednesday, October 4, 2023 3:56 PM
>> > To: Kahola, Mika <mika.kahola@intel.com>
>> > Cc: intel-gfx@lists.freedesktop.org
>> > Subject: Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation
>> >
>> > On Wed, Oct 04, 2023 at 01:25:04PM +0300, Mika Kahola wrote:
>> > > Every know and then we receive the following error when running for
>> > > example IGT test kms_flip.
>> > >
>> > > [drm] *ERROR* PHY G Read 0d80 failed after 3 retries.
>> > > [drm] *ERROR* PHY G Write 0d81 failed after 3 retries.
>> > >
>> > > Since the error is sporadic in nature, the patch proposes to reset the
>> > > message bus after every successful or unsuccessful read or write
>> > > operation. However, testing revealed that this alone is not sufficient
>> > > method an additiona delay is also introduces anything from 200us to
>> > > 300us. This delay is experimental value and has no specification to
>> > > back it up.
>> >
>> > have you tried the delays without the bus_reset?
>> Yes, we have bumped up the delay, first from 0x100 to 0x200 and then as per
>> BSpec change 0xa000 and I have tried 0xf000. Increasing the timeout reduces
>> the frequency of this error but doesn't solve this issue.
>
>what is exactly this BSPec's 0xa000? where can I see it? So maybe you can
>update the message above removing the 'no specification to back it up'.
(Resending this because I got a delivery failure notification)
I think we are confusing "delay" with the "timeout parameter" of the msgbus.
The PHY has a register to control the timeout parameter of msgbus transactions
(BSpec 65156). It's default value is 0x100. With commit e028d7a4235d
("drm/i915/cx0: Check and increase msgbus timeout threshold"), we had integrated
a workaround that bumped the timeout value to 0x200 in case timeouts were
observed. Later on, there was a BSpec update with the formal timeout value to be
programmed to 0xa000, which was incorporated with commit e35628968032
("drm/i915/cx0: Add step for programming msgbus timer").
I *believe* what Rodrigo has asked was about the usleep_range() calls added with
this patch, if we tried to only keep the usleed_range() without the bus reset.
--
Gustavo Sousa
>
>Oh, and my english is bad, but it looks to me that 'empirical' might
>sound better than 'experimental' for this case, since you really did
>a lot of experiments before coming to this final conclusion.
>
>>
>> > have you talked to hw architects about this?
>> Yes, HW guys requested traces which I provided but based on these the sequence we use in i915
>> is correct.
>>
>> >
>> > I wonder if we should add the delay inside the bus_reset itself?
>> > although the bit 15 clear check should be enough by itself and it doesn't look like it is a hw/fw reset involved to justify the extra
>> > delay.
>> That should be enough. To me, it looks like when reading/writing to the bus maybe too fast, the hw cannot handle that and we need
>> to reset and let things settle down before trying again.
>>
>> >
>> > well, at least some /* FIXME: */ or /* XXX: */ comments is desired along with the messages if we are going with this hack without
>> > understanding why...
>> True, I will add these the the patch.
>>
>> Thanks for review!
>>
>> -Mika-
>> >
>> > >
>> > > Signed-off-by: Mika Kahola <mika.kahola@intel.com>
>> > > ---
>> > > drivers/gpu/drm/i915/display/intel_cx0_phy.c | 6 ++++++
>> > > 1 file changed, 6 insertions(+)
>> > >
>> > > diff --git a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
>> > > b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
>> > > index abd607b564f1..a71b8a29d6b0 100644
>> > > --- a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
>> > > +++ b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
>> > > @@ -220,9 +220,12 @@ static u8 __intel_cx0_read(struct drm_i915_private *i915, enum port port,
>> > > /* 3 tries is assumed to be enough to read successfully */
>> > > for (i = 0; i < 3; i++) {
>> > > status = __intel_cx0_read_once(i915, port, lane, addr);
>> > > + intel_cx0_bus_reset(i915, port, lane);
>> > >
>> > > if (status >= 0)
>> > > return status;
>> > > +
>> > > + usleep_range(200, 300);
>> > > }
>> > >
>> > > drm_err_once(&i915->drm, "PHY %c Read %04x failed after %d
>> > > retries.\n", @@ -299,9 +302,12 @@ static void __intel_cx0_write(struct drm_i915_private *i915, enum port port,
>> > > /* 3 tries is assumed to be enough to write successfully */
>> > > for (i = 0; i < 3; i++) {
>> > > status = __intel_cx0_write_once(i915, port, lane, addr, data,
>> > > committed);
>> > > + intel_cx0_bus_reset(i915, port, lane);
>> > >
>> > > if (status == 0)
>> > > return;
>> > > +
>> > > + usleep_range(200, 300);
>> > > }
>> > >
>> > > drm_err_once(&i915->drm,
>> > > --
>> > > 2.34.1
>> > >
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation
2023-10-05 15:40 ` Gustavo Sousa
@ 2023-10-05 16:10 ` Rodrigo Vivi
2023-10-06 6:49 ` Kahola, Mika
0 siblings, 1 reply; 12+ messages in thread
From: Rodrigo Vivi @ 2023-10-05 16:10 UTC (permalink / raw)
To: Gustavo Sousa; +Cc: intel-gfx
On Thu, Oct 05, 2023 at 12:40:35PM -0300, Gustavo Sousa wrote:
> Quoting Rodrigo Vivi (2023-10-05 12:13:34-03:00)
> >On Thu, Oct 05, 2023 at 03:05:31AM -0400, Kahola, Mika wrote:
> >> > -----Original Message-----
> >> > From: Vivi, Rodrigo <rodrigo.vivi@intel.com>
> >> > Sent: Wednesday, October 4, 2023 3:56 PM
> >> > To: Kahola, Mika <mika.kahola@intel.com>
> >> > Cc: intel-gfx@lists.freedesktop.org
> >> > Subject: Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation
> >> >
> >> > On Wed, Oct 04, 2023 at 01:25:04PM +0300, Mika Kahola wrote:
> >> > > Every know and then we receive the following error when running for
> >> > > example IGT test kms_flip.
> >> > >
> >> > > [drm] *ERROR* PHY G Read 0d80 failed after 3 retries.
> >> > > [drm] *ERROR* PHY G Write 0d81 failed after 3 retries.
> >> > >
> >> > > Since the error is sporadic in nature, the patch proposes to reset the
> >> > > message bus after every successful or unsuccessful read or write
> >> > > operation. However, testing revealed that this alone is not sufficient
> >> > > method an additiona delay is also introduces anything from 200us to
> >> > > 300us. This delay is experimental value and has no specification to
> >> > > back it up.
> >> >
> >> > have you tried the delays without the bus_reset?
> >> Yes, we have bumped up the delay, first from 0x100 to 0x200 and then as per
> >> BSpec change 0xa000 and I have tried 0xf000. Increasing the timeout reduces
> >> the frequency of this error but doesn't solve this issue.
> >
> >what is exactly this BSPec's 0xa000? where can I see it? So maybe you can
> >update the message above removing the 'no specification to back it up'.
>
> (Resending this because I got a delivery failure notification)
>
> I think we are confusing "delay" with the "timeout parameter" of the msgbus.
>
> The PHY has a register to control the timeout parameter of msgbus transactions
> (BSpec 65156). It's default value is 0x100. With commit e028d7a4235d
> ("drm/i915/cx0: Check and increase msgbus timeout threshold"), we had integrated
> a workaround that bumped the timeout value to 0x200 in case timeouts were
> observed. Later on, there was a BSpec update with the formal timeout value to be
> programmed to 0xa000, which was incorporated with commit e35628968032
> ("drm/i915/cx0: Add step for programming msgbus timer").
>
> I *believe* what Rodrigo has asked was about the usleep_range() calls added with
> this patch, if we tried to only keep the usleed_range() without the bus reset.
yes, that was my original question.
>
> --
> Gustavo Sousa
>
> >
> >Oh, and my english is bad, but it looks to me that 'empirical' might
> >sound better than 'experimental' for this case, since you really did
> >a lot of experiments before coming to this final conclusion.
> >
> >>
> >> > have you talked to hw architects about this?
> >> Yes, HW guys requested traces which I provided but based on these the sequence we use in i915
> >> is correct.
> >>
> >> >
> >> > I wonder if we should add the delay inside the bus_reset itself?
> >> > although the bit 15 clear check should be enough by itself and it doesn't look like it is a hw/fw reset involved to justify the extra
> >> > delay.
> >> That should be enough. To me, it looks like when reading/writing to the bus maybe too fast, the hw cannot handle that and we need
> >> to reset and let things settle down before trying again.
> >>
> >> >
> >> > well, at least some /* FIXME: */ or /* XXX: */ comments is desired along with the messages if we are going with this hack without
> >> > understanding why...
> >> True, I will add these the the patch.
> >>
> >> Thanks for review!
> >>
> >> -Mika-
> >> >
> >> > >
> >> > > Signed-off-by: Mika Kahola <mika.kahola@intel.com>
> >> > > ---
> >> > > drivers/gpu/drm/i915/display/intel_cx0_phy.c | 6 ++++++
> >> > > 1 file changed, 6 insertions(+)
> >> > >
> >> > > diff --git a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> >> > > b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> >> > > index abd607b564f1..a71b8a29d6b0 100644
> >> > > --- a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> >> > > +++ b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> >> > > @@ -220,9 +220,12 @@ static u8 __intel_cx0_read(struct drm_i915_private *i915, enum port port,
> >> > > /* 3 tries is assumed to be enough to read successfully */
> >> > > for (i = 0; i < 3; i++) {
> >> > > status = __intel_cx0_read_once(i915, port, lane, addr);
> >> > > + intel_cx0_bus_reset(i915, port, lane);
> >> > >
> >> > > if (status >= 0)
> >> > > return status;
> >> > > +
> >> > > + usleep_range(200, 300);
> >> > > }
> >> > >
> >> > > drm_err_once(&i915->drm, "PHY %c Read %04x failed after %d
> >> > > retries.\n", @@ -299,9 +302,12 @@ static void __intel_cx0_write(struct drm_i915_private *i915, enum port port,
> >> > > /* 3 tries is assumed to be enough to write successfully */
> >> > > for (i = 0; i < 3; i++) {
> >> > > status = __intel_cx0_write_once(i915, port, lane, addr, data,
> >> > > committed);
> >> > > + intel_cx0_bus_reset(i915, port, lane);
> >> > >
> >> > > if (status == 0)
> >> > > return;
> >> > > +
> >> > > + usleep_range(200, 300);
> >> > > }
> >> > >
> >> > > drm_err_once(&i915->drm,
> >> > > --
> >> > > 2.34.1
> >> > >
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation
2023-10-05 16:10 ` Rodrigo Vivi
@ 2023-10-06 6:49 ` Kahola, Mika
2023-10-06 11:57 ` Gustavo Sousa
0 siblings, 1 reply; 12+ messages in thread
From: Kahola, Mika @ 2023-10-06 6:49 UTC (permalink / raw)
To: Vivi, Rodrigo, Sousa, Gustavo; +Cc: intel-gfx@lists.freedesktop.org
> -----Original Message-----
> From: Vivi, Rodrigo <rodrigo.vivi@intel.com>
> Sent: Thursday, October 5, 2023 7:10 PM
> To: Sousa, Gustavo <gustavo.sousa@intel.com>
> Cc: Kahola, Mika <mika.kahola@intel.com>; intel-gfx@lists.freedesktop.org
> Subject: Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation
>
> On Thu, Oct 05, 2023 at 12:40:35PM -0300, Gustavo Sousa wrote:
> > Quoting Rodrigo Vivi (2023-10-05 12:13:34-03:00)
> > >On Thu, Oct 05, 2023 at 03:05:31AM -0400, Kahola, Mika wrote:
> > >> > -----Original Message-----
> > >> > From: Vivi, Rodrigo <rodrigo.vivi@intel.com>
> > >> > Sent: Wednesday, October 4, 2023 3:56 PM
> > >> > To: Kahola, Mika <mika.kahola@intel.com>
> > >> > Cc: intel-gfx@lists.freedesktop.org
> > >> > Subject: Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message
> > >> > bus after each read/write operation
> > >> >
> > >> > On Wed, Oct 04, 2023 at 01:25:04PM +0300, Mika Kahola wrote:
> > >> > > Every know and then we receive the following error when running
> > >> > > for example IGT test kms_flip.
> > >> > >
> > >> > > [drm] *ERROR* PHY G Read 0d80 failed after 3 retries.
> > >> > > [drm] *ERROR* PHY G Write 0d81 failed after 3 retries.
> > >> > >
> > >> > > Since the error is sporadic in nature, the patch proposes to
> > >> > > reset the message bus after every successful or unsuccessful
> > >> > > read or write operation. However, testing revealed that this
> > >> > > alone is not sufficient method an additiona delay is also
> > >> > > introduces anything from 200us to 300us. This delay is
> > >> > > experimental value and has no specification to back it up.
> > >> >
> > >> > have you tried the delays without the bus_reset?
> > >> Yes, we have bumped up the delay, first from 0x100 to 0x200 and
> > >> then as per BSpec change 0xa000 and I have tried 0xf000. Increasing
> > >> the timeout reduces the frequency of this error but doesn't solve this issue.
> > >
> > >what is exactly this BSPec's 0xa000? where can I see it? So maybe you
> > >can update the message above removing the 'no specification to back it up'.
> >
> > (Resending this because I got a delivery failure notification)
> >
> > I think we are confusing "delay" with the "timeout parameter" of the msgbus.
> >
> > The PHY has a register to control the timeout parameter of msgbus
> > transactions (BSpec 65156). It's default value is 0x100. With commit
> > e028d7a4235d
> > ("drm/i915/cx0: Check and increase msgbus timeout threshold"), we had
> > integrated a workaround that bumped the timeout value to 0x200 in case
> > timeouts were observed. Later on, there was a BSpec update with the
> > formal timeout value to be programmed to 0xa000, which was
> > incorporated with commit e35628968032
> > ("drm/i915/cx0: Add step for programming msgbus timer").
> >
> > I *believe* what Rodrigo has asked was about the usleep_range() calls
> > added with this patch, if we tried to only keep the usleed_range() without the bus reset.
>
> yes, that was my original question.
I have no good explanation why usleep_range() is needed. Without it, the kms_flip test eventually
throws these read/write failures. As these are a bit sporadic in nature, it takes some time to catch
these errors.
The patch is a hack and my idea was to set message bus at reset state after each read/write operation.
Unfortunately, this alone is not enough to pass kms_flip without these dmesg errors on read/write.
However, the kms_flip test itself, which triggers these, passes without issues.
And I missed to mention that these errors show up (at least more frequently) when 2x 4k monitors are
connected. These may not be visible with only one monitor connected. For such a system, I haven't
been testing that much.
-Mika-
>
> >
> > --
> > Gustavo Sousa
> >
> > >
> > >Oh, and my english is bad, but it looks to me that 'empirical' might
> > >sound better than 'experimental' for this case, since you really did
> > >a lot of experiments before coming to this final conclusion.
> > >
> > >>
> > >> > have you talked to hw architects about this?
> > >> Yes, HW guys requested traces which I provided but based on these
> > >> the sequence we use in i915 is correct.
> > >>
> > >> >
> > >> > I wonder if we should add the delay inside the bus_reset itself?
> > >> > although the bit 15 clear check should be enough by itself and it
> > >> > doesn't look like it is a hw/fw reset involved to justify the extra delay.
> > >> That should be enough. To me, it looks like when reading/writing to
> > >> the bus maybe too fast, the hw cannot handle that and we need to reset and let things settle down before trying again.
> > >>
> > >> >
> > >> > well, at least some /* FIXME: */ or /* XXX: */ comments is
> > >> > desired along with the messages if we are going with this hack without understanding why...
> > >> True, I will add these the the patch.
> > >>
> > >> Thanks for review!
> > >>
> > >> -Mika-
> > >> >
> > >> > >
> > >> > > Signed-off-by: Mika Kahola <mika.kahola@intel.com>
> > >> > > ---
> > >> > > drivers/gpu/drm/i915/display/intel_cx0_phy.c | 6 ++++++
> > >> > > 1 file changed, 6 insertions(+)
> > >> > >
> > >> > > diff --git a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> > >> > > b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> > >> > > index abd607b564f1..a71b8a29d6b0 100644
> > >> > > --- a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> > >> > > +++ b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> > >> > > @@ -220,9 +220,12 @@ static u8 __intel_cx0_read(struct drm_i915_private *i915, enum port port,
> > >> > > /* 3 tries is assumed to be enough to read successfully */
> > >> > > for (i = 0; i < 3; i++) {
> > >> > > status = __intel_cx0_read_once(i915, port,
> > >> > > lane, addr);
> > >> > > + intel_cx0_bus_reset(i915, port, lane);
> > >> > >
> > >> > > if (status >= 0)
> > >> > > return status;
> > >> > > +
> > >> > > + usleep_range(200, 300);
> > >> > > }
> > >> > >
> > >> > > drm_err_once(&i915->drm, "PHY %c Read %04x failed
> > >> > > after %d retries.\n", @@ -299,9 +302,12 @@ static void __intel_cx0_write(struct drm_i915_private *i915, enum port
> port,
> > >> > > /* 3 tries is assumed to be enough to write successfully */
> > >> > > for (i = 0; i < 3; i++) {
> > >> > > status = __intel_cx0_write_once(i915, port,
> > >> > > lane, addr, data, committed);
> > >> > > + intel_cx0_bus_reset(i915, port, lane);
> > >> > >
> > >> > > if (status == 0)
> > >> > > return;
> > >> > > +
> > >> > > + usleep_range(200, 300);
> > >> > > }
> > >> > >
> > >> > > drm_err_once(&i915->drm,
> > >> > > --
> > >> > > 2.34.1
> > >> > >
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation
2023-10-06 6:49 ` Kahola, Mika
@ 2023-10-06 11:57 ` Gustavo Sousa
2023-10-06 12:31 ` Kahola, Mika
0 siblings, 1 reply; 12+ messages in thread
From: Gustavo Sousa @ 2023-10-06 11:57 UTC (permalink / raw)
To: Kahola, Mika, Vivi, Rodrigo; +Cc: intel-gfx
Quoting Kahola, Mika (2023-10-06 03:49:15-03:00)
>> -----Original Message-----
>> From: Vivi, Rodrigo <rodrigo.vivi@intel.com>
>> Sent: Thursday, October 5, 2023 7:10 PM
>> To: Sousa, Gustavo <gustavo.sousa@intel.com>
>> Cc: Kahola, Mika <mika.kahola@intel.com>; intel-gfx@lists.freedesktop.org
>> Subject: Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation
>>
>> On Thu, Oct 05, 2023 at 12:40:35PM -0300, Gustavo Sousa wrote:
>> > Quoting Rodrigo Vivi (2023-10-05 12:13:34-03:00)
>> > >On Thu, Oct 05, 2023 at 03:05:31AM -0400, Kahola, Mika wrote:
>> > >> > -----Original Message-----
>> > >> > From: Vivi, Rodrigo <rodrigo.vivi@intel.com>
>> > >> > Sent: Wednesday, October 4, 2023 3:56 PM
>> > >> > To: Kahola, Mika <mika.kahola@intel.com>
>> > >> > Cc: intel-gfx@lists.freedesktop.org
>> > >> > Subject: Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message
>> > >> > bus after each read/write operation
>> > >> >
>> > >> > On Wed, Oct 04, 2023 at 01:25:04PM +0300, Mika Kahola wrote:
>> > >> > > Every know and then we receive the following error when running
>> > >> > > for example IGT test kms_flip.
>> > >> > >
>> > >> > > [drm] *ERROR* PHY G Read 0d80 failed after 3 retries.
>> > >> > > [drm] *ERROR* PHY G Write 0d81 failed after 3 retries.
>> > >> > >
>> > >> > > Since the error is sporadic in nature, the patch proposes to
>> > >> > > reset the message bus after every successful or unsuccessful
>> > >> > > read or write operation. However, testing revealed that this
>> > >> > > alone is not sufficient method an additiona delay is also
>> > >> > > introduces anything from 200us to 300us. This delay is
>> > >> > > experimental value and has no specification to back it up.
>> > >> >
>> > >> > have you tried the delays without the bus_reset?
>> > >> Yes, we have bumped up the delay, first from 0x100 to 0x200 and
>> > >> then as per BSpec change 0xa000 and I have tried 0xf000. Increasing
>> > >> the timeout reduces the frequency of this error but doesn't solve this issue.
>> > >
>> > >what is exactly this BSPec's 0xa000? where can I see it? So maybe you
>> > >can update the message above removing the 'no specification to back it up'.
>> >
>> > (Resending this because I got a delivery failure notification)
>> >
>> > I think we are confusing "delay" with the "timeout parameter" of the msgbus.
>> >
>> > The PHY has a register to control the timeout parameter of msgbus
>> > transactions (BSpec 65156). It's default value is 0x100. With commit
>> > e028d7a4235d
>> > ("drm/i915/cx0: Check and increase msgbus timeout threshold"), we had
>> > integrated a workaround that bumped the timeout value to 0x200 in case
>> > timeouts were observed. Later on, there was a BSpec update with the
>> > formal timeout value to be programmed to 0xa000, which was
>> > incorporated with commit e35628968032
>> > ("drm/i915/cx0: Add step for programming msgbus timer").
>> >
>> > I *believe* what Rodrigo has asked was about the usleep_range() calls
>> > added with this patch, if we tried to only keep the usleed_range() without the bus reset.
>>
>> yes, that was my original question.
>
>I have no good explanation why usleep_range() is needed. Without it, the kms_flip test eventually
>throws these read/write failures. As these are a bit sporadic in nature, it takes some time to catch
>these errors.
I think the question is whether the bus reset is really necessary. Maybe only
the usleep_range() hack would be "enough" to mitigate the issue?
--
Gustavo Sousa
>
>The patch is a hack and my idea was to set message bus at reset state after each read/write operation.
>Unfortunately, this alone is not enough to pass kms_flip without these dmesg errors on read/write.
>However, the kms_flip test itself, which triggers these, passes without issues.
>
>And I missed to mention that these errors show up (at least more frequently) when 2x 4k monitors are
>connected. These may not be visible with only one monitor connected. For such a system, I haven't
>been testing that much.
>
>-Mika-
>
>>
>> >
>> > --
>> > Gustavo Sousa
>> >
>> > >
>> > >Oh, and my english is bad, but it looks to me that 'empirical' might
>> > >sound better than 'experimental' for this case, since you really did
>> > >a lot of experiments before coming to this final conclusion.
>> > >
>> > >>
>> > >> > have you talked to hw architects about this?
>> > >> Yes, HW guys requested traces which I provided but based on these
>> > >> the sequence we use in i915 is correct.
>> > >>
>> > >> >
>> > >> > I wonder if we should add the delay inside the bus_reset itself?
>> > >> > although the bit 15 clear check should be enough by itself and it
>> > >> > doesn't look like it is a hw/fw reset involved to justify the extra delay.
>> > >> That should be enough. To me, it looks like when reading/writing to
>> > >> the bus maybe too fast, the hw cannot handle that and we need to reset and let things settle down before trying again.
>> > >>
>> > >> >
>> > >> > well, at least some /* FIXME: */ or /* XXX: */ comments is
>> > >> > desired along with the messages if we are going with this hack without understanding why...
>> > >> True, I will add these the the patch.
>> > >>
>> > >> Thanks for review!
>> > >>
>> > >> -Mika-
>> > >> >
>> > >> > >
>> > >> > > Signed-off-by: Mika Kahola <mika.kahola@intel.com>
>> > >> > > ---
>> > >> > > drivers/gpu/drm/i915/display/intel_cx0_phy.c | 6 ++++++
>> > >> > > 1 file changed, 6 insertions(+)
>> > >> > >
>> > >> > > diff --git a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
>> > >> > > b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
>> > >> > > index abd607b564f1..a71b8a29d6b0 100644
>> > >> > > --- a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
>> > >> > > +++ b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
>> > >> > > @@ -220,9 +220,12 @@ static u8 __intel_cx0_read(struct drm_i915_private *i915, enum port port,
>> > >> > > /* 3 tries is assumed to be enough to read successfully */
>> > >> > > for (i = 0; i < 3; i++) {
>> > >> > > status = __intel_cx0_read_once(i915, port,
>> > >> > > lane, addr);
>> > >> > > + intel_cx0_bus_reset(i915, port, lane);
>> > >> > >
>> > >> > > if (status >= 0)
>> > >> > > return status;
>> > >> > > +
>> > >> > > + usleep_range(200, 300);
>> > >> > > }
>> > >> > >
>> > >> > > drm_err_once(&i915->drm, "PHY %c Read %04x failed
>> > >> > > after %d retries.\n", @@ -299,9 +302,12 @@ static void __intel_cx0_write(struct drm_i915_private *i915, enum port
>> port,
>> > >> > > /* 3 tries is assumed to be enough to write successfully */
>> > >> > > for (i = 0; i < 3; i++) {
>> > >> > > status = __intel_cx0_write_once(i915, port,
>> > >> > > lane, addr, data, committed);
>> > >> > > + intel_cx0_bus_reset(i915, port, lane);
>> > >> > >
>> > >> > > if (status == 0)
>> > >> > > return;
>> > >> > > +
>> > >> > > + usleep_range(200, 300);
>> > >> > > }
>> > >> > >
>> > >> > > drm_err_once(&i915->drm,
>> > >> > > --
>> > >> > > 2.34.1
>> > >> > >
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation
2023-10-06 11:57 ` Gustavo Sousa
@ 2023-10-06 12:31 ` Kahola, Mika
0 siblings, 0 replies; 12+ messages in thread
From: Kahola, Mika @ 2023-10-06 12:31 UTC (permalink / raw)
To: Sousa, Gustavo, Vivi, Rodrigo; +Cc: intel-gfx@lists.freedesktop.org
> -----Original Message-----
> From: Sousa, Gustavo <gustavo.sousa@intel.com>
> Sent: Friday, October 6, 2023 2:57 PM
> To: Kahola, Mika <mika.kahola@intel.com>; Vivi, Rodrigo <rodrigo.vivi@intel.com>
> Cc: intel-gfx@lists.freedesktop.org
> Subject: RE: [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation
>
> Quoting Kahola, Mika (2023-10-06 03:49:15-03:00)
> >> -----Original Message-----
> >> From: Vivi, Rodrigo <rodrigo.vivi@intel.com>
> >> Sent: Thursday, October 5, 2023 7:10 PM
> >> To: Sousa, Gustavo <gustavo.sousa@intel.com>
> >> Cc: Kahola, Mika <mika.kahola@intel.com>;
> >> intel-gfx@lists.freedesktop.org
> >> Subject: Re: [Intel-gfx] [PATCH] drm/i915/display: Reset message bus
> >> after each read/write operation
> >>
> >> On Thu, Oct 05, 2023 at 12:40:35PM -0300, Gustavo Sousa wrote:
> >> > Quoting Rodrigo Vivi (2023-10-05 12:13:34-03:00)
> >> > >On Thu, Oct 05, 2023 at 03:05:31AM -0400, Kahola, Mika wrote:
> >> > >> > -----Original Message-----
> >> > >> > From: Vivi, Rodrigo <rodrigo.vivi@intel.com>
> >> > >> > Sent: Wednesday, October 4, 2023 3:56 PM
> >> > >> > To: Kahola, Mika <mika.kahola@intel.com>
> >> > >> > Cc: intel-gfx@lists.freedesktop.org
> >> > >> > Subject: Re: [Intel-gfx] [PATCH] drm/i915/display: Reset
> >> > >> > message bus after each read/write operation
> >> > >> >
> >> > >> > On Wed, Oct 04, 2023 at 01:25:04PM +0300, Mika Kahola wrote:
> >> > >> > > Every know and then we receive the following error when
> >> > >> > > running for example IGT test kms_flip.
> >> > >> > >
> >> > >> > > [drm] *ERROR* PHY G Read 0d80 failed after 3 retries.
> >> > >> > > [drm] *ERROR* PHY G Write 0d81 failed after 3 retries.
> >> > >> > >
> >> > >> > > Since the error is sporadic in nature, the patch proposes to
> >> > >> > > reset the message bus after every successful or unsuccessful
> >> > >> > > read or write operation. However, testing revealed that this
> >> > >> > > alone is not sufficient method an additiona delay is also
> >> > >> > > introduces anything from 200us to 300us. This delay is
> >> > >> > > experimental value and has no specification to back it up.
> >> > >> >
> >> > >> > have you tried the delays without the bus_reset?
> >> > >> Yes, we have bumped up the delay, first from 0x100 to 0x200 and
> >> > >> then as per BSpec change 0xa000 and I have tried 0xf000.
> >> > >> Increasing the timeout reduces the frequency of this error but doesn't solve this issue.
> >> > >
> >> > >what is exactly this BSPec's 0xa000? where can I see it? So maybe
> >> > >you can update the message above removing the 'no specification to back it up'.
> >> >
> >> > (Resending this because I got a delivery failure notification)
> >> >
> >> > I think we are confusing "delay" with the "timeout parameter" of the msgbus.
> >> >
> >> > The PHY has a register to control the timeout parameter of msgbus
> >> > transactions (BSpec 65156). It's default value is 0x100. With
> >> > commit e028d7a4235d
> >> > ("drm/i915/cx0: Check and increase msgbus timeout threshold"), we
> >> > had integrated a workaround that bumped the timeout value to 0x200
> >> > in case timeouts were observed. Later on, there was a BSpec update
> >> > with the formal timeout value to be programmed to 0xa000, which was
> >> > incorporated with commit e35628968032
> >> > ("drm/i915/cx0: Add step for programming msgbus timer").
> >> >
> >> > I *believe* what Rodrigo has asked was about the usleep_range()
> >> > calls added with this patch, if we tried to only keep the usleed_range() without the bus reset.
> >>
> >> yes, that was my original question.
> >
> >I have no good explanation why usleep_range() is needed. Without it,
> >the kms_flip test eventually throws these read/write failures. As these
> >are a bit sporadic in nature, it takes some time to catch these errors.
>
> I think the question is whether the bus reset is really necessary. Maybe only the usleep_range() hack would be "enough" to
> mitigate the issue?
I have been scratching my head with this. I tested without reset and left only the delay i.e. usleep_range() in place but still I had these read/write failures. The same thing vice versa. It's like we would need both of them. I would like to understand why or is there something else that we are missing from our sequence?
>
> --
> Gustavo Sousa
>
> >
> >The patch is a hack and my idea was to set message bus at reset state after each read/write operation.
> >Unfortunately, this alone is not enough to pass kms_flip without these dmesg errors on read/write.
> >However, the kms_flip test itself, which triggers these, passes without issues.
> >
> >And I missed to mention that these errors show up (at least more
> >frequently) when 2x 4k monitors are connected. These may not be visible
> >with only one monitor connected. For such a system, I haven't been testing that much.
> >
> >-Mika-
> >
> >>
> >> >
> >> > --
> >> > Gustavo Sousa
> >> >
> >> > >
> >> > >Oh, and my english is bad, but it looks to me that 'empirical'
> >> > >might sound better than 'experimental' for this case, since you
> >> > >really did a lot of experiments before coming to this final conclusion.
> >> > >
> >> > >>
> >> > >> > have you talked to hw architects about this?
> >> > >> Yes, HW guys requested traces which I provided but based on
> >> > >> these the sequence we use in i915 is correct.
> >> > >>
> >> > >> >
> >> > >> > I wonder if we should add the delay inside the bus_reset itself?
> >> > >> > although the bit 15 clear check should be enough by itself and
> >> > >> > it doesn't look like it is a hw/fw reset involved to justify the extra delay.
> >> > >> That should be enough. To me, it looks like when reading/writing
> >> > >> to the bus maybe too fast, the hw cannot handle that and we need to reset and let things settle down before trying again.
> >> > >>
> >> > >> >
> >> > >> > well, at least some /* FIXME: */ or /* XXX: */ comments is
> >> > >> > desired along with the messages if we are going with this hack without understanding why...
> >> > >> True, I will add these the the patch.
> >> > >>
> >> > >> Thanks for review!
> >> > >>
> >> > >> -Mika-
> >> > >> >
> >> > >> > >
> >> > >> > > Signed-off-by: Mika Kahola <mika.kahola@intel.com>
> >> > >> > > ---
> >> > >> > > drivers/gpu/drm/i915/display/intel_cx0_phy.c | 6 ++++++
> >> > >> > > 1 file changed, 6 insertions(+)
> >> > >> > >
> >> > >> > > diff --git a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> >> > >> > > b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> >> > >> > > index abd607b564f1..a71b8a29d6b0 100644
> >> > >> > > --- a/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> >> > >> > > +++ b/drivers/gpu/drm/i915/display/intel_cx0_phy.c
> >> > >> > > @@ -220,9 +220,12 @@ static u8 __intel_cx0_read(struct drm_i915_private *i915, enum port port,
> >> > >> > > /* 3 tries is assumed to be enough to read successfully */
> >> > >> > > for (i = 0; i < 3; i++) {
> >> > >> > > status = __intel_cx0_read_once(i915, port,
> >> > >> > > lane, addr);
> >> > >> > > + intel_cx0_bus_reset(i915, port, lane);
> >> > >> > >
> >> > >> > > if (status >= 0)
> >> > >> > > return status;
> >> > >> > > +
> >> > >> > > + usleep_range(200, 300);
> >> > >> > > }
> >> > >> > >
> >> > >> > > drm_err_once(&i915->drm, "PHY %c Read %04x failed
> >> > >> > > after %d retries.\n", @@ -299,9 +302,12 @@ static void
> >> > >> > > __intel_cx0_write(struct drm_i915_private *i915, enum port
> >> port,
> >> > >> > > /* 3 tries is assumed to be enough to write successfully */
> >> > >> > > for (i = 0; i < 3; i++) {
> >> > >> > > status = __intel_cx0_write_once(i915, port,
> >> > >> > > lane, addr, data, committed);
> >> > >> > > + intel_cx0_bus_reset(i915, port, lane);
> >> > >> > >
> >> > >> > > if (status == 0)
> >> > >> > > return;
> >> > >> > > +
> >> > >> > > + usleep_range(200, 300);
> >> > >> > > }
> >> > >> > >
> >> > >> > > drm_err_once(&i915->drm,
> >> > >> > > --
> >> > >> > > 2.34.1
> >> > >> > >
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2023-10-09 13:20 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-10-04 10:25 [Intel-gfx] [PATCH] drm/i915/display: Reset message bus after each read/write operation Mika Kahola
2023-10-04 12:47 ` [Intel-gfx] ✓ Fi.CI.BAT: success for " Patchwork
2023-10-04 12:55 ` [Intel-gfx] [PATCH] " Rodrigo Vivi
2023-10-05 7:05 ` Kahola, Mika
2023-10-05 15:13 ` Rodrigo Vivi
2023-10-05 15:32 ` Gustavo Sousa
2023-10-05 15:40 ` Gustavo Sousa
2023-10-05 16:10 ` Rodrigo Vivi
2023-10-06 6:49 ` Kahola, Mika
2023-10-06 11:57 ` Gustavo Sousa
2023-10-06 12:31 ` Kahola, Mika
2023-10-04 17:46 ` [Intel-gfx] ✗ Fi.CI.IGT: failure for " Patchwork
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox