Re: [PATCH i-g-t 2/3] lib: print the name of the currently running subtest

[Daniel Vetter <daniel@ffwll.ch>]

On Tue, Mar 11, 2014 at 04:18:49PM +0000, Thomas Wood wrote:
> Signed-off-by: Thomas Wood <thomas.wood@intel.com>
> ---
>  lib/drmtest.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/drmtest.c b/lib/drmtest.c
> index 9f6e701..b8d3542 100644
> --- a/lib/drmtest.c
> +++ b/lib/drmtest.c
> @@ -1007,6 +1007,9 @@ bool __igt_run_subtest(const char *subtest_name)
>  		return false;
>  	}
>  
> +	printf("Subtest %s: ", subtest_name);
> +	fflush(stdout);

A lot of subtest spam stdout quite a bit, so we'd need to instaed print
start/stop markers - splitt up the lines like this will result in really
ugly output. See e.g. kms_flip.
-Daniel

> +
>  	return (in_subtest = subtest_name);
>  }
>  
> @@ -1028,7 +1031,7 @@ static int igt_exitcode;
>  static void exit_subtest(const char *) __attribute__((noreturn));
>  static void exit_subtest(const char *result)
>  {
> -	printf("Subtest %s: %s\n", in_subtest, result);
> +	printf("%s\n", result);
>  	in_subtest = NULL;
>  	longjmp(igt_subtest_jmpbuf, 1);
>  }
> -- 
> 1.8.5.3
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch