[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