diff options
author | Tianci.Yin <tianci.yin@amd.com> | 2021-08-25 15:47:07 +0800 |
---|---|---|
committer | Rodrigo Siqueira <Rodrigo.Siqueira@amd.com> | 2021-08-29 12:24:29 -0400 |
commit | 925655c30f4dcc0fa750ba7f827776509f8b9700 (patch) | |
tree | d474315b8572cc4b48c47e0ece78ed6ee64a8d1b /tests/kms_atomic_interruptible.c | |
parent | 845b9768a45e81e245f6ba3b8ad00f9b9abed6db (diff) |
tests/kms_atomic_interruptible: fix a warning message on AMD GPU
[why]
For subtest legacy-setmode and atomic-setmode, fb buffer is
unbinded from plane by block_plane(), then igt_display_commit2()
will trigger a dmesg warning message in drm_atomic_helper_commit(),
since for block commit, drm will wait fences that represents
finishing fb writing, when it find this fence has no accompanying
fb, it gives out a warning.
[how]
Just remove igt_plane_set_fb() from block_plane() to retain
the fb binding with plane.
Verified on AMD Radeon Pro W5500 and Intel UHD Graphics 630
(inside i7-9700K), both pass.
Signed-off-by: Tianci.Yin <tianci.yin@amd.com>
Reviewed-by: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>
Diffstat (limited to 'tests/kms_atomic_interruptible.c')
-rw-r--r-- | tests/kms_atomic_interruptible.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/tests/kms_atomic_interruptible.c b/tests/kms_atomic_interruptible.c index 0a127921..03980d6e 100644 --- a/tests/kms_atomic_interruptible.c +++ b/tests/kms_atomic_interruptible.c @@ -49,7 +49,6 @@ static int block_plane(igt_display_t *display, igt_output_t *output, enum plane_ if (test_type == test_legacy_modeset || test_type == test_atomic_modeset) { igt_output_set_pipe(output, PIPE_NONE); - igt_plane_set_fb(plane, NULL); } igt_plane_set_fence_fd(plane, sw_sync_timeline_create_fence(timeline, 1)); |