diff options
| author | Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> | 2021-04-14 11:25:56 +0300 |
|---|---|---|
| committer | Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com> | 2021-04-19 11:53:00 +0300 |
| commit | 71ee6cdc849c381186f681527e19fb217d32510d (patch) | |
| tree | 01b061a7cf5aceb07e70180493001e9575a74d70 /tests/kms_cursor_crc.c | |
| parent | 161b31f1bfabd8dff88de26774f7ce0fa5318ec4 (diff) | |
tests/kms_cursor_crc: Let's not create full screen framebuffers in a loop
Avoid creating full screen size fbs in a loop just to throw them
away.
Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>
Reviewed-by: Mika Kahola <mika.kahola@intel.com>
Diffstat (limited to 'tests/kms_cursor_crc.c')
| -rw-r--r-- | tests/kms_cursor_crc.c | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c index 2619e15e..1463c5b3 100644 --- a/tests/kms_cursor_crc.c +++ b/tests/kms_cursor_crc.c @@ -492,13 +492,6 @@ static void test_cursor_opaque(data_t *data) test_cursor_alpha(data, 1.0); } -static void run_test(data_t *data, void (*testfunc)(data_t *), int cursor_w, int cursor_h) -{ - prepare_crtc(data, data->output, cursor_w, cursor_h); - testfunc(data); - cleanup_crtc(data); -} - static void create_cursor_fb(data_t *data, int cur_w, int cur_h) { cairo_t *cr; @@ -567,6 +560,16 @@ static void require_cursor_size(data_t *data, int w, int h) igt_skip_on_f(ret, "Cursor size %dx%d not supported by driver\n", w, h); } +static void run_test(data_t *data, void (*testfunc)(data_t *), int cursor_w, int cursor_h) +{ + if (data->fb.fb_id != 0) + require_cursor_size(data, cursor_w, cursor_h); + + prepare_crtc(data, data->output, cursor_w, cursor_h); + testfunc(data); + cleanup_crtc(data); +} + static void test_cursor_size(data_t *data) { igt_display_t *display = &data->display; @@ -687,7 +690,6 @@ static void run_size_tests(data_t *data, enum pipe pipe, w, h); } create_cursor_fb(data, w, h); - require_cursor_size(data, w, h); } /* Using created cursor FBs to test cursor support */ |