* [RFC PATCH] i915/gt: Reapply workarounds in case the previous attempt failed.
@ 2024-12-05 15:47 Sebastian Brzezinka
2024-12-05 21:45 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
` (5 more replies)
0 siblings, 6 replies; 10+ messages in thread
From: Sebastian Brzezinka @ 2024-12-05 15:47 UTC (permalink / raw)
To: intel-gfx; +Cc: andi.shyti, vidya.srinivas
`wa_verify`sporadically detects lost workaround on application; this
is unusual behavior since wa are applied at `intel_gt_init_hw` and
verified right away by `intel_gt_verify_workarounds`, and `wa_verify`
doesn't fail on initialization as one might suspect would happen.
One approach that may be somewhat beneficial is to reapply workarounds
in the event of failure, or even get rid of verify on application,
since it's redundant to `intel_gt_verify_workarounds`.
This patch aims to resolve: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12668
Signed-off-by: Sebastian Brzezinka <sebastian.brzezinka@intel.com>
---
drivers/gpu/drm/i915/gt/intel_workarounds.c | 11 +++++++++++
1 file changed, 11 insertions(+)
diff --git a/drivers/gpu/drm/i915/gt/intel_workarounds.c b/drivers/gpu/drm/i915/gt/intel_workarounds.c
index 570c91878189..4ee623448223 100644
--- a/drivers/gpu/drm/i915/gt/intel_workarounds.c
+++ b/drivers/gpu/drm/i915/gt/intel_workarounds.c
@@ -1761,6 +1761,17 @@ static void wa_list_apply(const struct i915_wa_list *wal)
intel_gt_mcr_read_any_fw(gt, wa->mcr_reg) :
intel_uncore_read_fw(uncore, wa->reg);
+ if ((val ^ wa->set) & wa->read) {
+ if (wa->is_mcr)
+ intel_gt_mcr_multicast_write_fw(gt, wa->mcr_reg, val);
+ else
+ intel_uncore_write_fw(uncore, wa->reg, val);
+ }
+
+ val = wa->is_mcr ?
+ intel_gt_mcr_read_any_fw(gt, wa->mcr_reg) :
+ intel_uncore_read_fw(uncore, wa->reg);
+
wa_verify(gt, wa, val, wal->name, "application");
}
}
--
2.34.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* ✗ Fi.CI.CHECKPATCH: warning for i915/gt: Reapply workarounds in case the previous attempt failed.
2024-12-05 15:47 [RFC PATCH] i915/gt: Reapply workarounds in case the previous attempt failed Sebastian Brzezinka
@ 2024-12-05 21:45 ` Patchwork
2024-12-05 22:04 ` ✗ i915.CI.BAT: failure " Patchwork
` (4 subsequent siblings)
5 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2024-12-05 21:45 UTC (permalink / raw)
To: Sebastian Brzezinka; +Cc: intel-gfx
== Series Details ==
Series: i915/gt: Reapply workarounds in case the previous attempt failed.
URL : https://patchwork.freedesktop.org/series/142188/
State : warning
== Summary ==
Error: dim checkpatch failed
254bcd8fe90b i915/gt: Reapply workarounds in case the previous attempt failed.
-:19: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#19:
This patch aims to resolve: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12668
-:31: ERROR:TRAILING_WHITESPACE: trailing whitespace
#31: FILE: drivers/gpu/drm/i915/gt/intel_workarounds.c:1764:
+^I^I^Iif ((val ^ wa->set) & wa->read) { $
total: 1 errors, 1 warnings, 0 checks, 17 lines checked
^ permalink raw reply [flat|nested] 10+ messages in thread
* ✗ i915.CI.BAT: failure for i915/gt: Reapply workarounds in case the previous attempt failed.
2024-12-05 15:47 [RFC PATCH] i915/gt: Reapply workarounds in case the previous attempt failed Sebastian Brzezinka
2024-12-05 21:45 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
@ 2024-12-05 22:04 ` Patchwork
2024-12-06 12:36 ` ✗ Fi.CI.CHECKPATCH: warning for i915/gt: Reapply workarounds in case the previous attempt failed. (rev2) Patchwork
` (3 subsequent siblings)
5 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2024-12-05 22:04 UTC (permalink / raw)
To: Sebastian Brzezinka; +Cc: intel-gfx
== Series Details ==
Series: i915/gt: Reapply workarounds in case the previous attempt failed.
URL : https://patchwork.freedesktop.org/series/142188/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_15796 -> Patchwork_142188v1
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_142188v1 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_142188v1, please notify your bug team (I915-ci-infra@lists.freedesktop.org) to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v1/index.html
Participating hosts (44 -> 43)
------------------------------
Missing (1): fi-snb-2520m
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_142188v1:
### IGT changes ###
#### Possible regressions ####
* igt@i915_pm_rpm@module-reload:
- fi-rkl-11600: [PASS][1] -> [ABORT][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15796/fi-rkl-11600/igt@i915_pm_rpm@module-reload.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v1/fi-rkl-11600/igt@i915_pm_rpm@module-reload.html
Known issues
------------
Here are the changes found in Patchwork_142188v1 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_pm_rpm@module-reload:
- bat-adls-6: [PASS][3] -> [FAIL][4] ([i915#12903])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15796/bat-adls-6/igt@i915_pm_rpm@module-reload.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v1/bat-adls-6/igt@i915_pm_rpm@module-reload.html
- bat-rpls-4: [PASS][5] -> [FAIL][6] ([i915#12903])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15796/bat-rpls-4/igt@i915_pm_rpm@module-reload.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v1/bat-rpls-4/igt@i915_pm_rpm@module-reload.html
* igt@i915_selftest@live@workarounds:
- bat-arlh-3: [PASS][7] -> [ABORT][8] ([i915#12061]) +1 other test abort
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15796/bat-arlh-3/igt@i915_selftest@live@workarounds.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v1/bat-arlh-3/igt@i915_selftest@live@workarounds.html
#### Possible fixes ####
* igt@i915_module_load@reload:
- bat-jsl-3: [DMESG-WARN][9] ([i915#1982]) -> [PASS][10]
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15796/bat-jsl-3/igt@i915_module_load@reload.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v1/bat-jsl-3/igt@i915_module_load@reload.html
* igt@i915_pm_rpm@module-reload:
- bat-dg1-7: [FAIL][11] ([i915#12903]) -> [PASS][12]
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15796/bat-dg1-7/igt@i915_pm_rpm@module-reload.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v1/bat-dg1-7/igt@i915_pm_rpm@module-reload.html
* igt@i915_selftest@live:
- bat-mtlp-8: [ABORT][13] ([i915#12061]) -> [PASS][14] +1 other test pass
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15796/bat-mtlp-8/igt@i915_selftest@live.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v1/bat-mtlp-8/igt@i915_selftest@live.html
- bat-adlp-6: [ABORT][15] ([i915#9413]) -> [PASS][16] +1 other test pass
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15796/bat-adlp-6/igt@i915_selftest@live.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v1/bat-adlp-6/igt@i915_selftest@live.html
* igt@i915_selftest@live@workarounds:
- {bat-arls-6}: [ABORT][17] ([i915#12061]) -> [PASS][18] +1 other test pass
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15796/bat-arls-6/igt@i915_selftest@live@workarounds.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v1/bat-arls-6/igt@i915_selftest@live@workarounds.html
* igt@kms_chamelium_edid@hdmi-edid-read:
- bat-dg2-13: [DMESG-WARN][19] ([i915#12253]) -> [PASS][20]
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15796/bat-dg2-13/igt@kms_chamelium_edid@hdmi-edid-read.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v1/bat-dg2-13/igt@kms_chamelium_edid@hdmi-edid-read.html
* igt@kms_pipe_crc_basic@hang-read-crc:
- fi-cfl-8109u: [DMESG-WARN][21] ([i915#12914]) -> [PASS][22] +1 other test pass
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15796/fi-cfl-8109u/igt@kms_pipe_crc_basic@hang-read-crc.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v1/fi-cfl-8109u/igt@kms_pipe_crc_basic@hang-read-crc.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[i915#12061]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12061
[i915#12253]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12253
[i915#12903]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12903
[i915#12914]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12914
[i915#1982]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/1982
[i915#9413]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9413
Build changes
-------------
* Linux: CI_DRM_15796 -> Patchwork_142188v1
CI-20190529: 20190529
CI_DRM_15796: 1b7746f770882ce40dacae683e8e65657c40c2b7 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_8138: 8138
Patchwork_142188v1: 1b7746f770882ce40dacae683e8e65657c40c2b7 @ git://anongit.freedesktop.org/gfx-ci/linux
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v1/index.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* ✗ Fi.CI.CHECKPATCH: warning for i915/gt: Reapply workarounds in case the previous attempt failed. (rev2)
2024-12-05 15:47 [RFC PATCH] i915/gt: Reapply workarounds in case the previous attempt failed Sebastian Brzezinka
2024-12-05 21:45 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
2024-12-05 22:04 ` ✗ i915.CI.BAT: failure " Patchwork
@ 2024-12-06 12:36 ` Patchwork
2024-12-06 12:45 ` ✓ i915.CI.BAT: success " Patchwork
` (2 subsequent siblings)
5 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2024-12-06 12:36 UTC (permalink / raw)
To: Sebastian Brzezinka; +Cc: intel-gfx
== Series Details ==
Series: i915/gt: Reapply workarounds in case the previous attempt failed. (rev2)
URL : https://patchwork.freedesktop.org/series/142188/
State : warning
== Summary ==
Error: dim checkpatch failed
70c3e0476ce2 i915/gt: Reapply workarounds in case the previous attempt failed.
-:19: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#19:
This patch aims to resolve: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12668
-:31: ERROR:TRAILING_WHITESPACE: trailing whitespace
#31: FILE: drivers/gpu/drm/i915/gt/intel_workarounds.c:1764:
+^I^I^Iif ((val ^ wa->set) & wa->read) { $
total: 1 errors, 1 warnings, 0 checks, 17 lines checked
^ permalink raw reply [flat|nested] 10+ messages in thread
* ✓ i915.CI.BAT: success for i915/gt: Reapply workarounds in case the previous attempt failed. (rev2)
2024-12-05 15:47 [RFC PATCH] i915/gt: Reapply workarounds in case the previous attempt failed Sebastian Brzezinka
` (2 preceding siblings ...)
2024-12-06 12:36 ` ✗ Fi.CI.CHECKPATCH: warning for i915/gt: Reapply workarounds in case the previous attempt failed. (rev2) Patchwork
@ 2024-12-06 12:45 ` Patchwork
2024-12-06 14:03 ` ✓ i915.CI.Full: " Patchwork
2024-12-06 15:38 ` [RFC PATCH] i915/gt: Reapply workarounds in case the previous attempt failed Rodrigo Vivi
5 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2024-12-06 12:45 UTC (permalink / raw)
To: Sebastian Brzezinka; +Cc: intel-gfx
== Series Details ==
Series: i915/gt: Reapply workarounds in case the previous attempt failed. (rev2)
URL : https://patchwork.freedesktop.org/series/142188/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_15800 -> Patchwork_142188v2
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/index.html
Participating hosts (42 -> 41)
------------------------------
Missing (1): fi-snb-2520m
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_142188v2:
### IGT changes ###
#### Suppressed ####
The following results come from untrusted machines, tests, or statuses.
They do not affect the overall result.
* igt@prime_vgem@basic-fence-flip:
- {bat-mtlp-9}: NOTRUN -> [WARN][1]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/bat-mtlp-9/igt@prime_vgem@basic-fence-flip.html
Known issues
------------
Here are the changes found in Patchwork_142188v2 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@debugfs_test@basic-hwmon:
- bat-adlp-6: NOTRUN -> [SKIP][2] ([i915#9318])
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/bat-adlp-6/igt@debugfs_test@basic-hwmon.html
* igt@gem_lmem_swapping@random-engines:
- bat-adlp-6: NOTRUN -> [SKIP][3] ([i915#4613]) +3 other tests skip
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/bat-adlp-6/igt@gem_lmem_swapping@random-engines.html
* igt@gem_tiled_pread_basic:
- bat-adlp-6: NOTRUN -> [SKIP][4] ([i915#3282])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/bat-adlp-6/igt@gem_tiled_pread_basic.html
* igt@i915_pm_rps@basic-api:
- bat-adlp-6: NOTRUN -> [SKIP][5] ([i915#6621])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/bat-adlp-6/igt@i915_pm_rps@basic-api.html
* igt@i915_selftest@live:
- bat-mtlp-8: [PASS][6] -> [ABORT][7] ([i915#12061]) +1 other test abort
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/bat-mtlp-8/igt@i915_selftest@live.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/bat-mtlp-8/igt@i915_selftest@live.html
* igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy:
- bat-adlp-6: NOTRUN -> [SKIP][8] ([i915#4103]) +1 other test skip
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/bat-adlp-6/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html
* igt@kms_dsc@dsc-basic:
- bat-adlp-6: NOTRUN -> [SKIP][9] ([i915#3555] / [i915#3840])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/bat-adlp-6/igt@kms_dsc@dsc-basic.html
* igt@kms_force_connector_basic@force-load-detect:
- bat-adlp-6: NOTRUN -> [SKIP][10]
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/bat-adlp-6/igt@kms_force_connector_basic@force-load-detect.html
* igt@kms_pipe_crc_basic@read-crc-frame-sequence:
- bat-dg2-11: [PASS][11] -> [SKIP][12] ([i915#9197])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/bat-dg2-11/igt@kms_pipe_crc_basic@read-crc-frame-sequence.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/bat-dg2-11/igt@kms_pipe_crc_basic@read-crc-frame-sequence.html
* igt@kms_setmode@basic-clone-single-crtc:
- bat-adlp-6: NOTRUN -> [SKIP][13] ([i915#3555])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/bat-adlp-6/igt@kms_setmode@basic-clone-single-crtc.html
* igt@prime_vgem@basic-write:
- bat-adlp-6: NOTRUN -> [SKIP][14] ([i915#3291] / [i915#3708]) +2 other tests skip
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/bat-adlp-6/igt@prime_vgem@basic-write.html
#### Possible fixes ####
* igt@dmabuf@all-tests@dma_fence_chain:
- fi-bsw-nick: [INCOMPLETE][15] ([i915#12904]) -> [PASS][16] +1 other test pass
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/fi-bsw-nick/igt@dmabuf@all-tests@dma_fence_chain.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/fi-bsw-nick/igt@dmabuf@all-tests@dma_fence_chain.html
* igt@i915_module_load@load:
- bat-twl-1: [DMESG-WARN][17] ([i915#1982]) -> [PASS][18]
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/bat-twl-1/igt@i915_module_load@load.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/bat-twl-1/igt@i915_module_load@load.html
- {bat-mtlp-9}: [ABORT][19] ([i915#13268]) -> [PASS][20]
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/bat-mtlp-9/igt@i915_module_load@load.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/bat-mtlp-9/igt@i915_module_load@load.html
* igt@i915_selftest@live@gt_pm:
- bat-jsl-3: [DMESG-FAIL][21] ([i915#13132]) -> [PASS][22] +1 other test pass
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/bat-jsl-3/igt@i915_selftest@live@gt_pm.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/bat-jsl-3/igt@i915_selftest@live@gt_pm.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[i915#10216]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/10216
[i915#11681]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/11681
[i915#12061]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12061
[i915#12695]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12695
[i915#12904]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12904
[i915#13132]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13132
[i915#13268]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13268
[i915#1982]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/1982
[i915#3282]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3282
[i915#3291]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3291
[i915#3555]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3555
[i915#3708]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3708
[i915#3840]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3840
[i915#4077]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4077
[i915#4079]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4079
[i915#4083]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4083
[i915#4103]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4103
[i915#4212]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4212
[i915#4213]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4213
[i915#4613]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4613
[i915#5190]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/5190
[i915#5274]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/5274
[i915#6621]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/6621
[i915#8809]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8809
[i915#9159]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9159
[i915#9197]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9197
[i915#9318]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9318
[i915#9688]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9688
Build changes
-------------
* Linux: CI_DRM_15800 -> Patchwork_142188v2
CI-20190529: 20190529
CI_DRM_15800: 2e90353f058f2487798ad9236f0926df4ea54f02 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_8141: e776f39da6b3666a2834f7e02a1eed9a87f21d74 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_142188v2: 2e90353f058f2487798ad9236f0926df4ea54f02 @ git://anongit.freedesktop.org/gfx-ci/linux
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/index.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* ✓ i915.CI.Full: success for i915/gt: Reapply workarounds in case the previous attempt failed. (rev2)
2024-12-05 15:47 [RFC PATCH] i915/gt: Reapply workarounds in case the previous attempt failed Sebastian Brzezinka
` (3 preceding siblings ...)
2024-12-06 12:45 ` ✓ i915.CI.BAT: success " Patchwork
@ 2024-12-06 14:03 ` Patchwork
2024-12-06 15:38 ` [RFC PATCH] i915/gt: Reapply workarounds in case the previous attempt failed Rodrigo Vivi
5 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2024-12-06 14:03 UTC (permalink / raw)
To: Sebastian Brzezinka; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 29031 bytes --]
== Series Details ==
Series: i915/gt: Reapply workarounds in case the previous attempt failed. (rev2)
URL : https://patchwork.freedesktop.org/series/142188/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_15800_full -> Patchwork_142188v2_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
Participating hosts (9 -> 9)
------------------------------
No changes in participating hosts
Known issues
------------
Here are the changes found in Patchwork_142188v2_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@device_reset@unbind-reset-rebind:
- shard-dg1: NOTRUN -> [ABORT][1] ([i915#11814] / [i915#11815])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg1-12/igt@device_reset@unbind-reset-rebind.html
* igt@drm_fdinfo@busy-idle@bcs0:
- shard-dg2: NOTRUN -> [SKIP][2] ([i915#8414]) +15 other tests skip
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@drm_fdinfo@busy-idle@bcs0.html
* igt@drm_fdinfo@virtual-busy-idle:
- shard-dg1: NOTRUN -> [SKIP][3] ([i915#8414])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg1-12/igt@drm_fdinfo@virtual-busy-idle.html
* igt@gem_ccs@ctrl-surf-copy:
- shard-dg1: NOTRUN -> [SKIP][4] ([i915#3555] / [i915#9323])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg1-12/igt@gem_ccs@ctrl-surf-copy.html
* igt@gem_exec_balancer@sliced:
- shard-glk: NOTRUN -> [ABORT][5] ([i915#13218])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-glk9/igt@gem_exec_balancer@sliced.html
* igt@gem_exec_capture@capture-invisible:
- shard-tglu: NOTRUN -> [SKIP][6] ([i915#6334]) +1 other test skip
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-tglu-5/igt@gem_exec_capture@capture-invisible.html
* igt@gem_exec_fence@concurrent:
- shard-dg2: NOTRUN -> [SKIP][7] ([i915#4812])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@gem_exec_fence@concurrent.html
* igt@gem_exec_flush@basic-batch-kernel-default-wb:
- shard-dg2: NOTRUN -> [SKIP][8] ([i915#3539] / [i915#4852])
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@gem_exec_flush@basic-batch-kernel-default-wb.html
* igt@gem_exec_flush@basic-wb-pro-default:
- shard-dg1: NOTRUN -> [SKIP][9] ([i915#3539] / [i915#4852])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg1-14/igt@gem_exec_flush@basic-wb-pro-default.html
* igt@gem_exec_reloc@basic-write-cpu-active:
- shard-dg1: NOTRUN -> [SKIP][10] ([i915#3281])
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg1-14/igt@gem_exec_reloc@basic-write-cpu-active.html
* igt@gem_exec_reloc@basic-write-gtt:
- shard-dg2: NOTRUN -> [SKIP][11] ([i915#3281]) +1 other test skip
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@gem_exec_reloc@basic-write-gtt.html
* igt@gem_exec_schedule@semaphore-power:
- shard-dg2: NOTRUN -> [SKIP][12] ([i915#4537] / [i915#4812])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@gem_exec_schedule@semaphore-power.html
* igt@gem_lmem_swapping@heavy-verify-random:
- shard-glk: NOTRUN -> [SKIP][13] ([i915#4613])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-glk9/igt@gem_lmem_swapping@heavy-verify-random.html
* igt@gem_mmap@basic:
- shard-dg2: NOTRUN -> [SKIP][14] ([i915#4083]) +1 other test skip
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@gem_mmap@basic.html
* igt@gem_mmap_wc@write:
- shard-dg1: NOTRUN -> [SKIP][15] ([i915#4083])
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg1-12/igt@gem_mmap_wc@write.html
* igt@gem_partial_pwrite_pread@write-uncached:
- shard-dg1: NOTRUN -> [SKIP][16] ([i915#3282])
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg1-12/igt@gem_partial_pwrite_pread@write-uncached.html
* igt@gem_pxp@verify-pxp-stale-ctx-execution:
- shard-dg2: NOTRUN -> [SKIP][17] ([i915#4270])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@gem_pxp@verify-pxp-stale-ctx-execution.html
* igt@gem_render_copy@linear-to-vebox-y-tiled:
- shard-dg2: NOTRUN -> [SKIP][18] ([i915#5190] / [i915#8428])
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-1/igt@gem_render_copy@linear-to-vebox-y-tiled.html
* igt@gen7_exec_parse@bitmasks:
- shard-dg2: NOTRUN -> [SKIP][19]
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@gen7_exec_parse@bitmasks.html
* igt@gen9_exec_parse@batch-zero-length:
- shard-dg2: NOTRUN -> [SKIP][20] ([i915#2856]) +1 other test skip
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@gen9_exec_parse@batch-zero-length.html
* igt@i915_module_load@reload-with-fault-injection:
- shard-rkl: [PASS][21] -> [ABORT][22] ([i915#9820])
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/shard-rkl-1/igt@i915_module_load@reload-with-fault-injection.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-rkl-7/igt@i915_module_load@reload-with-fault-injection.html
* igt@i915_pm_rps@min-max-config-idle:
- shard-dg1: NOTRUN -> [SKIP][23] ([i915#11681] / [i915#6621])
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg1-12/igt@i915_pm_rps@min-max-config-idle.html
* igt@i915_pm_rps@thresholds-park:
- shard-dg1: NOTRUN -> [SKIP][24] ([i915#11681])
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg1-14/igt@i915_pm_rps@thresholds-park.html
* igt@i915_suspend@fence-restore-untiled:
- shard-dg2: NOTRUN -> [SKIP][25] ([i915#4077]) +4 other tests skip
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-1/igt@i915_suspend@fence-restore-untiled.html
* igt@kms_big_fb@4-tiled-64bpp-rotate-180:
- shard-mtlp: [PASS][26] -> [FAIL][27] ([i915#5138])
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/shard-mtlp-5/igt@kms_big_fb@4-tiled-64bpp-rotate-180.html
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-mtlp-6/igt@kms_big_fb@4-tiled-64bpp-rotate-180.html
* igt@kms_big_fb@x-tiled-32bpp-rotate-270:
- shard-dg1: NOTRUN -> [SKIP][28] ([i915#3638])
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg1-12/igt@kms_big_fb@x-tiled-32bpp-rotate-270.html
* igt@kms_big_fb@y-tiled-64bpp-rotate-270:
- shard-tglu-1: NOTRUN -> [SKIP][29] +3 other tests skip
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-tglu-1/igt@kms_big_fb@y-tiled-64bpp-rotate-270.html
* igt@kms_ccs@bad-pixel-format-4-tiled-mtl-rc-ccs@pipe-a-hdmi-a-1:
- shard-tglu-1: NOTRUN -> [SKIP][30] ([i915#6095]) +4 other tests skip
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-tglu-1/igt@kms_ccs@bad-pixel-format-4-tiled-mtl-rc-ccs@pipe-a-hdmi-a-1.html
* igt@kms_ccs@crc-primary-rotation-180-y-tiled-gen12-rc-ccs-cc@pipe-b-hdmi-a-2:
- shard-dg2: NOTRUN -> [SKIP][31] ([i915#10307] / [i915#6095]) +42 other tests skip
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-11/igt@kms_ccs@crc-primary-rotation-180-y-tiled-gen12-rc-ccs-cc@pipe-b-hdmi-a-2.html
* igt@kms_ccs@crc-primary-suspend-4-tiled-mtl-rc-ccs-cc@pipe-c-hdmi-a-4:
- shard-dg1: NOTRUN -> [SKIP][32] ([i915#6095]) +19 other tests skip
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg1-14/igt@kms_ccs@crc-primary-suspend-4-tiled-mtl-rc-ccs-cc@pipe-c-hdmi-a-4.html
* igt@kms_ccs@crc-primary-suspend-y-tiled-gen12-mc-ccs@pipe-a-hdmi-a-2:
- shard-dg2: NOTRUN -> [SKIP][33] ([i915#6095]) +3 other tests skip
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-11/igt@kms_ccs@crc-primary-suspend-y-tiled-gen12-mc-ccs@pipe-a-hdmi-a-2.html
* igt@kms_ccs@missing-ccs-buffer-y-tiled-ccs@pipe-b-hdmi-a-1:
- shard-rkl: NOTRUN -> [SKIP][34] ([i915#6095]) +15 other tests skip
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-rkl-2/igt@kms_ccs@missing-ccs-buffer-y-tiled-ccs@pipe-b-hdmi-a-1.html
* igt@kms_ccs@random-ccs-data-y-tiled-gen12-mc-ccs:
- shard-tglu: NOTRUN -> [SKIP][35] ([i915#6095]) +4 other tests skip
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-tglu-5/igt@kms_ccs@random-ccs-data-y-tiled-gen12-mc-ccs.html
* igt@kms_chamelium_audio@dp-audio-edid:
- shard-dg2: NOTRUN -> [SKIP][36] ([i915#7828]) +1 other test skip
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@kms_chamelium_audio@dp-audio-edid.html
* igt@kms_chamelium_frames@hdmi-crc-single:
- shard-tglu: NOTRUN -> [SKIP][37] ([i915#7828])
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-tglu-2/igt@kms_chamelium_frames@hdmi-crc-single.html
* igt@kms_chamelium_hpd@vga-hpd-with-enabled-mode:
- shard-tglu-1: NOTRUN -> [SKIP][38] ([i915#7828])
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-tglu-1/igt@kms_chamelium_hpd@vga-hpd-with-enabled-mode.html
* igt@kms_content_protection@lic-type-0:
- shard-glk: NOTRUN -> [SKIP][39] +13 other tests skip
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-glk9/igt@kms_content_protection@lic-type-0.html
* igt@kms_cursor_crc@cursor-rapid-movement-max-size:
- shard-dg2: NOTRUN -> [SKIP][40] ([i915#3555])
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@kms_cursor_crc@cursor-rapid-movement-max-size.html
* igt@kms_display_modes@mst-extended-mode-negative:
- shard-dg2: NOTRUN -> [SKIP][41] ([i915#8588])
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@kms_display_modes@mst-extended-mode-negative.html
* igt@kms_dither@fb-8bpc-vs-panel-6bpc:
- shard-dg2: [PASS][42] -> [SKIP][43] ([i915#3555])
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/shard-dg2-10/igt@kms_dither@fb-8bpc-vs-panel-6bpc.html
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-8/igt@kms_dither@fb-8bpc-vs-panel-6bpc.html
* igt@kms_dither@fb-8bpc-vs-panel-6bpc@pipe-a-hdmi-a-1:
- shard-rkl: NOTRUN -> [SKIP][44] ([i915#3804])
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-rkl-7/igt@kms_dither@fb-8bpc-vs-panel-6bpc@pipe-a-hdmi-a-1.html
* igt@kms_feature_discovery@display-4x:
- shard-tglu-1: NOTRUN -> [SKIP][45] ([i915#1839])
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-tglu-1/igt@kms_feature_discovery@display-4x.html
- shard-dg2: NOTRUN -> [SKIP][46] ([i915#1839])
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@kms_feature_discovery@display-4x.html
* igt@kms_flip@2x-blocking-absolute-wf_vblank:
- shard-dg1: NOTRUN -> [SKIP][47] ([i915#9934])
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg1-12/igt@kms_flip@2x-blocking-absolute-wf_vblank.html
* igt@kms_flip@flip-vs-blocking-wf-vblank:
- shard-rkl: [PASS][48] -> [FAIL][49] ([i915#11989])
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/shard-rkl-2/igt@kms_flip@flip-vs-blocking-wf-vblank.html
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-rkl-1/igt@kms_flip@flip-vs-blocking-wf-vblank.html
* igt@kms_flip@flip-vs-blocking-wf-vblank@a-hdmi-a1:
- shard-tglu: [PASS][50] -> [FAIL][51] ([i915#13131])
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/shard-tglu-9/igt@kms_flip@flip-vs-blocking-wf-vblank@a-hdmi-a1.html
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-tglu-9/igt@kms_flip@flip-vs-blocking-wf-vblank@a-hdmi-a1.html
* igt@kms_flip@flip-vs-blocking-wf-vblank@a-hdmi-a2:
- shard-rkl: NOTRUN -> [FAIL][52] ([i915#11989])
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-rkl-1/igt@kms_flip@flip-vs-blocking-wf-vblank@a-hdmi-a2.html
* igt@kms_flip@flip-vs-blocking-wf-vblank@c-hdmi-a1:
- shard-tglu: [PASS][53] -> [FAIL][54] ([i915#11989]) +6 other tests fail
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/shard-tglu-9/igt@kms_flip@flip-vs-blocking-wf-vblank@c-hdmi-a1.html
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-tglu-9/igt@kms_flip@flip-vs-blocking-wf-vblank@c-hdmi-a1.html
* igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytilegen12rcccs-upscaling:
- shard-dg2: NOTRUN -> [SKIP][55] ([i915#2672] / [i915#3555] / [i915#5190])
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytilegen12rcccs-upscaling.html
* igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytilegen12rcccs-upscaling@pipe-a-valid-mode:
- shard-dg2: NOTRUN -> [SKIP][56] ([i915#2672])
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytilegen12rcccs-upscaling@pipe-a-valid-mode.html
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-indfb-pgflip-blt:
- shard-dg1: NOTRUN -> [SKIP][57] +1 other test skip
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg1-12/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-indfb-pgflip-blt.html
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-shrfb-draw-mmap-gtt:
- shard-dg2: NOTRUN -> [SKIP][58] ([i915#8708]) +5 other tests skip
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-shrfb-draw-mmap-gtt.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-indfb-pgflip-blt:
- shard-tglu: NOTRUN -> [SKIP][59] +6 other tests skip
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-tglu-5/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-indfb-pgflip-blt.html
* igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-indfb-draw-blt:
- shard-dg1: NOTRUN -> [SKIP][60] ([i915#3458]) +2 other tests skip
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg1-12/igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-indfb-draw-blt.html
* igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-shrfb-draw-mmap-wc:
- shard-dg1: NOTRUN -> [SKIP][61] ([i915#8708])
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg1-13/igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-shrfb-draw-mmap-wc.html
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-blt:
- shard-dg2: NOTRUN -> [SKIP][62] ([i915#3458]) +4 other tests skip
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-blt.html
* igt@kms_frontbuffer_tracking@psr-2p-primscrn-shrfb-msflip-blt:
- shard-dg2: NOTRUN -> [SKIP][63] ([i915#5354]) +3 other tests skip
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@kms_frontbuffer_tracking@psr-2p-primscrn-shrfb-msflip-blt.html
* igt@kms_hdr@static-toggle:
- shard-dg2: [PASS][64] -> [SKIP][65] ([i915#3555] / [i915#8228])
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/shard-dg2-10/igt@kms_hdr@static-toggle.html
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-8/igt@kms_hdr@static-toggle.html
* igt@kms_joiner@basic-big-joiner:
- shard-tglu-1: NOTRUN -> [SKIP][66] ([i915#10656])
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-tglu-1/igt@kms_joiner@basic-big-joiner.html
- shard-dg2: NOTRUN -> [SKIP][67] ([i915#10656])
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@kms_joiner@basic-big-joiner.html
* igt@kms_pipe_crc_basic@hang-read-crc:
- shard-rkl: [PASS][68] -> [DMESG-WARN][69] ([i915#12964]) +1 other test dmesg-warn
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/shard-rkl-3/igt@kms_pipe_crc_basic@hang-read-crc.html
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-rkl-1/igt@kms_pipe_crc_basic@hang-read-crc.html
* igt@kms_pm_dc@dc6-psr:
- shard-dg2: NOTRUN -> [SKIP][70] ([i915#9685])
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-1/igt@kms_pm_dc@dc6-psr.html
* igt@kms_pm_rpm@dpms-lpsp:
- shard-dg2: NOTRUN -> [SKIP][71] ([i915#9519])
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@kms_pm_rpm@dpms-lpsp.html
* igt@kms_pm_rpm@dpms-non-lpsp:
- shard-rkl: [PASS][72] -> [SKIP][73] ([i915#9519])
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/shard-rkl-5/igt@kms_pm_rpm@dpms-non-lpsp.html
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-rkl-2/igt@kms_pm_rpm@dpms-non-lpsp.html
* igt@kms_psr2_sf@fbc-pr-cursor-plane-move-continuous-exceed-sf:
- shard-dg2: NOTRUN -> [SKIP][74] ([i915#11520]) +1 other test skip
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@kms_psr2_sf@fbc-pr-cursor-plane-move-continuous-exceed-sf.html
* igt@kms_psr2_sf@fbc-psr2-overlay-plane-update-sf-dmg-area:
- shard-tglu-1: NOTRUN -> [SKIP][75] ([i915#11520]) +1 other test skip
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-tglu-1/igt@kms_psr2_sf@fbc-psr2-overlay-plane-update-sf-dmg-area.html
* igt@kms_psr2_sf@pr-overlay-plane-update-continuous-sf:
- shard-glk: NOTRUN -> [SKIP][76] ([i915#11520]) +1 other test skip
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-glk9/igt@kms_psr2_sf@pr-overlay-plane-update-continuous-sf.html
* igt@kms_psr2_sf@psr2-overlay-plane-update-sf-dmg-area:
- shard-dg1: NOTRUN -> [SKIP][77] ([i915#11520])
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg1-12/igt@kms_psr2_sf@psr2-overlay-plane-update-sf-dmg-area.html
* igt@kms_psr@fbc-pr-basic:
- shard-dg2: NOTRUN -> [SKIP][78] ([i915#1072] / [i915#9732]) +2 other tests skip
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@kms_psr@fbc-pr-basic.html
* igt@kms_psr@psr-primary-mmap-gtt:
- shard-tglu: NOTRUN -> [SKIP][79] ([i915#9732])
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-tglu-5/igt@kms_psr@psr-primary-mmap-gtt.html
* igt@kms_tiled_display@basic-test-pattern:
- shard-glk: NOTRUN -> [FAIL][80] ([i915#10959])
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-glk9/igt@kms_tiled_display@basic-test-pattern.html
* igt@kms_writeback@writeback-invalid-parameters:
- shard-tglu-1: NOTRUN -> [SKIP][81] ([i915#2437])
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-tglu-1/igt@kms_writeback@writeback-invalid-parameters.html
* igt@perf@gen12-invalid-class-instance:
- shard-dg2: NOTRUN -> [ABORT][82] ([i915#13218]) +1 other test abort
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@perf@gen12-invalid-class-instance.html
* igt@perf@global-sseu-config-invalid@0-rcs0:
- shard-tglu: NOTRUN -> [ABORT][83] ([i915#13218]) +3 other tests abort
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-tglu-2/igt@perf@global-sseu-config-invalid@0-rcs0.html
* igt@perf_pmu@idle-no-semaphores:
- shard-dg1: NOTRUN -> [ABORT][84] ([i915#13218]) +3 other tests abort
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg1-14/igt@perf_pmu@idle-no-semaphores.html
* igt@perf_pmu@module-unload:
- shard-tglu-1: NOTRUN -> [ABORT][85] ([i915#13218]) +4 other tests abort
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-tglu-1/igt@perf_pmu@module-unload.html
- shard-dg2: NOTRUN -> [FAIL][86] ([i915#11823])
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-5/igt@perf_pmu@module-unload.html
#### Possible fixes ####
* igt@gem_userptr_blits@stress-mm:
- shard-rkl: [DMESG-WARN][87] ([i915#12964]) -> [PASS][88] +1 other test pass
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/shard-rkl-5/igt@gem_userptr_blits@stress-mm.html
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-rkl-2/igt@gem_userptr_blits@stress-mm.html
* igt@i915_suspend@fence-restore-tiled2untiled:
- shard-glk: [INCOMPLETE][89] ([i915#4817]) -> [PASS][90]
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/shard-glk8/igt@i915_suspend@fence-restore-tiled2untiled.html
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-glk9/igt@i915_suspend@fence-restore-tiled2untiled.html
* igt@kms_flip@flip-vs-absolute-wf_vblank:
- shard-rkl: [FAIL][91] ([i915#11989]) -> [PASS][92]
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/shard-rkl-2/igt@kms_flip@flip-vs-absolute-wf_vblank.html
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-rkl-3/igt@kms_flip@flip-vs-absolute-wf_vblank.html
#### Warnings ####
* igt@i915_module_load@reload-with-fault-injection:
- shard-dg2: [WARN][93] ([i915#12653]) -> [ABORT][94] ([i915#9820])
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/shard-dg2-10/igt@i915_module_load@reload-with-fault-injection.html
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-8/igt@i915_module_load@reload-with-fault-injection.html
* igt@kms_frontbuffer_tracking@fbcpsr-shrfb-scaledprimary:
- shard-dg2: [SKIP][95] ([i915#3458]) -> [SKIP][96] ([i915#10433] / [i915#3458]) +1 other test skip
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/shard-dg2-3/igt@kms_frontbuffer_tracking@fbcpsr-shrfb-scaledprimary.html
[96]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-dg2-4/igt@kms_frontbuffer_tracking@fbcpsr-shrfb-scaledprimary.html
* igt@kms_hdr@brightness-with-hdr:
- shard-rkl: [SKIP][97] ([i915#12713]) -> [SKIP][98] ([i915#1187] / [i915#12713])
[97]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_15800/shard-rkl-2/igt@kms_hdr@brightness-with-hdr.html
[98]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_142188v2/shard-rkl-3/igt@kms_hdr@brightness-with-hdr.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[i915#10055]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/10055
[i915#10307]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/10307
[i915#10433]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/10433
[i915#10656]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/10656
[i915#1072]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/1072
[i915#10959]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/10959
[i915#11520]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/11520
[i915#11681]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/11681
[i915#11814]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/11814
[i915#11815]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/11815
[i915#11823]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/11823
[i915#1187]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/1187
[i915#11989]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/11989
[i915#12339]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12339
[i915#12653]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12653
[i915#12713]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12713
[i915#12964]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12964
[i915#13131]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13131
[i915#13218]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13218
[i915#1839]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/1839
[i915#2437]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/2437
[i915#2672]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/2672
[i915#2856]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/2856
[i915#3281]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3281
[i915#3282]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3282
[i915#3458]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3458
[i915#3539]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3539
[i915#3555]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3555
[i915#3638]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3638
[i915#3804]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3804
[i915#4077]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4077
[i915#4083]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4083
[i915#4103]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4103
[i915#4213]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4213
[i915#4270]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4270
[i915#4537]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4537
[i915#4538]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4538
[i915#4613]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4613
[i915#4812]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4812
[i915#4817]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4817
[i915#4852]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4852
[i915#4860]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4860
[i915#4885]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4885
[i915#5138]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/5138
[i915#5190]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/5190
[i915#5354]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/5354
[i915#6095]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/6095
[i915#6334]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/6334
[i915#6621]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/6621
[i915#7828]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/7828
[i915#8228]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8228
[i915#8381]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8381
[i915#8414]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8414
[i915#8428]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8428
[i915#8588]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8588
[i915#8708]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8708
[i915#8850]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8850
[i915#9323]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9323
[i915#9519]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9519
[i915#9685]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9685
[i915#9732]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9732
[i915#9820]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9820
[i915#9917]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9917
[i915#9934]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9934
Build changes
-------------
* Linux: CI_DRM_15800 -> Patchwork_142188v2
CI-20190529: 20190529
CI_DRM_15800: 2e90353f058f2487798ad9236f0926df4ea54f02 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_8141: e776f39da6b3666a2834f7e02a1eed9a87f21d74 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_142188v2: 2e90353f058f2487798ad9236f0926df4ea54f02 @ 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_142188v2/index.html
[-- Attachment #2: Type: text/html, Size: 34021 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [RFC PATCH] i915/gt: Reapply workarounds in case the previous attempt failed.
2024-12-05 15:47 [RFC PATCH] i915/gt: Reapply workarounds in case the previous attempt failed Sebastian Brzezinka
` (4 preceding siblings ...)
2024-12-06 14:03 ` ✓ i915.CI.Full: " Patchwork
@ 2024-12-06 15:38 ` Rodrigo Vivi
2024-12-12 14:51 ` Andi Shyti
5 siblings, 1 reply; 10+ messages in thread
From: Rodrigo Vivi @ 2024-12-06 15:38 UTC (permalink / raw)
To: Sebastian Brzezinka; +Cc: intel-gfx, andi.shyti, vidya.srinivas
On Thu, Dec 05, 2024 at 03:47:35PM +0000, Sebastian Brzezinka wrote:
> `wa_verify`sporadically detects lost workaround on application; this
> is unusual behavior since wa are applied at `intel_gt_init_hw` and
> verified right away by `intel_gt_verify_workarounds`, and `wa_verify`
> doesn't fail on initialization as one might suspect would happen.
>
> One approach that may be somewhat beneficial is to reapply workarounds
> in the event of failure, or even get rid of verify on application,
> since it's redundant to `intel_gt_verify_workarounds`.
>
> This patch aims to resolve: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12668
It should be:
Closes: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12668
>
> Signed-off-by: Sebastian Brzezinka <sebastian.brzezinka@intel.com>
> ---
> drivers/gpu/drm/i915/gt/intel_workarounds.c | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/gt/intel_workarounds.c b/drivers/gpu/drm/i915/gt/intel_workarounds.c
> index 570c91878189..4ee623448223 100644
> --- a/drivers/gpu/drm/i915/gt/intel_workarounds.c
> +++ b/drivers/gpu/drm/i915/gt/intel_workarounds.c
> @@ -1761,6 +1761,17 @@ static void wa_list_apply(const struct i915_wa_list *wal)
> intel_gt_mcr_read_any_fw(gt, wa->mcr_reg) :
> intel_uncore_read_fw(uncore, wa->reg);
>
> + if ((val ^ wa->set) & wa->read) {
> + if (wa->is_mcr)
> + intel_gt_mcr_multicast_write_fw(gt, wa->mcr_reg, val);
> + else
> + intel_uncore_write_fw(uncore, wa->reg, val);
> + }
instead of duplicating the code you should extract that to an aux function
to write it...
> +
> + val = wa->is_mcr ?
> + intel_gt_mcr_read_any_fw(gt, wa->mcr_reg) :
> + intel_uncore_read_fw(uncore, wa->reg);
and another one to read it...
> +
> wa_verify(gt, wa, val, wal->name, "application");
However my biggest concern with this patch is the brute force solution
and only on CONFIG_DRM_I915_DEBUG_GEM case...
and as duplication because I see that the second write attempt is
already happening above if (val != old || !wa->clr)
So, something is not quite right in here and this deserves another alternative..
> }
> }
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [RFC PATCH] i915/gt: Reapply workarounds in case the previous attempt failed.
2024-12-06 15:38 ` [RFC PATCH] i915/gt: Reapply workarounds in case the previous attempt failed Rodrigo Vivi
@ 2024-12-12 14:51 ` Andi Shyti
2024-12-16 21:27 ` Matt Roper
0 siblings, 1 reply; 10+ messages in thread
From: Andi Shyti @ 2024-12-12 14:51 UTC (permalink / raw)
To: Rodrigo Vivi; +Cc: Sebastian Brzezinka, intel-gfx, andi.shyti, vidya.srinivas
Hi Rodrigo,
On Fri, Dec 06, 2024 at 10:38:24AM -0500, Rodrigo Vivi wrote:
> On Thu, Dec 05, 2024 at 03:47:35PM +0000, Sebastian Brzezinka wrote:
> > `wa_verify`sporadically detects lost workaround on application; this
> > is unusual behavior since wa are applied at `intel_gt_init_hw` and
> > verified right away by `intel_gt_verify_workarounds`, and `wa_verify`
> > doesn't fail on initialization as one might suspect would happen.
> >
> > One approach that may be somewhat beneficial is to reapply workarounds
> > in the event of failure, or even get rid of verify on application,
> > since it's redundant to `intel_gt_verify_workarounds`.
> >
> > This patch aims to resolve: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12668
>
> It should be:
>
> Closes: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12668
aapart from the formatting issues this was suggested by me. We
have observed some sporadic vailures in applying the specific
workaround added by Ville (now cc'ed to the thread) in commit
0ddae025ab6c ("drm/i915: Disable compression tricks on JSL").
Because it's sporadic, we could give it one more chance and try
to re-apply it.
> >
> > Signed-off-by: Sebastian Brzezinka <sebastian.brzezinka@intel.com>
> > ---
> > drivers/gpu/drm/i915/gt/intel_workarounds.c | 11 +++++++++++
> > 1 file changed, 11 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/i915/gt/intel_workarounds.c b/drivers/gpu/drm/i915/gt/intel_workarounds.c
> > index 570c91878189..4ee623448223 100644
> > --- a/drivers/gpu/drm/i915/gt/intel_workarounds.c
> > +++ b/drivers/gpu/drm/i915/gt/intel_workarounds.c
> > @@ -1761,6 +1761,17 @@ static void wa_list_apply(const struct i915_wa_list *wal)
> > intel_gt_mcr_read_any_fw(gt, wa->mcr_reg) :
> > intel_uncore_read_fw(uncore, wa->reg);
> >
> > + if ((val ^ wa->set) & wa->read) {
> > + if (wa->is_mcr)
> > + intel_gt_mcr_multicast_write_fw(gt, wa->mcr_reg, val);
> > + else
> > + intel_uncore_write_fw(uncore, wa->reg, val);
> > + }
>
> instead of duplicating the code you should extract that to an aux function
> to write it...
a for loop can decrease the amount of duplicated code.
> > +
> > + val = wa->is_mcr ?
> > + intel_gt_mcr_read_any_fw(gt, wa->mcr_reg) :
> > + intel_uncore_read_fw(uncore, wa->reg);
>
> and another one to read it...
this, indeed it's just reading, but we are trying to re-write. If
we wrote the unwanted value, we will keep reading the unwanted
value.
> > +
> > wa_verify(gt, wa, val, wal->name, "application");
>
> However my biggest concern with this patch is the brute force solution
> and only on CONFIG_DRM_I915_DEBUG_GEM case...
this is a good point, indeed, I don't understand why the
confirmation should be within the DEBUG section.
Andi
> and as duplication because I see that the second write attempt is
> already happening above if (val != old || !wa->clr)
>
> So, something is not quite right in here and this deserves another alternative..
>
>
> > }
> > }
> > --
> > 2.34.1
> >
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [RFC PATCH] i915/gt: Reapply workarounds in case the previous attempt failed.
2024-12-12 14:51 ` Andi Shyti
@ 2024-12-16 21:27 ` Matt Roper
2024-12-18 14:48 ` Andi Shyti
0 siblings, 1 reply; 10+ messages in thread
From: Matt Roper @ 2024-12-16 21:27 UTC (permalink / raw)
To: Andi Shyti; +Cc: Rodrigo Vivi, Sebastian Brzezinka, intel-gfx, vidya.srinivas
On Thu, Dec 12, 2024 at 03:51:12PM +0100, Andi Shyti wrote:
> Hi Rodrigo,
>
> On Fri, Dec 06, 2024 at 10:38:24AM -0500, Rodrigo Vivi wrote:
> > On Thu, Dec 05, 2024 at 03:47:35PM +0000, Sebastian Brzezinka wrote:
> > > `wa_verify`sporadically detects lost workaround on application; this
> > > is unusual behavior since wa are applied at `intel_gt_init_hw` and
> > > verified right away by `intel_gt_verify_workarounds`, and `wa_verify`
> > > doesn't fail on initialization as one might suspect would happen.
> > >
> > > One approach that may be somewhat beneficial is to reapply workarounds
> > > in the event of failure, or even get rid of verify on application,
> > > since it's redundant to `intel_gt_verify_workarounds`.
> > >
> > > This patch aims to resolve: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12668
> >
> > It should be:
> >
> > Closes: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12668
>
> aapart from the formatting issues this was suggested by me. We
> have observed some sporadic vailures in applying the specific
> workaround added by Ville (now cc'ed to the thread) in commit
> 0ddae025ab6c ("drm/i915: Disable compression tricks on JSL").
>
> Because it's sporadic, we could give it one more chance and try
> to re-apply it.
That sounds like it's just papering over the issue without really
figuring out what's truly going on.
Looking at the current implementation, it looks like at least one
possible problem is that it was implemented in rcs_engine_wa_init, but
the CACHE_MODE_0 register itself is part of the LRC (according to bspec
18907). So we want to move it to icl_ctx_workarounds_init() instead to
make sure it gets recorded in the golden context image. Our
initialization and reset handling for workarounds touching registers in
the context are different from those that aren't.
BTW, I'm a bit surprised to see us needing to implement this workaround
in the kernel at all. CACHE_MODE_0 is a register that's under userspace
control (according to bspec 14181), so we usually expect the userspace
drivers to own implementing any workarounds dealing with the registers
they control. Indeed, it looks like Mesa's Iris driver already has an
implementation of this workaround in iris_state.c:
if (devinfo->disable_ccs_repack) {
iris_emit_reg(batch, GENX(CACHE_MODE_0), reg) {
reg.DisableRepackingforCompression = true;
reg.DisableRepackingforCompressionMask = true;
}
}
and that workaround was added back in mid-2019 so it should be in all
recent Mesa releases.
Matt
>
> > >
> > > Signed-off-by: Sebastian Brzezinka <sebastian.brzezinka@intel.com>
> > > ---
> > > drivers/gpu/drm/i915/gt/intel_workarounds.c | 11 +++++++++++
> > > 1 file changed, 11 insertions(+)
> > >
> > > diff --git a/drivers/gpu/drm/i915/gt/intel_workarounds.c b/drivers/gpu/drm/i915/gt/intel_workarounds.c
> > > index 570c91878189..4ee623448223 100644
> > > --- a/drivers/gpu/drm/i915/gt/intel_workarounds.c
> > > +++ b/drivers/gpu/drm/i915/gt/intel_workarounds.c
> > > @@ -1761,6 +1761,17 @@ static void wa_list_apply(const struct i915_wa_list *wal)
> > > intel_gt_mcr_read_any_fw(gt, wa->mcr_reg) :
> > > intel_uncore_read_fw(uncore, wa->reg);
> > >
> > > + if ((val ^ wa->set) & wa->read) {
> > > + if (wa->is_mcr)
> > > + intel_gt_mcr_multicast_write_fw(gt, wa->mcr_reg, val);
> > > + else
> > > + intel_uncore_write_fw(uncore, wa->reg, val);
> > > + }
> >
> > instead of duplicating the code you should extract that to an aux function
> > to write it...
>
> a for loop can decrease the amount of duplicated code.
>
> > > +
> > > + val = wa->is_mcr ?
> > > + intel_gt_mcr_read_any_fw(gt, wa->mcr_reg) :
> > > + intel_uncore_read_fw(uncore, wa->reg);
> >
> > and another one to read it...
>
> this, indeed it's just reading, but we are trying to re-write. If
> we wrote the unwanted value, we will keep reading the unwanted
> value.
>
> > > +
> > > wa_verify(gt, wa, val, wal->name, "application");
> >
> > However my biggest concern with this patch is the brute force solution
> > and only on CONFIG_DRM_I915_DEBUG_GEM case...
>
> this is a good point, indeed, I don't understand why the
> confirmation should be within the DEBUG section.
>
> Andi
>
> > and as duplication because I see that the second write attempt is
> > already happening above if (val != old || !wa->clr)
> >
> > So, something is not quite right in here and this deserves another alternative..
> >
> >
> > > }
> > > }
> > > --
> > > 2.34.1
> > >
--
Matt Roper
Graphics Software Engineer
Linux GPU Platform Enablement
Intel Corporation
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [RFC PATCH] i915/gt: Reapply workarounds in case the previous attempt failed.
2024-12-16 21:27 ` Matt Roper
@ 2024-12-18 14:48 ` Andi Shyti
0 siblings, 0 replies; 10+ messages in thread
From: Andi Shyti @ 2024-12-18 14:48 UTC (permalink / raw)
To: Matt Roper
Cc: Andi Shyti, Rodrigo Vivi, Sebastian Brzezinka, intel-gfx,
vidya.srinivas, Ville Syrjala
Hi Matt,
On Mon, Dec 16, 2024 at 01:27:51PM -0800, Matt Roper wrote:
> On Thu, Dec 12, 2024 at 03:51:12PM +0100, Andi Shyti wrote:
> > On Fri, Dec 06, 2024 at 10:38:24AM -0500, Rodrigo Vivi wrote:
> > > On Thu, Dec 05, 2024 at 03:47:35PM +0000, Sebastian Brzezinka wrote:
> > > > `wa_verify`sporadically detects lost workaround on application; this
> > > > is unusual behavior since wa are applied at `intel_gt_init_hw` and
> > > > verified right away by `intel_gt_verify_workarounds`, and `wa_verify`
> > > > doesn't fail on initialization as one might suspect would happen.
> > > >
> > > > One approach that may be somewhat beneficial is to reapply workarounds
> > > > in the event of failure, or even get rid of verify on application,
> > > > since it's redundant to `intel_gt_verify_workarounds`.
> > > >
> > > > This patch aims to resolve: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12668
> > >
> > > It should be:
> > >
> > > Closes: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12668
> >
> > aapart from the formatting issues this was suggested by me. We
> > have observed some sporadic vailures in applying the specific
> > workaround added by Ville (now cc'ed to the thread) in commit
> > 0ddae025ab6c ("drm/i915: Disable compression tricks on JSL").
> >
> > Because it's sporadic, we could give it one more chance and try
> > to re-apply it.
>
> That sounds like it's just papering over the issue without really
> figuring out what's truly going on.
Independently from your next comments and the implementation, the
sporadic errors we've seen have an extremely low rate and we
weren't able to make sense to them or fully test them.
We had a short chat with Ville (who implemented the workaround)
amd he suggested to leave it as it is, while I suggested to give
it another chance.
> Looking at the current implementation, it looks like at least one
> possible problem is that it was implemented in rcs_engine_wa_init, but
> the CACHE_MODE_0 register itself is part of the LRC (according to bspec
> 18907). So we want to move it to icl_ctx_workarounds_init() instead to
> make sure it gets recorded in the golden context image. Our
> initialization and reset handling for workarounds touching registers in
> the context are different from those that aren't.
> BTW, I'm a bit surprised to see us needing to implement this workaround
> in the kernel at all. CACHE_MODE_0 is a register that's under userspace
> control (according to bspec 14181), so we usually expect the userspace
> drivers to own implementing any workarounds dealing with the registers
> they control. Indeed, it looks like Mesa's Iris driver already has an
> implementation of this workaround in iris_state.c:
>
> if (devinfo->disable_ccs_repack) {
> iris_emit_reg(batch, GENX(CACHE_MODE_0), reg) {
> reg.DisableRepackingforCompression = true;
> reg.DisableRepackingforCompressionMask = true;
> }
> }
>
> and that workaround was added back in mid-2019 so it should be in all
> recent Mesa releases.
Ville? Any comment here?
Andi
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2024-12-18 14:48 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-12-05 15:47 [RFC PATCH] i915/gt: Reapply workarounds in case the previous attempt failed Sebastian Brzezinka
2024-12-05 21:45 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
2024-12-05 22:04 ` ✗ i915.CI.BAT: failure " Patchwork
2024-12-06 12:36 ` ✗ Fi.CI.CHECKPATCH: warning for i915/gt: Reapply workarounds in case the previous attempt failed. (rev2) Patchwork
2024-12-06 12:45 ` ✓ i915.CI.BAT: success " Patchwork
2024-12-06 14:03 ` ✓ i915.CI.Full: " Patchwork
2024-12-06 15:38 ` [RFC PATCH] i915/gt: Reapply workarounds in case the previous attempt failed Rodrigo Vivi
2024-12-12 14:51 ` Andi Shyti
2024-12-16 21:27 ` Matt Roper
2024-12-18 14:48 ` Andi Shyti
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox