Allowing modeset may prevent the test case from failing in case the atomic check phase finds the userspace doesn't allow modeset for the commit and returns -EINVAL. A real case is to run the test case on imx-drm which requires a full modeset when we change an active plane's configuration, e.g., pixel format and stride.
Cc: Daniel Vetter <[email protected]> Cc: Marius Vlad <[email protected]> Cc: Micah Fedke <[email protected]> Cc: Daniel Stone <[email protected]> Signed-off-by: Liu Ying <[email protected]> --- tests/kms_atomic.c | 17 ++++++++++++++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/tests/kms_atomic.c b/tests/kms_atomic.c index bc6e575..1441fdf 100644 --- a/tests/kms_atomic.c +++ b/tests/kms_atomic.c @@ -910,6 +910,8 @@ static void plane_primary(struct kms_atomic_crtc_state *crtc, uint32_t format = plane_get_igt_format(&plane); drmModeAtomicReq *req = drmModeAtomicAlloc(); struct igt_fb fb; + uint32_t flags = 0; + int ret; igt_require(format != 0); @@ -926,12 +928,21 @@ static void plane_primary(struct kms_atomic_crtc_state *crtc, plane.crtc_w, plane.crtc_h, format, I915_TILING_NONE, &fb); + drmModeAtomicSetCursor(req, 0); + crtc_populate_req(crtc, req); + plane_populate_req(&plane, req); + ret = drmModeAtomicCommit(crtc->state->desc->fd, req, + DRM_MODE_ATOMIC_TEST_ONLY, NULL); + /* Try harder in case the failure is caused by disallowing modeset. */ + if (ret == -EINVAL) + flags |= DRM_MODE_ATOMIC_ALLOW_MODESET; + /* Flip the primary plane using the atomic API, and double-check * state is what we think it should be. */ - crtc_commit_atomic(crtc, &plane, req, ATOMIC_RELAX_NONE, 0); + crtc_commit_atomic(crtc, &plane, req, ATOMIC_RELAX_NONE, flags); /* Restore the primary plane and check the state matches the old. */ - crtc_commit_atomic(crtc, plane_old, req, ATOMIC_RELAX_NONE, 0); + crtc_commit_atomic(crtc, plane_old, req, ATOMIC_RELAX_NONE, flags); /* Re-enable the plane through the legacy CRTC/primary-plane API, and * verify through atomic. */ @@ -942,7 +953,7 @@ static void plane_primary(struct kms_atomic_crtc_state *crtc, crtc_commit_legacy(crtc, plane_old, CRTC_RELAX_MODE); /* Finally, restore to the original state. */ - crtc_commit_atomic(crtc, plane_old, req, ATOMIC_RELAX_NONE, 0); + crtc_commit_atomic(crtc, plane_old, req, ATOMIC_RELAX_NONE, flags); drmModeAtomicFree(req); } -- 2.7.4 _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
