I've accidentally broken the new cursor size extensions, but it
wasn't quite correct before already: Variables which are set in
igt_fixtures _must_ be outside of the stackframe of the igt_fixture
block.

Signed-off-by: Daniel Vetter <[email protected]>
---
 tests/kms_cursor_crc.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c
index 7423f51409fb..f95448f47356 100644
--- a/tests/kms_cursor_crc.c
+++ b/tests/kms_cursor_crc.c
@@ -341,11 +341,12 @@ static void run_test_generic(data_t *data, int 
cursor_max_size)
 
 }
 
+uint64_t cursor_width, cursor_height;
+
 igt_main
 {
        data_t data = {};
-       int cursor_max_size, ret;
-       uint64_t cursor_width, cursor_height;
+       int ret;
 
        igt_skip_on_simulation();
 
@@ -371,7 +372,7 @@ igt_main
 
        }
 
-       run_test_generic(&data, cursor_max_size);
+       run_test_generic(&data, cursor_width);
 
        igt_fixture {
                free(data.pipe_crc);
-- 
1.8.5.2

_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to