* [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry
@ 2021-07-28 17:16 Mark Yacoub
2021-07-28 20:32 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
` (8 more replies)
0 siblings, 9 replies; 16+ messages in thread
From: Mark Yacoub @ 2021-07-28 17:16 UTC (permalink / raw)
To: igt-dev; +Cc: robdclark, petri.latvala, seanpaul, Mark Yacoub
From: Mark Yacoub <markyacoub@google.com>
[Why]
On slower devices, non blocking do_page_flip followed by wait_for_events does not finish fast enough to send an event to be consumed by wait_for_events.
Instead, wait_for_events ends up finding an unconsumed event from a
previous do_page_flip (i.e. a skipped preceding subtest) and signals
that the userspace is ready to do another do_page_flip while the first
one hasn't finished yet, resulting in an EBUSY.
[How]
At the beginning of every test running on crtc set, check for any stale
events and consume them so the kernel has no pending events.
Fixes: flip-vs-suspend and flip-vs-suspend-interruptible.
Tested on ChromeOS Volteer(i915-TGL) and Trogdor(msm)
Signed-off-by: Mark Yacoub <markyacoub@chromium.org>
Change-Id: I254a84a202140c332312aba1d9edf15ea78b7a61
---
tests/kms_flip.c | 23 +++++++++++++++++++++++
1 file changed, 23 insertions(+)
diff --git a/tests/kms_flip.c b/tests/kms_flip.c
index f2fce8d2..0a2041c5 100755
--- a/tests/kms_flip.c
+++ b/tests/kms_flip.c
@@ -1261,6 +1261,26 @@ static bool needs_retry_after_link_reset(struct udev_monitor *mon)
return hotplug_detected;
}
+static void discard_any_stale_events() {
+ fd_set fds;
+ FD_ZERO(&fds);
+ FD_SET(drm_fd, &fds);
+ struct timeval timeout = { .tv_sec = 3, .tv_usec = 0 };
+ int ret = select(drm_fd + 1, &fds, NULL, NULL, &timeout);
+
+ if (ret > 0) {
+ drmEventContext evctx;
+ memset(&evctx, 0, sizeof evctx);
+ evctx.version = 2;
+ igt_info("Stale Event found - Discarding now\n");
+ drmHandleEvent(drm_fd, &evctx);
+ }
+ else {
+ igt_debug("No stale Event found\n");
+ }
+}
+
+
static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
int crtc_count, int duration_ms)
{
@@ -1311,6 +1331,9 @@ static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
kmstest_dump_mode(&o->kmode[i]);
retry:
+ // Discard any pending event that hasn't been consumed from a previous retry or subtest.
+ discard_any_stale_events();
+
memset(&o->vblank_state, 0, sizeof(o->vblank_state));
memset(&o->flip_state, 0, sizeof(o->flip_state));
o->flip_state.name = "flip";
--
2.32.0.432.gabb21c7263-goog
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_flip: Discard any stale event at each retry
2021-07-28 17:16 [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry Mark Yacoub
@ 2021-07-28 20:32 ` Patchwork
2021-07-28 23:15 ` [igt-dev] ✗ GitLab.Pipeline: warning " Patchwork
` (7 subsequent siblings)
8 siblings, 0 replies; 16+ messages in thread
From: Patchwork @ 2021-07-28 20:32 UTC (permalink / raw)
To: Mark Yacoub; +Cc: igt-dev
[-- Attachment #1.1: Type: text/plain, Size: 3185 bytes --]
== Series Details ==
Series: tests/kms_flip: Discard any stale event at each retry
URL : https://patchwork.freedesktop.org/series/93133/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10413 -> IGTPW_6072
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/index.html
Known issues
------------
Here are the changes found in IGTPW_6072 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@amdgpu/amd_basic@semaphore:
- fi-bdw-5557u: NOTRUN -> [SKIP][1] ([fdo#109271]) +29 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/fi-bdw-5557u/igt@amdgpu/amd_basic@semaphore.html
* igt@core_hotunplug@unbind-rebind:
- fi-bdw-5557u: NOTRUN -> [WARN][2] ([i915#3718])
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/fi-bdw-5557u/igt@core_hotunplug@unbind-rebind.html
* igt@i915_pm_rpm@basic-rte:
- fi-bdw-5557u: NOTRUN -> [FAIL][3] ([i915#579])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/fi-bdw-5557u/igt@i915_pm_rpm@basic-rte.html
* igt@i915_selftest@live@execlists:
- fi-bsw-nick: [PASS][4] -> [INCOMPLETE][5] ([i915#2782] / [i915#2940])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/fi-bsw-nick/igt@i915_selftest@live@execlists.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/fi-bsw-nick/igt@i915_selftest@live@execlists.html
* igt@kms_chamelium@dp-crc-fast:
- fi-bdw-5557u: NOTRUN -> [SKIP][6] ([fdo#109271] / [fdo#111827]) +8 similar issues
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/fi-bdw-5557u/igt@kms_chamelium@dp-crc-fast.html
* igt@runner@aborted:
- fi-bsw-nick: NOTRUN -> [FAIL][7] ([fdo#109271] / [i915#1436] / [i915#2722])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/fi-bsw-nick/igt@runner@aborted.html
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
[i915#1436]: https://gitlab.freedesktop.org/drm/intel/issues/1436
[i915#2722]: https://gitlab.freedesktop.org/drm/intel/issues/2722
[i915#2782]: https://gitlab.freedesktop.org/drm/intel/issues/2782
[i915#2940]: https://gitlab.freedesktop.org/drm/intel/issues/2940
[i915#3718]: https://gitlab.freedesktop.org/drm/intel/issues/3718
[i915#579]: https://gitlab.freedesktop.org/drm/intel/issues/579
Participating hosts (42 -> 36)
------------------------------
Missing (6): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-ctg-p8600 fi-bdw-samus bat-jsl-1
Build changes
-------------
* CI: CI-20190529 -> None
* IGT: IGT_6155 -> IGTPW_6072
CI-20190529: 20190529
CI_DRM_10413: 33ace46253ab0ea580c1ff117e8ec2c6096d3297 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_6072: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/index.html
IGT_6155: 4b51398dcd7559012b85776e7353d516ff1e6ce6 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/index.html
[-- Attachment #1.2: Type: text/html, Size: 4039 bytes --]
[-- Attachment #2: Type: text/plain, Size: 154 bytes --]
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 16+ messages in thread
* [igt-dev] ✗ GitLab.Pipeline: warning for tests/kms_flip: Discard any stale event at each retry
2021-07-28 17:16 [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry Mark Yacoub
2021-07-28 20:32 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
@ 2021-07-28 23:15 ` Patchwork
2021-07-29 1:07 ` [igt-dev] ✓ Fi.CI.IGT: success " Patchwork
` (6 subsequent siblings)
8 siblings, 0 replies; 16+ messages in thread
From: Patchwork @ 2021-07-28 23:15 UTC (permalink / raw)
To: Mark Yacoub; +Cc: igt-dev
== Series Details ==
Series: tests/kms_flip: Discard any stale event at each retry
URL : https://patchwork.freedesktop.org/series/93133/
State : warning
== Summary ==
Pipeline status: FAILED.
see https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/369629 for the overview.
test:ninja-test has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/12280799):
9/305 lib igt_exit_handler TIMEOUT 30.09 s
--- command ---
/builds/gfx-ci/igt-ci-tags/build/lib/tests/igt_exit_handler
-------
10/305 lib igt_fork TIMEOUT 30.07 s
--- command ---
/builds/gfx-ci/igt-ci-tags/build/lib/tests/igt_fork
-------
Full log written to /builds/gfx-ci/igt-ci-tags/build/meson-logs/testlog.txt
FAILED: meson-test
/usr/bin/meson test --no-rebuild --print-errorlogs
ninja: build stopped: subcommand failed.
section_end:1627513727:step_script
ERROR: Job failed: execution took longer than 1h0m0s seconds
test:ninja-test-arm64 has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/12280801):
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 805, in run
return th.doit()
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 555, in doit
self.run_tests(tests)
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 715, in run_tests
self.drain_futures(futures)
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 732, in drain_futures
self.print_stats(numlen, tests, name, result.result(), i)
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 505, in print_stats
result_str += "\n\n" + result.get_log()
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 178, in get_log
res += self.stde
TypeError: can only concatenate str (not "bytes") to str
1/291 lib igt_assert TIMEOUT 32.06 s
FAILED: meson-test
/usr/bin/meson test --no-rebuild --print-errorlogs
ninja: build stopped: subcommand failed.
section_end:1627513732:step_script
ERROR: Job failed: execution took longer than 1h0m0s seconds
test:ninja-test-armhf has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/12280802):
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 805, in run
return th.doit()
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 555, in doit
self.run_tests(tests)
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 715, in run_tests
self.drain_futures(futures)
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 732, in drain_futures
self.print_stats(numlen, tests, name, result.result(), i)
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 505, in print_stats
result_str += "\n\n" + result.get_log()
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 178, in get_log
res += self.stde
TypeError: can only concatenate str (not "bytes") to str
1/291 lib igt_assert TIMEOUT 32.03 s
FAILED: meson-test
/usr/bin/meson test --no-rebuild --print-errorlogs
ninja: build stopped: subcommand failed.
section_end:1627513732:step_script
ERROR: Job failed: execution took longer than 1h0m0s seconds
test:ninja-test-minimal has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/12280800):
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 805, in run
return th.doit()
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 555, in doit
self.run_tests(tests)
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 715, in run_tests
self.drain_futures(futures)
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 732, in drain_futures
self.print_stats(numlen, tests, name, result.result(), i)
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 505, in print_stats
result_str += "\n\n" + result.get_log()
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 178, in get_log
res += self.stde
TypeError: can only concatenate str (not "bytes") to str
1/24 lib igt_assert TIMEOUT 32.03 s
FAILED: meson-test
/usr/bin/meson test --no-rebuild --print-errorlogs
ninja: build stopped: subcommand failed.
section_end:1627513731:step_script
ERROR: Job failed: execution took longer than 1h0m0s seconds
test:ninja-test-mips has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/12280803):
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 805, in run
return th.doit()
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 555, in doit
self.run_tests(tests)
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 715, in run_tests
self.drain_futures(futures)
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 732, in drain_futures
self.print_stats(numlen, tests, name, result.result(), i)
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 505, in print_stats
result_str += "\n\n" + result.get_log()
File "/usr/lib/python3/dist-packages/mesonbuild/mtest.py", line 178, in get_log
res += self.stde
TypeError: can only concatenate str (not "bytes") to str
1/291 lib igt_assert TIMEOUT 32.04 s
FAILED: meson-test
/usr/bin/meson test --no-rebuild --print-errorlogs
ninja: build stopped: subcommand failed.
section_end:1627513736:step_script
ERROR: Job failed: execution took longer than 1h0m0s seconds
== Logs ==
For more details see: https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/369629
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 16+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for tests/kms_flip: Discard any stale event at each retry
2021-07-28 17:16 [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry Mark Yacoub
2021-07-28 20:32 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2021-07-28 23:15 ` [igt-dev] ✗ GitLab.Pipeline: warning " Patchwork
@ 2021-07-29 1:07 ` Patchwork
2021-08-26 12:55 ` [igt-dev] [PATCH] " Rodrigo Siqueira
` (5 subsequent siblings)
8 siblings, 0 replies; 16+ messages in thread
From: Patchwork @ 2021-07-29 1:07 UTC (permalink / raw)
To: Mark Yacoub; +Cc: igt-dev
[-- Attachment #1.1: Type: text/plain, Size: 30271 bytes --]
== Series Details ==
Series: tests/kms_flip: Discard any stale event at each retry
URL : https://patchwork.freedesktop.org/series/93133/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10413_full -> IGTPW_6072_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/index.html
Known issues
------------
Here are the changes found in IGTPW_6072_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_create@create-massive:
- shard-kbl: NOTRUN -> [DMESG-WARN][1] ([i915#3002])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl2/igt@gem_create@create-massive.html
* igt@gem_ctx_isolation@preservation-s3@vecs0:
- shard-apl: [PASS][2] -> [DMESG-WARN][3] ([i915#180])
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-apl3/igt@gem_ctx_isolation@preservation-s3@vecs0.html
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-apl1/igt@gem_ctx_isolation@preservation-s3@vecs0.html
* igt@gem_ctx_persistence@legacy-engines-hang@render:
- shard-apl: NOTRUN -> [FAIL][4] ([i915#2410])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-apl2/igt@gem_ctx_persistence@legacy-engines-hang@render.html
- shard-glk: NOTRUN -> [FAIL][5] ([i915#2410])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-glk8/igt@gem_ctx_persistence@legacy-engines-hang@render.html
* igt@gem_ctx_persistence@legacy-engines-persistence:
- shard-snb: NOTRUN -> [SKIP][6] ([fdo#109271] / [i915#1099]) +4 similar issues
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-snb6/igt@gem_ctx_persistence@legacy-engines-persistence.html
* igt@gem_ctx_persistence@many-contexts:
- shard-iclb: [PASS][7] -> [INCOMPLETE][8] ([i915#3057])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-iclb3/igt@gem_ctx_persistence@many-contexts.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb4/igt@gem_ctx_persistence@many-contexts.html
- shard-tglb: [PASS][9] -> [FAIL][10] ([i915#2410])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-tglb7/igt@gem_ctx_persistence@many-contexts.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb6/igt@gem_ctx_persistence@many-contexts.html
* igt@gem_eio@unwedge-stress:
- shard-iclb: NOTRUN -> [TIMEOUT][11] ([i915#2369] / [i915#2481] / [i915#3070])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb8/igt@gem_eio@unwedge-stress.html
* igt@gem_exec_fair@basic-deadline:
- shard-apl: NOTRUN -> [FAIL][12] ([i915#2846])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-apl6/igt@gem_exec_fair@basic-deadline.html
* igt@gem_exec_fair@basic-none-solo@rcs0:
- shard-kbl: [PASS][13] -> [FAIL][14] ([i915#2842]) +1 similar issue
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-kbl6/igt@gem_exec_fair@basic-none-solo@rcs0.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl2/igt@gem_exec_fair@basic-none-solo@rcs0.html
* igt@gem_exec_flush@basic-batch-kernel-default-cmd:
- shard-snb: NOTRUN -> [SKIP][15] ([fdo#109271]) +405 similar issues
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-snb7/igt@gem_exec_flush@basic-batch-kernel-default-cmd.html
* igt@gem_exec_whisper@basic-queues-forked-all:
- shard-glk: NOTRUN -> [DMESG-WARN][16] ([i915#118] / [i915#95])
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-glk3/igt@gem_exec_whisper@basic-queues-forked-all.html
* igt@gem_huc_copy@huc-copy:
- shard-apl: NOTRUN -> [SKIP][17] ([fdo#109271] / [i915#2190])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-apl2/igt@gem_huc_copy@huc-copy.html
* igt@gem_mmap_gtt@coherency:
- shard-tglb: NOTRUN -> [SKIP][18] ([fdo#111656])
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb3/igt@gem_mmap_gtt@coherency.html
- shard-iclb: NOTRUN -> [SKIP][19] ([fdo#109292])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb5/igt@gem_mmap_gtt@coherency.html
* igt@gem_mmap_gtt@cpuset-big-copy:
- shard-iclb: [PASS][20] -> [FAIL][21] ([i915#307])
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-iclb3/igt@gem_mmap_gtt@cpuset-big-copy.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb5/igt@gem_mmap_gtt@cpuset-big-copy.html
* igt@gem_pread@exhaustion:
- shard-apl: NOTRUN -> [WARN][22] ([i915#2658])
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-apl7/igt@gem_pread@exhaustion.html
* igt@gem_render_copy@yf-tiled-mc-ccs-to-vebox-y-tiled:
- shard-iclb: NOTRUN -> [SKIP][23] ([i915#768])
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb7/igt@gem_render_copy@yf-tiled-mc-ccs-to-vebox-y-tiled.html
* igt@gem_userptr_blits@dmabuf-sync:
- shard-kbl: NOTRUN -> [SKIP][24] ([fdo#109271] / [i915#3323])
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl2/igt@gem_userptr_blits@dmabuf-sync.html
* igt@gem_userptr_blits@vma-merge:
- shard-snb: NOTRUN -> [FAIL][25] ([i915#2724])
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-snb5/igt@gem_userptr_blits@vma-merge.html
* igt@gen9_exec_parse@bb-start-cmd:
- shard-tglb: NOTRUN -> [SKIP][26] ([i915#2856])
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb5/igt@gen9_exec_parse@bb-start-cmd.html
* igt@gen9_exec_parse@bb-start-far:
- shard-iclb: NOTRUN -> [SKIP][27] ([i915#2856]) +1 similar issue
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb7/igt@gen9_exec_parse@bb-start-far.html
* igt@i915_pm_dc@dc6-dpms:
- shard-iclb: [PASS][28] -> [FAIL][29] ([i915#454])
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-iclb4/igt@i915_pm_dc@dc6-dpms.html
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb5/igt@i915_pm_dc@dc6-dpms.html
* igt@i915_pm_rpm@basic-rte:
- shard-kbl: NOTRUN -> [FAIL][30] ([i915#579])
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl1/igt@i915_pm_rpm@basic-rte.html
- shard-apl: NOTRUN -> [FAIL][31] ([i915#579])
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-apl6/igt@i915_pm_rpm@basic-rte.html
- shard-glk: NOTRUN -> [FAIL][32] ([i915#579])
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-glk2/igt@i915_pm_rpm@basic-rte.html
* igt@i915_pm_rpm@dpms-mode-unset-non-lpsp:
- shard-tglb: NOTRUN -> [SKIP][33] ([i915#579]) +1 similar issue
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb7/igt@i915_pm_rpm@dpms-mode-unset-non-lpsp.html
* igt@i915_pm_rpm@gem-execbuf-stress-pc8:
- shard-iclb: NOTRUN -> [SKIP][34] ([i915#579]) +2 similar issues
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb1/igt@i915_pm_rpm@gem-execbuf-stress-pc8.html
* igt@i915_selftest@live@hangcheck:
- shard-snb: NOTRUN -> [INCOMPLETE][35] ([i915#2782])
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-snb2/igt@i915_selftest@live@hangcheck.html
* igt@i915_suspend@sysfs-reader:
- shard-kbl: [PASS][36] -> [DMESG-WARN][37] ([i915#180])
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-kbl6/igt@i915_suspend@sysfs-reader.html
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl7/igt@i915_suspend@sysfs-reader.html
* igt@kms_big_fb@linear-8bpp-rotate-90:
- shard-tglb: NOTRUN -> [SKIP][38] ([fdo#111614])
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb7/igt@kms_big_fb@linear-8bpp-rotate-90.html
- shard-iclb: NOTRUN -> [SKIP][39] ([fdo#110725] / [fdo#111614])
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb1/igt@kms_big_fb@linear-8bpp-rotate-90.html
* igt@kms_big_fb@x-tiled-max-hw-stride-32bpp-rotate-180-hflip:
- shard-apl: NOTRUN -> [SKIP][40] ([fdo#109271] / [i915#3777]) +2 similar issues
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-apl6/igt@kms_big_fb@x-tiled-max-hw-stride-32bpp-rotate-180-hflip.html
* igt@kms_big_fb@x-tiled-max-hw-stride-64bpp-rotate-0-hflip:
- shard-kbl: NOTRUN -> [SKIP][41] ([fdo#109271] / [i915#3777])
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl3/igt@kms_big_fb@x-tiled-max-hw-stride-64bpp-rotate-0-hflip.html
- shard-glk: NOTRUN -> [SKIP][42] ([fdo#109271] / [i915#3777])
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-glk8/igt@kms_big_fb@x-tiled-max-hw-stride-64bpp-rotate-0-hflip.html
* igt@kms_big_fb@y-tiled-32bpp-rotate-0:
- shard-glk: [PASS][43] -> [DMESG-WARN][44] ([i915#118] / [i915#95])
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-glk7/igt@kms_big_fb@y-tiled-32bpp-rotate-0.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-glk2/igt@kms_big_fb@y-tiled-32bpp-rotate-0.html
* igt@kms_big_fb@yf-tiled-max-hw-stride-64bpp-rotate-0-async-flip:
- shard-iclb: NOTRUN -> [SKIP][45] ([fdo#110723]) +2 similar issues
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb3/igt@kms_big_fb@yf-tiled-max-hw-stride-64bpp-rotate-0-async-flip.html
* igt@kms_ccs@pipe-b-crc-primary-basic-yf_tiled_ccs:
- shard-tglb: NOTRUN -> [SKIP][46] ([i915#3689]) +4 similar issues
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb8/igt@kms_ccs@pipe-b-crc-primary-basic-yf_tiled_ccs.html
* igt@kms_chamelium@hdmi-crc-nonplanar-formats:
- shard-glk: NOTRUN -> [SKIP][47] ([fdo#109271] / [fdo#111827]) +8 similar issues
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-glk3/igt@kms_chamelium@hdmi-crc-nonplanar-formats.html
* igt@kms_chamelium@vga-edid-read:
- shard-apl: NOTRUN -> [SKIP][48] ([fdo#109271] / [fdo#111827]) +23 similar issues
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-apl7/igt@kms_chamelium@vga-edid-read.html
* igt@kms_chamelium@vga-hpd-for-each-pipe:
- shard-kbl: NOTRUN -> [SKIP][49] ([fdo#109271] / [fdo#111827]) +13 similar issues
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl1/igt@kms_chamelium@vga-hpd-for-each-pipe.html
* igt@kms_color@pipe-d-degamma:
- shard-iclb: NOTRUN -> [SKIP][50] ([fdo#109278] / [i915#1149])
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb6/igt@kms_color@pipe-d-degamma.html
* igt@kms_color_chamelium@pipe-b-ctm-red-to-blue:
- shard-iclb: NOTRUN -> [SKIP][51] ([fdo#109284] / [fdo#111827]) +6 similar issues
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb5/igt@kms_color_chamelium@pipe-b-ctm-red-to-blue.html
* igt@kms_color_chamelium@pipe-d-ctm-red-to-blue:
- shard-tglb: NOTRUN -> [SKIP][52] ([fdo#109284] / [fdo#111827]) +4 similar issues
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb6/igt@kms_color_chamelium@pipe-d-ctm-red-to-blue.html
- shard-iclb: NOTRUN -> [SKIP][53] ([fdo#109278] / [fdo#109284] / [fdo#111827])
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb7/igt@kms_color_chamelium@pipe-d-ctm-red-to-blue.html
* igt@kms_color_chamelium@pipe-invalid-ctm-matrix-sizes:
- shard-snb: NOTRUN -> [SKIP][54] ([fdo#109271] / [fdo#111827]) +19 similar issues
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-snb7/igt@kms_color_chamelium@pipe-invalid-ctm-matrix-sizes.html
* igt@kms_content_protection@dp-mst-type-1:
- shard-tglb: NOTRUN -> [SKIP][55] ([i915#3116])
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb7/igt@kms_content_protection@dp-mst-type-1.html
* igt@kms_content_protection@legacy:
- shard-kbl: NOTRUN -> [TIMEOUT][56] ([i915#1319])
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl2/igt@kms_content_protection@legacy.html
- shard-apl: NOTRUN -> [TIMEOUT][57] ([i915#1319])
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-apl8/igt@kms_content_protection@legacy.html
* igt@kms_cursor_crc@pipe-a-cursor-suspend:
- shard-kbl: NOTRUN -> [DMESG-WARN][58] ([i915#180])
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl6/igt@kms_cursor_crc@pipe-a-cursor-suspend.html
* igt@kms_cursor_crc@pipe-c-cursor-32x32-rapid-movement:
- shard-tglb: NOTRUN -> [SKIP][59] ([i915#3319])
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb6/igt@kms_cursor_crc@pipe-c-cursor-32x32-rapid-movement.html
* igt@kms_cursor_crc@pipe-c-cursor-512x512-rapid-movement:
- shard-iclb: NOTRUN -> [SKIP][60] ([fdo#109278] / [fdo#109279])
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb8/igt@kms_cursor_crc@pipe-c-cursor-512x512-rapid-movement.html
* igt@kms_cursor_crc@pipe-c-cursor-max-size-sliding:
- shard-tglb: NOTRUN -> [SKIP][61] ([i915#3359]) +3 similar issues
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb6/igt@kms_cursor_crc@pipe-c-cursor-max-size-sliding.html
* igt@kms_cursor_crc@pipe-d-cursor-32x10-sliding:
- shard-iclb: NOTRUN -> [SKIP][62] ([fdo#109278]) +18 similar issues
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb7/igt@kms_cursor_crc@pipe-d-cursor-32x10-sliding.html
* igt@kms_cursor_crc@pipe-d-cursor-512x512-onscreen:
- shard-tglb: NOTRUN -> [SKIP][63] ([fdo#109279] / [i915#3359]) +2 similar issues
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb5/igt@kms_cursor_crc@pipe-d-cursor-512x512-onscreen.html
* igt@kms_cursor_crc@pipe-d-cursor-64x21-random:
- shard-glk: NOTRUN -> [SKIP][64] ([fdo#109271]) +79 similar issues
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-glk2/igt@kms_cursor_crc@pipe-d-cursor-64x21-random.html
* igt@kms_cursor_legacy@cursora-vs-flipb-atomic:
- shard-iclb: NOTRUN -> [SKIP][65] ([fdo#109274] / [fdo#109278]) +2 similar issues
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb3/igt@kms_cursor_legacy@cursora-vs-flipb-atomic.html
* igt@kms_cursor_legacy@pipe-d-torture-bo:
- shard-apl: NOTRUN -> [SKIP][66] ([fdo#109271] / [i915#533]) +3 similar issues
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-apl3/igt@kms_cursor_legacy@pipe-d-torture-bo.html
* igt@kms_dither@fb-8bpc-vs-panel-8bpc@edp-1-pipe-a:
- shard-iclb: NOTRUN -> [SKIP][67] ([i915#3788])
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb2/igt@kms_dither@fb-8bpc-vs-panel-8bpc@edp-1-pipe-a.html
* igt@kms_dp_tiled_display@basic-test-pattern:
- shard-iclb: NOTRUN -> [SKIP][68] ([i915#426])
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb3/igt@kms_dp_tiled_display@basic-test-pattern.html
* igt@kms_fbcon_fbt@fbc-suspend:
- shard-apl: [PASS][69] -> [INCOMPLETE][70] ([i915#180] / [i915#1982])
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-apl3/igt@kms_fbcon_fbt@fbc-suspend.html
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-apl3/igt@kms_fbcon_fbt@fbc-suspend.html
* igt@kms_flip@2x-flip-vs-absolute-wf_vblank-interruptible:
- shard-tglb: NOTRUN -> [SKIP][71] ([fdo#111825]) +12 similar issues
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb6/igt@kms_flip@2x-flip-vs-absolute-wf_vblank-interruptible.html
- shard-iclb: NOTRUN -> [SKIP][72] ([fdo#109274]) +1 similar issue
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb4/igt@kms_flip@2x-flip-vs-absolute-wf_vblank-interruptible.html
* igt@kms_flip@flip-vs-modeset-vs-hang@a-dp1:
- shard-apl: [PASS][73] -> [SKIP][74] ([fdo#109271]) +1 similar issue
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-apl8/igt@kms_flip@flip-vs-modeset-vs-hang@a-dp1.html
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-apl7/igt@kms_flip@flip-vs-modeset-vs-hang@a-dp1.html
* igt@kms_flip@flip-vs-modeset-vs-hang@a-hdmi-a1:
- shard-glk: [PASS][75] -> [SKIP][76] ([fdo#109271]) +3 similar issues
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-glk1/igt@kms_flip@flip-vs-modeset-vs-hang@a-hdmi-a1.html
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-glk2/igt@kms_flip@flip-vs-modeset-vs-hang@a-hdmi-a1.html
* igt@kms_flip@flip-vs-modeset-vs-hang@c-dp1:
- shard-kbl: [PASS][77] -> [SKIP][78] ([fdo#109271]) +2 similar issues
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-kbl3/igt@kms_flip@flip-vs-modeset-vs-hang@c-dp1.html
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl7/igt@kms_flip@flip-vs-modeset-vs-hang@c-dp1.html
* igt@kms_flip@flip-vs-suspend-interruptible@c-dp1:
- shard-apl: NOTRUN -> [DMESG-WARN][79] ([i915#180]) +2 similar issues
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-apl8/igt@kms_flip@flip-vs-suspend-interruptible@c-dp1.html
* igt@kms_flip_tiling@flip-to-yf-tiled:
- shard-tglb: NOTRUN -> [SKIP][80] ([fdo#111615]) +2 similar issues
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb8/igt@kms_flip_tiling@flip-to-yf-tiled.html
* igt@kms_force_connector_basic@force-load-detect:
- shard-tglb: NOTRUN -> [SKIP][81] ([fdo#109285])
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb6/igt@kms_force_connector_basic@force-load-detect.html
* igt@kms_frontbuffer_tracking@psr-2p-primscrn-cur-indfb-draw-render:
- shard-iclb: NOTRUN -> [SKIP][82] ([fdo#109280]) +9 similar issues
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb5/igt@kms_frontbuffer_tracking@psr-2p-primscrn-cur-indfb-draw-render.html
* igt@kms_hdr@static-toggle-suspend:
- shard-tglb: NOTRUN -> [SKIP][83] ([i915#1187])
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb1/igt@kms_hdr@static-toggle-suspend.html
- shard-iclb: NOTRUN -> [SKIP][84] ([i915#1187])
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb8/igt@kms_hdr@static-toggle-suspend.html
* igt@kms_pipe_crc_basic@hang-read-crc-pipe-d:
- shard-kbl: NOTRUN -> [SKIP][85] ([fdo#109271] / [i915#533])
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl3/igt@kms_pipe_crc_basic@hang-read-crc-pipe-d.html
- shard-glk: NOTRUN -> [SKIP][86] ([fdo#109271] / [i915#533])
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-glk8/igt@kms_pipe_crc_basic@hang-read-crc-pipe-d.html
* igt@kms_plane_alpha_blend@pipe-b-constant-alpha-max:
- shard-apl: NOTRUN -> [FAIL][87] ([fdo#108145] / [i915#265]) +2 similar issues
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-apl1/igt@kms_plane_alpha_blend@pipe-b-constant-alpha-max.html
* igt@kms_plane_alpha_blend@pipe-c-alpha-transparent-fb:
- shard-glk: NOTRUN -> [FAIL][88] ([i915#265])
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-glk1/igt@kms_plane_alpha_blend@pipe-c-alpha-transparent-fb.html
- shard-kbl: NOTRUN -> [FAIL][89] ([i915#265])
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl4/igt@kms_plane_alpha_blend@pipe-c-alpha-transparent-fb.html
* igt@kms_prime@basic-crc@first-to-second:
- shard-iclb: NOTRUN -> [SKIP][90] ([i915#1836])
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb6/igt@kms_prime@basic-crc@first-to-second.html
- shard-tglb: NOTRUN -> [SKIP][91] ([i915#1836])
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb3/igt@kms_prime@basic-crc@first-to-second.html
* igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-1:
- shard-kbl: NOTRUN -> [SKIP][92] ([fdo#109271] / [i915#658]) +1 similar issue
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl4/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-1.html
- shard-glk: NOTRUN -> [SKIP][93] ([fdo#109271] / [i915#658])
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-glk2/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-1.html
* igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-3:
- shard-apl: NOTRUN -> [SKIP][94] ([fdo#109271] / [i915#658]) +4 similar issues
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-apl6/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-3.html
* igt@kms_psr@psr2_basic:
- shard-iclb: NOTRUN -> [SKIP][95] ([fdo#109441])
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb6/igt@kms_psr@psr2_basic.html
* igt@kms_psr@psr2_primary_blt:
- shard-tglb: NOTRUN -> [FAIL][96] ([i915#132] / [i915#3467]) +1 similar issue
[96]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb2/igt@kms_psr@psr2_primary_blt.html
* igt@kms_psr@psr2_sprite_plane_move:
- shard-iclb: [PASS][97] -> [SKIP][98] ([fdo#109441])
[97]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-iclb2/igt@kms_psr@psr2_sprite_plane_move.html
[98]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb3/igt@kms_psr@psr2_sprite_plane_move.html
* igt@kms_selftest@all@damage_iter_no_damage:
- shard-snb: NOTRUN -> [INCOMPLETE][99] ([i915#3871])
[99]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-snb7/igt@kms_selftest@all@damage_iter_no_damage.html
- shard-kbl: NOTRUN -> [INCOMPLETE][100] ([i915#3871])
[100]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl3/igt@kms_selftest@all@damage_iter_no_damage.html
* igt@kms_sysfs_edid_timing:
- shard-apl: NOTRUN -> [FAIL][101] ([IGT#2])
[101]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-apl3/igt@kms_sysfs_edid_timing.html
- shard-kbl: NOTRUN -> [FAIL][102] ([IGT#2])
[102]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl4/igt@kms_sysfs_edid_timing.html
* igt@nouveau_crc@pipe-a-ctx-flip-detection:
- shard-iclb: NOTRUN -> [SKIP][103] ([i915#2530])
[103]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb7/igt@nouveau_crc@pipe-a-ctx-flip-detection.html
* igt@nouveau_crc@pipe-b-ctx-flip-skip-current-frame:
- shard-apl: NOTRUN -> [SKIP][104] ([fdo#109271]) +277 similar issues
[104]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-apl6/igt@nouveau_crc@pipe-b-ctx-flip-skip-current-frame.html
* igt@nouveau_crc@pipe-d-ctx-flip-detection:
- shard-tglb: NOTRUN -> [SKIP][105] ([i915#2530]) +1 similar issue
[105]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb5/igt@nouveau_crc@pipe-d-ctx-flip-detection.html
- shard-iclb: NOTRUN -> [SKIP][106] ([fdo#109278] / [i915#2530])
[106]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb5/igt@nouveau_crc@pipe-d-ctx-flip-detection.html
* igt@perf@gen12-mi-rpc:
- shard-iclb: NOTRUN -> [SKIP][107] ([fdo#109289]) +1 similar issue
[107]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb8/igt@perf@gen12-mi-rpc.html
* igt@perf@polling-parameterized:
- shard-glk: [PASS][108] -> [FAIL][109] ([i915#1542])
[108]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-glk3/igt@perf@polling-parameterized.html
[109]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-glk1/igt@perf@polling-parameterized.html
* igt@prime_nv_pcopy@test2:
- shard-kbl: NOTRUN -> [SKIP][110] ([fdo#109271]) +154 similar issues
[110]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl7/igt@prime_nv_pcopy@test2.html
- shard-tglb: NOTRUN -> [SKIP][111] ([fdo#109291])
[111]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb3/igt@prime_nv_pcopy@test2.html
* igt@prime_nv_pcopy@test3_3:
- shard-iclb: NOTRUN -> [SKIP][112] ([fdo#109291]) +1 similar issue
[112]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb6/igt@prime_nv_pcopy@test3_3.html
* igt@sysfs_clients@busy:
- shard-tglb: NOTRUN -> [SKIP][113] ([i915#2994])
[113]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-tglb3/igt@sysfs_clients@busy.html
- shard-iclb: NOTRUN -> [SKIP][114] ([i915#2994])
[114]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb6/igt@sysfs_clients@busy.html
- shard-glk: NOTRUN -> [SKIP][115] ([fdo#109271] / [i915#2994]) +1 similar issue
[115]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-glk4/igt@sysfs_clients@busy.html
* igt@sysfs_clients@recycle:
- shard-apl: NOTRUN -> [SKIP][116] ([fdo#109271] / [i915#2994]) +2 similar issues
[116]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-apl1/igt@sysfs_clients@recycle.html
* igt@sysfs_clients@split-50:
- shard-kbl: NOTRUN -> [SKIP][117] ([fdo#109271] / [i915#2994]) +3 similar issues
[117]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl3/igt@sysfs_clients@split-50.html
#### Possible fixes ####
* igt@gem_eio@in-flight-suspend:
- shard-kbl: [DMESG-WARN][118] ([i915#180]) -> [PASS][119]
[118]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-kbl7/igt@gem_eio@in-flight-suspend.html
[119]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl4/igt@gem_eio@in-flight-suspend.html
* igt@gem_exec_fair@basic-none-share@rcs0:
- shard-iclb: [FAIL][120] ([i915#2842]) -> [PASS][121]
[120]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-iclb1/igt@gem_exec_fair@basic-none-share@rcs0.html
[121]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb2/igt@gem_exec_fair@basic-none-share@rcs0.html
* igt@gem_exec_fair@basic-none-vip@rcs0:
- shard-glk: [FAIL][122] ([i915#2842]) -> [PASS][123]
[122]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-glk1/igt@gem_exec_fair@basic-none-vip@rcs0.html
[123]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-glk5/igt@gem_exec_fair@basic-none-vip@rcs0.html
* igt@gem_exec_fair@basic-pace@vcs1:
- shard-kbl: [FAIL][124] ([i915#2842]) -> [PASS][125]
[124]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-kbl1/igt@gem_exec_fair@basic-pace@vcs1.html
[125]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl6/igt@gem_exec_fair@basic-pace@vcs1.html
* igt@kms_big_fb@linear-32bpp-rotate-0:
- shard-iclb: [DMESG-WARN][126] ([i915#3621]) -> [PASS][127]
[126]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-iclb1/igt@kms_big_fb@linear-32bpp-rotate-0.html
[127]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb2/igt@kms_big_fb@linear-32bpp-rotate-0.html
- shard-glk: [DMESG-WARN][128] ([i915#118] / [i915#95]) -> [PASS][129] +1 similar issue
[128]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-glk7/igt@kms_big_fb@linear-32bpp-rotate-0.html
[129]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-glk1/igt@kms_big_fb@linear-32bpp-rotate-0.html
* igt@kms_cursor_crc@pipe-b-cursor-256x85-onscreen:
- shard-glk: [FAIL][130] ([i915#3444]) -> [PASS][131]
[130]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-glk6/igt@kms_cursor_crc@pipe-b-cursor-256x85-onscreen.html
[131]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-glk4/igt@kms_cursor_crc@pipe-b-cursor-256x85-onscreen.html
- shard-kbl: [FAIL][132] ([i915#3444]) -> [PASS][133]
[132]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-kbl6/igt@kms_cursor_crc@pipe-b-cursor-256x85-onscreen.html
[133]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl7/igt@kms_cursor_crc@pipe-b-cursor-256x85-onscreen.html
* igt@kms_flip@flip-vs-expired-vblank-interruptible@a-hdmi-a1:
- shard-glk: [FAIL][134] ([i915#79]) -> [PASS][135]
[134]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-glk7/igt@kms_flip@flip-vs-expired-vblank-interruptible@a-hdmi-a1.html
[135]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-glk3/igt@kms_flip@flip-vs-expired-vblank-interruptible@a-hdmi-a1.html
* igt@kms_psr@psr2_cursor_render:
- shard-iclb: [SKIP][136] ([fdo#109441]) -> [PASS][137]
[136]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-iclb6/igt@kms_psr@psr2_cursor_render.html
[137]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb2/igt@kms_psr@psr2_cursor_render.html
* igt@perf_pmu@rc6-suspend:
- shard-kbl: [INCOMPLETE][138] ([i915#155] / [i915#794]) -> [PASS][139]
[138]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-kbl4/igt@perf_pmu@rc6-suspend.html
[139]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-kbl2/igt@perf_pmu@rc6-suspend.html
* igt@prime_vgem@sync@rcs0:
- shard-iclb: [INCOMPLETE][140] ([i915#409]) -> [PASS][141]
[140]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-iclb4/igt@prime_vgem@sync@rcs0.html
[141]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/shard-iclb3/igt@prime_vgem@sync@rcs0.html
#### Warnings ####
* igt@gem_exec_fair@basic-pace@rcs0:
- shard-tglb: [FAIL][142] ([i915#2876]) -> [FAIL][143] ([i915#2842])
[142]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10413/shard-tgl
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6072/index.html
[-- Attachment #1.2: Type: text/html, Size: 33932 bytes --]
[-- Attachment #2: Type: text/plain, Size: 154 bytes --]
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry
2021-07-28 17:16 [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry Mark Yacoub
` (2 preceding siblings ...)
2021-07-29 1:07 ` [igt-dev] ✓ Fi.CI.IGT: success " Patchwork
@ 2021-08-26 12:55 ` Rodrigo Siqueira
2021-08-27 5:47 ` Petri Latvala
2021-08-27 7:52 ` [igt-dev] ✗ Fi.CI.BAT: failure for tests/kms_flip: Discard any stale event at each retry (rev2) Patchwork
` (4 subsequent siblings)
8 siblings, 1 reply; 16+ messages in thread
From: Rodrigo Siqueira @ 2021-08-26 12:55 UTC (permalink / raw)
To: Mark Yacoub, Lakshminarayana Vudum
Cc: igt-dev, robdclark, petri.latvala, seanpaul, Mark Yacoub
Hi Lakshminarayana,
I noticed that the CI pipeline for GitLab failed in this patch. I look
at the log, and I think this failed due to CI instabilities. Could you
trigger the CI again for this patch?
Thanks
Siqueira
On 07/28, Mark Yacoub wrote:
> From: Mark Yacoub <markyacoub@google.com>
>
> [Why]
> On slower devices, non blocking do_page_flip followed by wait_for_events does not finish fast enough to send an event to be consumed by wait_for_events.
> Instead, wait_for_events ends up finding an unconsumed event from a
> previous do_page_flip (i.e. a skipped preceding subtest) and signals
> that the userspace is ready to do another do_page_flip while the first
> one hasn't finished yet, resulting in an EBUSY.
>
> [How]
> At the beginning of every test running on crtc set, check for any stale
> events and consume them so the kernel has no pending events.
>
> Fixes: flip-vs-suspend and flip-vs-suspend-interruptible.
> Tested on ChromeOS Volteer(i915-TGL) and Trogdor(msm)
>
> Signed-off-by: Mark Yacoub <markyacoub@chromium.org>
> Change-Id: I254a84a202140c332312aba1d9edf15ea78b7a61
> ---
> tests/kms_flip.c | 23 +++++++++++++++++++++++
> 1 file changed, 23 insertions(+)
>
> diff --git a/tests/kms_flip.c b/tests/kms_flip.c
> index f2fce8d2..0a2041c5 100755
> --- a/tests/kms_flip.c
> +++ b/tests/kms_flip.c
> @@ -1261,6 +1261,26 @@ static bool needs_retry_after_link_reset(struct udev_monitor *mon)
> return hotplug_detected;
> }
>
> +static void discard_any_stale_events() {
> + fd_set fds;
> + FD_ZERO(&fds);
> + FD_SET(drm_fd, &fds);
> + struct timeval timeout = { .tv_sec = 3, .tv_usec = 0 };
> + int ret = select(drm_fd + 1, &fds, NULL, NULL, &timeout);
> +
> + if (ret > 0) {
> + drmEventContext evctx;
> + memset(&evctx, 0, sizeof evctx);
> + evctx.version = 2;
> + igt_info("Stale Event found - Discarding now\n");
> + drmHandleEvent(drm_fd, &evctx);
> + }
> + else {
> + igt_debug("No stale Event found\n");
> + }
> +}
> +
> +
> static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
> int crtc_count, int duration_ms)
> {
> @@ -1311,6 +1331,9 @@ static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
> kmstest_dump_mode(&o->kmode[i]);
>
> retry:
> + // Discard any pending event that hasn't been consumed from a previous retry or subtest.
> + discard_any_stale_events();
> +
> memset(&o->vblank_state, 0, sizeof(o->vblank_state));
> memset(&o->flip_state, 0, sizeof(o->flip_state));
> o->flip_state.name = "flip";
> --
> 2.32.0.432.gabb21c7263-goog
>
> _______________________________________________
> igt-dev mailing list
> igt-dev@lists.freedesktop.org
> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Figt-dev&data=04%7C01%7CRodrigo.Siqueira%40amd.com%7C5bc54d07bd7b47a66e0d08d951eb6c94%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637630893864077242%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=nqwaQ2ruIdx27uUE3kXgGGf5LcJaO7w0GmfSyoHOHP8%3D&reserved=0
--
Rodrigo Siqueira
https://siqueira.tech
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry
2021-08-26 12:55 ` [igt-dev] [PATCH] " Rodrigo Siqueira
@ 2021-08-27 5:47 ` Petri Latvala
0 siblings, 0 replies; 16+ messages in thread
From: Petri Latvala @ 2021-08-27 5:47 UTC (permalink / raw)
To: Rodrigo Siqueira
Cc: Mark Yacoub, Lakshminarayana Vudum, igt-dev, robdclark, seanpaul,
Mark Yacoub
On Thu, Aug 26, 2021 at 08:55:16AM -0400, Rodrigo Siqueira wrote:
> Hi Lakshminarayana,
>
> I noticed that the CI pipeline for GitLab failed in this patch. I look
> at the log, and I think this failed due to CI instabilities. Could you
> trigger the CI again for this patch?
Triggered a full test round. Repeating just the gitlab parts would
just repeat with the timeouting setup.
--
Petri Latvala
>
> Thanks
> Siqueira
>
> On 07/28, Mark Yacoub wrote:
> > From: Mark Yacoub <markyacoub@google.com>
> >
> > [Why]
> > On slower devices, non blocking do_page_flip followed by wait_for_events does not finish fast enough to send an event to be consumed by wait_for_events.
> > Instead, wait_for_events ends up finding an unconsumed event from a
> > previous do_page_flip (i.e. a skipped preceding subtest) and signals
> > that the userspace is ready to do another do_page_flip while the first
> > one hasn't finished yet, resulting in an EBUSY.
> >
> > [How]
> > At the beginning of every test running on crtc set, check for any stale
> > events and consume them so the kernel has no pending events.
> >
> > Fixes: flip-vs-suspend and flip-vs-suspend-interruptible.
> > Tested on ChromeOS Volteer(i915-TGL) and Trogdor(msm)
> >
> > Signed-off-by: Mark Yacoub <markyacoub@chromium.org>
> > Change-Id: I254a84a202140c332312aba1d9edf15ea78b7a61
> > ---
> > tests/kms_flip.c | 23 +++++++++++++++++++++++
> > 1 file changed, 23 insertions(+)
> >
> > diff --git a/tests/kms_flip.c b/tests/kms_flip.c
> > index f2fce8d2..0a2041c5 100755
> > --- a/tests/kms_flip.c
> > +++ b/tests/kms_flip.c
> > @@ -1261,6 +1261,26 @@ static bool needs_retry_after_link_reset(struct udev_monitor *mon)
> > return hotplug_detected;
> > }
> >
> > +static void discard_any_stale_events() {
> > + fd_set fds;
> > + FD_ZERO(&fds);
> > + FD_SET(drm_fd, &fds);
> > + struct timeval timeout = { .tv_sec = 3, .tv_usec = 0 };
> > + int ret = select(drm_fd + 1, &fds, NULL, NULL, &timeout);
> > +
> > + if (ret > 0) {
> > + drmEventContext evctx;
> > + memset(&evctx, 0, sizeof evctx);
> > + evctx.version = 2;
> > + igt_info("Stale Event found - Discarding now\n");
> > + drmHandleEvent(drm_fd, &evctx);
> > + }
> > + else {
> > + igt_debug("No stale Event found\n");
> > + }
> > +}
> > +
> > +
> > static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
> > int crtc_count, int duration_ms)
> > {
> > @@ -1311,6 +1331,9 @@ static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
> > kmstest_dump_mode(&o->kmode[i]);
> >
> > retry:
> > + // Discard any pending event that hasn't been consumed from a previous retry or subtest.
> > + discard_any_stale_events();
> > +
> > memset(&o->vblank_state, 0, sizeof(o->vblank_state));
> > memset(&o->flip_state, 0, sizeof(o->flip_state));
> > o->flip_state.name = "flip";
> > --
> > 2.32.0.432.gabb21c7263-goog
> >
> > _______________________________________________
> > igt-dev mailing list
> > igt-dev@lists.freedesktop.org
> > https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Figt-dev&data=04%7C01%7CRodrigo.Siqueira%40amd.com%7C5bc54d07bd7b47a66e0d08d951eb6c94%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637630893864077242%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=nqwaQ2ruIdx27uUE3kXgGGf5LcJaO7w0GmfSyoHOHP8%3D&reserved=0
>
> --
> Rodrigo Siqueira
> https://siqueira.tech
^ permalink raw reply [flat|nested] 16+ messages in thread
* [igt-dev] ✗ Fi.CI.BAT: failure for tests/kms_flip: Discard any stale event at each retry (rev2)
2021-07-28 17:16 [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry Mark Yacoub
` (3 preceding siblings ...)
2021-08-26 12:55 ` [igt-dev] [PATCH] " Rodrigo Siqueira
@ 2021-08-27 7:52 ` Patchwork
2021-09-01 19:06 ` [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry Arkadiusz Hiler
` (3 subsequent siblings)
8 siblings, 0 replies; 16+ messages in thread
From: Patchwork @ 2021-08-27 7:52 UTC (permalink / raw)
To: Mark Yacoub; +Cc: igt-dev
[-- Attachment #1: Type: text/plain, Size: 3399 bytes --]
== Series Details ==
Series: tests/kms_flip: Discard any stale event at each retry (rev2)
URL : https://patchwork.freedesktop.org/series/93133/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_10528 -> IGTPW_6165
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with IGTPW_6165 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in IGTPW_6165, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6165/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in IGTPW_6165:
### IGT changes ###
#### Possible regressions ####
* igt@kms_flip@basic-flip-vs-modeset:
- fi-rkl-11600: NOTRUN -> [SKIP][1]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6165/fi-rkl-11600/igt@kms_flip@basic-flip-vs-modeset.html
* igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-b:
- fi-rkl-guc: [PASS][2] -> [SKIP][3] +1 similar issue
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10528/fi-rkl-guc/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-b.html
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6165/fi-rkl-guc/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-b.html
Known issues
------------
Here are the changes found in IGTPW_6165 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_selftest@live@execlists:
- fi-bsw-kefka: [PASS][4] -> [INCOMPLETE][5] ([i915#2940])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10528/fi-bsw-kefka/igt@i915_selftest@live@execlists.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6165/fi-bsw-kefka/igt@i915_selftest@live@execlists.html
* igt@i915_selftest@live@gt_lrc:
- fi-rkl-guc: [PASS][6] -> [DMESG-WARN][7] ([i915#3958])
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10528/fi-rkl-guc/igt@i915_selftest@live@gt_lrc.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6165/fi-rkl-guc/igt@i915_selftest@live@gt_lrc.html
* igt@runner@aborted:
- fi-bsw-kefka: NOTRUN -> [FAIL][8] ([fdo#109271] / [i915#1436])
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6165/fi-bsw-kefka/igt@runner@aborted.html
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[i915#1436]: https://gitlab.freedesktop.org/drm/intel/issues/1436
[i915#2940]: https://gitlab.freedesktop.org/drm/intel/issues/2940
[i915#3958]: https://gitlab.freedesktop.org/drm/intel/issues/3958
Participating hosts (39 -> 33)
------------------------------
Missing (6): fi-ilk-m540 bat-adls-5 fi-tgl-1115g4 fi-bsw-cyan fi-bdw-samus bat-jsl-1
Build changes
-------------
* CI: CI-20190529 -> None
* IGT: IGT_6187 -> IGTPW_6165
CI-20190529: 20190529
CI_DRM_10528: c0d72b2775df87cc8a852e5be38e9d84e7e42a05 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_6165: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6165/index.html
IGT_6187: 1afd52c1471dafdf521eae431f3e228826de6de2 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6165/index.html
[-- Attachment #2: Type: text/html, Size: 4126 bytes --]
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry
2021-07-28 17:16 [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry Mark Yacoub
` (4 preceding siblings ...)
2021-08-27 7:52 ` [igt-dev] ✗ Fi.CI.BAT: failure for tests/kms_flip: Discard any stale event at each retry (rev2) Patchwork
@ 2021-09-01 19:06 ` Arkadiusz Hiler
2021-09-01 19:21 ` Mark Yacoub
2021-09-24 18:39 ` [igt-dev] [PATCH v2] " Mark Yacoub
` (2 subsequent siblings)
8 siblings, 1 reply; 16+ messages in thread
From: Arkadiusz Hiler @ 2021-09-01 19:06 UTC (permalink / raw)
To: Mark Yacoub
Cc: igt-dev, seanpaul, petri.latvala, abhinavk, robdclark, Mark Yacoub
On Wed, Jul 28, 2021 at 01:16:16PM -0400, Mark Yacoub wrote:
> From: Mark Yacoub <markyacoub@google.com>
>
> [Why]
> On slower devices, non blocking do_page_flip followed by wait_for_events does not finish fast enough to send an event to be consumed by wait_for_events.
This sentence should be wrapped like the rest of the commit message.
> Instead, wait_for_events ends up finding an unconsumed event from a
> previous do_page_flip (i.e. a skipped preceding subtest) and signals
> that the userspace is ready to do another do_page_flip while the first
> one hasn't finished yet, resulting in an EBUSY.
>
> [How]
> At the beginning of every test running on crtc set, check for any stale
> events and consume them so the kernel has no pending events.
I am not opposed to this approach, it's nice to know that we can trust
the state as the test start, but I wonder where and how do we leak those
pending flips? Have you looked into where those are coming from?
> Fixes: flip-vs-suspend and flip-vs-suspend-interruptible.
> Tested on ChromeOS Volteer(i915-TGL) and Trogdor(msm)
>
> Signed-off-by: Mark Yacoub <markyacoub@chromium.org>
> Change-Id: I254a84a202140c332312aba1d9edf15ea78b7a61
It would be nice to not include your internal gerrit's Change-Id
(or discard them upon commit). Those does not mean anything in the
upstream.
> ---
> tests/kms_flip.c | 23 +++++++++++++++++++++++
> 1 file changed, 23 insertions(+)
>
> diff --git a/tests/kms_flip.c b/tests/kms_flip.c
> index f2fce8d2..0a2041c5 100755
> --- a/tests/kms_flip.c
> +++ b/tests/kms_flip.c
> @@ -1261,6 +1261,26 @@ static bool needs_retry_after_link_reset(struct udev_monitor *mon)
> return hotplug_detected;
> }
>
> +static void discard_any_stale_events() {
> + fd_set fds;
> + FD_ZERO(&fds);
> + FD_SET(drm_fd, &fds);
> + struct timeval timeout = { .tv_sec = 3, .tv_usec = 0 };
Is the 3s timeout necessary? I can imagine that in a lot of cases this
will end up being sleep(3), as we don't expect any stale events in a
normal case. That's a significant time increase if you count all the
invocations.
> + int ret = select(drm_fd + 1, &fds, NULL, NULL, &timeout);
> +
> + if (ret > 0) {
> + drmEventContext evctx;
> + memset(&evctx, 0, sizeof evctx);
> + evctx.version = 2;
> + igt_info("Stale Event found - Discarding now\n");
> + drmHandleEvent(drm_fd, &evctx);
> + }
> + else {
> + igt_debug("No stale Event found\n");
> + }
> +}
I think it should plural (and lowercase) "events" as it will go through
all the pending events on that fd.
> +
> +
> static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
> int crtc_count, int duration_ms)
> {
> @@ -1311,6 +1331,9 @@ static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
> kmstest_dump_mode(&o->kmode[i]);
>
> retry:
> + // Discard any pending event that hasn't been consumed from a previous retry or subtest.
Generally C-style comments (/* bla */) are preferred, but I don't think
IGT or the kernel coding style explicitly mentions that. Also there are
occurrences of C++-style comments in the code already. Oh well...
All the cosmetic nitpicks can be fixed by me or another committer when
merging the patch (assuming that you are ok with them), so there's not
need to resend unless you want to make changes to the actual code.
Cheers,
Arek
> + discard_any_stale_events();
> +
> memset(&o->vblank_state, 0, sizeof(o->vblank_state));
> memset(&o->flip_state, 0, sizeof(o->flip_state));
> o->flip_state.name = "flip";
> --
> 2.32.0.432.gabb21c7263-goog
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry
2021-09-01 19:06 ` [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry Arkadiusz Hiler
@ 2021-09-01 19:21 ` Mark Yacoub
2021-09-01 19:44 ` Arkadiusz Hiler
0 siblings, 1 reply; 16+ messages in thread
From: Mark Yacoub @ 2021-09-01 19:21 UTC (permalink / raw)
To: Arkadiusz Hiler
Cc: Development mailing list for IGT GPU Tools, Sean Paul,
Petri Latvala, abhinavk, Rob Clark, Mark Yacoub
On Wed, Sep 1, 2021 at 3:06 PM Arkadiusz Hiler <arek@hiler.eu> wrote:
>
> On Wed, Jul 28, 2021 at 01:16:16PM -0400, Mark Yacoub wrote:
> > From: Mark Yacoub <markyacoub@google.com>
> >
> > [Why]
> > On slower devices, non blocking do_page_flip followed by wait_for_events does not finish fast enough to send an event to be consumed by wait_for_events.
>
> This sentence should be wrapped like the rest of the commit message.
>
> > Instead, wait_for_events ends up finding an unconsumed event from a
> > previous do_page_flip (i.e. a skipped preceding subtest) and signals
> > that the userspace is ready to do another do_page_flip while the first
> > one hasn't finished yet, resulting in an EBUSY.
> >
> > [How]
> > At the beginning of every test running on crtc set, check for any stale
> > events and consume them so the kernel has no pending events.
>
> I am not opposed to this approach, it's nice to know that we can trust
> the state as the test start, but I wonder where and how do we leak those
> pending flips? Have you looked into where those are coming from?
Yeah I spent a fair amount of time debugging this but it's been a
month so I vaguely remember it was cause the first subtest doesn't
finish all the way to the end (can't remember if it was an assertion
or requirement) so it ends early but with an event that's not been
consumed, then comes the second subtest working on top of an
unconsumed event. The bug exists in all devices, but they're probably
fast enough to not run into this case while my test device was slow
and suffered from this.
If you need it, I can debug the code again and report back exactly
what is happening.
>
> > Fixes: flip-vs-suspend and flip-vs-suspend-interruptible.
> > Tested on ChromeOS Volteer(i915-TGL) and Trogdor(msm)
> >
> > Signed-off-by: Mark Yacoub <markyacoub@chromium.org>
> > Change-Id: I254a84a202140c332312aba1d9edf15ea78b7a61
>
> It would be nice to not include your internal gerrit's Change-Id
> (or discard them upon commit). Those does not mean anything in the
> upstream.
My bad. I added a script that will prevent me from sending a patch if
it has the gerrit ID.
>
> > ---
> > tests/kms_flip.c | 23 +++++++++++++++++++++++
> > 1 file changed, 23 insertions(+)
> >
> > diff --git a/tests/kms_flip.c b/tests/kms_flip.c
> > index f2fce8d2..0a2041c5 100755
> > --- a/tests/kms_flip.c
> > +++ b/tests/kms_flip.c
> > @@ -1261,6 +1261,26 @@ static bool needs_retry_after_link_reset(struct udev_monitor *mon)
> > return hotplug_detected;
> > }
> >
> > +static void discard_any_stale_events() {
> > + fd_set fds;
> > + FD_ZERO(&fds);
> > + FD_SET(drm_fd, &fds);
> > + struct timeval timeout = { .tv_sec = 3, .tv_usec = 0 };
>
> Is the 3s timeout necessary? I can imagine that in a lot of cases this
> will end up being sleep(3), as we don't expect any stale events in a
> normal case. That's a significant time increase if you count all the
> invocations.
>
I don't have a strong reason besides that I used the same
implementation in wait_for_events() which uses 3 seconds for timeout.
> > + int ret = select(drm_fd + 1, &fds, NULL, NULL, &timeout);
> > +
> > + if (ret > 0) {
> > + drmEventContext evctx;
> > + memset(&evctx, 0, sizeof evctx);
> > + evctx.version = 2;
> > + igt_info("Stale Event found - Discarding now\n");
> > + drmHandleEvent(drm_fd, &evctx);
> > + }
> > + else {
> > + igt_debug("No stale Event found\n");
> > + }
> > +}
>
> I think it should plural (and lowercase) "events" as it will go through
> all the pending events on that fd.
>
> > +
> > +
> > static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
> > int crtc_count, int duration_ms)
> > {
> > @@ -1311,6 +1331,9 @@ static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
> > kmstest_dump_mode(&o->kmode[i]);
> >
> > retry:
> > + // Discard any pending event that hasn't been consumed from a previous retry or subtest.
>
> Generally C-style comments (/* bla */) are preferred, but I don't think
> IGT or the kernel coding style explicitly mentions that. Also there are
> occurrences of C++-style comments in the code already. Oh well...
>
Sorry, it's just a habit, my bad.
> All the cosmetic nitpicks can be fixed by me or another committer when
> merging the patch (assuming that you are ok with them), so there's not
> need to resend unless you want to make changes to the actual code.
>
> Cheers,
> Arek
>
Thanks for reviewing!
> > + discard_any_stale_events();
> > +
> > memset(&o->vblank_state, 0, sizeof(o->vblank_state));
> > memset(&o->flip_state, 0, sizeof(o->flip_state));
> > o->flip_state.name = "flip";
> > --
> > 2.32.0.432.gabb21c7263-goog
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry
2021-09-01 19:21 ` Mark Yacoub
@ 2021-09-01 19:44 ` Arkadiusz Hiler
2021-09-24 18:31 ` Mark Yacoub
0 siblings, 1 reply; 16+ messages in thread
From: Arkadiusz Hiler @ 2021-09-01 19:44 UTC (permalink / raw)
To: Mark Yacoub
Cc: Development mailing list for IGT GPU Tools, Sean Paul,
Petri Latvala, abhinavk, Rob Clark, Mark Yacoub
On Wed, Sep 01, 2021 at 03:21:02PM -0400, Mark Yacoub wrote:
> On Wed, Sep 1, 2021 at 3:06 PM Arkadiusz Hiler <arek@hiler.eu> wrote:
> >
> > On Wed, Jul 28, 2021 at 01:16:16PM -0400, Mark Yacoub wrote:
> > > From: Mark Yacoub <markyacoub@google.com>
> > >
> > > [Why]
> > > On slower devices, non blocking do_page_flip followed by wait_for_events does not finish fast enough to send an event to be consumed by wait_for_events.
> >
> > This sentence should be wrapped like the rest of the commit message.
> >
> > > Instead, wait_for_events ends up finding an unconsumed event from a
> > > previous do_page_flip (i.e. a skipped preceding subtest) and signals
> > > that the userspace is ready to do another do_page_flip while the first
> > > one hasn't finished yet, resulting in an EBUSY.
> > >
> > > [How]
> > > At the beginning of every test running on crtc set, check for any stale
> > > events and consume them so the kernel has no pending events.
> >
> > I am not opposed to this approach, it's nice to know that we can trust
> > the state as the test start, but I wonder where and how do we leak those
> > pending flips? Have you looked into where those are coming from?
> Yeah I spent a fair amount of time debugging this but it's been a
> month so I vaguely remember it was cause the first subtest doesn't
> finish all the way to the end (can't remember if it was an assertion
> or requirement) so it ends early but with an event that's not been
> consumed, then comes the second subtest working on top of an
> unconsumed event. The bug exists in all devices, but they're probably
> fast enough to not run into this case while my test device was slow
> and suffered from this.
> If you need it, I can debug the code again and report back exactly
> what is happening.
I would appreciate that. I think it may be better here to make sure that
we are not leaking that flip in the first place.
> > > Fixes: flip-vs-suspend and flip-vs-suspend-interruptible.
> > > Tested on ChromeOS Volteer(i915-TGL) and Trogdor(msm)
> > >
> > > Signed-off-by: Mark Yacoub <markyacoub@chromium.org>
> > > Change-Id: I254a84a202140c332312aba1d9edf15ea78b7a61
> >
> > It would be nice to not include your internal gerrit's Change-Id
> > (or discard them upon commit). Those does not mean anything in the
> > upstream.
> My bad. I added a script that will prevent me from sending a patch if
> it has the gerrit ID.
> >
> > > ---
> > > tests/kms_flip.c | 23 +++++++++++++++++++++++
> > > 1 file changed, 23 insertions(+)
> > >
> > > diff --git a/tests/kms_flip.c b/tests/kms_flip.c
> > > index f2fce8d2..0a2041c5 100755
> > > --- a/tests/kms_flip.c
> > > +++ b/tests/kms_flip.c
> > > @@ -1261,6 +1261,26 @@ static bool needs_retry_after_link_reset(struct udev_monitor *mon)
> > > return hotplug_detected;
> > > }
> > >
> > > +static void discard_any_stale_events() {
> > > + fd_set fds;
> > > + FD_ZERO(&fds);
> > > + FD_SET(drm_fd, &fds);
> > > + struct timeval timeout = { .tv_sec = 3, .tv_usec = 0 };
> >
> > Is the 3s timeout necessary? I can imagine that in a lot of cases this
> > will end up being sleep(3), as we don't expect any stale events in a
> > normal case. That's a significant time increase if you count all the
> > invocations.
> >
> I don't have a strong reason besides that I used the same
> implementation in wait_for_events() which uses 3 seconds for timeout.
wait_for_events() is justified in having that 3s timeout - it is only
called when we just did something and we expect it to produce events.
We should never really spend the full 3s waiting.
In places where you want to call discard_any_stale_events() there should
be no pending events, so we would end up waiting those 3s until timeout
almost every time.
That's a significant difference.
I wonder if checking without any wait would be sufficient. I am a bit
afraid that may be too immediate for some of the slower devices.
> > > + int ret = select(drm_fd + 1, &fds, NULL, NULL, &timeout);
> > > +
> > > + if (ret > 0) {
> > > + drmEventContext evctx;
> > > + memset(&evctx, 0, sizeof evctx);
> > > + evctx.version = 2;
> > > + igt_info("Stale Event found - Discarding now\n");
> > > + drmHandleEvent(drm_fd, &evctx);
> > > + }
> > > + else {
> > > + igt_debug("No stale Event found\n");
> > > + }
> > > +}
> >
> > I think it should plural (and lowercase) "events" as it will go through
> > all the pending events on that fd.
> >
> > > +
> > > +
> > > static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
> > > int crtc_count, int duration_ms)
> > > {
> > > @@ -1311,6 +1331,9 @@ static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
> > > kmstest_dump_mode(&o->kmode[i]);
> > >
> > > retry:
> > > + // Discard any pending event that hasn't been consumed from a previous retry or subtest.
> >
> > Generally C-style comments (/* bla */) are preferred, but I don't think
> > IGT or the kernel coding style explicitly mentions that. Also there are
> > occurrences of C++-style comments in the code already. Oh well...
> >
> Sorry, it's just a habit, my bad.
No need to be sorry, code conventions are completetly arbitrary and we
don't really provide tools (you can theoretically use checkpatch.pl from
the kernel) to make it easy on people. This "oh well" was just me taking
a note that we don't even have "the comments rule" codified and that
there are examples of not following it in the IGT already, so expecting
people to follow it and enforcing it is not really fair.
Cheers,
Arek
> > All the cosmetic nitpicks can be fixed by me or another committer when
> > merging the patch (assuming that you are ok with them), so there's not
> > need to resend unless you want to make changes to the actual code.
> >
> > Cheers,
> > Arek
> >
> Thanks for reviewing!
> > > + discard_any_stale_events();
> > > +
> > > memset(&o->vblank_state, 0, sizeof(o->vblank_state));
> > > memset(&o->flip_state, 0, sizeof(o->flip_state));
> > > o->flip_state.name = "flip";
> > > --
> > > 2.32.0.432.gabb21c7263-goog
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry
2021-09-01 19:44 ` Arkadiusz Hiler
@ 2021-09-24 18:31 ` Mark Yacoub
0 siblings, 0 replies; 16+ messages in thread
From: Mark Yacoub @ 2021-09-24 18:31 UTC (permalink / raw)
To: Arkadiusz Hiler
Cc: Development mailing list for IGT GPU Tools, Sean Paul,
Petri Latvala, abhinavk, Rob Clark, Mark Yacoub
On Wed, Sep 1, 2021 at 3:45 PM Arkadiusz Hiler <arek@hiler.eu> wrote:
>
> On Wed, Sep 01, 2021 at 03:21:02PM -0400, Mark Yacoub wrote:
> > On Wed, Sep 1, 2021 at 3:06 PM Arkadiusz Hiler <arek@hiler.eu> wrote:
> > >
> > > On Wed, Jul 28, 2021 at 01:16:16PM -0400, Mark Yacoub wrote:
> > > > From: Mark Yacoub <markyacoub@google.com>
> > > >
> > > > [Why]
> > > > On slower devices, non blocking do_page_flip followed by wait_for_events does not finish fast enough to send an event to be consumed by wait_for_events.
> > >
> > > This sentence should be wrapped like the rest of the commit message.
> > >
> > > > Instead, wait_for_events ends up finding an unconsumed event from a
> > > > previous do_page_flip (i.e. a skipped preceding subtest) and signals
> > > > that the userspace is ready to do another do_page_flip while the first
> > > > one hasn't finished yet, resulting in an EBUSY.
> > > >
> > > > [How]
> > > > At the beginning of every test running on crtc set, check for any stale
> > > > events and consume them so the kernel has no pending events.
> > >
> > > I am not opposed to this approach, it's nice to know that we can trust
> > > the state as the test start, but I wonder where and how do we leak those
> > > pending flips? Have you looked into where those are coming from?
> > Yeah I spent a fair amount of time debugging this but it's been a
> > month so I vaguely remember it was cause the first subtest doesn't
> > finish all the way to the end (can't remember if it was an assertion
> > or requirement) so it ends early but with an event that's not been
> > consumed, then comes the second subtest working on top of an
> > unconsumed event. The bug exists in all devices, but they're probably
> > fast enough to not run into this case while my test device was slow
> > and suffered from this.
> > If you need it, I can debug the code again and report back exactly
> > what is happening.
>
> I would appreciate that. I think it may be better here to make sure that
> we are not leaking that flip in the first place.
>
What happens on CrOS is that the subtest *suspend* is achieved using
the RTC wakeup alarm. but if another service is holding /dev/rtc, IGT
will fail.
In this case, when the test is in its run_test_step(..), pages are
being flipped. Before the events are consumed, the test calls
`igt_system_suspend_autoresume(..)` which skips the test due to:
Test requirement not met in function suspend_via_rtcwake, file
../igt-gpu-tools-9999/lib/igt_aux.c:772:
Test requirement: ret == 0
rtcwake test failed with 1
This failure could mean that something is wrong with the rtcwake tool
or how your distro is set up.
When this happens, the subtest skips and page flip events would be pending.
When the next subtest starts, it's in an unclean state so an early
cleanup is what I'm doing here.
> > > > Fixes: flip-vs-suspend and flip-vs-suspend-interruptible.
> > > > Tested on ChromeOS Volteer(i915-TGL) and Trogdor(msm)
> > > >
> > > > Signed-off-by: Mark Yacoub <markyacoub@chromium.org>
> > >
> > > It would be nice to not include your internal gerrit's Change-Id
> > > (or discard them upon commit). Those does not mean anything in the
> > > upstream.
> > My bad. I added a script that will prevent me from sending a patch if
> > it has the gerrit ID.
> > >
> > > > ---
> > > > tests/kms_flip.c | 23 +++++++++++++++++++++++
> > > > 1 file changed, 23 insertions(+)
> > > >
> > > > diff --git a/tests/kms_flip.c b/tests/kms_flip.c
> > > > index f2fce8d2..0a2041c5 100755
> > > > --- a/tests/kms_flip.c
> > > > +++ b/tests/kms_flip.c
> > > > @@ -1261,6 +1261,26 @@ static bool needs_retry_after_link_reset(struct udev_monitor *mon)
> > > > return hotplug_detected;
> > > > }
> > > >
> > > > +static void discard_any_stale_events() {
> > > > + fd_set fds;
> > > > + FD_ZERO(&fds);
> > > > + FD_SET(drm_fd, &fds);
> > > > + struct timeval timeout = { .tv_sec = 3, .tv_usec = 0 };
> > >
> > > Is the 3s timeout necessary? I can imagine that in a lot of cases this
> > > will end up being sleep(3), as we don't expect any stale events in a
> > > normal case. That's a significant time increase if you count all the
> > > invocations.
> > >
> > I don't have a strong reason besides that I used the same
> > implementation in wait_for_events() which uses 3 seconds for timeout.
>
> wait_for_events() is justified in having that 3s timeout - it is only
> called when we just did something and we expect it to produce events.
> We should never really spend the full 3s waiting.
>
> In places where you want to call discard_any_stale_events() there should
> be no pending events, so we would end up waiting those 3s until timeout
> almost every time.
>
> That's a significant difference.
>
> I wonder if checking without any wait would be sufficient. I am a bit
> afraid that may be too immediate for some of the slower devices.
>
> > > > + int ret = select(drm_fd + 1, &fds, NULL, NULL, &timeout);
> > > > +
> > > > + if (ret > 0) {
> > > > + drmEventContext evctx;
> > > > + memset(&evctx, 0, sizeof evctx);
> > > > + evctx.version = 2;
> > > > + igt_info("Stale Event found - Discarding now\n");
> > > > + drmHandleEvent(drm_fd, &evctx);
> > > > + }
> > > > + else {
> > > > + igt_debug("No stale Event found\n");
> > > > + }
> > > > +}
> > >
> > > I think it should plural (and lowercase) "events" as it will go through
> > > all the pending events on that fd.
> > >
> > > > +
> > > > +
> > > > static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
> > > > int crtc_count, int duration_ms)
> > > > {
> > > > @@ -1311,6 +1331,9 @@ static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
> > > > kmstest_dump_mode(&o->kmode[i]);
> > > >
> > > > retry:
> > > > + // Discard any pending event that hasn't been consumed from a previous retry or subtest.
> > >
> > > Generally C-style comments (/* bla */) are preferred, but I don't think
> > > IGT or the kernel coding style explicitly mentions that. Also there are
> > > occurrences of C++-style comments in the code already. Oh well...
> > >
> > Sorry, it's just a habit, my bad.
>
> No need to be sorry, code conventions are completetly arbitrary and we
> don't really provide tools (you can theoretically use checkpatch.pl from
> the kernel) to make it easy on people. This "oh well" was just me taking
> a note that we don't even have "the comments rule" codified and that
> there are examples of not following it in the IGT already, so expecting
> people to follow it and enforcing it is not really fair.
>
> Cheers,
> Arek
>
> > > All the cosmetic nitpicks can be fixed by me or another committer when
> > > merging the patch (assuming that you are ok with them), so there's not
> > > need to resend unless you want to make changes to the actual code.
> > >
> > > Cheers,
> > > Arek
> > >
> > Thanks for reviewing!
> > > > + discard_any_stale_events();
> > > > +
> > > > memset(&o->vblank_state, 0, sizeof(o->vblank_state));
> > > > memset(&o->flip_state, 0, sizeof(o->flip_state));
> > > > o->flip_state.name = "flip";
> > > > --
> > > > 2.32.0.432.gabb21c7263-goog
^ permalink raw reply [flat|nested] 16+ messages in thread
* [igt-dev] [PATCH v2] tests/kms_flip: Discard any stale event at each retry
2021-07-28 17:16 [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry Mark Yacoub
` (5 preceding siblings ...)
2021-09-01 19:06 ` [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry Arkadiusz Hiler
@ 2021-09-24 18:39 ` Mark Yacoub
2021-10-26 8:39 ` Petri Latvala
2021-09-24 19:45 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_flip: Discard any stale event at each retry (rev3) Patchwork
2021-09-24 23:35 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
8 siblings, 1 reply; 16+ messages in thread
From: Mark Yacoub @ 2021-09-24 18:39 UTC (permalink / raw)
To: igt-dev, arek; +Cc: seanpaul, petri.latvala, Mark Yacoub, Mark Yacoub
From: Mark Yacoub <markyacoub@google.com>
[Why]
On slower devices, non blocking do_page_flip followed by wait_for_events
does not finish fast enough to send an event to be consumed by wait_for_events.
Instead, wait_for_events ends up finding an unconsumed event from a
previous do_page_flip (i.e. a skipped preceding subtest) and signals
that the userspace is ready to do another do_page_flip while the first
one hasn't finished yet, resulting in an EBUSY.
[How]
At the beginning of every test running on crtc set, check for any stale
events and consume them so the kernel has no pending events.
Fixes: flip-vs-suspend and flip-vs-suspend-interruptible.
Tested on ChromeOS Volteer(i915-TGL) and Trogdor(msm)
Changes since v1:
1. Update the discard_any_stale_events timeout
2. Correct a debug message
3. use C-style comments
Signed-off-by: Mark Yacoub <markyacoub@chromium.org>
---
tests/kms_flip.c | 23 +++++++++++++++++++++++
1 file changed, 23 insertions(+)
diff --git a/tests/kms_flip.c b/tests/kms_flip.c
index b4836a44..73a8e636 100755
--- a/tests/kms_flip.c
+++ b/tests/kms_flip.c
@@ -1265,6 +1265,26 @@ static bool needs_retry_after_link_reset(struct udev_monitor *mon)
return hotplug_detected;
}
+static void discard_any_stale_events() {
+ fd_set fds;
+ FD_ZERO(&fds);
+ FD_SET(drm_fd, &fds);
+ struct timeval timeout = { .tv_sec = 0, .tv_usec = 20000 };
+ int ret = select(drm_fd + 1, &fds, NULL, NULL, &timeout);
+
+ if (ret > 0) {
+ drmEventContext evctx;
+ memset(&evctx, 0, sizeof evctx);
+ evctx.version = 2;
+ igt_info("Stale Event found - Discarding now\n");
+ drmHandleEvent(drm_fd, &evctx);
+ }
+ else {
+ igt_debug("No stale events found\n");
+ }
+}
+
+
static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
int crtc_count, int duration_ms)
{
@@ -1315,6 +1335,9 @@ static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
kmstest_dump_mode(&o->kmode[i]);
retry:
+ /* Discard any pending event that hasn't been consumed from a previous retry or subtest. */
+ discard_any_stale_events();
+
memset(&o->vblank_state, 0, sizeof(o->vblank_state));
memset(&o->flip_state, 0, sizeof(o->flip_state));
o->flip_state.name = "flip";
--
2.33.0.685.g46640cef36-goog
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_flip: Discard any stale event at each retry (rev3)
2021-07-28 17:16 [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry Mark Yacoub
` (6 preceding siblings ...)
2021-09-24 18:39 ` [igt-dev] [PATCH v2] " Mark Yacoub
@ 2021-09-24 19:45 ` Patchwork
2021-09-24 23:35 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
8 siblings, 0 replies; 16+ messages in thread
From: Patchwork @ 2021-09-24 19:45 UTC (permalink / raw)
To: Mark Yacoub; +Cc: igt-dev
[-- Attachment #1: Type: text/plain, Size: 7101 bytes --]
== Series Details ==
Series: tests/kms_flip: Discard any stale event at each retry (rev3)
URL : https://patchwork.freedesktop.org/series/93133/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10641 -> IGTPW_6260
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/index.html
Known issues
------------
Here are the changes found in IGTPW_6260 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@amdgpu/amd_basic@query-info:
- fi-bsw-kefka: NOTRUN -> [SKIP][1] ([fdo#109271]) +17 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/fi-bsw-kefka/igt@amdgpu/amd_basic@query-info.html
* igt@kms_flip@basic-flip-vs-dpms:
- fi-kbl-8809g: NOTRUN -> [SKIP][2] ([fdo#109271]) +4 similar issues
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/fi-kbl-8809g/igt@kms_flip@basic-flip-vs-dpms.html
* igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-b:
- fi-kbl-8809g: [PASS][3] -> [SKIP][4] ([fdo#109271]) +15 similar issues
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/fi-kbl-8809g/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-b.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/fi-kbl-8809g/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-b.html
* igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d:
- fi-kbl-8809g: [PASS][5] -> [SKIP][6] ([fdo#109271] / [i915#533])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/fi-kbl-8809g/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/fi-kbl-8809g/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d.html
* igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a-frame-sequence:
- fi-rkl-11600: [PASS][7] -> [SKIP][8] ([i915#4098]) +2 similar issues
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/fi-rkl-11600/igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a-frame-sequence.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/fi-rkl-11600/igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a-frame-sequence.html
* igt@runner@aborted:
- fi-bdw-5557u: NOTRUN -> [FAIL][9] ([i915#1602] / [i915#2029])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/fi-bdw-5557u/igt@runner@aborted.html
#### Possible fixes ####
* igt@core_hotunplug@unbind-rebind:
- fi-kbl-8809g: [INCOMPLETE][10] ([i915#4130]) -> [PASS][11]
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/fi-kbl-8809g/igt@core_hotunplug@unbind-rebind.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/fi-kbl-8809g/igt@core_hotunplug@unbind-rebind.html
* igt@gem_exec_suspend@basic-s3:
- fi-tgl-1115g4: [FAIL][12] ([i915#1888]) -> [PASS][13]
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/fi-tgl-1115g4/igt@gem_exec_suspend@basic-s3.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/fi-tgl-1115g4/igt@gem_exec_suspend@basic-s3.html
* igt@i915_module_load@reload:
- {fi-tgl-dsi}: [DMESG-WARN][14] ([i915#1982] / [i915#4136]) -> [PASS][15]
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/fi-tgl-dsi/igt@i915_module_load@reload.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/fi-tgl-dsi/igt@i915_module_load@reload.html
* igt@i915_selftest@live@execlists:
- fi-bsw-kefka: [INCOMPLETE][16] ([i915#2940]) -> [PASS][17]
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/fi-bsw-kefka/igt@i915_selftest@live@execlists.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/fi-bsw-kefka/igt@i915_selftest@live@execlists.html
* igt@kms_addfb_basic@bad-pitch-65536:
- fi-kbl-8809g: [FAIL][18] -> [PASS][19] +1 similar issue
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/fi-kbl-8809g/igt@kms_addfb_basic@bad-pitch-65536.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/fi-kbl-8809g/igt@kms_addfb_basic@bad-pitch-65536.html
* igt@kms_addfb_basic@too-high:
- fi-kbl-8809g: [FAIL][20] ([i915#2774]) -> [PASS][21]
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/fi-kbl-8809g/igt@kms_addfb_basic@too-high.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/fi-kbl-8809g/igt@kms_addfb_basic@too-high.html
* igt@kms_addfb_basic@unused-modifier:
- fi-kbl-8809g: [SKIP][22] ([fdo#109271]) -> [PASS][23] +18 similar issues
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/fi-kbl-8809g/igt@kms_addfb_basic@unused-modifier.html
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/fi-kbl-8809g/igt@kms_addfb_basic@unused-modifier.html
* igt@kms_force_connector_basic@force-connector-state:
- fi-kbl-8809g: [DMESG-FAIL][24] -> [PASS][25]
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/fi-kbl-8809g/igt@kms_force_connector_basic@force-connector-state.html
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/fi-kbl-8809g/igt@kms_force_connector_basic@force-connector-state.html
* igt@kms_force_connector_basic@force-edid:
- fi-kbl-8809g: [CRASH][26] -> [PASS][27]
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/fi-kbl-8809g/igt@kms_force_connector_basic@force-edid.html
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/fi-kbl-8809g/igt@kms_force_connector_basic@force-edid.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[i915#1602]: https://gitlab.freedesktop.org/drm/intel/issues/1602
[i915#1888]: https://gitlab.freedesktop.org/drm/intel/issues/1888
[i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982
[i915#2029]: https://gitlab.freedesktop.org/drm/intel/issues/2029
[i915#2774]: https://gitlab.freedesktop.org/drm/intel/issues/2774
[i915#2940]: https://gitlab.freedesktop.org/drm/intel/issues/2940
[i915#4098]: https://gitlab.freedesktop.org/drm/intel/issues/4098
[i915#4130]: https://gitlab.freedesktop.org/drm/intel/issues/4130
[i915#4136]: https://gitlab.freedesktop.org/drm/intel/issues/4136
[i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533
Participating hosts (40 -> 34)
------------------------------
Missing (6): bat-dg1-6 bat-dg1-5 fi-bsw-cyan bat-adlp-4 fi-bdw-samus bat-jsl-1
Build changes
-------------
* CI: CI-20190529 -> None
* IGT: IGT_6218 -> IGTPW_6260
CI-20190529: 20190529
CI_DRM_10641: 7ed675211034f2d6fd57c0e6388081d5d008e2d4 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_6260: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/index.html
IGT_6218: 8d4169d9543d8e5c01f0c746f603801a4d65ead0 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/index.html
[-- Attachment #2: Type: text/html, Size: 8345 bytes --]
^ permalink raw reply [flat|nested] 16+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for tests/kms_flip: Discard any stale event at each retry (rev3)
2021-07-28 17:16 [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry Mark Yacoub
` (7 preceding siblings ...)
2021-09-24 19:45 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_flip: Discard any stale event at each retry (rev3) Patchwork
@ 2021-09-24 23:35 ` Patchwork
8 siblings, 0 replies; 16+ messages in thread
From: Patchwork @ 2021-09-24 23:35 UTC (permalink / raw)
To: Mark Yacoub; +Cc: igt-dev
[-- Attachment #1: Type: text/plain, Size: 30278 bytes --]
== Series Details ==
Series: tests/kms_flip: Discard any stale event at each retry (rev3)
URL : https://patchwork.freedesktop.org/series/93133/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10641_full -> IGTPW_6260_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/index.html
Known issues
------------
Here are the changes found in IGTPW_6260_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_create@create-massive:
- shard-snb: NOTRUN -> [DMESG-WARN][1] ([i915#3002])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-snb6/igt@gem_create@create-massive.html
- shard-kbl: NOTRUN -> [DMESG-WARN][2] ([i915#3002])
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl6/igt@gem_create@create-massive.html
- shard-glk: NOTRUN -> [DMESG-WARN][3] ([i915#3002])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-glk5/igt@gem_create@create-massive.html
- shard-apl: NOTRUN -> [DMESG-WARN][4] ([i915#3002])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-apl3/igt@gem_create@create-massive.html
* igt@gem_ctx_persistence@legacy-engines-mixed:
- shard-snb: NOTRUN -> [SKIP][5] ([fdo#109271] / [i915#1099]) +4 similar issues
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-snb5/igt@gem_ctx_persistence@legacy-engines-mixed.html
* igt@gem_ctx_sseu@engines:
- shard-tglb: NOTRUN -> [SKIP][6] ([i915#280])
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb6/igt@gem_ctx_sseu@engines.html
* igt@gem_exec_fair@basic-none-vip@rcs0:
- shard-kbl: [PASS][7] -> [FAIL][8] ([i915#2842]) +1 similar issue
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-kbl4/igt@gem_exec_fair@basic-none-vip@rcs0.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl2/igt@gem_exec_fair@basic-none-vip@rcs0.html
- shard-tglb: NOTRUN -> [FAIL][9] ([i915#2842])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb8/igt@gem_exec_fair@basic-none-vip@rcs0.html
- shard-glk: [PASS][10] -> [FAIL][11] ([i915#2842])
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-glk5/igt@gem_exec_fair@basic-none-vip@rcs0.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-glk3/igt@gem_exec_fair@basic-none-vip@rcs0.html
* igt@gem_exec_fair@basic-none@vcs1:
- shard-iclb: NOTRUN -> [FAIL][12] ([i915#2842])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb4/igt@gem_exec_fair@basic-none@vcs1.html
* igt@gem_exec_fair@basic-throttle@rcs0:
- shard-iclb: [PASS][13] -> [FAIL][14] ([i915#2849])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-iclb4/igt@gem_exec_fair@basic-throttle@rcs0.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb6/igt@gem_exec_fair@basic-throttle@rcs0.html
* igt@gem_huc_copy@huc-copy:
- shard-apl: NOTRUN -> [SKIP][15] ([fdo#109271] / [i915#2190])
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-apl3/igt@gem_huc_copy@huc-copy.html
* igt@gem_pread@exhaustion:
- shard-tglb: NOTRUN -> [WARN][16] ([i915#2658])
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb3/igt@gem_pread@exhaustion.html
- shard-glk: NOTRUN -> [WARN][17] ([i915#2658])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-glk2/igt@gem_pread@exhaustion.html
- shard-iclb: NOTRUN -> [WARN][18] ([i915#2658])
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb6/igt@gem_pread@exhaustion.html
- shard-kbl: NOTRUN -> [WARN][19] ([i915#2658])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl1/igt@gem_pread@exhaustion.html
* igt@gem_render_copy@linear-to-vebox-yf-tiled:
- shard-iclb: NOTRUN -> [SKIP][20] ([i915#768])
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb8/igt@gem_render_copy@linear-to-vebox-yf-tiled.html
* igt@gem_render_copy@x-tiled-to-vebox-yf-tiled:
- shard-kbl: NOTRUN -> [SKIP][21] ([fdo#109271]) +178 similar issues
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl6/igt@gem_render_copy@x-tiled-to-vebox-yf-tiled.html
* igt@gem_softpin@noreloc-s3:
- shard-apl: [PASS][22] -> [DMESG-WARN][23] ([i915#180])
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-apl3/igt@gem_softpin@noreloc-s3.html
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-apl8/igt@gem_softpin@noreloc-s3.html
* igt@gem_userptr_blits@vma-merge:
- shard-snb: NOTRUN -> [FAIL][24] ([i915#2724])
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-snb5/igt@gem_userptr_blits@vma-merge.html
* igt@gen9_exec_parse@bb-start-param:
- shard-tglb: NOTRUN -> [SKIP][25] ([i915#2856]) +2 similar issues
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb7/igt@gen9_exec_parse@bb-start-param.html
- shard-iclb: NOTRUN -> [SKIP][26] ([i915#2856]) +2 similar issues
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb6/igt@gen9_exec_parse@bb-start-param.html
* igt@i915_pm_dc@dc3co-vpb-simulation:
- shard-tglb: NOTRUN -> [SKIP][27] ([i915#1904])
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb2/igt@i915_pm_dc@dc3co-vpb-simulation.html
* igt@i915_pm_dc@dc6-psr:
- shard-iclb: [PASS][28] -> [FAIL][29] ([i915#454])
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-iclb1/igt@i915_pm_dc@dc6-psr.html
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb6/igt@i915_pm_dc@dc6-psr.html
* igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-dp:
- shard-kbl: NOTRUN -> [SKIP][30] ([fdo#109271] / [i915#1937])
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl2/igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-dp.html
- shard-apl: NOTRUN -> [SKIP][31] ([fdo#109271] / [i915#1937])
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-apl6/igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-dp.html
* igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-hdmi-a:
- shard-glk: NOTRUN -> [SKIP][32] ([fdo#109271] / [i915#1937])
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-glk7/igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-hdmi-a.html
* igt@i915_pm_rpm@dpms-mode-unset-non-lpsp:
- shard-tglb: NOTRUN -> [SKIP][33] ([fdo#111644] / [i915#1397] / [i915#2411])
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb2/igt@i915_pm_rpm@dpms-mode-unset-non-lpsp.html
* igt@i915_pm_sseu@full-enable:
- shard-tglb: NOTRUN -> [SKIP][34] ([fdo#109288])
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb7/igt@i915_pm_sseu@full-enable.html
- shard-iclb: NOTRUN -> [SKIP][35] ([fdo#109288])
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb6/igt@i915_pm_sseu@full-enable.html
* igt@kms_atomic@plane-primary-overlay-mutable-zpos:
- shard-tglb: NOTRUN -> [SKIP][36] ([i915#404])
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb8/igt@kms_atomic@plane-primary-overlay-mutable-zpos.html
- shard-iclb: NOTRUN -> [SKIP][37] ([i915#404])
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb1/igt@kms_atomic@plane-primary-overlay-mutable-zpos.html
* igt@kms_big_fb@linear-32bpp-rotate-270:
- shard-iclb: NOTRUN -> [SKIP][38] ([fdo#110725] / [fdo#111614])
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb3/igt@kms_big_fb@linear-32bpp-rotate-270.html
* igt@kms_big_fb@linear-8bpp-rotate-90:
- shard-tglb: NOTRUN -> [SKIP][39] ([fdo#111614]) +1 similar issue
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb1/igt@kms_big_fb@linear-8bpp-rotate-90.html
* igt@kms_big_fb@x-tiled-max-hw-stride-32bpp-rotate-0-hflip:
- shard-kbl: NOTRUN -> [SKIP][40] ([fdo#109271] / [i915#3777]) +1 similar issue
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl2/igt@kms_big_fb@x-tiled-max-hw-stride-32bpp-rotate-0-hflip.html
* igt@kms_big_fb@x-tiled-max-hw-stride-64bpp-rotate-0-hflip:
- shard-glk: NOTRUN -> [SKIP][41] ([fdo#109271] / [i915#3777])
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-glk7/igt@kms_big_fb@x-tiled-max-hw-stride-64bpp-rotate-0-hflip.html
* igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-180-hflip:
- shard-apl: NOTRUN -> [SKIP][42] ([fdo#109271] / [i915#3777]) +2 similar issues
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-apl8/igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-180-hflip.html
* igt@kms_big_fb@yf-tiled-8bpp-rotate-270:
- shard-iclb: NOTRUN -> [SKIP][43] ([fdo#110723]) +1 similar issue
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb4/igt@kms_big_fb@yf-tiled-8bpp-rotate-270.html
* igt@kms_big_fb@yf-tiled-max-hw-stride-64bpp-rotate-180-hflip:
- shard-tglb: NOTRUN -> [SKIP][44] ([fdo#111615]) +2 similar issues
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb8/igt@kms_big_fb@yf-tiled-max-hw-stride-64bpp-rotate-180-hflip.html
* igt@kms_big_joiner@invalid-modeset:
- shard-iclb: NOTRUN -> [SKIP][45] ([i915#2705])
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb5/igt@kms_big_joiner@invalid-modeset.html
- shard-tglb: NOTRUN -> [SKIP][46] ([i915#2705])
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb1/igt@kms_big_joiner@invalid-modeset.html
* igt@kms_ccs@pipe-a-bad-rotation-90-y_tiled_gen12_mc_ccs:
- shard-tglb: NOTRUN -> [SKIP][47] ([i915#3689] / [i915#3886]) +1 similar issue
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb6/igt@kms_ccs@pipe-a-bad-rotation-90-y_tiled_gen12_mc_ccs.html
* igt@kms_ccs@pipe-a-crc-primary-rotation-180-y_tiled_gen12_rc_ccs_cc:
- shard-kbl: NOTRUN -> [SKIP][48] ([fdo#109271] / [i915#3886]) +11 similar issues
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl4/igt@kms_ccs@pipe-a-crc-primary-rotation-180-y_tiled_gen12_rc_ccs_cc.html
* igt@kms_ccs@pipe-b-bad-pixel-format-yf_tiled_ccs:
- shard-tglb: NOTRUN -> [SKIP][49] ([i915#3689]) +1 similar issue
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb7/igt@kms_ccs@pipe-b-bad-pixel-format-yf_tiled_ccs.html
* igt@kms_ccs@pipe-b-crc-sprite-planes-basic-y_tiled_gen12_rc_ccs_cc:
- shard-glk: NOTRUN -> [SKIP][50] ([fdo#109271] / [i915#3886]) +7 similar issues
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-glk8/igt@kms_ccs@pipe-b-crc-sprite-planes-basic-y_tiled_gen12_rc_ccs_cc.html
- shard-iclb: NOTRUN -> [SKIP][51] ([fdo#109278] / [i915#3886]) +2 similar issues
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb3/igt@kms_ccs@pipe-b-crc-sprite-planes-basic-y_tiled_gen12_rc_ccs_cc.html
* igt@kms_ccs@pipe-c-ccs-on-another-bo-y_tiled_gen12_rc_ccs_cc:
- shard-apl: NOTRUN -> [SKIP][52] ([fdo#109271] / [i915#3886]) +14 similar issues
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-apl8/igt@kms_ccs@pipe-c-ccs-on-another-bo-y_tiled_gen12_rc_ccs_cc.html
* igt@kms_cdclk@mode-transition:
- shard-apl: NOTRUN -> [SKIP][53] ([fdo#109271]) +264 similar issues
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-apl1/igt@kms_cdclk@mode-transition.html
* igt@kms_chamelium@hdmi-edid-change-during-suspend:
- shard-apl: NOTRUN -> [SKIP][54] ([fdo#109271] / [fdo#111827]) +16 similar issues
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-apl3/igt@kms_chamelium@hdmi-edid-change-during-suspend.html
* igt@kms_chamelium@hdmi-hpd-enable-disable-mode:
- shard-iclb: NOTRUN -> [SKIP][55] ([fdo#109284] / [fdo#111827]) +3 similar issues
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb8/igt@kms_chamelium@hdmi-hpd-enable-disable-mode.html
* igt@kms_chamelium@hdmi-hpd-storm:
- shard-kbl: NOTRUN -> [SKIP][56] ([fdo#109271] / [fdo#111827]) +13 similar issues
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl4/igt@kms_chamelium@hdmi-hpd-storm.html
* igt@kms_chamelium@vga-hpd-without-ddc:
- shard-snb: NOTRUN -> [SKIP][57] ([fdo#109271] / [fdo#111827]) +17 similar issues
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-snb2/igt@kms_chamelium@vga-hpd-without-ddc.html
* igt@kms_color@pipe-d-ctm-blue-to-red:
- shard-iclb: NOTRUN -> [SKIP][58] ([fdo#109278] / [i915#1149])
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb4/igt@kms_color@pipe-d-ctm-blue-to-red.html
* igt@kms_color@pipe-d-invalid-ctm-matrix-sizes:
- shard-iclb: NOTRUN -> [SKIP][59] ([fdo#109278]) +17 similar issues
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb1/igt@kms_color@pipe-d-invalid-ctm-matrix-sizes.html
* igt@kms_color_chamelium@pipe-c-ctm-0-5:
- shard-glk: NOTRUN -> [SKIP][60] ([fdo#109271] / [fdo#111827]) +10 similar issues
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-glk4/igt@kms_color_chamelium@pipe-c-ctm-0-5.html
* igt@kms_color_chamelium@pipe-d-ctm-blue-to-red:
- shard-tglb: NOTRUN -> [SKIP][61] ([fdo#109284] / [fdo#111827]) +5 similar issues
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb5/igt@kms_color_chamelium@pipe-d-ctm-blue-to-red.html
- shard-iclb: NOTRUN -> [SKIP][62] ([fdo#109278] / [fdo#109284] / [fdo#111827])
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb2/igt@kms_color_chamelium@pipe-d-ctm-blue-to-red.html
* igt@kms_content_protection@atomic-dpms:
- shard-apl: NOTRUN -> [TIMEOUT][63] ([i915#1319])
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-apl8/igt@kms_content_protection@atomic-dpms.html
- shard-kbl: NOTRUN -> [TIMEOUT][64] ([i915#1319])
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl1/igt@kms_content_protection@atomic-dpms.html
* igt@kms_cursor_crc@pipe-b-cursor-32x10-sliding:
- shard-tglb: NOTRUN -> [SKIP][65] ([i915#3359]) +3 similar issues
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb6/igt@kms_cursor_crc@pipe-b-cursor-32x10-sliding.html
* igt@kms_cursor_crc@pipe-b-cursor-512x512-sliding:
- shard-iclb: NOTRUN -> [SKIP][66] ([fdo#109278] / [fdo#109279]) +1 similar issue
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb5/igt@kms_cursor_crc@pipe-b-cursor-512x512-sliding.html
* igt@kms_cursor_crc@pipe-d-cursor-512x512-rapid-movement:
- shard-tglb: NOTRUN -> [SKIP][67] ([fdo#109279] / [i915#3359]) +2 similar issues
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb8/igt@kms_cursor_crc@pipe-d-cursor-512x512-rapid-movement.html
* igt@kms_cursor_edge_walk@pipe-d-128x128-right-edge:
- shard-snb: NOTRUN -> [SKIP][68] ([fdo#109271]) +408 similar issues
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-snb5/igt@kms_cursor_edge_walk@pipe-d-128x128-right-edge.html
* igt@kms_cursor_legacy@cursora-vs-flipb-atomic:
- shard-tglb: NOTRUN -> [SKIP][69] ([fdo#111825]) +25 similar issues
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb8/igt@kms_cursor_legacy@cursora-vs-flipb-atomic.html
* igt@kms_cursor_legacy@cursora-vs-flipb-toggle:
- shard-iclb: NOTRUN -> [SKIP][70] ([fdo#109274] / [fdo#109278]) +1 similar issue
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb3/igt@kms_cursor_legacy@cursora-vs-flipb-toggle.html
* igt@kms_cursor_legacy@pipe-d-torture-bo:
- shard-kbl: NOTRUN -> [SKIP][71] ([fdo#109271] / [i915#533]) +1 similar issue
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl1/igt@kms_cursor_legacy@pipe-d-torture-bo.html
- shard-apl: NOTRUN -> [SKIP][72] ([fdo#109271] / [i915#533]) +1 similar issue
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-apl8/igt@kms_cursor_legacy@pipe-d-torture-bo.html
- shard-glk: NOTRUN -> [SKIP][73] ([fdo#109271] / [i915#533]) +2 similar issues
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-glk1/igt@kms_cursor_legacy@pipe-d-torture-bo.html
* igt@kms_fbcon_fbt@fbc-suspend:
- shard-kbl: [PASS][74] -> [INCOMPLETE][75] ([i915#155] / [i915#180] / [i915#636])
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-kbl4/igt@kms_fbcon_fbt@fbc-suspend.html
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl7/igt@kms_fbcon_fbt@fbc-suspend.html
* igt@kms_flip@2x-flip-vs-rmfb-interruptible:
- shard-iclb: NOTRUN -> [SKIP][76] ([fdo#109274]) +1 similar issue
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb8/igt@kms_flip@2x-flip-vs-rmfb-interruptible.html
* igt@kms_flip@flip-vs-suspend-interruptible@a-dp1:
- shard-kbl: NOTRUN -> [DMESG-WARN][77] ([i915#180]) +4 similar issues
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl3/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html
* igt@kms_flip@flip-vs-suspend@a-dp1:
- shard-apl: NOTRUN -> [DMESG-WARN][78] ([i915#180]) +1 similar issue
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-apl2/igt@kms_flip@flip-vs-suspend@a-dp1.html
* igt@kms_flip@flip-vs-suspend@a-edp1:
- shard-tglb: [PASS][79] -> [INCOMPLETE][80] ([i915#4173] / [i915#456])
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-tglb2/igt@kms_flip@flip-vs-suspend@a-edp1.html
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb7/igt@kms_flip@flip-vs-suspend@a-edp1.html
* igt@kms_frontbuffer_tracking@fbcpsr-2p-shrfb-fliptrack-mmap-gtt:
- shard-iclb: NOTRUN -> [SKIP][81] ([fdo#109280]) +15 similar issues
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb6/igt@kms_frontbuffer_tracking@fbcpsr-2p-shrfb-fliptrack-mmap-gtt.html
* igt@kms_frontbuffer_tracking@fbcpsr-suspend:
- shard-tglb: [PASS][82] -> [INCOMPLETE][83] ([i915#456])
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-tglb2/igt@kms_frontbuffer_tracking@fbcpsr-suspend.html
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb7/igt@kms_frontbuffer_tracking@fbcpsr-suspend.html
* igt@kms_frontbuffer_tracking@psr-2p-scndscrn-spr-indfb-draw-render:
- shard-glk: NOTRUN -> [SKIP][84] ([fdo#109271]) +113 similar issues
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-glk3/igt@kms_frontbuffer_tracking@psr-2p-scndscrn-spr-indfb-draw-render.html
* igt@kms_pipe_b_c_ivb@pipe-b-double-modeset-then-modeset-pipe-c:
- shard-tglb: NOTRUN -> [SKIP][85] ([fdo#109289]) +1 similar issue
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb6/igt@kms_pipe_b_c_ivb@pipe-b-double-modeset-then-modeset-pipe-c.html
- shard-iclb: NOTRUN -> [SKIP][86] ([fdo#109289])
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb8/igt@kms_pipe_b_c_ivb@pipe-b-double-modeset-then-modeset-pipe-c.html
* igt@kms_plane_alpha_blend@pipe-a-alpha-transparent-fb:
- shard-kbl: NOTRUN -> [FAIL][87] ([i915#265])
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl6/igt@kms_plane_alpha_blend@pipe-a-alpha-transparent-fb.html
* igt@kms_plane_alpha_blend@pipe-b-alpha-basic:
- shard-glk: NOTRUN -> [FAIL][88] ([fdo#108145] / [i915#265]) +1 similar issue
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-glk4/igt@kms_plane_alpha_blend@pipe-b-alpha-basic.html
* igt@kms_plane_alpha_blend@pipe-b-constant-alpha-max:
- shard-apl: NOTRUN -> [FAIL][89] ([fdo#108145] / [i915#265]) +1 similar issue
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-apl1/igt@kms_plane_alpha_blend@pipe-b-constant-alpha-max.html
* igt@kms_plane_lowres@pipe-a-tiling-y:
- shard-tglb: NOTRUN -> [SKIP][90] ([i915#3536]) +1 similar issue
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb7/igt@kms_plane_lowres@pipe-a-tiling-y.html
* igt@kms_plane_lowres@pipe-b-tiling-x:
- shard-iclb: NOTRUN -> [SKIP][91] ([i915#3536])
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb3/igt@kms_plane_lowres@pipe-b-tiling-x.html
* igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-2:
- shard-kbl: NOTRUN -> [SKIP][92] ([fdo#109271] / [i915#658]) +3 similar issues
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl1/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-2.html
- shard-iclb: NOTRUN -> [SKIP][93] ([i915#658])
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb6/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-2.html
- shard-tglb: NOTRUN -> [SKIP][94] ([i915#2920])
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb3/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-2.html
* igt@kms_psr2_su@frontbuffer:
- shard-glk: NOTRUN -> [SKIP][95] ([fdo#109271] / [i915#658]) +3 similar issues
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-glk6/igt@kms_psr2_su@frontbuffer.html
- shard-tglb: NOTRUN -> [SKIP][96] ([i915#1911])
[96]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb6/igt@kms_psr2_su@frontbuffer.html
- shard-iclb: NOTRUN -> [SKIP][97] ([fdo#109642] / [fdo#111068] / [i915#658])
[97]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb8/igt@kms_psr2_su@frontbuffer.html
* igt@kms_psr2_su@page_flip:
- shard-apl: NOTRUN -> [SKIP][98] ([fdo#109271] / [i915#658]) +4 similar issues
[98]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-apl1/igt@kms_psr2_su@page_flip.html
* igt@kms_psr@psr2_cursor_render:
- shard-iclb: [PASS][99] -> [SKIP][100] ([fdo#109441]) +1 similar issue
[99]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-iclb2/igt@kms_psr@psr2_cursor_render.html
[100]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb7/igt@kms_psr@psr2_cursor_render.html
* igt@kms_psr@psr2_sprite_mmap_gtt:
- shard-iclb: NOTRUN -> [SKIP][101] ([fdo#109441])
[101]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb7/igt@kms_psr@psr2_sprite_mmap_gtt.html
- shard-tglb: NOTRUN -> [FAIL][102] ([i915#132] / [i915#3467])
[102]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb1/igt@kms_psr@psr2_sprite_mmap_gtt.html
* igt@kms_sysfs_edid_timing:
- shard-apl: NOTRUN -> [FAIL][103] ([IGT#2])
[103]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-apl2/igt@kms_sysfs_edid_timing.html
- shard-kbl: NOTRUN -> [FAIL][104] ([IGT#2])
[104]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl2/igt@kms_sysfs_edid_timing.html
* igt@kms_vblank@pipe-b-ts-continuation-suspend:
- shard-kbl: [PASS][105] -> [DMESG-WARN][106] ([i915#180])
[105]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-kbl3/igt@kms_vblank@pipe-b-ts-continuation-suspend.html
[106]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl1/igt@kms_vblank@pipe-b-ts-continuation-suspend.html
* igt@kms_vrr@flip-dpms:
- shard-iclb: NOTRUN -> [SKIP][107] ([fdo#109502])
[107]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb7/igt@kms_vrr@flip-dpms.html
- shard-tglb: NOTRUN -> [SKIP][108] ([fdo#109502])
[108]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb1/igt@kms_vrr@flip-dpms.html
* igt@kms_writeback@writeback-fb-id:
- shard-glk: NOTRUN -> [SKIP][109] ([fdo#109271] / [i915#2437])
[109]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-glk4/igt@kms_writeback@writeback-fb-id.html
* igt@nouveau_crc@pipe-b-source-outp-complete:
- shard-tglb: NOTRUN -> [SKIP][110] ([i915#2530])
[110]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb5/igt@nouveau_crc@pipe-b-source-outp-complete.html
* igt@perf@polling-parameterized:
- shard-tglb: [PASS][111] -> [FAIL][112] ([i915#1542])
[111]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-tglb3/igt@perf@polling-parameterized.html
[112]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb6/igt@perf@polling-parameterized.html
* igt@prime_nv_api@i915_self_import:
- shard-iclb: NOTRUN -> [SKIP][113] ([fdo#109291]) +1 similar issue
[113]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb5/igt@prime_nv_api@i915_self_import.html
* igt@prime_nv_pcopy@test1_micro:
- shard-tglb: NOTRUN -> [SKIP][114] ([fdo#109291]) +1 similar issue
[114]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb8/igt@prime_nv_pcopy@test1_micro.html
* igt@sysfs_clients@busy:
- shard-tglb: NOTRUN -> [SKIP][115] ([i915#2994]) +2 similar issues
[115]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb7/igt@sysfs_clients@busy.html
* igt@sysfs_clients@fair-1:
- shard-apl: NOTRUN -> [SKIP][116] ([fdo#109271] / [i915#2994]) +6 similar issues
[116]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-apl1/igt@sysfs_clients@fair-1.html
* igt@sysfs_clients@split-25:
- shard-glk: NOTRUN -> [SKIP][117] ([fdo#109271] / [i915#2994]) +2 similar issues
[117]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-glk2/igt@sysfs_clients@split-25.html
* igt@sysfs_clients@split-50:
- shard-kbl: NOTRUN -> [SKIP][118] ([fdo#109271] / [i915#2994]) +3 similar issues
[118]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl2/igt@sysfs_clients@split-50.html
- shard-iclb: NOTRUN -> [SKIP][119] ([i915#2994]) +1 similar issue
[119]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-iclb1/igt@sysfs_clients@split-50.html
#### Possible fixes ####
* igt@gem_exec_fair@basic-deadline:
- shard-kbl: [FAIL][120] ([i915#2846]) -> [PASS][121]
[120]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-kbl2/igt@gem_exec_fair@basic-deadline.html
[121]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl2/igt@gem_exec_fair@basic-deadline.html
* igt@gem_exec_fair@basic-pace-share@rcs0:
- shard-tglb: [FAIL][122] ([i915#2842]) -> [PASS][123]
[122]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-tglb3/igt@gem_exec_fair@basic-pace-share@rcs0.html
[123]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb3/igt@gem_exec_fair@basic-pace-share@rcs0.html
* igt@i915_pm_dc@dc9-dpms:
- shard-apl: [SKIP][124] ([fdo#109271]) -> [PASS][125]
[124]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-apl6/igt@i915_pm_dc@dc9-dpms.html
[125]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-apl6/igt@i915_pm_dc@dc9-dpms.html
* igt@i915_suspend@fence-restore-tiled2untiled:
- shard-kbl: [DMESG-WARN][126] ([i915#180]) -> [PASS][127]
[126]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-kbl1/igt@i915_suspend@fence-restore-tiled2untiled.html
[127]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl2/igt@i915_suspend@fence-restore-tiled2untiled.html
* igt@kms_big_fb@linear-32bpp-rotate-180:
- shard-glk: [DMESG-WARN][128] ([i915#118] / [i915#95]) -> [PASS][129] +2 similar issues
[128]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-glk5/igt@kms_big_fb@linear-32bpp-rotate-180.html
[129]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-glk4/igt@kms_big_fb@linear-32bpp-rotate-180.html
* igt@kms_cursor_crc@pipe-a-cursor-size-change:
- shard-snb: [FAIL][130] ([i915#4024]) -> [PASS][131]
[130]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-snb7/igt@kms_cursor_crc@pipe-a-cursor-size-change.html
[131]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-snb5/igt@kms_cursor_crc@pipe-a-cursor-size-change.html
- shard-apl: [FAIL][132] ([i915#3444]) -> [PASS][133]
[132]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-apl3/igt@kms_cursor_crc@pipe-a-cursor-size-change.html
[133]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-apl2/igt@kms_cursor_crc@pipe-a-cursor-size-change.html
- shard-kbl: [FAIL][134] ([i915#3444]) -> [PASS][135]
[134]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-kbl3/igt@kms_cursor_crc@pipe-a-cursor-size-change.html
[135]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-kbl4/igt@kms_cursor_crc@pipe-a-cursor-size-change.html
* igt@kms_cursor_crc@pipe-b-cursor-suspend:
- shard-tglb: [INCOMPLETE][136] ([i915#456]) -> [PASS][137]
[136]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-tglb7/igt@kms_cursor_crc@pipe-b-cursor-suspend.html
[137]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb1/igt@kms_cursor_crc@pipe-b-cursor-suspend.html
* igt@kms_cursor_crc@pipe-d-cursor-alpha-opaque:
- shard-tglb: [FAIL][138] ([i915#2124]) -> [PASS][139]
[138]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10641/shard-tglb1/igt@kms_cursor_crc@pipe-d-cursor-alpha-opaque.html
[139]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/shard-tglb8/igt@kms_cursor_crc@pipe-d-c
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6260/index.html
[-- Attachment #2: Type: text/html, Size: 33912 bytes --]
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [igt-dev] [PATCH v2] tests/kms_flip: Discard any stale event at each retry
2021-09-24 18:39 ` [igt-dev] [PATCH v2] " Mark Yacoub
@ 2021-10-26 8:39 ` Petri Latvala
2021-10-26 14:57 ` Mark Yacoub
0 siblings, 1 reply; 16+ messages in thread
From: Petri Latvala @ 2021-10-26 8:39 UTC (permalink / raw)
To: Mark Yacoub; +Cc: igt-dev, arek, seanpaul, Mark Yacoub
On Fri, Sep 24, 2021 at 02:39:29PM -0400, Mark Yacoub wrote:
> From: Mark Yacoub <markyacoub@google.com>
>
> [Why]
> On slower devices, non blocking do_page_flip followed by wait_for_events
> does not finish fast enough to send an event to be consumed by wait_for_events.
> Instead, wait_for_events ends up finding an unconsumed event from a
> previous do_page_flip (i.e. a skipped preceding subtest) and signals
> that the userspace is ready to do another do_page_flip while the first
> one hasn't finished yet, resulting in an EBUSY.
>
> [How]
> At the beginning of every test running on crtc set, check for any stale
> events and consume them so the kernel has no pending events.
>
> Fixes: flip-vs-suspend and flip-vs-suspend-interruptible.
> Tested on ChromeOS Volteer(i915-TGL) and Trogdor(msm)
>
> Changes since v1:
> 1. Update the discard_any_stale_events timeout
> 2. Correct a debug message
> 3. use C-style comments
>
> Signed-off-by: Mark Yacoub <markyacoub@chromium.org>
Looks like Arek's comments have been addressed, and this LGTM.
Reviewed-by: Petri Latvala <petri.latvala@intel.com>
> ---
> tests/kms_flip.c | 23 +++++++++++++++++++++++
> 1 file changed, 23 insertions(+)
>
> diff --git a/tests/kms_flip.c b/tests/kms_flip.c
> index b4836a44..73a8e636 100755
> --- a/tests/kms_flip.c
> +++ b/tests/kms_flip.c
> @@ -1265,6 +1265,26 @@ static bool needs_retry_after_link_reset(struct udev_monitor *mon)
> return hotplug_detected;
> }
>
> +static void discard_any_stale_events() {
> + fd_set fds;
> + FD_ZERO(&fds);
> + FD_SET(drm_fd, &fds);
> + struct timeval timeout = { .tv_sec = 0, .tv_usec = 20000 };
> + int ret = select(drm_fd + 1, &fds, NULL, NULL, &timeout);
> +
> + if (ret > 0) {
> + drmEventContext evctx;
> + memset(&evctx, 0, sizeof evctx);
> + evctx.version = 2;
> + igt_info("Stale Event found - Discarding now\n");
> + drmHandleEvent(drm_fd, &evctx);
> + }
> + else {
> + igt_debug("No stale events found\n");
> + }
> +}
> +
> +
> static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
> int crtc_count, int duration_ms)
> {
> @@ -1315,6 +1335,9 @@ static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
> kmstest_dump_mode(&o->kmode[i]);
>
> retry:
> + /* Discard any pending event that hasn't been consumed from a previous retry or subtest. */
> + discard_any_stale_events();
> +
> memset(&o->vblank_state, 0, sizeof(o->vblank_state));
> memset(&o->flip_state, 0, sizeof(o->flip_state));
> o->flip_state.name = "flip";
> --
> 2.33.0.685.g46640cef36-goog
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [igt-dev] [PATCH v2] tests/kms_flip: Discard any stale event at each retry
2021-10-26 8:39 ` Petri Latvala
@ 2021-10-26 14:57 ` Mark Yacoub
0 siblings, 0 replies; 16+ messages in thread
From: Mark Yacoub @ 2021-10-26 14:57 UTC (permalink / raw)
To: Petri Latvala; +Cc: igt-dev, arek, seanpaul, Mark Yacoub
Applied. thanks!
On Tue, Oct 26, 2021 at 4:41 AM Petri Latvala <petri.latvala@intel.com> wrote:
>
> On Fri, Sep 24, 2021 at 02:39:29PM -0400, Mark Yacoub wrote:
> > From: Mark Yacoub <markyacoub@google.com>
> >
> > [Why]
> > On slower devices, non blocking do_page_flip followed by wait_for_events
> > does not finish fast enough to send an event to be consumed by wait_for_events.
> > Instead, wait_for_events ends up finding an unconsumed event from a
> > previous do_page_flip (i.e. a skipped preceding subtest) and signals
> > that the userspace is ready to do another do_page_flip while the first
> > one hasn't finished yet, resulting in an EBUSY.
> >
> > [How]
> > At the beginning of every test running on crtc set, check for any stale
> > events and consume them so the kernel has no pending events.
> >
> > Fixes: flip-vs-suspend and flip-vs-suspend-interruptible.
> > Tested on ChromeOS Volteer(i915-TGL) and Trogdor(msm)
> >
> > Changes since v1:
> > 1. Update the discard_any_stale_events timeout
> > 2. Correct a debug message
> > 3. use C-style comments
> >
> > Signed-off-by: Mark Yacoub <markyacoub@chromium.org>
>
> Looks like Arek's comments have been addressed, and this LGTM.
>
> Reviewed-by: Petri Latvala <petri.latvala@intel.com>
>
>
> > ---
> > tests/kms_flip.c | 23 +++++++++++++++++++++++
> > 1 file changed, 23 insertions(+)
> >
> > diff --git a/tests/kms_flip.c b/tests/kms_flip.c
> > index b4836a44..73a8e636 100755
> > --- a/tests/kms_flip.c
> > +++ b/tests/kms_flip.c
> > @@ -1265,6 +1265,26 @@ static bool needs_retry_after_link_reset(struct udev_monitor *mon)
> > return hotplug_detected;
> > }
> >
> > +static void discard_any_stale_events() {
> > + fd_set fds;
> > + FD_ZERO(&fds);
> > + FD_SET(drm_fd, &fds);
> > + struct timeval timeout = { .tv_sec = 0, .tv_usec = 20000 };
> > + int ret = select(drm_fd + 1, &fds, NULL, NULL, &timeout);
> > +
> > + if (ret > 0) {
> > + drmEventContext evctx;
> > + memset(&evctx, 0, sizeof evctx);
> > + evctx.version = 2;
> > + igt_info("Stale Event found - Discarding now\n");
> > + drmHandleEvent(drm_fd, &evctx);
> > + }
> > + else {
> > + igt_debug("No stale events found\n");
> > + }
> > +}
> > +
> > +
> > static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
> > int crtc_count, int duration_ms)
> > {
> > @@ -1315,6 +1335,9 @@ static void __run_test_on_crtc_set(struct test_output *o, int *crtc_idxs,
> > kmstest_dump_mode(&o->kmode[i]);
> >
> > retry:
> > + /* Discard any pending event that hasn't been consumed from a previous retry or subtest. */
> > + discard_any_stale_events();
> > +
> > memset(&o->vblank_state, 0, sizeof(o->vblank_state));
> > memset(&o->flip_state, 0, sizeof(o->flip_state));
> > o->flip_state.name = "flip";
> > --
> > 2.33.0.685.g46640cef36-goog
> >
^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2021-10-26 14:58 UTC | newest]
Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-28 17:16 [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry Mark Yacoub
2021-07-28 20:32 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2021-07-28 23:15 ` [igt-dev] ✗ GitLab.Pipeline: warning " Patchwork
2021-07-29 1:07 ` [igt-dev] ✓ Fi.CI.IGT: success " Patchwork
2021-08-26 12:55 ` [igt-dev] [PATCH] " Rodrigo Siqueira
2021-08-27 5:47 ` Petri Latvala
2021-08-27 7:52 ` [igt-dev] ✗ Fi.CI.BAT: failure for tests/kms_flip: Discard any stale event at each retry (rev2) Patchwork
2021-09-01 19:06 ` [igt-dev] [PATCH] tests/kms_flip: Discard any stale event at each retry Arkadiusz Hiler
2021-09-01 19:21 ` Mark Yacoub
2021-09-01 19:44 ` Arkadiusz Hiler
2021-09-24 18:31 ` Mark Yacoub
2021-09-24 18:39 ` [igt-dev] [PATCH v2] " Mark Yacoub
2021-10-26 8:39 ` Petri Latvala
2021-10-26 14:57 ` Mark Yacoub
2021-09-24 19:45 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_flip: Discard any stale event at each retry (rev3) Patchwork
2021-09-24 23:35 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.