[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