Re: [PATCH i-g-t v3 07/11] tests/xe_copy_basic: add testcase with large buffer size

[Francois Dugast <francois.dugast@intel.com>]

On Fri, May 23, 2025 at 10:01:22AM +0200, Zbigniew Kempczyński wrote:
> Maximum possible size for mem-copy/byte is 256KiB. Verify intel_blt
> is able to emit multiple mem-copy commands to copy buffer which
> is larger than 256KiB.
> 
> Cc: Francois Dugast <francois.dugast@intel.com>
> Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>

Reviewed-by: Francois Dugast <francois.dugast@intel.com>

> ---
>  tests/intel/xe_copy_basic.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/intel/xe_copy_basic.c b/tests/intel/xe_copy_basic.c
> index 569f250cc4..bed3e39426 100644
> --- a/tests/intel/xe_copy_basic.c
> +++ b/tests/intel/xe_copy_basic.c
> @@ -46,6 +46,7 @@ struct rect {
>   * @0x3fff: 0x3fff
>   * @0xfd: 0xfd
>   * @0xfffe: 0xfffe
> + * @0x8fffe: 0x8fffe
>   */
>  static void
>  mem_copy(int fd, uint32_t src_handle, uint32_t dst_handle, const intel_ctx_t *ctx,
> @@ -99,6 +100,7 @@ mem_copy(int fd, uint32_t src_handle, uint32_t dst_handle, const intel_ctx_t *ct
>   * @0x3fff: 0x3fff
>   * @0xfd: 0xfd
>   * @0xfffe: 0xfffe
> + * @0x8fffe: 0x8fffe
>   */
>  static void
>  mem_set(int fd, uint32_t dst_handle, const intel_ctx_t *ctx, uint32_t size,
> @@ -172,7 +174,8 @@ igt_main
>  	struct rect linear[] = { { 0, 0xfd, 1 },
>  				 { 0, 0x369, 1 },
>  				 { 0, 0x3fff, 1 },
> -				 { 0, 0xfffe, 1 } };
> +				 { 0, 0xfffe, 1 },
> +				 { 0, 0x8fffe, 1 } };
>  
>  	igt_fixture {
>  		fd = drm_open_driver(DRIVER_XE);
> -- 
> 2.43.0
>