diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2014-06-13 15:19:09 +0200 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2014-06-13 18:09:05 +0200 |
commit | 652e5441504ddded9d6b138cd6df3447ed9b2e74 (patch) | |
tree | 7acfd7afd9f12a794d8ade7ef0ac166d40d6a38a /tests | |
parent | ac0e606677823d27836708300a2c5ee05d24b458 (diff) |
tests/pm_psr_sink_crc: Fix longjmp fun
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@ffwll.ch>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/kms_psr_sink_crc.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tests/kms_psr_sink_crc.c b/tests/kms_psr_sink_crc.c index 999a0bfd..c2c12d11 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 { |