There was some confusion about whether we restore cursors correctly
after dpms and suspend/resume. Apparently we still do!

Signed-off-by: Daniel Vetter <daniel.vet...@intel.com>
---
 tests/kms_cursor_crc.c | 42 ++++++++++++++++++++++++++++++++++++++++--
 1 file changed, 40 insertions(+), 2 deletions(-)

diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c
index 0bfe15ee03d4..d33ee932d673 100644
--- a/tests/kms_cursor_crc.c
+++ b/tests/kms_cursor_crc.c
@@ -63,8 +63,12 @@ typedef struct {
        int cursor_max_w, cursor_max_h;
        igt_pipe_crc_t *pipe_crc;
        uint32_t devid;
+       unsigned flags;
 } data_t;
 
+#define TEST_DPMS (1<<0)
+#define TEST_SUSPEND (1<<1)
+
 static void draw_cursor(cairo_t *cr, int x, int y, int cw, int ch)
 {
        int wl, wr, ht, hb;
@@ -124,6 +128,27 @@ static void do_single_test(data_t *data, int x, int y)
        igt_display_commit(display);
        igt_wait_for_vblank(data->drm_fd, data->pipe);
        igt_pipe_crc_collect_crc(pipe_crc, &crc);
+
+       if (data->flags & (TEST_DPMS | TEST_SUSPEND)) {
+               igt_crc_t crc_after;
+
+               if (data->flags & TEST_DPMS) {
+                       igt_debug("dpms off/on cycle\n");
+                       kmstest_set_connector_dpms(data->drm_fd,
+                                                  
data->output->config.connector,
+                                                  DRM_MODE_DPMS_OFF);
+                       kmstest_set_connector_dpms(data->drm_fd,
+                                                  
data->output->config.connector,
+                                                  DRM_MODE_DPMS_ON);
+               }
+
+               if (data->flags & TEST_SUSPEND)
+                       igt_system_suspend_autoresume();
+
+               igt_pipe_crc_collect_crc(pipe_crc, &crc_after);
+               igt_assert_crc_equal(&crc, &crc_after);
+       }
+
        cursor_disable(data);
        igt_display_commit(display);
 
@@ -248,10 +273,12 @@ static void test_crc_sliding(data_t *data)
 
 static void test_crc_random(data_t *data)
 {
-       int i;
+       int i, max;
+
+       max = data->flags & (TEST_DPMS | TEST_SUSPEND) ? 2 : 50;
 
        /* Random cursor placement */
-       for (i = 0; i < 50; i++) {
+       for (i = 0; i < max; i++) {
                int x = rand() % (data->screenw + data->curw * 2) - data->curw;
                int y = rand() % (data->screenh + data->curh * 2) - data->curh;
                do_single_test(data, x, y);
@@ -474,6 +501,17 @@ static void run_test_generic(data_t *data)
                        run_test(data, test_crc_sliding, w, h);
                igt_subtest_f("cursor-%dx%d-random", w, h)
                        run_test(data, test_crc_random, w, h);
+               igt_subtest_f("cursor-%dx%d-dpms", w, h) {
+                       data->flags = TEST_DPMS;
+                       run_test(data, test_crc_random, w, h);
+                       data->flags = 0;
+               }
+
+               igt_subtest_f("cursor-%dx%d-suspend", w, h) {
+                       data->flags = TEST_SUSPEND;
+                       run_test(data, test_crc_random, w, h);
+                       data->flags = 0;
+               }
 
                igt_fixture
                        igt_remove_fb(data->drm_fd, &data->fb);
-- 
1.9.3

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to