On the fi-hsw-4770 machine, we have been getting the following error message quite often in pre-merge: (kms_cursor_legacy:4154) CRITICAL: Test assertion failure function basic_flip_cursor, file kms_cursor_legacy.c:612: (kms_cursor_legacy:4154) CRITICAL: Failed assertion: get_vblank(display->drm_fd, pipe, 0) == vblank_start (kms_cursor_legacy:4154) CRITICAL: error: 16071 != 16070 Full logs: - https://intel-gfx-ci.01.org/tree/drm-tip/Trybot_1085/fi-hsw-4770/igt@kms_cursor_legacy@basic-flip-before-cursor-varying-size.html - https://intel-gfx-ci.01.org/tree/drm-tip/Trybot_1085/fi-hsw-4770/igt@kms_cursor_legacy@basic-flip-after-cursor-varying-size.html
Created attachment 133838 [details] dmesg snippet pre-merge I removed timing so it is easier to diff
Created attachment 133839 [details] dmesg snippet CI I removed timing info to make it easier to diff with the pre-merge log
If I diff the logs it appear as if there are a lot more prints on the pre-merge.
History: https://intel-gfx-ci.01.org/cibuglog/index.html%3Faction_failures_history=231.html
Considering this was always failing before, maybe this fixed it? commit 1043c09ccbcba8e5c2ec5f2a358a442346348bd8 Author: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Date: Tue Sep 19 13:31:13 2017 +0200 tests/kms_cursor_legacy: Do not start collecting CRC after making FB busy Collecting CRC may force a modeset, which is a bad idea after we just forced a hang. The hang is intended to make sure the page flip doesn't complete before the cursor, making sure that works.
(In reply to Maarten Lankhorst from comment #5) > Considering this was always failing before, maybe this fixed it? > > commit 1043c09ccbcba8e5c2ec5f2a358a442346348bd8 > Author: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> > Date: Tue Sep 19 13:31:13 2017 +0200 > > tests/kms_cursor_legacy: Do not start collecting CRC after making FB busy > > Collecting CRC may force a modeset, which is a bad idea after we just > forced a hang. The hang is intended to make sure the page flip doesn't > complete before the cursor, making sure that works. Looks like it, indeed! I'll close the bug now and archive the issue.
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.