testdisplay does not have any subtests and should therefore exit with the appropriate exit code if the --list-subtests or --run-subtest options are used. Signed-off-by: Thomas Wood <thomas.wood@xxxxxxxxx> --- tests/testdisplay.c | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/tests/testdisplay.c b/tests/testdisplay.c index a187c16..f26d942 100644 --- a/tests/testdisplay.c +++ b/tests/testdisplay.c @@ -71,6 +71,8 @@ #include <stdlib.h> #include <signal.h> +#define SUBTEST_OPTS 1 + static int tio_fd; struct termios saved_tio; @@ -748,12 +750,17 @@ int main(int argc, char **argv) GMainLoop *mainloop; float force_clock; bool opt_dump_info = false; + struct option long_opts[] = { + {"list-subtests", 0, 0, SUBTEST_OPTS}, + {"run-subtest", 1, 0, SUBTEST_OPTS}, + { 0, 0, 0, 0 } + }; igt_skip_on_simulation(); enter_exec_path( argv ); - while ((c = getopt(argc, argv, optstr)) != -1) { + while ((c = getopt_long(argc, argv, optstr, long_opts, NULL)) != -1) { switch (c) { case '3': test_stereo_modes = 1; @@ -804,6 +811,10 @@ int main(int argc, char **argv) case 'o': sscanf(optarg, "%d,%d", &specified_disp_id, &specified_mode_num); break; + case SUBTEST_OPTS: + /* invalid subtest options */ + exit(IGT_EXIT_INVALID); + break; default: /* fall through */ case 'h': -- 1.9.3 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx