[PATCH] tests/intel/xe_exec_compute_mode: Fix assertion of batch buffer size

Kamil Konieczny kamil.konieczny at linux.intel.com
Thu Jun 12 08:48:35 UTC 2025


Hi Shuicheng,
On 2025-06-12 at 07:58:43 +0000, Shuicheng Lin wrote:
> The batch buffer pointer is data[EXEC_DATA] not data. Correct it in the assertion.
> 
> Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> Signed-off-by: Shuicheng Lin <shuicheng.lin at intel.com>

Nice catch,

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

> ---
>  tests/intel/xe_exec_compute_mode.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/intel/xe_exec_compute_mode.c b/tests/intel/xe_exec_compute_mode.c
> index 8fba9daef..f321e1d76 100644
> --- a/tests/intel/xe_exec_compute_mode.c
> +++ b/tests/intel/xe_exec_compute_mode.c
> @@ -393,7 +393,7 @@ static void non_block(int fd, int expect)
>  	data[EXEC_DATA].batch[b++] = sdi_addr >> 32;
>  	data[EXEC_DATA].batch[b++] = value;
>  	data[EXEC_DATA].batch[b++] = MI_BATCH_BUFFER_END;
> -	igt_assert(b <= ARRAY_SIZE(data->batch));
> +	igt_assert(b <= ARRAY_SIZE(data[EXEC_DATA].batch));
>  
>  	sync[0].addr = addr + (char *)&data[EXEC_DATA].exec_sync - (char *)data;
>  	exec.num_syncs = 0;
> -- 
> 2.25.1
> 


More information about the igt-dev mailing list