diff options
| author | Nicholas Kazlauskas <nicholas.kazlauskas@amd.com> | 2019-01-31 09:48:46 -0500 |
|---|---|---|
| committer | Harry Wentland <harry.wentland@amd.com> | 2019-02-05 14:17:22 -0500 |
| commit | 3bb24a9f2ae732a1299d18e921d88945c6bb00ea (patch) | |
| tree | 0059d51d24277c2734b3dd26e1564471ec90f02d /lib/igt_kms.c | |
| parent | 1f89f1a04016cef20aa278ad05508cafdb9976f5 (diff) | |
lib/igt_kms: Don't reset VRR_ENABLED on every commit
If the VRR tests failed then there was a chance that VRR could be
left enabled when exiting the test so the VRR_ENABLED property was
reset to 0 whenever the pipe was reset.
However, in doing so the pipe's state was considered changed even
if VRR_ENABLED was already 0. This causes the pipe to be added to
commits where it previously wasn't.
Other tests with properties that can persist for failures (like color
management) don't bother resetting the properties - so this patch
changes VRR_ENABLED to work the same.
Cc: Harry Wentland <harry.wentland@amd.com>
Cc: Petri Latvala <petri.latvala@intel.com>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=109490
Signed-off-by: Nicholas Kazlauskas <nicholas.kazlauskas@amd.com>
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Diffstat (limited to 'lib/igt_kms.c')
| -rw-r--r-- | lib/igt_kms.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/lib/igt_kms.c b/lib/igt_kms.c index d45b01bd..85a911e1 100644 --- a/lib/igt_kms.c +++ b/lib/igt_kms.c @@ -1791,9 +1791,6 @@ static void igt_pipe_reset(igt_pipe_t *pipe) igt_pipe_obj_set_prop_value(pipe, IGT_CRTC_ACTIVE, 0); igt_pipe_obj_clear_prop_changed(pipe, IGT_CRTC_OUT_FENCE_PTR); - if (igt_pipe_obj_has_prop(pipe, IGT_CRTC_VRR_ENABLED)) - igt_pipe_obj_set_prop_value(pipe, IGT_CRTC_VRR_ENABLED, 0); - pipe->out_fence_fd = -1; } |