Re: [igt-dev] [PATCH i-g-t] tests/i915/gem_render_copy: Skip the Y-tile test for MTL

[Karolina Stolarek <karolina.stolarek@intel.com>]

On 7.03.2023 17:53, Vikas Srivastava wrote:
> All the tests for y-tile or yf-tile , neither of these tiling formats
> are supported on MTL so skipping these tests.
> 
> Signed-off-by: Vikas Srivastava <vikas.srivastava@intel.com>
> Acked-by: Sinjan Kumar <sinjan.kumar@intel.com>
> ---
>   tests/i915/gem_render_copy.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/i915/gem_render_copy.c b/tests/i915/gem_render_copy.c
> index 70b96c8f4..2c150b2b0 100644
> --- a/tests/i915/gem_render_copy.c
> +++ b/tests/i915/gem_render_copy.c
> @@ -852,7 +852,7 @@ igt_main_args("dac", NULL, help_str, opt_handler, NULL)
>   			 force_vebox_dst_copy ? "vebox-" : "",
>   			 dst_mode);
>   		igt_subtest_with_dynamic(name) {
> -			igt_skip_on(IS_DG2(data.devid) &&
> +			igt_skip_on((IS_DG2(data.devid) || IS_METEORLAKE(data.devid)) &&

Ideally, we'd use intel_cmds_info/i915_blt helpers, but we lack 
definitions for render. This patch will do for now I think:

Reviewed-by: Karolina Stolarek <karolina.stolarek@intel.com>

>   				    ((t->src_tiling == I915_TILING_Y) ||
>   				     (t->src_tiling == I915_TILING_Yf) ||
>   				     (t->dst_tiling == I915_TILING_Y) ||