[igt-dev] [PATCH i-g-t] tests/i915/gem_ctx_create: lower active subtests timeout

[Kamil Konieczny]

Dnia 2022-03-14 at 18:17:47 +0100, Kamil Konieczny napisał(a):
> Active subtests measure time for one cycle of context creation
> and bufexec execution per engine and this is done until timeout
> of 20 seconds expires. On old gpu gens there are low number of
> engines so it runs in reasonable time, but on new ones it can
> take as long as over 800s. Lower timeout to only 2s to keep it
> low on CI. Below is example of dmesg output on new machine with
> long 20 seconds timeout:
> 
> [ 3683.594715] [IGT] gem_ctx_create: executing
> [ 3895.606195] [IGT] gem_ctx_create: starting subtest basic-active

[cut]

> [ 4551.486436] [IGT] gem_ctx_create: exiting, ret=0
> 
> so it took around 868s.
> 
> Cc: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
> Signed-off-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> ---
>  tests/i915/gem_ctx_create.c | 11 ++++++-----
>  1 file changed, 6 insertions(+), 5 deletions(-)
> 
> diff --git a/tests/i915/gem_ctx_create.c b/tests/i915/gem_ctx_create.c
> index 44846652..8052e17c 100644
> --- a/tests/i915/gem_ctx_create.c
> +++ b/tests/i915/gem_ctx_create.c
> @@ -37,6 +37,7 @@
>  #include "sw_sync.h"
>  
>  #define ENGINE_FLAGS  (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
> +#define ACTIVE_TIMEOUT 20

I forgot to change this to 2, but meanwhile I think about other
solution to this problem, so please drop this for now.

--
Kamil

>  
>  static unsigned all_engines[I915_EXEC_RING_MASK + 1];
>  static unsigned all_nengine;
> @@ -613,26 +614,26 @@ igt_main
>  
>  	/* NULL value means all engines */
>  	igt_subtest("active-all")
> -		active(fd, &cfg, NULL, 20, 1);
> +		active(fd, &cfg, NULL, ACTIVE_TIMEOUT, 1);

[cut]
> -- 
> 2.32.0
>