[PATCH i-g-t v1 2/2] tests/kms_async_flips: Use tiling modifiers

[Karthik B S]

Hi Santhosh,

On 4/15/2025 9:21 PM, Santhosh Reddy Guddati wrote:
> As async with linear modifiers is not a practical and common use case,
> override the default modifier with tiled for all the async flip subtests
>
> Signed-off-by: Santhosh Reddy Guddati <santhosh.reddy.guddati at intel.com>
> ---
>   tests/kms_async_flips.c | 4 ++++
>   1 file changed, 4 insertions(+)
>
> diff --git a/tests/kms_async_flips.c b/tests/kms_async_flips.c
> index df3e136b7..5d84df49a 100644
> --- a/tests/kms_async_flips.c
> +++ b/tests/kms_async_flips.c
> @@ -780,6 +780,10 @@ static void run_test(data_t *data, void (*test)(data_t *))
>   		test_init(data);
>   		data->allow_fail = false;
>   		data->modifier = default_modifier(data);
> +
> +		if (is_intel_device(data->drm_fd) && data->modifier == DRM_FORMAT_MOD_LINEAR)
> +			data->modifier = I915_FORMAT_MOD_4_TILED;
> +

Can we handle this in the default_modifier function?

Regards,
Karthik.B.S
>   		igt_dynamic_f("pipe-%s-%s", kmstest_pipe_name(data->pipe), data->output->name) {
>   			/*
>   			 * FIXME: joiner+async flip is busted currently in KMD.