Re: [PATCH i-g-t v2 3/3] tests/intel/kms_pipe_stress: use ARGB8888/LINEAR for cursor framebuffer

["Borah, Chaitanya Kumar" <chaitanya.kumar.borah@intel.com>]

On 6/2/2026 9:54 PM, Sowmiya S wrote:
> Cursor planes on Intel hardware accept only

nit: s/accept/accepts

> DRM_FORMAT_MOD_LINEAR, regardless of the overlay
> format and modifier used by the test variant. Using

and DRM_FORMAT_ARGB8888, unlike the formats and modifier
used for primary and overlay planes in the test.

Other than that LGTM.

Reviewed-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>


> data->modifier (e.g. 4-tiled) caused -EINVAL on the
> cursor commit.
> Bspec: 69833
> 
> v2: Add bspec reference in commit message (Swati)
> 
> Signed-off-by: Sowmiya S <sowmiya.s@intel.com>
> Reviewed-by: Swati Sharma <swati2.sharma@intel.com>
> ---
>   tests/intel/kms_pipe_stress.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/intel/kms_pipe_stress.c b/tests/intel/kms_pipe_stress.c
> index a84433b9d..5362fe9f3 100644
> --- a/tests/intel/kms_pipe_stress.c
> +++ b/tests/intel/kms_pipe_stress.c
> @@ -736,8 +736,8 @@ static void create_framebuffers(struct data *data)
>   		if (!data->cursor_fb[i].fb_id) {
>   			igt_create_color_fb(data->drm_fd,
>   					    cursor_width, cursor_height,
> -					    data->format,
> -					    data->modifier,
> +					    DRM_FORMAT_ARGB8888,
> +					    DRM_FORMAT_MOD_LINEAR,
>   					    1.0, 0.0, 0.0,
>   					    &data->cursor_fb[i]);
>   		}