[PATCH i-g-t] tests/xe_intel_bb: Reduce render subtest execution time

Kamil Konieczny kamil.konieczny at linux.intel.com
Wed Jul 3 10:12:22 UTC 2024


Hi Zbigniew,
On 2024-07-03 at 08:33:14 +0200, Zbigniew Kempczyński wrote:
> Test xe_intel_bb verifies intel-bb so it doesn't make sense to perform
> large render operations as those are covered in xe_render_copy test.
> Use 256px surface for render instead 512 and 1024 px.
> 
> Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
> Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>

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

I will merge this without waiting for CI as this works on my
test machine and it just reduced number of performed tests.

Regards,
Kamil

> ---
>  tests/intel/xe_intel_bb.c | 11 ++++++-----
>  1 file changed, 6 insertions(+), 5 deletions(-)
> 
> diff --git a/tests/intel/xe_intel_bb.c b/tests/intel/xe_intel_bb.c
> index 5dd794b186..647fd64e7e 100644
> --- a/tests/intel/xe_intel_bb.c
> +++ b/tests/intel/xe_intel_bb.c
> @@ -40,6 +40,7 @@
>  #define HEIGHT	64
>  #define STRIDE	(WIDTH * 4)
>  #define SIZE	(HEIGHT * STRIDE)
> +#define RENDER_WIDTH 256
>  
>  #define COLOR_00	0x00
>  #define COLOR_33	0x33
> @@ -1054,11 +1055,11 @@ igt_main_args("dpib", NULL, help_str, opt_handler, NULL)
>  			if (!render_supports_tiling(xe, tiling, false))
>  				continue;
>  
> -			for (width = 512; width <= 1024; width += 512)
> -				igt_dynamic_f("render-%s-%u",
> -					      blt_tiling_name(tiling), width)
> -					render(bops, blt_tile_to_i915_tile(tiling),
> -					       width, width);
> +			width = RENDER_WIDTH;
> +			igt_dynamic_f("render-%s-%u",
> +				      blt_tiling_name(tiling), width)
> +				render(bops, blt_tile_to_i915_tile(tiling),
> +				       width, width);
>  		}
>  	}
>  
> -- 
> 2.34.1
> 


More information about the igt-dev mailing list