[PATCH] tests/intel/xe_exec_threads: decrease number of batches for simulation

Kamil Konieczny kamil.konieczny at linux.intel.com
Fri Jul 26 11:45:14 UTC 2024


Hi Andrzej,
On 2024-07-26 at 10:24:19 +0200, Andrzej Hajda wrote:
> Tests like threads-mixed-shared-vm-rebind takes significantly longer time
> on simulation due to high number of batches - 6-8 minutes vs few seconds
> on hw. As a result CI tests frequently fails due to timeout.
> 
> Signed-off-by: Andrzej Hajda <andrzej.hajda at intel.com>

LGTM,
Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>

> ---
>  tests/intel/xe_exec_threads.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/intel/xe_exec_threads.c b/tests/intel/xe_exec_threads.c
> index e7a0a7cd0692..9a37730a9615 100644
> --- a/tests/intel/xe_exec_threads.c
> +++ b/tests/intel/xe_exec_threads.c
> @@ -944,6 +944,7 @@ static void *thread(void *data)
>  
>  static void threads(int fd, int flags)
>  {
> +	int n_execs = igt_run_in_simulation() ? 256 : 1024;
>  	struct thread_data *threads_data;
>  	struct drm_xe_engine_class_instance *hwe;
>  	uint64_t addr = 0x1a0000;
> @@ -1002,8 +1003,7 @@ static void threads(int fd, int flags)
>  		threads_data[i].eci = hwe;
>  #define N_EXEC_QUEUE	16
>  		threads_data[i].n_exec_queue = N_EXEC_QUEUE;
> -#define N_EXEC		1024
> -		threads_data[i].n_exec = N_EXEC;
> +		threads_data[i].n_exec = n_execs;
>  		threads_data[i].flags = flags;
>  		if (flags & MIXED_MODE) {
>  			threads_data[i].flags &= ~MIXED_MODE;
> @@ -1050,7 +1050,7 @@ static void threads(int fd, int flags)
>  						vm_legacy_mode;
>  					threads_data[i].class = class;
>  					threads_data[i].n_exec_queue = N_EXEC_QUEUE;
> -					threads_data[i].n_exec = N_EXEC;
> +					threads_data[i].n_exec = n_execs;
>  					threads_data[i].flags = flags;
>  					threads_data[i].flags &= ~BALANCER;
>  					threads_data[i].flags |= *data_flags;
> 
> ---
> base-commit: 0471a1097444389a901d78b260c24eaf4fc2d7df
> change-id: 20240726-xe_exec_threads_limit_simul-3dc20e0deac3
> 
> Best regards,
> -- 
> Andrzej Hajda <andrzej.hajda at intel.com>
> 


More information about the igt-dev mailing list