Re: [igt-dev] [Intel-gfx] [PATCH i-g-t] i915/selftest: Allow filtering of individual subtests

[Petri Latvala <petri.latvala@intel.com>]

On Tue, Jan 22, 2019 at 06:35:21PM +0000, Chris Wilson wrote:
> Take an environment variable, SELFTESTS=foo,bar, and pass that along to
> the kernel (as i915.st_filter=foo,bar) to provide fine grained test
> selection. This can be either as an exact match to select only that
> test, or to exclude only test. For example,
> 
> SELFTESTS=igt_vma_create,igt_vma_pin1 i915_selftest --run mock_vma
> SELFTESTS=!igt_vma_create i915_selftest --run mock_vma
> 
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> ---
>  tests/i915/selftest.c | 18 ++++++++++++------
>  1 file changed, 12 insertions(+), 6 deletions(-)
> 
> diff --git a/tests/i915/selftest.c b/tests/i915/selftest.c
> index 80e515c61..b8d7f0af1 100644
> --- a/tests/i915/selftest.c
> +++ b/tests/i915/selftest.c
> @@ -28,10 +28,16 @@ IGT_TEST_DESCRIPTION("Basic unit tests for i915.ko");
>  
>  igt_main
>  {
> -	igt_kselftests("i915",
> -		       "mock_selftests=-1 disable_display=1",
> -		       NULL, "mock");
> -	igt_kselftests("i915",
> -		       "live_selftests=-1 disable_display=1",
> -		       "live_selftests", "live");
> +	const char *env = getenv("SELFTESTS");
> +	char opts[1024];
> +
> +	igt_assert(snprintf(opts, sizeof(opts),
> +			    "mock_selftests=-1 disable_display=1 st_filter=%s",
> +			    env) < sizeof(opts));


I don't particularly like passing NULL to %s, even though glibc makes
it print "(null)".


-- 
Petri Latvala
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev