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

[Andi Shyti <andi.shyti@linux.intel.com>]

Hi Peter,

On Sat, Mar 30, 2024 at 03:59:34PM +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@petersenna.com>

Acked-by: Andi Shyti <andi.shyti@linux.intel.com>

Andi

> ---
> 
> 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