[PATCH] [PATCH i-g-t] tests/intel/gem_exec_capture: Reduce test size

Kamil Konieczny kamil.konieczny at linux.intel.com
Wed Jun 5 15:16:42 UTC 2024


Hi Peter,
On 2024-03-30 at 15:59:34 +0100, Peter Senna Tschudin wrote:
> Before this patch, when using a chromebook-level CPU such as the N3150,
> with 8GB of RAM, the test `gem_exec_capture --run-subtest
> many-4K-incremental` could take 4 minutes to complete, and the
> `gem_exec_capture --run-subtest many-4K-zero` could take a minute.
> 
> This patch reduces the test size by half, reducing the time to run by
> half as well.
> 
> Signed-off-by: Peter Senna Tschudin <me at petersenna.com>

This is still running too long for almost all many-* subtests,
one of runs took almost 600s (around 9 minutes).

Could you lower it below 20 seconds?
Andi - do we need an option to make it longer if developer want it?

Regards,
Kamil

> ---
> 
> This patch is only "testable" after my other patch that fixes the test,
> in the first place. As I write this, the other patch is here:
> 
>   https://patchwork.freedesktop.org/series/131602/
> 
>  tests/intel/gem_exec_capture.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/intel/gem_exec_capture.c b/tests/intel/gem_exec_capture.c
> index 2afb84283..3f454b897 100644
> --- a/tests/intel/gem_exec_capture.c
> +++ b/tests/intel/gem_exec_capture.c
> @@ -700,7 +700,7 @@ static void many(int fd, int dir, uint64_t size, unsigned int flags)
>  	igt_debug("Available objects in GTT:%"PRIu64", RAM:%"PRIu64"\n",
>  		  gtt, ram);
>  
> -	count = min(gtt, ram) / 4;
> +	count = min(gtt, ram) / 8;
>  	igt_require(count > 1);
>  
>  	igt_require_memory(count, size, CHECK_RAM);
> -- 
> 2.44.0
> 


More information about the igt-dev mailing list