[Intel-gfx] [PATCH i-g-t] testdisplay: ignore invalid subtest options

[Thomas Wood]

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.w...@intel.com>
---
 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@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx