[Intel-gfx] [PATCH] tests/pm_psr_sink_crc: Fix longjmp fun
Daniel Vetter
daniel.vetter at ffwll.ch
Fri Jun 13 15:21:04 CEST 2014
igt_fixture and igt_subtests use longjmp/setjmp internally, which
means local variables at the same stack frame are at risk. Best
practice is to move them out right in front of the igt_main block.
It would be awesome if someone could come up with a cocci patch to
auto-fix this, but unfortunately my attempts failed.
Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch>
---
tests/kms_psr_sink_crc.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/tests/kms_psr_sink_crc.c b/tests/kms_psr_sink_crc.c
index a36ae6c1800f..e2de5a754eb4 100644
--- a/tests/kms_psr_sink_crc.c
+++ b/tests/kms_psr_sink_crc.c
@@ -587,11 +587,11 @@ static void run_test(data_t *data)
}
}
+data_t data = {};
+enum tests test;
+
igt_main
{
- data_t data = {};
- enum tests test;
-
igt_skip_on_simulation();
igt_fixture {
--
2.0.0
More information about the Intel-gfx
mailing list