Re: [PATCH i-g-t 1/8] lib: warn when attempting to run an unknown subtest

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Looks fine to me

> -----Original Message-----
> From: Thomas Wood [mailto:thomas.wood@xxxxxxxxx]
> Sent: Wednesday, July 23, 2014 11:58 AM
> To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> Cc: Gore, Tim; daniel.vetter@xxxxxxxx
> Subject: [PATCH i-g-t 1/8] lib: warn when attempting to run an unknown
> subtest
> 
> Signed-off-by: Thomas Wood <thomas.wood@xxxxxxxxx>
Reviewed-by: Tim Gore <tim.gore@xxxxxxxxx>
> ---
>  lib/igt_core.c | 18 +++++++++++++++---
>  1 file changed, 15 insertions(+), 3 deletions(-)
> 
> diff --git a/lib/igt_core.c b/lib/igt_core.c index b197932..5c20581 100644
> --- a/lib/igt_core.c
> +++ b/lib/igt_core.c
> @@ -204,6 +204,7 @@ static unsigned int exit_handler_count;
>  /* subtests helpers */
>  static bool list_subtests = false;
>  static char *run_single_subtest = NULL;
> +static bool run_single_subtest_found = false;
>  static const char *in_subtest = NULL;
>  static bool in_fixture = false;
>  static bool test_with_subtests = false; @@ -484,9 +485,14 @@ bool
> __igt_run_subtest(const char *subtest_name)
>  		return false;
>  	}
> 
> -	if (run_single_subtest &&
> -	    strcmp(subtest_name, run_single_subtest) != 0)
> -		return false;
> +	if (run_single_subtest) {
> +		if (strcmp(subtest_name, run_single_subtest) != 0)
> +			return false;
> +		else
> +			run_single_subtest_found = true;
> +	}
> +
> +
> 
>  	if (skip_subtests_henceforth) {
>  		printf("Subtest %s: %s\n", subtest_name, @@ -722,6
> +728,12 @@ void igt_exit(void)  {
>  	igt_exit_called = true;
> 
> +	if (run_single_subtest && !run_single_subtest_found) {
> +		igt_warn("Unknown subtest: %s\n", run_single_subtest);
> +		exit(-1);
> +	}
> +
> +
>  	if (igt_only_list_subtests())
>  		exit(IGT_EXIT_SUCCESS);
> 
> --
> 1.9.3

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux