[i-g-t 2/4] tests/kms_vrr: Add Negative tests to validate VRR
Bhanuprakash Modem
bhanuprakash.modem at intel.com
Thu Apr 21 09:28:46 UTC 2022
This patch will try to enable VRR on Non-VRR panel. VRR should
not be enabled on the Non-VRR panel.
V2:
* Readback the "vrr_enabled" prop, since Kernel shouln't reject
the modeset.
Cc: Manasi Navare <manasi.d.navare at intel.com>
Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem at intel.com>
---
tests/kms_vrr.c | 73 +++++++++++++++++++++++++++++++++++++++++++++++--
1 file changed, 71 insertions(+), 2 deletions(-)
diff --git a/tests/kms_vrr.c b/tests/kms_vrr.c
index 8976d4a6..7e06d39d 100644
--- a/tests/kms_vrr.c
+++ b/tests/kms_vrr.c
@@ -45,6 +45,7 @@ enum {
TEST_DPMS = 1 << 0,
TEST_SUSPEND = 1 << 1,
TEST_FLIPLINE = 1 << 2,
+ TEST_NEGATIVE = 1 << 3,
};
typedef struct range {
@@ -244,6 +245,34 @@ static void prepare_test(data_t *data, igt_output_t *output, enum pipe pipe)
igt_display_commit2(&data->display, COMMIT_ATOMIC);
}
+static void prepare_negative_test(data_t *data, igt_output_t *output, enum pipe pipe)
+{
+ drmModeModeInfo *mode;
+
+ /* Reset output */
+ igt_display_reset(&data->display);
+ igt_output_set_pipe(output, pipe);
+
+ /* Capture VRR range */
+ data->range = get_vrr_range(data, output);
+
+ /* Prepare resources */
+ mode = igt_output_get_mode(output);
+ igt_assert(igt_create_fb(data->drm_fd, mode->hdisplay, mode->vdisplay,
+ DRM_FORMAT_XRGB8888, DRM_FORMAT_MOD_LINEAR,
+ &data->fb0));
+
+ /* Take care of any required modesetting before the test begins. */
+ data->primary = igt_output_get_plane_type(output, DRM_PLANE_TYPE_PRIMARY);
+ igt_plane_set_fb(data->primary, &data->fb0);
+
+ /* Clear vrr_enabled state before enabling it, because
+ * it might be left enabled if the previous test fails.
+ */
+ igt_pipe_set_prop_value(&data->display, pipe, IGT_CRTC_VRR_ENABLED, 0);
+ igt_display_commit2(&data->display, COMMIT_ATOMIC);
+}
+
/* Performs an atomic non-blocking page-flip on a pipe. */
static void
do_flip(data_t *data, igt_fb_t *fb)
@@ -430,6 +459,33 @@ test_basic(data_t *data, enum pipe pipe, igt_output_t *output, uint32_t flags)
igt_remove_fb(data->drm_fd, &data->fb0);
}
+/* VRR on Non-VRR panel: VRR should not be enabled on the Non-VRR panel. */
+static void
+test_negative_basic(data_t *data, enum pipe pipe,
+ igt_output_t *output, uint32_t flags)
+{
+ igt_info("VRR Negative Test execution on %s, PIPE_%s.\n",
+ output->name, kmstest_pipe_name(pipe));
+
+ prepare_negative_test(data, output, pipe);
+ set_vrr_on_pipe(data, pipe, true);
+
+ /*
+ * Readback the "vrr_enabled" prop to make sure that Kernel hasn't
+ * enabled the VRR on selected pipe.
+ */
+ igt_wait_for_vblank(data->drm_fd, data->display.pipes[pipe].crtc_offset);
+ igt_assert_f(!igt_pipe_get_prop(&data->display, pipe, IGT_CRTC_VRR_ENABLED),
+ "VRR shouln't be enabled on Non-VRR panel.\n");
+
+ /* Clean-up */
+ igt_plane_set_fb(data->primary, NULL);
+ igt_output_set_pipe(output, PIPE_NONE);
+ set_vrr_on_pipe(data, pipe, false);
+
+ igt_remove_fb(data->drm_fd, &data->fb0);
+}
+
/* Runs tests on outputs that are VRR capable. */
static void
run_vrr_test(data_t *data, test_t test, uint32_t flags)
@@ -439,8 +495,17 @@ run_vrr_test(data_t *data, test_t test, uint32_t flags)
for_each_connected_output(&data->display, output) {
enum pipe pipe;
- if (!has_vrr(output))
- continue;
+ /* For Negative tests, panel should be non-vrr. */
+ if (flags & TEST_NEGATIVE) {
+ if (!igt_output_has_prop(output, IGT_CONNECTOR_VRR_CAPABLE))
+ continue;
+
+ if (has_vrr(output))
+ continue;
+ } else {
+ if (!has_vrr(output))
+ continue;
+ }
for_each_pipe(&data->display, pipe) {
if (igt_pipe_connector_valid(pipe, output)) {
@@ -486,6 +551,10 @@ igt_main
igt_subtest_with_dynamic("flipline")
run_vrr_test(&data, test_basic, TEST_FLIPLINE);
+ igt_describe("Make sure that VRR should not be enabled on the Non-VRR panel.");
+ igt_subtest_with_dynamic("negative-basic")
+ run_vrr_test(&data, test_negative_basic, TEST_NEGATIVE);
+
igt_fixture {
igt_display_fini(&data.display);
}
--
2.35.1
More information about the Intel-gfx-trybot
mailing list