| Summary: | [PNV]igt/kms_flip_tiling/flip-changes-tiling tests timeout | ||||||
|---|---|---|---|---|---|---|---|
| Product: | DRI | Reporter: | Guo Jinxian <jinxianx.guo> | ||||
| Component: | DRM/Intel | Assignee: | Jesse Barnes <jbarnes> | ||||
| Status: | CLOSED FIXED | QA Contact: | Intel GFX Bugs mailing list <intel-gfx-bugs> | ||||
| Severity: | normal | ||||||
| Priority: | medium | CC: | intel-gfx-bugs | ||||
| Version: | unspecified | ||||||
| Hardware: | Other | ||||||
| OS: | All | ||||||
| Whiteboard: | |||||||
| i915 platform: | i915 features: | ||||||
| Attachments: |
|
||||||
|
Description
Guo Jinxian
2014-08-07 05:52:24 UTC
Tries to use pipe 0 with LVDS, fails to setup the output and blocks forever waiting for a vblank interrupt. Retested without the extra vblank patches, and it works with just fixing kms_flip_tiling to use the right pipe. Test still timeout on latest -nightly(e91331cf71c509af3dfae75cd9bca3db13d7da95) on PNV [root@x-pnv2 tests]# ./kms_flip_tiling --run-subtest flip-changes-tiling IGT-Version: 1.7-g50166d2 (i686) (Linux: 3.17.0-rc2_drm-intel-nightly_e91331_20140828+ i686) Subtest flip-changes-tiling: TIMEOUT I think it should be as simple as:
diff --git a/tests/kms_flip_tiling.c b/tests/kms_flip_tiling.c
index 1f2caac..9c2f7fe 100644
--- a/tests/kms_flip_tiling.c
+++ b/tests/kms_flip_tiling.c
@@ -67,9 +67,9 @@ test_flip_changes_tiling(data_t *data, igt_output_t *output)
igt_crc_t reference_crc, crc;
int fb_id, pipe, ret, width;
- pipe = 0;
+ pipe = output->config.pipe;
pipe_crc = igt_pipe_crc_new(pipe, INTEL_PIPE_CRC_SOURCE_AUTO);
- igt_output_set_pipe(output, 0);
+ igt_output_set_pipe(output, pipe);
but now the pipe crc fails to open.
commit be6bb4ce4a413df5ec385523313a4e2a3894d4c8 Author: Chris Wilson <chris@chris-wilson.co.uk> Date: Sun Sep 7 15:52:10 2014 +0100 igt/kms_flip_tiling: Use the associated pipe for the output This stops us from using an illegal pipe, such as pipe 0 for LVDS on PNV. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=82280 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Verified I will report a new bug for this failure. [root@x-pnv2 tests]# ./kms_flip_tiling --run-subtest flip-changes-tiling IGT-Version: 1.8-gd9d3f4b (i686) (Linux: 3.17.0-rc4_drm-intel-nightly_99f444_20140910+ i686) Test assertion failure function igt_primary_plane_commit_legacy, file igt_kms.c:1419: Failed assertion: ret == 0 Last errno: 13, Permission denied Subtest flip-changes-tiling: FAIL (0.202s) Closing verified+fixed |
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.