@@ -138,6 +138,16 @@ bool __igt_run_subtest(const char *subtest_name);
(setjmp(igt_subtest_jmpbuf) == 0); \
igt_success())
const char *igt_subtest_name(void);
+#define igt_main \
+ static void igt_tokencat(__real_main, __LINE__)(void); \
+ int main(int argc, char **argv) { \
+ igt_subtest_init(argc, argv); \
+ igt_tokencat(__real_main, __LINE__)(); \
+ igt_exit(); \
+ } \
+ static void igt_tokencat(__real_main, __LINE__)(void) \
+
+
/**
* igt_skip - subtest aware test skipping
*
@@ -321,11 +321,10 @@ static void create_cursor_fb(data_t *data,
igt_assert(cairo_status(cr) == 0);
}
-int main(int argc, char **argv)
+igt_main
{
data_t data = {};
- igt_subtest_init(argc, argv);
igt_skip_on_simulation();
igt_fixture {
@@ -375,6 +374,4 @@ int main(int argc, char **argv)
display_fini(&data);
fclose(data.ctl);
}
-
- igt_exit();
}
In the past new testcases with subtest often forgot to add the call to igt_exit at the end of their main() function. That is now caught with a bit more obnoxious asserts, but it's still a nuissance. This little igt_main macro takes care of that (and also of calling the subtest machinery initialization code correctly). If no one objects I'll roll this out for all the simple cases (i.e. those tests that don't have additional argv parsing on top of the subtest machinery). Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch> --- lib/drmtest.h | 10 ++++++++++ tests/kms_cursor_crc.c | 5 +---- 2 files changed, 11 insertions(+), 4 deletions(-)