[Intel-gfx] [PATCH 2/2] kms_atomic: plane_primary: Allow modeset if atomic check returns -EINVAL
Liu Ying
gnuiyl at gmail.com
Mon Sep 12 07:11:54 UTC 2016
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 <daniel.vetter at ffwll.ch>
Cc: Marius Vlad <marius.c.vlad at intel.com>
Cc: Micah Fedke <micah.fedke at collabora.com>
Cc: Daniel Stone <daniels at collabora.com>
Signed-off-by: Liu Ying <gnuiyl at gmail.com>
---
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
More information about the Intel-gfx
mailing list