[igt-dev] [PATCH i-g-t 1/4] i915/gem_sync: Reduce runtime

[Mika Kuoppala]

Chris Wilson <chris at chris-wilson.co.uk> writes:

> Reduce the upper timeout for stress tests from 150s to a mere 20s, and
> quick tests to 2s.
>
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>

This and the rest of the series,

Reviewed-by: Mika Kuoppala <mika.kuoppala at linux.intel.com>

> ---
>  tests/i915/gem_sync.c | 60 +++++++++++++++++++++----------------------
>  1 file changed, 30 insertions(+), 30 deletions(-)
>
> diff --git a/tests/i915/gem_sync.c b/tests/i915/gem_sync.c
> index 0010ac103..2ef55ecc0 100644
> --- a/tests/i915/gem_sync.c
> +++ b/tests/i915/gem_sync.c
> @@ -1224,67 +1224,67 @@ igt_main
>  
>  	for (e = intel_execution_engines; e->name; e++) {
>  		igt_subtest_f("%s", e->name)
> -			sync_ring(fd, eb_ring(e), 1, 150);
> +			sync_ring(fd, eb_ring(e), 1, 20);
>  		igt_subtest_f("idle-%s", e->name)
> -			idle_ring(fd, eb_ring(e), 150);
> +			idle_ring(fd, eb_ring(e), 20);
>  		igt_subtest_f("active-%s", e->name)
> -			active_ring(fd, eb_ring(e), 150);
> +			active_ring(fd, eb_ring(e), 20);
>  		igt_subtest_f("wakeup-%s", e->name)
> -			wakeup_ring(fd, eb_ring(e), 150, 1);
> +			wakeup_ring(fd, eb_ring(e), 20, 1);
>  		igt_subtest_f("active-wakeup-%s", e->name)
> -			active_wakeup_ring(fd, eb_ring(e), 150, 1);
> +			active_wakeup_ring(fd, eb_ring(e), 20, 1);
>  		igt_subtest_f("double-wakeup-%s", e->name)
> -			wakeup_ring(fd, eb_ring(e), 150, 2);
> +			wakeup_ring(fd, eb_ring(e), 20, 2);
>  		igt_subtest_f("store-%s", e->name)
> -			store_ring(fd, eb_ring(e), 1, 150);
> +			store_ring(fd, eb_ring(e), 1, 20);
>  		igt_subtest_f("switch-%s", e->name)
> -			switch_ring(fd, eb_ring(e), 1, 150);
> +			switch_ring(fd, eb_ring(e), 1, 20);
>  		igt_subtest_f("forked-switch-%s", e->name)
> -			switch_ring(fd, eb_ring(e), ncpus, 150);
> +			switch_ring(fd, eb_ring(e), ncpus, 20);
>  		igt_subtest_f("many-%s", e->name)
> -			store_many(fd, eb_ring(e), 150);
> +			store_many(fd, eb_ring(e), 20);
>  		igt_subtest_f("forked-%s", e->name)
> -			sync_ring(fd, eb_ring(e), ncpus, 150);
> +			sync_ring(fd, eb_ring(e), ncpus, 20);
>  		igt_subtest_f("forked-store-%s", e->name)
> -			store_ring(fd, eb_ring(e), ncpus, 150);
> +			store_ring(fd, eb_ring(e), ncpus, 20);
>  	}
>  
>  	igt_subtest("basic-each")
> -		sync_ring(fd, ALL_ENGINES, 1, 5);
> +		sync_ring(fd, ALL_ENGINES, 1, 2);
>  	igt_subtest("basic-store-each")
> -		store_ring(fd, ALL_ENGINES, 1, 5);
> +		store_ring(fd, ALL_ENGINES, 1, 2);
>  	igt_subtest("basic-many-each")
> -		store_many(fd, ALL_ENGINES, 5);
> +		store_many(fd, ALL_ENGINES, 2);
>  	igt_subtest("switch-each")
> -		switch_ring(fd, ALL_ENGINES, 1, 150);
> +		switch_ring(fd, ALL_ENGINES, 1, 20);
>  	igt_subtest("forked-switch-each")
> -		switch_ring(fd, ALL_ENGINES, ncpus, 150);
> +		switch_ring(fd, ALL_ENGINES, ncpus, 20);
>  	igt_subtest("forked-each")
> -		sync_ring(fd, ALL_ENGINES, ncpus, 150);
> +		sync_ring(fd, ALL_ENGINES, ncpus, 20);
>  	igt_subtest("forked-store-each")
> -		store_ring(fd, ALL_ENGINES, ncpus, 150);
> +		store_ring(fd, ALL_ENGINES, ncpus, 20);
>  	igt_subtest("active-each")
> -		active_ring(fd, ALL_ENGINES, 150);
> +		active_ring(fd, ALL_ENGINES, 20);
>  	igt_subtest("wakeup-each")
> -		wakeup_ring(fd, ALL_ENGINES, 150, 1);
> +		wakeup_ring(fd, ALL_ENGINES, 20, 1);
>  	igt_subtest("active-wakeup-each")
> -		active_wakeup_ring(fd, ALL_ENGINES, 150, 1);
> +		active_wakeup_ring(fd, ALL_ENGINES, 20, 1);
>  	igt_subtest("double-wakeup-each")
> -		wakeup_ring(fd, ALL_ENGINES, 150, 2);
> +		wakeup_ring(fd, ALL_ENGINES, 20, 2);
>  
>  	igt_subtest("basic-all")
> -		sync_all(fd, 1, 5);
> +		sync_all(fd, 1, 2);
>  	igt_subtest("basic-store-all")
> -		store_all(fd, 1, 5);
> +		store_all(fd, 1, 2);
>  
>  	igt_subtest("all")
> -		sync_all(fd, 1, 150);
> +		sync_all(fd, 1, 20);
>  	igt_subtest("store-all")
> -		store_all(fd, 1, 150);
> +		store_all(fd, 1, 20);
>  	igt_subtest("forked-all")
> -		sync_all(fd, ncpus, 150);
> +		sync_all(fd, ncpus, 20);
>  	igt_subtest("forked-store-all")
> -		store_all(fd, ncpus, 150);
> +		store_all(fd, ncpus, 20);
>  
>  	igt_subtest_group {
>  		igt_fixture {
> @@ -1298,7 +1298,7 @@ igt_main
>  
>  		for (e = intel_execution_engines; e->name; e++) {
>  			igt_subtest_f("preempt-%s", e->name)
> -				preempt(fd, eb_ring(e), ncpus, 150);
> +				preempt(fd, eb_ring(e), ncpus, 20);
>  		}
>  	}
>  
> -- 
> 2.25.0
>
> _______________________________________________
> igt-dev mailing list
> igt-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/igt-dev