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 <daniel.vetter@xxxxxxxx> --- 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 Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx