[Intel-gfx] [PATCH] tests/kms_cursor_crc: Fix up breakage

Daniel Vetter daniel.vetter at ffwll.ch
Sat Mar 22 15:37:09 CET 2014


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 at ffwll.ch>
---
 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




More information about the Intel-gfx mailing list